fix: revert "chore: bump d3-color and d3" — d3 v7 is ESM-only and crashes the CJS runtime - #323
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (2)
📝 WalkthroughWalkthroughThe pull request adds weekly Dependabot updates for root-level npm dependencies and excludes major updates for ChangesDependency updates
Estimated code review effort: 1 (Trivial) | ~5 minutes 🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Warning There were issues while running some tools. Please review the errors and either fix the tool's configuration or disable the tool if it's a critical failure. 🔧 ESLint
package.jsonParsing error: ESLint was configured to run on 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 |
Production went down (#320): every card endpoint returned FUNCTION_INVOCATION_FAILED. Runtime logs show the crash at module load:
d3 v7 (and d3-color v3) are ESM-only; this project compiles to CommonJS, so
require('d3')throws before any handler runs — the same failure mode as the July d3-color incident (#275). Local tests/tsc/ncc stay green because they resolve modules differently than Vercel's runtime loader, which is why #318 passed CI. The outage shipped inside v0.12.0 and was masked by the 48h CDN window until cached cards expired.Production is already restored via a hotfix push to
releasecarrying this same revert — this PR reconcilesmainso the next regular deploy doesn't re-ship the crash.Also adds
.github/dependabot.ymlignoring semver-major updates for d3/d3-color so this class of bump stops arriving until a proper ESM migration is planned.Refs #320 (left open for monitoring).
🤖 Generated with Claude Code
Summary by CodeRabbit