@fitlab-ai/agent-infra 0.9.14 → 0.10.0

package/lib/platform/github-data.ts

@@ -0,0 +1,322 @@
+import type { PlatformClient as GitHubClient } from './context.ts';
+import type { PlatformIssueSnapshot as IssueSnapshot, PlatformChangeRequestSnapshot as PullRequestSnapshot } from './snapshots.ts';
+import type { RequiredCheckSnapshot } from './provider-contract.ts';
+import { checkStatusBucket } from './check-status.ts';
+
+type RemoteIssue = {
+  number?: number;
+  id?: number;
+  node_id?: string;
+  html_url?: string;
+  state?: string;
+  title?: string;
+  body?: string | null;
+  labels?: Array<string | { name?: string }>;
+  assignees?: Array<{ login?: string }>;
+  milestone?: { title?: string } | null;
+  type?: { name?: string } | null;
+  pull_request?: unknown;
+};
+
+type IssueFieldSchema = {
+  id: string;
+  name: string;
+  kind: 'single-select' | 'date' | 'text' | 'number';
+  options: Array<{ id: string; name: string }>;
+};
+
+type IssueTypeSchema = { id: string; name: string; fields: IssueFieldSchema[] };
+
+type CurrentField = { id: string; name: string; kind: IssueFieldSchema['kind']; value: string | number | null };
+
+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 normalizeIssue(remote: RemoteIssue, repository: string, fallbackNumber?: number): IssueSnapshot | null {
+  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: { __typename?: string }): IssueFieldSchema['kind'] | null {
+  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: unknown): IssueFieldSchema[] {
+  if (!Array.isArray(value)) return [];
+  return value.flatMap((raw) => {
+    const item = raw as { id?: string; name?: string; __typename?: string; options?: Array<{ id?: string; name?: string }> };
+    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 normalizeCurrentFields(value: unknown): { issueId: string | null; type: IssueTypeSchema | null; values: CurrentField[] } {
+  const issue = (value as { data?: { repository?: { issue?: {
+    id?: string;
+    issueType?: { id?: string; name?: string; pinnedFields?: unknown[] } | null;
+    issueFieldValues?: { nodes?: unknown[] };
+  } } } })?.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 as { __typename?: string; name?: string; value?: string | number; field?: { id?: string; name?: string } };
+    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: GitHubClient, repository: string, issue: number, cwd: string) {
+  const [owner, name] = repository.split('/');
+  if (!owner || !name) return null;
+  const current = client.json<unknown>([
+    '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: GitHubClient, repository: string, issue: number, cwd: string) {
+  return client.json<RemoteIssue>(['api', `repos/${repository}/issues/${issue}`], { cwd });
+}
+
+function inspectGitHubIssue(client: GitHubClient, repository: string, issue: number, cwd: string) {
+  const fetched = fetchIssue(client, repository, issue, cwd);
+  if (!fetched.ok) return fetched;
+  const snapshot = normalizeIssue(fetched.value, repository, issue);
+  return snapshot
+    ? { ok: true as const, value: snapshot }
+    : { ok: false as const, error: { code: 'ISSUE_IDENTITY_INVALID', message: 'Remote resource is not a valid Issue', retryable: false } };
+}
+
+type RemotePullRequest = {
+  number?: number;
+  node_id?: string;
+  html_url?: string;
+  state?: string;
+  title?: string;
+  body?: string | null;
+  draft?: boolean;
+  pull_request?: unknown;
+  head?: { ref?: string; sha?: string; repo?: { full_name?: string } | null };
+  base?: { ref?: string; sha?: string; repo?: { full_name?: string } | null };
+  merged_at?: string | null;
+  merge_commit_sha?: string | null;
+  labels?: Array<string | { name?: string }>;
+  assignees?: Array<{ login?: string }>;
+  milestone?: { title?: string } | null;
+  mergeable?: boolean | null;
+  mergeable_state?: string | null;
+};
+
+type ClosingPullRequestNode = {
+  number?: number;
+  id?: string;
+  url?: string;
+  state?: string;
+  title?: string;
+  body?: string | null;
+  isDraft?: boolean;
+  headRefName?: string;
+  headRefOid?: string;
+  headRepository?: { nameWithOwner?: string } | null;
+  baseRefName?: string;
+  baseRefOid?: string;
+  baseRepository?: { nameWithOwner?: string } | null;
+  mergedAt?: string | null;
+  mergeCommit?: { oid?: string } | null;
+  labels?: { nodes?: Array<{ name?: string }> };
+  assignees?: { nodes?: Array<{ login?: string }> };
+  milestone?: { title?: string } | null;
+};
+
+type ClosingPullRequestPage = {
+  data?: { repository?: { issue?: { closedByPullRequestsReferences?: {
+    nodes?: ClosingPullRequestNode[];
+    pageInfo?: { hasNextPage?: boolean; endCursor?: string | null };
+  } } } };
+};
+
+const CLOSING_PULL_REQUESTS_QUERY = `query($owner:String!,$name:String!,$issue:Int!,$cursor:String){repository(owner:$owner,name:$name){issue(number:$issue){closedByPullRequestsReferences(first:100,after:$cursor){nodes{number id url state title body isDraft headRefName headRefOid headRepository{nameWithOwner} baseRefName baseRefOid baseRepository{nameWithOwner} mergedAt mergeCommit{oid} labels(first:100){nodes{name}} assignees(first:100){nodes{login}} milestone{title}} pageInfo{hasNextPage endCursor}}}}}`;
+
+function normalizePullRequest(remote: RemotePullRequest, repository: string): PullRequestSnapshot | null {
+  const number = Number(remote.number);
+  const headRepository = remote.head?.repo?.full_name;
+  const baseRepository = remote.base?.repo?.full_name;
+  if (!Number.isInteger(number) || number <= 0 || !remote.node_id || !remote.html_url ||
+      !remote.head?.ref || !remote.head.sha || !headRepository || !remote.base?.ref || !baseRepository) return null;
+  const mergeabilityDetail = remote.mergeable_state?.trim().toLowerCase() || null;
+  const mergeability = remote.mergeable === false
+    ? { state: 'conflicting' as const, detail: mergeabilityDetail }
+    : remote.mergeable === true && mergeabilityDetail !== 'dirty'
+      ? { state: 'mergeable' as const, detail: mergeabilityDetail }
+      : { state: 'unknown' as const, detail: mergeabilityDetail };
+  return {
+    repository,
+    number,
+    nodeId: remote.node_id,
+    url: remote.html_url,
+    state: remote.state === 'closed' ? 'closed' : 'open',
+    title: remote.title || '',
+    body: remote.body || '',
+    draft: Boolean(remote.draft),
+    head: { repository: headRepository, ref: remote.head.ref, sha: remote.head.sha },
+    base: { repository: baseRepository, ref: remote.base.ref, sha: remote.base.sha || '' },
+    mergedAt: remote.merged_at || null,
+    mergeCommitSha: remote.merge_commit_sha || null,
+    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,
+    mergeability
+  };
+}
+
+function normalizeClosingPullRequest(node: ClosingPullRequestNode, repository: string): PullRequestSnapshot | null {
+  return normalizePullRequest({
+    number: node.number,
+    node_id: node.id,
+    html_url: node.url,
+    state: node.state === 'OPEN' ? 'open' : 'closed',
+    title: node.title,
+    body: node.body,
+    draft: node.isDraft,
+    head: { ref: node.headRefName, sha: node.headRefOid, repo: node.headRepository ? { full_name: node.headRepository.nameWithOwner } : null },
+    base: { ref: node.baseRefName, sha: node.baseRefOid, repo: node.baseRepository ? { full_name: node.baseRepository.nameWithOwner } : null },
+    merged_at: node.mergedAt,
+    merge_commit_sha: node.mergeCommit?.oid,
+    labels: node.labels?.nodes,
+    assignees: node.assignees?.nodes,
+    milestone: node.milestone
+  }, repository);
+}
+
+function inspectGitHubIssueClosingChangeRequests(
+  client: GitHubClient,
+  repository: string,
+  issueNumber: number,
+  cwd: string
+) {
+  const [owner, name] = repository.split('/');
+  if (!owner || !name || !Number.isInteger(issueNumber) || issueNumber <= 0) return {
+    ok: false as const,
+    error: { code: 'PR_IDENTITY_INVALID', message: 'Repository or Issue identity is invalid', retryable: false }
+  };
+  const candidates: PullRequestSnapshot[] = [];
+  let cursor: string | null = null;
+  for (;;) {
+    const args = [
+      'api', 'graphql', '-f', `query=${CLOSING_PULL_REQUESTS_QUERY}`,
+      '-F', `owner=${owner}`, '-F', `name=${name}`, '-F', `issue=${issueNumber}`
+    ];
+    if (cursor) args.push('-F', `cursor=${cursor}`);
+    const response = client.json<ClosingPullRequestPage>(args, { cwd });
+    if (!response.ok) return response;
+    const connection = response.value?.data?.repository?.issue?.closedByPullRequestsReferences;
+    if (!connection || !Array.isArray(connection.nodes) || !connection.pageInfo) return {
+      ok: false as const,
+      error: { code: 'PR_IDENTITY_INVALID', message: 'Closing pull request response is incomplete', retryable: false }
+    };
+    for (const node of connection.nodes) {
+      const normalized = normalizeClosingPullRequest(node, repository);
+      if (!normalized) return {
+        ok: false as const,
+        error: { code: 'PR_IDENTITY_INVALID', message: 'Closing pull request identity is incomplete', retryable: false }
+      };
+      candidates.push(normalized);
+    }
+    if (!connection.pageInfo.hasNextPage) return { ok: true as const, value: candidates };
+    if (!connection.pageInfo.endCursor || connection.pageInfo.endCursor === cursor) return {
+      ok: false as const,
+      error: { code: 'PR_IDENTITY_INVALID', message: 'Closing pull request pagination cursor is invalid', retryable: false }
+    };
+    cursor = connection.pageInfo.endCursor;
+  }
+}
+
+function inspectGitHubPullRequest(client: GitHubClient, repository: string, number: number, cwd: string) {
+  const fetched = client.json<RemotePullRequest>(['api', `repos/${repository}/pulls/${number}`], { cwd });
+  if (!fetched.ok) return fetched;
+  const pullRequest = normalizePullRequest(fetched.value, repository);
+  return pullRequest
+    ? { ok: true as const, value: pullRequest }
+    : { ok: false as const, error: { code: 'PR_IDENTITY_INVALID', message: 'Remote resource is not a valid pull request', retryable: false } };
+}
+
+function normalizeGitHubChecks(value: unknown): RequiredCheckSnapshot[] {
+  if (!Array.isArray(value)) return [];
+  return value.flatMap((raw) => {
+    const item = raw as Record<string, unknown>;
+    const name = String(item.name || item.context || '');
+    return name ? [{
+      name,
+      status: checkStatusBucket(String(item.bucket || item.status || item.conclusion || item.state || '')),
+      conclusion: item.conclusion ? String(item.conclusion) : item.state ? String(item.state) : null,
+      detailsUrl: item.link ? String(item.link) : item.detailsUrl ? String(item.detailsUrl) : null
+    }] : [];
+  });
+}
+
+function inspectGitHubRequiredChecks(client: GitHubClient, repository: string, number: number, cwd: string) {
+  const inspected = client.json<unknown>([
+    'pr', 'checks', String(number), '--repo', repository,
+    '--json', 'name,state,bucket,link,workflow,startedAt,completedAt'
+  ], { cwd });
+  return inspected.ok
+    ? { ok: true as const, value: normalizeGitHubChecks(inspected.value) }
+    : { ok: false as const, error: inspected.error };
+}
+
+function parseRunJobIdentity(detailsUrl: string): { runId: number; jobId: number | null } | null {
+  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 fetchCheckLogText(client: GitHubClient, args: string[], cwd: string) {
+  if (!client.text) return { ok: false as const, 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 });
+}
+
+export { ISSUE_FIELDS_QUERY, graphState, inspectGitHubIssue, normalizePullRequest, inspectGitHubIssueClosingChangeRequests, inspectGitHubPullRequest, inspectGitHubRequiredChecks, parseRunJobIdentity, fetchCheckLogText };
+export type { IssueFieldSchema, RemotePullRequest };

package/lib/platform/github-provider.ts

@@ -1,3 +1,4 @@
+import { ISSUE_FIELDS_QUERY, graphState, inspectGitHubIssue, inspectGitHubPullRequest, inspectGitHubIssueClosingChangeRequests, inspectGitHubRequiredChecks, parseRunJobIdentity, fetchCheckLogText } from './github-data.ts';
 import { execFileSync } from 'node:child_process';
 import fs from 'node:fs';
 import os from 'node:os';
@@ -30,7 +31,6 @@ import type {
   RepositoryMetadataSnapshot,
   ReleaseSnapshot,
   RemoteCommentSnapshot,
-  RequiredCheckSnapshot,
   ReviewSnapshot,
   SecurityAlertKind,
   SecurityAlertSnapshot,
@@ -41,7 +41,6 @@ import { extractPullRequestFileNames, syncLabelDelta } from './in-label-sync.ts'
 
 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: string): string | null {
@@ -465,10 +464,9 @@ function createGitHubOperations(client: GitHubClient): Pick<PlatformProvider, 'i
     async inspect({ context, target }) {
       const number = resourceIdentityNumber(target);
       if (!number) return invalid('ISSUE_NUMBER_INVALID', 'Issue number must be positive');
-      const module = await import('./issues.ts');
-      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 }) {
@@ -648,8 +646,7 @@ function createGitHubOperations(client: GitHubClient): Pick<PlatformProvider, 'i
     async inspect({ context, target }) {
       const number = resourceIdentityNumber(target);
       if (!number) return invalid('PR_NUMBER_INVALID', 'Pull request number must be positive');
-      const module = await import('./pull-requests.ts');
-      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) };
     },
@@ -708,8 +705,7 @@ function createGitHubOperations(client: GitHubClient): Pick<PlatformProvider, 'i
     async listClosing({ context, issue }) {
       const number = resourceIdentityNumber(issue);
       if (!number) return invalid('ISSUE_NUMBER_INVALID', 'Issue number must be positive');
-      const module = await import('./pull-requests.ts');
-      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) };
     },
@@ -780,8 +776,7 @@ function createGitHubOperations(client: GitHubClient): Pick<PlatformProvider, 'i
     async resolveGitEvidence({ context, target, expected }) {
       const number = resourceIdentityNumber(target);
       if (!number) return invalid('PR_NUMBER_INVALID', 'Pull request number must be positive');
-      const module = await import('./github-provider.ts');
-      return module.resolveGitHubChangeRequestGitEvidence({
+      return resolveGitHubChangeRequestGitEvidence({
         cwd: context.workingDirectory,
         repository: repository(context),
         number,
@@ -795,16 +790,12 @@ function createGitHubOperations(client: GitHubClient): Pick<PlatformProvider, 'i
     async inspectRequired({ context, changeRequest }) {
       const number = resourceIdentityNumber(changeRequest);
       if (!number) return invalid('PR_NUMBER_INVALID', 'Pull request number must be positive');
-      const module = await import('./pr-checks.ts');
-      const fetched = module.inspectGitHubRequiredChecks(client, repository(context), number, context.workingDirectory);
-      if (!fetched.ok) return fetched;
-      return { ok: true, value: fetched.value.map((check: any): RequiredCheckSnapshot => ({ 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.ts');
-      const direct = detailsUrl ? module.parseRunJobIdentity(detailsUrl) : null;
+      const direct = detailsUrl ? parseRunJobIdentity(detailsUrl) : null;
       if (direct) {
         const run = client.json<any>(['api', `repos/${repository(context)}/actions/runs/${direct.runId}`], { cwd: context.workingDirectory });
         if (!run.ok) return run;
@@ -818,11 +809,10 @@ function createGitHubOperations(client: GitHubClient): Pick<PlatformProvider, 'i
         : invalid('CHECK_RUN_NOT_FOUND', 'Check run was not found');
     },
     async fetchLogs({ context, runId, jobId }) {
-      const module = await import('./pr-checks.ts');
       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 || '' } } as ProviderResult<CheckLogSnapshot>;
     }
@@ -891,7 +881,12 @@ function createGitHubOperations(client: GitHubClient): Pick<PlatformProvider, 'i
           }
           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/lib/platform/issue-comments.ts

@@ -17,11 +17,10 @@ import {
 import { resourceIdentityNumber } from './resource-identity.ts';
 import { taskIssueIdentity, taskIssueIdentityError } from './task-identities.ts';
 import {
-  CONTROL_MARKER_PATTERN,
+  canonicalizeCommentBody,
   escapeHtmlText,
   fenceRanges,
-  renderSafeCodeFence,
-  sanitizeMarkdownDocument
+  renderSafeCodeFence
 } from './comment-safety.ts';
 import type { FenceRange } from './comment-safety.ts';
 
@@ -51,6 +50,7 @@ const MARKERS = {
   summary: (taskId: string) => `<!-- sync-issue:${taskId}:summary -->`,
   cancel: (taskId: string) => `<!-- sync-issue:${taskId}:cancel -->`
 };
+const COMMENT_BYTE_LIMIT = 60_000;
 
 const ARTIFACT_TITLES: Record<string, string> = {
   analysis: '需求分析',
@@ -68,6 +68,10 @@ function normalizeCommentContent(content: string): string {
   return content.replace(/\r\n/g, '\n').replace(/\n+$/, '\n');
 }
 
+function isTaskCommentTooLarge(content: string): boolean {
+  return Buffer.byteLength(content, 'utf8') > COMMENT_BYTE_LIMIT;
+}
+
 function splitFrontmatter(content: string): { frontmatter: string | null; body: string } {
   const match = content.match(/^---\r?\n([\s\S]*?)\r?\n---(?:\r?\n|$)/);
   if (!match) return { frontmatter: null, body: content };
@@ -75,7 +79,7 @@ function splitFrontmatter(content: string): { frontmatter: string | null; body:
 }
 
 function sanitizeCommentBody(value: string, label: string): string {
-  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;
 }
@@ -270,7 +274,7 @@ function chunkArtifactComment(input: {
   byteLimit?: number;
   backfill?: boolean;
 }): RenderedChunk[] {
-  const byteLimit = input.byteLimit || 60_000;
+  const byteLimit = input.byteLimit || COMMENT_BYTE_LIMIT;
   const identity = artifactIdentity(input.artifact);
   const safeBody = sanitizeCommentBody(input.body, 'artifact body');
   const single = buildArtifactChunk(
@@ -324,12 +328,6 @@ function flattenComments(value: unknown): RemoteComment[] {
   );
 }
 
-function issueNumberFromTask(content: string): number | null {
-  const value = parseTaskFrontmatter(content).issue_number;
-  const number = Number(value);
-  return Number.isInteger(number) && number > 0 ? number : null;
-}
-
 function listRemoteComments(client: PlatformClient, repo: string, issue: number, cwd: string) {
   const result = client.json<unknown>([
     'api', '--paginate', '--slurp', `repos/${repo}/issues/${issue}/comments?per_page=100`
@@ -453,11 +451,18 @@ async function syncPlatformComment(taskRef: string, options: SyncOptions): Promi
   }
   const taskContent = fs.readFileSync(resolved.taskMdPath, 'utf8');
   let issueIdentityFromTask: ReturnType<typeof taskIssueIdentity>;
-  try { issueIdentityFromTask = taskIssueIdentity(parseTaskFrontmatter(taskContent), undefined, options.runtimeVersion); }
+  try { issueIdentityFromTask = taskIssueIdentity(parseTaskFrontmatter(taskContent)); }
   catch (error) { return platformResult('failed', { error: { ...taskIssueIdentityError(error), retryable: false } }); }
   if (!issueIdentityFromTask) {
     return platformResult('no-op', {
-      error: { code: 'ISSUE_NOT_LINKED', message: 'Task has no valid issue_number', retryable: false }
+      error: { code: 'ISSUE_NOT_LINKED', message: 'Task has no valid platform issue identity', retryable: false }
+    });
+  }
+  if (options.kind === 'task' && isTaskCommentTooLarge(taskContent)) {
+    return platformResult('no-op', {
+      resource: { kind: 'issue', number: resourceIdentityNumber(issueIdentityFromTask), identity: issueIdentityFromTask },
+      operations: [{ name: `comment:${MARKERS.task(resolved.taskId)}`, status: 'skipped', reasonCode: 'COMMENT_PAYLOAD_TOO_LARGE' }],
+      error: null
     });
   }
   let desired: RenderedChunk[];
@@ -615,7 +620,7 @@ async function checkPlatformCommentOwner(taskRef: string, options: { cwd?: strin
   if (!resolved.ok) return platformResult('failed', { error: { code: resolved.code, message: resolved.message, retryable: false } });
   const content = fs.readFileSync(resolved.taskMdPath, 'utf8');
   let issueIdentity: ReturnType<typeof taskIssueIdentity>;
-  try { issueIdentity = taskIssueIdentity(parseTaskFrontmatter(content), undefined, options.runtimeVersion); }
+  try { issueIdentity = taskIssueIdentity(parseTaskFrontmatter(content)); }
   catch (error) { return platformResult('failed', { error: { ...taskIssueIdentityError(error), retryable: false } }); }
   if (!issueIdentity) return platformResult('no-op', { error: { code: 'ISSUE_NOT_LINKED', message: 'Task has no valid platform issue identity', retryable: false } });
   const loaded = await resolvePlatformProviderContext({ cwd: resolved.repoRoot, client: options.client });
@@ -641,6 +646,7 @@ async function checkPlatformCommentOwner(taskRef: string, options: { cwd?: strin
 }
 
 export {
+  COMMENT_BYTE_LIMIT,
   MARKERS,
   chunkArtifactComment,
   findMarkerComments,
@@ -649,6 +655,7 @@ export {
   listPlatformComments,
   normalizeCommentContent,
   renderTaskComment,
+  isTaskCommentTooLarge,
   syncPlatformComment,
   validateRelatedMarkerSet,
   writeComment