@jungjaehoon/mama-os 0.9.2 → 0.9.4

package/dist/multi-agent/multi-agent-discord.js

@@ -45,7 +45,6 @@ const message_splitter_js_1 = require("../gateways/message-splitter.js");
 const delegation_format_validator_js_1 = require("./delegation-format-validator.js");
 const channel_history_js_1 = require("../gateways/channel-history.js");
 const prompt_enhancer_js_1 = require("../agent/prompt-enhancer.js");
-const pr_review_poller_js_1 = require("./pr-review-poller.js");
 const child_process_1 = require("child_process");
 const util_1 = require("util");
 const debugLogger = __importStar(require("@jungjaehoon/mama-core/debug-logger"));
@@ -97,16 +96,8 @@ class MultiAgentDiscordHandler extends multi_agent_base_js_1.MultiAgentHandlerBa
     historyInjected = new Set();
     /** Cleanup interval handle for periodic tasks */
     cleanupInterval = null;
-    /** PR poller summaries per channel for LEAD wake-up */
-    prPollerSummaries = new Map();
-    /** Tracks channels where APPROVE+commit was processed (prevents congratulation loops) */
-    approveProcessedChannels = new Map();
-    /** APPROVE cooldown period -- blocks agent-to-agent routing after commit (30 seconds) */
-    static APPROVE_COOLDOWN_MS = 30 * 1000;
     /** Cleanup interval period (1 minute) */
     static CLEANUP_INTERVAL_MS = 60_000;
-    /** PR review chain override (allows LEAD → DEV → REVIEWER → LEAD) */
-    static PR_REVIEW_MAX_CHAIN = 6;
     constructor(config, processOptions = {}, runtimeOptions = {}) {
         super(config, processOptions, runtimeOptions);
         this.multiBotManager = new multi_bot_manager_js_1.MultiBotManager(config);
@@ -115,7 +106,6 @@ class MultiAgentDiscordHandler extends multi_agent_base_js_1.MultiAgentHandlerBa
         this.cleanupInterval = setInterval(() => {
             this.messageQueue.clearExpired();
             this.cleanupProcessedMentions();
-            this.cleanupApproveChannels();
         }, MultiAgentDiscordHandler.CLEANUP_INTERVAL_MS);
         // Setup idle event listeners for all agents (F7)
         this.setupIdleListeners();
@@ -193,15 +183,6 @@ class MultiAgentDiscordHandler extends multi_agent_base_js_1.MultiAgentHandlerBa
             if (this.processedMentions.has(dedupKey))
                 return;
             this.processedMentions.set(dedupKey, Date.now());
-            // Block agent-to-LEAD mentions during post-APPROVE cooldown
-            // Only blocks mentions targeting the default (LEAD) agent -- not other agents like DEV
-            const approveCooldown = this.approveProcessedChannels.get(message.channel.id);
-            if (message.author.bot &&
-                approveCooldown &&
-                Date.now() - approveCooldown < MultiAgentDiscordHandler.APPROVE_COOLDOWN_MS &&
-                agentId === this.config.default_agent) {
-                return;
-            }
             const cleanContent = message.content.replace(/<@!?\d+>/g, '').trim();
             if (!cleanContent)
                 return;
@@ -213,7 +194,7 @@ class MultiAgentDiscordHandler extends multi_agent_base_js_1.MultiAgentHandlerBa
             // Chain depth check for mention_delegation
             if (isFromAgent && senderAgentId && senderAgentId !== 'main') {
                 const chainState = this.orchestrator.getChainState(message.channel.id);
-                const maxDepth = this.getEffectiveMaxMentionDepth(message.channel.id);
+                const maxDepth = this.getEffectiveMaxMentionDepth();
                 if (chainState.blocked) {
                     logger.info(`[MultiAgent] Mention chain blocked in channel ${message.channel.id}, ignoring`);
                     return;
@@ -300,7 +281,7 @@ class MultiAgentDiscordHandler extends multi_agent_base_js_1.MultiAgentHandlerBa
         if (this.config.mention_delegation) {
             const botUserIdMap = this.multiBotManager.getBotUserIdMap();
             // Include LEAD (default agent) which uses the main bot token
-            // Without this, @Sisyphus in other agents' personas won't resolve to <@userId>
+            // Without this, @Conductor in other agents' personas won't resolve to <@userId>
             const defaultAgentId = this.config.default_agent;
             if (defaultAgentId && !botUserIdMap.has(defaultAgentId)) {
                 const mainBotUserId = this.multiBotManager.getMainBotUserId();
@@ -311,16 +292,6 @@ class MultiAgentDiscordHandler extends multi_agent_base_js_1.MultiAgentHandlerBa
             this.processManager.setBotUserIdMap(botUserIdMap);
             this.processManager.setMentionDelegation(true);
             logger.info(`[MultiAgent] Mention delegation enabled with ${botUserIdMap.size} bot IDs`);
-            if (this.isPrReviewPollingEnabled()) {
-                // PR Review Poller: target LEAD directly.
-                // LEAD analyzes → delegates to DevBot → DevBot fixes → Reviewer reviews after.
-                const orchestratorId = defaultAgentId || 'sisyphus';
-                const orchestratorUserId = botUserIdMap.get(orchestratorId);
-                if (orchestratorUserId) {
-                    this.prReviewPoller.setTargetAgentUserId(orchestratorUserId);
-                    logger.info(`[MultiAgent] PR Poller target: ${orchestratorId} (LEAD)`);
-                }
-            }
         }
     }
     /**
@@ -354,109 +325,6 @@ class MultiAgentDiscordHandler extends multi_agent_base_js_1.MultiAgentHandlerBa
                 }
             }
         }, 'discord');
-        if (this.isPrReviewPollingEnabled()) {
-            // Capture compact batch summaries so LEAD receives actionable context.
-            this.prReviewPoller.setMessageSender(async () => { });
-            this.prReviewPoller.setOnBatchItem(async (channelId, summary, item) => {
-                const bucket = this.prPollerSummaries.get(channelId) ?? [];
-                const compact = this.compactPrReviewBatchSummary(summary);
-                if (!compact || bucket.some((candidate) => candidate.id === item.id)) {
-                    return;
-                }
-                const merged = {
-                    ...item,
-                    summary: compact,
-                };
-                // Keep bounded for prompt size and avoid repeats within one poll batch.
-                this.prPollerSummaries.set(channelId, [...bucket, merged].slice(-12));
-            });
-            // After poll cycle: post count to channel and wake up PR workflow agents.
-            this.prReviewPoller.setOnBatchComplete(async (channelId, digest) => {
-                const items = (digest?.items && digest.items.length > 0
-                    ? digest.items
-                    : this.prPollerSummaries.get(channelId)) ?? [];
-                if (items.length === 0) {
-                    return;
-                }
-                const actionItems = digest?.newItems ?? items.filter((item) => !item.isReminder);
-                const reminderItems = digest?.reminderItems ?? items.filter((item) => item.isReminder);
-                const allActionable = actionItems.length > 0;
-                const count = items.length;
-                if (!allActionable) {
-                    this.prPollerSummaries.delete(channelId);
-                    return;
-                }
-                const compactItems = items.map((item, idx) => `- ${idx + 1}. [${item.severity}] ${item.summary}`);
-                const summaryLines = compactItems.join('\n');
-                const sessions = this.prReviewPoller.getSessionDetails();
-                const session = sessions.find((s) => s.channelId === channelId);
-                if (!session) {
-                    // Clean up stale summary to prevent memory leak
-                    this.prPollerSummaries.delete(channelId);
-                    return;
-                }
-                const prLabel = `${session.owner}/${session.repo}#${session.prNumber}`;
-                const promptSummary = `\n${summaryLines}`;
-                const reminderSummary = reminderItems.length > 0 ? `\n🔔 Reminders only: ${reminderItems.length} item(s).` : '';
-                this.prPollerSummaries.delete(channelId);
-                // Post summary to channel (visible to humans)
-                const channel = await client.channels.fetch(channelId);
-                if (channel && 'send' in channel) {
-                    const content = `📊 PR ${prLabel} — ${count} new review item(s)\n\n${promptSummary}${reminderSummary}`;
-                    const chunks = (0, message_splitter_js_1.splitForDiscord)(content);
-                    for (const chunk of chunks) {
-                        await channel.send({
-                            content: chunk,
-                        });
-                    }
-                }
-                // Wake up LEAD with compact PR summary + workspace path.
-                this.orchestrator.resetChain(channelId);
-                const defaultAgentId = this.config.default_agent || 'sisyphus';
-                const workspace = `\`${session.workspaceDir}\``;
-                const workflowPrompt = `📊 PR Review follow-up (${count} new item(s))`;
-                const taskContext = `Target: ${prLabel}\nWorkspace: ${workspace}\n`;
-                const itemsContext = `Items:\n${summaryLines}`;
-                const basePrompt = `${workflowPrompt}\n${taskContext}${itemsContext}${reminderSummary}`;
-                this.messageQueue.enqueue(defaultAgentId, {
-                    prompt: `${basePrompt}\n\nClassify severity, implement fixable items, and coordinate with Dev/Reviewer.`,
-                    channelId,
-                    source: 'discord',
-                    enqueuedAt: Date.now(),
-                    context: { channelId, userId: 'pr-poller' },
-                });
-                logger.info(`[MultiAgent] PR Poller -> LEAD wake-up (${count} items)`);
-                this.tryDrainNow(defaultAgentId, 'discord', channelId).catch(() => { });
-                // Wake implementation/review agents in parallel so review loop can proceed autonomously.
-                const helperAgents = this.resolvePrReviewAssistants(defaultAgentId);
-                for (const helperAgentId of helperAgents) {
-                    const helperPrompt = helperAgentId === 'developer'
-                        ? `${basePrompt}\n\nImplement available fixes in the workspace and request Reviewer verification when changes are made.`
-                        : `${basePrompt}\n\nInspect the comments for correctness issues and provide reviewer guidance.`;
-                    this.messageQueue.enqueue(helperAgentId, {
-                        prompt: helperPrompt,
-                        channelId,
-                        source: 'discord',
-                        enqueuedAt: Date.now(),
-                        context: { channelId, userId: 'pr-poller' },
-                    });
-                    logger.info(`[MultiAgent] PR Poller -> ${helperAgentId} wake-up (${count} items)`);
-                    this.tryDrainNow(helperAgentId, 'discord', channelId).catch(() => { });
-                }
-            });
-            logger.info('[MultiAgent] PR Review Poller message sender configured for Discord');
-        }
-    }
-    /**
-     * Keep PR poller summaries short and remove duplicates.
-     */
-    compactPrReviewBatchSummary(summary) {
-        const compact = summary.replace(/[\r\n]+/g, ' ').trim();
-        const max = 230;
-        if (!compact) {
-            return '';
-        }
-        return compact.length > max ? `${compact.slice(0, max)}…` : compact;
     }
     /**
      * Set main bot token (to avoid duplicate logins in MultiBotManager)
@@ -472,10 +340,6 @@ class MultiAgentDiscordHandler extends multi_agent_base_js_1.MultiAgentHandlerBa
         this.orchestrator.updateConfig(config);
         this.processManager.updateConfig(config);
         this.multiBotManager.updateConfig(config);
-        if (!this.isPrReviewPollingEnabled()) {
-            this.prReviewPoller.stopAll();
-            this.prPollerSummaries.clear();
-        }
     }
     /**
      * Handle a Discord message with multi-agent logic
@@ -490,14 +354,9 @@ class MultiAgentDiscordHandler extends multi_agent_base_js_1.MultiAgentHandlerBa
         }
         // Build message context
         const context = this.buildMessageContext(message, cleanContent);
-        this.updateChainOverrideForChannel(context.channelId);
         // Record human message to shared context
         if (!context.isBot) {
             this.sharedContext.recordHumanMessage(context.channelId, message.author.username, cleanContent, message.id);
-            if (this.isPrReviewPollingEnabled()) {
-                // Auto-detect PR URLs in user messages and start polling
-                this.detectAndPollPRUrls(cleanContent, context.channelId, message);
-            }
         }
         // Select responding agents
         const selection = this.orchestrator.selectRespondingAgents(context);
@@ -532,10 +391,6 @@ class MultiAgentDiscordHandler extends multi_agent_base_js_1.MultiAgentHandlerBa
         if (responses.length === 0) {
             return null;
         }
-        // Auto-detect PR URLs in agent responses and start polling
-        for (const resp of responses) {
-            this.detectAndPollPRUrls(resp.rawContent, context.channelId, message);
-        }
         return {
             selectedAgents: selection.selectedAgents,
             reason: selection.reason,
@@ -637,7 +492,7 @@ class MultiAgentDiscordHandler extends multi_agent_base_js_1.MultiAgentHandlerBa
         // Inject agent availability status, active work, and channel context (Phase 2 + 3)
         const agentStatus = this.buildAgentStatusSection(agentId);
         const workSection = this.workTracker.buildWorkSection(agentId);
-        const channelInfo = `## Current Channel\nDiscord channel_id: ${context.channelId}`;
+        const channelInfo = `## Current Channel\nPlatform: Discord\nchannel_id: ${context.channelId}\nUse **discord_send** to send messages/files to this channel.`;
         const dynamicContextRaw = [agentStatus, workSection, channelInfo].filter(Boolean).join('\n');
         const dynamicContext = dynamicContextRaw.length > MAX_DYNAMIC_CONTEXT_CHARS
             ? `${dynamicContextRaw.slice(0, MAX_DYNAMIC_CONTEXT_CHARS)}\n...`
@@ -808,21 +663,41 @@ class MultiAgentDiscordHandler extends multi_agent_base_js_1.MultiAgentHandlerBa
                 }
                 let msg = '';
                 const modelTag = event.agentModel ? ` [${event.agentModel}]` : '';
+                const progress = event.totalSteps && event.completedSteps !== undefined
+                    ? ` [${event.completedSteps}/${event.totalSteps}]`
+                    : '';
                 if (event.type === 'step-started') {
-                    msg = `  ${event.agentDisplayName}${modelTag} 시작...`;
+                    msg = `  ${event.agentDisplayName}${modelTag}${progress} 시작...`;
                 }
                 else if (event.type === 'step-completed') {
                     const sec = event.duration_ms ? Math.round(event.duration_ms / 1000) : 0;
-                    msg = `${event.agentDisplayName}${modelTag} (${sec}s) 완료`;
+                    const pct = event.totalSteps && event.completedSteps !== undefined
+                        ? ` (${Math.round((event.completedSteps / event.totalSteps) * 100)}%)`
+                        : '';
+                    msg = `${event.agentDisplayName}${modelTag} (${sec}s)${pct} 완료`;
                 }
                 else if (event.type === 'step-failed') {
-                    msg = `${event.agentDisplayName}${modelTag} ❌ 실패: ${event.error?.substring(0, 100)}`;
+                    msg = `${event.agentDisplayName}${modelTag}${progress} ❌ 실패: ${event.error?.substring(0, 100)}`;
                 }
                 if (msg) {
                     this.sendChannelNotification(context.channelId, msg).catch(() => { });
                 }
             });
             if (workflowResult) {
+                if (workflowResult.failed) {
+                    this.logger.warn(`[MultiAgentDiscord] Workflow failed: ${workflowResult.failed}, sending feedback to conductor`);
+                    const feedback = `[SYSTEM] Your workflow_plan failed to execute.\nReason: ${workflowResult.failed}\nPlease adjust and retry, or respond without a workflow_plan.`;
+                    const retryResult = await agentProcess.sendMessage(feedback);
+                    const cleanedRetry = await this.executeAgentToolCalls(agentId, retryResult.response);
+                    const formattedResponse = this.formatAgentResponse(agent, cleanedRetry);
+                    return {
+                        agentId,
+                        agent,
+                        content: formattedResponse,
+                        rawContent: cleanedRetry,
+                        duration: Date.now() - workflowStart + (result.duration_ms ?? 0),
+                    };
+                }
                 const display = workflowResult.directMessage
                     ? `${workflowResult.directMessage}\n\n${workflowResult.result}`
                     : workflowResult.result;
@@ -836,7 +711,47 @@ class MultiAgentDiscordHandler extends multi_agent_base_js_1.MultiAgentHandlerBa
                     duration: totalDuration,
                 };
             }
-            const cleanedResponse = await this.executeAgentToolCalls(agentId, result.response);
+            // Check for council plan (after workflow, before tool calls)
+            const councilResult = await this.tryExecuteCouncil(result.response, context.channelId, 'discord', (event) => {
+                if (!this.discordClient)
+                    return;
+                let msg = '';
+                if (event.type === 'council-round-started') {
+                    msg = `🗣️ ${event.agentDisplayName} Round ${event.round} 시작...`;
+                }
+                else if (event.type === 'council-round-completed') {
+                    const sec = event.duration_ms ? Math.round(event.duration_ms / 1000) : 0;
+                    msg = `🗣️ ${event.agentDisplayName} Round ${event.round} (${sec}s) 완료`;
+                }
+                else if (event.type === 'council-round-failed') {
+                    msg = `🗣️ ${event.agentDisplayName} Round ${event.round} ❌ 실패: ${event.error?.substring(0, 100)}`;
+                }
+                if (msg) {
+                    this.sendChannelNotification(context.channelId, msg).catch(() => { });
+                }
+            });
+            if (councilResult) {
+                const display = councilResult.directMessage
+                    ? `${councilResult.directMessage}\n\n${councilResult.result}`
+                    : councilResult.result;
+                const formattedResponse = this.formatAgentResponse(agent, display);
+                return {
+                    agentId,
+                    agent,
+                    content: formattedResponse,
+                    rawContent: display,
+                    duration: Date.now() - workflowStart + (result.duration_ms ?? 0),
+                };
+            }
+            // Strip any workflow/council plan JSON that wasn't executed
+            let responseForProcessing = result.response;
+            if (this.workflowEngine?.isEnabled()) {
+                responseForProcessing = this.workflowEngine.extractNonPlanContent(responseForProcessing);
+            }
+            if (this.councilEngine) {
+                responseForProcessing = this.councilEngine.extractNonPlanContent(responseForProcessing);
+            }
+            const cleanedResponse = await this.executeAgentToolCalls(agentId, responseForProcessing);
             // Detect API error responses — skip mention resolution and delegation to prevent error loops
             const isErrorResponse = /API Error:\s*\d{3}\b/.test(cleanedResponse);
             const resolvedResponse = isErrorResponse
@@ -849,8 +764,9 @@ class MultiAgentDiscordHandler extends multi_agent_base_js_1.MultiAgentHandlerBa
                 let submittedCount = 0;
                 const submittedAgents = [];
                 for (const delegation of bgDelegations) {
-                    if (!delegation.background)
+                    if (!delegation.background) {
                         continue;
+                    }
                     const check = this.delegationManager.isDelegationAllowed(delegation.fromAgentId, delegation.toAgentId);
                     if (check.allowed) {
                         this.backgroundTaskManager.submit({
@@ -975,8 +891,9 @@ class MultiAgentDiscordHandler extends multi_agent_base_js_1.MultiAgentHandlerBa
         if (delegations.length > 0 && hasBackground) {
             let submittedCount = 0;
             for (const delegation of delegations) {
-                if (!delegation.background)
+                if (!delegation.background) {
                     continue;
+                }
                 const check = this.delegationManager.isDelegationAllowed(delegation.fromAgentId, delegation.toAgentId);
                 if (check.allowed) {
                     this.backgroundTaskManager.submit({
@@ -1119,7 +1036,7 @@ class MultiAgentDiscordHandler extends multi_agent_base_js_1.MultiAgentHandlerBa
                 console.error(`[MultiAgent] Failed to send response for agent ${response.agentId}:`, err);
             }
         }
-        // Post-send: Auto-review trigger for default agent (Armed Sisyphus) self-implementations
+        // Post-send: Auto-review trigger for default agent (Armed Conductor) self-implementations
         const defaultAgentId = this.config.default_agent;
         if (defaultAgentId) {
             const selfImplemented = responses.find((r) => r.agentId === defaultAgentId && this.detectSelfImplementation(r.rawContent));
@@ -1130,7 +1047,7 @@ class MultiAgentDiscordHandler extends multi_agent_base_js_1.MultiAgentHandlerBa
         return sentMessages;
     }
     /**
-     * Detect if the default agent (Sisyphus) performed direct code edits.
+     * Detect if the default agent (Conductor) performed direct code edits.
      * Checks for Claude CLI tool-use markers that indicate Edit/Write operations.
      */
     detectSelfImplementation(rawContent) {
@@ -1149,7 +1066,7 @@ class MultiAgentDiscordHandler extends multi_agent_base_js_1.MultiAgentHandlerBa
         return editIndicators.some((pattern) => pattern.test(rawContent));
     }
     /**
-     * Check git diff size after Sisyphus self-implementation.
+     * Check git diff size after Conductor self-implementation.
      * If diff exceeds thresholds, auto-trigger Reviewer for quality gate.
      *
      * Thresholds (PAIR mode auto-escalation):
@@ -1189,7 +1106,7 @@ class MultiAgentDiscordHandler extends multi_agent_base_js_1.MultiAgentHandlerBa
                 }
             }
             if (filesChanged > MAX_FILES || totalLines > MAX_LINES) {
-                logger.info(`[AutoReview] Sisyphus self-implementation exceeded thresholds: ${filesChanged} files, ${totalLines} lines -> auto-triggering Reviewer`);
+                logger.info(`[AutoReview] Conductor self-implementation exceeded thresholds: ${filesChanged} files, ${totalLines} lines -> auto-triggering Reviewer`);
                 // Find reviewer agent using shared helper
                 const reviewerEntry = this.findReviewerAgent();
                 const reviewerAgentId = reviewerEntry?.[0];
@@ -1199,7 +1116,7 @@ class MultiAgentDiscordHandler extends multi_agent_base_js_1.MultiAgentHandlerBa
                 }
             }
             else {
-                logger.info(`[AutoReview] Sisyphus self-implementation within thresholds: ${filesChanged} files, ${totalLines} lines -- no auto-review needed`);
+                logger.info(`[AutoReview] Conductor self-implementation within thresholds: ${filesChanged} files, ${totalLines} lines -- no auto-review needed`);
             }
         }
         catch {
@@ -1256,33 +1173,14 @@ class MultiAgentDiscordHandler extends multi_agent_base_js_1.MultiAgentHandlerBa
                 }
             }
             logger.info(`[MultiAgent] Auto-routing mentions from ${response.agentId}: -> ${mentionedAgentIds.join(', ')}`);
-            const defaultAgentId = this.config.default_agent;
-            const hasApproveSignal = /\b(APPROVE[DS]?|LGTM)\b/i.test(response.rawContent) &&
-                !/\b(not?\s+approve|don'?t\s+approve|cannot\s+approve|rejected)\b/i.test(response.rawContent);
-            const isReviewerApproveToLead = this.isReviewerAgent(response.agentId) &&
-                mentionedAgentIds.includes(defaultAgentId || '') &&
-                hasApproveSignal;
-            if (isReviewerApproveToLead) {
-                // Dedup: skip if APPROVE already processed for this channel
-                const lastApprove = this.approveProcessedChannels.get(originalMessage.channel.id);
-                if (lastApprove &&
-                    Date.now() - lastApprove < MultiAgentDiscordHandler.APPROVE_COOLDOWN_MS) {
-                    logger.info(`[MultiAgent] APPROVE already processed for ${originalMessage.channel.id}, skipping`);
-                    continue;
-                }
-            }
             // Route to all mentioned agents in parallel
-            await Promise.all(mentionedAgentIds.map((targetAgentId) => this.handleDelegatedMention(targetAgentId, originalMessage, response, isReviewerApproveToLead ? 'REVIEWER_APPROVED' : null)));
-            // Set APPROVE cooldown AFTER routing to LEAD (so LEAD gets the first one)
-            if (isReviewerApproveToLead) {
-                this.approveProcessedChannels.set(originalMessage.channel.id, Date.now());
-            }
+            await Promise.all(mentionedAgentIds.map((targetAgentId) => this.handleDelegatedMention(targetAgentId, originalMessage, response)));
         }
     }
     /**
      * Handle a delegated mention: process target agent response and recursively route.
      */
-    async handleDelegatedMention(targetAgentId, originalMessage, sourceResponse, reviewerSignal) {
+    async handleDelegatedMention(targetAgentId, originalMessage, sourceResponse) {
         // Dedup: prevent double processing
         const dedupKey = `${targetAgentId}:${sourceResponse.messageId || originalMessage.id}`;
         if (this.processedMentions.has(dedupKey))
@@ -1333,16 +1231,10 @@ class MultiAgentDiscordHandler extends multi_agent_base_js_1.MultiAgentHandlerBa
             channelId,
             timestamp: Date.now(),
         });
-        let delegationContent = sourceResponse.rawContent
+        const delegationContent = sourceResponse.rawContent
             .replace(/<@!?\d+>/g, '')
             .replace(/DELEGATE(?:_BG)?::[\w-]+::/g, '')
             .trim();
-        if (reviewerSignal === 'REVIEWER_APPROVED' && targetAgentId === this.config.default_agent) {
-            delegationContent +=
-                '\n\n🔔 [SYSTEM] Reviewer has APPROVED the changes.\n' +
-                    'Review the changes yourself, then commit and push when you are satisfied.\n' +
-                    'Use your judgement on the commit message and which files to include.';
-        }
         try {
             const response = await this.processAgentResponse(targetAgentId, {
                 channelId,
@@ -1455,7 +1347,7 @@ class MultiAgentDiscordHandler extends multi_agent_base_js_1.MultiAgentHandlerBa
     }
     /**
      * Resolve @Name mentions in LLM response text to <@userId> Discord format.
-     * LLMs generate plain text like "@LEAD", "@Sisyphus", "@DevBot" which won't
+     * LLMs generate plain text like "@LEAD", "@Conductor", "@DevBot" which won't
      * trigger Discord mentions or routeResponseMentions detection.
      */
     resolveResponseMentions(text) {
@@ -1503,122 +1395,9 @@ class MultiAgentDiscordHandler extends multi_agent_base_js_1.MultiAgentHandlerBa
         }
         return resolved;
     }
-    cleanupApproveChannels() {
-        const now = Date.now();
-        for (const [channelId, ts] of this.approveProcessedChannels) {
-            if (now - ts > MultiAgentDiscordHandler.APPROVE_COOLDOWN_MS) {
-                this.approveProcessedChannels.delete(channelId);
-            }
-        }
-    }
-    /**
-     * Detect PR URLs in text and auto-start polling.
-     * Sends a notification to the channel when polling starts.
-     */
-    detectAndPollPRUrls(text, channelId, message) {
-        if (!this.isPrReviewPollingEnabled())
-            return;
-        const prUrls = pr_review_poller_js_1.PRReviewPoller.extractPRUrls(text);
-        if (prUrls.length === 0)
-            return;
-        for (const prUrl of prUrls) {
-            this.prReviewPoller
-                .startPolling(prUrl, channelId)
-                .then((started) => {
-                const parsed = this.prReviewPoller.parsePRUrl(prUrl);
-                const key = parsed ? `${parsed.owner}/${parsed.repo}#${parsed.prNumber}` : prUrl;
-                if (started) {
-                    this.updateChainOverrideForChannel(channelId);
-                    if ('send' in message.channel) {
-                        message.channel
-                            .send({
-                            content: `👀 **PR Review Poller Started** -- ${key}\nDetecting new review comments every 60 seconds.`,
-                        })
-                            .catch((err) => {
-                            console.warn(`[MultiAgent] Failed to send PR polling start message to Discord channel ${channelId}:`, err?.message || err);
-                        });
-                    }
-                    logger.info(`[MultiAgent] PR Poller started for ${key} in Discord channel ${channelId}`);
-                }
-                else {
-                    if ('send' in message.channel) {
-                        message.channel
-                            .send({
-                            content: `ℹ️ PR Review Poller is already active for ${key}.`,
-                        })
-                            .catch((err) => {
-                            console.warn(`[MultiAgent] Failed to send PR polling already-running message to Discord channel ${channelId}:`, err?.message || err);
-                        });
-                    }
-                }
-            })
-                .catch((err) => {
-                console.error(`[MultiAgent] Failed to start PR polling:`, err);
-                if ('send' in message.channel) {
-                    message.channel
-                        .send({
-                        content: `⚠️ Failed to start PR Review Poller for ${prUrl}: ${String(err)}`,
-                    })
-                        .catch((sendErr) => {
-                        console.warn(`[MultiAgent] Failed to send PR polling error message to Discord channel ${channelId}:`, sendErr?.message || sendErr);
-                    });
-                }
-            });
-        }
-    }
-    /**
-     * Resolve helper agents for PR review workflow in addition to default lead.
-     */
-    resolvePrReviewAssistants(defaultAgentId) {
-        const candidateIds = ['developer', 'reviewer', 'dev', 'lead'];
-        const helperAgents = [];
-        for (const id of candidateIds) {
-            const cfg = this.config.agents[id];
-            if (!cfg || id === defaultAgentId) {
-                continue;
-            }
-            if (cfg.enabled === false) {
-                continue;
-            }
-            helperAgents.push(id);
-        }
-        return [...new Set(helperAgents)];
-    }
-    /**
-     * Update chain/mention depth overrides when PR review polling is active.
-     */
-    updateChainOverrideForChannel(channelId) {
-        if (this.isPrReviewActive(channelId)) {
-            this.orchestrator.setChannelChainLimit(channelId, MultiAgentDiscordHandler.PR_REVIEW_MAX_CHAIN);
-        }
-        else {
-            this.orchestrator.clearChannelChainLimit(channelId);
-        }
-    }
-    getEffectiveMaxMentionDepth(channelId) {
-        if (this.isPrReviewActive(channelId)) {
-            return MultiAgentDiscordHandler.PR_REVIEW_MAX_CHAIN;
-        }
+    getEffectiveMaxMentionDepth() {
         return this.config.max_mention_depth ?? 3;
     }
-    isPrReviewActive(channelId) {
-        if (!this.isPrReviewPollingEnabled()) {
-            return false;
-        }
-        return this.prReviewPoller
-            .getSessionDetails()
-            .some((session) => session.channelId === channelId);
-    }
-    isPrReviewPollingEnabled() {
-        return this.config.pr_review_poller?.enabled === true;
-    }
-    /**
-     * Check if an agent ID is the reviewer agent
-     */
-    isReviewerAgent(agentId) {
-        return (agentId.toLowerCase().includes('review') ||
-            this.config.agents[agentId]?.name?.toLowerCase().includes('review') === true);
-    }
     /**
      * Find the reviewer agent entry from config
      */