Plugin/protocol improvements#380
Merged
Merged
Conversation
SKILL.md: - Require imperative verb in `action` field - Add pattern-field guidance: omit if it just rephrases the summary - Ban project/codebase names from verification timestamps - Add near-duplicate check before calling propose reflect.md: - Drop estimated_relevance from candidate fields and all templates - Exclude already-proposed insights from reflect candidates - Inline VIBE√ dimensions (V/I/B/E) so Step 2.5 is self-contained - Add query-before-propose step in Step 5 to catch near-duplicates
…y trigger SKILL.md: - Replace prose frontmatter with INVOKE WHEN / SKIP WHEN structure; single safe SKIP WHEN condition (already queried this topic this session) replaces the Dunning-Kruger "tasks you already know" bullet - Mid-task propose trigger now requires user approval before calling `propose` — prevents unconfirmed auto-proposes - Add field-role table clarifying domains/languages/frameworks/pattern: use the most specific field, no duplication across fields, avoid meta-labels (gotchas, tips) in domains - Add POSIX/shell coverage rule: include both a generic tag and the specific one where the issue was observed - Fix GitHub Actions/Rust example: rust moved from domains to languages reflect.md: - Mark candidates that resolved earlier-session errors with ⏱ and surface count in Step 6 summary (missed mid-task propose visibility)
Contributor
There was a problem hiding this comment.
Pull request overview
This PR refines the cq “propose immediately” workflow and clarifies how agents should classify/tag knowledge units, with corresponding updates to the /cq:reflect command guidance.
Changes:
- Rewrites the cq skill frontmatter guidance (invoke/skip conditions) and strengthens mid-task propose expectations (incl. near-duplicate check).
- Tightens domain/language/framework/pattern field guidance to avoid cross-field duplication and updates examples accordingly.
- Updates
/cq:reflectinstructions to exclude already-proposed insights, remove the relevance field from presentation, and add a “query-before-propose” step.
Reviewed changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
| plugins/cq/skills/cq/SKILL.md | Updates skill frontmatter protocol guidance and adds stricter tagging/field guidance for query/propose. |
| plugins/cq/commands/reflect.md | Adjusts reflection workflow/content rules, presentation templates, and proposal flow (query before propose). |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Address PR #380 review feedback: - SKILL.md frontmatter used "stabilises" (British) while the protocol body used "stabilizes" (American); standardize on US English per the project convention. - reflect.md Step 2 introduced the ⏱ marker for candidates that resolved earlier-session errors, but the Step 3 templates and Step 6 summary template had no slot for it. Add a metadata-block line mirroring the existing⚠️ pattern (first line, above⚠️ when both apply) and a conditional Step 6 summary line that follows the same omit-when-zero rule as the {excluded} line. SDK copies regenerated via make sync-prompts.
This was referenced May 20, 2026
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.
Strengthen the propose-immediately protocol and tighten field guidance.