fix(compliance): drop comply_test_controller from canonical_format_validate_input required_tools (OR-gate)

.changeset/canonical-format-validate-input-required-tools.md

@@ -0,0 +1,5 @@
+---
+"adcontextprotocol": patch
+---
+
+`canonical_format_validate_input` no longer lists `comply_test_controller` in `required_tools`. The storyboard runner's per-storyboard gate admits a storyboard when any listed tool is present, so any agent exposing the (universal) test controller was selected and then failed all 17 steps on the missing `validate_input`. Agents without `validate_input` now receive a coverage-gap skip; agents implementing it run unchanged. Same shape as #6774 (fixes #7404, bug 1).

.github/workflows/training-agent-storyboards.yml

@@ -80,7 +80,7 @@ jobs:
           - surface: current
             tenant: governance
             min_clean_storyboards: 47
-            min_passing_steps: 161
+            min_passing_steps: 160
           - surface: current
             tenant: creative
             min_clean_storyboards: 49
@@ -92,7 +92,7 @@ jobs:
           - surface: current
             tenant: brand
             min_clean_storyboards: 45
-            min_passing_steps: 116
+            min_passing_steps: 115
           - surface: current
             tenant: si
             min_clean_storyboards: 42

scripts/run-storyboards-matrix.sh

@@ -269,10 +269,10 @@ else
     # clean-result-row and passing-step regression floors.
     "signals:45:80"
     "sales:133:632"
-    "governance:47:161"
+    "governance:47:160"
     "creative:49:209"
     "creative-builder:50:184"
-    "brand:45:116"
+    "brand:45:115"
     "si:42:50"
   )
 fi

static/compliance/source/universal/canonical-format-validate-input.yaml

@@ -7,7 +7,6 @@ track: creative
 introduced_in: "3.1"
 required_tools:
   - validate_input
-  - comply_test_controller
 
 requires:
   - controller

tests/run-storyboards-sharding.test.cjs

@@ -300,13 +300,19 @@ test('creative-builder uses bounded isolated children in local and CI matrices',
 test('current training-agent floors are ratcheted and mirrored by local and CI runners', () => {
   const workflow = fs.readFileSync(STORYBOARD_WORKFLOW, 'utf8');
   const matrixRunner = fs.readFileSync(MATRIX_RUNNER, 'utf8');
+  // governance and brand dropped one passing step each when
+  // canonical_format_validate_input stopped listing comply_test_controller in
+  // required_tools: both tenants expose the controller but not validate_input,
+  // so the storyboard is no longer selected for them and the single step it
+  // contributed is gone. Deliberate de-ratchet, not a regression — the
+  // clean-storyboard floors are untouched.
   const baselines = [
     ['signals', 45, 80],
     ['sales', 133, 632],
-    ['governance', 47, 161],
+    ['governance', 47, 160],
     ['creative', 49, 209],
     ['creative-builder', 50, 184],
-    ['brand', 45, 116],
+    ['brand', 45, 115],
     ['si', 42, 50],
   ];