Repository navigation
docs: add agent instructions with public repository confidentiality rules - #311
Conversation
…ules Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
🔒 Security Scan Complete✅ No security issues detected Scanned files against 45 rules. · Reviewed commit: 📜 Scan history (1 scan)
ℹ️ About PR Security AgentThis scan was performed by PR Security Agent, an automated security review tool for pull requests. See the User Guide for details on severity levels, feedback reactions, and scan configuration. Reply |
Codex Review SummaryThis comment shows the latest Codex review activity on this pull request.
ℹ️ About Codex in GitHubYour team has set up Codex to review pull requests in this repo. Reviews are triggered when you
Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings. |
There was a problem hiding this comment.
I reviewed this PR and didn't find any bugs. It's a docs-only change, but both AGENTS.md and CLAUDE.md are explicitly listed in .github/CODEOWNERS under @ ROKT/sdk-engineering, so a human from that team should still sign off.
What was reviewed: the new AGENTS.md confidentiality guidance (public-repo warning, partner/brand/internal-detail redaction rules) and the one-line CLAUDE.md that imports it via @ AGENTS.md — content and import syntax look correct and self-contained, no code or build files touched.
Extended reasoning...
The PR adds two new docs files (AGENTS.md with confidentiality guidance for this public repo, and a one-line CLAUDE.md importing it) with no code, config, or security-sensitive logic touched. Both changed paths are explicitly listed in .github/CODEOWNERS under @ ROKT/sdk-engineering, which per review policy means a CODEOWNER should review even though the change itself is simple and the bug hunter found nothing.
mattbodle
left a comment
There was a problem hiding this comment.
SDK Team PR Review
Docs-only and accurate. The CLAUDE.md import loads AGENTS.md, and neither file is included in the published package.
Adds an AGENTS.md whose first rule is that this repo is public: branch names, commit messages, PR text and assets are permanent, and closing a PR or renaming a branch does not take back what was already pushed. A one-line CLAUDE.md imports it so Claude Code loads it too.
Docs only.
🤖 Generated with Claude Code