@fitlab-ai/agent-infra 0.9.14 → 0.10.0

package/dist/lib/task/pr-delivery-fact.js

@@ -1,4 +1,3 @@
-import { isLegacyCompatibilityEnabled, legacyCompatibilityError, VERSION } from '../version.js';
 import { isResourceIdentity, parseResourceIdentity, resourceIdentityNumber, serializeResourceIdentity } from '../platform/resource-identity.js';
 const PR_DELIVERY_FACT_KEY = 'pr_delivery_fact';
 function factError(message) {
@@ -28,11 +27,6 @@ function timestamp(value, label) {
         throw factError(`${label} must be a valid timestamp`);
     return new Date(result).toISOString();
 }
-function positiveNumber(value, label) {
-    if (!Number.isSafeInteger(value) || value <= 0)
-        throw factError(`${label} must be a positive integer`);
-    return value;
-}
 function parseRef(value, label) {
     if (!isRecord(value))
         throw factError(`${label} must be an object`);
@@ -51,36 +45,17 @@ function canonicalIdentity(value) {
         base: parseRef(value.base, 'identity.base')
     };
 }
-function decodeLegacyIdentity(value) {
-    if (!isRecord(value))
-        throw factError('legacy identity must be an object');
-    // TODO(compat): Remove the v1 identity decoder before the first stable v1.0.0 release.
-    exactKeys(value, ['repository', 'number', 'nodeId', 'url', 'head', 'base'], 'legacy identity');
-    const legacy = {
-        number: positiveNumber(value.number, 'legacy identity.number'),
-        repository: text(value.repository, 'identity.repository'),
-        nodeId: text(value.nodeId, 'legacy identity.nodeId'),
-        url: text(value.url, 'identity.url'),
-        head: parseRef(value.head, 'identity.head'),
-        base: parseRef(value.base, 'identity.base')
-    };
-    return {
-        resource: { kind: 'number', value: legacy.number },
-        repository: legacy.repository,
-        url: legacy.url,
-        head: legacy.head,
-        base: legacy.base
-    };
-}
 function provenanceForSource(source) { return source === 'created' ? 'create-post' : source === 'reused' ? 'reuse' : source; }
-function parseFact(value, runtimeVersion = VERSION) {
+function legacyFactError() {
+    return Object.assign(new Error('PLATFORM_IDENTITY_LEGACY_UNSUPPORTED: v1 PR delivery fact is unsupported; use the previous tool version to view this task or migrate its fact to version 2 before using this version'), { code: 'PLATFORM_IDENTITY_LEGACY_UNSUPPORTED' });
+}
+function parseFact(value) {
     if (!isRecord(value))
         throw factError('fact must be an object');
-    if ((value.version !== 1 && value.version !== 2) || typeof value.state !== 'string')
+    if (value.version === 1)
+        throw legacyFactError();
+    if (value.version !== 2 || typeof value.state !== 'string')
         throw factError('version or state is invalid');
-    if (value.version === 1 && !isLegacyCompatibilityEnabled(runtimeVersion)) {
-        throw legacyCompatibilityError('v1 PR delivery fact', runtimeVersion);
-    }
     if (value.state === 'unbound') {
         exactKeys(value, ['version', 'state', 'reason'], 'unbound fact');
         if (value.reason !== 'initial')
@@ -95,25 +70,6 @@ function parseFact(value, runtimeVersion = VERSION) {
     }
     if (value.state !== 'bound')
         throw factError('state is invalid');
-    if (value.version === 1) {
-        // TODO(compat): Remove this v1-to-v2 shape conversion before the first stable v1.0.0 release.
-        exactKeys(value, ['version', 'state', 'identity', 'binding', 'provenance'], 'legacy bound fact');
-        if (!isRecord(value.binding) || !isRecord(value.provenance))
-            throw factError('legacy binding or provenance is invalid');
-        exactKeys(value.binding, ['status', 'source', 'verifiedAt', 'issueNumber', 'remoteState', 'mergedAt', 'mergeCommitSha'], 'legacy binding');
-        const issueNumber = value.binding.issueNumber === null ? null : positiveNumber(value.binding.issueNumber, 'legacy binding.issueNumber');
-        return parseFact({
-            version: 2,
-            state: 'bound',
-            identity: decodeLegacyIdentity(value.identity),
-            binding: {
-                status: 'verified', source: value.binding.source, verifiedAt: timestamp(value.binding.verifiedAt, 'binding.verifiedAt'),
-                issueIdentity: issueNumber === null ? null : { kind: 'number', value: issueNumber }, remoteState: value.binding.remoteState,
-                mergedAt: value.binding.mergedAt === null ? null : timestamp(value.binding.mergedAt, 'binding.mergedAt'), mergeCommitSha: nullableText(value.binding.mergeCommitSha, 'binding.mergeCommitSha')
-            },
-            provenance: value.provenance
-        }, runtimeVersion);
-    }
     exactKeys(value, ['version', 'state', 'identity', 'binding', 'provenance'], 'bound fact');
     if (!isRecord(value.binding) || !isRecord(value.provenance))
         throw factError('binding or provenance is invalid');
@@ -142,7 +98,7 @@ function parseFact(value, runtimeVersion = VERSION) {
         provenance: { establishedBy: value.provenance.establishedBy }
     };
 }
-function decodePrDeliveryFact(value, runtimeVersion = VERSION) {
+function decodePrDeliveryFact(value, _runtimeVersion) {
     if (typeof value === 'string') {
         let decoded;
         try {
@@ -151,16 +107,16 @@ function decodePrDeliveryFact(value, runtimeVersion = VERSION) {
         catch {
             throw factError('value is not valid JSON');
         }
-        return parseFact(decoded, runtimeVersion);
+        return parseFact(decoded);
     }
-    return parseFact(value, runtimeVersion);
+    return parseFact(value);
 }
 function encodePrDeliveryFact(fact) { return JSON.stringify(parseFact(fact)); }
-function readPrDeliveryFact(metadata, runtimeVersion = VERSION) {
+function readPrDeliveryFact(metadata, _runtimeVersion) {
     if (!Object.hasOwn(metadata, PR_DELIVERY_FACT_KEY) || metadata[PR_DELIVERY_FACT_KEY] === '')
         return { status: 'missing', fact: null };
     try {
-        return { status: 'valid', fact: decodePrDeliveryFact(metadata[PR_DELIVERY_FACT_KEY], runtimeVersion) };
+        return { status: 'valid', fact: decodePrDeliveryFact(metadata[PR_DELIVERY_FACT_KEY]) };
     }
     catch (error) {
         return { status: 'invalid', fact: null, error: asFactError(error) };

package/dist/lib/task/qualification-audit.js

@@ -1,6 +1,8 @@
 import { createHash } from 'node:crypto';
+import { parseArtifactName } from './artifact-name.js';
 import { parseTypedTaskFrontmatter } from './frontmatter.js';
-import { parseTable } from './sections.js';
+import { findSectionRange, parseTable } from './sections.js';
+import { scanVisibleMarkdown } from './markdown.js';
 const TASK_CONSTRAINT_HEADINGS = ['约束', 'Constraints'];
 const TASK_CANDIDATE_HEADINGS = ['候选与否决方案', 'Candidate and Rejected Options', 'Candidates and Rejected Options', 'Candidates and Rejected Alternatives'];
 const AUDIT_HEADINGS = ['资格审计', '资格审计复核', 'Qualification Audit', 'Qualification Audit Review'];
@@ -72,21 +74,8 @@ function taskInputDigest(qualification) {
     return digest(canonicalTaskInput(qualification));
 }
 function sectionBody(content, aliases, level = 2) {
-    const heading = aliases.map((value) => value.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')).join('|');
-    const marker = '#'.repeat(level);
-    const match = new RegExp(`^${marker}\\s+(${heading})\\s*$`, 'm').exec(content);
-    if (!match)
-        return null;
-    const rest = content.slice((match.index ?? 0) + match[0].length);
-    const end = rest.search(new RegExp(`^#{2,${level}}\\s+`, 'm'));
-    return rest.slice(0, end < 0 ? rest.length : end);
-}
-function stripSection(content, aliases) {
-    const body = sectionBody(content, aliases);
-    if (body === null)
-        return normalizeText(content);
-    const heading = aliases.map((value) => value.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')).join('|');
-    return normalizeText(content.replace(new RegExp(`^##\\s+(?:${heading})\\s*$[\\s\\S]*?(?=^##\\s+|(?![\\s\\S]))`, 'm'), ''));
+    const section = findSectionRange(content, aliases, level);
+    return section ? content.slice(section.bodyStart, section.end) : null;
 }
 function stripFrontmatter(content) {
     const opening = /^---(?:\r?\n)/.exec(content);
@@ -119,7 +108,7 @@ function nonConstraintInputDigest(content) {
         ['返工意图', 'Rework Intents'],
         CONFIRMATION_HEADINGS,
         ['完成检查清单', 'Completion Checklist']
-    ].reduce((value, aliases) => stripSection(value, aliases), projection);
+    ].reduce((value, aliases) => stripSectionAtLevel(value, aliases, 2), projection);
     return digest(projection);
 }
 function taskInputSection(content, aliases) {
@@ -130,12 +119,8 @@ function stripTaskInputSection(content, aliases) {
     return stripSectionAtLevel(result, aliases, 2);
 }
 function stripSectionAtLevel(content, aliases, level) {
-    const body = sectionBody(content, aliases, level);
-    if (body === null)
-        return normalizeText(content);
-    const heading = aliases.map((value) => value.replace(/[.*+?^${}()|[\]\\]/g, '\\$&')).join('|');
-    const marker = '#'.repeat(level);
-    return normalizeText(content.replace(new RegExp(`^${marker}\\s+(?:${heading})\\s*$[\\s\\S]*?(?=^#{2,${level}}\\s+|(?![\\s\\S]))`, 'm'), ''));
+    const section = findSectionRange(content, aliases, level);
+    return normalizeText(section ? content.slice(0, section.start) + content.slice(section.end) : content);
 }
 function parseTaskQualification(content) {
     const constraintBody = taskInputSection(content, TASK_CONSTRAINT_HEADINGS);
@@ -144,12 +129,14 @@ function parseTaskQualification(content) {
     const hasCandidateHeading = candidateBody !== null;
     if ((!hasConstraintHeading && !hasCandidateHeading)
         || (hasConstraintHeading && hasCandidateHeading
-            && !/^\s*\|/m.test(constraintBody) && !/^\s*\|/m.test(candidateBody))) {
+            && !scanVisibleMarkdown(constraintBody).lines.some((line) => line.text.trim().startsWith('|'))
+            && !scanVisibleMarkdown(candidateBody).lines.some((line) => line.text.trim().startsWith('|')))) {
         return { ok: true, qualification: { present: false, constraints: [], candidates: [], constraintDigest: digest([]), taskInputDigest: digest({ constraints: [], candidates: [] }), nonConstraintInputDigest: nonConstraintInputDigest(content) } };
     }
     if (!hasConstraintHeading || !hasCandidateHeading) {
-        const hasCanonicalTable = new RegExp(`\\|\\s*${CONSTRAINT_COLUMNS.join('\\s*\\|\\s*')}\\s*\\|`).test(content)
-            || new RegExp(`\\|\\s*${CANDIDATE_COLUMNS.join('\\s*\\|\\s*')}\\s*\\|`).test(content);
+        const visible = scanVisibleMarkdown(content).lines.map((line) => line.text).join('\n');
+        const hasCanonicalTable = new RegExp(`\\|\\s*${CONSTRAINT_COLUMNS.join('\\s*\\|\\s*')}\\s*\\|`).test(visible)
+            || new RegExp(`\\|\\s*${CANDIDATE_COLUMNS.join('\\s*\\|\\s*')}\\s*\\|`).test(visible);
         if (hasCanonicalTable)
             return { ok: false, code: 'QUALIFICATION_TASK_CONTRACT_INVALID', message: 'task qualification requires both constraints and candidates sections' };
         return { ok: true, qualification: { present: false, constraints: [], candidates: [], constraintDigest: digest([]), taskInputDigest: digest({ constraints: [], candidates: [] }), nonConstraintInputDigest: nonConstraintInputDigest(content) } };
@@ -228,17 +215,16 @@ function parseAuditTable(content, heading, columns) {
     const table = parseTable(`## Qualification Audit Body\n${subsection}`, { sectionAliases: ['Qualification Audit Body'], columns });
     return table?.rows.map((row) => ({ ...row.values })) ?? null;
 }
-function parseArtifactName(value) {
-    const match = /^(analysis|review-analysis|plan|review-plan|code|review-code)(?:-r([2-9]|[1-9]\d+))?\.md$/.exec(value);
-    if (!match)
-        return null;
-    return { family: match[1], round: match[2] ? Number(match[2]) : 1 };
+function parseAuditArtifactName(value) {
+    const identity = parseArtifactName(value);
+    return identity && ARTIFACT_FAMILIES.includes(identity.family)
+        ? { family: identity.family, round: identity.round } : null;
 }
 function upstreamArtifactDigest(rows) {
     return digest([...rows].map((row) => ({ ...row })).sort((a, b) => `${a.upstreamFamily}/${a.upstreamArtifact}/${a.relation}`.localeCompare(`${b.upstreamFamily}/${b.upstreamArtifact}/${b.relation}`)));
 }
 function validateUpstreamRelation(row) {
-    const identity = parseArtifactName(row.upstreamArtifact);
+    const identity = parseAuditArtifactName(row.upstreamArtifact);
     if (!ARTIFACT_FAMILIES.includes(row.upstreamFamily) || !identity || identity.family !== row.upstreamFamily
         || identity.round !== row.upstreamRound || !Number.isSafeInteger(row.upstreamRound)
         || !/^[a-f0-9]{64}$/i.test(row.upstreamSha256) || !RELATIONS.includes(row.relation)) {
@@ -283,7 +269,7 @@ function expectedQualificationRelations(taskContent, family) {
     const inputSha256 = family === 'code' ? frontmatter.code_input_sha256 : family.startsWith('review-') ? frontmatter.review_input_sha256 : '';
     if (typeof inputName !== 'string' || !inputName || typeof inputSha256 !== 'string' || !inputSha256)
         return { ok: true, relations: undefined };
-    const identity = parseArtifactName(inputName);
+    const identity = parseAuditArtifactName(inputName);
     if (!identity)
         return { ok: false, code: 'QUALIFICATION_STARTED_INPUT_INVALID', message: `started input '${inputName}' is not canonical` };
     const relation = {
@@ -444,7 +430,7 @@ function validateQualificationAudit(taskContent, artifactContent, options = {})
         || upstreamArtifactDigest(options.expectedUpstreamRelations) !== upstreamArtifactDigest(audit.audit.upstreamRelations)))
         return { ok: false, code: 'QUALIFICATION_UPSTREAM_RELATION_MISMATCH', message: 'qualification audit upstream relations do not match the started artifact inputs' };
     if (options.family && options.artifact) {
-        const identity = parseArtifactName(options.artifact);
+        const identity = parseAuditArtifactName(options.artifact);
         if (!identity || identity.family !== options.family)
             return { ok: false, code: 'QUALIFICATION_ARTIFACT_IDENTITY_INVALID', message: `artifact '${options.artifact}' is not canonical for '${options.family}'` };
     }

package/dist/lib/task/recovery-warning.js

@@ -0,0 +1,19 @@
+function warningText(value) {
+    return typeof value === 'string' && value.length > 0 && value.trim() === value && !/[\r\n]/u.test(value);
+}
+export function isRecoveryWarning(value) {
+    if (!value || typeof value !== 'object' || Array.isArray(value))
+        return false;
+    const warning = value;
+    return Object.keys(warning).sort().join(',') === 'action,code,message'
+        && warningText(warning.code)
+        && warningText(warning.message)
+        && warningText(warning.action);
+}
+export function sameRecoveryWarning(left, right) {
+    return isRecoveryWarning(left) && isRecoveryWarning(right)
+        && left.code === right.code
+        && left.message === right.message
+        && left.action === right.action;
+}
+//# sourceMappingURL=recovery-warning.js.map

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

@@ -1,6 +1,6 @@
 import fs from 'node:fs';
 import path from 'node:path';
-import { scanVisibleMarkdown } from './decision-details.js';
+import { scanVisibleMarkdown } from './markdown.js';
 const SUMMARY_HEADINGS = new Set(['审查摘要', 'Review Summary']);
 const PLACEHOLDERS = [
     '{unresolved-blockers}',
@@ -10,22 +10,6 @@ const PLACEHOLDERS = [
 function escapeRegExp(value) {
     return value.replace(/[.*+?^${}()|[\]\\]/g, '\\$&');
 }
-function maxRound(entries, stem) {
-    let max = 0;
-    for (const entry of entries) {
-        if (entry === `${stem}.md`) {
-            max = Math.max(max, 1);
-            continue;
-        }
-        const match = entry.match(new RegExp(`^${escapeRegExp(stem)}-r(\\d+)\\.md$`));
-        if (match)
-            max = Math.max(max, Number(match[1]));
-    }
-    return max;
-}
-function artifactName(stem, round) {
-    return round === 1 ? `${stem}.md` : `${stem}-r${round}.md`;
-}
 function normalizeVerdict(raw) {
     const value = String(raw).trim().toLowerCase();
     if (value === '通过' || value === 'approved')
@@ -199,5 +183,5 @@ function parseVerdict(reviewPath) {
     }
     return resolveCanonicalVerdict(parsed.summary);
 }
-export { artifactName, equalCounts, escapeRegExp, extractSection, finalizeReviewSummaryContent, maxRound, normalizeVerdict, parseReviewSummary, parseVerdict, resolveCanonicalVerdict };
+export { equalCounts, escapeRegExp, extractSection, finalizeReviewSummaryContent, normalizeVerdict, parseReviewSummary, parseVerdict, resolveCanonicalVerdict };
 //# sourceMappingURL=review-artifacts.js.map

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

@@ -0,0 +1,26 @@
+export function parseReviewCommand(args) {
+    if (!args[0] || args[1] !== 'finalize-summary')
+        throw new Error('task ref and finalize-summary are required');
+    const values = {};
+    const flags = {
+        '--stage': 'stage', '--artifact': 'artifact', '--orchestrated': 'orchestrated', '--dry-run': 'dryRun',
+        '--recovery-id': 'recoveryId',
+        '--override-ticket': 'overrideTicket', '--override-target': 'overrideTarget', '--override-scope': 'overrideScope'
+    };
+    for (let index = 2; index < args.length; index += 1) {
+        const flag = args[index];
+        if (!Object.hasOwn(flags, flag))
+            throw new Error(`unknown option '${flag}'`);
+        const key = flags[flag];
+        if (Object.hasOwn(values, key))
+            throw new Error(`duplicate option '${flag}'`);
+        const value = flag === '--orchestrated' || flag === '--dry-run' ? true : args[++index];
+        if (!value || typeof value === 'string' && value.startsWith('--'))
+            throw new Error(`option '${flag}' requires a value`);
+        values[key] = value;
+    }
+    if (!values.stage || !values.artifact)
+        throw new Error("options '--stage' and '--artifact' are required");
+    return { taskRef: args[0], ...values };
+}
+//# sourceMappingURL=review-command.js.map