Skip to content

Upgrade Node to v24#50

Merged
AButler merged 9 commits into
mainfrom
node-upgrade
May 2, 2026
Merged

Upgrade Node to v24#50
AButler merged 9 commits into
mainfrom
node-upgrade

Conversation

@AButler
Copy link
Copy Markdown
Owner

@AButler AButler commented May 2, 2026

  • Added tests
  • Refactored

- Added tests
- Refactored

Co-authored-by: Copilot <copilot@github.com>
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the GitHub Action to run on Node.js 24, modernizes dependencies/tooling, and adds a Vitest-based test suite while refactoring the action logic into a separate run() module.

Changes:

  • Upgrade runtime to node24 (action + CI) and update dependencies/@types.
  • Refactor action implementation into src/run.ts and add Vitest tests + coverage config.
  • Switch build output generation to an esbuild bundle targeting Node 24.

Reviewed changes

Copilot reviewed 8 out of 13 changed files in this pull request and generated 7 comments.

Show a summary per file
File Description
action.yml Moves action runtime from node20 to node24.
.github/workflows/ci.yml Runs CI on Node 24; adds audit + test steps.
package.json Updates deps; adds Vitest tooling; changes build to tsc --noEmit + esbuild.
package-lock.json Lockfile updates for new dependency set.
tsconfig.json Updates TS target to ES2024; adjusts excludes.
src/run.ts New extracted run() implementation for the action logic.
src/main.ts Now delegates to run().
__tests__/main.test.ts New Vitest suite covering release-id/tag/context + uploads.
vitest.config.ts Adds Vitest config + V8 coverage settings.
vitest.config.js / vitest.config.js.map Added generated Vitest config artifacts (see comments).
dist/main.js.map Updates committed sourcemap artifact.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread vitest.config.js Outdated
Comment thread package.json Outdated
Comment thread package.json Outdated
Comment thread src/main.ts Outdated
Comment thread src/run.ts Outdated
Comment thread src/run.ts Outdated
Comment thread src/run.ts Outdated
AButler and others added 2 commits May 2, 2026 11:42
Co-authored-by: Copilot <copilot@github.com>
Co-authored-by: Copilot <copilot@github.com>
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 7 out of 11 changed files in this pull request and generated 4 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/run.ts Outdated
Comment thread src/run.ts Outdated
Comment thread src/run.ts Outdated
Comment thread src/run.ts
Co-authored-by: Copilot <copilot@github.com>
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 7 out of 11 changed files in this pull request and generated 5 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread action.yml
Comment thread .github/workflows/ci.yml
Comment thread src/run.ts Outdated
Comment thread src/run.ts
Comment thread src/run.ts Outdated
Co-authored-by: Copilot <copilot@github.com>
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 7 out of 11 changed files in this pull request and generated 4 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread action.yml
Comment thread src/run.ts Outdated
Comment thread src/run.ts Outdated
Comment thread src/run.ts Outdated
Co-authored-by: Copilot <copilot@github.com>
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 7 out of 11 changed files in this pull request and generated 4 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/run.ts Outdated
Comment thread src/run.ts Outdated
Comment thread src/run.ts Outdated
Comment thread src/main.ts Outdated
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 7 out of 11 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/run.ts Outdated
Comment thread src/run.ts Outdated
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 7 out of 11 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread src/run.ts Outdated
Comment thread src/main.ts
Co-authored-by: Copilot <copilot@github.com>
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 7 out of 11 changed files in this pull request and generated 2 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread .github/workflows/ci.yml
Comment thread src/run.ts
@AButler AButler merged commit 00b9d83 into main May 2, 2026
5 checks passed
@AButler AButler deleted the node-upgrade branch May 2, 2026 15:23
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