fix: clarify mark_review_as_passed guidance for dismissed findings

The instruction file's "After Review" section now covers all three
cases: no findings, findings fixed, or findings explicitly dismissed.
The MCP tool description is updated to match.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>

Author: nhorton

src/deepwork/jobs/mcp/server.py

@@ -490,8 +490,10 @@ async def get_configured_reviews(
     @mcp.tool(
         description=(
             "Mark a review as passed so it won't be re-run while reviewed files "
-            "remain unchanged. The review_id is provided in the instruction file's "
-            '"After Review" section.'
+            "remain unchanged. Call this when a review has no findings, when all "
+            "findings have been fixed, or when remaining findings have been "
+            "explicitly dismissed by the user. The review_id is provided in the "
+            'instruction file\'s "After Review" section.'
         )
     )
     async def mark_review_as_passed(review_id: str, ctx: Context) -> str:

src/deepwork/review/instructions.py

@@ -302,7 +302,8 @@ def build_instruction_file(
     if review_id:
         parts.append("## After Review\n")
         parts.append(
-            "If this review passes with no findings, call the `mark_review_as_passed` tool with:\n"
+            "If this review passes with no findings, or if all findings have been "
+            "addressed or explicitly dismissed, call the `mark_review_as_passed` tool with:\n"
         )
         parts.append(f'- `review_id`: `"{review_id}"`')
         parts.append("")