@fitlab-ai/agent-infra 0.9.10 → 0.9.12

package/dist/lib/platform/pr-checks.js

@@ -1,11 +1,12 @@
 import fs from 'node:fs';
-import { parseTaskFrontmatter } from '../task/frontmatter.js';
+import { parseTypedTaskFrontmatter } from '../task/frontmatter.js';
 import { resolveTaskRef } from '../task/resolve-ref.js';
 import { inspectPlatformRequiredChecks, registerPlatformCapabilities } from './adapters.js';
 import { resolvePlatformContext } from './context.js';
 import { createGitHubClient } from './github-client.js';
 import { inspectPlatformPullRequest } from './pull-requests.js';
 import { platformResult } from './types.js';
+import { readPrDeliveryFact } from '../task/pr-delivery-fact.js';
 function classifyRequiredChecks(required) {
     if (required.length === 0)
         return { state: 'no-required', required };
@@ -97,11 +98,13 @@ function resolvedTask(taskRef, options) {
     const resolved = resolveTaskRef(taskRef, options.cwd ? { repoRoot: options.cwd } : {});
     if (!resolved.ok)
         return { ok: false, output: checksResult('failed', { error: { code: resolved.code, message: resolved.message, retryable: false } }) };
-    const frontmatter = parseTaskFrontmatter(fs.readFileSync(resolved.taskMdPath, 'utf8'));
-    const pr = Number(frontmatter.pr_number);
-    const prNumber = Number.isInteger(pr) && pr > 0 ? pr : null;
+    const frontmatter = parseTypedTaskFrontmatter(fs.readFileSync(resolved.taskMdPath, 'utf8'));
+    const fact = readPrDeliveryFact(frontmatter);
+    if (fact.status === 'invalid')
+        return { ok: false, output: checksResult('failed', { error: { code: 'PR_DELIVERY_FACT_INVALID', message: fact.error.message, retryable: false } }) };
+    const prNumber = fact.status === 'valid' && fact.fact.state === 'bound' ? fact.fact.identity.number : null;
     if (!prNumber)
-        return { ok: false, output: checksResult('failed', { error: { code: 'PR_NOT_LINKED', message: 'Task has no valid pr_number', retryable: false } }) };
+        return { ok: false, output: checksResult('failed', { error: { code: fact.status === 'missing' ? 'PR_DELIVERY_FACT_MISSING' : 'PR_NOT_LINKED', message: 'Task has no verified bound pull request', retryable: false } }) };
     const client = options.client || createGitHubClient();
     const context = resolvePlatformContext({ cwd: resolved.repoRoot, client });
     if (!context.platform.repository || !['no-op', 'degraded'].includes(context.status))

package/dist/lib/platform/pr-summary.js

@@ -1,13 +1,14 @@
 import fs from 'node:fs';
 import { execFileSync } from 'node:child_process';
 import { enumerateArtifacts } from '../task/artifacts.js';
-import { parseTaskFrontmatter } from '../task/frontmatter.js';
+import { parseTypedTaskFrontmatter } from '../task/frontmatter.js';
 import { renderHumanOverrideAudit } from '../task/human-override.js';
 import { resolveTaskRef } from '../task/resolve-ref.js';
 import { resolvePlatformContext } from './context.js';
 import { createGitHubClient } from './github-client.js';
 import { listRemoteComments, normalizeCommentContent, writeComment } from './issue-comments.js';
 import { platformResult } from './types.js';
+import { readPrDeliveryFact } from '../task/pr-delivery-fact.js';
 function warningResultForPrimary(primaryResult) {
     if (primaryResult === 'pr_created')
         return 'pr_created_with_warnings';
@@ -52,9 +53,9 @@ function summaryContext(taskRef, options = {}) {
     const resolved = resolveTaskRef(taskRef, options.cwd ? { repoRoot: options.cwd } : {});
     if (!resolved.ok)
         return { ...platformResult('failed', { error: { code: resolved.code, message: resolved.message, retryable: false } }), task: { id: resolved.taskId, prNumber: null }, artifacts: [] };
-    const frontmatter = parseTaskFrontmatter(fs.readFileSync(resolved.taskMdPath, 'utf8'));
-    const value = Number(frontmatter.pr_number);
-    const prNumber = Number.isInteger(value) && value > 0 ? value : null;
+    const frontmatter = parseTypedTaskFrontmatter(fs.readFileSync(resolved.taskMdPath, 'utf8'));
+    const fact = readPrDeliveryFact(frontmatter);
+    const prNumber = fact.status === 'valid' && fact.fact.state === 'bound' ? fact.fact.identity.number : null;
     return { ...platformResult('no-op'), task: { id: resolved.taskId, prNumber }, artifacts: canonicalArtifacts(resolved.taskDir) };
 }
 function syncPullRequestSummary(taskRef, options) {
@@ -90,10 +91,13 @@ function syncPullRequestSummary(taskRef, options) {
     const resolved = resolveTaskRef(taskRef, options.cwd ? { repoRoot: options.cwd } : {});
     if (!resolved.ok)
         return softenFailure(platformResult('failed', { error: { code: resolved.code, message: resolved.message, retryable: false } }));
-    const frontmatter = parseTaskFrontmatter(fs.readFileSync(resolved.taskMdPath, 'utf8'));
-    const prNumber = Number(frontmatter.pr_number);
-    if (!Number.isInteger(prNumber) || prNumber <= 0)
-        return softenFailure(platformResult('failed', { error: { code: 'PR_NOT_LINKED', message: 'Task has no valid pr_number', retryable: false } }));
+    const frontmatter = parseTypedTaskFrontmatter(fs.readFileSync(resolved.taskMdPath, 'utf8'));
+    const fact = readPrDeliveryFact(frontmatter);
+    if (fact.status === 'invalid')
+        return softenFailure(platformResult('failed', { error: { code: 'PR_DELIVERY_FACT_INVALID', message: fact.error.message, retryable: false } }));
+    const prNumber = fact.status === 'valid' && fact.fact.state === 'bound' ? fact.fact.identity.number : null;
+    if (!prNumber)
+        return softenFailure(platformResult('failed', { error: { code: fact.status === 'missing' ? 'PR_DELIVERY_FACT_MISSING' : 'PR_NOT_LINKED', message: 'Task has no verified bound pull request', retryable: false } }));
     knownPrNumber = prNumber;
     const client = options.client || createGitHubClient();
     const context = resolvePlatformContext({ cwd: resolved.repoRoot, client });