@hassangameryt/freeflow 0.5.0 → 0.7.0

This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
Files changed (301) hide show
  1. package/.claude-plugin/marketplace.json +1 -1
  2. package/.claude-plugin/plugin.json +2 -2
  3. package/.codex-plugin/plugin.json +4 -5
  4. package/README.md +141 -163
  5. package/capabilities/cognitive-routing/SKILL.md +387 -0
  6. package/capabilities/context-virtualization/SKILL.md +103 -0
  7. package/capabilities/conversation-history/SKILL.md +53 -0
  8. package/command-surface.json +6 -37
  9. package/hooks/freeflow-runtime-context.mjs +50 -541
  10. package/hooks/hooks.json +1 -26
  11. package/hooks/tests/check-runtime-context-hook.sh +114 -308
  12. package/package.json +27 -17
  13. package/pi-extension/dist/cognitive-routing/commands.js +96 -0
  14. package/pi-extension/dist/cognitive-routing/config.js +232 -0
  15. package/pi-extension/dist/cognitive-routing/controller.js +1292 -0
  16. package/pi-extension/dist/cognitive-routing/history.js +451 -0
  17. package/pi-extension/dist/cognitive-routing/host.js +188 -0
  18. package/pi-extension/dist/cognitive-routing/pi-routing-session.js +414 -0
  19. package/pi-extension/dist/cognitive-routing/pi-session-control.js +320 -0
  20. package/pi-extension/dist/cognitive-routing/runtime.js +43 -0
  21. package/pi-extension/dist/cognitive-routing/tool.js +212 -0
  22. package/pi-extension/dist/cognitive-routing/types.js +6 -0
  23. package/pi-extension/dist/context-virtualization/commands.js +89 -0
  24. package/pi-extension/dist/context-virtualization/projector.js +42 -0
  25. package/pi-extension/dist/context-virtualization/runtime.js +356 -0
  26. package/pi-extension/dist/context-virtualization/state.js +57 -0
  27. package/pi-extension/dist/context-virtualization/types.js +10 -0
  28. package/pi-extension/dist/conversation-history/passages.js +66 -0
  29. package/pi-extension/dist/conversation-history/ranking.js +105 -0
  30. package/pi-extension/dist/conversation-history/runtime.js +226 -0
  31. package/pi-extension/dist/conversation-history/search.js +56 -0
  32. package/pi-extension/dist/conversation-history/types.js +1 -0
  33. package/pi-extension/dist/freeflow-context/resolver.js +120 -0
  34. package/pi-extension/dist/freeflow-context/runtime.js +80 -0
  35. package/pi-extension/dist/freeflow-context/source-projector.js +95 -0
  36. package/pi-extension/dist/freeflow-context/tool.js +308 -0
  37. package/pi-extension/dist/freeflow-context/types.js +12 -0
  38. package/pi-extension/dist/freeflow-context/visibility.js +28 -0
  39. package/pi-extension/dist/index.js +564 -129
  40. package/pi-extension/dist/runtime/runtime-context.js +357 -470
  41. package/pi-extension/dist/runtime/runtime-identity.js +5 -0
  42. package/pi-extension/dist/settings/settings-tui.js +78 -0
  43. package/pi-extension/dist/settings/settings-ui.js +572 -826
  44. package/pi-extension/dist/ui/text-component.js +57 -0
  45. package/pi-extension/src/cognitive-routing/commands.ts +129 -0
  46. package/pi-extension/src/cognitive-routing/config.ts +303 -0
  47. package/pi-extension/src/cognitive-routing/controller.ts +1628 -0
  48. package/pi-extension/src/cognitive-routing/history.ts +650 -0
  49. package/pi-extension/src/cognitive-routing/host.ts +266 -0
  50. package/pi-extension/src/cognitive-routing/pi-routing-session.ts +538 -0
  51. package/pi-extension/src/cognitive-routing/pi-session-control.ts +444 -0
  52. package/pi-extension/src/cognitive-routing/runtime.ts +57 -0
  53. package/pi-extension/src/cognitive-routing/tool.ts +273 -0
  54. package/pi-extension/src/cognitive-routing/types.ts +89 -0
  55. package/pi-extension/src/context-virtualization/commands.ts +111 -0
  56. package/pi-extension/src/context-virtualization/projector.ts +62 -0
  57. package/pi-extension/src/context-virtualization/runtime.ts +406 -0
  58. package/pi-extension/src/context-virtualization/state.ts +79 -0
  59. package/pi-extension/src/context-virtualization/types.ts +92 -0
  60. package/pi-extension/src/conversation-history/passages.ts +81 -0
  61. package/pi-extension/src/conversation-history/ranking.ts +155 -0
  62. package/pi-extension/src/conversation-history/runtime.ts +250 -0
  63. package/pi-extension/src/conversation-history/search.ts +79 -0
  64. package/pi-extension/src/conversation-history/types.ts +31 -0
  65. package/pi-extension/src/freeflow-context/resolver.ts +139 -0
  66. package/pi-extension/src/freeflow-context/runtime.ts +100 -0
  67. package/pi-extension/src/freeflow-context/source-projector.ts +106 -0
  68. package/pi-extension/src/freeflow-context/tool.ts +360 -0
  69. package/pi-extension/src/freeflow-context/types.ts +70 -0
  70. package/pi-extension/src/freeflow-context/visibility.ts +53 -0
  71. package/pi-extension/src/index.ts +597 -135
  72. package/pi-extension/src/runtime/runtime-context.ts +373 -491
  73. package/pi-extension/src/runtime/runtime-identity.ts +8 -0
  74. package/pi-extension/src/settings/settings-tui.ts +93 -0
  75. package/pi-extension/src/settings/settings-ui.ts +679 -910
  76. package/pi-extension/src/ui/text-component.ts +66 -0
  77. package/pi-extension/tests/cognitive-routing/commands.test.js +134 -0
  78. package/pi-extension/tests/cognitive-routing/controller.test.js +1128 -0
  79. package/pi-extension/tests/cognitive-routing/extension-lifecycle.test.js +1142 -0
  80. package/pi-extension/tests/cognitive-routing/history.test.js +274 -0
  81. package/pi-extension/tests/cognitive-routing/host-fixture.js +4 -0
  82. package/pi-extension/tests/cognitive-routing/host-identity.test.js +63 -0
  83. package/pi-extension/tests/cognitive-routing/pi-normal-lifecycle.test.js +434 -0
  84. package/pi-extension/tests/cognitive-routing/pi-session-control.test.js +174 -0
  85. package/pi-extension/tests/cognitive-routing/pi-session-history.test.js +91 -0
  86. package/pi-extension/tests/cognitive-routing/profile-resolution.test.js +279 -0
  87. package/pi-extension/tests/cognitive-routing/tool.test.js +230 -0
  88. package/pi-extension/tests/context-virtualization/extension.test.js +749 -0
  89. package/pi-extension/tests/context-virtualization/projector.test.js +110 -0
  90. package/pi-extension/tests/context-virtualization/runtime.test.js +216 -0
  91. package/pi-extension/tests/conversation-history/runtime.test.js +98 -0
  92. package/pi-extension/tests/conversation-history/search.test.js +97 -0
  93. package/pi-extension/tests/integration/mode-free-core.test.js +218 -0
  94. package/pi-extension/tests/integration/pi-extension.test.js +532 -3701
  95. package/pi-extension/tests/integration/prompt-architecture.test.js +350 -0
  96. package/pi-extension/tests/integration/unaffected-regressions.test.js +411 -0
  97. package/runtime/prompts/cognitive-routing.md +10 -0
  98. package/runtime/prompts/context-virtualization.md +3 -0
  99. package/runtime/prompts/conversation-history.md +3 -0
  100. package/runtime/prompts/core.md +84 -0
  101. package/runtime/prompts/interaction-contract.md +11 -0
  102. package/skills/action-selection/SKILL.md +140 -0
  103. package/skills/action-selection/references/trajectory-stalls.md +71 -0
  104. package/skills/bypass/SKILL.md +109 -38
  105. package/skills/commit-work/SKILL.md +94 -55
  106. package/skills/commit-work/references/git-state-edges.md +94 -0
  107. package/skills/commit-work/references/simple-push.md +92 -0
  108. package/skills/commit-work/references/staging-decisions.md +79 -46
  109. package/skills/decision-gate/SKILL.md +80 -22
  110. package/skills/design-for-depth/SKILL.md +206 -60
  111. package/skills/design-for-depth/references/design-pressure-signals.md +108 -240
  112. package/skills/design-for-depth/references/interface-design-loop.md +77 -97
  113. package/skills/design-for-depth/references/module-and-dependency-design.md +147 -0
  114. package/skills/design-for-depth/references/software-design-philosophy.md +128 -174
  115. package/skills/design-for-depth/references/state-and-failure-boundaries.md +136 -0
  116. package/skills/diagnose-failure/SKILL.md +200 -48
  117. package/skills/diagnose-failure/references/diagnostic-loop-catalog.md +18 -6
  118. package/skills/diagnose-failure/references/{flaky-and-performance.md → flaky-and-performance-diagnosis.md} +13 -5
  119. package/skills/discuss/SKILL.md +119 -65
  120. package/skills/discuss/references/discussion-continuity.md +123 -0
  121. package/skills/evaluate-skill/SKILL.md +166 -52
  122. package/skills/evaluate-skill/references/{definition-schema.md → evaluation-definition-schema.md} +43 -3
  123. package/skills/evaluate-skill/references/evaluation-design.md +56 -30
  124. package/skills/evaluate-skill/references/execution-and-evidence.md +8 -3
  125. package/skills/evaluate-skill/references/review-and-revision.md +62 -21
  126. package/skills/evaluate-skill/scripts/lib/definitions.mjs +114 -3
  127. package/skills/evaluate-skill/scripts/lib/grade-interaction.mjs +156 -1
  128. package/skills/evaluate-skill/scripts/lib/grade.mjs +9 -1
  129. package/skills/evaluate-skill/scripts/lib/pi.mjs +147 -32
  130. package/skills/evaluate-skill/scripts/lib/rpc.mjs +4 -4
  131. package/skills/evaluate-skill/scripts/lib/runner.mjs +36 -14
  132. package/skills/evaluate-skill/scripts/lib/sandbox.mjs +92 -0
  133. package/skills/evaluate-skill/scripts/pi-guard.mjs +1 -1
  134. package/skills/evaluate-skill/scripts/pi-observer.mjs +66 -0
  135. package/skills/evaluate-skill/scripts/skill-eval.mjs +6 -5
  136. package/skills/execute-work/SKILL.md +99 -57
  137. package/skills/execute-work/references/code-practices.md +4 -4
  138. package/skills/execute-work/references/execute-work-edges.md +62 -0
  139. package/skills/finish-branch/SKILL.md +2 -2
  140. package/skills/finish-branch/references/integration-options.md +1 -1
  141. package/skills/handoff/SKILL.md +3 -3
  142. package/skills/launch-work/SKILL.md +1 -1
  143. package/skills/launch-work/references/launch-readiness.md +2 -2
  144. package/skills/migration-work/SKILL.md +2 -2
  145. package/skills/migration-work/references/migration-lifecycle.md +1 -1
  146. package/skills/release-work/SKILL.md +1 -1
  147. package/skills/release-work/references/release-evidence.md +1 -1
  148. package/skills/review-artifact/SKILL.md +129 -66
  149. package/skills/review-artifact/references/adjudicate-artifact-review.md +101 -0
  150. package/skills/review-artifact/references/independent-artifact-reviewer-contract.md +159 -0
  151. package/skills/review-work/SKILL.md +113 -68
  152. package/skills/review-work/references/adjudicate-work-review.md +97 -0
  153. package/skills/review-work/references/independent-work-reviewer-contract.md +139 -0
  154. package/skills/review-work/references/security-risk-lens.md +18 -16
  155. package/skills/setup-freeflow/SKILL.md +16 -17
  156. package/skills/setup-freeflow/references/activation-contract.md +24 -30
  157. package/skills/setup-freeflow/references/host-setup.md +6 -7
  158. package/skills/simplify-code/SKILL.md +97 -39
  159. package/skills/simplify-code/references/simplification-patterns.md +1 -1
  160. package/skills/tdd/SKILL.md +100 -53
  161. package/skills/tdd/references/test-design.md +3 -3
  162. package/skills/track-work/SKILL.md +195 -58
  163. package/skills/track-work/references/working-record-format.md +515 -0
  164. package/skills/track-work/scripts/lib/document.mjs +713 -0
  165. package/skills/track-work/scripts/lib/format.mjs +166 -0
  166. package/skills/track-work/scripts/lib/store.mjs +233 -0
  167. package/skills/track-work/scripts/lib/transitions.mjs +790 -0
  168. package/skills/track-work/scripts/lib/views.mjs +40 -0
  169. package/skills/track-work/scripts/working-record.mjs +316 -0
  170. package/skills/verify-work/SKILL.md +57 -19
  171. package/skills/verify-work/references/browser-runtime-evidence.md +2 -2
  172. package/skills/verify-work/references/integration-evidence.md +1 -7
  173. package/skills/verify-work/references/performance-evidence.md +2 -2
  174. package/skills/workflow/SKILL.md +137 -55
  175. package/skills/workflow/references/domain-skill-composition.md +26 -40
  176. package/skills/write-plan/SKILL.md +1 -1
  177. package/skills/write-skill/SKILL.md +218 -61
  178. package/skills/write-skill/references/activation-boundaries.md +63 -21
  179. package/skills/write-skill/references/agent-first-instructions.md +61 -27
  180. package/skills/write-skill/references/progressive-disclosure.md +97 -33
  181. package/skills/write-skill/references/skill-dependency-graphs.md +190 -0
  182. package/capabilities/output-router/SKILL.md +0 -206
  183. package/capabilities/output-router/references/safety-policy.md +0 -40
  184. package/capabilities/output-router/references/setup.md +0 -110
  185. package/pi-extension/dist/output-router/host-producer-identification.js +0 -232
  186. package/pi-extension/dist/output-router/native-safety-net.js +0 -216
  187. package/pi-extension/dist/output-router/observed-tool-routing.js +0 -122
  188. package/pi-extension/dist/output-router/renderers.js +0 -1023
  189. package/pi-extension/dist/output-router/router-tools.js +0 -554
  190. package/pi-extension/dist/output-router/schemas.js +0 -383
  191. package/pi-extension/dist/output-router/status.js +0 -744
  192. package/pi-extension/dist/output-router/utils.js +0 -502
  193. package/pi-extension/src/output-router/host-producer-identification.ts +0 -256
  194. package/pi-extension/src/output-router/native-safety-net.ts +0 -248
  195. package/pi-extension/src/output-router/observed-tool-routing.ts +0 -134
  196. package/pi-extension/src/output-router/renderers.ts +0 -1108
  197. package/pi-extension/src/output-router/router-tools.ts +0 -579
  198. package/pi-extension/src/output-router/schemas.ts +0 -387
  199. package/pi-extension/src/output-router/status.ts +0 -791
  200. package/pi-extension/src/output-router/utils.ts +0 -554
  201. package/pi-extension/tests/output-router/observed-producers.test.js +0 -132
  202. package/pi-extension/tests/output-router/search.test.js +0 -722
  203. package/router/dist/benchmarks/benchmark-harness.d.ts +0 -35
  204. package/router/dist/benchmarks/benchmark-harness.js +0 -92
  205. package/router/dist/benchmarks/benchmarks.d.ts +0 -109
  206. package/router/dist/benchmarks/benchmarks.js +0 -979
  207. package/router/dist/benchmarks/codex-qa-benchmarks.d.ts +0 -95
  208. package/router/dist/benchmarks/codex-qa-benchmarks.js +0 -500
  209. package/router/dist/benchmarks/command-benchmarks.d.ts +0 -143
  210. package/router/dist/benchmarks/command-benchmarks.js +0 -698
  211. package/router/dist/benchmarks/context-mode-normalized-benchmarks.d.ts +0 -82
  212. package/router/dist/benchmarks/context-mode-normalized-benchmarks.js +0 -773
  213. package/router/dist/benchmarks/context-mode-real-deep-benchmark.d.ts +0 -132
  214. package/router/dist/benchmarks/context-mode-real-deep-benchmark.js +0 -1380
  215. package/router/dist/benchmarks/context-saving-baseline.d.ts +0 -60
  216. package/router/dist/benchmarks/context-saving-baseline.js +0 -470
  217. package/router/dist/benchmarks/index-benchmarks.d.ts +0 -103
  218. package/router/dist/benchmarks/index-benchmarks.js +0 -862
  219. package/router/dist/benchmarks/storage-policy-benchmarks.d.ts +0 -101
  220. package/router/dist/benchmarks/storage-policy-benchmarks.js +0 -618
  221. package/router/dist/benchmarks/vault-index-benchmarks.d.ts +0 -56
  222. package/router/dist/benchmarks/vault-index-benchmarks.js +0 -322
  223. package/router/dist/config/config.d.ts +0 -103
  224. package/router/dist/config/config.js +0 -632
  225. package/router/dist/config/router-contract.d.ts +0 -10
  226. package/router/dist/config/router-contract.js +0 -54
  227. package/router/dist/config/schema.d.ts +0 -27
  228. package/router/dist/config/schema.js +0 -648
  229. package/router/dist/config/types.d.ts +0 -573
  230. package/router/dist/config/types.js +0 -41
  231. package/router/dist/evidence/bounded-evidence.d.ts +0 -40
  232. package/router/dist/evidence/bounded-evidence.js +0 -323
  233. package/router/dist/evidence/evidence-range-selector.d.ts +0 -24
  234. package/router/dist/evidence/evidence-range-selector.js +0 -232
  235. package/router/dist/evidence/evidence-search.d.ts +0 -27
  236. package/router/dist/evidence/evidence-search.js +0 -490
  237. package/router/dist/evidence/evidence.d.ts +0 -38
  238. package/router/dist/evidence/evidence.js +0 -200
  239. package/router/dist/evidence/failure-contracts.d.ts +0 -37
  240. package/router/dist/evidence/failure-contracts.js +0 -166
  241. package/router/dist/evidence/line-ranges.d.ts +0 -24
  242. package/router/dist/evidence/line-ranges.js +0 -22
  243. package/router/dist/experiments/local-index.d.ts +0 -64
  244. package/router/dist/experiments/local-index.js +0 -581
  245. package/router/dist/index.d.ts +0 -30
  246. package/router/dist/index.js +0 -32
  247. package/router/dist/local/local-traversal.d.ts +0 -18
  248. package/router/dist/local/local-traversal.js +0 -334
  249. package/router/dist/processing/engine.d.ts +0 -132
  250. package/router/dist/processing/engine.js +0 -533
  251. package/router/dist/processing/reducers.d.ts +0 -280
  252. package/router/dist/processing/reducers.js +0 -1673
  253. package/router/dist/processing/renderers.d.ts +0 -31
  254. package/router/dist/processing/renderers.js +0 -195
  255. package/router/dist/processing/scripts.d.ts +0 -91
  256. package/router/dist/processing/scripts.js +0 -386
  257. package/router/dist/repo/repo-traversal.d.ts +0 -18
  258. package/router/dist/repo/repo-traversal.js +0 -245
  259. package/router/dist/routing/observed-reducers.d.ts +0 -41
  260. package/router/dist/routing/observed-reducers.js +0 -661
  261. package/router/dist/routing/observed-routing.d.ts +0 -26
  262. package/router/dist/routing/observed-routing.js +0 -378
  263. package/router/dist/routing/parsers.d.ts +0 -16
  264. package/router/dist/routing/parsers.js +0 -521
  265. package/router/dist/routing/run-filters.d.ts +0 -60
  266. package/router/dist/routing/run-filters.js +0 -238
  267. package/router/dist/routing/run-reducers.d.ts +0 -49
  268. package/router/dist/routing/run-reducers.js +0 -122
  269. package/router/dist/sandbox/adapter-roots.d.ts +0 -46
  270. package/router/dist/sandbox/adapter-roots.js +0 -66
  271. package/router/dist/sandbox/eryx-python-adapter.d.ts +0 -13
  272. package/router/dist/sandbox/eryx-python-adapter.js +0 -868
  273. package/router/dist/sandbox/jq-wasm-adapter.d.ts +0 -9
  274. package/router/dist/sandbox/jq-wasm-adapter.js +0 -429
  275. package/router/dist/sandbox/quickjs-wasi-adapter.d.ts +0 -13
  276. package/router/dist/sandbox/quickjs-wasi-adapter.js +0 -384
  277. package/router/dist/sandbox/script-sandbox.d.ts +0 -134
  278. package/router/dist/sandbox/script-sandbox.js +0 -337
  279. package/router/dist/setup/script-transform-adapters.d.ts +0 -32
  280. package/router/dist/setup/script-transform-adapters.js +0 -278
  281. package/router/dist/tools/batch.d.ts +0 -33
  282. package/router/dist/tools/batch.js +0 -586
  283. package/router/dist/tools/run.d.ts +0 -62
  284. package/router/dist/tools/run.js +0 -1304
  285. package/router/dist/tools/search.d.ts +0 -66
  286. package/router/dist/tools/search.js +0 -1364
  287. package/router/dist/transform/engine.d.ts +0 -168
  288. package/router/dist/transform/engine.js +0 -2072
  289. package/router/dist/vault/vault-index.d.ts +0 -125
  290. package/router/dist/vault/vault-index.js +0 -416
  291. package/router/dist/vault/vault.d.ts +0 -138
  292. package/router/dist/vault/vault.js +0 -630
  293. package/runtime/interaction-contract.md +0 -16
  294. package/skills/discuss/references/checkpoints.md +0 -66
  295. package/skills/execute-work/references/execution-loop.md +0 -77
  296. package/skills/mode-contract/SKILL.md +0 -104
  297. package/skills/review-artifact/references/reviewer-prompt.md +0 -148
  298. package/skills/review-work/references/reviewer-prompt.md +0 -134
  299. package/skills/track-work/references/working-record-schema.md +0 -378
  300. package/skills/workflow/references/workflow-loop.md +0 -92
  301. /package/skills/handoff/references/{templates.md → handoff-templates.md} +0 -0
@@ -1,126 +1,189 @@
1
1
  ---
2
2
  name: review-artifact
3
- description: Use when reviewing a Working Record, spec, PRD, issue, API contract, technical design, migration contract, plan, decision record, ADR, handoff, or other durable artifact for fitness to guide its intended use.
3
+ description: Use when judging whether a durable artifact is aligned, sufficient, and fit to guide its intended use through self-review, independent review, or adjudication of a returned artifact review.
4
4
  ---
5
5
 
6
6
  # Review Artifact
7
7
 
8
- Judge whether an artifact is aligned, sufficient, and fit to guide its intended use.
8
+ Judge whether a durable artifact is aligned, sufficient, and fit to guide a named intended use.
9
9
 
10
- ## Choose The Review Role
10
+ Review provides judgment. [Verify Work](../../skills/verify-work/SKILL.md) establishes what direct evidence proves when factual claims require active verification. Review may find evidence insufficient, but it does not replace verification, settle owner decisions, or authorize revision.
11
11
 
12
- Choose the role before reviewing:
12
+ ## Select One Route
13
13
 
14
- - **Self-review:** you produced the artifact. Inspect it silently using the relevant boundary and lenses below. Correct clear local issues within existing authority. Create no formal review items, judgment, number, or cycle. Return unresolved material issues to [Workflow](../workflow/SKILL.md).
15
- - **Independent review:** you did not produce the reviewed state. Inspect and report without editing. Use the formal item, judgment, and report method below. The receiving active agent adjudicates and routes the result.
14
+ Choose the route before reviewing:
16
15
 
17
- Reading this skill does not create independence. If you produced the artifact, do not present your own judgment as independent review. Read [the reviewer prompt](references/reviewer-prompt.md) when preparing or performing a separately selected independent review.
16
+ - **Self-review:** you produced the reviewed artifact state.
17
+ - **Independent review:** you did not produce the reviewed state and are reviewing from a separate context.
18
+ - **Adjudication:** an independent reviewer returned a report and you are deciding what its findings establish and what follows.
19
+
20
+ Self-review and independent review use the same artifact review kernel below. Adjudication consumes a completed review; it is not another review.
21
+
22
+ Do not cross roles inside an independent reviewer context. The independent reviewer reports and stops. The receiving agent may then enter adjudication.
18
23
 
19
24
  ## Establish The Boundary
20
25
 
21
- Understand:
26
+ Identify only what the judgment needs:
22
27
 
23
- - artifact type, current state, and intended use;
24
- - accepted outcome, requirements, and non-goals;
25
- - owner decisions and unresolved questions;
28
+ - artifact path, type, exact reviewed state, and intended use;
29
+ - accepted outcome, requirements, non-goals, and owner decisions;
30
+ - unresolved questions and known evidence gaps;
31
+ - the artifact's job and any governing schema or format contract;
26
32
  - relevant code, tests, policies, ADRs, and established behavior;
27
- - upstream and downstream artifacts;
28
- - evidence gaps and prior review when applicable.
33
+ - upstream and downstream artifacts in dependency order;
34
+ - risks capable of changing whether the artifact may guide its intended use.
29
35
 
30
- Review the complete current artifact, not only its summary, change description, or author's reasoning. Review upstream authority before dependent artifacts. If an upstream issue invalidates downstream assumptions, mark affected material contingent instead of generating exhaustive findings against an unresolved basis.
36
+ Review the complete current artifact and source truth directly. Do not judge only its summary, change description, or producer reasoning.
31
37
 
32
- When reviewing a Working Record, read [Track Work](../track-work/SKILL.md), which owns its format and lifecycle.
38
+ Review upstream authority before dependent artifacts. When an upstream problem invalidates downstream assumptions, mark affected material contingent instead of generating exhaustive downstream findings against an unsettled basis.
33
39
 
34
- ## Judge The Artifact By Its Job
40
+ When reviewing a Working Record, first read [Track Work](../../skills/track-work/SKILL.md), which owns its format and lifecycle.
35
41
 
36
- - **Working Record:** living task memory with accurate current state, recoverable slices and decisions, compact evidence pointers, and one next useful action.
37
- - **Spec or durable content artifact:** accepted content, behavior, boundaries, evidence, and uncertainty needed for its stated use.
38
- - **Plan:** an inspectable ordered strategy with scope, dependencies, assumptions, checks, and stop conditions.
39
- - **Decision record or ADR:** decision, owner, alternatives, rationale, consequences, and revisit or supersession conditions.
40
- - **Handoff:** a point-in-time continuation package that preserves what its recipient needs without replacing live task memory.
41
- - **Other artifact:** its stated purpose without taking over another artifact's job.
42
+ When inspection could branch broadly among several useful sources or dependencies, use [Action Selection](../../skills/action-selection/SKILL.md). Skip it for an obvious artifact, schema, dependency, or source-truth read.
42
43
 
43
- ## Review Proportionately
44
+ ## Apply The Shared Artifact Review Kernel
44
45
 
45
- Apply only lenses that can materially change fitness:
46
+ Use only lenses capable of changing fitness for the intended use:
46
47
 
47
48
  - **Source alignment:** agrees with accepted requirements, owner decisions, and live facts.
48
49
  - **Fitness and sufficiency:** contains enough for its intended use without pretending every future question is settled.
49
50
  - **Decision clarity:** required, tentative, open, deferred, and superseded information cannot be confused.
50
51
  - **Evidence and acceptance:** load-bearing claims and acceptance conditions have suitable supporting or falsifying mechanisms.
51
52
  - **Behavior and failure contract:** consequential states, forbidden outcomes, observers, and recovery are explicit where required.
52
- - **Dependency integrity:** upstream and downstream artifacts remain consistent; provisional work is identified honestly.
53
+ - **Dependency integrity:** upstream and downstream artifacts remain consistent; provisional or contingent work is identified honestly.
53
54
  - **Scope and minimality:** avoids speculative design, unnecessary process, and hypothetical completeness.
54
55
  - **Clarity and continuity:** a future reader can use it without transcript memory or volatile copied context.
55
56
 
56
- A pass is valid. Do not invent items, require exhaustive edge cases, report wording or filename preferences, reopen intentional deferrals, or demand evidence needed only for a later boundary.
57
+ Review the artifact's resulting state, not the producer's intention. Use a high evidence bar, not a high item count. Finding no material issue is valid.
58
+
59
+ ### Judge The Artifact By Its Job
60
+
61
+ - **Working Record:** accurate living task memory with recoverable slices and decisions, compact evidence pointers, and one next useful action.
62
+ - **Spec or durable content artifact:** accepted content, behavior, boundaries, evidence, and uncertainty needed for its stated use.
63
+ - **Plan:** an inspectable ordered strategy with scope, dependencies, assumptions, checks, and stop conditions.
64
+ - **Decision record or ADR:** the decision, owner, alternatives, rationale, consequences, and revisit or supersession conditions.
65
+ - **Handoff:** a point-in-time continuation package that preserves what its recipient needs without replacing live task memory.
66
+ - **Other artifact:** its stated purpose without taking over another artifact's job.
67
+
68
+ Do not require one artifact to perform another artifact's job.
69
+
70
+ ### Classify Before Calling Something An Issue
71
+
72
+ Ask in order:
73
+
74
+ 1. **Does fitness for the intended use depend on content, source truth, or an owner decision that remains unsettled?**
75
+ - Yes: **Question**.
76
+ 2. **Does fitness depend on a load-bearing claim or condition that available evidence cannot establish?**
77
+ - Yes: **Needs evidence**.
78
+ 3. **Is there a supported artifact defect that requires revision for the intended use?**
79
+ - Yes, and the artifact cannot safely guide that use: **Blocking Issue**.
80
+ - Yes, but revision can be deferred safely: **Non-blocking Issue**.
81
+ 4. **Would the change merely make the artifact more useful beyond its intended use?**
82
+ - Omit it by default.
83
+ - Use **Improvement** only when materially relevant or explicitly requested.
84
+
85
+ A material **Needs evidence** observation must name the load-bearing claim or condition, required observing boundary, available evidence and its limit, why the gap affects intended-use fitness, and the smallest evidence that could disagree.
86
+
87
+ Do not turn wording preferences, exhaustive edge cases, intentional deferrals, polished presentation, or hypothetical completeness into Issues.
57
88
 
58
- ## Classify Independent Review Items
89
+ An Issue must identify:
59
90
 
60
- - **Blocking Issue:** a supported defect, inconsistency, omission, or risk that must be resolved before the artifact guides its intended use.
61
- - **Non-blocking Issue:** a real issue that can be deferred safely for this boundary.
62
- - **Question:** material intent, requirements, or an owner decision is unclear.
63
- - **Needs evidence:** a load-bearing claim or condition cannot be established.
64
- - **Improvement:** a materially useful enhancement not required by this boundary. It does not affect judgment or authorize revision.
91
+ - the exact location or affected dependency;
92
+ - the violated source truth or artifact responsibility;
93
+ - supporting evidence;
94
+ - the concrete consequence for the intended use.
65
95
 
66
- A Blocking Issue must name the exact location, violated source truth or artifact responsibility, evidence, concrete consequence for intended use, and smallest safe revision or owning activity to re-enter.
96
+ For a Blocking Issue, state why the artifact cannot safely guide its intended use. Describe revision constraints or the owning activity to re-enter. Recommend a specific revision only when source intent and dependency effects support it rather than one of several material alternatives.
67
97
 
68
- 1. **Blocking:** one or more Blocking Issues exist.
98
+ ## Self-Review
99
+
100
+ Apply the complete shared kernel to an artifact you produced. Self-review uses the same substantive scrutiny as independent review but does not create independence.
101
+
102
+ When evidence supports a clear local artifact defect, the revision basis is settled, and existing authority covers the revision:
103
+
104
+ 1. return to the artifact's owning activity;
105
+ 2. revise the defect without changing accepted intent;
106
+ 3. reconcile affected dependencies;
107
+ 4. re-verify affected load-bearing factual claims at their required observing boundaries;
108
+ 5. re-check source alignment and the affected review lenses once.
109
+
110
+ Do not create formal review items, a judgment, a review number, or a review cycle for self-review. Do not label it independent.
111
+
112
+ Do not settle an owner decision, revise upstream authority, or change accepted content merely to make the artifact internally consistent. Return unresolved material issues to [Workflow](../../skills/workflow/SKILL.md). Use [Discuss](../../skills/discuss/SKILL.md) when accepted content or strategy needs reconsideration, [Decision Gate](../../skills/decision-gate/SKILL.md) for a user-owned choice or source conflict, and [Diagnose Failure](../../skills/diagnose-failure/SKILL.md) when an artifact relies on an unsupported or repeated failure cause.
113
+
114
+ ## Independent Review
115
+
116
+ Before preparing or performing a separately selected independent review, read the [Independent Artifact Reviewer Contract](references/independent-artifact-reviewer-contract.md).
117
+
118
+ Use supplied and already available evidence. Do not start a missing active check unless the review authority explicitly covers that exact check. Otherwise report **Needs evidence** with the load-bearing claim and required observing boundary. When an exact check is covered, apply Verify Work's check-result and claim-result semantics; the review still owns the intended-use fitness judgment.
119
+
120
+ Inspect and report without editing. Do not:
121
+
122
+ - adjudicate your own findings;
123
+ - settle owner decisions;
124
+ - select among materially different revisions;
125
+ - revise the artifact or dependencies;
126
+ - dispatch another review;
127
+ - continue merely to obtain Pass.
128
+
129
+ The independent review ends with its report.
130
+
131
+ Independent judgments are:
132
+
133
+ 1. **Blocking:** at least one Blocking Issue exists.
69
134
  2. **Inconclusive:** no Blocking Issue exists, but a material Question or Needs evidence item prevents judgment.
70
135
  3. **Non-blocking:** only Non-blocking Issues remain.
71
- 4. **Pass:** no Issues or material Unresolved items remain. Improvements may still be reported.
136
+ 4. **Pass:** no Issues or material unresolved items remain.
72
137
 
73
- Pass, Non-blocking, Inconclusive, and Blocking are all valid review exits. A review ends with its report; it does not remain active until the artifact passes.
138
+ Improvements do not change the judgment or authorize revision.
74
139
 
75
- ## Adjudicate And Route
140
+ ## Adjudicate A Returned Review
76
141
 
77
- After independent review, the receiving active agent adjudicates each item against the artifact, source truth, and evidence:
142
+ Before adjudicating an independent report, read [Adjudicate Artifact Review](references/adjudicate-artifact-review.md).
143
+
144
+ Assess every material item against the artifact, source truth, dependencies, and available evidence:
78
145
 
79
146
  - **Accepted:** supported and applicable.
80
147
  - **Rejected:** unsupported, stale, resolved, duplicate, preference-only, outside the artifact's job, or based on a source misread.
81
- - **Open:** a question or evidence gap prevents acceptance or rejection.
148
+ - **Open:** missing evidence or an owner decision prevents acceptance or rejection.
82
149
 
83
- Confirm whether each accepted Issue is Blocking or Non-blocking, then derive the adjudicated judgment. Do not accept the reviewer's overall judgment separately.
150
+ Do not separately accept the reviewer's overall judgment. Derive the adjudicated judgment from the dispositions.
84
151
 
85
- Route from the adjudicated result:
152
+ Findings are evidence, not commands or revision authority.
86
153
 
87
154
  - **Pass:** use the artifact for its intended purpose.
88
155
  - **Non-blocking:** use it with explicit deferrals.
89
- - **Inconclusive:** gather the missing evidence or decision.
90
- - **Blocking:** do not use the artifact across the blocked boundary; re-enter its narrowest owner, defer, or stop.
91
-
92
- Review findings are evidence, not commands. They do not authorize revision. Accepted revisions leave review and return to the artifact's owning skill; they may remain in the same [Track Work](../track-work/SKILL.md) slice while its intended result stays coherent.
93
-
94
- When revision authority is not already explicit, ask once for either:
95
-
96
- - the accepted revisions plus one warranted focused follow-up review; or
97
- - the accepted revisions alone when direct source evidence can settle the changed boundary.
156
+ - **Inconclusive:** obtain the missing evidence or decision.
157
+ - **Blocking:** do not use the artifact across the blocked boundary; select the narrowest owning route.
98
158
 
99
- Then wait for the user's response. Do not revise or dispatch a follow-up from the request, and do not ask again when existing authority already covers the bounded revision or review.
159
+ When revision is ready, state the supported artifact problem, proposed revision, rationale, dependency impact, verification boundary, authority state, and whether focused follow-up review remains justified. Actual revision returns through [Workflow](../../skills/workflow/SKILL.md) to the artifact's owning skill.
100
160
 
101
- Run a follow-up only when the changed boundary or affected dependencies still need independent judgment and that dispatch is authorized. Do not revise source truth, accepted intent, or owner decisions merely to satisfy a reviewer or obtain Pass.
161
+ ## Limit Independent Review
102
162
 
103
- ## Limit The Review Cycle
163
+ Treat review count as a budget, not a schedule.
104
164
 
105
- For one independently reviewed artifact state and intended-use boundary:
165
+ - **Review 1:** the first selected independent review; broad by default.
166
+ - **Review 2:** a separately selected focused follow-up when accepted revisions, new evidence, affected dependencies, or remaining risk still require independent judgment.
167
+ - **Review 3:** exceptional, separately authorized, and final. Use it only after the revision basis and dependency boundary are understood.
106
168
 
107
- 1. Review 1 is the normal broad review.
108
- 2. Review 2, when needed and authorized, focuses on accepted revisions, affected dependencies, and remaining risk.
109
- 3. Review 3 is exceptional, authorized separately, and final for that cycle.
169
+ A review report always ends its review. Revision does not automatically authorize or require follow-up review.
110
170
 
111
- The budget is a cap, not dispatch authority. Do not request Review 4. At the cap, adjudicate and return control to Workflow; the artifact may be used, remain blocked, re-enter its owner, defer, or stop.
171
+ If Review 2 remains Blocking:
112
172
 
113
- After Review 2 is adjudicated Blocking, stop before proposing another revision or Review 3. Classify its relationship to the prior revision:
173
+ - diagnose when the blocker repeats, extends, invalidates, or exposes another consequence of prior revision, or when its basis remains uncertain;
174
+ - return an independent clear local artifact defect to its owner when diagnosis would add no useful understanding;
175
+ - do not automatically revise and dispatch Review 3.
114
176
 
115
- - If the blocker repeats, extends, invalidates, or exposes another consequence of the prior revision, or its cause remains unsupported, read [Diagnose Failure](../diagnose-failure/SKILL.md) and diagnose the shared cause before another revision.
116
- - If it is an independent clear local defect with a supported cause, return directly to the artifact's owner and state why diagnosis is unnecessary.
177
+ Do not request Review 4. A different reviewer, local revision, or renamed intended use does not reset the budget. Workflow may establish a new cycle only for a materially new artifact state and intended-use boundary.
117
178
 
118
- A reviewer judgment alone does not trigger diagnosis; active-agent adjudication and evidence do. Review 3 is final judgment after the cause and revision boundary are understood, not a third attempt to discover the cause through revisions. Workflow may later establish a new cycle only for a materially new artifact state and intended-use boundary; local edits, a different reviewer, or renamed scope do not reset it.
179
+ Self-review does not consume this budget.
119
180
 
120
- ## Report
181
+ ## Stop
121
182
 
122
- For self-review, report no formal review result. Correct clear local issues and surface only unresolved material issues that change fitness or route.
183
+ Stop when the selected route has produced its bounded result:
123
184
 
124
- For independent review, use the structured output in [the reviewer prompt](references/reviewer-prompt.md). The receiving agent reports the reviewer and adjudicated judgments, each material item's Accepted, Rejected, or Open outcome and reason, affected dependencies, remaining open items, and authorized or proposed next route.
185
+ - self-review: the supported artifact state is accepted or a material issue is routed;
186
+ - independent review: the report is complete;
187
+ - adjudication: material items are disposed and the next route is explicit.
125
188
 
126
- Omit empty groups. Support material judgments and adjudication with source or evidence rather than confidence.
189
+ Do not review until Pass, use review to authorize revision, or turn optional improvement into unfinished work.
@@ -0,0 +1,101 @@
1
+ # Adjudicate Artifact Review
2
+
3
+ Read this after an independent artifact review returns and before selecting what follows from its report.
4
+
5
+ Adjudication belongs to the receiving active agent. It tests the report against the artifact, source truth, dependencies, and evidence. It is not independent review and does not inherit the reviewer's authority or judgment.
6
+
7
+ ## Dispose Each Material Item
8
+
9
+ For every material item, choose one disposition.
10
+
11
+ ### Accepted
12
+
13
+ State:
14
+
15
+ - the supported artifact problem;
16
+ - its consequence for the intended use;
17
+ - whether it is Blocking or Non-blocking;
18
+ - whether the revision basis is supported or uncertain;
19
+ - which dependencies or contingent material it affects.
20
+
21
+ ### Rejected
22
+
23
+ State the concrete reason: unsupported, stale, resolved, duplicate, preference-only, outside the artifact's job, or based on a source or dependency misread.
24
+
25
+ ### Open
26
+
27
+ State:
28
+
29
+ - the unresolved content, claim, or dependency concern;
30
+ - its potential consequence for the intended use;
31
+ - the missing evidence or owner decision.
32
+
33
+ When factual evidence is missing, restate the item as a Verify Work input: load-bearing claim or condition, required observing boundary, available evidence and its limit, and the smallest evidence that could disagree.
34
+
35
+ A suggested revision may help explain a finding, but it does not settle source intent, select among material alternatives, or authorize revision.
36
+
37
+ After disposing the items, derive the adjudicated judgment. Do not separately accept the reviewer's overall judgment.
38
+
39
+ - **Pass:** no accepted Issues or material Open items remain.
40
+ - **Non-blocking:** only accepted Non-blocking Issues remain.
41
+ - **Inconclusive:** a material Open item prevents judgment.
42
+ - **Blocking:** at least one accepted Blocking Issue remains.
43
+
44
+ ## Use Verification Results
45
+
46
+ Route a known material evidence gap through [Workflow](../../../skills/workflow/SKILL.md) to [Verify Work](../../../skills/verify-work/SKILL.md). Verification produces factual evidence; it does not revise the reviewer's report or determine the review judgment.
47
+
48
+ - **Supported:** re-adjudicate the evidence gap. Do not infer Pass.
49
+ - **Contradicted:** re-adjudicate against source truth; the result may establish an Issue, invalidate one, or expose a source conflict.
50
+ - **Inconclusive:** keep the item Open. The adjudicated review remains Inconclusive when the gap is material.
51
+ - **Unavailable:** keep the item Open and state why the required boundary cannot currently be observed.
52
+
53
+ Select focused independent follow-up only when the new evidence still requires judgment from a separate context.
54
+
55
+ ## Decide Whether Revision Is Ready
56
+
57
+ Revision is ready when:
58
+
59
+ - the artifact's job and intended use are settled;
60
+ - the artifact problem and consequence are supported;
61
+ - sufficient source or decision basis exists to bound the revision;
62
+ - affected upstream and downstream dependencies are known sufficiently;
63
+ - the revision will not silently alter accepted content or an owner decision;
64
+ - no material alternative or user-owned choice remains.
65
+
66
+ When revision is ready, state:
67
+
68
+ - the proposed revision and rationale;
69
+ - affected locations and dependencies;
70
+ - required reconciliation or ordering;
71
+ - evidence or acceptance checks;
72
+ - whether focused independent follow-up remains justified;
73
+ - the authority state.
74
+
75
+ Use existing revision authority only when it covers that result and dependency impact. Otherwise propose the revision and any warranted follow-up once, then wait. A finding, adjudication, or selected review does not authorize mutation.
76
+
77
+ Actual revision returns through [Workflow](../../../skills/workflow/SKILL.md) to the artifact's owning skill. Use [Track Work](../../../skills/track-work/SKILL.md) for a Working Record, [Write Spec](../../../skills/write-spec/SKILL.md) for stable content or decision artifacts, [Write Plan](../../../skills/write-plan/SKILL.md) for a Plan, and [Handoff](../../../skills/handoff/SKILL.md) for a Handoff.
78
+
79
+ Do not revise upstream authority, accepted intent, or owner decisions merely to satisfy a reviewer or obtain Pass.
80
+
81
+ ## Route When Revision Is Not Ready
82
+
83
+ - unsettled content, strategy, or materially different revisions: [Discuss](../../../skills/discuss/SKILL.md);
84
+ - user-owned choice or source conflict: [Decision Gate](../../../skills/decision-gate/SKILL.md);
85
+ - unsupported or repeated failure cause: [Diagnose Failure](../../../skills/diagnose-failure/SKILL.md);
86
+ - missing factual support: route the named claim through [Workflow](../../../skills/workflow/SKILL.md) to [Verify Work](../../../skills/verify-work/SKILL.md);
87
+ - uncovered revision or dependency update: propose the bounded change and wait.
88
+
89
+ When findings interact, upstream authority is challenged, dependency effects remain uncertain, or user input could change the revision, report a problem checkpoint and stop before selecting or executing it.
90
+
91
+ ## After Authorized Revision
92
+
93
+ Re-verify affected load-bearing factual claims at their required observing boundaries. Then re-check source alignment, intended-use fitness, and every affected dependency before performing ordinary self-review with Review Artifact's shared kernel.
94
+
95
+ Select focused independent follow-up only when accepted revisions, affected dependencies, new evidence, or remaining risk still require judgment from a separate context. Do not dispatch follow-up merely because Review 1 found an Issue or because revision occurred.
96
+
97
+ If Review 2 remains Blocking, adjudicate it before acting again. Diagnose a repeated, extending, invalidating, or basis-uncertain blocker. Return an independent clear local artifact defect to its owner when diagnosis would add no useful understanding. Review 3 remains exceptional, separately authorized, and final.
98
+
99
+ ## Stop
100
+
101
+ Stop when every material item is Accepted, Rejected, or Open; the adjudicated judgment is supported; affected dependencies and contingent material are explicit; and the next route and authority state are clear.
@@ -0,0 +1,159 @@
1
+ # Independent Artifact Reviewer Contract
2
+
3
+ Read this before preparing or performing a separately selected independent review of a durable artifact.
4
+
5
+ The reviewer did not produce the reviewed artifact state. Give them the complete artifact, source truth, dependencies, and evidence needed to judge it directly. Do not provide only the producer's summary, reasoning, or change description.
6
+
7
+ ## Required Context
8
+
9
+ Provide:
10
+
11
+ - artifact path, type, exact reviewed state, and intended use;
12
+ - accepted outcome, requirements, non-goals, and owner decisions;
13
+ - unresolved questions and known evidence gaps;
14
+ - the artifact-specific schema or format contract when one exists;
15
+ - relevant code, tests, policies, ADRs, and established behavior;
16
+ - upstream and downstream artifacts in dependency order;
17
+ - only the review lenses material to the intended use;
18
+ - active evidence authority: none, or each exact covered check with its expected evidence and stop condition;
19
+ - review number and, for follow-up, prior items, adjudication, revisions, and new evidence.
20
+
21
+ For an artifact package, identify upstream authority and which material remains provisional or contingent.
22
+
23
+ ## Reviewer Prompt
24
+
25
+ ```md
26
+ # Independent Artifact Review
27
+
28
+ Review an artifact state you did not produce. Inspect the complete artifact, source truth, dependencies, and evidence directly. Report without editing. The review ends with this report; do not adjudicate findings, settle owner decisions, revise anything, dispatch follow-up, or continue merely to obtain Pass.
29
+
30
+ A pass is valid. Use a high evidence bar, not a high item count. Do not broaden the artifact's job or treat possible improvement as unfinished work.
31
+
32
+ Use supplied and already available evidence. Do not start a missing active check unless the review contract explicitly authorizes that exact check. Otherwise report **Needs evidence** with the load-bearing claim and required observing boundary. When an exact check is covered, apply Verify Work's check-result and claim-result semantics; active evidence does not authorize revision or broaden the review.
33
+
34
+ ## Artifact And Intended Use
35
+
36
+ - Artifact: [path]
37
+ - Type: [working record | spec | PRD | issue | API contract | technical design | migration contract | plan | decision record | ADR | handoff | other]
38
+ - Reviewed state: [revision, hash, or other exact identity]
39
+ - Intended use: [decision or future action this artifact must support]
40
+ - Dependency order: [upstream -> downstream, or standalone]
41
+
42
+ ## Accepted Outcome And Non-Goals
43
+
44
+ - Outcome: [accepted outcome]
45
+ - Requirements: [accepted requirements]
46
+ - Non-goals: [intentional exclusions]
47
+ - Owner decisions: [settled decisions or none]
48
+ - Unresolved questions: [questions or none]
49
+
50
+ ## Source Truth
51
+
52
+ - [code, tests, policies, ADRs, established behavior, upstream artifacts]
53
+
54
+ ## Evidence And Known Gaps
55
+
56
+ - Supporting evidence: [evidence or none]
57
+ - Known gaps: [gaps or none]
58
+
59
+ ## Review Scope
60
+
61
+ - Review number: [1 | 2 | 3]
62
+ - Material lenses: [source alignment | fitness and sufficiency | decision clarity | evidence and acceptance | behavior and failure contract | dependency integrity | scope and minimality | clarity and continuity]
63
+ - Active evidence authority: [none | exact covered check, expected evidence, and stop condition]
64
+
65
+ For Review 2 or 3:
66
+
67
+ - Prior items and adjudication: [Accepted | Rejected | Open]
68
+ - Revisions and new evidence: [bounded changes and evidence]
69
+ - Affected dependencies: [dependencies or none]
70
+ - Remaining risk: [narrow unresolved scope]
71
+
72
+ ## Judge The Artifact By Its Job
73
+
74
+ - **Working Record:** accurate living task memory with recoverable slices and decisions, compact evidence pointers, and one next useful action.
75
+ - **Spec or durable content artifact:** accepted content, behavior, boundaries, evidence, and uncertainty needed for its stated use.
76
+ - **Plan:** an inspectable ordered strategy with scope, dependencies, assumptions, checks, and stop conditions.
77
+ - **Decision record or ADR:** the decision, owner, alternatives, rationale, consequences, and revisit or supersession conditions.
78
+ - **Handoff:** a point-in-time continuation package that preserves what its recipient needs without replacing live task memory.
79
+ - **Other artifact:** its stated purpose without taking over another artifact's job.
80
+
81
+ Do not require one artifact to perform another artifact's job.
82
+
83
+ ## Check
84
+
85
+ Apply only the selected lenses:
86
+
87
+ - the artifact agrees with accepted requirements, owner decisions, and live facts;
88
+ - it contains enough for its intended use without pretending every future question is settled;
89
+ - required, tentative, open, deferred, and superseded information cannot be confused;
90
+ - load-bearing claims and acceptance conditions have suitable supporting or falsifying mechanisms;
91
+ - consequential behavior, failure states, forbidden outcomes, observers, and recovery are explicit where required;
92
+ - upstream and downstream artifacts remain consistent and provisional material is identified honestly;
93
+ - it avoids speculative scope, unnecessary process, and duplicated information owned elsewhere;
94
+ - a future reader can use it without transcript memory or volatile copied context.
95
+
96
+ Review upstream authority before dependent artifacts. When an upstream problem invalidates downstream assumptions, mark affected material contingent rather than generating exhaustive downstream findings against an unsettled basis.
97
+
98
+ Before calling something an Issue, ask:
99
+
100
+ 1. Does fitness for the intended use depend on content, source truth, or an owner decision that remains unsettled? If yes, use **Question**.
101
+ 2. Does fitness depend on a load-bearing claim or condition that available evidence cannot establish? If yes, use **Needs evidence**.
102
+ 3. Is there a supported artifact defect requiring revision for the intended use? Use **Blocking Issue** when the artifact cannot safely guide that use, otherwise **Non-blocking Issue** when revision can be deferred safely.
103
+ 4. Is the change merely useful beyond the intended use? Omit it by default; use **Improvement** only when materially relevant or requested.
104
+
105
+ A **Needs evidence** item must identify the load-bearing claim or condition, required observing boundary, available evidence and its limit, why the gap affects intended-use fitness, and the smallest evidence that could disagree.
106
+
107
+ A Blocking Issue must identify the exact location or dependency, violated source truth or artifact responsibility, evidence, concrete consequence for intended use, and revision constraints or owning activity to re-enter. Recommend a specific revision only when source intent and dependency effects support it.
108
+
109
+ For Review 2 or 3, inspect only accepted revisions, affected dependencies, new evidence, and remaining risk. Do not reopen Rejected items without contradictory evidence. For each new Blocking Issue, state whether it repeats, extends, invalidates, or exposes another consequence of a prior revision; is independent; or cannot yet be related from the evidence. Report a related pattern rather than proposing another revision. Review 3 is final; do not recommend Review 4.
110
+
111
+ ## Determine The Judgment
112
+
113
+ 1. **Blocking:** at least one Blocking Issue exists.
114
+ 2. **Inconclusive:** no Blocking Issue exists, but a material Question or Needs evidence item prevents judgment.
115
+ 3. **Non-blocking:** only Non-blocking Issues remain.
116
+ 4. **Pass:** no Issues or material unresolved items remain.
117
+
118
+ Improvements do not change the judgment.
119
+
120
+ ## Output
121
+
122
+ Omit empty groups. Do not include Improvements unless they are materially relevant or explicitly requested.
123
+
124
+ Review type: independent
125
+ Review number: [1 | 2 | 3]
126
+ Artifact: [reviewed artifact]
127
+ Intended use: [reviewed boundary]
128
+ Reviewed state: [state identity]
129
+ Judgment: Pass | Non-blocking | Inconclusive | Blocking
130
+ Reasoning: [concise evidence-backed judgment]
131
+ Relationship to prior items (Review 2 or 3 only): [related consequence | independent defect | unclear, with evidence]
132
+
133
+ ### Review Items
134
+
135
+ #### Issues — Blocking
136
+ - [location or dependency, issue, violated source or responsibility, evidence, consequence, revision constraints or owning activity]
137
+
138
+ #### Issues — Non-blocking
139
+ - [location or dependency, issue, evidence, why it can be deferred]
140
+
141
+ #### Unresolved — Questions
142
+ - [question, effect on judgment, required answer]
143
+
144
+ #### Unresolved — Needs evidence
145
+ - [load-bearing claim or condition, required observing boundary, available evidence and limit, effect on intended-use fitness, smallest evidence that could disagree]
146
+
147
+ #### Improvements
148
+ - [only when materially relevant or requested: improvement, benefit, evidence]
149
+
150
+ Dependencies affected: [dependencies or none]
151
+ Contingent material: [material and upstream condition, or none]
152
+ Evidence gaps: [unproved claims or none]
153
+ ```
154
+
155
+ ## Calibration
156
+
157
+ Lead with the few items that can change fitness for the intended use. Prior review, artifact length, polished language, or producer confidence does not establish fitness.
158
+
159
+ The report is evidence for adjudication. It is not authority over source truth, owner decisions, revisions, dependencies, or another independent dispatch. Pass, Non-blocking, Inconclusive, and Blocking are all valid exits.