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
Copy Markdown
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
Copy Markdown
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.

Comment thread .github/skills/vmr-codeflow-status/SKILL.md Outdated
Comment thread .github/skills/vmr-codeflow-status/SKILL.md Outdated
Comment thread .github/skills/vmr-codeflow-status/SKILL.md Outdated
@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
Copy Markdown
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.

Comment thread .github/skills/vmr-codeflow-status/SKILL.md Outdated
Comment thread .github/skills/vmr-codeflow-status/SKILL.md Outdated
Comment thread .github/skills/vmr-codeflow-status/SKILL.md Outdated
@lewing lewing merged commit 500d83e into main Feb 9, 2026
17 checks passed
@jkotas jkotas deleted the lewing/vmr-skill-improvements branch March 1, 2026 18:36
iremyux pushed a commit to iremyux/dotnet-runtime that referenced this pull request Mar 2, 2026
@github-actions github-actions Bot locked and limited conversation to collaborators Apr 1, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

area-skills Agent Skills

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants