Prune skill authoring guidance - #281
Conversation
ariobarin
left a comment
ariobarin
left a comment
There was a problem hiding this comment.
Current-head review at 2ded4b4bfb2554964fbd09f1a10cc87a9d69fe6e: no blocking findings. The authoring guidance remains complete on role, surface selection, wiring, provenance, retirement, validation, and authority while removing duplicated routing/checklist material. portable checks run 1058 completed successfully, with no existing review comments or unresolved threads. Ready for merge under the user's explicit authority.
![chatgpt-codex-connector[bot]](https://avatars.githubusercontent.com/in/1144995?s=60&v=4){kind=small}
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 2ded4b4bfb
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| - role; | ||
| - desired terminal result; | ||
| - why the role exists at runtime; | ||
| - recurring failure it corrects; | ||
| - evidence that matters; | ||
| - desired result; | ||
| - recurring failure the role corrects; | ||
| - evidence that proves success; | ||
| - authority boundary. |
There was a problem hiding this comment.
This shortened first-screen checklist drops the required explanation of why the role exists at runtime. When this skill is used to author or review new SKILL.md files, it can now treat a contract as complete even though the runtime rationale is missing, which weakens trigger judgment and diverges from the repository's skill-authoring contract; restore the omitted why item or equivalent rationale requirement here.
AGENTS.md reference: AGENTS.md:L73-L80
Useful? React with 👍 / 👎.
1 participant
The two authoring skills repeated routing and checklist material that already belongs to Compass workflows. This keeps the runtime contract complete while removing 12 net lines and making every paragraph justify its context cost.