@fitlab-ai/agent-infra 0.9.14 → 0.9.15

package/dist/lib/platform/github-provider.js

@@ -1,3 +1,4 @@
+import { ISSUE_FIELDS_QUERY, graphState, inspectGitHubIssue, inspectGitHubPullRequest, inspectGitHubIssueClosingChangeRequests, inspectGitHubRequiredChecks, parseRunJobIdentity, fetchCheckLogText } from './github-data.js';
 import { execFileSync } from 'node:child_process';
 import fs from 'node:fs';
 import os from 'node:os';
@@ -8,7 +9,6 @@ import { resourceIdentityNumber, resourceIdentityString } from './resource-ident
 import { extractPullRequestFileNames, syncLabelDelta } from './in-label-sync.js';
 const CURRENT_USER_QUERY = 'query { viewer { login } }';
 const ISSUE_TYPES_QUERY = `query($owner:String!){organization(login:$owner){issueTypes(first:20){nodes{id name pinnedFields{__typename ... on IssueFieldSingleSelect{id name options{id name}} ... on IssueFieldDate{id name} ... on IssueFieldText{id name} ... on IssueFieldNumber{id name}}}}}}`;
-const ISSUE_FIELDS_QUERY = `query($owner:String!,$name:String!,$number:Int!){repository(owner:$owner,name:$name){issue(number:$number){id issueType{id name pinnedFields{__typename ... on IssueFieldSingleSelect{id name options{id name}} ... on IssueFieldDate{id name} ... on IssueFieldText{id name} ... on IssueFieldNumber{id name}}} issueFieldValues(first:50){nodes{__typename ... on IssueFieldSingleSelectValue{name optionId field{... on IssueFieldSingleSelect{id name}}} ... on IssueFieldDateValue{value field{... on IssueFieldDate{id name}}} ... on IssueFieldTextValue{value field{... on IssueFieldText{id name}}} ... on IssueFieldNumberValue{value field{... on IssueFieldNumber{id name}}}}}}}}`;
 const CLOSING_ISSUES_QUERY = 'query($owner:String!,$name:String!,$number:Int!,$cursor:String){repository(owner:$owner,name:$name){pullRequest(number:$number){closingIssuesReferences(first:100,after:$cursor){nodes{number} pageInfo{hasNextPage endCursor}}}}}';
 function parseGitHubRemote(remote) {
     const trimmed = remote.trim().replace(/\.git$/, '');
@@ -399,11 +399,10 @@ function createGitHubOperations(client) {
             const number = resourceIdentityNumber(target);
             if (!number)
                 return invalid('ISSUE_NUMBER_INVALID', 'Issue number must be positive');
-            const module = await import('./issues.js');
-            const fetched = module.inspectGitHubIssue(client, repository(context), number, context.workingDirectory);
+            const fetched = inspectGitHubIssue(client, repository(context), number, context.workingDirectory);
             if (!fetched.ok)
                 return fetched;
-            const graph = module.graphState(client, repository(context), number, context.workingDirectory);
+            const graph = graphState(client, repository(context), number, context.workingDirectory);
             return { ok: true, value: issueSnapshot(fetched.value, graph || undefined) };
         },
         async create({ context, desired }) {
@@ -606,8 +605,7 @@ function createGitHubOperations(client) {
             const number = resourceIdentityNumber(target);
             if (!number)
                 return invalid('PR_NUMBER_INVALID', 'Pull request number must be positive');
-            const module = await import('./pull-requests.js');
-            const fetched = module.inspectGitHubPullRequest(client, repository(context), number, context.workingDirectory);
+            const fetched = inspectGitHubPullRequest(client, repository(context), number, context.workingDirectory);
             if (!fetched.ok)
                 return fetched;
             return { ok: true, value: changeRequestSnapshot(fetched.value) };
@@ -671,8 +669,7 @@ function createGitHubOperations(client) {
             const number = resourceIdentityNumber(issue);
             if (!number)
                 return invalid('ISSUE_NUMBER_INVALID', 'Issue number must be positive');
-            const module = await import('./pull-requests.js');
-            const fetched = module.inspectGitHubIssueClosingChangeRequests(client, repository(context), number, context.workingDirectory);
+            const fetched = inspectGitHubIssueClosingChangeRequests(client, repository(context), number, context.workingDirectory);
             if (!fetched.ok)
                 return fetched;
             return { ok: true, value: fetched.value.map(changeRequestSnapshot) };
@@ -756,8 +753,7 @@ function createGitHubOperations(client) {
             const number = resourceIdentityNumber(target);
             if (!number)
                 return invalid('PR_NUMBER_INVALID', 'Pull request number must be positive');
-            const module = await import('./github-provider.js');
-            return module.resolveGitHubChangeRequestGitEvidence({
+            return resolveGitHubChangeRequestGitEvidence({
                 cwd: context.workingDirectory,
                 repository: repository(context),
                 number,
@@ -771,18 +767,13 @@ function createGitHubOperations(client) {
             const number = resourceIdentityNumber(changeRequest);
             if (!number)
                 return invalid('PR_NUMBER_INVALID', 'Pull request number must be positive');
-            const module = await import('./pr-checks.js');
-            const fetched = module.inspectGitHubRequiredChecks(client, repository(context), number, context.workingDirectory);
-            if (!fetched.ok)
-                return fetched;
-            return { ok: true, value: fetched.value.map((check) => ({ name: check.name, status: check.bucket, conclusion: check.conclusion, detailsUrl: check.detailsUrl })) };
+            return inspectGitHubRequiredChecks(client, repository(context), number, context.workingDirectory);
         },
         async resolveRun({ context, changeRequest, checkName, detailsUrl }) {
             const number = resourceIdentityNumber(changeRequest);
             if (!number)
                 return invalid('PR_NUMBER_INVALID', 'Pull request number must be positive');
-            const module = await import('./pr-checks.js');
-            const direct = detailsUrl ? module.parseRunJobIdentity(detailsUrl) : null;
+            const direct = detailsUrl ? parseRunJobIdentity(detailsUrl) : null;
             if (direct) {
                 const run = client.json(['api', `repos/${repository(context)}/actions/runs/${direct.runId}`], { cwd: context.workingDirectory });
                 if (!run.ok)
@@ -798,11 +789,10 @@ function createGitHubOperations(client) {
                 : invalid('CHECK_RUN_NOT_FOUND', 'Check run was not found');
         },
         async fetchLogs({ context, runId, jobId }) {
-            const module = await import('./pr-checks.js');
             const args = jobId
                 ? ['api', `repos/${repository(context)}/actions/jobs/${jobId}/logs`]
                 : ['run', 'view', runId, '--repo', repository(context), '--log-failed'];
-            const fetched = module.fetchCheckLogText(client, args, context.workingDirectory);
+            const fetched = fetchCheckLogText(client, args, context.workingDirectory);
             if (!fetched.ok)
                 return fetched;
             return { ok: true, value: { runId, jobId, text: fetched.value || '' } };
@@ -883,7 +873,12 @@ function createGitHubOperations(client) {
                     }
                     continue;
                 }
-                const response = client.json(['api', '--method', 'POST', `repos/${repository(context)}/milestones`, '-f', `title=${item.title}`, '-f', `description=${item.description}`], { cwd: context.workingDirectory });
+                const response = client.json([
+                    'api', '--method', 'POST', `repos/${repository(context)}/milestones`,
+                    '-f', `title=${item.title}`,
+                    '-f', `description=${item.description}`,
+                    '-f', `state=${item.state}`
+                ], { cwd: context.workingDirectory });
                 if (!response.ok)
                     return response;
                 created.push(item.title);

package/dist/lib/platform/issue-comments.js

@@ -7,7 +7,7 @@ import { platformResult } from './types.js';
 import { providerError, providerOperationContext, providerStatus, providerResourceToken, unsupportedProviderOperation } from './provider-bridge.js';
 import { resourceIdentityNumber } from './resource-identity.js';
 import { taskIssueIdentity, taskIssueIdentityError } from './task-identities.js';
-import { CONTROL_MARKER_PATTERN, escapeHtmlText, fenceRanges, renderSafeCodeFence, sanitizeMarkdownDocument } from './comment-safety.js';
+import { canonicalizeCommentBody, escapeHtmlText, fenceRanges, renderSafeCodeFence } from './comment-safety.js';
 function providerCommentId(id, provider) {
     return provider.identity?.comment === 'number' && /^\d+$/.test(id) ? Number(id) : id;
 }
@@ -39,7 +39,7 @@ function splitFrontmatter(content) {
     return { frontmatter: match[1], body: content.slice(match[0].length).replace(/^\r?\n/, '') };
 }
 function sanitizeCommentBody(value, label) {
-    const result = sanitizeMarkdownDocument(value, { reservedMarkers: [CONTROL_MARKER_PATTERN] });
+    const result = canonicalizeCommentBody(value);
     if (!result.ok)
         throw new Error(`${result.error.code}: ${label}: ${result.error.message} at offset ${result.error.offset}`);
     return result.value;

package/dist/lib/platform/issues.js

@@ -14,8 +14,6 @@ import { labelDelta, planInLabelUpdate, syncLabelDelta, validateInLabelMapping,
 import { providerError, providerOperationContext, providerStatus, providerResourceIdentity, providerResourceToken, unsupportedProviderOperation } from './provider-bridge.js';
 import { resourceIdentityEquals, resourceIdentityNumber, resourceIdentityString, serializeResourceIdentity } from './resource-identity.js';
 import { taskIssueIdentity, taskIssueIdentityError } from './task-identities.js';
-const ISSUE_TYPES_QUERY = `query($owner:String!){organization(login:$owner){issueTypes(first:20){nodes{id name pinnedFields{__typename ... on IssueFieldSingleSelect{id name options{id name}} ... on IssueFieldDate{id name} ... on IssueFieldText{id name} ... on IssueFieldNumber{id name}}}}}}`;
-const ISSUE_FIELDS_QUERY = `query($owner:String!,$name:String!,$number:Int!){repository(owner:$owner,name:$name){issue(number:$number){id issueType{id name pinnedFields{__typename ... on IssueFieldSingleSelect{id name options{id name}} ... on IssueFieldDate{id name} ... on IssueFieldText{id name} ... on IssueFieldNumber{id name}}} issueFieldValues(first:50){nodes{__typename ... on IssueFieldSingleSelectValue{name optionId field{... on IssueFieldSingleSelect{id name}}} ... on IssueFieldDateValue{value field{... on IssueFieldDate{id name}}} ... on IssueFieldTextValue{value field{... on IssueFieldText{id name}}} ... on IssueFieldNumberValue{value field{... on IssueFieldNumber{id name}}}}}}}}`;
 function result(status, taskId, issueNumber, overrides = {}) {
     return {
         ...platformResult(status),
@@ -133,115 +131,6 @@ async function inspectExternalIssue(base, issueIdentity) {
         error: null
     });
 }
-function normalizeIssue(remote, repository, fallbackNumber) {
-    const number = Number.isInteger(remote.number) && Number(remote.number) > 0
-        ? Number(remote.number)
-        : fallbackNumber;
-    if (!number || !Number.isSafeInteger(number) || remote.pull_request)
-        return null;
-    return {
-        repository,
-        number,
-        databaseId: Number.isInteger(remote.id) ? Number(remote.id) : null,
-        nodeId: remote.node_id || `issue-${number}`,
-        url: remote.html_url || `https://github.com/${repository}/issues/${number}`,
-        state: String(remote.state || '').toLowerCase() === 'closed' ? 'closed' : 'open',
-        title: remote.title || '',
-        body: remote.body || '',
-        labels: (remote.labels || []).map((label) => typeof label === 'string' ? label : label.name || '').filter(Boolean).sort(),
-        assignees: (remote.assignees || []).map((assignee) => assignee.login || '').filter(Boolean).sort(),
-        milestone: remote.milestone?.title || null,
-        issueType: remote.type?.name || null,
-        fields: {}
-    };
-}
-function fieldKind(value) {
-    if (value.__typename === 'IssueFieldSingleSelect' || value.__typename === 'IssueFieldSingleSelectValue')
-        return 'single-select';
-    if (value.__typename === 'IssueFieldDate' || value.__typename === 'IssueFieldDateValue')
-        return 'date';
-    if (value.__typename === 'IssueFieldText' || value.__typename === 'IssueFieldTextValue')
-        return 'text';
-    if (value.__typename === 'IssueFieldNumber' || value.__typename === 'IssueFieldNumberValue')
-        return 'number';
-    return null;
-}
-function normalizeFieldSchemas(value) {
-    if (!Array.isArray(value))
-        return [];
-    return value.flatMap((raw) => {
-        const item = raw;
-        const kind = fieldKind(item);
-        return item.id && item.name && kind ? [{
-                id: item.id,
-                name: item.name,
-                kind,
-                options: (item.options || []).flatMap((option) => option.id && option.name ? [{ id: option.id, name: option.name }] : [])
-            }] : [];
-    });
-}
-function normalizeIssueTypes(value) {
-    const nodes = value?.data?.organization?.issueTypes?.nodes;
-    return (nodes || []).flatMap((raw) => {
-        const item = raw;
-        return item.id && item.name ? [{ id: item.id, name: item.name, fields: normalizeFieldSchemas(item.pinnedFields) }] : [];
-    });
-}
-function normalizeCurrentFields(value) {
-    const issue = value?.data?.repository?.issue;
-    const type = issue?.issueType?.name ? {
-        id: issue.issueType.id || issue.issueType.name,
-        name: issue.issueType.name,
-        fields: normalizeFieldSchemas(issue.issueType.pinnedFields)
-    } : null;
-    const values = (issue?.issueFieldValues?.nodes || []).flatMap((raw) => {
-        const item = raw;
-        const kind = fieldKind(item);
-        const value = kind === 'single-select' ? item.name : item.value;
-        return item.field?.name && kind && value !== undefined
-            ? [{ id: item.field.id || item.field.name, name: item.field.name, kind, value: value ?? null }]
-            : [];
-    });
-    return { issueId: issue?.id || null, type, values };
-}
-function graphState(client, repository, issue, cwd) {
-    const [owner, name] = repository.split('/');
-    if (!owner || !name)
-        return null;
-    const current = client.json([
-        'api', 'graphql', '-f', `query=${ISSUE_FIELDS_QUERY}`, '-F', `owner=${owner}`, '-F', `name=${name}`, '-F', `number=${issue}`
-    ], { cwd });
-    if (!current.ok)
-        return null;
-    return normalizeCurrentFields(current.value);
-}
-function fetchIssue(client, repository, issue, cwd) {
-    return client.json(['api', `repos/${repository}/issues/${issue}`], { cwd });
-}
-function inspectGitHubIssue(client, repository, issue, cwd) {
-    const fetched = fetchIssue(client, repository, issue, cwd);
-    if (!fetched.ok)
-        return fetched;
-    const snapshot = normalizeIssue(fetched.value, repository, issue);
-    return snapshot
-        ? { ok: true, value: snapshot }
-        : { ok: false, error: { code: 'ISSUE_IDENTITY_INVALID', message: 'Remote resource is not a valid Issue', retryable: false } };
-}
-function inspectGitHubIssueMetadata(client, repository, issue, cwd) {
-    const fetched = fetchIssue(client, repository, issue, cwd);
-    if (!fetched.ok)
-        return fetched;
-    const remote = fetched.value;
-    return {
-        ok: true,
-        value: {
-            state: String(remote.state || '').toLowerCase() === 'closed' ? 'closed' : 'open',
-            labels: (remote.labels || []).map((label) => typeof label === 'string' ? label : label.name || '').filter(Boolean).sort(),
-            body: remote.body || '',
-            milestone: remote.milestone?.title || null
-        }
-    };
-}
 async function inspectPlatformIssue(taskRef, options = {}) {
     const base = await resolvedContext(taskRef, options);
     if (!base.ok)
@@ -450,12 +339,6 @@ function requirementsFromTask(content) {
         checked: match[1].toLowerCase() === 'x', text: match[2]
     }));
 }
-function flattenPages(value) {
-    return Array.isArray(value) ? value.flatMap((item) => Array.isArray(item) ? item : [item]) : [];
-}
-function listNames(value) {
-    return flattenPages(value).map((item) => typeof item === 'string' ? item : String(item.name || '')).filter(Boolean);
-}
 function milestoneBasis(repoRoot, current) {
     if (!current || !/^\d+\.\d+\.x$/.test(current))
         return current;
@@ -552,67 +435,6 @@ function desiredFieldValues(frontmatter, fields) {
     }
     return desired;
 }
-function planGraphMetadata(client, repository, issue, cwd, frontmatter, options, capabilities) {
-    if (!options.issueType && !options.fields)
-        return { operations: [], issueId: null };
-    if (!capabilities.push)
-        return {
-            operations: [
-                ...(options.issueType ? [{ name: 'issue-type', status: 'skipped', reasonCode: 'PUSH_REQUIRED' }] : []),
-                ...(options.fields ? [{ name: 'fields', status: 'skipped', reasonCode: 'PUSH_REQUIRED' }] : [])
-            ], issueId: null
-        };
-    const [owner] = repository.split('/');
-    const typesResult = client.json(['api', 'graphql', '-f', `query=${ISSUE_TYPES_QUERY}`, '-F', `owner=${owner}`], { cwd });
-    if (!typesResult.ok)
-        return { operations: [{ name: 'issue-schema', status: 'failed', reasonCode: typesResult.error.code }], issueId: null };
-    const types = normalizeIssueTypes(typesResult.value);
-    if (types.length === 0)
-        return {
-            operations: [
-                ...(options.issueType ? [{ name: 'issue-type', status: 'skipped', reasonCode: 'ISSUE_TYPES_UNSUPPORTED' }] : []),
-                ...(options.fields ? [{ name: 'fields', status: 'skipped', reasonCode: 'ISSUE_TYPES_UNSUPPORTED' }] : [])
-            ], issueId: null
-        };
-    const current = graphState(client, repository, issue, cwd);
-    if (!current?.issueId)
-        return { operations: [{ name: 'issue-schema', status: 'failed', reasonCode: 'ISSUE_GRAPH_IDENTITY_INVALID' }], issueId: null };
-    const target = types.find((type) => type.name === desiredIssueType(frontmatter.type || ''));
-    if (!target)
-        return { operations: [{ name: 'issue-type', status: 'failed', reasonCode: 'ISSUE_TYPE_NOT_FOUND' }], issueId: current.issueId };
-    const operations = [];
-    if (options.issueType)
-        operations.push(current.type?.id === target.id
-            ? { name: 'issue-type', status: 'no-op', reasonCode: null }
-            : { name: 'issue-type', status: 'planned', reasonCode: null, value: { issueTypeId: target.id } });
-    if (options.fields) {
-        const desired = desiredFieldValues(frontmatter, target.fields);
-        const inputs = desired.filter((field) => current.values.find((value) => value.name === field.name)?.value !== field.value).map((field) => field.input);
-        if (current.type?.id !== target.id) {
-            const targetNames = new Set(target.fields.map((field) => field.name));
-            for (const old of current.values)
-                if (!targetNames.has(old.name))
-                    inputs.push({ fieldId: old.id, delete: true });
-        }
-        operations.push(inputs.length === 0
-            ? { name: 'fields', status: 'no-op', reasonCode: null }
-            : { name: 'fields', status: 'planned', reasonCode: null, value: inputs });
-    }
-    return { operations, issueId: current.issueId };
-}
-function executeGraphOperation(client, cwd, issueId, operation) {
-    if (operation.name === 'issue-type') {
-        const query = 'mutation($issueId:ID!,$issueTypeId:ID){updateIssueIssueType(input:{issueId:$issueId,issueTypeId:$issueTypeId}){issue{id}}}';
-        const typeId = operation.value.issueTypeId;
-        return client.json(['api', 'graphql', '--input', '-'], {
-            cwd, method: 'POST', input: JSON.stringify({ query, variables: { issueId, issueTypeId: typeId } })
-        });
-    }
-    const query = 'mutation($issueId:ID!,$issueFields:[IssueFieldCreateOrUpdateInput!]!){setIssueFieldValue(input:{issueId:$issueId,issueFields:$issueFields}){issue{id}}}';
-    return client.json(['api', 'graphql', '--input', '-'], {
-        cwd, method: 'POST', input: JSON.stringify({ query, variables: { issueId, issueFields: operation.value } })
-    });
-}
 async function syncPlatformIssue(taskRef, options) {
     const base = await resolvedContext(taskRef, options);
     if (!base.ok)
@@ -761,5 +583,5 @@ async function syncPlatformIssue(taskRef, options) {
         operations: plan.operations.map((operation) => operation.status === 'planned' ? { ...operation, status: 'applied' } : operation), error: null
     });
 }
-export { bindPlatformIssue, createPlatformIssue, graphState, inspectGitHubIssue, inspectGitHubIssueMetadata, inspectPlatformIssue, normalizeIssue, requirementSectionAnchors, syncPlatformIssue };
+export { bindPlatformIssue, createPlatformIssue, inspectPlatformIssue, requirementSectionAnchors, syncPlatformIssue };
 //# sourceMappingURL=issues.js.map

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

@@ -1,6 +1,7 @@
 import fs from 'node:fs';
 import { parseTypedTaskFrontmatter } from '../task/frontmatter.js';
 import { resolveTaskRef } from '../task/resolve-ref.js';
+import { checkStatusBucket } from './check-status.js';
 import { resolvePlatformProviderContext } from './context.js';
 import { inspectPlatformPullRequest } from './pull-requests.js';
 import { platformResult } from './types.js';
@@ -26,20 +27,6 @@ function classifyPullRequestReadiness(input) {
         return { state: 'pending', headSha: input.headSha };
     return { state: 'ready', headSha: input.headSha };
 }
-function parseRunJobIdentity(detailsUrl) {
-    try {
-        const url = new URL(detailsUrl);
-        if (url.hostname !== 'github.com')
-            return null;
-        const match = url.pathname.match(/\/actions\/runs\/(\d+)(?:\/job\/(\d+))?/);
-        if (!match)
-            return null;
-        return { runId: Number(match[1]), jobId: match[2] ? Number(match[2]) : null };
-    }
-    catch {
-        return null;
-    }
-}
 function resolveRunCandidate(candidates, headSha, checkName) {
     const matches = candidates.filter((candidate) => candidate.headSha === headSha && candidate.name === checkName);
     if (matches.length === 1)
@@ -54,42 +41,6 @@ function checksResult(status, overrides = {}) {
         ...overrides
     };
 }
-function normalizeBucket(value) {
-    const raw = String(value.bucket || value.status || value.conclusion || value.state || '').toLowerCase();
-    if (['pass', 'success', 'successful', 'neutral'].includes(raw))
-        return 'pass';
-    if (['fail', 'failure', 'failed', 'error', 'timed_out', 'action_required'].includes(raw))
-        return 'fail';
-    if (['cancel', 'cancelled', 'canceled', 'skipped', 'stale'].includes(raw))
-        return 'cancel';
-    return 'pending';
-}
-function normalizeChecks(value) {
-    if (!Array.isArray(value))
-        return [];
-    return value.flatMap((raw) => {
-        const item = raw;
-        const name = String(item.name || item.context || '');
-        return name ? [{
-                name,
-                bucket: normalizeBucket(item),
-                workflow: item.workflow ? String(item.workflow) : null,
-                conclusion: item.conclusion ? String(item.conclusion) : item.state ? String(item.state) : null,
-                detailsUrl: item.link ? String(item.link) : item.detailsUrl ? String(item.detailsUrl) : null,
-                startedAt: item.startedAt ? String(item.startedAt) : null,
-                completedAt: item.completedAt ? String(item.completedAt) : null
-            }] : [];
-    });
-}
-function inspectGitHubRequiredChecks(client, repository, number, cwd) {
-    const inspected = client.json([
-        'pr', 'checks', String(number), '--repo', repository,
-        '--json', 'name,state,bucket,link,workflow,startedAt,completedAt'
-    ], { cwd });
-    return inspected.ok
-        ? { ok: true, value: normalizeChecks(inspected.value) }
-        : { ok: false, error: inspected.error };
-}
 async function resolvedTask(taskRef, options) {
     const resolved = resolveTaskRef(taskRef, options.cwd ? { repoRoot: options.cwd } : {});
     if (!resolved.ok)
@@ -117,10 +68,9 @@ async function inspectRequiredChecks(taskRef, options = {}) {
     const base = await resolvedTask(taskRef, options);
     if (!base.ok)
         return base.output;
-    return inspectChecksForResolvedTask(base, options);
+    return inspectChecksForResolvedTask(base);
 }
-async function inspectChecksForResolvedTask(base, options) {
-    const repository = base.context.platform.repository;
+async function inspectChecksForResolvedTask(base) {
     {
         const inspected = base.provider.checks?.inspectRequired
             ? await base.provider.checks.inspectRequired({
@@ -137,7 +87,13 @@ async function inspectChecksForResolvedTask(base, options) {
                 pullRequest: base.pullRequest,
                 error: providerError(inspected.error, 'PLATFORM_PROVIDER_OPERATION_FAILED')
             });
-        const required = normalizeChecks(inspected.value);
+        const required = inspected.value.map((check) => ({
+            name: check.name,
+            bucket: checkStatusBucket(check.status),
+            conclusion: check.conclusion || null,
+            detailsUrl: check.detailsUrl || null,
+            workflow: null, startedAt: null, completedAt: null
+        }));
         const classified = classifyRequiredChecks(required);
         const status = classified.state === 'passed' || classified.state === 'no-required'
             ? 'no-op'
@@ -157,7 +113,7 @@ async function inspectPullRequestReadiness(taskRef, options = {}) {
     const base = await resolvedTask(taskRef, options);
     if (!base.ok)
         return base.output;
-    const checked = await inspectChecksForResolvedTask(base, options);
+    const checked = await inspectChecksForResolvedTask(base);
     const classifiedCodes = new Set(['REQUIRED_CHECKS_PENDING', 'REQUIRED_CHECKS_FAILED', 'REQUIRED_CHECKS_CANCELLED']);
     if (checked.error && !classifiedCodes.has(checked.error.code))
         return checked;
@@ -242,15 +198,6 @@ async function resolvePlatformCheckRun(taskRef, options) {
         });
     }
 }
-function fetchCheckLogText(client, args, cwd) {
-    if (!client.text)
-        return { ok: false, error: { code: 'PLATFORM_CLIENT_TEXT_UNAVAILABLE', message: 'Platform client does not support text responses', retryable: false } };
-    const fetched = client.text(args, { cwd });
-    if (fetched.ok || args[0] !== 'api' || !/response contains terminal escape sequences/i.test(fetched.error.message)) {
-        return fetched;
-    }
-    return client.text([...args, '--allow-escape-sequences'], { cwd });
-}
 async function fetchPlatformCheckLogs(taskRef, options) {
     const base = await resolvedTask(taskRef, options);
     if (!base.ok)
@@ -281,5 +228,5 @@ async function fetchPlatformCheckLogs(taskRef, options) {
         });
     }
 }
-export { classifyPullRequestReadiness, classifyRequiredChecks, fetchCheckLogText, fetchPlatformCheckLogs, inspectGitHubRequiredChecks, inspectRequiredChecks, inspectPullRequestReadiness, parseRunJobIdentity, resolvePlatformCheckRun, resolveRunCandidate, watchPullRequestReadiness };
+export { classifyPullRequestReadiness, classifyRequiredChecks, fetchPlatformCheckLogs, inspectRequiredChecks, inspectPullRequestReadiness, resolvePlatformCheckRun, resolveRunCandidate, watchPullRequestReadiness };
 //# sourceMappingURL=pr-checks.js.map

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

@@ -12,6 +12,8 @@ import { CONTROL_MARKER_PATTERN, escapeHtmlText, sanitizeMarkdownDocument, split
 import { platformResult } from './types.js';
 import { providerError, providerOperationContext, providerStatus, providerResourceToken, resourceIdentityNumber, unsupportedProviderOperation } from './provider-bridge.js';
 import { buildPrChangeReport, CANONICAL_REPORT_PLACEHOLDER, readPrChangeReport, replaceCanonicalReportPlaceholder, runMechanicalChangeReport, taskIntentDigest, validateMechanicalReport, validatePrecheckCandidate, writePrChangeReportAtomic } from './pr-change-report.js';
+import { manualValidationFinalSummaryProjectionMatches } from '../task/manual-validation-receipt.js';
+import { readManualValidationCompletion } from '../task/manual-validation-completion.js';
 function summaryMarker(taskId) {
     return `<!-- sync-pr:${taskId}:summary -->`;
 }
@@ -286,6 +288,25 @@ async function summaryContext(taskRef, options = {}) {
         artifacts: canonicalArtifacts(resolved.taskDir)
     };
 }
+async function summaryCommentState(taskRef, options = {}) {
+    const context = await summaryContext(taskRef, options);
+    if (!context.task.prNumber || !context.pullRequest)
+        return { ...context, comment: null };
+    const loaded = await resolvePlatformProviderContext({ cwd: options.cwd, client: options.client });
+    if (!loaded.ok || !loaded.value.provider.comments?.list)
+        return { ...context, comment: null };
+    const listed = await loaded.value.provider.comments.list({
+        context: providerOperationContext(loaded.value),
+        parent: providerResourceToken(loaded.value.provider, 'pull-request', String(context.task.prNumber))
+    });
+    if (!listed.ok)
+        return { ...context, comment: null };
+    const matches = listed.value.filter((comment) => comment.body.includes(summaryMarker(context.task.id)));
+    return {
+        ...context,
+        comment: matches.length === 1 ? { id: /^\d+$/u.test(matches[0].id) ? Number(matches[0].id) : matches[0].id, body: matches[0].body } : null
+    };
+}
 async function reportWrite(taskRef, options) {
     const resolved = resolveTaskRef(taskRef, options.cwd ? { repoRoot: options.cwd } : {});
     if (!resolved.ok)
@@ -444,7 +465,7 @@ async function syncPullRequestSummary(taskRef, options) {
         return preserveFailure(context);
     }
     try {
-        return await withTaskExecutionLock(resolved.repoRoot, resolved.taskId, options.agent, async () => {
+        const execute = async () => {
             const { taskContent, boundFact } = readBoundTaskSnapshot(resolved.taskMdPath);
             if (!boundFact)
                 return {
@@ -463,6 +484,34 @@ async function syncPullRequestSummary(taskRef, options) {
             const initial = await inspectBoundPullRequest(context, resolved.repoRoot, boundPrNumber, loaded.value);
             if (!initial.ok)
                 return fail(initial.status, context, initial.error);
+            const manual = options.manualValidation;
+            const hasFinalManualValidation = /###\s+✅\s+(?:Manual Validation Passed|人工验证已通过)/u.test(options.body);
+            if (hasFinalManualValidation && (!manual || manual.phase !== 'final'))
+                return fail('failed', context, { code: 'MANUAL_VALIDATION_TRANSACTION_REQUIRED', message: 'final manual-validation summary requires the transaction coordinator', retryable: false }, prNumber);
+            if (manual && (manual.phase === 'pending' ? hasFinalManualValidation : !hasFinalManualValidation))
+                return fail('failed', context, { code: 'MANUAL_VALIDATION_SUMMARY_PHASE_INVALID', message: 'manual-validation summary phase does not match the requested writer phase', retryable: false }, prNumber);
+            if (manual) {
+                if (manual.phase === 'final' && (!manual.transactionId || !manual.receiptDigest || !manual.prHeadSha || manual.prHeadSha !== initial.value.head.sha))
+                    return fail('failed', context, { code: 'MANUAL_VALIDATION_TRANSACTION_REQUIRED', message: 'final manual-validation summary requires transaction, receipt, and current head identity', retryable: false }, prNumber);
+                if (manual.phase === 'final') {
+                    if (manual.authority !== 'coordinator')
+                        return fail('failed', context, { code: 'MANUAL_VALIDATION_TRANSACTION_REQUIRED', message: 'final manual-validation summary requires coordinator authority', retryable: false }, prNumber);
+                    const completion = readManualValidationCompletion(resolved.taskDir, {
+                        transactionId: manual.transactionId,
+                        taskId: resolved.taskId,
+                        prNumber,
+                        prHeadSha: initial.value.head.sha,
+                        receiptDigest: manual.receiptDigest
+                    });
+                    if (!completion.ok)
+                        return fail('failed', context, platformError(completion.error), prNumber);
+                    const { receipt, transaction } = completion.value;
+                    if (transaction.phase !== 'final-promotion-in-progress' || !transaction.eventAppended)
+                        return fail('failed', context, { code: 'MANUAL_VALIDATION_TRANSACTION_PHASE_INVALID', message: 'final summary requires a receipt-backed promotion intent', retryable: false }, prNumber);
+                    if (!manualValidationFinalSummaryProjectionMatches(options.body, receipt))
+                        return fail('failed', context, { code: 'MANUAL_VALIDATION_SUMMARY_PROJECTION_INVALID', message: 'final summary body does not match the canonical receipt projection', retryable: false }, prNumber);
+                }
+            }
             const expectedReportPath = taskReportPath(resolved.taskDir);
             const suppliedReportPath = path.resolve(resolved.repoRoot, options.changeReportFile);
             if (suppliedReportPath !== expectedReportPath)
@@ -610,7 +659,10 @@ async function syncPullRequestSummary(taskRef, options) {
                 operations: [{ name: `summary:${reconciliation.action}`, status: 'applied', reasonCode: null }],
                 result: null, warnings: [], ...info
             };
-        });
+        };
+        return await (options.lockAlreadyHeld
+            ? execute()
+            : withTaskExecutionLock(resolved.repoRoot, resolved.taskId, options.agent, execute));
     }
     catch (error) {
         const lockError = error instanceof TaskExecutionLockError
@@ -619,5 +671,5 @@ async function syncPullRequestSummary(taskRef, options) {
         return fail(lockError.retryable ? 'blocked' : 'failed', context, lockError);
     }
 }
-export { buildPullRequestSummary, reconcileSummaryComment, reportWrite, summaryContext, summaryMarker, syncPullRequestSummary, warningResultForPrimary };
+export { buildPullRequestSummary, reconcileSummaryComment, reportWrite, summaryCommentState, summaryContext, summaryMarker, syncPullRequestSummary, warningResultForPrimary };
 //# sourceMappingURL=pr-summary.js.map