Skip to content

Improve vmr-codeflow-status SKILL.md structure#124150

Merged
lewing merged 5 commits intomainfrom
lewing/vmr-skill-improvements
Feb 9, 2026
Merged

Improve vmr-codeflow-status SKILL.md structure#124150
lewing merged 5 commits intomainfrom
lewing/vmr-skill-improvements

Conversation

@lewing
Copy link
Member

@lewing lewing commented Feb 8, 2026

Improves the SKILL.md for the vmr-codeflow-status skill based on patterns from the skill-builder methodology:

Changes:

  • Critical rules — No PR approval/blocking (--approve/--request-changes), no branch switching (git checkout/gh pr checkout)
  • Expanded trigger description — Added flow status, missing backflow, blocked keywords that match common user queries
  • Two Modes section — Decision table clarifying PR analysis vs flow health (-CheckMissing) as separate modes
  • Inline anti-pattern — Warning not to combine -PRNumber with -CheckMissing
  • Split 'What the Script Does' — Separated into PR Analysis Mode (8 steps) and Flow Health Mode (4 steps) instead of a mixed 11-item list
  • Inline warningsUnknown health ≠ healthy, aka.ms 301 vs 302 redirect behavior

No script changes — SKILL.md documentation improvements only.

cc @dotnet/runtime-infrastructure

- Expand trigger description with flow status and missing backflow keywords
- Add Two Modes section with decision table (PR analysis vs flow health)
- Add inline anti-pattern warning about -PRNumber vs -CheckMissing
- Split 'What the Script Does' into PR Analysis vs Flow Health modes
- Add inline warnings: Unknown health, aka.ms redirect behavior
- Add net11 trigger phrase to When to Use examples
Copilot AI review requested due to automatic review settings February 8, 2026 20:22
@github-actions github-actions bot added the needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners label Feb 8, 2026
@lewing lewing added area-skills Agent Skills and removed needs-area-label An area label is needed to ensure this gets routed to the appropriate area owners labels Feb 8, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Documentation-only update to the vmr-codeflow-status skill guide, aiming to make it easier for users to choose the correct invocation mode and interpret results when diagnosing VMR backflow/forward-flow health.

Changes:

  • Expanded “When to use”/trigger language to better match common user queries (flow status, missing backflow, blocked).
  • Added a “Two Modes” section with a decision table and an explicit anti-pattern warning.
  • Reorganized “What the Script Does” into separate PR Analysis vs Flow Health (-CheckMissing) sections, with added interpretation warnings.

@lewing lewing requested a review from steveisok February 8, 2026 21:21
Copilot AI review requested due to automatic review settings February 8, 2026 21:55
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 3 comments.

@lewing lewing merged commit 500d83e into main Feb 9, 2026
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-skills Agent Skills

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants