Skip to content

Comments

Improve jit-regression-test skill with structured description and stop signals#124734

Merged
lewing merged 1 commit intodotnet:mainfrom
lewing:improve-jit-regression-test-skill
Feb 23, 2026
Merged

Improve jit-regression-test skill with structured description and stop signals#124734
lewing merged 1 commit intodotnet:mainfrom
lewing:improve-jit-regression-test-skill

Conversation

@lewing
Copy link
Member

@lewing lewing commented Feb 23, 2026

Summary

Improve the jit-regression-test skill with structured description, stop signals, and consolidated tips.

Changes

  1. Structured description — Added USE FOR / DO NOT USE FOR routing keywords for better skill activation. Trigger phrases include: creating JIT regression tests, extracting repro code, "write a test for this JIT bug", "create a regression test for issue #NNNNN". Anti-triggers: non-JIT tests, debugging without repro, performance benchmarks.

  2. Stop signal blockquote — Added explicit guidance on when NOT to create a test (no reproducible code, duplicate of existing test, bug is in libraries not JIT). This prevents the skill from being misapplied on invalid inputs.

  3. Consolidated tips — Reduced from 4 verbose items to 2 concise items. Removed tips that duplicate the conventions section (NoInlining, minimize repro).

Validation

Multi-model eval results (5 questions testing trigger routing, stop compliance, structural output, convention compliance, negative routing):

Model Before After
Claude Sonnet 25/25 25/25
Claude Haiku 22/25 25/25

Key improvement: Haiku stop-signal compliance went from 2/5 to 5/5. The blockquote pattern causes weaker models to correctly refuse when stop conditions are met, rather than inferring from general knowledge.

Context

These patterns (structured descriptions, blockquote stop signals) are documented in the Copilot Skills Guidance and have been validated across multiple skills (ci-analysis, flow-analysis, flow-tracing).

Copilot AI review requested due to automatic review settings February 23, 2026 00:06
@github-actions github-actions bot added the area-skills Agent Skills label Feb 23, 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

This PR improves the jit-regression-test skill documentation with better structured descriptions, explicit stop signals, and consolidated tips to enhance skill activation accuracy and prevent misapplication.

Changes:

  • Added USE FOR / DO NOT USE FOR routing keywords to the skill description for better activation matching
  • Added blockquote stop signal (line 15) specifying when NOT to create tests (no repro, duplicate, non-JIT bugs)
  • Consolidated tips from 4 verbose items to 2 concise items, removing duplicates of conventions already covered

@lewing lewing marked this pull request as ready for review February 23, 2026 00:17
@lewing lewing merged commit b1f031d into dotnet:main Feb 23, 2026
24 checks passed
@EgorBo
Copy link
Member

EgorBo commented Feb 23, 2026

If it continues to kick in too often when not asked, let us know, we can consider removing it

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