@agent-relay/factory 0.1.26 → 0.1.27

package/dist/orchestrator/factory.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"factory.d.ts","sourceRoot":"","sources":["../../src/orchestrator/factory.ts"],"names":[],"mappings":"AAIA,OAAO,EAAuB,KAAK,aAAa,EAAE,MAAM,kBAAkB,CAAA;AAI1E,OAAO,KAAK,EAKV,WAAW,EAKX,WAAW,EAKZ,MAAM,UAAU,CAAA;AA+BjB,OAAO,KAAK,EACV,cAAc,EACd,OAAO,EACP,mBAAmB,EACnB,YAAY,EACZ,aAAa,EACb,mBAAmB,EAEnB,qBAAqB,EACrB,oBAAoB,EACpB,mBAAmB,EAGnB,QAAQ,EACR,eAAe,EACf,WAAW,EAGX,cAAc,EAEf,MAAM,UAAU,CAAA;AAOjB,KAAK,YAAY,GAAG,cAAc,GAAG,YAAY,GAAG,YAAY,GAAG,oBAAoB,GAAG,OAAO,CAAA;AACjG,KAAK,QAAQ,GAAG,CAAC,OAAO,EAAE,mBAAmB,KAAK,IAAI,CAAA;AAiEtD,KAAK,iBAAiB,GAAG;IACvB,KAAK,EAAE,MAAM,CAAA;IACb,QAAQ,EAAE,MAAM,CAAA;IAChB,IAAI,EAAE,MAAM,CAAA;IACZ,MAAM,EAAE,MAAM,CAAA;IACd,KAAK,EAAE,MAAM,CAAA;IACb,IAAI,EAAE,MAAM,CAAA;IACZ,GAAG,EAAE,MAAM,CAAA;IACX,KAAK,EAAE,MAAM,CAAA;IACb,MAAM,EAAE,MAAM,EAAE,CAAA;IAChB,MAAM,CAAC,EAAE,MAAM,CAAA;IACf,IAAI,EAAE,MAAM,CAAA;IACZ,GAAG,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAA;CAC7B,CAAA;AA8ED,eAAO,MAAM,mCAAmC,iDAAiD,CAAA;AACjG,eAAO,MAAM,kCAAkC,gDAAgD,CAAA;AAM/F,wBAAgB,aAAa,CAAC,MAAM,EAAE,aAAa,EAAE,KAAK,EAAE,YAAY,GAAG,OAAO,CAEjF;AAED,qBAAa,WAAY,YAAW,OAAO;;gBAgI7B,MAAM,EAAE,aAAa,EAAE,KAAK,EAAE,YAAY;IA+GhD,uBAAuB,CAAC,KAAK,EAAE,QAAQ,GAAG,OAAO,CAAC,IAAI,CAAC;IAiEvD,KAAK,CAAC,IAAI,GAAE,mBAAwB,GAAG,OAAO,CAAC,IAAI,CAAC;IAiGpD,IAAI,IAAI,OAAO,CAAC,IAAI,CAAC;IAsHrB,OAAO,IAAI,OAAO,CAAC,IAAI,CAAC;IAklBxB,OAAO,CAAC,IAAI,GAAE;QAAE,MAAM,CAAC,EAAE,OAAO,CAAA;KAAO,GAAG,OAAO,CAAC,eAAe,CAAC;IA2NlE,OAAO,CAAC,IAAI,GAAE,qBAA0B,GAAG,OAAO,CAAC,eAAe,EAAE,CAAC;IAyDrE,WAAW,CAAC,KAAK,EAAE,WAAW,GAAG,OAAO,CAAC,cAAc,CAAC;IAOxD,QAAQ,CAAC,QAAQ,EAAE,cAAc,EAAE,IAAI,GAAE;QAAE,MAAM,CAAC,EAAE,OAAO,CAAA;KAAO,GAAG,OAAO,CAAC,cAAc,CAAC;IA6OlG,MAAM,IAAI,aAAa;IAWvB,EAAE,CAAC,KAAK,EAAE,YAAY,EAAE,QAAQ,EAAE,QAAQ,GAAG,MAAM,IAAI;CA2wMxD;AAED,wBAAgB,gBAAgB,CAAC,IAAI,EAAE,MAAM,EAAE,OAAO,EAAE,OAAO,GAAG,WAAW,CAiC5E;AAyBD,wBAAsB,oCAAoC,CACxD,KAAK,EAAE,IAAI,CAAC,WAAW,EAAE,UAAU,CAAC,EACpC,IAAI,EAAE,MAAM,GACX,OAAO,CAAC,WAAW,CAAC,CAetB;AAED,wBAAgB,gBAAgB,CAAC,IAAI,EAAE,MAAM,EAAE,OAAO,EAAE,OAAO,GAAG,iBAAiB,CA6BlF;AAED,wBAAgB,uBAAuB,CAAC,IAAI,EAAE,MAAM,EAAE,OAAO,EAAE,OAAO,GAAG,WAAW,CAEnF;AA2CD,wBAAsB,wBAAwB,CAC5C,IAAI,SAAsC,GACzC,OAAO,CAAC,oBAAoB,GAAG,SAAS,CAAC,CAM3C;AAED,wBAAgB,wBAAwB,CACtC,SAAS,EAAE,oBAAoB,GAAG,SAAS,EAC3C,IAAI,GAAE;IAAE,KAAK,CAAC,EAAE,MAAM,CAAC;IAAC,OAAO,CAAC,EAAE,MAAM,CAAA;CAAO,GAC9C,mBAAmB,CAgBrB;AAED,wBAAgB,iBAAiB,CAAC,KAAK,EAAE,WAAW,GAAG,OAAO,CAO7D;AAED,wBAAgB,mBAAmB,CAAC,KAAK,EAAE,WAAW,GAAG,OAAO,CAc/D;AAufD,eAAO,MAAM,oBAAoB,GAAI,MAAM,MAAM,KAAG;IAAE,KAAK,EAAE,MAAM,CAAC;IAAC,IAAI,EAAE,MAAM,CAAC;IAAC,MAAM,EAAE,MAAM,CAAC;IAAC,IAAI,CAAC,EAAE,MAAM,CAAA;CAAE,GAAG,SAwBpH,CAAA;AAiTD,eAAO,MAAM,2BAA2B,GAAI,QAAQ,aAAa,KAAG,MAAM,EAItE,CAAA;AA6gBJ,eAAO,MAAM,WAAW,GAAI,MAAM,MAAM,KAAG,MAC4B,CAAA;AA0SvE,eAAO,MAAM,eAAe,GAAI,OAAO,WAAW,KAAG,MAAM,GAAG,SAG7D,CAAA"}
+{"version":3,"file":"factory.d.ts","sourceRoot":"","sources":["../../src/orchestrator/factory.ts"],"names":[],"mappings":"AAIA,OAAO,EAAuB,KAAK,aAAa,EAAE,MAAM,kBAAkB,CAAA;AAI1E,OAAO,KAAK,EAKV,WAAW,EAMX,WAAW,EAKZ,MAAM,UAAU,CAAA;AA+BjB,OAAO,KAAK,EACV,cAAc,EACd,OAAO,EACP,mBAAmB,EACnB,YAAY,EACZ,aAAa,EACb,mBAAmB,EAEnB,qBAAqB,EACrB,oBAAoB,EACpB,mBAAmB,EAGnB,QAAQ,EACR,eAAe,EACf,WAAW,EAGX,cAAc,EAEf,MAAM,UAAU,CAAA;AAOjB,KAAK,YAAY,GAAG,cAAc,GAAG,YAAY,GAAG,YAAY,GAAG,oBAAoB,GAAG,OAAO,CAAA;AACjG,KAAK,QAAQ,GAAG,CAAC,OAAO,EAAE,mBAAmB,KAAK,IAAI,CAAA;AAqEtD,KAAK,iBAAiB,GAAG;IACvB,KAAK,EAAE,MAAM,CAAA;IACb,QAAQ,EAAE,MAAM,CAAA;IAChB,IAAI,EAAE,MAAM,CAAA;IACZ,MAAM,EAAE,MAAM,CAAA;IACd,KAAK,EAAE,MAAM,CAAA;IACb,IAAI,EAAE,MAAM,CAAA;IACZ,GAAG,EAAE,MAAM,CAAA;IACX,KAAK,EAAE,MAAM,CAAA;IACb,MAAM,EAAE,MAAM,EAAE,CAAA;IAChB,MAAM,CAAC,EAAE,MAAM,CAAA;IACf,IAAI,EAAE,MAAM,CAAA;IACZ,GAAG,EAAE,MAAM,CAAC,MAAM,EAAE,OAAO,CAAC,CAAA;CAC7B,CAAA;AA8ED,eAAO,MAAM,mCAAmC,iDAAiD,CAAA;AACjG,eAAO,MAAM,kCAAkC,gDAAgD,CAAA;AAM/F,wBAAgB,aAAa,CAAC,MAAM,EAAE,aAAa,EAAE,KAAK,EAAE,YAAY,GAAG,OAAO,CAEjF;AAED,qBAAa,WAAY,YAAW,OAAO;;gBAmI7B,MAAM,EAAE,aAAa,EAAE,KAAK,EAAE,YAAY;IA+GhD,uBAAuB,CAAC,KAAK,EAAE,QAAQ,GAAG,OAAO,CAAC,IAAI,CAAC;IAiEvD,KAAK,CAAC,IAAI,GAAE,mBAAwB,GAAG,OAAO,CAAC,IAAI,CAAC;IAiGpD,IAAI,IAAI,OAAO,CAAC,IAAI,CAAC;IAsHrB,OAAO,IAAI,OAAO,CAAC,IAAI,CAAC;IAulBxB,OAAO,CAAC,IAAI,GAAE;QAAE,MAAM,CAAC,EAAE,OAAO,CAAA;KAAO,GAAG,OAAO,CAAC,eAAe,CAAC;IA4YlE,OAAO,CAAC,IAAI,GAAE,qBAA0B,GAAG,OAAO,CAAC,eAAe,EAAE,CAAC;IAyDrE,WAAW,CAAC,KAAK,EAAE,WAAW,GAAG,OAAO,CAAC,cAAc,CAAC;IAOxD,QAAQ,CAAC,QAAQ,EAAE,cAAc,EAAE,IAAI,GAAE;QAAE,MAAM,CAAC,EAAE,OAAO,CAAA;KAAO,GAAG,OAAO,CAAC,cAAc,CAAC;IA6OlG,MAAM,IAAI,aAAa;IAWvB,EAAE,CAAC,KAAK,EAAE,YAAY,EAAE,QAAQ,EAAE,QAAQ,GAAG,MAAM,IAAI;CA8zMxD;AAED,wBAAgB,gBAAgB,CAAC,IAAI,EAAE,MAAM,EAAE,OAAO,EAAE,OAAO,GAAG,WAAW,CAiC5E;AAyBD,wBAAsB,oCAAoC,CACxD,KAAK,EAAE,IAAI,CAAC,WAAW,EAAE,UAAU,CAAC,EACpC,IAAI,EAAE,MAAM,GACX,OAAO,CAAC,WAAW,CAAC,CAetB;AAED,wBAAgB,gBAAgB,CAAC,IAAI,EAAE,MAAM,EAAE,OAAO,EAAE,OAAO,GAAG,iBAAiB,CA6BlF;AAED,wBAAgB,uBAAuB,CAAC,IAAI,EAAE,MAAM,EAAE,OAAO,EAAE,OAAO,GAAG,WAAW,CAEnF;AA2CD,wBAAsB,wBAAwB,CAC5C,IAAI,SAAsC,GACzC,OAAO,CAAC,oBAAoB,GAAG,SAAS,CAAC,CAM3C;AAED,wBAAgB,wBAAwB,CACtC,SAAS,EAAE,oBAAoB,GAAG,SAAS,EAC3C,IAAI,GAAE;IAAE,KAAK,CAAC,EAAE,MAAM,CAAC;IAAC,OAAO,CAAC,EAAE,MAAM,CAAA;CAAO,GAC9C,mBAAmB,CAgBrB;AAED,wBAAgB,iBAAiB,CAAC,KAAK,EAAE,WAAW,GAAG,OAAO,CAO7D;AAED,wBAAgB,mBAAmB,CAAC,KAAK,EAAE,WAAW,GAAG,OAAO,CAc/D;AAufD,eAAO,MAAM,oBAAoB,GAAI,MAAM,MAAM,KAAG;IAAE,KAAK,EAAE,MAAM,CAAC;IAAC,IAAI,EAAE,MAAM,CAAC;IAAC,MAAM,EAAE,MAAM,CAAC;IAAC,IAAI,CAAC,EAAE,MAAM,CAAA;CAAE,GAAG,SAwBpH,CAAA;AAiVD,eAAO,MAAM,2BAA2B,GAAI,QAAQ,aAAa,KAAG,MAAM,EAItE,CAAA;AA8gBJ,eAAO,MAAM,WAAW,GAAI,MAAM,MAAM,KAAG,MAC4B,CAAA;AA0SvE,eAAO,MAAM,eAAe,GAAI,OAAO,WAAW,KAAG,MAAM,GAAG,SAG7D,CAAA"}

package/dist/orchestrator/factory.js

@@ -129,6 +129,9 @@ export class FactoryLoop {
     #githubIssueCommentWatchers = new Map();
     #githubIssueCommentWatchStates = new Map();
     #githubIssueCommentQueues = new Map();
+    #githubIssueAuthors = new Map();
+    #githubIssueAuthorLookups = new Map();
+    #reconciledGithubInProgress = new Set();
     #resolvedSlackChannelDir;
     #slackChannelDirRefresh;
     // Agents we've already logged an ambiguous-PID-lookup warning for, so the
@@ -947,12 +950,16 @@ export class FactoryLoop {
             return { dispatchRelayflow: true };
         }
         if (isGithubIssueFilePath(path)) {
-            const sourceKey = `github:${path}`;
+            const parts = githubIssuePathParts(path);
+            const sourceKey = parts
+                ? `github:${githubIssueIdentity(parts.owner, parts.repo, parts.number)}`
+                : `github:${path}`;
             if (seenIssueKeys.has(sourceKey)) {
                 this.#increment('liveDuplicateIssueEventsSuppressed');
-                this.#logger.debug?.('[factory] suppressed duplicate live GitHub issue event in current drain', {
+                this.#logger.debug?.('[factory] suppressed duplicate live GitHub issue alias in current drain', {
                     id: event.id,
                     path,
+                    issue: parts ? sourceKey.slice('github:'.length) : undefined,
                 });
                 return { dispatchRelayflow: false };
             }
@@ -1131,7 +1138,8 @@ export class FactoryLoop {
         });
     }
     async #resolveIssuePr(issue, opts = {}) {
-        const key = issueStateKey(issueRef(issue));
+        const issueKey = issueStateKey(issueRef(issue));
+        const key = opts.openOnly ? `${issueKey}:open` : issueKey;
         const now = this.#clock.now();
         const cached = this.#probePrResolvedCache.get(key);
         if (cached && cached.expiresAtMs > now) {
@@ -1176,6 +1184,9 @@ export class FactoryLoop {
                 await this.#ensureGithubIngestionReady();
             }
             const paths = await this.#readyIssuePaths();
+            const orphanRecovery = issueSource === 'github'
+                ? await this.#githubOrphanRecoveryContext()
+                : undefined;
             const pulled = [];
             const triaged = [];
             const dispatched = [];
@@ -1195,36 +1206,74 @@ export class FactoryLoop {
                     continue;
                 }
                 pulled.push(issueRef(issue));
-                const dispatchBlock = await this.#dispatchBlockReason(issue);
-                if (dispatchBlock) {
-                    skipped.push({ issue: issueRef(issue), reason: dispatchBlock });
-                    continue;
+                const wasReady = this.#isIssueReady(issue);
+                const labels = isGithubIssue(issue)
+                    ? new Set(issue.labels.map((label) => label.trim().toLowerCase()))
+                    : undefined;
+                const requiredLabel = this.#config.safety.requireLabel.trim().toLowerCase();
+                const mayRecoverGithubOrphan = !wasReady &&
+                    !dryRun &&
+                    issueSource === 'github' &&
+                    Boolean(orphanRecovery) &&
+                    Boolean(requiredLabel) &&
+                    Boolean(labels?.has(requiredLabel)) &&
+                    Boolean(labels?.has('factory:in-progress')) &&
+                    !labels?.has('factory:human-review');
+                if (!mayRecoverGithubOrphan) {
+                    const dispatchBlock = await this.#dispatchBlockReason(issue);
+                    if (dispatchBlock) {
+                        skipped.push({ issue: issueRef(issue), reason: dispatchBlock });
+                        continue;
+                    }
                 }
                 const batch = await this.#batch();
                 if (batch.isInFlight(issue) || batch.isQueued(issue)) {
                     skipped.push({ issue: issueRef(issue), reason: 'already tracked' });
                     continue;
                 }
-                if (!this.#isIssueReady(issue)) {
+                const recoveredOrphan = mayRecoverGithubOrphan &&
+                    await this.#reconcileOrphanedGithubInProgress(issue, orphanRecovery, dryRun);
+                if (!wasReady && !recoveredOrphan) {
+                    if (mayRecoverGithubOrphan) {
+                        const dispatchBlock = await this.#dispatchBlockReason(issue);
+                        if (dispatchBlock) {
+                            skipped.push({ issue: issueRef(issue), reason: dispatchBlock });
+                            continue;
+                        }
+                    }
                     skipped.push({ issue: issueRef(issue), reason: 'live state is not ready-for-agent' });
                     continue;
                 }
-                if (!isInFactoryScope(issue, this.#config.safety)) {
-                    skipped.push({ issue: issueRef(issue), reason: 'not factory-e2e scope' });
-                    continue;
-                }
-                if (!isDispatchableIssue(issue)) {
-                    skipped.push({ issue: issueRef(issue), reason: 'not reconciled real Linear issue' });
-                    continue;
-                }
-                const decision = await this.triageIssue(issue);
-                triaged.push(decision);
-                const result = await this.dispatch(decision, { dryRun });
-                if (result.agents.length === 0 && !dryRun) {
-                    skipped.push({ issue: decision.issue, reason: 'queued or escalated' });
+                const recoveredIdentity = recoveredOrphan ? githubIssueRefIdentity(issueRef(issue)) : undefined;
+                try {
+                    if (recoveredOrphan) {
+                        const dispatchBlock = await this.#dispatchBlockReason(issue);
+                        if (dispatchBlock) {
+                            skipped.push({ issue: issueRef(issue), reason: dispatchBlock });
+                            continue;
+                        }
+                    }
+                    if (!isInFactoryScope(issue, this.#config.safety)) {
+                        skipped.push({ issue: issueRef(issue), reason: 'not factory-e2e scope' });
+                        continue;
+                    }
+                    if (!isDispatchableIssue(issue)) {
+                        skipped.push({ issue: issueRef(issue), reason: 'not reconciled real Linear issue' });
+                        continue;
+                    }
+                    const decision = await this.triageIssue(issue);
+                    triaged.push(decision);
+                    const result = await this.dispatch(decision, { dryRun });
+                    if (result.agents.length === 0 && !dryRun) {
+                        skipped.push({ issue: decision.issue, reason: 'queued or escalated' });
+                    }
+                    else {
+                        dispatched.push(result);
+                    }
                 }
-                else {
-                    dispatched.push(result);
+                finally {
+                    if (recoveredIdentity)
+                        this.#reconciledGithubInProgress.delete(recoveredIdentity);
                 }
             }
             report = { pulled, triaged, dispatched, skipped, dryRun, slackDegraded: this.#slackDegraded };
@@ -1255,6 +1304,140 @@ export class FactoryLoop {
             }
         }
     }
+    async #githubOrphanRecoveryContext() {
+        try {
+            const [registry, roster, lifecycles, waitingClarifications] = await Promise.all([
+                readFactoryInFlightRegistry(this.#config.loop.registryPath),
+                this.#fleet.roster(),
+                this.#state.listDispatchLifecycles(this.#workspaceId),
+                this.#state.listWaitingClarifications(this.#workspaceId),
+            ]);
+            const onlineAgents = new Set(roster.agents.map((agent) => agent.name));
+            const activeIssueIdentities = new Set();
+            for (const [, lifecycle] of lifecycles) {
+                if (isTerminalDispatchLifecycle(lifecycle))
+                    continue;
+                const identity = githubIssueRefIdentity(lifecycle.issue);
+                if (identity)
+                    activeIssueIdentities.add(identity);
+            }
+            for (const [, waiting] of waitingClarifications) {
+                const identity = githubIssueRefIdentity(waiting.issue);
+                if (identity)
+                    activeIssueIdentities.add(identity);
+            }
+            for (const agent of registry?.agents ?? []) {
+                if (!onlineAgents.has(agent.name) || !agent.issue)
+                    continue;
+                const identity = githubIssueRefIdentity(agent.issue);
+                if (identity)
+                    activeIssueIdentities.add(identity);
+            }
+            return {
+                activeIssueIdentities,
+                onlineAgentNames: onlineAgents,
+            };
+        }
+        catch (error) {
+            this.#increment('githubOrphanRecoveryContextFailures');
+            this.#logger.warn?.('[factory] could not establish orphan-recovery safety context; preserving in-progress issues', {
+                error: describeError(error).errorMessage,
+            });
+            return undefined;
+        }
+    }
+    async #reconcileOrphanedGithubInProgress(issue, context, dryRun) {
+        if (dryRun || !context || !isGithubIssue(issue))
+            return false;
+        const labels = new Set(issue.labels.map((label) => label.trim().toLowerCase()));
+        const required = this.#config.safety.requireLabel.trim().toLowerCase();
+        if (!required ||
+            !labels.has(required) ||
+            !labels.has('factory:in-progress') ||
+            labels.has('factory:human-review'))
+            return false;
+        const identity = githubIssueRefIdentity(issueRef(issue));
+        if (!identity ||
+            context.activeIssueIdentities.has(identity) ||
+            [...context.onlineAgentNames].some((name) => githubAgentNameMatchesIssue(name, issue))) {
+            this.#increment('githubOrphanRecoveriesBlockedActive');
+            return false;
+        }
+        const getProviderStatus = this.#githubWriteback.getIssueStatus;
+        if (!getProviderStatus) {
+            this.#increment('githubOrphanRecoveryStatusLookupUnavailable');
+            return false;
+        }
+        let providerStatus;
+        try {
+            providerStatus = await getProviderStatus.call(this.#githubWriteback, issue);
+        }
+        catch (error) {
+            this.#increment('githubOrphanRecoveryStatusLookupFailures');
+            this.#logger.warn?.('[factory] could not verify provider-authoritative GitHub issue status; preserving it', {
+                issue: issue.key,
+                error: describeError(error).errorMessage,
+            });
+            return false;
+        }
+        if (!providerStatus || providerStatus === 'human-review') {
+            this.#increment('githubOrphanRecoveriesBlockedProviderStatus');
+            return false;
+        }
+        let hasOpenPr;
+        try {
+            hasOpenPr = await this.#hasOpenCompletionPr(issue);
+        }
+        catch (error) {
+            this.#increment('githubOrphanRecoveryPrProbeFailures');
+            this.#logger.warn?.('[factory] could not prove an in-progress GitHub issue has no open PR; preserving it', {
+                issue: issue.key,
+                error: describeError(error).errorMessage,
+            });
+            return false;
+        }
+        if (hasOpenPr) {
+            this.#increment('githubOrphanRecoveriesBlockedOpenPr');
+            this.#logger.info?.('[factory] preserved in-progress GitHub issue because a matching open PR exists', {
+                issue: issue.key,
+            });
+            return false;
+        }
+        try {
+            if (providerStatus === 'in-progress') {
+                await this.#githubWriteback.setStatus(issue, 'ready');
+            }
+            // A crashed dispatch may leave its durable attempt marked in-flight even
+            // after every agent and lifecycle disappeared. Only clear that stale bit
+            // after all provider, agent, lifecycle, and open-PR safety checks pass.
+            await this.#clearDispatchInFlight(issue);
+            this.#reconciledGithubInProgress.add(identity);
+            this.#increment('githubOrphanedInProgressRecovered');
+            this.#logger.warn?.('[factory] recovered orphaned GitHub in-progress issue for redispatch', {
+                issue: issue.key,
+                path: issue.path,
+            });
+            return true;
+        }
+        catch (error) {
+            this.#increment('githubOrphanRecoveryWritebackFailures');
+            this.#logger.warn?.('[factory] failed to clear orphaned GitHub lifecycle status; preserving in-progress issue', {
+                issue: issue.key,
+                error: describeError(error).errorMessage,
+            });
+            return false;
+        }
+    }
+    async #hasOpenCompletionPr(issue) {
+        if (this.#customProbePrResolver) {
+            return Boolean(await this.#probePrResolver(issue));
+        }
+        return Boolean(await this.#resolveIssuePr(issue, {
+            titleMarker: FACTORY_E2E_MARKER,
+            openOnly: true,
+            failOnLookupError: true,
+        }));
+    }
     async #listRelayfileTree(prefix, phase) {
         return this.#withRelayfileOperation('listTree', { phase, prefix }, () => this.#mount.listTree(prefix), {
             count: (paths) => paths.length,
@@ -2185,11 +2368,14 @@ export class FactoryLoop {
         if (githubState === 'closed') {
             return false;
         }
-        if (issue.labels.some((label) => GITHUB_LIFECYCLE_LABELS.has(label.trim().toLowerCase()))) {
+        const labels = new Set(issue.labels.map((label) => label.trim().toLowerCase()));
+        if (labels.has('factory:human-review'))
+            return false;
+        const identity = githubIssueRefIdentity(issueRef(issue));
+        if (labels.has('factory:in-progress') && (!identity || !this.#reconciledGithubInProgress.has(identity)))
             return false;
-        }
         const required = this.#config.safety.requireLabel.trim().toLowerCase();
-        return Boolean(required) && issue.labels.some((label) => label.trim().toLowerCase() === required);
+        return Boolean(required) && labels.has(required);
     }
     async #postIssueComment(issue, body) {
         if (isGithubIssue(issue)) {
@@ -2274,12 +2460,22 @@ export class FactoryLoop {
     }
     async #githubIssuePaths() {
         try {
-            const issuePaths = new Set();
+            const issuePaths = new Map();
             for (const root of githubIssueScanRoots(this.#config)) {
                 const paths = await this.#listRelayfileTree(root, 'GitHub issue ingestion');
                 for (const path of paths) {
-                    if (githubIssuePathParts(path) !== undefined) {
-                        issuePaths.add(path);
+                    const parts = githubIssuePathParts(path);
+                    if (parts) {
+                        const identity = githubIssueIdentity(parts.owner, parts.repo, parts.number);
+                        const existing = issuePaths.get(identity);
+                        if (!existing || githubIssuePathPreference(path) < githubIssuePathPreference(existing)) {
+                            if (existing)
+                                this.#increment('githubIssueAliasPathsSuppressed');
+                            issuePaths.set(identity, path);
+                        }
+                        else {
+                            this.#increment('githubIssueAliasPathsSuppressed');
+                        }
                     }
                     else if (githubIssueDirectoryPathParts(path) !== undefined) {
                         // listTree returns the issue directory entry alongside its
@@ -2295,7 +2491,7 @@ export class FactoryLoop {
                     }
                 }
             }
-            return [...issuePaths].sort();
+            return [...issuePaths.values()].sort();
         }
         catch (error) {
             this.#increment('githubIssueListFailures');
@@ -4274,7 +4470,7 @@ export class FactoryLoop {
         const correlationId = githubEscalationCorrelationId('agent-question', record.issue, question.question);
         const issue = await this.#readIssue(record.issue.path);
         const source = issue ? githubIssueSourceRef(issue) : undefined;
-        const authorizedAuthor = issue ? githubIssueAuthor(issue) : undefined;
+        const authorizedAuthor = issue ? await this.#resolveGithubIssueAuthor(issue) : undefined;
         if (!issue || !source || !authorizedAuthor) {
             this.#surfaceEscalationDeliveryFailure('agent-question', record.issue, correlationId, fallbackReason
                 ? `${fallbackReason}; no GitHub issue write path with an identifiable issue reporter is available`
@@ -4770,7 +4966,7 @@ export class FactoryLoop {
         };
         const correlationId = githubEscalationCorrelationId('agent-question', record.issue, `${comment.commentId}:${question.question}`);
         const issue = await this.#readIssue(record.issue.path);
-        const authorizedAuthor = issue ? githubIssueAuthor(issue) : undefined;
+        const authorizedAuthor = issue ? await this.#resolveGithubIssueAuthor(issue) : undefined;
         if (!authorizedAuthor) {
             this.#increment('githubAgentQuestionsIgnoredMissingAuthorizedAuthor');
             this.#surfaceEscalationDeliveryFailure('agent-question', record.issue, correlationId, 'source GitHub issue has no identifiable reporter authorized to answer the durable question');
@@ -6407,7 +6603,7 @@ export class FactoryLoop {
         const correlationId = githubEscalationCorrelationId('triage', decision.issue, question);
         const issue = await this.#readIssue(decision.issue.path);
         const source = issue ? githubIssueSourceRef(issue) : undefined;
-        const authorizedAuthor = issue ? githubIssueAuthor(issue) : undefined;
+        const authorizedAuthor = issue ? await this.#resolveGithubIssueAuthor(issue) : undefined;
         if (!issue || !source || !authorizedAuthor) {
             this.#surfaceEscalationDeliveryFailure('triage', decision.issue, correlationId, 'no Slack channel or GitHub issue write path with an identifiable issue reporter is available');
             return;
@@ -6482,7 +6678,7 @@ export class FactoryLoop {
             return;
         const source = githubIssueSourceRef(issue);
         const stakeholderMentions = slackMentions(this.#config.slack.stakeholderUserIds);
-        const reporter = githubIssueAuthor(issue);
+        const reporter = await this.#resolveGithubIssueAuthor(issue);
         const audience = [stakeholderMentions, reporter ? `GitHub reporter: @${reporter}.` : undefined]
             .filter((part) => Boolean(part))
             .join(' ');
@@ -6501,6 +6697,46 @@ export class FactoryLoop {
         this.#increment('triageEscalationsMirroredToSlack');
         this.#recordSlackWritebackSuccess('triage-escalation-mirror');
     }
+    async #resolveGithubIssueAuthor(issue) {
+        const embeddedAuthor = githubIssueAuthor(issue);
+        if (embeddedAuthor)
+            return embeddedAuthor;
+        const source = githubIssueSourceRef(issue);
+        const lookup = this.#githubWriteback.getIssueAuthor;
+        if (!source || !lookup)
+            return undefined;
+        const key = githubIssueSourceKey(source);
+        if (this.#githubIssueAuthors.has(key)) {
+            return this.#githubIssueAuthors.get(key);
+        }
+        const existing = this.#githubIssueAuthorLookups.get(key);
+        if (existing)
+            return existing;
+        const pending = lookup.call(this.#githubWriteback, issue)
+            .then((author) => {
+            const normalized = author?.trim() || undefined;
+            this.#githubIssueAuthors.set(key, normalized);
+            if (normalized) {
+                this.#increment('githubIssueAuthorsResolvedFromProvider');
+            }
+            return normalized;
+        })
+            .catch((error) => {
+            this.#increment('githubIssueAuthorLookupFailures');
+            this.#logger.warn?.('[factory] provider-authoritative GitHub issue author lookup failed', {
+                owner: source.owner,
+                repo: source.repo,
+                issue: source.number,
+                error: describeError(error).errorMessage,
+            });
+            return undefined;
+        })
+            .finally(() => {
+            this.#githubIssueAuthorLookups.delete(key);
+        });
+        this.#githubIssueAuthorLookups.set(key, pending);
+        return pending;
+    }
     async #postAndWatchSlackEscalationThread(decision, reason) {
         if (!this.#slack || !this.#config.slack) {
             return;
@@ -8136,6 +8372,26 @@ export const githubIssuePathParts = (path) => {
         slug: match[7],
     };
 };
+const githubIssueIdentity = (owner, repo, number) => `${owner.toLowerCase()}/${repo.toLowerCase()}#${number}`;
+const githubIssueRefIdentity = (issue) => {
+    const parts = githubIssuePathParts(issue.path);
+    return parts ? githubIssueIdentity(parts.owner, parts.repo, parts.number) : undefined;
+};
+const githubIssuePathPreference = (path) => {
+    if (path.includes('/issues/by-id/'))
+        return 0;
+    if (path.endsWith('/meta.json'))
+        return 1;
+    if (path.endsWith('.json'))
+        return 2;
+    return 3;
+};
+const githubAgentNameMatchesIssue = (name, issue) => {
+    const parts = githubIssuePathParts(issue.path);
+    if (!parts)
+        return false;
+    return name.startsWith(`ar-${parts.number}-`) && name.endsWith(`-${sanitizeAgentSlug(parts.repo)}`);
+};
 const githubIssueCommentPathParts = (path) => {
     const match = path.match(/^\/github\/repos\/(?:([^/]+)\/([^/]+)|([A-Za-z0-9-]+)__([^/]+))\/issues\/(\d+)(?:__[^/]*)?\/comments\/([^/]+?)(?:\.json|\/(?:meta|metadata)\.json)$/u);
     const owner = match?.[1] ?? match?.[3];
@@ -8295,6 +8551,8 @@ const resolveIssuePrFromMount = async (mount, config, issue, opts = {}) => {
             if (!path.endsWith('.json'))
                 continue;
             const pr = await readProbePrCandidate(mount, path);
+            if (opts.openOnly && normalizePrState(pr?.state) !== 'OPEN')
+                continue;
             const score = pr
                 ? issuePrMatchScore(pr, issue, opts.titleMarker ?? config.safety.requireTitlePrefix, opts)
                 : 0;
@@ -8307,6 +8565,7 @@ const resolveIssuePrFromMount = async (mount, config, issue, opts = {}) => {
 };
 const resolveIssuePrFromGh = async (run, config, issue, opts = {}, logger) => {
     const candidates = [];
+    let lookupFailures = 0;
     for (const repo of reposFromConfig(config)) {
         let payload;
         try {
@@ -8323,26 +8582,33 @@ const resolveIssuePrFromGh = async (run, config, issue, opts = {}, logger) => {
                 String(PROBE_PR_GH_CANDIDATE_LIMIT),
             ]);
             if (!result.stdout.trim()) {
+                lookupFailures += 1;
                 logger?.warn?.('[factory] gh PR resolver returned empty output', { issue: issue.key, repo });
                 continue;
             }
             payload = parseJsonContent(result.stdout);
         }
         catch (error) {
+            lookupFailures += 1;
             logger?.warn?.('[factory] gh PR resolver failed', { issue: issue.key, repo, error });
             continue;
         }
         if (!Array.isArray(payload)) {
+            lookupFailures += 1;
             logger?.warn?.('[factory] gh PR resolver returned non-array payload', { issue: issue.key, repo });
             continue;
         }
         if (payload.length >= PROBE_PR_GH_CANDIDATE_LIMIT) {
             logger?.warn?.('[factory] gh PR resolver hit candidate limit', { issue: issue.key, repo, limit: PROBE_PR_GH_CANDIDATE_LIMIT });
+            if (opts.failOnLookupError)
+                lookupFailures += 1;
         }
         for (const entry of payload) {
             const pr = ghProbePrCandidate(entry);
             if (!pr || !containsIssueKey(pr.headRef, issue.key))
                 continue;
+            if (opts.openOnly && normalizePrState(pr.state) !== 'OPEN')
+                continue;
             const score = issuePrMatchScore(pr, issue, opts.titleMarker ?? config.safety.requireTitlePrefix, opts);
             if (score <= 0)
                 continue;
@@ -8355,9 +8621,13 @@ const resolveIssuePrFromGh = async (run, config, issue, opts = {}, logger) => {
             });
         }
     }
-    return candidates.sort((a, b) => b.score - a.score ||
+    const resolved = candidates.sort((a, b) => b.score - a.score ||
         Number(b.open) - Number(a.open) ||
         b.prNumber - a.prNumber)[0];
+    if (!resolved && opts.failOnLookupError && lookupFailures > 0) {
+        throw new Error(`Unable to confirm open pull request state for ${issue.key} in ${lookupFailures} configured repository lookup(s)`);
+    }
+    return resolved;
 };
 const reposFromConfig = (config) => {
     const repos = new Set([
@@ -8452,6 +8722,7 @@ const readProbePrCandidate = async (mount, path) => {
             body: stringValue(payload.body) ?? '',
             headRef: refName(payload.headRef) ?? refName(payload.head) ?? stringValue(payload.head_ref) ?? '',
             draft: booleanValue(payload.isDraft) ?? booleanValue(payload.draft),
+            state: stringValue(payload.state),
         };
     }
     catch {