Skip to content

Align GitHub Actions Bun version with the repo lockfile - #457

Merged
frankinkfnd merged 2 commits into
mainfrom
copilot/fix-failing-github-actions-job-test
Sep 4, 2026
Merged

Align GitHub Actions Bun version with the repo lockfile#457
frankinkfnd merged 2 commits into
mainfrom
copilot/fix-failing-github-actions-job-test

Conversation

Copilot AI commented Sep 4, 2026

Copy link
Copy Markdown
Contributor

The test GitHub Actions job was failing before build/test execution because CI installed a newer Bun than the repo declares, which caused tracked lockfile drift and tripped the dirty-workspace check.

  • Root cause

    • setup-bun in CI was pinned to latest while the repository declares bun@1.3.14 in package.json.
    • bun install under the newer runtime rewrote tracked state, so git diff --exit-code --quiet failed immediately.
  • Workflow changes

    • Removed bun-version: latest from the unit-test workflow so oven-sh/setup-bun resolves Bun from the repository’s packageManager.
    • Removed bun-version: latest from the E2E workflow for the same reason, keeping both pipelines consistent.
  • Install behavior

    • Updated the unit-test workflow to use a frozen install:
      - uses: oven-sh/setup-bun@v2
      
      - name: Install dependencies
        run: bun install --frozen-lockfile
    • This preserves the existing clean-workspace gate while making lockfile drift fail explicitly instead of mutating the checkout.
  • Net effect

    • CI now uses the same Bun version the repository declares.
    • The test job no longer fails on install-induced workspace changes before reaching build and test steps.

Copilot AI changed the title [WIP] Fix failing GitHub Actions job test Align GitHub Actions Bun version with the repo lockfile Sep 4, 2026
Copilot AI requested a review from frankinkfnd September 4, 2026 18:55
@frankinkfnd
frankinkfnd marked this pull request as ready for review September 4, 2026 18:55

@claude claude Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Claude Code Review

This repository is configured for manual code reviews. Comment @claude review for a one-time review, or @claude review always to subscribe this PR to a review on every future push.

Tip: disable this comment in your organization's Code Review settings.

@frankinkfnd
frankinkfnd merged commit 6cf39f2 into main Sep 4, 2026
1 check passed
@frankinkfnd
frankinkfnd deleted the copilot/fix-failing-github-actions-job-test branch September 4, 2026 18:56
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