skills.rest
ohmyhotelco
ohmyhotelcoOFFICIAL
Explore all skills by this author →
GitHub

fm-test-review

Audits generated Vitest and Playwright migration tests for quality, coverage, and anti-patterns.

1|Updated Feb 19, 2026
Search Tags:#vitest#migration#test quality#code audit#playwright#testing library#test review
One-click install
npx skills add https://github.com/ohmyhotelco/hare-cc-plugins --skill fm-test-review-ohmyhotelco
Or copy as Structured Prompt for Agent▼
Please help me install this Agent Skill.
Skill: fm-test-review
Source: https://github.com/ohmyhotelco/hare-cc-plugins/tree/main/frontend-migration-plugin/skills/fm-test-review
Command: npx skills add https://github.com/ohmyhotelco/hare-cc-plugins --skill fm-test-review-ohmyhotelco

SYSTEM DOCUMENTATION & REQUIREMENTS

What problem does it solve? Generated migration tests often contain weak assertions, flaky async handling, and Testing Library misuse that pass silently until they fail in CI. This Skill runs a standalone quality audit on Vitest and Playwright tests so problems surface before they erode trust in the test suite. ## Core Features & Use Cases - Multi-dimension test audit: Reviews assertions, Testing Library usage, async handling, coverage, timing/flakiness, and anti-patterns via a dedicated test-reviewer agent. - Standalone operation: Runs independently of the migration pipeline with no progress tracking, no lock, and no state mutation. - Flexible targeting: Accepts an explicit test path or defaults to the current app's Vitest and Playwright e2e directories, with per-app config resolution (pc, mobile, hana). - Use Case: After generating migration tests for the mobile app, run the audit to get a scored report with file:line findings and concrete fixes before merging. ## Quick Start Ask the assistant to review the generated tests for the mobile app with fm-test-review and report the top quality issues.

Frequently Asked Questions about fm-test-review

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

FAQPage Schema
How do I audit generated Vitest and Playwright tests for quality?▼

Run the test review with an optional test path or let it default to the current app's Vitest and Playwright e2e directories. It launches a test-reviewer agent that scores assertions, async handling, coverage, and anti-patterns, then reports findings with file:line references.

What test quality issues does a migration test review check?▼

The review checks assertion strength, Testing Library usage, async handling, coverage gaps, timing and flakiness risks, and common test anti-patterns. Each finding includes the file and line plus a suggested fix.

Can I review tests for a specific app like mobile or hana?▼

Yes, pass the --app flag with pc, mobile, or hana. The app entry must exist in the config with the required keys; an unconfigured app stops with a clear message instead of failing inside the agent.

Does the test review modify my tests or pipeline state?▼

No. The review is read-only: it never changes tests, takes the pipeline lock, or mutates progress tracking. It only reads the tests and writes a report in the configured working language.

What happens if the config file is missing when running a test review?▼

The skill stops and instructs you to run fm-init first. Config presence alone is not enough; the target app entry must also exist with the keys this stage reads.

Related Skills

Explore More Software Engineering Skills→
hkcm91hkcm91

testing-widgets

Test StickerNest widgets with Playwright E2E and Vitest unit tests.

Community
Intermediate
notquenotque
415

testing-preferred-patterns

Detect flaky, brittle, and skipped test patterns in test suites.

Community
Advanced
JamieMasonJamieMason
2.1k

testing-gaps

Identify uncovered branches and missing test scenarios from cargo llvm-cov coverage output.

Community
Intermediate
Obsidian-OwlObsidian-Owl
9

gate-tests

Audit automated test quality for assertion strength, boundary coverage, mock discipline, and error paths.

Official
Advanced