Repository navigation
fix(skills/reporting-a-skill-gap): let the location block hold several skills - #89
Merged
Merged
Conversation
…han one skill The template said that `## Where in the plugin` holds "exactly these three lines", and the same comment said to repeat the Skill and Section lines when more than one skill is involved. A report about two skills cannot obey both rules. The guidance now asks for one Skill and Section pair for each skill, then one Plugin version line at the end. Fixes #82 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Contributor
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
The focused guidance change resolves the formatting conflict and matches #82’s requirements.
Review effort: Balanced
Findings: None
What changed in this PR
Resolves #82 by clarifying the skill-gap report template’s guidance for gaps involving multiple skills.
Changes:
- Replaces the three-line restriction with one Skill/Section pair per skill.
- Specifies one Plugin version line at the end.
| File | Description |
|---|---|
| skills/reporting-a-skill-gap/templates/skill-gap.md | Updates the guidance comment to define multi-skill location blocks. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
…shape for two skills The title rule named "the skill" in the singular, and `## Reproduction` gave no shape for a gap that spans an agent and the agents it dispatches. Test runs on a two-skill report guessed at both. One wrote a predicate that needed a second run which the scenario did not describe. The title rule now names each skill, joined with "and". The reproduction guidance asks for one scenario for the agent that dispatches, whose predicate checks the dispatch prompts and what the dispatched agents do in the same run. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Fixes #82
A skill-gap report about two skills could not follow its own template. The guidance under
## Where in the pluginsaid "Exactly these three lines" and, in the same comment, told the agent to repeat the Skill and Section lines for each skill. Now it asks for one Skill and Section pair for each skill, then one Plugin version line at the end.The test runs showed two more gaps for a report about two skills, and this PR fixes them too. The title rule named "the skill" in the singular.
## Reproductiongave no shape for a gap that spans an agent and the agents it dispatches, so one run wrote a predicate that needed a second run that the scenario did not describe.Changes
templates/skill-gap.md: the guidance comment under## Where in the plugindefines the shape for any number of skills, and says where the version line goes.templates/skill-gap.md: the title rule names each skill, joined with "and", when more than one skill is involved.templates/skill-gap.md: the## Reproductionguidance says that a gap across a dispatching agent and its dispatched agents gets one scenario for the agent that dispatches. Its Task prompt is the message that agent receives, its Environment facts name the skill that each dispatched agent follows, and its Failure predicate checks the dispatch prompts and what the dispatched agents do, in the same run.Testing
Tested with isolated subagents and the issue's two-skill task.
skills/says "three lines".🤖 Generated with Claude Code