fix(ci): bump unit-tests bun-version to 1.3.14 to fix coverage WriteFailed by duyet · Pull Request #2242 · chmonitor/chmonitor · GitHub
Skip to content

fix(ci): bump unit-tests bun-version to 1.3.14 to fix coverage WriteFailed - #2242

Merged
duyet merged 1 commit into
mainfrom
fix/bun-version-coverage-writer
Jul 3, 2026
Merged

fix(ci): bump unit-tests bun-version to 1.3.14 to fix coverage WriteFailed#2242
duyet merged 1 commit into
mainfrom
fix/bun-version-coverage-writer

Conversation

@duyet

@duyet duyet commented Jul 3, 2026

Copy link
Copy Markdown
Member

Summary

Bun 1.3.13's lcov coverage writer intermittently throws error: An internal error occurred (WriteFailed) right after all tests finish (all tests pass beforehand, crash is in the writer). Reproduced on main's own recent Test workflow runs (e.g. run 28663719011 on commit db35d2d), not tied to any single PR's diff — this was blocking unit-tests CI on multiple unrelated open PRs (#2232, #2235, #2240) independently of their actual code changes.

Change

Bump the unit-tests job's pinned bun-version from 1.3.13 to 1.3.14 in .github/workflows/test.yml.

Verification

  • bun run test:coverage passes clean locally under 1.3.14 (thousands of tests, 0 fail).
  • No application code touched.

Co-Authored-By: duyetbot bot@duyet.net

…ailed

Bun 1.3.13's lcov coverage writer intermittently throws
`error: An internal error occurred (WriteFailed)` right after all tests
finish (all tests pass beforehand). Reproduced repo-wide on main's own
Test workflow runs, not tied to any single PR's diff.

Co-Authored-By: duyetbot <bot@duyet.net>
@duyet
duyet enabled auto-merge (squash) July 3, 2026 16:19
@github-actions github-actions Bot added the area: ci CI / workflows / .github label Jul 3, 2026
@github-actions

github-actions Bot commented Jul 3, 2026

Copy link
Copy Markdown
Contributor

@duyet
duyet merged commit cc3b790 into main Jul 3, 2026
13 of 14 checks passed
@duyet
duyet deleted the fix/bun-version-coverage-writer branch July 3, 2026 16:20
duyet added a commit that referenced this pull request Jul 3, 2026
…iled #2242)

Resolves conflicts in user-connections.ts / user-connections/$id.ts where
this branch's outbound webhook bus (emitEvent) and main's audit log
(logEvent/logSessionEvent) both hook the connection create/delete path —
kept both, audit log first then webhook emit. Adds emitEvent mocks to
user-connections.test.ts and user-connections/$id.test.ts (main's
audit-log tests didn't know about the webhook bus, so the route now
transitively pulled in @chm/platform's cloudflare:workers binding
under bun test).

Co-Authored-By: duyetbot <bot@duyet.net>
duyet added a commit that referenced this pull request Jul 3, 2026
…NT) (#2252)

bun 1.3.3's `install --frozen-lockfile` regresses with
"ENOENT: could not open the node_modules directory" on jobs without a
pre-existing node_modules, breaking the Landing/Blog/Docs/a11y checks and the
Cloudflare deploy's landing job on every main commit. #2242 already proved
1.3.14 fixes it for unit-tests; align the remaining 8 pins to match.

Co-authored-by: duyetbot <bot@duyet.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area: ci CI / workflows / .github

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant