Move assertion-quality, test-gap-analysis, test-smell-detection, and
test-tagging from dotnet-experimental to dotnet-test. These skills have
demonstrated sufficient eval coverage, unique value beyond baseline model
capabilities, and are already integrated into the test-quality-auditor
agent pipeline.
Changes:
- Move skills and tests, dropping the exp- prefix
- Update all cross-references in skill descriptions and routing tables
- Update test-quality-auditor agent and eval scenarios
- Update CODEOWNERS for the moved paths
- Delete duplicate exp-dotnet-test-frameworks (identical to dotnet-test-frameworks)
- Fix stale exp-dotnet-test-frameworks reference in exp-test-maintainability
Skills held back for further iteration:
- exp-mock-usage-analysis: baseline model scores 5/5 on key scenarios
- exp-test-maintainability: quality gains offset by token overhead
* Fix dotnet-test-frameworks skill activation for evaluations
The skill had user-invocable: false and a description saying 'DO NOT USE
directly', which prevented the model from activating it during evaluation.
All 6 eval scenarios default to expect_activation: true, causing
SkillNotActivated failures on CI.
- Remove user-invocable: false from both dotnet-test and experimental copies
- Rewrite description with activation keywords matching eval prompts
(assertion mapping, framework identification, skip annotations, exception
assertions, lifecycle conversion, integration test markers)
- Preserve note that other analysis skills also load this as reference data
* Restore user-invocable: false, fix cross-refs, add expect_activation: false
Address review comment: remove exp-* skill references from dotnet-test-
frameworks description since those skills reference exp-dotnet-test-
frameworks instead.
Optimize eval results: this reference data skill's content is knowledge
the model already has (baseline scores 5.0/5 on most scenarios). Direct
activation adds token/time overhead without quality improvement:
- Restore user-invocable: false (correct for reference data skill)
- Shorten description to not trigger activation
- Add expect_activation: false to all 6 scenarios
The skill's value is realized when loaded by consumer skills (e.g.
test-anti-patterns), not via direct user queries.
* Deduplicate test skill references and clarify skill boundaries
- Move platform-detection.md and filter-syntax.md to plugins/dotnet-test/shared/,
removing 3 identical copies of each from run-tests, mtp-hot-reload, and
migrate-vstest-to-mtp reference directories.
- Move dotnet.md from exp-test-smell-detection/extensions/ to shared/ as
dotnet-test-frameworks.md in both dotnet-test and dotnet-experimental plugins.
Update exp-assertion-quality, exp-test-boilerplate-detection, exp-test-tagging,
and test-anti-patterns to reference the shared file instead of inlining
framework detection tables.
- Differentiate test-anti-patterns (quick pragmatic review) from
exp-test-smell-detection (deep formal audit with academic taxonomy) by updating
descriptions and cross-referencing each other in When Not to Use sections.
- Update skill-validator to allow ../../shared/ file references while still
blocking other parent-directory traversals. Add tests for the new rule.
* Switch from shared/ directories to hidden reference skills
Replace the plugin-level shared/ directories with non-invocable reference
skills (user-invocable: false) that other skills reference by name.
- Create platform-detection, filter-syntax, and dotnet-test-frameworks as
hidden skills under plugins/dotnet-test/skills/. These contain the
detection tables and syntax references previously duplicated across
run-tests, mtp-hot-reload, and migrate-vstest-to-mtp.
- Create exp-dotnet-test-frameworks as a hidden skill under
plugins/dotnet-experimental/skills/ for the experimental test analysis
skills (exp-test-smell-detection, exp-assertion-quality, etc.).
- Update all consuming skills to reference these by skill name in backtick
notation instead of file links.
- Revert the skill-validator ../../shared/ exception — no longer needed
since all references now use the standard skill name mechanism.
* Merge exp-test-boilerplate-detection into exp-test-maintainability
exp-test-maintainability was only 6 calibration rules with no workflow.
exp-test-boilerplate-detection had the full 5-category detection workflow,
examples, calibration, and validation. Merge the boilerplate content into
exp-test-maintainability (the broader, more user-facing name) and add the
two unique maintainability rules (DisplayName guidance, DataRow vs
DynamicData preference) to Category 3.
- Replace exp-test-maintainability SKILL.md with the merged content
- Move test fixtures from exp-test-boilerplate-detection to exp-test-maintainability
- Merge eval.yaml scenarios (4 total: 2 from each original skill)
- Delete exp-test-boilerplate-detection skill and tests
- Update all cross-references in exp-test-smell-detection, dotnet-test-frameworks,
exp-dotnet-test-frameworks, and CODEOWNERS
* Add cross-references to test-anti-patterns for deep mock and duplication analysis
Point users to exp-mock-usage-analysis from the Over-mocking entry and
to exp-test-maintainability from the Duplicate tests entry.
* Add exp-dotnet-test-frameworks to CODEOWNERS
* Improve run-tests SDK 10 MTP detection for blame-hang scenario
Inline the critical SDK 10 detection signal (global.json test.runner)
directly in Step 1 instead of deferring entirely to the platform-detection
skill. This makes the distinction between SDK 10 (no -- separator) and
SDK 8/9 (requires -- separator) more prominent.
Add a quick detection summary table, strengthen the Common Pitfalls entry
for SDK 10 with a blame-hang-timeout example, and keep the
platform-detection skill reference for the full detection logic.
* Improve skill activation keywords in descriptions
- exp-test-maintainability: Add 'suggest a better test structure',
'consolidate similar test methods', 'convert copy-paste tests to
data-driven parameterized tests' to match prompts like 'each new case
needs a whole new method, suggest a better structure'.
- test-anti-patterns: Add 'review tests', 'find test problems', 'check
test quality', 'audit tests for common mistakes' to match review-style
prompts that don't use the word 'anti-pattern'.
- run-tests: Add 'hang timeout', 'blame-hang', 'blame-crash', 'TUnit'
to match SDK 10 blame scenarios and TUnit filter scenarios that were
intermittently not activating.
* Add exp-mock-usage-analysis skill and evaluation
New experimental skill that detects unused, redundant, or unnecessary
mocks in .NET test suites. Covers Moq, NSubstitute, FakeItEasy, and
manual test doubles.
- SKILL.md with 6-step workflow: gather code, classify mocks, detect
anti-patterns, incorporate runtime data, generate recommendations,
report findings
- eval.yaml with 10 scenarios: trivial type mocking, unused setups,
redundant configs, stable utility mocks, well-placed mocks (no false
positives), NSubstitute support, FakeItEasy support, setup sprawl,
and 2 non-activation boundary tests
- Fixture files for each scenario with production + test code
- CODEOWNERS entry for @dotnet/dotnet-testing
* Remove no-headroom scenarios, increase NSubstitute timeout
Remove 4 scenarios where baseline already scores 5.0/5 (model handles
these well without skill guidance):
- Detect mocking of DTOs, records, and enums
- Recognize well-placed mocks without inventing false positives
- Decline when asked about non-mock test anti-patterns
- Decline mock framework migration request
Increase NSubstitute scenario timeout from 120s to 180s to avoid
baseline timeouts impacting scoring.
Remaining 6 scenarios all show positive skill impact (3→5, 3→4, etc).
* Document areas the model handles well without the skill
Add a subsection under 'When Not to Use' listing mock-related tasks
where the base model already scores 5.0/5 without skill guidance:
- Mocking trivial types (DTOs, records, enums)
- Recognizing well-placed mocks (no false positives)
* Fix md
* Improve
* Add exp-test-smell-detection skill and evaluation
- Language-agnostic core skill detecting 10 test smells (conditional logic,
mystery guest, sleepy test, assertion-free, eager test, magic numbers,
sensitive equality, exception handling, general fixture, ignored tests)
- .NET extension with framework-specific detection tables (MSTest/xUnit/NUnit/TUnit)
- Reference catalog with 19 smells from testsmells.org research
- Evaluation with 4 scenarios: smelly suite, clean suite, integration tests, non-activation
- CODEOWNERS entry for @dotnet/dotnet-testing
* Improve integration test fixture and non-activation timeout
- Add real smells to integration test fixture (Thread.Sleep, conditional
logic, assertion-free test) mixed with legitimate integration patterns
- Update rubric to test nuanced distinction between real smells and
acceptable integration test patterns
- Increase non-activation scenario timeout from 120s to 300s to prevent
timeout-related scoring penalties
* Fix test projects
* Add domains for testsmells
* Remove extra domains
New experimental skill that performs pseudo-mutation analysis on .NET
production code to find gaps in existing test suites. Covers boundary,
boolean/logic, return value, exception removal, arithmetic, and
null-check mutation categories.
Includes eval.yaml with 4 scenarios (boundary gaps, logic gaps,
well-tested code recognition, non-activation) and fixture projects.
Two new experimental skills for dotnet-experimental plugin:
- exp-test-boilerplate-detection: Detects duplicate boilerplate patterns
across .NET test suites and identifies refactoring opportunities.
Categories: repeated construction, assertion patterns, copy-paste
methods, duplicated setup/teardown, repeated infrastructure.
- exp-assertion-quality: Analyzes assertion variety and depth across
.NET test suites. Produces metrics dashboard including assertion
count, type spread, trivial %, assertion-free tests, and negative
assertion coverage.
Both skills include eval.yaml with 3-4 scenarios each (positive,
negative, and non-activation) plus test fixtures.
New experimental skill that assesses test maintainability: duplication,
test size, data-driven patterns, display names, builder/helper extraction,
and shared setup.
Amaury Levé