testing-review

Analyze code diffs to identify missing regression coverage and brittle test patterns.

3|Updated May 12, 2026
One-click install
npx skills add https://github.com/gao-hongnan/omniagents --skill testing-review-gao-hongnan
Or copy as Structured Prompt for Agent
Please help me install this Agent Skill.
Skill: testing-review
Source: https://github.com/gao-hongnan/omniagents/tree/main/plugins/reviewer/skills/testing-review
Command: npx skills add https://github.com/gao-hongnan/omniagents --skill testing-review-gao-hongnan

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve?

This skill addresses the common issue of superficial test coverage, where tests exist but fail to catch actual regressions or verify critical edge cases.

Core Features & Use Cases

  • Regression Pinning: Ensures bug fixes are protected by tests that would have failed prior to the fix.
  • Branch & Contract Validation: Verifies that new logic branches and public API changes are explicitly covered by tests.
  • Mock & Flake Analysis: Identifies brittle mocks that mask real interface changes and detects non-deterministic test triggers like improper async handling or shared state.

Quick Start

Run the testing-review skill on the current diff to identify gaps in regression coverage and potential test fragility.

Frequently Asked Questions about testing-review

High-intent search queries and answers about installing and using this skill.

FAQPage Schema
How do I check if my unit tests actually catch regressions instead of just hitting coverage targets?

Test adequacy is verified by analyzing code diffs to identify missing regression coverage, ensuring bug fixes are pinned by tests that would have failed prior to the fix. It evaluates branch coverage and validates that new logic branches are explicitly tested.

Why does my test suite pass but still allow production bugs to slip through?

Superficial test coverage often stems from weak assertions and brittle test patterns. Evaluating test effectiveness against production diffs identifies missing branch coverage, mock boundary integrity issues, and nondeterministic flake triggers that mask real interface changes.

How do I detect brittle mocks that hide real API interface changes in my code review?

Mock boundary integrity is analyzed during code review to identify brittle mocks that mask real interface changes. The evaluation checks if tests properly validate public API contract changes and new logic branches rather than relying on shallow mock implementations.

What is the best way to find nondeterministic flake triggers in async test suites?

Nondeterministic flake triggers are detected by analyzing test patterns for improper async handling and shared state. The review process validates that test suites provide meaningful failure signals rather than passing inconsistently due to hidden state dependencies.

Can I use automated test review to validate branch coverage for new feature implementations?

Automated test review validates branch and contract coverage by evaluating the effectiveness of new tests against production diffs. It explicitly checks that new logic branches and public API changes are covered, ensuring meaningful failure signals for feature implementations.

When should I audit my test suite for weak assertions and superficial coverage?

You should audit tests when adding bug fixes or new features to ensure they provide meaningful failure signals. The audit identifies weak assertions, missing regression pinning, and brittle patterns that fail to verify critical edge cases against actual production diffs.