Repository navigation
chore: refresh dist/ for undici 6.29.0 - #146
Conversation
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
|
| Topic | Details | |||||||||
|---|---|---|---|---|---|---|---|---|---|---|
| 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)
Latest Contributors(2)
| |||||||||
| 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)
Latest Contributors(2)
|
|
Important Review skippedReview was skipped due to path filters ⛔ Files ignored due to path filters (2)
CodeRabbit blocks several paths by default. You can override this behavior by explicitly including those paths in the path filters. For example, including ⚙️ Run configuration
You can disable this status message by setting the Use the checkbox below for a quick retry:
Comment |
Problem
mainis red. TheCheck Transpiled JavaScriptworkflow fails on0c857f0(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, whilenpm run bundlenow produces the 6.29.0 code —check-dist.ymldiffs the two and fails.The diff CI reported is exactly the undici 6.29.0 websocket changes, e.g.:
Fix
Ran
npm run bundleand committed the result. No source changes — only the two generated bundles:dist/index.jsdist/post.jsVerification
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 checkcheck-dist.ymlruns)npm run packagea second time → byte-identical output, so the build is deterministic and this won't drift again on the next runnpm run format:check→ passesnpm run lint→ passesnpm test→ 39/39 passing🤖 Generated with Claude Code
https://claude.ai/code/session_015LJpmQCqwZ4GxHZusLGMTk
Generated by Claude Code
Need help on this PR? Tag
@codesmith-botwith what you need. Autofix is disabled.