@fitlab-ai/agent-infra 0.12.1 → 0.12.2

package/lib/internal/manual-validation.ts

@@ -1,7 +1,8 @@
 import fs from 'node:fs';
 import path from 'node:path';
 
-import { locateActivityLog, pairEntries } from '../task/activity-log.ts';
+import { activityWarnings } from '../task/activity-log.ts';
+import { lifecycleExecutions } from '../task/lifecycle-execution.ts';
 import { resolveTaskRef } from '../task/resolve-ref.ts';
 import { prepareAndRenderPrSummaryContent } from '../platform/pr-summary.ts';
 import { applyTaskEvent } from '../task/events.ts';
@@ -58,15 +59,13 @@ function openManualValidationStarted(taskMdPath: string): { present: boolean; tr
   let content: string;
   try { content = fs.readFileSync(taskMdPath, 'utf8'); }
   catch { return { present: false, transactionId: null }; }
-  const section = locateActivityLog(content);
-  if (!section) return { present: false, transactionId: null };
-  const row = pairEntries(section.entries)
-    .filter((item) => item.step === 'Complete Manual Validation' && item.started && !item.done)
+  const row = lifecycleExecutions(content)
+    .filter((item) => item.family === 'manual-validation')
     .at(-1);
   if (!row) return { present: false, transactionId: null };
   return {
     present: true,
-    transactionId: /(?:^|;\s*)transaction=([A-Za-z0-9][A-Za-z0-9._-]{0,127})(?:;|$)/u.exec(row.note)?.[1] ?? null
+    transactionId: row.transactionId ?? null
   };
 }
 
@@ -120,7 +119,7 @@ async function executeManualValidationTransactionAtResolvedTask(
     transactionResult = { ok: false, error: { code: 'MANUAL_VALIDATION_TRANSACTION_MISSING', message: 'previous failed transaction was archived for a new pull-request head' } };
   }
   if (!transactionResult.ok && transactionResult.error.code !== 'MANUAL_VALIDATION_TRANSACTION_MISSING') return result('failed', transactionResult.error);
-  if (transactionResult.ok && transactionResult.value.phase === 'committed') return result('applied', null, { transaction: transactionResult.value, receipt: transactionResult.value.committedReceipt, idempotent: true });
+  if (transactionResult.ok && transactionResult.value.phase === 'committed') return result('applied', null, { transaction: transactionResult.value, receipt: transactionResult.value.committedReceipt, idempotent: true, warnings: activityWarnings(taskContent) });
   const openStarted = transactionResult.ok ? { present: false, transactionId: null } : openManualValidationStarted(resolved.taskMdPath);
   if (openStarted.present && !openStarted.transactionId) {
     return result('failed', { code: 'MANUAL_VALIDATION_TRANSACTION_RECOVERY_REQUIRED', message: 'open manual-validation started event has no recoverable transactionId' });
@@ -148,11 +147,16 @@ async function executeManualValidationTransactionAtResolvedTask(
   });
   let transaction: ManualValidationTransaction;
   const idempotent = transactionResult.ok;
+  const warnings: { code: string; message: string }[] = [];
+  const currentWarnings = () => warnings.length
+    ? warnings.filter((warning, index) => warnings.findIndex((item) => item.code === warning.code && item.message === warning.message) === index)
+    : activityWarnings(taskContent);
   let startedEventApplied = false;
   if (!transactionResult.ok) {
     transaction = createTransaction();
     const started = applyTaskEvent({ taskRef, event: 'manual-validation.started', agent, initiator: 'model', requestId: transactionId, reasonCode: 'user-request', transactionId }, { repoRoot: cwd });
     if (started.status === 'failed') return result('failed', started.error ?? { code: 'MANUAL_VALIDATION_TRANSACTION_FAILED', message: 'manual-validation started event failed' });
+    warnings.push(...(started.warnings ?? []));
     startedEventApplied = true;
     if (previousTransaction) {
       try { archiveGeneration(resolved.taskDir, previousTransaction, previousTransaction.phase === 'committed'); }
@@ -171,8 +175,9 @@ async function executeManualValidationTransactionAtResolvedTask(
   if (transaction.phase === 'prepared' && !startedEventApplied) {
     const started = applyTaskEvent({ taskRef, event: 'manual-validation.started', agent, initiator: 'model', requestId: transaction.transactionId, reasonCode: 'user-request', transactionId: transaction.transactionId }, { repoRoot: cwd });
     if (started.status === 'failed') return result('failed', started.error ?? { code: 'MANUAL_VALIDATION_TRANSACTION_FAILED', message: 'manual-validation started event failed' });
+    warnings.push(...(started.warnings ?? []));
   }
-  if (prepareOnly) return result('applied', null, { transaction, prepared: true, idempotent });
+  if (prepareOnly) return result('applied', null, { transaction, prepared: true, idempotent, warnings: currentWarnings() });
   const transactionFailure = (failure: { code: string; message: string }): never => {
     throw new Error(`${failure.code}: ${failure.message}`);
   };
@@ -227,6 +232,7 @@ async function executeManualValidationTransactionAtResolvedTask(
     if (transaction.phase === 'receipt-committed' && !transaction.eventAppended) {
       const completed = applyTaskEvent({ taskRef, event: 'manual-validation.completed', agent, initiator: 'model', requestId: transaction.transactionId, reasonCode: 'user-request', artifact: values.artifact, summaryResult: 'human-confirmed validation and committed receipt', transactionId: transaction.transactionId, receiptDigest: receipt.receiptDigest, prHeadSha: receipt.prHeadSha }, { repoRoot: cwd });
       if (completed.status === 'failed') transactionFailure(completed.error ?? { code: 'MANUAL_VALIDATION_TRANSACTION_FAILED', message: 'manual-validation completion event failed' });
+      warnings.push(...(completed.warnings ?? []));
       transitionAndPersist('receipt-committed', { eventAppended: true });
     }
     if (transaction.phase === 'receipt-committed') {
@@ -240,7 +246,7 @@ async function executeManualValidationTransactionAtResolvedTask(
     const postWrite = await summaryCommentState(taskRef, { cwd, client: options.client });
     if (!/^###\s+✅\s+(?:Manual Validation Passed|人工验证已通过)\s*$/mu.test(postWrite.comment?.body ?? '') || postWrite.pullRequest?.head.sha !== receipt.prHeadSha || !manualValidationFinalSummaryProjectionMatches(postWrite.comment?.body ?? '', receipt)) transactionFailure({ code: 'MANUAL_VALIDATION_TRANSACTION_RECOVERY_REQUIRED', message: 'final summary post-write verification failed' });
     transitionAndPersist('committed', { postWriteVerified: true });
-    return result('applied', null, { transaction, receipt, idempotent: false });
+    return result('applied', null, { transaction, receipt, idempotent: false, warnings: currentWarnings() });
   } catch (error) {
     const message = error instanceof Error ? error.message : String(error);
     if (transaction.phase === 'final-promotion-in-progress') {
@@ -292,6 +298,7 @@ async function manualValidation(args: string[] = []): Promise<void> {
   const format = values.format === 'text' ? 'text' : 'json';
   const output = await executeManualValidationTransaction(taskRef, values, path.resolve(values.cwd ?? process.cwd()));
   process.stdout.write(format === 'json' ? `${JSON.stringify(output)}\n` : `${output.status === 'applied' ? 'Manual validation transaction committed.' : `${output.error?.code}: ${output.error?.message}`}\n`);
+  if (format === 'text' && Array.isArray(output.warnings)) for (const warning of output.warnings) process.stderr.write(`${warning.code}: ${warning.message}\n`);
   if (output.status === 'failed') process.exitCode = 1;
 }
 

package/lib/internal/task-create.ts

@@ -11,7 +11,6 @@ import { validateTaskCreateCandidate, type TaskCreateCandidateV1 } from '../task
 import {
   createTask,
   parseTaskCreateResult,
-  projectTaskCreateResult,
   taskCreateExitCode,
   taskCreateFailure,
   type TaskCreateControl,

package/lib/internal/task-event.ts

@@ -4,7 +4,7 @@ import type { TaskEventRequest, Verdict } from '../task/events.ts';
 import { resolveTaskRef } from '../task/resolve-ref.ts';
 import { ensureInternalHandlerRoute, internalHandlerRoute } from './cli-route-inventory.ts';
 
-const USAGE = `Usage: agent-infra-internal task-event <N | TASK-id> <event> --agent <agent> [event options] [--orchestrated] [--dry-run]
+const USAGE = `Usage: agent-infra-internal task-event <N | TASK-id> <event> --agent <agent> [event options] [--dry-run]
 
 Apply one closed-set task lifecycle event and print a structured JSON result.
 Events: ${eventCatalog.join(', ')}
@@ -35,10 +35,6 @@ export function parseTaskEventRequest(args: readonly string[]): TaskEventRequest
   const seen = new Set<string>();
   for (let index = 2; index < args.length; index += 1) {
     const flag = args[index]!;
-    if (flag === '--orchestrated') {
-      if (seen.has(flag)) throw new Error(`duplicate option '${flag}'`);
-      seen.add(flag); request.orchestrated = true; continue;
-    }
     if (flag === '--dry-run') {
       if (seen.has(flag)) throw new Error(`duplicate option '${flag}'`);
       seen.add(flag); request.dryRun = true; continue;

package/lib/internal/task-finalization.ts

@@ -1,4 +1,3 @@
-import path from 'node:path';
 import {
   createDirectHostExecutionContext,
   dispatchTaskControlOperation,

package/lib/internal/task-lifecycle.ts

@@ -6,12 +6,10 @@ import {
 } from '../task/control-authority.ts';
 import { lifecycleIntentCatalog } from '../task/lifecycle.ts';
 import { detectRepoRoot, resolveTaskRef } from '../task/resolve-ref.ts';
-import type { TaskLifecycleResult } from '../task/lifecycle.ts';
-import type { AgentClientLifecycleRecoveryResult as LifecycleRecoveryResult } from '../agent-clients/adapter.ts';
 import { ensureInternalHandlerRoute, internalHandlerRoute } from './cli-route-inventory.ts';
 import { loadSandboxControlLocation } from '../sandbox/config.ts';
 
-const USAGE = `Usage: agent-infra-internal task-lifecycle <N | TASK-id> <intent> --agent <agent> [intent flags] [--dry-run]\n\nIntents: ${lifecycleIntentCatalog.filter((intent) => intent !== 'complete').join(', ')}\nrecover-started: --auto\n`;
+const USAGE = `Usage: agent-infra-internal task-lifecycle <N | TASK-id> <intent> --agent <agent> [intent flags] [--dry-run]\n\nIntents: ${lifecycleIntentCatalog.filter((intent) => intent !== 'complete').join(', ')}\n`;
 
 function usageFailure(message: string): void {
   process.stdout.write(`${JSON.stringify({ status: 'failed', changed: false, error: { code: 'LIFECYCLE_PAYLOAD_INVALID', message } })}\n`);
@@ -64,9 +62,9 @@ async function taskLifecycle(args: string[] = []): Promise<void> {
   const result = await dispatchTaskControlOperation(
     createDirectHostExecutionContext({ repoRoot, ...(controlLocation ?? {}) }),
     operation
-  ) as TaskLifecycleResult | LifecycleRecoveryResult;
+  );
   process.stdout.write(`${JSON.stringify(result)}\n`);
-  if (result.status === 'failed' || result.status === 'owner-unknown' || result.status === 'conflict') process.exitCode = 1;
+  if (result.status === 'failed') process.exitCode = 1;
 }
 
 export { taskLifecycle };

package/lib/internal/task-operation-registry.ts

@@ -1,9 +1,7 @@
 import fs from 'node:fs';
-import path from 'node:path';
 
 import {
   accessSandboxTaskView,
-  parseSandboxTaskView,
   taskViewFromStatus,
   type SandboxTaskView,
   type TaskViewAccessEffect
@@ -14,7 +12,7 @@ import {
   PUBLIC_CLI_SELECTOR_ALIASES,
   internalRouteSelector
 } from './cli-route-inventory.ts';
-import { hasSandboxControlMarker, resolveSandboxControlTransport, sandboxTaskMarkerState } from '../sandbox/environment.ts';
+import { hasSandboxControlMarker, sandboxTaskMarkerState } from '../sandbox/environment.ts';
 export { resolveSandboxControlTransport } from '../sandbox/environment.ts';
 
 export type TaskOperationDispatcher = 'public' | 'internal';
@@ -54,12 +52,6 @@ function internalTaskRoutes(): TaskOperationDescriptor[] {
     descriptor('internal', 'sandbox-control', 'client', 'conditional', 'progress', 'delegated'),
     descriptor('internal', 'agent-client', 'next-steps', 'non-task', 'diagnostic', 'none'),
     descriptor('internal', 'agent-client', 'model-selection', 'non-task', 'diagnostic', 'none'),
-    descriptor('internal', 'agent-client', 'launch-carrier', 'non-task', 'diagnostic', 'none'),
-    descriptor('internal', 'codex-lifecycle', 'preflight', 'non-task', 'diagnostic', 'none'),
-    descriptor('internal', 'codex-lifecycle', 'hook-event', 'conditional', 'progress'),
-    descriptor('internal', 'codex-lifecycle', 'resolve-start', 'conditional', 'progress'),
-    descriptor('internal', 'codex-lifecycle', 'resolve-stop', 'conditional', 'progress'),
-    descriptor('internal', 'codex-lifecycle', 'consume', 'conditional', 'progress'),
     descriptor('internal', 'git-workflow', 'inspect', 'conditional', 'diagnostic'),
     descriptor('internal', 'git-workflow', 'preview-tree', 'conditional', 'diagnostic'),
     descriptor('internal', 'git-workflow', 'snapshot', 'conditional', 'diagnostic'),
@@ -132,8 +124,6 @@ function internalTaskRoutes(): TaskOperationDescriptor[] {
     descriptor('internal', 'task-artifact', 'init', 'task-bound', 'artifact-write'),
     descriptor('internal', 'task-artifact', 'preflight', 'task-bound', 'artifact-write'),
     descriptor('internal', 'task-artifact', 'finalize-local', 'task-bound', 'artifact-write'),
-    descriptor('internal', 'task-orchestration', 'status', 'task-bound', 'diagnostic'),
-    descriptor('internal', 'task-orchestration', 'progress', 'task-bound', 'progress'),
     descriptor('internal', 'task-review', 'preflight', 'task-bound', 'artifact-write'),
     descriptor('internal', 'task-review', 'finalize-summary', 'task-bound', 'progress'),
     descriptor('internal', 'task-event', 'event', 'task-bound', 'progress'),
@@ -212,11 +202,11 @@ export const TASK_OPERATION_DESCRIPTORS = Object.freeze([
 ]);
 
 export const INTERNAL_DISPATCHER_ROUTES = Object.freeze([
-  'task-create', 'task-qualification', 'sandbox-control', 'agent-client', 'codex-lifecycle',
+  'task-create', 'task-qualification', 'sandbox-control', 'agent-client',
   'git-workflow', 'task-delivery', 'release-workflow', 'platform-release-notes', 'platform-security', 'platform-metadata', 'platform-context',
   'platform-comment', 'platform-issue', 'platform-pr', 'platform-pr-review', 'pr-review-grade',
   'platform-checks', 'task-context', 'task-workspace', 'task-ledger', 'task-warning', 'task-activity', 'task-artifact',
-  'task-orchestration', 'task-review', 'task-event', 'task-lifecycle',
+  'task-review', 'task-event', 'task-lifecycle',
   'task-finalization', 'task-platform-recovery', 'task-short-id', 'task-snapshot', 'task-verify', 'task-validate', 'manual-validation'
 ]);
 export const PUBLIC_DISPATCHER_ROUTES = Object.freeze([
@@ -271,7 +261,6 @@ export function resolveTaskOperation(
 export function resolveDelegatedTaskOperation(args: readonly string[]): TaskOperationDescriptor | null {
   const [operation, ...rest] = args;
   if (operation !== 'client' || !rest[0]) return null;
-  if (rest[0] === 'task-orchestration') return null;
   return resolveTaskOperation('internal', rest[0], rest.slice(1));
 }
 
@@ -298,7 +287,6 @@ function explicitTaskRefForOperation(
     }
     return command === 'merge' ? null : first(args) || null;
   }
-  if (command === 'codex-lifecycle') return optionValue(args, '--task-id') ?? null;
   if (command === 'platform-comment' || command === 'platform-issue') {
     return first(args) === 'list' ? null : args[1] ?? null;
   }
@@ -449,7 +437,3 @@ export function guardTaskOperation(
   }
   return { descriptor: selected, taskView };
 }
-
-export function assertTaskViewStatus(value: unknown): SandboxTaskView {
-  return parseSandboxTaskView(value);
-}

package/lib/internal/task-review.ts

@@ -3,7 +3,7 @@ import { finalizeReviewSummary, preflightReviewSummary } from '../task/review-fi
 import { resolveTaskRef } from '../task/resolve-ref.ts';
 import { ensureInternalHandlerRoute } from './cli-route-inventory.ts';
 
-const USAGE = 'Usage: agent-infra-internal task-review <task-ref> <preflight|finalize-summary> --stage <analysis|plan|code> --artifact <review-*.md> [--orchestrated] [--dry-run]\n';
+const USAGE = 'Usage: agent-infra-internal task-review <task-ref> <preflight|finalize-summary> --stage <analysis|plan|code> --artifact <review-*.md> [--dry-run]\n';
 
 function failUsage(message: string): void {
   process.stdout.write(`${JSON.stringify({
@@ -25,13 +25,13 @@ async function taskReview(args: string[] = []): Promise<void> {
   let request;
   try { request = parseReviewCommand(args); }
   catch (error) { failUsage(error instanceof Error ? error.message : String(error)); return; }
-  const { stage, artifact, dryRun, orchestrated } = request;
+  const { stage, artifact, dryRun } = request;
   const operation = args[1]!;
   const resolved = resolveTaskRef(args[0]!);
   if (!resolved.ok) { failUsage(resolved.message); return; }
   const result = operation === 'preflight'
-    ? preflightReviewSummary({ taskRef: args[0]!, stage, artifact, orchestrated, dryRun })
-    : finalizeReviewSummary({ taskRef: args[0]!, stage, artifact, orchestrated, dryRun });
+    ? preflightReviewSummary({ taskRef: args[0]!, stage, artifact, dryRun })
+    : finalizeReviewSummary({ taskRef: args[0]!, stage, artifact, dryRun });
   process.stdout.write(`${JSON.stringify(result)}\n`);
   if (result.status === 'failed') process.exitCode = 1;
 }

package/lib/platform/comment-safety.ts

@@ -94,14 +94,6 @@ function escapeHtmlText(value: string): string {
   return escapeHtml(value);
 }
 
-function escapeHtmlAttribute(value: string): string {
-  return escapeHtml(value);
-}
-
-function escapeMarkdownLiteral(value: string): string {
-  return value.replace(/[\\`*_{}\[\]()#+.!|<>-]/g, '\\$&');
-}
-
 function markerSafe(value: string, markers: readonly RegExp[]): string {
   let result = value;
   for (const marker of markers) {
@@ -250,9 +242,7 @@ function splitDocumentPlaceholder(value: string, placeholder: string): Sanitizat
 
 export {
   CONTROL_MARKER_PATTERN,
-  escapeHtmlAttribute,
   escapeHtmlText,
-  escapeMarkdownLiteral,
   fenceRanges,
   renderSafeCodeFence,
   canonicalizeCommentBody,

package/lib/platform/github-provider.ts

@@ -15,7 +15,6 @@ import type {
   ChangeRequestSnapshot,
   CheckLogSnapshot,
   CheckRunSnapshot,
-  GitEvidenceSnapshot,
   IssueSnapshot,
   LabelReconciliation,
   MutationReceipt,
@@ -32,7 +31,6 @@ import type {
   ReleaseSnapshot,
   RemoteCommentSnapshot,
   ReviewSnapshot,
-  SecurityAlertKind,
   SecurityAlertSnapshot,
   VerificationRemoteFacts
 } from './provider-contract.ts';

package/lib/platform/in-label-sync.ts

@@ -182,16 +182,6 @@ function flattenPages(value: unknown): unknown[] {
   return value.flatMap((page) => Array.isArray(page) ? page : [page]);
 }
 
-function extractRepositoryLabelNames(value: unknown): string[] {
-  return flattenPages(value).flatMap((entry) => {
-    if (typeof entry === 'string') return [entry];
-    if (entry && typeof entry === 'object' && typeof (entry as { name?: unknown }).name === 'string') {
-      return [(entry as { name: string }).name];
-    }
-    return [];
-  }).filter(Boolean).sort();
-}
-
 function extractPullRequestFileNames(value: unknown): string[] | null {
   const entries = flattenPages(value);
   if (!Array.isArray(value) || entries.some((entry) => !entry || typeof entry !== 'object')) return null;
@@ -200,20 +190,14 @@ function extractPullRequestFileNames(value: unknown): string[] | null {
   return [...new Set(names as string[])].sort();
 }
 
-function resourceSnapshot(kind: InLabelResource['kind'], number: number, labels: readonly string[]): InLabelResource {
-  return { kind, number, labels: [...labels].sort() };
-}
-
 export {
   extractPullRequestFileNames,
-  extractRepositoryLabelNames,
   flattenPages,
   inLabels,
   inLabelMappingDigest,
   labelDelta,
   mergeInLabels,
   planInLabelUpdate,
-  resourceSnapshot,
   syncLabelDelta,
   validateInLabelMapping,
   validateRepositoryLabelPayload

package/lib/platform/issue-comments.ts

@@ -178,10 +178,6 @@ function renderTaskComment(content: string, taskId: string, _agent: string, lang
   return renderTaskCommentResult(content, taskId, language).body;
 }
 
-function isTaskCommentTooLarge(content: string, taskId = 'TASK-UNKNOWN', _agent = 'codex'): boolean {
-  return renderTaskCommentResult(content, taskId).byteLength > COMMENT_BYTE_LIMIT;
-}
-
 function taskCommentLanguage(repoRoot: string): string {
   try {
     const config = JSON.parse(fs.readFileSync(path.join(repoRoot, '.agents', '.airc.json'), 'utf8')) as { language?: unknown };
@@ -409,21 +405,6 @@ function findMarkerComments(comments: RemoteComment[], marker: string): RemoteCo
   return comments.filter((comment) => normalizeCommentContent(String(comment.body || '')).split('\n', 1)[0] === marker);
 }
 
-function flattenComments(value: unknown): RemoteComment[] {
-  if (!Array.isArray(value)) return [];
-  const flattened = value.flatMap((entry) => Array.isArray(entry) ? entry : [entry]);
-  return flattened.filter((entry): entry is RemoteComment =>
-    Boolean(entry && typeof entry === 'object' && (typeof entry.id === 'number' || typeof entry.id === 'string') && typeof entry.body === 'string')
-  );
-}
-
-function listRemoteComments(client: PlatformClient, repo: string, issue: number, cwd: string) {
-  const result = client.json<unknown>([
-    'api', '--paginate', '--slurp', `repos/${repo}/issues/${issue}/comments?per_page=100`
-  ], { cwd });
-  return result.ok ? { ok: true as const, value: flattenComments(result.value) } : result;
-}
-
 function contextFields(context: PlatformResult): Partial<PlatformResult> {
   return {
     platform: context.platform,
@@ -1060,14 +1041,12 @@ export {
   chunkArtifactComment,
   inspectPlatformCommentOperation,
   findMarkerComments,
-  listRemoteComments,
   checkPlatformCommentOwner,
   listPlatformComments,
   normalizeCommentContent,
   renderTaskComment,
   renderTaskCommentResult,
   syncCommentPlan,
-  isTaskCommentTooLarge,
   syncPlatformComment,
   validateRelatedMarkerSet
 };

package/lib/platform/issues.ts

@@ -27,9 +27,7 @@ import {
   labelDelta,
   inLabelMappingDigest,
   planInLabelUpdate,
-  syncLabelDelta,
   validateInLabelMapping,
-  validateRepositoryLabelPayload
 } from './in-label-sync.ts';
 import {
   providerError,

package/lib/platform/operation-coordinator.ts

@@ -37,7 +37,8 @@ async function coordinatePlatformWrite<T extends OperationResult>(input: Coordin
     });
   }
 
-  const selection = input.operation.kind === 'issue-metadata' && input.operation.issueMetadata?.state === 'closed'
+  const boundedPr = input.operation.kind === 'pull-request' && input.operation.dependency === 'required';
+  const selection = boundedPr || input.operation.kind === 'issue-metadata' && input.operation.issueMetadata?.state === 'closed'
     ? 'all' : input.operation.dependency;
   const drained = input.drain
     ? await input.drain(input.operation.taskRef, operationId, input.agent, input.operation.cwd)
@@ -45,7 +46,8 @@ async function coordinatePlatformWrite<T extends OperationResult>(input: Coordin
       agent: input.agent,
       client: input.client,
       cwd: input.operation.cwd,
-      excludeId: operationId
+      excludeId: operationId,
+      ...(boundedPr ? { beforeId: operationId } : {})
     });
   if (drained.status !== 'applied' && drained.status !== 'no-op') {
     return input.block(drained.error ? { ...drained.error, retryable: true } : {

package/lib/platform/pr-change-report.ts

@@ -493,10 +493,6 @@ function buildPrChangeReport(
   });
 }
 
-function reportFilePath(taskDir: string): string {
-  return path.join(taskDir, REPORT_FILE_NAME);
-}
-
 function readPrChangeReport(file: string): ValidationResult<PrChangeReport> {
   try {
     const stat = fs.lstatSync(file);
@@ -765,7 +761,6 @@ export {
   readPrChangeReport,
   renderCanonicalChangeReport,
   replaceCanonicalReportPlaceholder,
-  reportFilePath,
   runMechanicalChangeReport,
   taskIntentDigest,
   taskIntentProjection,

package/lib/platform/pr-checks.ts

@@ -15,7 +15,6 @@ import {
   providerError,
   providerOperationContext,
   providerStatus,
-  resourceIdentity,
   resourceIdentityNumber,
   unsupportedProviderOperation
 } from './provider-bridge.ts';
@@ -56,14 +55,6 @@ function classifyPullRequestReadiness(input: {
   return { state: 'ready', headSha: input.headSha };
 }
 
-function resolveRunCandidate(candidates: RunCandidate[], headSha: string, checkName: string):
-  | { status: 'resolved'; runId: number; jobId: number | null }
-  | { status: 'missing' | 'ambiguous'; runId: null; jobId: null } {
-  const matches = candidates.filter((candidate) => candidate.headSha === headSha && candidate.name === checkName);
-  if (matches.length === 1) return { status: 'resolved', runId: matches[0]!.id, jobId: matches[0]!.jobId || null };
-  return { status: matches.length === 0 ? 'missing' : 'ambiguous', runId: null, jobId: null };
-}
-
 function checksResult(status: PlatformResult['status'], overrides: Partial<ChecksResult> = {}): ChecksResult {
   return {
     ...platformResult(status),
@@ -256,5 +247,5 @@ async function fetchPlatformCheckLogs(taskRef: string, options: SharedOptions &
   }
 }
 
-export { classifyPullRequestReadiness, classifyRequiredChecks, fetchPlatformCheckLogs, inspectRequiredChecks, inspectPullRequestReadiness, resolvePlatformCheckRun, resolveRunCandidate, watchPullRequestReadiness };
+export { classifyPullRequestReadiness, classifyRequiredChecks, fetchPlatformCheckLogs, inspectRequiredChecks, inspectPullRequestReadiness, resolvePlatformCheckRun, watchPullRequestReadiness };
 export type { CheckBucket, CheckSnapshot, ChecksResult, ChecksSnapshot, CheckState, ReadinessSnapshot, ReadinessState, RunCandidate };

package/lib/platform/pr-review.ts

@@ -7,7 +7,7 @@ import { resolvePlatformProviderContext } from './context.ts';
 import type { PlatformClient } from './context.ts';
 import { providerErrorRetryable } from './provider-validation.ts';
 import { platformResult } from './types.ts';
-import type { PlatformOperation, PlatformResult } from './types.ts';
+import type { PlatformResult } from './types.ts';
 import {
   providerError,
   providerOperationContext,

package/lib/platform/provider-bridge.ts

@@ -4,7 +4,6 @@ import type {
   ProviderError,
   ProviderOperationContext,
   PlatformProvider,
-  ProviderResult,
   ResourceIdentity
 } from './provider-contract.ts';
 import {

package/lib/platform/provider-loader.ts

@@ -10,11 +10,7 @@ import {
   isPlatformProviderFactory,
   validatePlatformProvider
 } from './provider-contract.ts';
-import type {
-  JsonValue,
-  PlatformProvider,
-  PlatformProviderFactoryInput
-} from './provider-contract.ts';
+import type { JsonValue, PlatformProviderFactoryInput } from './provider-contract.ts';
 import { createGitHubProvider } from './github-provider.ts';
 import { createNoneProvider } from './none-provider.ts';
 import { wrapProviderOperations } from './provider-validation.ts';

package/lib/platform/provider-validation.ts

@@ -1,33 +1,6 @@
 import { parseResourceIdentity, serializeResourceIdentity } from './resource-identity.ts';
 import type { ProviderIdentityDeclaration, ResourceIdentityKind } from './resource-identity.ts';
-import type {
-  ChangeRequestSnapshot,
-  CheckLogSnapshot,
-  CheckRunSnapshot,
-  GitEvidenceSnapshot,
-  LabelDefinition,
-  LabelReconciliation,
-  IssueSnapshot,
-  MilestoneDefinition,
-  MilestoneInitialization,
-  MilestoneReconciliation,
-  MutationReceipt,
-  PlatformContextSnapshot,
-  ProviderError,
-  PlatformProvider,
-  ProviderResult,
-  ReleaseNoteAuthor,
-  ReleaseNoteIssue,
-  ReleaseNotesFacts,
-  ReleaseSnapshot,
-  RemoteCommentSnapshot,
-  RepositoryMetadataSnapshot,
-  RequiredCheckSnapshot,
-  ReviewSnapshot,
-  SecurityAlertKind,
-  SecurityAlertSnapshot,
-  VerificationRemoteFacts
-} from './provider-contract.ts';
+import type { ChangeRequestSnapshot, CheckLogSnapshot, CheckRunSnapshot, GitEvidenceSnapshot, LabelReconciliation, IssueSnapshot, MilestoneInitialization, MilestoneReconciliation, MutationReceipt, PlatformContextSnapshot, ProviderError, PlatformProvider, ReleaseNoteAuthor, ReleaseNoteIssue, ReleaseNotesFacts, ReleaseSnapshot, RemoteCommentSnapshot, RepositoryMetadataSnapshot, RequiredCheckSnapshot, ReviewSnapshot, SecurityAlertKind, SecurityAlertSnapshot, VerificationRemoteFacts } from './provider-contract.ts';
 import type { JsonValue } from './provider-contract.ts';
 import type { PlatformError } from './types.ts';
 
@@ -336,16 +309,6 @@ function validateSecurityAlert(value: unknown): SecurityAlertSnapshot {
   };
 }
 
-function validateLabelDefinition(value: unknown, label: string): LabelDefinition {
-  const item = record(value, label);
-  exactKeys(item, ['name', 'color', 'description'], label);
-  return {
-    name: stringValue(item.name, `${label}.name`),
-    color: stringValue(item.color, `${label}.color`),
-    description: stringValue(item.description, `${label}.description`, true)
-  };
-}
-
 function validateLabelReconciliation(value: unknown): LabelReconciliation {
   const item = record(value, 'label reconciliation');
   exactKeys(item, ['changed', 'created', 'updated', 'removed', 'skipped'], 'label reconciliation');
@@ -359,17 +322,6 @@ function validateLabelReconciliation(value: unknown): LabelReconciliation {
   };
 }
 
-function validateMilestoneDefinition(value: unknown, label: string): MilestoneDefinition {
-  const item = record(value, label);
-  exactKeys(item, ['title', 'description', 'state'], label);
-  if (item.state !== 'open' && item.state !== 'closed') throw new Error(`${label}.state is invalid`);
-  return {
-    title: stringValue(item.title, `${label}.title`),
-    description: stringValue(item.description, `${label}.description`, true),
-    state: item.state
-  };
-}
-
 function validateMilestoneInitialization(value: unknown): MilestoneInitialization {
   const item = record(value, 'milestone initialization');
   exactKeys(item, ['changed', 'created', 'skipped'], 'milestone initialization');