[pipeline-dcdad92d][slice-2/12] Determinism boundary: computed...

.changeset/review-computed-verdict.md

@@ -0,0 +1,5 @@
+---
+"review": minor
+---
+
+Make the review verdict a deterministic computation instead of a model judgment: structured findings flow through a computed verdict (severity/confidence truth table, with a hold-for-human outcome for policy conflicts), templated comment rendering, and a missing-dimension gate that refuses to conclude when a required review dimension produced no findings and no explicit all-clear. The model authors findings; code decides the verdict and renders the comments.

.changeset/review-finding-schema-foundations.md

@@ -0,0 +1,5 @@
+---
+"review": minor
+---
+
+Add the versioned structured finding schema (`workflows/review/lib/finding-schema.ts`): every sub-agent finding now carries id, lens, anchor (line/range/file/pr-level), severity, confidence, evidence trace, optional suggested patch and pre-merge obligation, validated against an exported `FINDING_SCHEMA_VERSION`. Review submission is standardized on a single robust `submit-pull-request-review` call with a guaranteed non-empty body (the empty-body retry fallback is removed), and PR context (`pr-context.json`) is staged on disk once per run for all sub-agents, extending the existing diff staging.

package.json

@@ -23,11 +23,8 @@
         "fast-glob": "^3.3.3",
         "memfs": "^4.51.0",
         "prettier": "^2.6.2",
+        "typescript": "^5.9.3",
         "vitest": "^4.0.10"
     },
-    "packageManager": "pnpm@10.0.0+sha512.b8fef5494bd3fe4cbd4edabd0745df2ee5be3e4b0b8b08fa643aa3e4c6702ccc0f00d68fa8a8c9858a735a0032485a44990ed2810526c875e416f001b17df12b",
-    "dependencies": {
-        "@swc-node/register": "^1.11.1",
-        "typescript": "^5.9.3"
-    }
+    "packageManager": "pnpm@10.0.0+sha512.b8fef5494bd3fe4cbd4edabd0745df2ee5be3e4b0b8b08fa643aa3e4c6702ccc0f00d68fa8a8c9858a735a0032485a44990ed2810526c875e416f001b17df12b"
 }