From forgeweb-review
This skill should be used when the user asks to 'run pre-push checks', 'check before push', 'what did I forget', 'run quality gate', 'pre-push gate', 'check artifacts', 'check i18n parity', 'check missing tests', or before pushing a feature branch. Analyzes git diff against staging, checks for missing artifacts (pgTAP tests, i18n translations, data-test attributes, schema sync, unit tests, E2E coverage), and optionally dispatches agents to generate fixes.
How this skill is triggered — by the user, by Claude, or both
Slash command
/forgeweb-review:pre-push-gateThe summary Claude sees in its skill listing — used to decide when to auto-load this skill
Analyze the git diff against staging and generate a dynamic checklist of missing artifacts. Optionally dispatch agents to fix findings.
Analyze the git diff against staging and generate a dynamic checklist of missing artifacts. Optionally dispatch agents to fix findings.
Arguments: "$ARGUMENTS"
check: Read-only checklist only (no fix offered)fix: Fix all findingsfix <category>: Fix only the named category (e.g. fix i18n, fix db)Run standard quality checks first. Stop immediately on failure.
pnpm typecheck
On typecheck failure: report errors and STOP. Do not continue to Phase 1.
pnpm lint:fix
pnpm format:fix
After lint:fix and format:fix, check for auto-fixed files via git diff --name-only. Report which files were modified. Continue to Phase 1.
git fetch origin staging
git diff --name-only origin/staging...HEAD
Match each changed file against the trigger matrix. A file can trigger multiple categories.
| Category | ID | File Patterns |
|---|---|---|
| DB & Migrations | db | supabase/migrations/*.sql, schemas/20-corpus-health.sql, database.types.ts |
| i18n | i18n | public/locales/**/*.json; changed .tsx with new <Trans or t( calls |
| UI Completeness | ui | New .tsx in _components/; new page.tsx under (user)/ |
| Server Actions | actions | *-actions.ts; files with 'use server' directive |
| Schemas & Validation | schemas | *.schema.ts |
| Services & Logic | services | _lib/**/*.ts (excl. tests), *-service.ts, *-loader.ts |
| E2E Impact | e2e | Changed pages under (user)/; changed _components/**/*.tsx; changed data-test attrs; deleted routes |
| Edge Functions | ef | supabase/functions/**/*.ts (excl. _shared/) |
Skip categories with zero matching files. Report active categories:
Active categories: db, i18n, e2e (3/8)
Skipped: ui, actions, schemas, services, ef
For fix <category> arguments, validate the category ID and process only that category.
Detect whether TeamCreate is available.
Team Mode (TeamCreate available):
TeamCreate({ team_name: "pre-push-check", description: "Read-only pre-push quality gate checks" })TaskCreate per active categoryAgent with team_name and subagent_type: "feature-dev:code-explorer"Fallback Mode (no TeamCreate):
Agent with run_in_background: true, all subagent_type: "feature-dev:code-explorer"Read-only pre-push quality gate checker for a MakerKit (Next.js App Router + Supabase) project.
CATEGORY: {category_name}
CHANGED FILES IN THIS CATEGORY:
{file_list}
FULL DIFF CONTEXT (all changed files on this branch):
{all_changed_files}
INSTRUCTIONS:
1. Read `.claude/rules/` and `CLAUDE.md` for project conventions
2. Execute each check listed below against the changed files
3. For checks that look for "missing" artifacts, search the codebase to verify presence/absence
YOUR CHECKS:
{checks_for_this_category}
OUTPUT FORMAT — For each check, report exactly:
- check: {check description}
- status: PASS | FAIL | WARN
- details: {what was found or missing, with file paths}
- fix_keywords: {2-3 keywords for ToolSearch to find a fix agent, e.g. "database sql pgTAP"}
IMPORTANT: This is READ-ONLY. Do NOT modify any files.
Load the detailed checks for each category from references/checks-per-category.md. Insert the relevant section as {checks_for_this_category} in the agent prompt.
After all check agents complete, collect fix_keywords from each FAIL/WARN finding.
For each unique set of fix keywords:
ToolSearch(query: "{fix_keywords}", max_results: 3)
Map each finding to the best-matching agent/skill capable of writing code. Fallback: bmad-dev.
Present results as tables per category:
# Pre-Push Gate — Checkliste
Branch: {branch_name} → staging
Geänderte Dateien: {count} | Aktive Kategorien: {active}/{total}
## {Category Name} ({pass_count}/{total_count})
| # | Status | Prüfung | Fix-Agent |
|---|--------|---------|-----------|
| 1 | ✅ | {check description} | — |
| 2 | ❌ | {check description + details} | {recommended agent/skill} |
| 3 | ⚠️ | {check description + details} | {recommended agent/skill} |
Rules:
## {Category} ({n}/{n}) ✅## Übersprungen: {list}Footer:
**Ergebnis: {fail_count} ❌ · {warn_count} ⚠️ · {pass_count} ✅**
For check argument: show checklist and STOP.
Otherwise present options:
Optionen:
(1) Alles fixen — Agents für alle ❌ Items dispatchen
(2) Kategorie wählen — z.B. "fix i18n"
(3) Selbst fixen — Checkliste steht, du fixst manuell
(4) Detail zu einem Item — mehr Kontext anzeigen
Wait for user input before proceeding.
Runs only on user choice (option 1/2) or fix argument.
When two findings modify the same file:
⚠️ Konflikt-Warnung: Items #{a} und #{b} betreffen beide {filename}
→ Empfehlung: sequenziell ausführen
Present conflicts to the Team Lead. They decide parallel vs sequential.
Team Mode: TeamCreate → one task per agent group → one teammate per task.
Fallback: Sequential or parallel Agent calls per Team Lead decision.
Each fix agent receives:
Fix agent for a forgeweb pre-push gate finding (MakerKit/Next.js/Supabase).
CONTEXT:
- Branch: {branch_name}
- Project rules: Read `.claude/rules/` and `CLAUDE.md`
FINDING:
{finding_description}
AFFECTED FILES:
{file_list}
INSTRUCTIONS:
- Follow existing artifacts in the same directory as templates
- Read existing tests/locale files/schemas for patterns
- Write the fixes
- Do NOT commit — the Team Lead reviews and commits
{additional_context_per_category}
Load additional fix context per category from references/checks-per-category.md (bottom section).
After all fix agents complete:
| ID | Category |
|---|---|
db | DB & Migrations |
i18n | i18n |
ui | UI Completeness |
actions | Server Actions |
schemas | Schemas & Validation |
services | Services & Logic |
e2e | E2E Impact |
ef | Edge Functions |
references/checks-per-category.md — Detailed check definitions per category and fix contextnpx claudepluginhub lets-forge/forge-plugins --plugin forgeweb-reviewAudits git code changes across 13 quality dimensions before or after merge, auto-scoping based on branch/commits/uncommitted state or user input, outputs prioritized tasks by file.
Pre-commit quality gate validating logic correctness, error handling, regressions, and completeness in code changes. Auto-fires before commit via cook or on large diffs.
Reviews and verifies code before merge via triage-first checks (up to 16 parallel agents). Pipeline mode verifies vs plans; general mode for PRs/branches/staged changes. Flags findings only.