Skip to content

Conversation

@ho991217
Copy link
Collaborator

This pull request updates the ESLint configuration and related code to improve TypeScript support, enforce stricter linting rules, and fix minor issues. The most important changes are grouped below.

ESLint Configuration Improvements:

  • Switched from js.configs.recommended to eslint.configs.recommended for base ESLint configuration, ensuring better compatibility with the latest ESLint ecosystem.
  • Updated TypeScript configuration to use typescript-eslint parser and stricter rules, including enabling the @typescript-eslint/strict-boolean-expressions rule for enhanced type safety.
  • Changed TypeScript project glob pattern from ./packages/*/tsconfig.json to ./packages/**/tsconfig.json for broader coverage.
  • Removed the unused eslint-config-prettier dependency from package.json to clean up project dependencies.

Code Quality and Bug Fixes:

  • Fixed an asynchronous bug in getConfig by awaiting the pathExists check, ensuring correct config file resolution.
  • Added an ESLint directive to suppress the no-unused-vars warning for the abstract action method parameter in Base.ts, improving code clarity for future maintainers.

@ho991217 ho991217 self-assigned this Dec 26, 2025
@ho991217 ho991217 merged commit 6bfe23b into main Dec 26, 2025
2 checks passed
@ho991217 ho991217 deleted the remove-unused branch December 26, 2025 07:35
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