@fitlab-ai/agent-infra 0.11.0 → 0.11.1

package/dist/lib/task/review-finalization.js

@@ -69,7 +69,6 @@ function preflightReviewSummaryUnlocked(request, options = {}) {
     let taskContent;
     let content;
     let recovery;
-    let existingIntent;
     try {
         taskContent = fs.readFileSync(resolved.taskMdPath, 'utf8');
         if (request.recoveryId) {
@@ -89,7 +88,6 @@ function preflightReviewSummaryUnlocked(request, options = {}) {
                 return preflightFailed(request, validated.error.code === 'ARTIFACT_NOT_REGULAR' ? 'REVIEW_ARTIFACT_NOT_REGULAR' : 'REVIEW_ARTIFACT_IDENTITY_INVALID', validated.error.message, resolved.taskId);
             content = fs.readFileSync(validated.artifact.path, 'utf8');
             const intent = readArtifactRecoveryIntent(resolved.repoRoot, resolved.taskId, spec.family, request.artifact);
-            existingIntent = intent;
             if (intent?.state === 'preflight-ready')
                 recovery = recoveryContextFromIntent(resolved.repoRoot, resolved.taskDir, intent);
             if (intent?.state === 'consumed')
@@ -143,12 +141,6 @@ function preflightReviewSummaryUnlocked(request, options = {}) {
             ...(recovery ? { recovery: recoveryInfo(recovery) } : {})
         });
     }
-    if (!recovery && existingIntent?.state === 'passed') {
-        return {
-            ...preflightFailed(request, 'REVIEW_ARTIFACT_CONFLICT', '', resolved.taskId, { artifactSha256, semanticDigest }),
-            status: 'passed', changed: false, error: null
-        };
-    }
     if (request.dryRun || recovery && readArtifactRecoveryIntent(resolved.repoRoot, resolved.taskId, spec.family, request.artifact)?.state === 'preflight-ready') {
         return {
             ...preflightFailed(request, 'REVIEW_ARTIFACT_CONFLICT', '', resolved.taskId, { artifactSha256, semanticDigest, ...(recovery ? { recovery: recoveryInfo(recovery) } : {}) }),
@@ -316,7 +308,9 @@ function prepareReviewSummaryCandidate(request, artifactContent, options = {}) {
             ...(recovery ? { recovery } : {})
         };
     }
-    const transformed = finalizeReviewSummaryContent(artifactContent, stageStatus.unresolvedFindingCounts);
+    const transformed = finalizeReviewSummaryContent(artifactContent, stageStatus.unresolvedFindingCounts, {
+        refreshNumericCounts: Boolean(request.recoveryId && recovery)
+    });
     if (!transformed.ok)
         return reject(transformed.code, transformed.message);
     const finalDigests = { artifactSha256: sha256Content(transformed.content), semanticDigest: canonicalSemanticDigest(transformed.content) };

package/dist/lib/task/verification-engine.js

@@ -29,6 +29,7 @@ import { validateLocalArtifact } from "./local-artifact-finalization.js";
 import { validateQualificationAudit } from "./qualification-audit.js";
 import { getArtifactSchema } from "./artifact-schema.js";
 import { inspectArtifactContract } from "./artifact-operations.js";
+import { normalizeVerificationRecord } from './gate-policy.js';
 import { manualValidationFinalSummaryProjectionMatches } from "./manual-validation-receipt.js";
 import { readManualValidationCompletion } from "./manual-validation-completion.js";
 import { sha256File } from "./artifact-receipts.js";
@@ -1198,19 +1199,19 @@ function interpolate(template, taskDir, artifactFile) {
         .replace(/\{artifact-stem\}/g, artifactStem);
 }
 function summarizeGate(checks) {
-    if (checks.some((check) => check.status === "blocked")) {
+    if (checks.some((check) => check.effectiveStatus === "blocked")) {
         return "blocked";
     }
-    if (checks.some((check) => check.status === "fail")) {
+    if (checks.some((check) => check.effectiveStatus === "fail")) {
         return "fail";
     }
     return "pass";
 }
 function summarizeChecks(checks) {
     const counts = {
-        pass: checks.filter((check) => check.status === "pass").length,
-        fail: checks.filter((check) => check.status === "fail").length,
-        blocked: checks.filter((check) => check.status === "blocked").length
+        pass: checks.filter((check) => check.effectiveStatus === "pass").length,
+        fail: checks.filter((check) => check.effectiveStatus === "fail").length,
+        blocked: checks.filter((check) => check.effectiveStatus === "blocked").length
     };
     if (counts.blocked > 0) {
         return `${counts.pass} passed, ${counts.fail} failed, ${counts.blocked} blocked`;
@@ -1221,7 +1222,7 @@ function buildAction(gate, checks) {
     if (gate === "pass") {
         return "All declared checks passed";
     }
-    const firstFailure = checks.find((check) => check.status !== "pass");
+    const firstFailure = checks.find((check) => check.effectiveStatus !== "pass");
     if (!firstFailure) {
         return "Review validation output";
     }
@@ -1277,16 +1278,16 @@ async function verifyInProcess({ mode, skillName, taskDir, artifactFile, checks:
         for (const [type, checkConfig] of Object.entries(verifyConfig.checks || {})) {
             if (checkConfig === null)
                 continue;
-            const result = await runCheck(type, {
+            const result = normalizeVerificationRecord(await runCheck(type, {
                 mode,
                 skillName,
                 taskDir: path.resolve(taskDir),
                 artifactFile,
                 repositoryRoot: repoRoot,
                 config: checkConfig
-            }, shared);
-            checks.push(result);
-            if (result.status === "blocked")
+            }, shared));
+            checks.push(...(result.subchecks?.map(normalizeVerificationRecord) ?? [result]));
+            if (checks.at(-1)?.effectiveStatus === "blocked")
                 break;
         }
         const gate = summarizeGate(checks);
@@ -1298,15 +1299,25 @@ async function verifyInProcess({ mode, skillName, taskDir, artifactFile, checks:
         return { skill: skillName, ...failResult(type, `Unknown check type '${type}' for skill '${skillName}'.`) };
     if (config === null)
         return { skill: skillName, ...passResult(type, `Check '${type}' is disabled for skill '${skillName}'.`) };
+    const result = normalizeVerificationRecord(await runCheck(type, {
+        skillName,
+        taskDir: path.resolve(taskDir),
+        artifactFile,
+        repositoryRoot: repoRoot,
+        config
+    }, shared));
+    if (!result.subchecks?.length)
+        return { skill: skillName, ...result };
+    const checks = result.subchecks.map(normalizeVerificationRecord);
+    const gate = summarizeGate(checks);
     return {
         skill: skillName,
-        ...await runCheck(type, {
-            skillName,
-            taskDir: path.resolve(taskDir),
-            artifactFile,
-            repositoryRoot: repoRoot,
-            config
-        }, shared)
+        ...result,
+        status: gate,
+        effectiveStatus: gate,
+        reason: gate === 'pass' ? 'OK' : checks.find((check) => check.effectiveStatus !== 'pass')?.reason ?? 'CHECK_FAILED',
+        action: buildAction(gate, checks),
+        checks
     };
 }
 export { verifyInProcess };

package/dist/lib/task/verification.js

@@ -80,7 +80,7 @@ async function verifyTaskEvent(request, options = {}) {
     else {
         for (const check of spec.checks ?? []) {
             const payload = await engine({ mode: 'checks', skillName: spec.skill, taskDir: resolved.taskDir, artifactFile: request.artifact, checks: [check], repositoryRoot: resolved.repoRoot });
-            const status = payload.status;
+            const status = (payload.effectiveStatus ?? payload.status);
             if (status !== 'pass' && allowsManualOverride(options.manualOverride, 'verification-engine', status === 'blocked' ? 'CHECK_BLOCKED' : 'CHECK_FAILED')) {
                 invocations.push({ status: 'pass', exitCode: 0, payload: { ...payload, status: 'pass', humanOverride: 'human-approved', originalStatus: status } });
                 break;
@@ -103,19 +103,35 @@ function statusLabel(status) {
 function renderPayload(payload) {
     const lines = [];
     if (Array.isArray(payload.checks)) {
-        lines.push(`Verification: ${payload.gate} | Skill: ${payload.skill}`, '');
+        const gateMode = typeof payload.gate === 'string';
+        lines.push(gateMode
+            ? `Verification: ${payload.gate} | Skill: ${payload.skill}`
+            : `Check: ${payload.status} | Skill: ${payload.skill} | Type: ${payload.type}`, '');
         for (const check of payload.checks) {
-            lines.push(`  [${statusLabel(check.status)}] ${check.type} - ${check.message}`);
+            const raw = statusLabel(check.status);
+            const effective = statusLabel(check.effectiveStatus ?? check.status);
+            const reason = check.reason ?? 'CHECK_FAILED';
+            const message = check.message ?? reason;
+            lines.push(`  [${effective}] ${check.checkId ?? check.type} (${check.classification ?? 'hard'}; raw ${raw}; reason ${reason}) - ${message}; ${check.action ?? 'Review validation output'}`);
         }
-        lines.push('', `Result: ${payload.summary} - ${payload.action}`);
+        const summary = payload.summary ?? (() => {
+            const checks = payload.checks;
+            const statuses = checks.map((check) => check.effectiveStatus ?? check.status);
+            return `${statuses.filter((status) => status === 'pass').length} passed, ${statuses.filter((status) => status === 'fail').length} failed${statuses.includes('blocked') ? `, ${statuses.filter((status) => status === 'blocked').length} blocked` : ''}`;
+        })();
+        lines.push('', `Result: ${summary} - ${payload.action}`);
     }
     else {
         lines.push(`Check: ${payload.status} | Skill: ${payload.skill} | Type: ${payload.type}`, '');
-        lines.push(`  [${statusLabel(payload.status)}] ${payload.type} - ${payload.message}`);
-        const summary = payload.status === 'pass' ? '1 passed, 0 failed'
-            : payload.status === 'blocked' ? '0 passed, 0 failed, 1 blocked' : '0 passed, 1 failed';
-        const action = payload.status === 'pass' ? 'Requested check passed'
-            : payload.status === 'blocked' ? `Resolve blocked ${payload.type} check and re-run check`
+        const raw = statusLabel(payload.status);
+        const effective = statusLabel(payload.effectiveStatus ?? payload.status);
+        const reason = payload.reason ?? 'CHECK_FAILED';
+        lines.push(`  [${effective}] ${payload.checkId ?? payload.type} (${payload.classification ?? 'hard'}; raw ${raw}; reason ${reason}) - ${payload.message ?? reason}; ${payload.action ?? 'Review validation output'}`);
+        const status = payload.effectiveStatus ?? payload.status;
+        const summary = status === 'pass' ? '1 passed, 0 failed'
+            : status === 'blocked' ? '0 passed, 0 failed, 1 blocked' : '0 passed, 1 failed';
+        const action = status === 'pass' ? 'Requested check passed'
+            : status === 'blocked' ? `Resolve blocked ${payload.type} check and re-run check`
                 : `Fix ${payload.type} issues and re-run check`;
         lines.push('', `Result: ${summary} - ${action}`);
     }

package/dist/lifecycle-build-manifest.json

@@ -1,6 +1,6 @@
 {
   "protocolVersion": 3,
-  "packageVersion": "0.11.0",
-  "sourceInputHash": "c67e66a46040d5246d67c751af605b796cd61913b98add01ff327829c01ce3db",
-  "internalExecutableBuildHash": "f6b4be06df12111b3a7614d335c9e802049c3cea194c6e7c040f3251188c694f"
+  "packageVersion": "0.11.1",
+  "sourceInputHash": "0d66fb32bbdc0982d38a04b86b0278768bb6091eb945ebd90a432f9bae363712",
+  "internalExecutableBuildHash": "4b1edd20ed87d3d742e5e7975fd49b1ac1fb0ff14f837d52ee9ba3ab11318365"
 }

package/lib/internal/manual-validation.ts

@@ -44,6 +44,7 @@ function printFailure(format: OutputFormat, error: { code: string; message: stri
 }
 
 const MANUAL_VALIDATION_STATUS_SECTION = /^###\s+(?:⚠️\s+(?:需人工校验|Manual Validation Required)|✅\s+(?:人工验证已通过|无需人工校验|Manual Validation Passed|No Manual Validation Required)|⏳\s+(?:人工验证待收尾|Manual Validation Pending))\s*$[\s\S]*?(?=^#{1,3}\s|(?![\s\S]))/gmu;
+const TRAILING_CANONICAL_REPORT_PLACEHOLDER = /\n*<!--\s*canonical-pr-change-report\s*-->\s*$/u;
 
 function manualSummaryBody(body: string, phase: 'pending' | 'final', transactionId: string, receiptDigest = '', evidenceDigest = '', prHeadSha = ''): string {
   const chinese = /###\s+(?:⚠️\s+需人工校验|✅\s+(?:人工验证已通过|无需人工校验)|⏳\s+人工验证待收尾)\s*$/mu.test(body);
@@ -55,10 +56,13 @@ function manualSummaryBody(body: string, phase: 'pending' | 'final', transaction
       ? `### ✅ 人工验证已通过\n\n人工验证已通过；transaction=${transactionId}; receipt=${receiptDigest}; evidence=${evidenceDigest}; head=${prHeadSha}.`
       : `### ✅ Manual Validation Passed\n\nManual validation passed; transaction=${transactionId}; receipt=${receiptDigest}; evidence=${evidenceDigest}; head=${prHeadSha}.`;
   let inserted = false;
-  const updated = body.replace(MANUAL_VALIDATION_STATUS_SECTION, () => {
+  const updated = body.replace(MANUAL_VALIDATION_STATUS_SECTION, (matched) => {
     if (inserted) return '';
     inserted = true;
-    return `${section}\n\n`;
+    const trailingPlaceholder = TRAILING_CANONICAL_REPORT_PLACEHOLDER.exec(matched)?.[0];
+    return trailingPlaceholder
+      ? `${trailingPlaceholder.trim()}\n\n${section}\n\n`
+      : `${section}\n\n`;
   });
   return inserted ? `${updated.replace(/\s+$/u, '')}\n` : `${body.replace(/\s+$/u, '')}\n\n${section}\n`;
 }

package/lib/platform/issues.ts

@@ -574,9 +574,8 @@ async function syncPlatformIssue(taskRef: string, options: SyncOptions): Promise
   const failure = plan.operations.find((operation) => operation.status === 'failed');
   if (failure) return result('failed', base.resolved.taskId, base.issueNumber, { issue: snapshot, operations: plan.operations, error: { code: failure.reasonCode || 'ISSUE_SYNC_FAILED', message: `Operation ${failure.name} failed`, retryable: false } });
   const planned = plan.operations.filter((operation) => operation.status === 'planned');
-  const skipped = plan.operations.some((operation) => operation.status === 'skipped');
-  if (options.dryRun) return result(planned.length ? 'planned' : skipped ? 'degraded' : 'no-op', base.resolved.taskId, base.issueNumber, { issue: snapshot, operations: plan.operations, error: null });
-  if (planned.length === 0) return result(skipped ? 'degraded' : 'no-op', base.resolved.taskId, base.issueNumber, { issue: snapshot, operations: plan.operations, error: null });
+  if (options.dryRun) return result(planned.length ? 'planned' : 'no-op', base.resolved.taskId, base.issueNumber, { issue: snapshot, operations: plan.operations, error: null });
+  if (planned.length === 0) return result('no-op', base.resolved.taskId, base.issueNumber, { issue: snapshot, operations: plan.operations, error: null });
   const payload = applyRestOperations(snapshot, planned) as Partial<{
     title: string; body: string; labels: string[]; assignees: string[];
     milestone: string | null; state: 'open' | 'closed'; fields: Record<string, string | number | null>;
@@ -611,7 +610,7 @@ async function syncPlatformIssue(taskRef: string, options: SyncOptions): Promise
       error: { code: 'IN_LABEL_SYNC_PARTIAL', message: 'Issue labels did not converge after update', retryable: true }
     });
   }
-  return result(skipped ? 'degraded' : 'applied', base.resolved.taskId, base.issueNumber, {
+  return result('applied', base.resolved.taskId, base.issueNumber, {
     changed: updated.value.changed, platform: base.context.platform, capabilities: base.context.capabilities,
     resource: { kind: 'issue', number: base.issueNumber, identity: base.issueIdentity }, issue: finalIssue,
     operations: plan.operations.map((operation) => operation.status === 'planned' ? { ...operation, status: 'applied' } : operation), error: null