Skip to content

chore: refresh dist/ for undici 6.29.0 - #146

Merged
twk3 merged 1 commit into
mainfrom
claude/inspiring-allen-kroyde
Oct 5, 2026
Merged

twk3 merged 1 commit into
mainfrom
claude/inspiring-allen-kroyde

Conversation

@twk3

@twk3 twk3 commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

Problem

main is red. The Check Transpiled JavaScript workflow fails on 0c857f0 (run 37334229323).

#145 (undici 6.28.0 → 6.29.0) was merged before that check finished, so dist/ was never regenerated. The committed bundles still carry the undici 6.28.0 code, while npm run bundle now produces the 6.29.0 code — check-dist.yml diffs the two and fails.

The diff CI reported is exactly the undici 6.29.0 websocket changes, e.g.:

-        if (!requestProtocols.includes(secProtocol)) {
+        if (requestProtocols === null || !requestProtocols.includes(secProtocol)) {

Fix

Ran npm run bundle and committed the result. No source changes — only the two generated bundles:

  • dist/index.js
  • dist/post.js

Verification

Reproduced the failure locally first (git diff --ignore-space-at-eol --text dist/ → 1816 lines, matching CI), then confirmed it clean after the rebuild:

  • npm run bundle → dist/ diff is now empty (the exact check check-dist.yml runs)
  • Re-ran npm run package a second time → byte-identical output, so the build is deterministic and this won't drift again on the next run
  • npm run format:check → passes
  • npm run lint → passes
  • npm test → 39/39 passing

🤖 Generated with Claude Code

https://claude.ai/code/session_015LJpmQCqwZ4GxHZusLGMTk


Generated by Claude Code


View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

The undici 6.28.0 -> 6.29.0 bump (#145) was merged before the
"Check Transpiled JavaScript" workflow finished, so dist/ was never
regenerated and main is now red: the committed bundles still contain
the undici 6.28.0 code.

Rebuild with `npm run bundle` so dist/index.js and dist/post.js match
the current sources and lockfile. No source changes.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_015LJpmQCqwZ4GxHZusLGMTk
@baz-reviewer

baz-reviewer Bot commented Oct 5, 2026 •

Copy link
Copy Markdown

Review this PR on Baz

Baz Summary

Regenerate the committed dist/index.js and dist/post.js bundles with undici 6.29.0. Synchronize HTTP, HTTP/2, WebSocket, and EventSource runtime behavior without changing source modules.

Topics

TopicDetails
EventSource parsing Update the bundled EventSourceStream parser to process chunked input more efficiently while preserving BOM, field validation, and event assembly behavior.
Modified files (2)
  • dist/index.js
  • dist/post.js
Latest Contributors(2)
UserCommitDate
noreply@anthropic.comchore: refresh dist/ f...October 05, 2026
dj@currents.devchore: refresh dist (#...August 07, 2026
Network runtime bundle Refresh bundled HTTP, HTTP/2, and WebSocket implementations from undici 6.29.0, including upgrade handling, socket validation, retry behavior, and protocol safety fixes.
Modified files (2)
  • dist/index.js
  • dist/post.js
Latest Contributors(2)
UserCommitDate
noreply@anthropic.comchore: refresh dist/ f...October 05, 2026
dj@currents.devchore: refresh dist (#...August 07, 2026

Merger  Activate to get a short verdict whether this PR is good to go or not

Skills  Activate Skill Maintainer to keep your skills up to date

Planner  This PR would have been improved with Baz Planner - Try it now

@coderabbitai

coderabbitai Bot commented Oct 5, 2026

Copy link
Copy Markdown

Important

Review skipped

Review was skipped due to path filters

⛔ Files ignored due to path filters (2)
  • dist/index.js is excluded by !**/dist/**
  • dist/post.js is excluded by !**/dist/**

CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including **/dist/** will override the default block on the dist directory, by removing the pattern from both the lists.

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Essentials
  • Run ID: 94960f11-14ef-4721-8806-9b2fd1302f9b

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Comment @coderabbitai help to get the list of available commands.

@twk3
twk3 merged commit 260fe80 into main Oct 5, 2026
29 checks passed
@twk3
twk3 deleted the claude/inspiring-allen-kroyde branch October 5, 2026 18:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants