Skip to content

chore(scripts): reorganize static analysis validations#8191

Merged
kuhe merged 1 commit into
mainfrom
kuhe/chore/static
Jul 14, 2026
Merged

chore(scripts): reorganize static analysis validations#8191
kuhe merged 1 commit into
mainfrom
kuhe/chore/static

Conversation

@kuhe

@kuhe kuhe commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Issue

JS-7014

Description

  • reorganizes the static analysis validators to share a "bus" 🚌 for AST traversal
  • this is to eliminate the redundant AST traversals when running all the validators at the same time. They now share a bus that traverses the AST and each validator looks at different parts of the code.

Excluding eslint, the static analyzers now run in ~7s instead of taking ~60s.

@kuhe kuhe requested a review from a team as a code owner July 14, 2026 16:25
@kuhe kuhe force-pushed the kuhe/chore/static branch from 0d7b320 to 5e73876 Compare July 14, 2026 17:02
@kuhe kuhe force-pushed the kuhe/chore/static branch from 5e73876 to 18fa647 Compare July 14, 2026 17:26
@kuhe kuhe merged commit 4871e09 into main Jul 14, 2026
7 checks passed
@kuhe kuhe deleted the kuhe/chore/static branch July 14, 2026 19:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants