Skip to content

fix(linter/array-type): handle TSSatisfiesExpression#16904

Closed
tt-a1i wants to merge 1 commit intooxc-project:mainfrom
tt-a1i:fix/array-type-satisfies-expression
Closed

fix(linter/array-type): handle TSSatisfiesExpression#16904
tt-a1i wants to merge 1 commit intooxc-project:mainfrom
tt-a1i:fix/array-type-satisfies-expression

Conversation

@tt-a1i
Copy link
Contributor

@tt-a1i tt-a1i commented Dec 15, 2025

Fixes #16897

The array-type rule was not checking array types in satisfies expressions. This adds handling for TSSatisfiesExpression AST node type to enforce array type style consistency.

Changes

  • Added AstKind::TSSatisfiesExpression case in the run method
  • Regenerated rule_runner_impls.rs to include the new node type in NODE_TYPES
  • Added pass/fail/fix test cases including nested scenarios

Fixes oxc-project#16897

The `array-type` rule was not checking array types in `satisfies`
expressions. This adds handling for `TSSatisfiesExpression` AST node
type to enforce array type style consistency.
@tt-a1i tt-a1i requested a review from camc314 as a code owner December 15, 2025 14:06
@github-actions github-actions bot added A-linter Area - Linter C-bug Category - Bug labels Dec 15, 2025
@tt-a1i
Copy link
Contributor Author

tt-a1i commented Dec 15, 2025

Closing as duplicate - #16903 was merged first with the same fix.

@tt-a1i tt-a1i closed this Dec 15, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-linter Area - Linter C-bug Category - Bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Linter: typescript/array-type doesn’t catch all readonly array types

1 participant