Conversation
|
Caution Review failedThe pull request is closed. WalkthroughUpdated documentation: README reordered TOC, added contributor badges, new Caveats entry for "TypeError: example is not a function", replaced Node.js tsconfig guidance with ESM-run instructions, adjusted examples to use Changes
Sequence Diagram(s)The changes are documentation-only and do not modify runtime control flow; no sequence diagram is provided. Estimated code review effort🎯 1 (Trivial) | ⏱️ ~3 minutes Possibly related PRs
Suggested labels
Poem
Pre-merge checks and finishing touches✅ Passed checks (5 passed)
📜 Recent review detailsConfiguration used: CodeRabbit UI Review profile: CHILL Plan: Pro 📒 Files selected for processing (1)
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 |
|
@coderabbitai, the cause of the issue is described here: #2981 (comment) |
|
Tip For best results, initiate chat on the files or code changes. 🧩 Analysis chain🏁 Script executed: Length of output: 570 @RobinTail, thank you for the clarification! I now understand the root cause: The issue occurs when users run v25 (ESM-only) with CommonJS code. Since Express Zod API imports and patches Zod's ESM bundle with the This PR correctly addresses the issue by updating the documentation to emphasize ESM requirements, including:
The documentation changes will help users properly configure their projects to use ESM, ensuring they use the same Zod instance that Express Zod API patches. ✏️ Learnings added
|
|
@coderabbitai, |
|
Tip For best results, initiate chat on the files or code changes. @RobinTail, thank you for the correction! You're absolutely right—
I've updated my understanding accordingly. ✏️ Learnings added
⛔ Learnings removed🧠 Learnings used |
closes #2981
Summary by CodeRabbit
Documentation
Changelog
Contributors