🧹 chore: Apply ESLint auto-fixes and manual corrections #10549
+79
−75
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Summary
This PR applies ESLint code style fixes across the codebase to ensure consistent formatting and compliance with the project's linting rules. The majority of changes were automatically fixed by running
npm run lint:fix, with three additional manual fixes applied to the touched files.Auto-fixed changes (via
npm run lint:fix):eslint-disablecomments where intentional rule violations are necessary (e.g., import cycles, disabled tests, commented-out tests)Manual fixes:
These changes improve code consistency and maintainability without affecting functionality. All changes are purely stylistic and do not modify any business logic or behavior.
Note: This PR only addresses the auto-fixable lint issues and three manual fixes in the specific files that were touched. There are still remaining lint errors and warnings in other parts of the codebase (186 problems: 85 errors, 101 warnings) that are outside the scope of this PR and will need to be addressed separately.
I added two entries to the eslint ignore list to prevent scanning of build artifacts and Python virtual environments:
Change Type
Please delete any irrelevant options.
Testing
All existing tests should continue to pass as these are purely stylistic changes. The changes were generated by running
npm run lint:fixfollowed by three manual fixes.Test Configuration:
No special test configuration required. Standard test suite execution should verify that:
Testing Steps:
npm testNote: Running
npm run lintwill still show errors/warnings in other files that are not part of this PR. This PR only fixes the auto-fixable issues and three manual fixes in the specific files that were modified.Checklist
Please delete any irrelevant options.