fix(webapp): stop logging full batch item contents in batchTriggerV3 - #4404
fix(webapp): stop logging full batch item contents in batchTriggerV3#4404carderne wants to merge 2 commits into
Conversation
|
The idempotency grouping step logged every batch item verbatim, including its payload and options (which can carry arbitrary customer metadata). Replace that with a summary of task identifiers and item counts, and add a unit test covering the summary shape.
7f5f033 to
a978755
Compare
WalkthroughAdded an exported helper that summarizes grouped batch items by task identifier and count. Updated debug logging to emit the summary instead of raw item payloads. Added tests covering populated inputs, sensitive-field exclusion, and empty input behavior. 🚥 Pre-merge checks | ✅ 3 | ❌ 2❌ Failed checks (2 warnings)
✅ Passed checks (3 passed)
✨ Finishing Touches📝 Generate docstrings
🧪 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 |
![devin-ai-integration[bot]](https://avatars.githubusercontent.com/in/811515?s=60&v=4){kind=small}
1 participant
Summary
The idempotency grouping diagnostic now records task identifiers and item counts rather than full batch items. This keeps the log useful for troubleshooting without including payloads or options.
Performance notes
Should be better than baseline.