docs(subagent): clarify unattended review contracts

examples/acp-agent/tests/fixtures/subagent-result-diagnostic.ts

@@ -44,7 +44,7 @@ class DiagnosticProvider implements SubagentProvider {
   }
 }
 
-/** Register the fixed snapshot provider under the public product provider name. */
+/** Register the fixed provider behind the public Codex-shaped snapshot tool. */
 export function apply(ctx: Context): void {
   ctx.subagents.registerProvider(new DiagnosticProvider())
 }

packages/subagent/subagent-claude-code/src/index.ts

@@ -29,8 +29,8 @@ import {
 export const name = 'subagent-claude-code'
 export const inject = ['subagents', 'subprocess']
 
-/* jscpd:ignore-start -- sibling product providers intentionally expose the
- * same two deployment-owned fields without adding a shared config owner. */
+/* jscpd:ignore-start -- sibling product providers intentionally expose
+ * overlapping deployment-owned fields without adding a shared config owner. */
 /** Deployment-owned permission, environment, and process-release settings. */
 export interface Config {
   /**

packages/subagent/subagent-claude-code/tests/messages-fixture.ts

@@ -193,7 +193,7 @@ export async function startMessagesFixture(
       ) {
         complete(response, body, behavior.finalText)
       }
-      // A hold deliberately leaves the response pending until client abort.
+      // A hold, or a tool-use without final text, waits for client abort.
     })
   })
   await new Promise<void>((resolve, reject) => {

packages/subagent/subagent/src/run-settlement.ts

@@ -42,7 +42,7 @@ function runOutcome(result: SubagentResult): JobOutcome {
     case 'max-tokens':
     case 'refusal':
       return { status: 'failed', detail: failureDetail(result) }
-    // Merge-extensible reasons remain failures with their raw detail.
+    // Merge-extensible reasons remain failures with provider-authored detail.
     default:
       return { status: 'failed', detail: failureDetail(result) }
   }