docs: cite contribution-guidelines and directory-coverage refs#47
Conversation
…le analysis Cite ai-contribution-guidelines.md (3 Cs framework) and directory-coverage.md (scoped AGENTS.md coverage rationale). Remove references/analysis.md - dated cross-project survey, no longer load-bearing. Signed-off-by: Sebastian Mendel <github@sebastianmendel.de>
CybotTM
force-pushed
the
docs/cite-contribution-and-coverage-refs
branch
from
bc2f9e7 to
de7bde5
Compare
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request updates the SKILL.md file by adding references to AI contribution guidelines and directory coverage rationale, while also removing a large analysis document concerning AGENTS.md patterns across projects. A review comment identified a discrepancy in the description of the "3 Cs" framework, noting that it should be updated to "Comprehension, Context, Continuity" to align with the referenced documentation.
There was a problem hiding this comment.
Pull request overview
This PR updates the skill’s discovery surface by ensuring key reference documents are reachable from skills/agent-rules/SKILL.md, and removes an obsolete reference document that is no longer used.
Changes:
- Add SKILL.md reference links to
references/ai-contribution-guidelines.md(“3 Cs” framework) andreferences/directory-coverage.md(scoped coverage rationale). - Remove
references/analysis.md(confirmed no longer referenced anywhere in the repo).
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| skills/agent-rules/SKILL.md | Adds two missing reference links so existing guidance documents are discoverable from the skill entrypoint. |
| skills/agent-rules/references/analysis.md | Deletes an unused, dated reference document (no remaining repo references found). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: Sebastian Mendel <github@sebastianmendel.de>
CybotTM
force-pushed
the
docs/cite-contribution-and-coverage-refs
branch
from
69d3d84 to
fd93295
Compare
Summary
references/ai-contribution-guidelines.md(3 Cs framework) from SKILL.md.references/directory-coverage.md(scoped AGENTS.md coverage rationale) from SKILL.md.references/analysis.md— dated cross-project survey, no longer load-bearing.Why
Both reference files existed but were not cited by SKILL.md, so they were unreachable from the skill's discovery surface. The deleted
analysis.mdwas a one-off survey artifact whose findings have been superseded by the current SKILL.md content.Test plan
git statusclean after commitreferences/*.mdfiles all cited from SKILL.md