docs: Remove old idempotencyKey warning#3306
Conversation
|
Preview deployment for your docs. Learn more about Mintlify Previews.
|
|
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (1)
💤 Files with no reviewable changes (1)
WalkthroughThis is a documentation update removing a warning block from Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes 🚥 Pre-merge checks | ✅ 2 | ❌ 1❌ Failed checks (1 warning)
✅ Passed checks (2 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 |
There was a problem hiding this comment.
Devin Review found 1 potential issue.
⚠️ 1 issue in files not directly in the diff
⚠️ Stale idempotencyKey warning remains in .cursor/rules/writing-tasks.mdc after removal from docs (.cursor/rules/writing-tasks.mdc:1055)
The PR removes the warning about idempotencyKey not being available with triggerAndWait/batchTriggerAndWait from docs/triggering.mdx, but the identical stale warning still exists at .cursor/rules/writing-tasks.mdc:1055: "Not available with triggerAndWait or batchTriggerAndWait in v3.3.0+ due to a bug". Since the bug is now fixed (as evidenced by packages/trigger-sdk/src/v3/shared.ts:2378-2406 properly handling idempotencyKey in triggerAndWait_internal), this stale note in the cursor rules file will cause AI coding agents to give incorrect guidance, telling users they cannot use idempotency keys with triggerAndWait/batchTriggerAndWait when they actually can.
View 1 additional finding in Devin Review.
No description provided.