Skip to content

Conversation

@ludamad
Copy link
Collaborator

@ludamad ludamad commented Jan 16, 2026

  • Replace eslint-plugin-import with eslint-plugin-import-x (rust-based resolver) for faster linter time, particularly for no-cycle

@socket-security
Copy link

socket-security bot commented Jan 16, 2026

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff Package Supply Chain
Security
Vulnerability Quality Maintenance License
Addednpm/​eslint-plugin-import-x@​4.16.19710010088100

View full report

@ludamad ludamad force-pushed the ad/eslint-import-x-optimization branch 3 times, most recently from 935ea58 to 5c89f4b Compare January 16, 2026 19:10
Replace eslint-plugin-import with eslint-plugin-import-x which uses a
rust-based resolver for significantly faster import cycle detection.

- Lint time reduced from ~3:22 to ~22 seconds (9x speedup)
- Updated rule names from import/* to import-x/*
- Updated eslint-disable comments in constants.ts

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@ludamad ludamad force-pushed the ad/eslint-import-x-optimization branch from 5c89f4b to 9cffdb1 Compare January 16, 2026 19:20
@AztecBot
Copy link
Collaborator

Flakey Tests

🤖 says: This CI run detected 2 tests that failed, but were tolerated due to a .test_patterns.yml entry.

\033FLAKED\033 (8;;http://ci.aztec-labs.com/dd04ea5e4bab013b�dd04ea5e4bab013b8;;�):  yarn-project/end-to-end/scripts/run_test.sh simple src/e2e_p2p/gossip_network_no_cheat.test.ts (53s) (code: 1) group:e2e-p2p-epoch-flakes (\033ludamad\033: chore: switch to eslint-plugin-import-x for faster linting)
\033FLAKED\033 (8;;http://ci.aztec-labs.com/52322ffc3776f8fb�52322ffc3776f8fb8;;�): yarn-project/scripts/run_test.sh prover-client/src/proving_broker/proving_broker.test.ts (30s) (code: 1) (\033ludamad\033: chore: switch to eslint-plugin-import-x for faster linting)

@ludamad ludamad enabled auto-merge January 16, 2026 22:46
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.

3 participants