@fitlab-ai/agent-infra 0.9.13 → 0.9.15

package/dist/lib/platform/pull-requests.js

@@ -1,3 +1,4 @@
+import { normalizePullRequest } from './github-data.js';
 import fs from 'node:fs';
 import path from 'node:path';
 import { execFileSync } from 'node:child_process';
@@ -5,12 +6,10 @@ import { parseTypedTaskFrontmatter } from '../task/frontmatter.js';
 import { resolveTaskRef } from '../task/resolve-ref.js';
 import { extractSection } from '../task/sections.js';
 import { captureTaskWriteMetadata, writeTask } from '../task/write.js';
-import { parseGitHubRemote, resolvePlatformContext, resolvePlatformProviderContext } from './context.js';
-import { createGitHubClient } from './github-client.js';
+import { resolvePlatformProviderContext } from './context.js';
 import { inspectPlatformIssue } from './issues.js';
-import { inspectGitHubIssue } from './issues.js';
 import { planPullRequestMetadata } from './pull-request-metadata.js';
-import { extractPullRequestFileNames, planInLabelUpdate, syncLabelDelta, validateInLabelMapping, validateRepositoryLabelPayload } from './in-label-sync.js';
+import { planInLabelUpdate, validateInLabelMapping } from './in-label-sync.js';
 import { platformResult } from './types.js';
 import { inspectCompletionArtifacts } from '../task/finalization-artifacts.js';
 import { TaskExecutionLockError, withTaskExecutionLock } from '../task/task-execution-lock.js';
@@ -18,7 +17,7 @@ import { resolveDeliveryTarget } from '../task/delivery-target.js';
 import { mergeOperationWarnings } from '../task/operation-outcome.js';
 import { buildBoundFact, buildSkippedFact, factFrontmatterMutation, readPrDeliveryFact } from '../task/pr-delivery-fact.js';
 import { providerError, providerOperationContext, providerStatus, providerResourceToken, providerResourceIdentity, resourceIdentity, unsupportedProviderOperation } from './provider-bridge.js';
-import { isResourceIdentity, resourceIdentityEquals, resourceIdentityNumber } from './resource-identity.js';
+import { isResourceIdentity, resourceIdentityEquals, resourceIdentityNumber, serializeResourceIdentity } from './resource-identity.js';
 import { taskIssueIdentity, taskIssueIdentityError } from './task-identities.js';
 function warningResultForPrimary(primaryResult) {
     if (primaryResult === 'pr_created')
@@ -56,56 +55,6 @@ function withCreation(output, creation) {
 const PRECONDITION_NOT_CREATED = {
     kind: 'not-created', reason: 'precondition-failed', createdByCurrentOperation: false
 };
-function normalizePullRequest(remote, repository) {
-    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', detail: mergeabilityDetail }
-        : remote.mergeable === true && mergeabilityDetail !== 'dirty'
-            ? { state: 'mergeable', detail: mergeabilityDetail }
-            : { state: 'unknown', 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, repository) {
-    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 repositoryKey(repository) {
     return repository.trim().toLowerCase();
 }
@@ -185,51 +134,6 @@ function selectExternalPullRequest(candidates, repository, existingPrIdentity, e
         };
     return { status: 'selected', source: 'unique', selected: eligible[0], candidates, eligible };
 }
-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 inspectGitHubIssueClosingChangeRequests(client, repository, issueNumber, cwd) {
-    const [owner, name] = repository.split('/');
-    if (!owner || !name || !Number.isInteger(issueNumber) || issueNumber <= 0)
-        return {
-            ok: false,
-            error: { code: 'PR_IDENTITY_INVALID', message: 'Repository or Issue identity is invalid', retryable: false }
-        };
-    const candidates = [];
-    let cursor = 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(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,
-                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,
-                    error: { code: 'PR_IDENTITY_INVALID', message: 'Closing pull request identity is incomplete', retryable: false }
-                };
-            candidates.push(normalized);
-        }
-        if (!connection.pageInfo.hasNextPage)
-            return { ok: true, value: candidates };
-        if (!connection.pageInfo.endCursor || connection.pageInfo.endCursor === cursor)
-            return {
-                ok: false,
-                error: { code: 'PR_IDENTITY_INVALID', message: 'Closing pull request pagination cursor is invalid', retryable: false }
-            };
-        cursor = connection.pageInfo.endCursor;
-    }
-}
 function expectedHead(repository, head) {
     const colon = head.indexOf(':');
     if (colon === -1)
@@ -422,15 +326,6 @@ async function inspectExternalPullRequest(base, prIdentity) {
         error: null
     });
 }
-function inspectGitHubPullRequest(client, repository, number, cwd) {
-    const fetched = client.json(['api', `repos/${repository}/pulls/${number}`], { cwd });
-    if (!fetched.ok)
-        return fetched;
-    const pullRequest = normalizePullRequest(fetched.value, repository);
-    return pullRequest
-        ? { ok: true, value: pullRequest }
-        : { ok: false, error: { code: 'PR_IDENTITY_INVALID', message: 'Remote resource is not a valid pull request', retryable: false } };
-}
 async function inspectPlatformPullRequest(taskRef, options = {}) {
     const base = await resolvedContext(taskRef, options);
     if (!base.ok)
@@ -487,7 +382,6 @@ async function inspectPlatformPullRequestByNumber(prNumber, options = {}) {
         error: { code: 'PLATFORM_PROVIDER_LOAD_FAILED', message: 'Selected provider failed to load', retryable: false }
     });
 }
-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 readInLabelMapping(repoRoot) {
     try {
         const config = JSON.parse(fs.readFileSync(path.join(repoRoot, '.agents', '.airc.json'), 'utf8'));
@@ -528,105 +422,19 @@ function taskInLabelTarget(repoRoot, frontmatter, currentLabels, repositoryLabel
         return { ok: false, error: planned.error };
     return { ok: true, value: planned.target };
 }
-function inspectClosingIssueNumbers(client, repository, pullRequest, cwd) {
-    const [owner, name] = repository.split('/');
-    if (!owner || !name)
-        return {
-            ok: false,
-            error: { code: 'IN_LABEL_SYNC_REPOSITORY_INVALID', message: 'Repository identity is invalid', retryable: false }
-        };
-    let cursor = null;
-    const numbers = [];
-    for (;;) {
-        const args = ['api', 'graphql', '-f', `query=${CLOSING_ISSUES_QUERY}`, '-F', `owner=${owner}`, '-F', `name=${name}`, '-F', `number=${pullRequest}`];
-        if (cursor)
-            args.push('-F', `cursor=${cursor}`);
-        const response = client.json(args, { cwd });
-        if (!response.ok)
-            return response;
-        const connection = response.value?.data?.repository?.pullRequest?.closingIssuesReferences;
-        if (!connection || !Array.isArray(connection.nodes) || !connection.pageInfo)
-            return {
-                ok: false,
-                error: { code: 'IN_LABEL_SYNC_ASSOCIATION_INVALID', message: 'Closing Issue response is incomplete', retryable: false }
-            };
-        for (const node of connection.nodes) {
-            const issue = Number(node?.number);
-            if (!Number.isSafeInteger(issue) || issue <= 0)
-                return {
-                    ok: false,
-                    error: { code: 'IN_LABEL_SYNC_ASSOCIATION_INVALID', message: 'Closing Issue identity is invalid', retryable: false }
-                };
-            numbers.push(issue);
-        }
-        if (!connection.pageInfo.hasNextPage)
-            return { ok: true, value: [...new Set(numbers)] };
-        if (!connection.pageInfo.endCursor || connection.pageInfo.endCursor === cursor)
-            return {
-                ok: false,
-                error: { code: 'IN_LABEL_SYNC_ASSOCIATION_INVALID', message: 'Closing Issue pagination cursor is invalid', retryable: false }
-            };
-        cursor = connection.pageInfo.endCursor;
-    }
-}
-function inspectRepositoryLabels(client, repository, cwd) {
-    const response = client.json(['api', '--paginate', '--slurp', `repos/${repository}/labels?per_page=100`], { cwd });
-    if (!response.ok)
-        return response;
-    const labels = validateRepositoryLabelPayload(response.value);
-    return labels.ok ? { ok: true, value: new Set(labels.value) } : labels;
+function resourceIdentityLabel(identity) {
+    const number = resourceIdentityNumber(identity);
+    return number === null ? `${identity.kind} ${identity.value}` : `#${number}`;
 }
-function inspectPullRequestFiles(client, repository, pullRequest, cwd) {
-    const response = client.json(['api', '--paginate', '--slurp', `repos/${repository}/pulls/${pullRequest}/files?per_page=100`], { cwd });
-    if (!response.ok)
-        return response;
-    const files = extractPullRequestFileNames(response.value);
-    return files ? { ok: true, value: files } : {
-        ok: false,
-        error: { code: 'IN_LABEL_SYNC_FILES_INVALID', message: 'Pull request files response is incomplete', retryable: false }
-    };
-}
-function inspectTaskInLabelTarget(repoRoot, frontmatter, pullRequest, repository, client) {
-    const baseRef = typeof frontmatter.delivery_base_ref === 'string' ? frontmatter.delivery_base_ref.trim() : '';
-    if (!baseRef)
-        return {
-            ok: false,
-            error: { code: 'IN_LABEL_SYNC_BASE_MISSING', message: 'Task has no delivery_base_ref for in-label evidence', retryable: false }
-        };
-    let changedFiles;
-    try {
-        changedFiles = execFileSync('git', ['diff', `${baseRef}...HEAD`, '--name-only'], {
-            cwd: repoRoot, encoding: 'utf8'
-        }).trim().split(/\r?\n/).filter(Boolean);
-    }
-    catch (error) {
-        return {
-            ok: false,
-            error: { code: 'IN_LABEL_SYNC_EVIDENCE_UNAVAILABLE', message: error instanceof Error ? error.message : String(error), retryable: false }
-        };
-    }
-    const mapping = readInLabelMapping(repoRoot);
-    if (!mapping.ok)
-        return mapping;
-    const labels = inspectRepositoryLabels(client, repository, repoRoot);
-    if (!labels.ok)
-        return labels;
-    const planned = planInLabelUpdate({
-        changedFiles, currentLabels: pullRequest.labels, mapping: mapping.value, repositoryLabels: labels.value
-    });
-    if (planned.error)
-        return { ok: false, error: planned.error };
-    return { ok: true, value: { changedFiles, target: planned.target } };
-}
-function inLabelResource(kind, number, before, expected, after, effect) {
-    return { kind, number, before: [...before].sort(), expected: [...expected].sort(), after: after ? [...after].sort() : null, effect };
+function inLabelResource(kind, identity, before, expected, after, effect) {
+    return { kind, number: resourceIdentityNumber(identity), identity, before: [...before].sort(), expected: [...expected].sort(), after: after ? [...after].sort() : null, effect };
 }
 async function syncPlatformPullRequestInLabels(prNumber, options = {}) {
     const cwd = path.resolve(options.cwd || process.cwd());
-    const client = options.client || createGitHubClient();
-    const context = await resolvePlatformContext({ cwd, client });
+    const loaded = await resolvePlatformProviderContext({ cwd, client: options.client });
+    const context = loaded.ok ? loaded.value.context : loaded.context;
     const usable = (context.status === 'no-op' || context.status === 'degraded') && context.platform.repository;
-    if (!usable)
+    if (!loaded.ok || !usable)
         return result(context.status, null, null, prNumber, {
             platform: context.platform, capabilities: context.capabilities, operations: context.operations, error: context.error
         });
@@ -635,68 +443,79 @@ async function syncPlatformPullRequestInLabels(prNumber, options = {}) {
             platform: context.platform, capabilities: context.capabilities,
             error: { code: 'PR_NUMBER_INVALID', message: 'PR number must be positive', retryable: false }
         });
+    const provider = loaded.value.provider;
+    const operationContext = providerOperationContext(loaded.value);
     const repository = context.platform.repository;
-    const fetched = inspectGitHubPullRequest(client, repository, prNumber, cwd);
-    if (!fetched.ok)
-        return result(fetched.error.retryable ? 'blocked' : 'failed', null, null, prNumber, {
-            platform: context.platform, capabilities: context.capabilities, resource: { kind: 'pull-request', number: prNumber }, error: fetched.error
-        });
-    const files = inspectPullRequestFiles(client, repository, prNumber, cwd);
+    const pullRequestIdentity = providerResourceToken(provider, 'pull-request', String(prNumber));
+    const providerFailure = (error) => result(error.code === 'PLATFORM_CAPABILITY_UNSUPPORTED' ? 'degraded' : providerStatus(error), null, null, prNumber, {
+        platform: context.platform,
+        capabilities: context.capabilities,
+        resource: { kind: 'pull-request', number: prNumber, identity: pullRequestIdentity },
+        error: providerError(error, 'PLATFORM_PROVIDER_OPERATION_FAILED')
+    });
+    const inspected = provider.changeRequests?.inspect
+        ? await provider.changeRequests.inspect({ context: operationContext, target: pullRequestIdentity })
+        : unsupportedProviderOperation(provider, 'changeRequests.inspect');
+    if (!inspected.ok)
+        return providerFailure(inspected.error);
+    const fetched = normalizeProviderPullRequest(inspected.value, repository, prNumber);
+    const files = provider.changeRequests?.listFiles
+        ? await provider.changeRequests.listFiles({ context: operationContext, target: pullRequestIdentity })
+        : unsupportedProviderOperation(provider, 'changeRequests.listFiles');
     if (!files.ok)
-        return result(files.error.retryable ? 'blocked' : 'failed', null, null, prNumber, {
-            platform: context.platform, capabilities: context.capabilities, pullRequest: fetched.value,
-            resource: { kind: 'pull-request', number: prNumber }, error: files.error
-        });
+        return providerFailure(files.error);
     const mapping = readInLabelMapping(cwd);
     if (!mapping.ok)
         return result('failed', null, null, prNumber, {
-            platform: context.platform, capabilities: context.capabilities, pullRequest: fetched.value,
-            resource: { kind: 'pull-request', number: prNumber }, error: mapping.error
-        });
-    const repositoryLabels = inspectRepositoryLabels(client, repository, cwd);
-    if (!repositoryLabels.ok)
-        return result(repositoryLabels.error.retryable ? 'blocked' : 'failed', null, null, prNumber, {
-            platform: context.platform, capabilities: context.capabilities, pullRequest: fetched.value,
-            resource: { kind: 'pull-request', number: prNumber }, error: repositoryLabels.error
+            platform: context.platform, capabilities: context.capabilities, pullRequest: fetched,
+            resource: { kind: 'pull-request', number: prNumber, identity: pullRequestIdentity }, error: mapping.error
         });
+    let repositoryLabels = new Set();
+    if (Object.keys(mapping.value).length > 0) {
+        const labels = provider.issues?.listLabels
+            ? await provider.issues.listLabels({ context: operationContext })
+            : unsupportedProviderOperation(provider, 'issues.listLabels');
+        if (!labels.ok)
+            return providerFailure(labels.error);
+        repositoryLabels = new Set(labels.value);
+    }
     const plannedTarget = planInLabelUpdate({
-        changedFiles: files.value, currentLabels: fetched.value.labels,
-        mapping: mapping.value, repositoryLabels: repositoryLabels.value
+        changedFiles: files.value, currentLabels: fetched.labels,
+        mapping: mapping.value, repositoryLabels
     });
     if (plannedTarget.error)
         return result('failed', null, null, prNumber, {
-            platform: context.platform, capabilities: context.capabilities, pullRequest: fetched.value,
-            resource: { kind: 'pull-request', number: prNumber }, error: plannedTarget.error
+            platform: context.platform, capabilities: context.capabilities, pullRequest: fetched,
+            resource: { kind: 'pull-request', number: prNumber, identity: pullRequestIdentity }, error: plannedTarget.error
         });
-    const target = plannedTarget.target;
-    const association = inspectClosingIssueNumbers(client, repository, prNumber, cwd);
+    const association = provider.changeRequests?.listClosingIssues
+        ? await provider.changeRequests.listClosingIssues({ context: operationContext, target: pullRequestIdentity })
+        : unsupportedProviderOperation(provider, 'changeRequests.listClosingIssues');
     if (!association.ok)
-        return result(association.error.retryable ? 'blocked' : 'failed', null, null, prNumber, {
-            platform: context.platform, capabilities: context.capabilities, pullRequest: fetched.value,
-            resource: { kind: 'pull-request', number: prNumber }, evidence: { kind: 'pull-request-files', pullRequestFiles: files.value }, error: association.error
-        });
-    const issueNumber = association.value.length === 1 ? association.value[0] : null;
+        return providerFailure(association.error);
+    const issueIdentity = association.value.length === 1 ? association.value[0] : null;
+    const issueNumber = issueIdentity ? resourceIdentityNumber(issueIdentity) : null;
+    const closingIssueIdentities = association.value;
     let issue = null;
-    if (issueNumber) {
-        const fetchedIssue = inspectGitHubIssue(client, repository, issueNumber, cwd);
+    if (issueIdentity) {
+        const fetchedIssue = provider.issues?.inspect
+            ? await provider.issues.inspect({ context: operationContext, target: issueIdentity })
+            : unsupportedProviderOperation(provider, 'issues.inspect');
         if (!fetchedIssue.ok)
-            return result(fetchedIssue.error.retryable ? 'blocked' : 'failed', null, issueNumber, prNumber, {
-                platform: context.platform, capabilities: context.capabilities, pullRequest: fetched.value,
-                resource: { kind: 'issue', number: issueNumber }, evidence: { kind: 'pull-request-files', pullRequestFiles: files.value, closingIssues: association.value }, error: fetchedIssue.error
-            });
+            return providerFailure(fetchedIssue.error);
         issue = fetchedIssue.value;
     }
     const resources = issue
         ? [
-            { kind: 'issue', number: issue.number, snapshot: issue },
-            { kind: 'pull-request', number: prNumber, snapshot: fetched.value }
+            { kind: 'issue', identity: issueIdentity, number: issueNumber, snapshot: issue },
+            { kind: 'pull-request', identity: pullRequestIdentity, number: prNumber, snapshot: fetched }
         ]
-        : [{ kind: 'pull-request', number: prNumber, snapshot: fetched.value }];
+        : [{ kind: 'pull-request', identity: pullRequestIdentity, number: prNumber, snapshot: fetched }];
     const plans = resources.map((resource) => ({
         ...resource,
         plan: planInLabelUpdate({
             changedFiles: files.value, currentLabels: resource.snapshot.labels,
-            mapping: mapping.value, repositoryLabels: repositoryLabels.value
+            mapping: mapping.value, repositoryLabels
         })
     }));
     const operations = [
@@ -708,63 +527,86 @@ async function syncPlatformPullRequestInLabels(prNumber, options = {}) {
         }))
     ];
     const willWrite = context.capabilities.triage && !options.dryRun;
-    const resourcesState = plans.map((item) => inLabelResource(item.kind, item.number, item.snapshot.labels, target, item.plan.changed && willWrite ? null : item.snapshot.labels, item.plan.changed && willWrite ? 'unknown' : 'no-op'));
+    const resourcesState = plans.map((item) => inLabelResource(item.kind, item.identity, item.snapshot.labels, item.plan.target, item.plan.changed && willWrite ? null : item.snapshot.labels, item.plan.changed && willWrite ? 'unknown' : 'no-op'));
     if (options.dryRun || !context.capabilities.triage)
         return result(!context.capabilities.triage || association.value.length !== 1 ? 'degraded' : plans.some((item) => item.plan.changed) ? 'planned' : 'no-op', null, issueNumber, prNumber, {
-            platform: context.platform, capabilities: context.capabilities, pullRequest: fetched.value,
-            resource: { kind: 'pull-request', number: prNumber }, operations, resources: resourcesState,
-            evidence: { kind: 'pull-request-files', pullRequestFiles: files.value, closingIssues: association.value }, error: null
+            platform: context.platform, capabilities: context.capabilities, pullRequest: fetched,
+            resource: { kind: 'pull-request', number: prNumber, identity: pullRequestIdentity }, operations, resources: resourcesState,
+            evidence: { kind: 'pull-request-files', pullRequestFiles: files.value, closingIssues: closingIssueIdentities }, error: null
         });
-    let currentPullRequest = fetched.value;
+    let currentPullRequest = fetched;
     let successfulWrites = 0;
     for (const item of plans) {
         if (!item.plan.changed)
             continue;
-        const synced = syncLabelDelta(client, repository, item.number, cwd, item.snapshot.labels, item.plan.target);
-        if (synced.status === 'failed' || synced.status === 'blocked') {
-            const state = resourcesState.find((resource) => resource.kind === item.kind && resource.number === item.number);
-            state.effect = synced.status === 'blocked' ? 'unknown' : 'no-op';
-            state.after = synced.status === 'blocked' ? null : item.snapshot.labels;
-            const partial = successfulWrites > 0 || synced.status === 'blocked';
-            const error = partial && synced.error
+        const targetIdentity = item.kind === 'issue' ? issueIdentity : pullRequestIdentity;
+        const synced = item.kind === 'issue'
+            ? provider.issues?.update
+                ? await provider.issues.update({
+                    context: operationContext,
+                    target: targetIdentity,
+                    currentLabels: item.snapshot.labels,
+                    patch: { labels: item.plan.labels },
+                    mutation: { idempotencyKey: `issue:update:${serializeResourceIdentity(targetIdentity)}:in-labels`, target: targetIdentity }
+                })
+                : unsupportedProviderOperation(provider, 'issues.update')
+            : provider.changeRequests?.update
+                ? await provider.changeRequests.update({
+                    context: operationContext,
+                    target: targetIdentity,
+                    currentLabels: item.snapshot.labels,
+                    patch: { labels: item.plan.labels },
+                    mutation: { idempotencyKey: `pull-request:update:${serializeResourceIdentity(targetIdentity)}:in-labels`, target: targetIdentity }
+                })
+                : unsupportedProviderOperation(provider, 'changeRequests.update');
+        if (!synced.ok) {
+            const state = resourcesState.find((resource) => resource.kind === item.kind && resourceIdentityEquals(resource.identity, item.identity));
+            state.effect = synced.error.retryable ? 'unknown' : 'no-op';
+            state.after = state.effect === 'unknown' ? null : item.snapshot.labels;
+            const partial = successfulWrites > 0 || synced.error.retryable;
+            const error = partial
                 ? { ...synced.error, code: 'IN_LABEL_SYNC_PARTIAL', message: `In-label synchronization is partial or unknown: ${synced.error.message}` }
                 : synced.error;
-            return result(partial ? 'blocked' : synced.status, null, issueNumber, prNumber, {
-                changed: successfulWrites > 0 || synced.changed, platform: context.platform, capabilities: context.capabilities,
-                pullRequest: currentPullRequest, resource: { kind: 'pull-request', number: prNumber }, operations, resources: resourcesState,
-                evidence: { kind: 'pull-request-files', pullRequestFiles: files.value, closingIssues: association.value }, error
+            return result(partial ? 'blocked' : providerStatus(synced.error), null, issueNumber, prNumber, {
+                changed: successfulWrites > 0, platform: context.platform, capabilities: context.capabilities,
+                pullRequest: currentPullRequest, resource: { kind: 'pull-request', number: prNumber, identity: pullRequestIdentity }, operations, resources: resourcesState,
+                evidence: { kind: 'pull-request-files', pullRequestFiles: files.value, closingIssues: closingIssueIdentities }, error: providerError(error, 'PLATFORM_PROVIDER_OPERATION_FAILED')
             });
         }
-        successfulWrites += synced.changed ? 1 : 0;
+        successfulWrites += synced.value.changed ? 1 : 0;
         const reread = item.kind === 'issue'
-            ? inspectGitHubIssue(client, repository, item.number, cwd)
-            : inspectGitHubPullRequest(client, repository, item.number, cwd);
+            ? provider.issues?.inspect
+                ? await provider.issues.inspect({ context: operationContext, target: targetIdentity })
+                : unsupportedProviderOperation(provider, 'issues.inspect')
+            : provider.changeRequests?.inspect
+                ? await provider.changeRequests.inspect({ context: operationContext, target: targetIdentity })
+                : unsupportedProviderOperation(provider, 'changeRequests.inspect');
         if (!reread.ok)
             return result('blocked', null, issueNumber, prNumber, {
                 changed: true, platform: context.platform, capabilities: context.capabilities,
-                pullRequest: currentPullRequest, resource: { kind: 'pull-request', number: prNumber }, operations, resources: resourcesState,
-                evidence: { kind: 'pull-request-files', pullRequestFiles: files.value, closingIssues: association.value },
-                error: { ...reread.error, code: 'IN_LABEL_SYNC_PARTIAL', message: `In-label synchronization is partial or unknown: ${reread.error.message}` }
+                pullRequest: currentPullRequest, resource: { kind: 'pull-request', number: prNumber, identity: pullRequestIdentity }, operations, resources: resourcesState,
+                evidence: { kind: 'pull-request-files', pullRequestFiles: files.value, closingIssues: closingIssueIdentities },
+                error: { ...providerError(reread.error, 'PLATFORM_PROVIDER_OPERATION_FAILED'), code: 'IN_LABEL_SYNC_PARTIAL', message: `In-label synchronization is partial or unknown: ${reread.error.message}` }
             });
-        const after = reread.value.labels;
-        const state = resourcesState.find((resource) => resource.kind === item.kind && resource.number === item.number);
+        const after = reread.value.labels || [];
+        const state = resourcesState.find((resource) => resource.kind === item.kind && resourceIdentityEquals(resource.identity, item.identity));
         state.after = after;
-        state.effect = after.filter((label) => label.startsWith('in:')).sort().join('\0') === target.join('\0') ? 'applied' : 'unknown';
+        state.effect = after.filter((label) => label.startsWith('in:')).sort().join('\0') === item.plan.target.join('\0') ? 'applied' : 'unknown';
         if (state.effect === 'unknown')
             return result('blocked', null, issueNumber, prNumber, {
                 changed: true, platform: context.platform, capabilities: context.capabilities,
-                pullRequest: currentPullRequest, resource: { kind: 'pull-request', number: prNumber }, operations, resources: resourcesState,
-                evidence: { kind: 'pull-request-files', pullRequestFiles: files.value, closingIssues: association.value },
-                error: { code: 'IN_LABEL_SYNC_PARTIAL', message: `${item.kind} #${item.number} did not converge to the expected in: labels`, retryable: true }
+                pullRequest: currentPullRequest, resource: { kind: 'pull-request', number: prNumber, identity: pullRequestIdentity }, operations, resources: resourcesState,
+                evidence: { kind: 'pull-request-files', pullRequestFiles: files.value, closingIssues: closingIssueIdentities },
+                error: { code: 'IN_LABEL_SYNC_PARTIAL', message: `${item.kind} ${resourceIdentityLabel(item.identity)} did not converge to the expected in: labels`, retryable: true }
             });
         if (item.kind === 'pull-request')
-            currentPullRequest = reread.value;
+            currentPullRequest = normalizeProviderPullRequest(reread.value, repository, prNumber, currentPullRequest);
     }
     return result(association.value.length !== 1 ? 'degraded' : successfulWrites > 0 ? 'applied' : 'no-op', null, issueNumber, prNumber, {
         changed: successfulWrites > 0, platform: context.platform, capabilities: context.capabilities,
-        pullRequest: currentPullRequest, resource: { kind: 'pull-request', number: prNumber },
+        pullRequest: currentPullRequest, resource: { kind: 'pull-request', number: prNumber, identity: pullRequestIdentity },
         operations: operations.map((operation) => operation.status === 'planned' ? { ...operation, status: 'applied' } : operation),
-        resources: resourcesState, evidence: { kind: 'pull-request-files', pullRequestFiles: files.value, closingIssues: association.value }, error: null
+        resources: resourcesState, evidence: { kind: 'pull-request-files', pullRequestFiles: files.value, closingIssues: closingIssueIdentities }, error: null
     });
 }
 function appendActivity(content, line) {
@@ -1518,6 +1360,6 @@ async function skipPlatformPullRequestFact(taskRef, options) {
         });
     }
 }
-export { bindPlatformPullRequest, createPlatformPullRequest, inspectGitHubPullRequest, inspectGitHubIssueClosingChangeRequests, inspectPlatformPullRequest, inspectPlatformPullRequestByNumber, normalizePullRequest, resolveExternalPullRequest, skipPlatformPullRequestFact, selectExternalPullRequest, selectPullRequest, syncPlatformPullRequest, syncPlatformPullRequestInLabels };
+export { bindPlatformPullRequest, createPlatformPullRequest, inspectPlatformPullRequest, inspectPlatformPullRequestByNumber, resolveExternalPullRequest, skipPlatformPullRequestFact, selectExternalPullRequest, selectPullRequest, syncPlatformPullRequest, syncPlatformPullRequestInLabels };
 export { warningResultForPrimary };
 //# sourceMappingURL=pull-requests.js.map