Skip to content

chore(lint): add ESLint (type-aware) + Prettier and wire into CI#7

Merged
NagaYu merged 1 commit into
mainfrom
chore/eslint-prettier
Jun 28, 2026
Merged

chore(lint): add ESLint (type-aware) + Prettier and wire into CI#7
NagaYu merged 1 commit into
mainfrom
chore/eslint-prettier

Conversation

@NagaYu

@NagaYu NagaYu commented Jun 28, 2026

Copy link
Copy Markdown
Owner

Resolves #4.

What

  • ESLint (flat config) with typescript-eslint recommendedTypeChecked (type-aware) rules.
  • Prettier for formatting + eslint-config-prettier so the two never conflict.
  • tsconfig.eslint.json so test/** gets type info during linting (build tsconfig only includes src/**).
  • npm scripts: lint, lint:fix, format, format:check.
  • CI gains Lint and Format check steps across Node 18/20/22.

Lint findings fixed

  • Removed unnecessary type assertions in extractor.ts / sync.ts.
  • prefer-const for Google's rejected counter.
  • require-await / no-misused-promises: refactored the cron scheduler to fire-and-forget an async tick() via () => void tick().
  • Applied Prettier formatting repo-wide.

Verification (local, all green)

npm run typecheck · npm run lint · npm run format:check · npm test (24/24) · npm run build.

- Flat ESLint config with typescript-eslint recommendedTypeChecked rules
- Prettier for formatting; eslint-config-prettier to avoid conflicts
- tsconfig.eslint.json so test/** gets type information for linting
- npm scripts: lint, lint:fix, format, format:check
- CI runs lint + format:check alongside typecheck/test/build
- Apply Prettier formatting and fix lint findings (unnecessary casts,
  prefer-const, require-await, no-misused-promises in the scheduler)

Closes #4

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
@NagaYu NagaYu merged commit c9e21c5 into main Jun 28, 2026
3 checks passed
@NagaYu NagaYu deleted the chore/eslint-prettier branch June 28, 2026 11:09
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.

Add ESLint + Prettier and wire into CI

1 participant