Remove undici 7.18.2 override, bump wrangler to 4.83.0#49
Conversation
miniflare 4.20260415.0 (via wrangler 4.83.0) pulls undici 7.24.8, which includes the backported fix for the 401-with-request-body crash (nodejs/undici#4941, cloudflare/workers-sdk#13450). The override is no longer needed.
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 57 minutes and 56 seconds. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ℹ️ Review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
📝 WalkthroughWalkthroughDocumentation and configuration updates addressing a Linux CI blocker for DO sync e2e testing, reorganizing kanban task statuses, removing outdated undici override documentation, and bumping Cloudflare devDependency versions while eliminating the associated undici override constraint. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~10 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Deploying with
|
| Status | Name | Latest Commit | Updated (UTC) |
|---|---|---|---|
| ✅ Deployment successful! View logs |
cloudstash | 5b9ee37 | Apr 20 2026, 10:45 AM |
There was a problem hiding this comment.
🧹 Nitpick comments (1)
docs/kanban.md (1)
20-21: Nit: "Done" section contains an unchecked item.Not part of this diff, but worth flagging while touching this file — Line 26 has
- [ ] [[todos/publish-raycast-extension|...]]under the Done heading. Either move it back to In Progress/Todo or mark it[x]to keep the board consistent.🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@docs/kanban.md` around lines 20 - 21, Under the "Done" section there's an unchecked item `- [ ] [[todos/publish-raycast-extension|...]]`; either move that checklist line back to the TODO/In Progress section or mark it completed by changing the checkbox to `[x]` so the board's "Done" heading contains only completed items; update the `- [ ] [[todos/publish-raycast-extension|...]]` line accordingly and keep the rest of the Done section unchanged.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In `@docs/kanban.md`:
- Around line 20-21: Under the "Done" section there's an unchecked item `- [ ]
[[todos/publish-raycast-extension|...]]`; either move that checklist line back
to the TODO/In Progress section or mark it completed by changing the checkbox to
`[x]` so the board's "Done" heading contains only completed items; update the `-
[ ] [[todos/publish-raycast-extension|...]]` line accordingly and keep the rest
of the Done section unchanged.
ℹ️ Review info
⚙️ Run configuration
Configuration used: defaults
Review profile: CHILL
Plan: Pro
Run ID: 28d43265-7f80-46b6-abe3-f77699189536
⛔ Files ignored due to path filters (1)
bun.lockis excluded by!**/*.lock
📒 Files selected for processing (4)
docs/kanban.mddocs/todos/e2e-do-sync-testing.mddocs/todos/remove-undici-override.mdpackage.json
💤 Files with no reviewable changes (1)
- docs/todos/remove-undici-override.md
- oxfmt pass on docs/kanban.md - Sync link-processor excalidraw diagram
Summary
wrangler4.82.2 → 4.83.0 and@cloudflare/vite-plugin1.32.2 → 1.32.3overrides.undici: 7.18.2workaroundWhy now
The override worked around a regression in undici 7.24.4 (shipped by miniflare) where
isTraversableNavigable()incorrectly returnedtrue, triggering browser-only 401 credential-retry logic against stream-backed bodies and crashing any POST withTypeError: fetch failed.The fix has landed end-to-end:
After this PR,
undiciresolves to 7.25.0 via miniflare's range.Verified
bun run typecheckclean.POST /api/ingestwith bad/missing key now returns a 401 JSON body instead of crashing withTypeError: fetch failed.Summary by CodeRabbit
Release Notes
Chores
Documentation