@fitlab-ai/agent-infra 0.9.13 → 0.9.14

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

@@ -5,12 +5,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 +16,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')
@@ -487,7 +485,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 +525,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 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 resourceIdentityLabel(identity) {
+    const number = resourceIdentityNumber(identity);
+    return number === null ? `${identity.kind} ${identity.value}` : `#${number}`;
 }
-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 +546,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 +630,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) {