Conversation
|
Warning Review limit reachedNext included review available in 59 minutes. View limit detailsLimit details: You’ve used all 2 included reviews currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (3)
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 |
5c7f84b to
cbe870d
Compare
|
Claims to be a "one-time" notice but prints to stderr on every engine start. The cost it warns about is opt-in and already in the first paragraph of the TEA guide so leaving as is for now. thanks for the effort! |
What: Add a one-time, non-blocking stderr notice inside TeaPlugin.validate() stating that enabling tea's six true-by-default settings injects 6 extra agent sessions per story, pointing to docs/tea-plugin-guide.md.
Why: The six cost-affecting settings all default to true with no signal at the moment of activation; the guide documents the cost but an operator who edits policy.toml or toggles the plugin without reading it first gets no in-the-moment warning.
How:
Testing: Add a test to tests/test_plugin_tea.py asserting the notice is emitted on validate() when the defaults are in effect, and that the plugin still activates normally.
Changelog: "Added: the TEA plugin now prints a one-time notice at activation about the extra agent sessions its default settings add per story."
🤖 Generated with Claude Code