Skip to content

ci: pin bun to 1.2.15 to avoid bundler output drift#14

Merged
johnstcn merged 1 commit intomainfrom
ci/pin-bun-version
Mar 27, 2026
Merged

ci: pin bun to 1.2.15 to avoid bundler output drift#14
johnstcn merged 1 commit intomainfrom
ci/pin-bun-version

Conversation

@johnstcn
Copy link
Copy Markdown
Member

@johnstcn johnstcn commented Mar 26, 2026

Relates to #12

Problem: latest recently jumped to 1.3.11 which changed bundler preamble, causing check_unstaged.sh failures

  • Pin bun-version from latest to 1.2.15 (version from codercom/oss-dogfood) in CI workflow
  • Rebuild dist/index.js with pinned version to match

🤖 Created by Coder Agents, reviewed by me.

Copilot AI review requested due to automatic review settings March 26, 2026 12:19
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

Pins the Bun version used in CI to prevent dist/index.js bundler output from drifting when latest updates, which was causing check_unstaged.sh failures in the workflow for this GitHub Action.

Changes:

  • Pin oven-sh/setup-bun to Bun 1.2.15 in the CI workflow.
  • Regenerate dist/index.js so the committed build artifact matches the pinned Bun bundler output.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File Description
dist/index.js Rebuilt bundled artifact to match Bun 1.2.15 output and avoid CI “unstaged changes” drift.
.github/workflows/ci.yaml Pins Bun to 1.2.15 to stabilize builds across CI runs.

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

The CI workflow used bun-version: latest, which recently jumped to
1.3.11. Bun 1.3.10 changed the bundler preamble (WeakMap caching,
__accessProp, __exportSetter), so dist/index.js built locally with
1.2.x no longer matches what CI produces — instant check_unstaged
failure for anyone not running bleeding-edge bun.

ci: rebuild dist/index.js with bun 1.2.15
@johnstcn johnstcn force-pushed the ci/pin-bun-version branch from e53fdfa to b51313a Compare March 26, 2026 12:24
@johnstcn johnstcn self-assigned this Mar 26, 2026
@johnstcn johnstcn changed the title [DNM] ci: pin bun to 1.2.15 to avoid bundler output drift ci: pin bun to 1.2.15 to avoid bundler output drift Mar 26, 2026
@johnstcn johnstcn requested a review from mafredri March 26, 2026 12:36
@johnstcn johnstcn merged commit b6890d3 into main Mar 27, 2026
1 check passed
@johnstcn johnstcn deleted the ci/pin-bun-version branch March 27, 2026 09:38
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.

3 participants