Skip to content

chore: upgrade CI to Node 22, zero ESLint warnings, drop activeTab#68

Merged
heznpc merged 1 commit intomainfrom
fix/ci-lint-permissions
Apr 9, 2026
Merged

chore: upgrade CI to Node 22, zero ESLint warnings, drop activeTab#68
heznpc merged 1 commit intomainfrom
fix/ci-lint-permissions

Conversation

@heznpc
Copy link
Copy Markdown
Owner

@heznpc heznpc commented Apr 9, 2026

Summary

  • CI Node.js 20 → 22 LTS: Preempts forced migration on June 2, 2026
  • ESLint 24 → 0 warnings: Added caughtErrorsIgnorePattern to config, fixed all unused vars across 12 files
  • Remove activeTab permission: Not referenced anywhere in source — reduces Chrome Web Store review friction

Test plan

  • ESLint: 0 errors, 0 warnings
  • 309 tests pass
  • Prettier passes
  • CI passes (first run on Node 22)

🤖 Generated with Claude Code

- Upgrade CI workflow from Node.js 20 to 22 LTS (20 deprecated June 2026)
- Add caughtErrorsIgnorePattern to ESLint config for catch clause vars
- Fix all 24 ESLint warnings across src/ and tests/ (now 0 warnings)
- Remove unused activeTab permission (reduces attack surface for store review)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@heznpc heznpc merged commit 5b49f54 into main Apr 9, 2026
2 checks passed
@heznpc heznpc deleted the fix/ci-lint-permissions branch April 9, 2026 14:56
heznpc added a commit that referenced this pull request Apr 12, 2026
- PRIVACY_POLICY.md: drop activeTab (removed in #68), list alarms with its
  actual purpose (24h cache cleanup + 7d update check), and bump the
  "Last updated" date to reflect this edit.
- docs/i18n/README_*.md: add the VERSION_START/END marker to all six
  translated READMEs so generate-docs.js can keep them in sync with
  manifest.json from now on.
- src/data/*.json: bump _meta.version from the 3.3.0 residual to the
  current 3.5.4 across all ten premium dictionaries (lastUpdated stays
  2026-04-02 because the curated contents were not edited).
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.

1 participant