fix(eslint-plugin): [consistent-type-definitions] correct fixer with declare keyword

[armano2]

PR Checklist

• Addresses an existing issue: fixes [consistent-type-definitions] invalid fixer for declare type #4333
• That issue was marked as accepting prs
• Steps in CONTRIBUTING.md were taken

Overview

Fixer incorrectly changes declare keyword to interface

[nx-cloud]

☁️ Nx Cloud Report

CI ran the following commands for commit 8b4288a. Click to see the status, the terminal output, and the build insights.

📂 See all runs for this branch

---

✅ Successfully ran 44 targets

• Node 12.x - nx test @typescript-eslint/eslint-plugin-internal
• Node 12.x - nx test @typescript-eslint/eslint-plugin-tslint
• Node 12.x - nx test @typescript-eslint/eslint-plugin
• Node 12.x - nx test @typescript-eslint/eslint-plugin-tslint
• Node 12.x - nx test @typescript-eslint/eslint-plugin
• Node 16.x - nx test @typescript-eslint/eslint-plugin-tslint
• Node 16.x - nx test @typescript-eslint/eslint-plugin
• Node 14.x - nx test @typescript-eslint/eslint-plugin-tslint
• Node 14.x - nx test @typescript-eslint/eslint-plugin
• Node 12.x - nx run-many --target=typecheck --all --parallel
• Node 12.x - nx test @typescript-eslint/parser
• Node 12.x - nx test @typescript-eslint/experimental-utils
• Node 12.x - nx test @typescript-eslint/scope-manager
• Node 12.x - nx test @typescript-eslint/visitor-keys
• Node 12.x - nx test @typescript-eslint/typescript-estree
• Node 12.x - nx run-many --target=build --all --parallel --exclude website --exclude website
• Node 12.x - nx test @typescript-eslint/parser
• Node 12.x - nx test @typescript-eslint/experimental-utils
• Node 12.x - nx test @typescript-eslint/scope-manager
• Node 12.x - nx test @typescript-eslint/visitor-keys
• Node 12.x - nx test @typescript-eslint/typescript-estree
• Node 12.x - nx run-many --target=build --all --parallel --exclude website --exclude website
• Node 12.x - nx run-many --target=build --all --parallel --exclude website --exclude website
• Node 16.x - nx test @typescript-eslint/parser
• Node 16.x - nx test @typescript-eslint/experimental-utils
• Node 16.x - nx test @typescript-eslint/scope-manager
• Node 14.x - nx test @typescript-eslint/parser
• Node 12.x - nx run-many --target=build --all --parallel --exclude website --exclude website
• Node 14.x - nx test @typescript-eslint/experimental-utils
• Node 16.x - nx test @typescript-eslint/visitor-keys
• Node 14.x - nx test @typescript-eslint/scope-manager
• Node 12.x - nx run-many --target=build --all --parallel --exclude website --exclude website
• Node 16.x - nx test @typescript-eslint/typescript-estree
• Node 14.x - nx test @typescript-eslint/visitor-keys
• Node 14.x - nx test @typescript-eslint/typescript-estree
• Node 16.x - nx run-many --target=build --all --parallel --exclude website --exclude website
• Node 14.x - nx run-many --target=build --all --parallel --exclude website --exclude website
• Node 12.x - nx prebuild @typescript-eslint/types
• Node 12.x - nx prebuild @typescript-eslint/types
• Node 12.x - nx prebuild @typescript-eslint/types
• Node 12.x - nx prebuild @typescript-eslint/types
• Node 16.x - nx prebuild @typescript-eslint/types
• Node 12.x - nx prebuild @typescript-eslint/types
• Node 14.x - nx prebuild @typescript-eslint/types

---

Sent with 💌 from NxCloud.

[netlify]

❌ Deploy Preview for typescript-eslint failed.

🔨 Explore the source changes: 8b4288a

🔍 Inspect the deploy log: https://app.netlify.com/sites/typescript-eslint/deploys/61c0fa45d8c444000897bf3a

[codecov]

Codecov Report

Merging #4334 (8b4288a) into main (c3e1834) will increase coverage by 0.67%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #4334      +/-   ##
==========================================
+ Coverage   93.52%   94.20%   +0.67%     
==========================================
  Files         336      152     -184     
  Lines       11596     8104    -3492     
  Branches     3291     2592     -699     
==========================================
- Hits        10845     7634    -3211     
+ Misses        479      266     -213     
+ Partials      272      204      -68     

Flag	Coverage Δ
unittest	94.20% <100.00%> (+0.67%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files	Coverage Δ
...nt-plugin/src/rules/consistent-type-definitions.ts	97.29% <100.00%> (ø)
...es/scope-manager/src/scope/ConditionalTypeScope.ts
packages/scope-manager/src/scope/SwitchScope.ts
...l-utils/src/ast-utils/eslint-utils/astUtilities.ts
...rimental-utils/src/ast-utils/eslint-utils/index.ts
packages/scope-manager/src/lib/es2020.intl.ts
...ges/experimental-utils/src/ast-utils/predicates.ts
...scope-manager/src/definition/VariableDefinition.ts
...s/scope-manager/src/scope/ClassStaticBlockScope.ts
packages/typescript-estree/src/ast-converter.ts
... and 175 more

[JoshuaKGoldberg]

👍