Repository navigation
fix: cap web_execute debug log output - #1343
Conversation
|
Warning Review limit reachedYou've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Next included review available in 10 minutes. View limit details
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 |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1343 +/- ##
==========================================
+ Coverage 92.11% 92.21% +0.09%
==========================================
Files 50 50
Lines 7723 7728 +5
Branches 1340 1341 +1
==========================================
+ Hits 7114 7126 +12
Misses 400 400
+ Partials 209 202 -7
Flags with carried forward coverage won't be shown. Click here to find out more.
... and 1 file with indirect coverage changes Continue to review full report in Codecov by Harness.
🚀 New features to boost your workflow:
|
ℹ️ Description
web_executelogged every script and its full result at DEBUG level. Reading Astro island props produced a single log line of ~121,000 characters with the complete ad and user data, which users attach to public issues.📋 Changes Summary
misc.truncate_for_log(): collapses whitespace to one line and caps the text (default 300 chars) with a… (+N chars)marker.web_executeuses it for both the script and the result in its debug line.web_executelog line.⚙️ Type of Change
✅ Checklist
pdm run test).pdm run format).pdm run lint).By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.