fix: format inline-content review tasks as "inline content" (not "0 files")

Follow-up to the issue #350 string-output review fix. The formatter's
_task_name() used len(files_to_review) to describe task scope, which fell
through to the multi-file branch for inline-content tasks and produced
"review of 0 files" — cosmetic wart visible in the parallel-task listing.

Also adds a bespoke test_job (.deepwork/jobs/test_job/) that exercises the
full quality gate end-to-end with four outputs covering every review-level
combination: file_path + output-ref review, string + output-ref review
(NEW), string + arg-level review (NEW, inherited), and string + no review
(control). Used as a manual smoke test when verifying the fix.

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

Author: nhorton

.deepwork/jobs/test_job/fixtures/report.md

@@ -0,0 +1,4 @@
+# Test Job Report
+
+This is a short markdown report produced by the test_job workflow to exercise
+the file_path review path of the DeepWork quality-gate review system.

.deepwork/jobs/test_job/job.yml

@@ -0,0 +1,74 @@
+name: test_job
+summary: "Exercise the DeepWork quality-gate review system with string and file_path outputs to verify string-output reviews run"
+
+step_arguments:
+  - name: report_file
+    description: "A short markdown report file produced by the step — exercises the file_path review path."
+    type: file_path
+
+  - name: summary_text
+    description: "A 1-2 sentence narrative summary of the step's work — exercises a string output review defined at the step output-ref level."
+    type: string
+
+  - name: metrics_text
+    description: "A concise metrics blurb containing at least one numeric value — exercises a string output review defined at the step_argument level (inherited by any step that outputs it)."
+    type: string
+    review:
+      strategy: individual
+      instructions: |
+        Verify the value is a concise metrics summary of 1-3 sentences that contains at
+        least one numeric value. Flag it if the value is empty, a placeholder, or does
+        not mention any numbers.
+
+  - name: notes_text
+    description: "A freeform notes string with no review attached — control case that should produce no review task."
+    type: string
+
+workflows:
+  test_review_system:
+    summary: "Produce four outputs of mixed types and review levels to verify the quality gate reviews them correctly."
+    common_job_info_provided_to_all_steps_at_runtime: |
+      This bespoke test job verifies that the DeepWork quality-gate review system
+      executes reviews on string outputs as well as file_path outputs. Four outputs are
+      produced in a single step, each hitting a distinct review code path. The control
+      case (notes_text) must NOT produce a review task.
+    steps:
+      - name: produce_outputs
+        instructions: |
+          # Produce test outputs for the quality gate
+
+          Create each of the four outputs below, then call `finished_step` with all four
+          values. Keep the content short.
+
+          1. **report_file**: write a small markdown file at
+             `.deepwork/jobs/test_job/fixtures/report.md` (create the parent directory
+             first if needed). Two or three lines — a heading plus a sentence is fine.
+
+          2. **summary_text**: a 1-2 sentence narrative summary describing the work
+             performed in this step.
+
+          3. **metrics_text**: a 1-3 sentence metrics blurb that includes at least one
+             number (e.g., "Produced 4 outputs, 3 reviewed, in under 1 second.").
+
+          4. **notes_text**: any freeform string — this one has no review attached.
+
+          Expected quality-gate result: three review tasks (one per output that has a
+          review configured). `notes_text` should not produce a review task.
+        outputs:
+          report_file:
+            review:
+              strategy: individual
+              instructions: |
+                Verify the file is a valid markdown document with a recognisable heading
+                or non-empty first line. Flag it if the file is empty or contains no
+                markdown structure.
+          summary_text:
+            review:
+              strategy: individual
+              instructions: |
+                Verify the value is a non-empty narrative summary of 1-2 sentences
+                written in plain English that describes some concrete outcome of the
+                step. Flag it if the value is empty, a placeholder, or does not describe
+                anything.
+          metrics_text: {}
+          notes_text: {}

src/deepwork/review/formatter.py

@@ -137,13 +137,18 @@ def _task_name(task: ReviewTask) -> str:
     rule comes from a subdirectory .deepreview file.  This disambiguates
     same-named rules from different directories (REVIEW-REQ-004.10).
 
+    For inline-content tasks (type: string step outputs per JOBS-REQ-004.8)
+    the scope reads ``inline content`` instead of a file count.
+
     Args:
         task: The ReviewTask to name.
 
     Returns:
         Task name string.
     """
     prefix = _scope_prefix(task)
+    if not task.files_to_review and task.inline_content is not None:
+        return f"{prefix}{task.rule_name} review of inline content"
     if len(task.files_to_review) == 1:
         return f"{prefix}{task.rule_name} review of {task.files_to_review[0]}"
     return f"{prefix}{task.rule_name} review of {len(task.files_to_review)} files"

tests/unit/review/test_formatter.py

@@ -56,6 +56,20 @@ def test_grouped_task_name_includes_file_count(self, tmp_path: Path) -> None:
         result = format_for_claude([(task, file_path)], tmp_path)
         assert 'name: "py_review review of 3 files"' in result
 
+    def test_inline_content_task_name_says_inline_content(self, tmp_path: Path) -> None:
+        """Inline-content tasks render as "review of inline content", not "0 files"."""
+        task = ReviewTask(
+            rule_name="string_rule",
+            files_to_review=[],
+            instructions="Review the value.",
+            agent_name=None,
+            inline_content="the value",
+        )
+        file_path = tmp_path / "instructions.md"
+        result = format_for_claude([(task, file_path)], tmp_path)
+        assert 'name: "string_rule review of inline content"' in result
+        assert "0 files" not in result
+
     # THIS TEST VALIDATES A HARD REQUIREMENT (REVIEW-REQ-006.3.3b).
     # YOU MUST NOT MODIFY THIS TEST UNLESS THE REQUIREMENT CHANGES
     def test_default_subagent_type_when_no_agent(self, tmp_path: Path) -> None: