feat: upgrade to node 24 + migrate to bun#65
Conversation
Smart Diff (Fails): FailureExpected tolerance |
Smart Diff (Works (sticky)): SuccessCheck succeeded with tolerance |
Smart Diff (Works): SuccessCheck succeeded with tolerance |
Smart Diff (Fails (sticky)): FailureExpected tolerance |
There was a problem hiding this comment.
Pull request overview
Upgrades the GitHub Action’s runtime/tooling toward Node 24 and switches the project build/test workflow from npm/Jest/ncc to Bun + Biome 2.
Changes:
- Migrate CI to Bun (install, test, pack) and replace ncc packaging with
bun buildgeneratingdist/index.js - Update Action metadata to run on
node24and modernize dependency versions / Biome configuration - Add a workflow to maintain “aggregate” tags (e.g.,
v1pointing to latestv1.x)
Reviewed changes
Copilot reviewed 10 out of 16 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| src/processing.ts | Reorders returned fields and ensures diff output is included in the result object. |
| src/notifications.ts | Refactors GitHub Actions context typing/import and reorders API payload fields. |
| src/inputs.ts | Reorders object literal fields for notifications and returned args. |
| package.json | Switches scripts/tooling to Bun; updates dependencies and removes Jest/ncc. |
| biome.json | Updates to Biome v2 schema/config and excludes dist/index.js from formatting/linting. |
| action.yml | Updates action metadata formatting and runtime to node24. |
| tests/main.test.ts | Adapts tests for new build artifact location and minor refactors. |
| .github/workflows/update-tags.yaml | Adds a tagging workflow to update major/minor aggregate tags. |
| .github/workflows/build.yml | Replaces setup-node/npm with setup-bun/bun commands. |
| .github/dependabot.yaml | Updates formatting and switches dependency ecosystem monitoring to Bun. |
Comments suppressed due to low confidence (1)
tests/main.test.ts:1
- The test harness asserts outputs via
::set-output, which is deprecated and not how outputs are produced on actual runners whenGITHUB_OUTPUTis set (file-based). Consider updating the tests to pointprocess.env.GITHUB_OUTPUTto a temp file and assert by reading/parsing that file, so tests match real GitHub Actions behavior more closely.`
import * as cp from "node:child_process";
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Smart Diff (Fails): FailureExpected tolerance |
Smart Diff (Works): SuccessCheck succeeded with tolerance |
Smart Diff (Works): SuccessCheck succeeded with tolerance |
Smart Diff (Fails): FailureExpected tolerance |
Smart Diff (Works): SuccessCheck succeeded with tolerance |
Smart Diff (Fails): FailureExpected tolerance |
Smart Diff (Fails): FailureExpected tolerance |
Smart Diff (Works): SuccessCheck succeeded with tolerance |
|



No description provided.