Skip to content

chore: add ESLint config, root package.json, and fix type import ordering#16

Merged
kumilingus merged 14 commits into
clientIO:mainfrom
Geliogabalus:eslint
Mar 5, 2026
Merged

chore: add ESLint config, root package.json, and fix type import ordering#16
kumilingus merged 14 commits into
clientIO:mainfrom
Geliogabalus:eslint

Conversation

@Geliogabalus
Copy link
Copy Markdown
Contributor

@Geliogabalus Geliogabalus commented Feb 24, 2026

Summary

  • Add root eslint.config.mjs with shared ESLint flat config for JS and TS files across all demos
  • Add root package.json with lint, build, and screenshot scripts, plus ESLint devDependencies
  • Fix type import ordering across 252 TypeScript files (regular imports before type imports)
  • Update README.md with a Root Scripts section documenting the new npm scripts
  • Add GitHub Actions workflow that runs ESLint on every non-draft PR and blocks merging on lint errors

Details

eslint.config.mjs

  • Covers .js, .jsx, .mjs and .ts, .tsx, .mts files
  • Uses @typescript-eslint/eslint-plugin flat config preset (flat/recommended)
  • Enforces 4-space indent, single quotes, semicolons, prefer-const, consistent-type-imports
  • Ignores .github/, build/, dist/, node_modules/

Type import ordering

  • Reordered imports in 252 TS/TSX files so import type statements always follow regular import statements
  • Files with intentional section comments were left untouched

.github/workflows/lint.yml

  • Triggers on PR open, sync, reopen, and ready-for-review events
  • Skips draft PRs — only runs when the PR is marked as ready
  • Runs npm run lint; a non-zero exit blocks merging

Test plan

  • Run npm install at the repo root
  • Run npm run lint — should report no errors
  • Open a draft PR and confirm the lint workflow does not run
  • Mark the PR as ready and confirm the lint workflow runs and passes
  • Spot-check a few TS files to confirm type imports appear after regular imports

🤖 Generated with Claude Code

@Geliogabalus Geliogabalus requested a review from kumilingus March 4, 2026 12:20
@kumilingus kumilingus merged commit c062891 into clientIO:main Mar 5, 2026
1 check passed
@Geliogabalus Geliogabalus deleted the eslint branch March 9, 2026 09:53
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