fix: address plugin security review - #48
Conversation
Qodo reviews are paused for this user.Troubleshooting steps vary by plan Learn more → On a Teams plan? Using GitHub Enterprise Server, GitLab Self-Managed, or Bitbucket Data Center? |
|
Warning Review limit reachedNext included review available in 47 minutes. View limit detailsLimit details: You’ve used the included review 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: defaults Review profile: CHILL Plan: Pro Plus Run ID: 📒 Files selected for processing (11)
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 |
Summary
exportexamples from plugin, core, DOX, protocol, and generated agent guidanceValidation
python3 scripts/validate-plugin-packages.pypython3 /Users/lazy/.codex/skills/.system/plugin-creator/scripts/validate_plugin.py plugins/tree-ring-memoryclaude plugin validate .claude plugin validate plugins/tree-ring-memorycargo fmt --all -- --checkcargo test -p tree-ring-memory-cli agent_awareness --locked(14 passed)git diff --checkAddresses the actionable CodeRabbit review comments on #47.
High-level PR Summary
This PR addresses security review findings by removing unsafe token-bearing
exportexamples from all documentation and replacing them with guidance to use history-safe, no-echo prompts or secret-manager injection for theTREE_RING_COORDINATOR_TOKEN. It also updates security, privacy, and support links to point to the canonical repository, enables private vulnerability reporting, and adds validation checks to prevent regressions of these security practices.⏱️ Estimated Review Time: 30-90 minutes
💡 Review Order Suggestion
scripts/validate-plugin-packages.pyplugins/tree-ring-memory/SECURITY.mdplugins/tree-ring-memory/PRIVACY.mdplugins/tree-ring-memory/TERMS.mdREADME.mdskills/tree-ring-memory/SKILL.mdplugins/tree-ring-memory/skills/tree-ring-memory/SKILL.mddocs/integrations/agent-skill.mdtemplates/dox/AGENTS.mddocs/protocol/memory-event.mdcrates/tree-ring-memory-cli/src/agent_awareness.rs