Uh oh!
There was an error while loading. Please reload this page.
feat: update Node.js version and dependencies, use rolldown, enhance configurations, fix lint issues - #28
Conversation
9fa26cb to
cf0b101CompareThere was a problem hiding this comment.
Pull request overview
This PR modernizes the action’s toolchain by bumping the Node.js baseline to v24, migrating the build output from ncc to rolldown, and introducing updated formatting/linting/commit tooling aligned with newer ESLint + TypeScript conventions.
Changes:
- Update runtime/tooling to Node.js 24 (engines, action runtime, CI workflow, TS target).
- Replace
nccbuild withrolldownand add/refresh repo-wide Prettier, ESLint (flat config), lint-staged, and commitlint configs. - Fix lint/type issues by tightening TS annotations and adjusting execution style (top-level
await).
Reviewed changes
Copilot reviewed 18 out of 21 changed files in this pull request and generated 7 comments.
Show a summary per file
| File | Description |
|---|---|
tsconfig.json | Updates TS target to ES2024 and normalizes exclude formatting. |
src/lab.ts | Adds top-level await and tightens typing for URL param / Octokit response. |
src/index.ts | Adds top-level await, adds lint suppression for GitHub payload access, tightens fetch param types. |
prettier.config.js | Introduces JS-based (ESM) Prettier configuration. |
package.json | Updates Node engine, deps, and switches build script to rolldown; adds CI format/lint scripts. |
lint-staged.config.js | Adds lint-staged configuration for formatting/linting staged files. |
eslint.config.js | Migrates to ESLint flat config with type-aware TypeScript rules. |
commitlint.config.js | Adds commitlint JS config. |
action.yml | Bumps action runtime to node24. |
README.md | Reformats workflow/docs content and updates development section formatting. |
.prettierrc.yml | Removes legacy Prettier YAML config in favor of JS config. |
.prettierignore | Ignores dist for formatting. |
.nvmrc | Updates local Node version to 24. |
.npmrc | Disables funding prompts. |
.lintstagedrc.yml | Removes legacy lint-staged YAML config in favor of JS config. |
.husky/pre-commit | Converts pre-commit hook to multi-line script and keeps building dist. |
.github/workflows/test.yml | Updates CI to use .nvmrc, adds formatting and linting checks, pins actions by SHA. |
.eslintrc.yml | Removes legacy ESLint YAML config in favor of flat config. |
.commitlintrc.yml | Removes legacy commitlint YAML config in favor of JS config. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
…configurations, fix lint issues
cf0b101 to
e6df3baCompareUh oh!
There was an error while loading. Please reload this page.
No description provided.