docs(agents): focus the guide on working in the repo - #88
Conversation
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.
Copilot review overview
🟢 Approval recommended
It is a documentation-only change whose scripts, command classifications, and release/CI claims were all verified accurate against the codebase.
Review effort: Balanced
Findings: None
What changed in this PR
This PR reshapes the AGENTS.md agent guide to focus on concrete, repo-specific consequences of actions rather than abstract process rules. It replaces an "ask-first / done definition" pair of lines with a new "Ways To Hurt Yourself" section that names what a merge publishes and which commands touch a shared Roku, and adds explicit proof/delivery guidance. All referenced scripts, commands, and release wiring were verified to exist in the codebase.
Changes:
- Adds a "Ways To Hurt Yourself" section explaining that releasable commits on
mainpublish to npm (versions can't be reused) and which commands mutate the device vs. only read. - Adds a "Which proof a change needs" map (docs →
check/skills:lint; source →verify; device-facing →live:smoke/live:probe) and a new "Delivery" section describing CI and semantic-release. - Removes the prior "without pausing" and "ask before publishing/live runs" lines and the blanket "Done means…" definition.
| File | Description |
|---|---|
| AGENTS.md | Replaces ask-first/done-definition guidance with a consequences-focused "Ways To Hurt Yourself" section, a per-change proof map, and a "Delivery" section; all claims verified accurate against scripts, CLI metadata, and CI/release config. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
The agent guide now names what a merge publishes and which commands touch a shared Roku, instead of an ask-first rule and a blanket done definition.
mainpublish to npm (version numbers can't be reused), and which commands replace the sideloaded developer app or change the screen versus only reading.vp run check(plusskills:lintforskills/) with no device run; source changes runverify; device-facing changes addlive:smokeorlive:probe.main, semantic-release to npm, linked to Distribution.vp run checkpasses.Written by an agent (Claude Code, Opus 5.5)