@fitlab-ai/agent-infra 0.11.3 → 0.11.4

package/dist/lib/task/capabilities.js

@@ -2,7 +2,7 @@ import fs from 'node:fs';
 import path from 'node:path';
 import { parseArtifactName } from './artifact-name.js';
 import { invalidationBlocks, isArtifactInvalidated, parseInvalidationDocument } from './invalidation.js';
-import { receiptForOutput } from './artifact-receipts.js';
+import { parseArtifactReceipts, receiptForOutput } from './artifact-receipts.js';
 import { parseTypedTaskFrontmatter } from './frontmatter.js';
 import { parseLedgerDocument, summarizeLedgerStage, validateLedgerRows } from './ledger.js';
 import { parseReviewSummary, resolveCanonicalVerdict } from './review-artifacts.js';
@@ -10,6 +10,8 @@ import { parseReworkIntentDocument } from './rework-intent.js';
 import { sha256File } from './artifact-receipts.js';
 import { hasOpenLifecycleExecution } from './activity-log.js';
 import { parseQualificationAudit, parseTaskQualification } from './qualification-audit.js';
+import { parseLifecyclePathDecision, pathIncludes } from './lifecycle-path.js';
+import { parseImplementationInputs } from './implementation-inputs.js';
 const ARTIFACT_AUDIT_FAMILIES = new Set(['analysis', 'review-analysis', 'plan', 'review-plan', 'code', 'review-code']);
 const REASONS = new Set([
     'user-request', 'new-requirement', 'upstream-fact-doubt', 'review-finding', 'retry', 'validation-rerun'
@@ -23,6 +25,11 @@ function allow(...evidence) {
 function hasArtifact(facts, action) {
     return (facts.artifacts[action]?.length ?? 0) > 0;
 }
+function effectiveReworkTarget(target, pathState) {
+    if (target === 'plan' && pathState?.status === 'valid' && !pathIncludes(pathState, 'plan'))
+        return 'analysis';
+    return target;
+}
 function canStart(action, facts, trigger) {
     if (!trigger || trigger.requestedAction !== action)
         return deny('TRIGGER_ACTION_MISMATCH', `requested=${trigger?.requestedAction ?? 'missing'}`);
@@ -45,11 +52,30 @@ function canStart(action, facts, trigger) {
         return deny('TASK_NOT_ACTIVE', `state=${facts.taskState}`);
     if (invalidationBlocks(facts.invalidation))
         return deny('INVALIDATION_INCOMPLETE');
-    if (facts.qualificationStale && facts.recommendedAction !== action) {
-        return deny('QUALIFICATION_STALE', ...(facts.qualificationStaleArtifacts ?? ['qualification audit is stale']));
-    }
     if (facts.executionBusy)
         return deny('EXECUTION_BUSY');
+    const pendingIntent = (facts.reworkIntents ?? []).find((intent) => intent.status === 'pending');
+    if (pendingIntent) {
+        const target = effectiveReworkTarget(pendingIntent.target, facts.pathState);
+        const requirementRestart = action === 'analysis' && trigger.reasonCode === 'new-requirement';
+        if (!requirementRestart && (target === 'pause' || target !== action)) {
+            return deny(target === 'pause' ? 'REWORK_PAUSED' : 'REWORK_INTENT_TARGET_MISMATCH', pendingIntent.intentId);
+        }
+    }
+    if (facts.pathState && facts.pathState.status !== 'valid') {
+        return action === 'analysis'
+            ? allow(`lifecycle-path-${facts.pathState.status}`)
+            : deny('LIFECYCLE_PATH_INVALID', facts.pathState.message);
+    }
+    if (facts.pathState?.status === 'valid'
+        && ['analysis', 'review-analysis', 'plan', 'review-plan', 'code', 'review-code'].includes(action)
+        && !pathIncludes(facts.pathState, action)) {
+        return deny('ARTIFACT_STAGE_NOT_SELECTED', `path=${facts.pathState.decision.path}`, `stage=${action}`);
+    }
+    if (facts.reworkClassificationRequired?.includes(action)
+        && !(facts.reworkIntents ?? []).some((intent) => intent.status === 'pending')) {
+        return deny('REWORK_CLASSIFICATION_REQUIRED', `stage=${action}`);
+    }
     const implicit = trigger.explicitRequest === false;
     if (action === 'analysis') {
         if (!implicit || hasArtifact(facts, 'analysis') || facts.recommendedAction === 'analysis')
@@ -60,6 +86,9 @@ function canStart(action, facts, trigger) {
         return hasArtifact(facts, 'analysis') ? allow('analysis-artifact') : deny('ANALYSIS_ARTIFACT_REQUIRED');
     }
     if (action === 'plan') {
+        if (facts.pathState?.status === 'valid' && facts.pathState.decision.path === 'standard') {
+            return hasArtifact(facts, 'analysis') ? allow('analysis-artifact') : deny('ANALYSIS_ARTIFACT_REQUIRED');
+        }
         if (!hasArtifact(facts, 'review-analysis'))
             return deny('ANALYSIS_REVIEW_REQUIRED');
         if (!reviewMatchesLatest(facts, 'analysis', 'review-analysis'))
@@ -75,6 +104,8 @@ function canStart(action, facts, trigger) {
     }
     if (action === 'code') {
         if (trigger.implementationInput) {
+            if (facts.resolvedHumanDecisions?.code === 'implementation')
+                return allow('human-decision-implementation-input');
             if (!hasArtifact(facts, 'review-code'))
                 return deny('CODE_REVIEW_REQUIRED');
             if (!reviewMatchesLatest(facts, 'code', 'review-code'))
@@ -83,6 +114,20 @@ function canStart(action, facts, trigger) {
                 return deny('CODE_REVIEW_NOT_APPROVED');
             return allow('implementation-input');
         }
+        if (facts.pathState?.status === 'valid' && facts.pathState.decision.path === 'streamlined') {
+            if (!hasArtifact(facts, 'analysis'))
+                return deny('ANALYSIS_ARTIFACT_REQUIRED');
+            if (facts.unresolvedLedger.analysis > 0)
+                return deny('ANALYSIS_LEDGER_BLOCKED', `unresolved=${facts.unresolvedLedger.analysis}`);
+            return allow('analysis-path-approved');
+        }
+        if (facts.pathState?.status === 'valid' && facts.pathState.decision.path === 'standard') {
+            if (!hasArtifact(facts, 'plan'))
+                return deny('PLAN_ARTIFACT_REQUIRED');
+            if (facts.unresolvedLedger.plan > 0)
+                return deny('PLAN_LEDGER_BLOCKED', `unresolved=${facts.unresolvedLedger.plan}`);
+            return allow('plan-path-approved');
+        }
         if (!hasArtifact(facts, 'review-plan'))
             return deny('PLAN_REVIEW_REQUIRED');
         if (!reviewMatchesLatest(facts, 'plan', 'review-plan'))
@@ -113,14 +158,6 @@ function latestArtifact(names) {
     return names.map(parseArtifactName).filter((identity) => identity !== null)
         .sort((left, right) => right.round - left.round || left.name.localeCompare(right.name))[0]?.name ?? null;
 }
-function reviewedInputName(content, expectedFamily) {
-    const header = content.split(/\r?\n/).findIndex((line) => /\*\*(?:审查输入|Review Input)\*\*[:：]/.test(line));
-    if (header < 0)
-        return null;
-    const referenceBlock = content.split(/\r?\n/).slice(header, header + 12).join('\n');
-    const expected = new RegExp('`' + expectedFamily + '(?:-r[2-9]|-r[1-9]\\d+)?\\.md`');
-    return expected.exec(referenceBlock)?.[0].slice(1, -1) ?? null;
-}
 function reviewMatchesLatest(facts, source, review) {
     const latestSource = latestArtifact(facts.artifacts[source] ?? []);
     return Boolean(latestSource && facts.reviewedInputs?.[review] === latestSource);
@@ -133,30 +170,56 @@ function qualificationCandidateSnapshotMatches(audit, qualification) {
             && candidate.evidence === row.evidence && candidate.constraintIds.join(',') === row.constraintIds.join(','));
     });
 }
-const QUALIFICATION_RECOVERY_ORDER = [
-    { action: 'analysis', pattern: /^analysis(?:-r\d+)?\.md$/ },
-    { action: 'review-analysis', pattern: /^review-analysis(?:-r\d+)?\.md$/ },
-    { action: 'plan', pattern: /^plan(?:-r\d+)?\.md$/ },
-    { action: 'review-plan', pattern: /^review-plan(?:-r\d+)?\.md$/ },
-    { action: 'code', pattern: /^code(?:-r\d+)?\.md$/ },
-    { action: 'review-code', pattern: /^review-code(?:-r\d+)?\.md$/ }
-];
-function qualificationRecoveryAction(facts) {
-    const stale = facts.qualificationStaleArtifacts ?? [];
-    return QUALIFICATION_RECOVERY_ORDER.find(({ pattern }) => stale.some((name) => pattern.test(name)))?.action ?? null;
-}
 function recommendNext(facts) {
-    if (facts.qualificationStale) {
-        const action = qualificationRecoveryAction(facts);
-        return {
-            action,
-            reasonCode: 'QUALIFICATION_RECOVERY_REQUIRED',
-            evidence: facts.qualificationStaleArtifacts ?? ['qualification audit is stale']
-        };
+    if ((facts.reworkClassificationRequired?.length ?? 0) > 0
+        && !(facts.reworkIntents ?? []).some((intent) => intent.status === 'pending')) {
+        return { action: null, reasonCode: 'REWORK_CLASSIFICATION_REQUIRED', evidence: facts.reworkClassificationRequired ?? [] };
     }
     const pendingIntent = (facts.reworkIntents ?? []).find((intent) => intent.status === 'pending');
-    if (pendingIntent)
-        return { action: pendingIntent.target, reasonCode: 'REWORK_INTENT_PENDING', evidence: [pendingIntent.intentId, pendingIntent.findingId] };
+    if (pendingIntent) {
+        const target = effectiveReworkTarget(pendingIntent.target, facts.pathState);
+        return { action: target === 'pause' ? null : target, reasonCode: target === 'pause' ? 'REWORK_PAUSED' : 'REWORK_INTENT_PENDING', evidence: [pendingIntent.intentId, pendingIntent.findingId] };
+    }
+    if (facts.pathState && facts.pathState.status !== 'valid') {
+        return { action: 'analysis', reasonCode: facts.pathState.status === 'missing' ? 'LIFECYCLE_PATH_MISSING' : 'LIFECYCLE_PATH_INVALID', evidence: [facts.pathState.message] };
+    }
+    if (facts.pathState?.status === 'valid') {
+        for (const stage of facts.pathState.decision.stages) {
+            if (stage === 'analysis' && !hasArtifact(facts, stage))
+                return { action: stage, reasonCode: 'ANALYSIS_ARTIFACT_MISSING', evidence: ['analysis artifact is absent'] };
+            if (stage === 'review-analysis' && (!reviewMatchesLatest(facts, 'analysis', stage) || facts.reviews[stage] !== 'approved')) {
+                if (facts.reviews[stage] === 'changes-requested' && facts.resolvedHumanDecisions?.analysis === 'review') {
+                    return { action: stage, reasonCode: 'HUMAN_DECISION_REVIEW_REQUIRED', evidence: ['analysis decision was resolved'] };
+                }
+                return facts.reviews[stage] === 'changes-requested'
+                    ? { action: 'analysis', reasonCode: 'ANALYSIS_REWORK_REQUIRED', evidence: ['analysis review is not approved'] }
+                    : { action: stage, reasonCode: 'ANALYSIS_REVIEW_MISSING', evidence: ['analysis review does not bind the latest analysis artifact'] };
+            }
+            if (stage === 'plan' && !hasArtifact(facts, stage))
+                return { action: stage, reasonCode: 'PLAN_ARTIFACT_MISSING', evidence: ['plan artifact is absent'] };
+            if (stage === 'review-plan' && (!reviewMatchesLatest(facts, 'plan', stage) || facts.reviews[stage] !== 'approved')) {
+                if (facts.reviews[stage] === 'changes-requested' && facts.resolvedHumanDecisions?.plan === 'review') {
+                    return { action: stage, reasonCode: 'HUMAN_DECISION_REVIEW_REQUIRED', evidence: ['plan decision was resolved'] };
+                }
+                return facts.reviews[stage] === 'changes-requested'
+                    ? { action: 'plan', reasonCode: 'PLAN_REWORK_REQUIRED', evidence: ['plan review is not approved'] }
+                    : { action: stage, reasonCode: 'PLAN_REVIEW_MISSING', evidence: ['plan review does not bind the latest plan artifact'] };
+            }
+            if (stage === 'code' && !hasArtifact(facts, stage))
+                return { action: stage, reasonCode: 'CODE_ARTIFACT_MISSING', evidence: ['code artifact is absent'] };
+            if (stage === 'review-code' && (!reviewMatchesLatest(facts, 'code', stage) || facts.reviews[stage] !== 'approved')) {
+                if (facts.reviews[stage] === 'changes-requested' && facts.resolvedHumanDecisions?.code) {
+                    return facts.resolvedHumanDecisions.code === 'implementation'
+                        ? { action: 'code', reasonCode: 'HUMAN_DECISION_IMPLEMENTATION_REQUIRED', evidence: ['code decision requires implementation'] }
+                        : { action: stage, reasonCode: 'HUMAN_DECISION_REVIEW_REQUIRED', evidence: ['code decision requires no implementation'] };
+                }
+                return facts.reviews[stage] === 'changes-requested'
+                    ? { action: 'code', reasonCode: 'CODE_REWORK_REQUIRED', evidence: ['code review is not approved'] }
+                    : { action: stage, reasonCode: 'CODE_REVIEW_MISSING', evidence: ['code review does not bind the latest code artifact'] };
+            }
+        }
+        return { action: null, reasonCode: 'LIFECYCLE_REVIEWED', evidence: ['selected lifecycle path is complete'] };
+    }
     if (!hasArtifact(facts, 'analysis'))
         return { action: 'analysis', reasonCode: 'ANALYSIS_ARTIFACT_MISSING', evidence: ['analysis artifact is absent'] };
     if (!hasArtifact(facts, 'review-analysis') || !reviewMatchesLatest(facts, 'analysis', 'review-analysis')) {
@@ -213,9 +276,12 @@ function buildLifecycleFacts(taskDir, content, taskState = 'active', executionBu
             const family = familyFor(name);
             return !family || !isArtifactInvalidated(invalidation.document, family, name);
         });
+        const allArtifactHashes = {};
+        for (const name of files)
+            allArtifactHashes[name] = sha256File(path.join(taskDir, name));
         const artifactHashes = {};
         for (const name of activeFiles)
-            artifactHashes[name] = sha256File(path.join(taskDir, name));
+            artifactHashes[name] = allArtifactHashes[name];
         const artifacts = Object.fromEntries(artifactFamilies.map((family) => [
             family, activeFiles.filter((name) => familyFor(name) === family)
         ]));
@@ -264,9 +330,12 @@ function buildLifecycleFacts(taskDir, content, taskState = 'active', executionBu
                 continue;
             const reviewContent = fs.readFileSync(path.join(taskDir, latest), 'utf8');
             const expectedFamily = family === 'review-analysis' ? 'analysis' : family === 'review-plan' ? 'plan' : 'code';
-            const input = reviewedInputName(reviewContent, expectedFamily);
-            const receipt = input ? receiptForOutput(content, latest) : null;
-            if (input && receipt?.input === input && artifactHashes[input] === receipt.inputSha256)
+            const receipt = receiptForOutput(content, latest);
+            const input = receipt?.input;
+            if (input
+                && parseArtifactName(input)?.family === expectedFamily
+                && (artifacts[expectedFamily] ?? []).includes(input)
+                && artifactHashes[input] === receipt.inputSha256)
                 reviewedInputs[family] = input;
             const parsed = parseReviewSummary(reviewContent);
             if (!parsed.ok)
@@ -275,11 +344,58 @@ function buildLifecycleFacts(taskDir, content, taskState = 'active', executionBu
             if (verdict.ok)
                 reviews[family] = verdict.verdict === 'Approved' ? 'approved' : verdict.verdict === 'Changes Requested' ? 'changes-requested' : 'rejected';
         }
+        const reworkClassificationRequired = [];
+        const receipts = parseArtifactReceipts(content).rows;
+        for (const [reviewFamily, stage] of [['review-analysis', 'analysis'], ['review-plan', 'plan'], ['review-code', 'code']]) {
+            const cycles = receipts.filter((receipt) => receipt.event === `${reviewFamily}.completed` && files.includes(receipt.output)
+                && allArtifactHashes[receipt.input] === receipt.inputSha256).map((receipt) => {
+                const parsed = parseReviewSummary(fs.readFileSync(path.join(taskDir, receipt.output), 'utf8'));
+                const verdict = parsed.ok ? resolveCanonicalVerdict(parsed.summary) : null;
+                return {
+                    input: receipt.input,
+                    output: receipt.output,
+                    outputRound: parseArtifactName(receipt.output)?.round ?? 0,
+                    inputRound: parseArtifactName(receipt.input)?.round ?? 0,
+                    verdict: verdict?.ok ? verdict.verdict : null
+                };
+            }).sort((left, right) => left.inputRound - right.inputRound || left.outputRound - right.outputRound || left.output.localeCompare(right.output));
+            const latest = cycles.at(-1);
+            const trailingInputRounds = new Set();
+            for (let index = cycles.length - 1; index >= 0 && cycles[index].verdict === 'Changes Requested'; index -= 1) {
+                trailingInputRounds.add(cycles[index].inputRound);
+            }
+            const handled = latest && (rework.intents ?? []).some((intent) => intent.sourceArtifact === latest.output
+                && intent.sourceSha256 === allArtifactHashes[latest.output]);
+            const latestActiveReview = latestArtifact(artifacts[reviewFamily] ?? []);
+            const latestActiveInput = latestArtifact(artifacts[stage] ?? []);
+            const latestCycleIsActive = latest?.output === latestActiveReview && latest.input === latestActiveInput;
+            if (latestCycleIsActive && latest.verdict === 'Changes Requested' && trailingInputRounds.size >= 2 && !handled)
+                reworkClassificationRequired.push(stage);
+        }
         const unresolvedLedger = { analysis: 0, plan: 0, code: 0 };
         if (ledger.present) {
             for (const stage of ['analysis', 'plan', 'code'])
                 unresolvedLedger[stage] = summarizeLedgerStage(ledger.rows, stage).unresolved.length;
         }
+        const latestAnalysis = latestArtifact(artifacts.analysis ?? []);
+        const pathState = latestAnalysis
+            ? parseLifecyclePathDecision(fs.readFileSync(path.join(taskDir, latestAnalysis), 'utf8'), latestAnalysis, artifactHashes[latestAnalysis] ?? '')
+            : { status: 'missing', decision: null, message: 'analysis artifact is missing' };
+        const resolvedHumanDecisions = {};
+        const implementationInputs = parseImplementationInputs(content).rows;
+        for (const stage of ['analysis', 'plan', 'code']) {
+            const reviewFamily = `review-${stage}`;
+            const latestReview = latestArtifact(artifacts[reviewFamily] ?? []);
+            if (!latestReview || reviews[reviewFamily] !== 'changes-requested')
+                continue;
+            const latestHash = artifactHashes[latestReview];
+            const decision = rework.intents.find((intent) => intent.status === 'consumed'
+                && intent.classification === 'human-decision' && intent.sourceArtifact === latestReview && intent.sourceSha256 === latestHash);
+            if (!decision)
+                continue;
+            const implementationInput = implementationInputs.find((input) => input.ledgerId === decision.findingId);
+            resolvedHumanDecisions[stage] = stage === 'code' && implementationInput?.status === 'pending' ? 'implementation' : 'review';
+        }
         const facts = {
             taskState, currentStep: String(metadata.current_step ?? ''), artifacts, reviews,
             staleArtifacts, reviewedInputs, artifactHashes,
@@ -288,7 +404,10 @@ function buildLifecycleFacts(taskDir, content, taskState = 'active', executionBu
             executionBusy: executionBusy || hasOpenLifecycleExecution(content),
             recommendedAction: null,
             qualificationStale: qualificationStaleArtifacts.length > 0,
-            qualificationStaleArtifacts
+            qualificationStaleArtifacts,
+            pathState,
+            reworkClassificationRequired,
+            resolvedHumanDecisions
         };
         facts.recommendedAction = recommendNext(facts).action;
         return { ok: true, facts };
@@ -297,5 +416,5 @@ function buildLifecycleFacts(taskDir, content, taskState = 'active', executionBu
         return { ok: false, code: 'TASK_CAPABILITY_FACTS_INVALID', message: error instanceof Error ? error.message : String(error) };
     }
 }
-export { buildLifecycleFacts, canStart, recommendNext };
+export { buildLifecycleFacts, canStart, effectiveReworkTarget, recommendNext };
 //# sourceMappingURL=capabilities.js.map

package/dist/lib/task/commands/status.js

@@ -325,7 +325,11 @@ function renderStatus(model) {
         ['processed', `${model.invalidation?.processed ?? 0}/${model.invalidation?.total ?? 0}`],
         ['pending_intents', String(model.invalidation?.pendingIntents ?? 0)]
     ]));
-    lines.push('', 'Recommendation', ...renderPairs([['action', model.recommendation ?? DASH]]));
+    lines.push('', 'Recommendation', ...renderPairs([
+        ['path', model.lifecyclePath ?? DASH],
+        ['action', model.recommendation ?? DASH],
+        ['reason', model.recommendationReason ?? DASH]
+    ]));
     lines.push('', 'Orchestration', ...renderPairs([
         ['status', model.orchestration.status],
         ['run_id', model.orchestration.runId],
@@ -411,7 +415,8 @@ function buildFromResolved(input) {
             pendingIntents
         };
     const facts = buildLifecycleFacts(input.taskDir, content, fm.status ?? 'active');
-    const recommendation = facts.ok ? (recommendNext(facts.facts).action ?? DASH) : DASH;
+    const next = facts.ok ? recommendNext(facts.facts) : null;
+    const recommendation = next?.action ?? DASH;
     return {
         taskId: input.taskId,
         shortId: input.shortId ?? loadShortIdByTaskId(input.repoRoot).get(input.taskId) ?? DASH,
@@ -424,6 +429,8 @@ function buildFromResolved(input) {
         orchestration: collectOrchestration(input.taskDir),
         invalidation: invalidationInfo,
         recommendation,
+        recommendationReason: next?.reasonCode ?? 'TASK_CAPABILITY_FACTS_INVALID',
+        lifecyclePath: facts.ok && facts.facts.pathState?.status === 'valid' ? facts.facts.pathState.decision.path : facts.ok ? facts.facts.pathState?.status ?? DASH : DASH,
         git: collectGit(fm.branch ?? '', run)
     };
 }

package/dist/lib/task/control-authority.js

@@ -6,7 +6,7 @@ import { isAgentClientId } from '../agent-clients/types.js';
 import { getAgentClientAdapter } from '../agent-clients/registry.js';
 import { consumeHumanOverride, failureId, overrideDryRunConflict } from './human-override.js';
 import { activateMatchingOrchestrationDelegation, activateOrchestrationDelegation, advanceOrchestration, awaitOrchestrationDelegationActivation, beginOrResumeOrchestration, dispatchOrchestrationDelegation, OrchestrationStateError, pauseOrchestration, prepareOrchestrationDelegation, recoverPreparedOrchestrationDelegation, routeOrchestration, sealMatchingOrchestrationDelegation, sealOrchestrationDelegation, statusOrchestration } from './orchestration.js';
-import { applyTaskFinalization } from './finalization.js';
+import { applyTaskFinalization, commitPreparedTaskFinalization } from './finalization.js';
 import { applyTaskLifecycle, taskLifecycleFailure } from './lifecycle.js';
 import { locateActivityLog } from './activity-log.js';
 import { resolveTaskRef, TASK_ID_RE } from './resolve-ref.js';
@@ -590,7 +590,8 @@ export function dispatchTaskControlOperation(context, operation) {
     }
     if (operation.family === 'task-finalization') {
         operationTaskId(context, operation.request.taskRef);
-        return applyTaskFinalization(operation.request, {
+        const finalize = context.source === 'sandbox-executor' ? commitPreparedTaskFinalization : applyTaskFinalization;
+        return finalize(operation.request, {
             repoRoot: context.repoRoot,
             ...(context.source === 'sandbox-executor' ? {
                 controlBinding: { generation: context.generation, requestId: context.requestId }

package/dist/lib/task/decision-intents.js

@@ -7,6 +7,7 @@ import { resolveTaskContext } from './resolve-ref.js';
 import { extractSection, extractSubSection, findSectionHeading } from './sections.js';
 import { captureTaskWriteMetadata, writeTask } from './write.js';
 import { allowsManualOverride } from './guard-override.js';
+import { consumeHumanDecisionReworkIntents, parseReworkIntentDocument, reworkIntentMutation } from './rework-intent.js';
 const DECISION_ALIASES = ['人工裁决', 'Human Rulings', 'Human Decisions', 'Human Decision'];
 const ACTIVITY_ALIASES = ['活动日志', 'Activity Log'];
 function failed(code, message, taskId = null, ledgerId = null) {
@@ -160,6 +161,17 @@ function applyHumanDecision(request, options = {}) {
         kind: 'section', aliases: ACTIVITY_ALIASES, heading: activity.heading,
         body: appendActivityEntry(activity, { time: metadata.timestamp, step: 'Human Decision', agent: 'human', note: `${row.id} decided → ${recordId}` })
     });
+    try {
+        const parsedRework = parseReworkIntentDocument(content);
+        if (!parsedRework.ok)
+            return failed('DECISION_DOCUMENT_INVALID', parsedRework.message, resolved.taskId, row.id);
+        const consumed = consumeHumanDecisionReworkIntents(parsedRework.intents, row.id, metadata.timestamp);
+        if (consumed.changed)
+            mutations.push(reworkIntentMutation(content, consumed.intents));
+    }
+    catch (error) {
+        return failed('DECISION_DOCUMENT_INVALID', error instanceof Error ? error.message : String(error), resolved.taskId, row.id);
+    }
     const writeResult = writeTask({ taskRef: resolved.taskId, expectedState: stateOverride ? resolved.state : 'active', mutations, dryRun: request.dryRun }, {
         ...options,
         taskLocation: { repoRoot: resolved.repoRoot, taskId: resolved.taskId, taskMdPath: resolved.taskMdPath, state: resolved.state },

package/dist/lib/task/delivery.js

@@ -3,8 +3,7 @@ import { spawnSync } from 'node:child_process';
 import path from 'node:path';
 import { parseTaskFrontmatter } from './frontmatter.js';
 import { resolveTaskRef } from './resolve-ref.js';
-import { resolveDeliveryTarget, resolveDiffBase, resolveTargetHead, validateBaseRef, validateRemote } from './delivery-target.js';
-import { extractReviewBaseline, extractReviewDiffBase, extractReviewTargetHead, extractReviewedHead, findAuthoritativeReviewCodeArtifact } from './review-fingerprint.js';
+import { resolveDeliveryTarget, validateBaseRef, validateRemote } from './delivery-target.js';
 import { captureTaskWriteMetadata, writeTask } from './write.js';
 import { withRepositoryMutationLock, withTaskExecutionLock } from './task-execution-lock.js';
 function classifyDeliveryState(localHead, remoteHead, lastDelivered) {
@@ -27,41 +26,6 @@ function gitValue(cwd, args) {
 function validateBranch(cwd, branch) {
     return git(cwd, ['check-ref-format', `refs/heads/${branch}`]).status === 0;
 }
-function validateReviewTargetCompatibility(repoRoot, taskDir, target, reviewedHead) {
-    const artifact = findAuthoritativeReviewCodeArtifact(taskDir);
-    if (!artifact.ok || !artifact.path)
-        return { ok: false, code: 'DELIVERY_REVIEW_REQUIRED', message: 'A current review-code artifact is required before delivery' };
-    let content;
-    try {
-        content = fs.readFileSync(artifact.path, 'utf8');
-    }
-    catch (error) {
-        return { ok: false, code: 'DELIVERY_REVIEW_REQUIRED', message: error instanceof Error ? error.message : String(error) };
-    }
-    const savedReviewedHead = extractReviewedHead(content) || extractReviewBaseline(content);
-    const savedTargetHead = extractReviewTargetHead(content);
-    const savedDiffBase = extractReviewDiffBase(content);
-    if (!savedReviewedHead || !savedTargetHead || !savedDiffBase) {
-        return { ok: false, code: 'DELIVERY_REVIEW_TARGET_INVALID', message: 'review-code must save reviewed head, target head, and diff base before delivery' };
-    }
-    if (savedReviewedHead !== reviewedHead) {
-        return { ok: false, code: 'DELIVERY_REVIEW_REQUIRED', message: 'review-code reviewed head does not match last_reviewed_commit' };
-    }
-    const saved = resolveDiffBase(repoRoot, reviewedHead, savedTargetHead);
-    if (!saved.ok || saved.diffBase !== savedDiffBase) {
-        return { ok: false, code: 'DELIVERY_REVIEW_TARGET_INVALID', message: saved.ok ? 'saved review diff base does not match saved target head' : saved.message };
-    }
-    const currentTarget = resolveTargetHead(repoRoot, target);
-    if (!currentTarget.ok)
-        return { ok: false, code: currentTarget.code, message: currentTarget.message };
-    const current = resolveDiffBase(repoRoot, reviewedHead, currentTarget.head);
-    if (!current.ok)
-        return { ok: false, code: current.code, message: current.message };
-    if (current.diffBase !== savedDiffBase) {
-        return { ok: false, code: 'DELIVERY_REVIEW_TARGET_CHANGED', message: 'current delivery target changed the reviewed diff base; re-run review-code' };
-    }
-    return { ok: true };
-}
 function failure(taskId, code, message, retryable = false, status = retryable ? 'blocked' : 'failed') {
     return { status, changed: false, taskId, remoteHead: null, localHead: null, state: null, error: { code, message, retryable } };
 }
@@ -102,12 +66,6 @@ function deliverUnlocked(taskId, taskDir, repoRoot, options) {
         return failure(taskId, 'DELIVERY_GIT_INSPECT_FAILED', status.stderr.trim() || 'Unable to inspect Git state');
     if (status.stdout.trim())
         return failure(taskId, 'DELIVERY_WORKTREE_DIRTY', 'Working tree must be clean before branch delivery');
-    const reviewed = frontmatter.last_reviewed_commit ?? '';
-    if (reviewed !== localHead)
-        return failure(taskId, 'DELIVERY_REVIEW_REQUIRED', 'Local HEAD must equal last_reviewed_commit before delivery');
-    const reviewTarget = validateReviewTargetCompatibility(repoRoot, taskDir, target.value, reviewed);
-    if (!reviewTarget.ok)
-        return failure(taskId, reviewTarget.code, reviewTarget.message, reviewTarget.code.includes('UNAVAILABLE'), reviewTarget.code.includes('UNAVAILABLE') ? 'blocked' : 'failed');
     const ref = `refs/heads/${branch}`;
     const remoteOutput = git(repoRoot, ['ls-remote', '--refs', target.value.remote, ref]);
     if (remoteOutput.status !== 0)