@agent-relay/factory 0.1.7 → 0.1.8

package/dist/orchestrator/factory.js

@@ -10,7 +10,7 @@ import { isInFactoryScope } from '../safety/factory-scope.js';
 import { deriveDescriptorsFromMount, prescriptiveInstructions, } from '@agent-relay/integration-prompts';
 import { renderAgentTask } from '../dispatch/templates.js';
 import { HeuristicTriage, TieredTriage, babysitterSpec, isShapeLabel, scopeFromLabels } from '../triage/index.js';
-import { MountGithubRead, MountLinearWriteback, MountSlackWriteback } from '../writeback/index.js';
+import { MountGithubRead, MountLinearWriteback, MountSlackWriteback, slackChannelAliases, slackChannelSegment } from '../writeback/index.js';
 import { asRecord, parseJsonContent, stableHash, wrappedPayload } from '../writeback/shared.js';
 import { issueKey } from './batch-tracker.js';
 import { findAgentProcessByName, readProcessIdentity } from './process-identity.js';
@@ -98,6 +98,8 @@ export class FactoryLoop {
     #resumeInFlight = new Map();
     #slackWatchers = new Map();
     #slackWatcherStarts = new Map();
+    #resolvedSlackChannelDir;
+    #slackChannelDirRefresh;
     // Agents we've already logged an ambiguous-PID-lookup warning for, so the
     // reaper doesn't spam the same benign "ambiguous process lookup" line on every
     // poll (a joined/cloud agent has no local PID to resolve — expected).
@@ -2513,12 +2515,15 @@ export class FactoryLoop {
             });
             if (this.#slack && this.#config.slack && !await this.#shouldSkipSlackWriteback('merge-done-thread')) {
                 try {
-                    const root = await this.#slack.postThread({
-                        channel: this.#config.slack.channel,
-                        text: `${issue.key}: PR merged; Linear state set to Done.`,
-                    });
-                    await this.#slack.reply(root.threadId, `${issue.key}: Linear state set to Done.`);
-                    this.#recordSlackWritebackSuccess('merge-done-thread');
+                    const channel = await this.#slackChannelDir();
+                    if (channel) {
+                        const root = await this.#slack.postThread({
+                            channel,
+                            text: `${issue.key}: PR merged; Linear state set to Done.`,
+                        });
+                        await this.#slack.reply(root.threadId, `${issue.key}: Linear state set to Done.`);
+                        this.#recordSlackWritebackSuccess('merge-done-thread');
+                    }
                 }
                 catch (error) {
                     this.#markSlackWritebackFailure('merge-done-thread', error);
@@ -2752,21 +2757,24 @@ export class FactoryLoop {
             }
             if (this.#slack && this.#config.slack && !await this.#shouldSkipSlackWriteback('completion-thread')) {
                 try {
-                    const merged = opts.completionReason === 'pr-merged';
-                    const completionText = merged
-                        ? `${record.issue.key}: PR merged; Linear state set to ${statusLabel}.`
-                        : `${record.issue.key}: factory agents completed${humanReview ? '; awaiting human review' : ''}.\nStatus: ${statusLabel}\nMerge policy: ${this.#config.mergePolicy}`;
-                    const stateText = merged
-                        ? `${record.issue.key}: PR merged; Linear state set to ${statusLabel}.`
-                        : humanReview
-                            ? `${record.issue.key}: awaiting human review; Linear state set to ${statusLabel}.`
-                            : `${record.issue.key}: Linear state set to ${statusLabel}.`;
-                    const root = await this.#slack.postThread({
-                        channel: this.#config.slack.channel,
-                        text: completionText,
-                    });
-                    await this.#slack.reply(root.threadId, stateText);
-                    this.#recordSlackWritebackSuccess('completion-thread');
+                    const channel = await this.#slackChannelDir();
+                    if (channel) {
+                        const merged = opts.completionReason === 'pr-merged';
+                        const completionText = merged
+                            ? `${record.issue.key}: PR merged; Linear state set to ${statusLabel}.`
+                            : `${record.issue.key}: factory agents completed${humanReview ? '; awaiting human review' : ''}.\nStatus: ${statusLabel}\nMerge policy: ${this.#config.mergePolicy}`;
+                        const stateText = merged
+                            ? `${record.issue.key}: PR merged; Linear state set to ${statusLabel}.`
+                            : humanReview
+                                ? `${record.issue.key}: awaiting human review; Linear state set to ${statusLabel}.`
+                                : `${record.issue.key}: Linear state set to ${statusLabel}.`;
+                        const root = await this.#slack.postThread({
+                            channel,
+                            text: completionText,
+                        });
+                        await this.#slack.reply(root.threadId, stateText);
+                        this.#recordSlackWritebackSuccess('completion-thread');
+                    }
                 }
                 catch (error) {
                     this.#markSlackWritebackFailure('completion-thread', error);
@@ -3012,7 +3020,7 @@ export class FactoryLoop {
             return;
         }
         const root = await this.#slack.postThread({
-            channel: this.#config.slack.channel,
+            channel: await this.#slackChannelDir() ?? this.#config.slack.channel,
             text: [
                 `${record.issue.key}: factory agents dispatched.`,
                 `State: ${result.stateId ?? 'dispatching'}`,
@@ -3073,7 +3081,7 @@ export class FactoryLoop {
         }
         const issue = await this.#readIssue(decision.issue.path);
         const root = await this.#slack.postThread({
-            channel: this.#config.slack.channel,
+            channel: await this.#slackChannelDir() ?? this.#config.slack.channel,
             text: [
                 `${decision.issue.key}: factory triage escalation for ${issue?.title ?? decision.issue.key}`,
                 `Reason: ${reason}`,
@@ -3092,7 +3100,7 @@ export class FactoryLoop {
         if (this.#slackWatchers.has(key)) {
             return;
         }
-        const channelDir = this.#config.slack.channel;
+        const channelDir = await this.#slackChannelDir() ?? this.#config.slack.channel;
         const messagesPrefix = slackChannelMessagesPrefix(channelDir);
         const preExistingPaths = new Set();
         const seenReplies = new Set();
@@ -3311,13 +3319,78 @@ export class FactoryLoop {
     #integrationsMountRoot() {
         return resolve(process.cwd(), '.integrations');
     }
+    async #slackChannelDir() {
+        if (!this.#config.slack) {
+            return undefined;
+        }
+        if (this.#resolvedSlackChannelDir) {
+            return this.#resolvedSlackChannelDir;
+        }
+        if (this.#slackChannelDirRefresh) {
+            return this.#slackChannelDirRefresh;
+        }
+        this.#slackChannelDirRefresh = this.#resolveSlackChannelDir()
+            .finally(() => {
+            this.#slackChannelDirRefresh = undefined;
+        });
+        return this.#slackChannelDirRefresh;
+    }
+    async #resolveSlackChannelDir() {
+        const configured = this.#config.slack?.channel.trim();
+        if (!configured) {
+            return undefined;
+        }
+        const configuredSegment = slackChannelSegment(configured);
+        const configuredAliases = slackChannelAliases(configured);
+        if (configuredAliases.size === 0) {
+            return undefined;
+        }
+        let paths;
+        try {
+            paths = await this.#mount.listTree('/slack/channels');
+        }
+        catch (error) {
+            this.#logger.warn?.('[factory] unable to resolve Slack channel from mount; using configured channel value', {
+                channel: configured,
+                error,
+            });
+            this.#resolvedSlackChannelDir = configuredSegment;
+            return this.#resolvedSlackChannelDir;
+        }
+        const channelDirs = [...new Set(paths
+                .map((path) => path.match(/^\/slack\/channels\/([^/]+)/u)?.[1])
+                .filter((channelDir) => Boolean(channelDir)))]
+            .sort();
+        const matches = channelDirs.filter((channelDir) => {
+            const aliases = slackChannelAliases(channelDir);
+            return [...aliases].some((alias) => configuredAliases.has(alias));
+        });
+        if (matches.length === 0) {
+            this.#logger.warn?.('[factory] Slack channel was not found in mount; using configured channel value', {
+                channel: configured,
+            });
+            this.#resolvedSlackChannelDir = configuredSegment;
+            return this.#resolvedSlackChannelDir;
+        }
+        const exact = matches.find((channelDir) => slackChannelSegment(channelDir).toLowerCase() === configuredSegment.toLowerCase());
+        this.#resolvedSlackChannelDir = exact ?? matches[0];
+        if (matches.length > 1 && !exact) {
+            this.#logger.warn?.('[factory] Slack channel name matched multiple mount directories; using first match', {
+                channel: configured,
+                selected: this.#resolvedSlackChannelDir,
+                matches,
+            });
+        }
+        return this.#resolvedSlackChannelDir;
+    }
     async #slackDispatchThreadFor(record) {
         if (!this.#config.slack) {
             return undefined;
         }
         const threadId = await this.#state.getSlackThread(this.#workspaceId, issueKey(record.issue));
+        const channel = await this.#slackChannelDir() ?? this.#config.slack.channel;
         return threadId
-            ? { channel: this.#config.slack.channel, threadId, mountRoot: this.#integrationsMountRoot() }
+            ? { channel, threadId, mountRoot: this.#integrationsMountRoot() }
             : undefined;
     }
     async #runCompletionMergeGate(issue) {
@@ -3701,8 +3774,25 @@ function githubMirrorRepoForIssue(issue) {
     return githubRepoFromUrl(sourceUrlLine?.slice(GITHUB_MIRROR_SOURCE_PREFIX.length));
 }
 function githubRepoFromUrl(url) {
-    const match = url?.match(/^https:\/\/github\.com\/([^/]+)\/([^/]+)\/issues\/\d+(?:[/?#].*)?$/iu);
-    return match?.[1] && match[2] ? `${match[1]}/${match[2]}` : undefined;
+    for (const candidate of githubUrlCandidates(url)) {
+        const match = candidate.match(/^https:\/\/github\.com\/([^/]+)\/([^/]+)\/issues\/\d+(?:[/?#].*)?$/iu);
+        if (match?.[1] && match[2]) {
+            return `${match[1]}/${match[2]}`;
+        }
+    }
+    return undefined;
+}
+function githubUrlCandidates(value) {
+    if (!value) {
+        return [];
+    }
+    const candidates = new Set();
+    const trimmed = value.trim();
+    candidates.add(trimmed.replace(/^<|>$/gu, ''));
+    for (const match of trimmed.matchAll(/https:\/\/github\.com\/[^\s<>)\]]+/giu)) {
+        candidates.add(match[0].replace(/[),.;]+$/gu, ''));
+    }
+    return [...candidates];
 }
 function labelRoutesForIssue(issue, config) {
     const labels = uniqueNormalizedLabels(issue.labels).filter((label) => !isShapeLabel(label));