fix: simplify quality change review

Author: shinpr

agents/quality-fixer-frontend.md

@@ -37,11 +37,9 @@ Use the appropriate run command based on the `packageManager` field in package.j
 
 ### Step 1: Incomplete Implementation Check [BLOCKING — before any quality checks]
 
-Review the diff of changed files to detect stub or incomplete implementations. This step runs before any quality checks because verifying the quality of unfinished code is meaningless.
+Review the current uncommitted changes for incomplete implementation using the current task and repository context. This step runs before any quality checks because verifying the quality of unfinished code is meaningless.
 
-Derive the current uncommitted write set from `git status --short`. Inspect tracked changes with `git diff HEAD`, read untracked files directly, and account for deleted paths without attempting to read them. The workflow commits each completed unit before the next begins, so this repository state is the quality scope.
-
-Apply the indicators below to each existing file in this write set.
+Use the indicators below for this review.
 
 **Indicators of incomplete implementation** (stub_detected):
 - `// TODO`, `// FIXME`, `// HACK`, `throw new Error("not implemented")` or equivalent

agents/quality-fixer.md

@@ -34,11 +34,9 @@ Executes applicable quality checks, fixes in-scope failures, and reports blocker
 
 ### Step 1: Incomplete Implementation Check [BLOCKING — before any quality checks]
 
-Review the diff of changed files to detect stub or incomplete implementations. This step runs before any quality checks because verifying the quality of unfinished code is meaningless.
+Review the current uncommitted changes for incomplete implementation using the current task and repository context. This step runs before any quality checks because verifying the quality of unfinished code is meaningless.
 
-Derive the current uncommitted write set from `git status --short`. Inspect tracked changes with `git diff HEAD`, read untracked files directly, and account for deleted paths without attempting to read them. The workflow commits each completed unit before the next begins, so this repository state is the quality scope.
-
-Apply the indicators below to each existing file in this write set.
+Use the indicators below for this review.
 
 **Indicators of incomplete implementation** (stub_detected):
 - `// TODO`, `// FIXME`, `// HACK`, `throw new Error("not implemented")` or equivalent

dev-workflows-frontend/agents/quality-fixer-frontend.md

@@ -37,11 +37,9 @@ Use the appropriate run command based on the `packageManager` field in package.j
 
 ### Step 1: Incomplete Implementation Check [BLOCKING — before any quality checks]
 
-Review the diff of changed files to detect stub or incomplete implementations. This step runs before any quality checks because verifying the quality of unfinished code is meaningless.
+Review the current uncommitted changes for incomplete implementation using the current task and repository context. This step runs before any quality checks because verifying the quality of unfinished code is meaningless.
 
-Derive the current uncommitted write set from `git status --short`. Inspect tracked changes with `git diff HEAD`, read untracked files directly, and account for deleted paths without attempting to read them. The workflow commits each completed unit before the next begins, so this repository state is the quality scope.
-
-Apply the indicators below to each existing file in this write set.
+Use the indicators below for this review.
 
 **Indicators of incomplete implementation** (stub_detected):
 - `// TODO`, `// FIXME`, `// HACK`, `throw new Error("not implemented")` or equivalent

dev-workflows-frontend/skills/recipe-front-adjust/SKILL.md

@@ -115,7 +115,6 @@ When the project-tier file declares no automated verification mechanism for an a
   - `subagent_type: "dev-workflows-frontend:quality-fixer-frontend"`
   - `description: "Quality verification for adjustment unit"`
   - Pass `qualityCommand` when available (caller first, otherwise current task).
-  - The agent derives the adjustment unit's uncommitted write set from repository status.
 - Route the quality-fixer-frontend response by `status`:
   - `approved` → proceed to Step 7
   - `stub_detected` → return to Step 5 to complete the implementation for this unit, then re-invoke quality-fixer-frontend

dev-workflows-frontend/skills/recipe-front-build/SKILL.md

@@ -100,7 +100,7 @@ For EACH task in the Consumed Task Set, YOU MUST:
      - `blocked` → STOP and escalate to user
      - `needs_revision` → Pass `qualityIssues` unchanged into the Review Resolution Gate; return to step 1 for rerouted corrections and derive convergence from correction re-review `prior_feedback_reconciliation`
    - `status: completed` → Proceed to step 3
-3. **QUALITY-FIX**: Invoke quality-fixer-frontend with `task_file`, upstream `mutationEvidence`, and `qualityCommand` when available (caller first, otherwise current task); it derives the uncommitted write set from repository status
+3. **QUALITY-FIX**: Invoke quality-fixer-frontend with `task_file`, upstream `mutationEvidence`, and `qualityCommand` when available (caller first, otherwise current task)
    - `stub_detected` → Return to step 1 with `incompleteImplementations[]` details
    - `blocked` → STOP and escalate to user
    - `approved` → Proceed to step 4

dev-workflows-frontend/skills/recipe-front-review/SKILL.md

@@ -142,7 +142,7 @@ Invoke task-executor-frontend using Agent tool:
 Invoke quality-fixer-frontend using Agent tool:
 - `subagent_type`: "dev-workflows-frontend:quality-fixer-frontend"
 - `description`: "Quality gate check"
-- Pass Step 6 `mutationEvidence`; quality-fixer-frontend derives the uncommitted correction write set from repository status.
+- Pass Step 6 `mutationEvidence`.
 - `prompt`: "Confirm quality gate passage for fixed files."
 
 ### Step 8: Re-validate code-reviewer

dev-workflows-frontend/skills/subagents-orchestration-guide/SKILL.md

@@ -197,7 +197,7 @@ For Small, execute one direct-scope 4-step cycle and complete when quality-fixer
      - `blocked` → Escalate to user
      - `needs_revision` → Pass `qualityIssues` objects unchanged into Review Resolution. On correction re-review, derive the next transition only from `prior_feedback_reconciliation`; return to step 1 for rerouted corrections and proceed to step 3 only at convergence
    - Otherwise → Proceed to step 3
-3. **Quality-fix**: invoke quality-fixer with upstream `mutationEvidence`, plus `task_file` when available and `qualityCommand` from the caller first or task otherwise; quality-fixer derives the current uncommitted write set from repository status
+3. **Quality-fix**: invoke quality-fixer with upstream `mutationEvidence`, plus `task_file` when available and `qualityCommand` from the caller first or task otherwise
    - `stub_detected` → Return to step 1 with `incompleteImplementations[]` details
    - `blocked` → Escalate to user
    - `approved` → Proceed to step 4

dev-workflows-fullstack/agents/quality-fixer-frontend.md

@@ -37,11 +37,9 @@ Use the appropriate run command based on the `packageManager` field in package.j
 
 ### Step 1: Incomplete Implementation Check [BLOCKING — before any quality checks]
 
-Review the diff of changed files to detect stub or incomplete implementations. This step runs before any quality checks because verifying the quality of unfinished code is meaningless.
+Review the current uncommitted changes for incomplete implementation using the current task and repository context. This step runs before any quality checks because verifying the quality of unfinished code is meaningless.
 
-Derive the current uncommitted write set from `git status --short`. Inspect tracked changes with `git diff HEAD`, read untracked files directly, and account for deleted paths without attempting to read them. The workflow commits each completed unit before the next begins, so this repository state is the quality scope.
-
-Apply the indicators below to each existing file in this write set.
+Use the indicators below for this review.
 
 **Indicators of incomplete implementation** (stub_detected):
 - `// TODO`, `// FIXME`, `// HACK`, `throw new Error("not implemented")` or equivalent

dev-workflows-fullstack/agents/quality-fixer.md

@@ -34,11 +34,9 @@ Executes applicable quality checks, fixes in-scope failures, and reports blocker
 
 ### Step 1: Incomplete Implementation Check [BLOCKING — before any quality checks]
 
-Review the diff of changed files to detect stub or incomplete implementations. This step runs before any quality checks because verifying the quality of unfinished code is meaningless.
+Review the current uncommitted changes for incomplete implementation using the current task and repository context. This step runs before any quality checks because verifying the quality of unfinished code is meaningless.
 
-Derive the current uncommitted write set from `git status --short`. Inspect tracked changes with `git diff HEAD`, read untracked files directly, and account for deleted paths without attempting to read them. The workflow commits each completed unit before the next begins, so this repository state is the quality scope.
-
-Apply the indicators below to each existing file in this write set.
+Use the indicators below for this review.
 
 **Indicators of incomplete implementation** (stub_detected):
 - `// TODO`, `// FIXME`, `// HACK`, `throw new Error("not implemented")` or equivalent

dev-workflows-fullstack/skills/recipe-add-integration-tests/SKILL.md

@@ -117,7 +117,7 @@ Invoke quality-fixer for the current layer:
 - Backend or single-layer → `subagent_type`: "dev-workflows-fullstack:quality-fixer"
 - Frontend → `subagent_type`: "dev-workflows-fullstack:quality-fixer-frontend"
 - `description`: "Final quality assurance"
-- Pass the latest executor's `mutationEvidence`; quality-fixer derives the current layer's uncommitted write set from repository status.
+- Pass the latest executor's `mutationEvidence`.
 - `prompt`: "Final quality assurance for test files added in this workflow. Run all tests and verify coverage."
 
 **Expected output**: `status` (approved/stub_detected/blocked)