@link-assistant/hive-mind 1.21.4 → 1.22.0

package/CHANGELOG.md

@@ -1,5 +1,19 @@
 # @link-assistant/hive-mind
 
+## 1.22.0
+
+### Minor Changes
+
+- c000f7b: Add `--attach-solution-summary` and `--auto-attach-solution-summary` options
+
+  This feature allows users to automatically attach the AI's result summary as a PR/issue comment:
+  - `--attach-solution-summary`: Always attach the solution summary when available
+  - `--auto-attach-solution-summary`: Only attach the summary if the AI didn't create any comments during the session
+
+  The solution summary is extracted from the JSON output stream of all AI tools (claude, agent, codex, opencode). Each tool captures the last text content from various JSON event types (text, assistant, message, result) to provide a summary of the work done.
+
+  Fixes #1263
+
 ## 1.21.4
 
 ### Patch Changes

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@link-assistant/hive-mind",
-  "version": "1.21.4",
+  "version": "1.22.0",
   "description": "AI-powered issue solver and hive mind for collaborative problem solving",
   "main": "src/hive.mjs",
   "type": "module",

package/src/agent.lib.mjs

@@ -553,6 +553,7 @@ export const executeAgentCommand = async params => {
       let limitReached = false;
       let limitResetTime = null;
       let lastMessage = '';
+      let lastTextContent = ''; // Issue #1263: Track last text content for result summary
       let fullOutput = ''; // Collect all output for error detection (kept for backward compatibility)
       // Issue #1201: Track error events detected during streaming for reliable error detection
       // Post-hoc detection on fullOutput can miss errors if NDJSON lines get concatenated without newlines
@@ -613,6 +614,33 @@ export const executeAgentCommand = async params => {
                 streamingErrorMessage = data.message || data.error || line.substring(0, 100);
                 await log(`⚠️  Error event detected in stream: ${streamingErrorMessage}`, { level: 'warning' });
               }
+              // Issue #1263: Track text content for result summary
+              // Agent outputs text via 'text', 'assistant', or 'message' type events
+              if (data.type === 'text' && data.text) {
+                lastTextContent = data.text;
+              } else if (data.type === 'assistant' && data.message?.content) {
+                // Extract text from assistant message content
+                const content = Array.isArray(data.message.content) ? data.message.content : [data.message.content];
+                for (const item of content) {
+                  if (item.type === 'text' && item.text) {
+                    lastTextContent = item.text;
+                  }
+                }
+              } else if (data.type === 'message' && data.content) {
+                // Direct message content
+                if (typeof data.content === 'string') {
+                  lastTextContent = data.content;
+                } else if (Array.isArray(data.content)) {
+                  for (const item of data.content) {
+                    if (item.type === 'text' && item.text) {
+                      lastTextContent = item.text;
+                    }
+                  }
+                }
+              } else if (data.type === 'result' && data.result) {
+                // Explicit result message (like Claude outputs)
+                lastTextContent = data.result;
+              }
             } catch {
               // Not JSON - log as plain text
               await log(line);
@@ -647,6 +675,29 @@ export const executeAgentCommand = async params => {
                   streamingErrorMessage = stderrData.message || stderrData.error || stderrLine.substring(0, 100);
                   await log(`⚠️  Error event detected in stream: ${streamingErrorMessage}`, { level: 'warning' });
                 }
+                // Issue #1263: Track text content for result summary (stderr)
+                if (stderrData.type === 'text' && stderrData.text) {
+                  lastTextContent = stderrData.text;
+                } else if (stderrData.type === 'assistant' && stderrData.message?.content) {
+                  const content = Array.isArray(stderrData.message.content) ? stderrData.message.content : [stderrData.message.content];
+                  for (const item of content) {
+                    if (item.type === 'text' && item.text) {
+                      lastTextContent = item.text;
+                    }
+                  }
+                } else if (stderrData.type === 'message' && stderrData.content) {
+                  if (typeof stderrData.content === 'string') {
+                    lastTextContent = stderrData.content;
+                  } else if (Array.isArray(stderrData.content)) {
+                    for (const item of stderrData.content) {
+                      if (item.type === 'text' && item.text) {
+                        lastTextContent = item.text;
+                      }
+                    }
+                  }
+                } else if (stderrData.type === 'result' && stderrData.result) {
+                  lastTextContent = stderrData.result;
+                }
               } catch {
                 // Not JSON - log as plain text
                 await log(stderrLine);
@@ -814,6 +865,7 @@ export const executeAgentCommand = async params => {
           tokenUsage,
           pricingInfo,
           publicPricingEstimate: pricingInfo.totalCostUSD,
+          resultSummary: lastTextContent || null, // Issue #1263: Use last text content from JSON output stream
         };
       }
 
@@ -867,6 +919,11 @@ export const executeAgentCommand = async params => {
         }
       }
 
+      // Issue #1263: Log if result summary was captured
+      if (lastTextContent) {
+        await log('📝 Captured result summary from Agent output', { verbose: true });
+      }
+
       return {
         success: true,
         sessionId,
@@ -875,6 +932,7 @@ export const executeAgentCommand = async params => {
         tokenUsage,
         pricingInfo,
         publicPricingEstimate: pricingInfo.totalCostUSD,
+        resultSummary: lastTextContent || null, // Issue #1263: Use last text content from JSON output stream
       };
     } catch (error) {
       reportError(error, {
@@ -893,6 +951,7 @@ export const executeAgentCommand = async params => {
         tokenUsage: null,
         pricingInfo: null,
         publicPricingEstimate: null,
+        resultSummary: null, // Issue #1263: No result summary available on error
       };
     }
   };

package/src/claude.lib.mjs

@@ -861,6 +861,7 @@ export const executeClaudeCommand = async params => {
     let stderrErrors = [];
     let anthropicTotalCostUSD = null; // Capture Anthropic's official total_cost_usd from result
     let errorDuringExecution = false; // Issue #1088: Track if error_during_execution subtype occurred
+    let resultSummary = null; // Issue #1263: Capture AI result summary for --attach-solution-summary
 
     // Create interactive mode handler if enabled
     let interactiveHandler = null;
@@ -901,12 +902,10 @@ export const executeClaudeCommand = async params => {
       await log('---BEGIN USER PROMPT---', { verbose: true });
       await log(prompt, { verbose: true });
       await log('---END USER PROMPT---', { verbose: true });
-      await log('', { verbose: true });
       await log('📋 System prompt:', { verbose: true });
       await log('---BEGIN SYSTEM PROMPT---', { verbose: true });
       await log(systemPrompt, { verbose: true });
       await log('---END SYSTEM PROMPT---', { verbose: true });
-      await log('', { verbose: true });
     }
     try {
       // Resolve thinking settings (handles translation between --think and --thinking-budget based on Claude version)
@@ -1001,6 +1000,12 @@ export const executeClaudeCommand = async params => {
                 } else if (data.total_cost_usd !== undefined && data.total_cost_usd !== null) {
                   await log(`💰 Anthropic cost from ${data.subtype || 'unknown'} result ignored: $${data.total_cost_usd.toFixed(6)}`, { verbose: true });
                 }
+                // Issue #1263: Extract result summary for --attach-solution-summary and --auto-attach-solution-summary
+                // The result field contains the AI's summary of the work done
+                if (data.subtype === 'success' && data.result && typeof data.result === 'string') {
+                  resultSummary = data.result;
+                  await log('📝 Captured result summary from Claude output', { verbose: true });
+                }
                 if (data.is_error === true) {
                   lastMessage = data.result || JSON.stringify(data);
                   const subtype = data.subtype || 'unknown';
@@ -1067,8 +1072,7 @@ export const executeClaudeCommand = async params => {
                 const termsAcceptancePattern = /\[ACTION REQUIRED\].*terms|must run.*claude.*review.*terms/i;
                 if (termsAcceptancePattern.test(line)) {
                   commandFailed = true;
-                  await log('\n❌ Claude Code requires terms acceptance - please run `claude` interactively to accept the updated terms', { level: 'error' });
-                  await log('   This is not an error in your code, but Claude CLI needs human interaction.', { level: 'error' });
+                  await log('\n❌ Claude Code requires terms acceptance - please run `claude` interactively to accept the updated terms\n   This is not an error in your code, but Claude CLI needs human interaction.', { level: 'error' });
                 }
               }
             }
@@ -1125,8 +1129,7 @@ export const executeClaudeCommand = async params => {
         // Specifically detect "command not found" via exit code 127
         if (resultExitCode === 127 && !commandFailed) {
           commandFailed = true;
-          await log(`\n❌ Command not found (exit code 127) - "${claudePath}" is not installed or not in PATH`, { level: 'error' });
-          await log('   Please ensure Claude CLI is installed: npm install -g @anthropic-ai/claude-code', { level: 'error' });
+          await log(`\n❌ Command not found (exit code 127) - "${claudePath}" is not installed or not in PATH\n   Please ensure Claude CLI is installed: npm install -g @anthropic-ai/claude-code`, { level: 'error' });
         }
       }
 
@@ -1151,8 +1154,7 @@ export const executeClaudeCommand = async params => {
           retryCount++;
           return await executeWithRetry();
         } else {
-          await log(`\n\n❌ API overload error persisted after ${maxRetries} retries`, { level: 'error' });
-          await log('   The API appears to be heavily loaded. Please try again later.', { level: 'error' });
+          await log(`\n\n❌ API overload error persisted after ${maxRetries} retries\n   The API appears to be heavily loaded. Please try again later.`, { level: 'error' });
           return {
             success: false,
             sessionId,
@@ -1162,6 +1164,7 @@ export const executeClaudeCommand = async params => {
             messageCount,
             toolUseCount,
             anthropicTotalCostUSD, // Issue #1104: Include cost even on failure
+            resultSummary, // Issue #1263: Include result summary
           };
         }
       }
@@ -1196,11 +1199,7 @@ export const executeClaudeCommand = async params => {
           retryCount++;
           return await executeWithRetry();
         } else {
-          await log(`\n\n❌ 503 network error persisted after ${retryLimits.max503Retries} retries`, {
-            level: 'error',
-          });
-          await log('   The Anthropic API appears to be experiencing network issues.', { level: 'error' });
-          await log('   Please try again later or check https://status.anthropic.com/', { level: 'error' });
+          await log(`\n\n❌ 503 network error persisted after ${retryLimits.max503Retries} retries\n   The Anthropic API appears to be experiencing network issues.\n   Please try again later or check https://status.anthropic.com/`, { level: 'error' });
           return {
             success: false,
             sessionId,
@@ -1211,6 +1210,7 @@ export const executeClaudeCommand = async params => {
             toolUseCount,
             is503Error: true,
             anthropicTotalCostUSD, // Issue #1104: Include cost even on failure
+            resultSummary, // Issue #1263: Include result summary
           };
         }
       }
@@ -1265,11 +1265,11 @@ export const executeClaudeCommand = async params => {
       // See: docs/dependencies-research/claude-code-issues/README.md for full details
       if (!commandFailed && stderrErrors.length > 0 && messageCount === 0 && toolUseCount === 0) {
         commandFailed = true;
-        await log('\n\n❌ Command failed: No messages processed and errors detected in stderr', { level: 'error' });
-        await log('Stderr errors:', { level: 'error' });
-        for (const err of stderrErrors.slice(0, 5)) {
-          await log(`   ${err.substring(0, 200)}`, { level: 'error' });
-        }
+        const errorsPreview = stderrErrors
+          .slice(0, 5)
+          .map(e => `   ${e.substring(0, 200)}`)
+          .join('\n');
+        await log(`\n\n❌ Command failed: No messages processed and errors detected in stderr\nStderr errors:\n${errorsPreview}`, { level: 'error' });
       }
       if (commandFailed) {
         // Take resource snapshot after failure
@@ -1277,8 +1277,6 @@ export const executeClaudeCommand = async params => {
         await log('\n📈 System resources after execution:', { verbose: true });
         await log(`   Memory: ${resourcesAfter.memory.split('\n')[1]}`, { verbose: true });
         await log(`   Load: ${resourcesAfter.load}`, { verbose: true });
-        // Log attachment will be handled by solve.mjs when it receives success=false
-        await log('', { verbose: true });
         await showResumeCommand(sessionId, tempDir, claudePath, argv.model, log);
         return {
           success: false,
@@ -1290,6 +1288,7 @@ export const executeClaudeCommand = async params => {
           toolUseCount,
           errorDuringExecution,
           anthropicTotalCostUSD, // Issue #1104: Include cost even on failure
+          resultSummary, // Issue #1263: Include result summary
         };
       }
       // Issue #1088: If error_during_execution occurred but command didn't fail,
@@ -1361,6 +1360,7 @@ export const executeClaudeCommand = async params => {
         toolUseCount,
         anthropicTotalCostUSD, // Pass Anthropic's official total cost
         errorDuringExecution, // Issue #1088: Track if error_during_execution subtype occurred
+        resultSummary, // Issue #1263: Include result summary for --attach-solution-summary
       };
     } catch (error) {
       reportError(error, {
@@ -1409,6 +1409,7 @@ export const executeClaudeCommand = async params => {
         messageCount,
         toolUseCount,
         anthropicTotalCostUSD, // Issue #1104: Include cost even on failure
+        resultSummary, // Issue #1263: Include result summary
       };
     }
   }; // End of executeWithRetry function

package/src/codex.lib.mjs

@@ -288,6 +288,7 @@ export const executeCodexCommand = async params => {
       let limitReached = false;
       let limitResetTime = null;
       let lastMessage = '';
+      let lastTextContent = ''; // Issue #1263: Track last text content for result summary
       let authError = false;
 
       for await (const chunk of execCommand.stream()) {
@@ -325,6 +326,31 @@ export const executeCodexCommand = async params => {
                 await log('   This error cannot be resolved by retrying.', { level: 'error' });
                 await log('   💡 Please run: codex login', { level: 'error' });
               }
+
+              // Issue #1263: Track text content for result summary
+              // Codex outputs text via 'text', 'assistant', 'message', or 'result' type events
+              if (data.type === 'text' && data.text) {
+                lastTextContent = data.text;
+              } else if (data.type === 'assistant' && data.message?.content) {
+                const content = Array.isArray(data.message.content) ? data.message.content : [data.message.content];
+                for (const item of content) {
+                  if (item.type === 'text' && item.text) {
+                    lastTextContent = item.text;
+                  }
+                }
+              } else if (data.type === 'message' && data.content) {
+                if (typeof data.content === 'string') {
+                  lastTextContent = data.content;
+                } else if (Array.isArray(data.content)) {
+                  for (const item of data.content) {
+                    if (item.type === 'text' && item.text) {
+                      lastTextContent = item.text;
+                    }
+                  }
+                }
+              } else if (data.type === 'result' && data.result) {
+                lastTextContent = data.result;
+              }
             }
           } catch {
             // Not JSON, continue
@@ -386,16 +412,23 @@ export const executeCodexCommand = async params => {
           sessionId,
           limitReached,
           limitResetTime,
+          resultSummary: lastTextContent || null, // Issue #1263: Use last text content from JSON output stream
         };
       }
 
       await log('\n\n✅ Codex command completed');
 
+      // Issue #1263: Log if result summary was captured
+      if (lastTextContent) {
+        await log('📝 Captured result summary from Codex output', { verbose: true });
+      }
+
       return {
         success: true,
         sessionId,
         limitReached,
         limitResetTime,
+        resultSummary: lastTextContent || null, // Issue #1263: Use last text content from JSON output stream
       };
     } catch (error) {
       // Don't report auth errors to Sentry as they are user configuration issues
@@ -420,6 +453,7 @@ export const executeCodexCommand = async params => {
         sessionId: null,
         limitReached: false,
         limitResetTime: null,
+        resultSummary: null, // Issue #1263: No result summary available on error
       };
     }
   };

package/src/opencode.lib.mjs

@@ -307,6 +307,7 @@ export const executeOpenCodeCommand = async params => {
       let limitReached = false;
       let limitResetTime = null;
       let lastMessage = '';
+      let lastTextContent = ''; // Issue #1263: Track last text content for result summary
       let allOutput = ''; // Collect all output for error detection
 
       for await (const chunk of execCommand.stream()) {
@@ -315,6 +316,41 @@ export const executeOpenCodeCommand = async params => {
           await log(output);
           lastMessage = output;
           allOutput += output;
+
+          // Issue #1263: Try to parse JSON output to extract text content for result summary
+          try {
+            const lines = output.split('\n');
+            for (const line of lines) {
+              if (!line.trim()) continue;
+              const data = JSON.parse(line);
+              // Track text content for result summary
+              // OpenCode outputs text via 'text', 'assistant', 'message', or 'result' type events
+              if (data.type === 'text' && data.text) {
+                lastTextContent = data.text;
+              } else if (data.type === 'assistant' && data.message?.content) {
+                const content = Array.isArray(data.message.content) ? data.message.content : [data.message.content];
+                for (const item of content) {
+                  if (item.type === 'text' && item.text) {
+                    lastTextContent = item.text;
+                  }
+                }
+              } else if (data.type === 'message' && data.content) {
+                if (typeof data.content === 'string') {
+                  lastTextContent = data.content;
+                } else if (Array.isArray(data.content)) {
+                  for (const item of data.content) {
+                    if (item.type === 'text' && item.text) {
+                      lastTextContent = item.text;
+                    }
+                  }
+                }
+              } else if (data.type === 'result' && data.result) {
+                lastTextContent = data.result;
+              }
+            }
+          } catch {
+            // Not JSON, continue
+          }
         }
 
         if (chunk.type === 'stderr') {
@@ -322,6 +358,39 @@ export const executeOpenCodeCommand = async params => {
           if (errorOutput) {
             await log(errorOutput, { stream: 'stderr' });
             allOutput += errorOutput;
+
+            // Issue #1263: Also try to parse stderr for text content
+            try {
+              const lines = errorOutput.split('\n');
+              for (const line of lines) {
+                if (!line.trim()) continue;
+                const data = JSON.parse(line);
+                if (data.type === 'text' && data.text) {
+                  lastTextContent = data.text;
+                } else if (data.type === 'assistant' && data.message?.content) {
+                  const content = Array.isArray(data.message.content) ? data.message.content : [data.message.content];
+                  for (const item of content) {
+                    if (item.type === 'text' && item.text) {
+                      lastTextContent = item.text;
+                    }
+                  }
+                } else if (data.type === 'message' && data.content) {
+                  if (typeof data.content === 'string') {
+                    lastTextContent = data.content;
+                  } else if (Array.isArray(data.content)) {
+                    for (const item of data.content) {
+                      if (item.type === 'text' && item.text) {
+                        lastTextContent = item.text;
+                      }
+                    }
+                  }
+                } else if (data.type === 'result' && data.result) {
+                  lastTextContent = data.result;
+                }
+              }
+            } catch {
+              // Not JSON, continue
+            }
           }
         } else if (chunk.type === 'exit') {
           exitCode = chunk.code;
@@ -374,6 +443,7 @@ export const executeOpenCodeCommand = async params => {
           limitReached: false,
           limitResetTime: null,
           permissionPromptDetected: true,
+          resultSummary: lastTextContent || null, // Issue #1263: Use last text content from JSON output stream
         };
       }
 
@@ -409,16 +479,23 @@ export const executeOpenCodeCommand = async params => {
           sessionId,
           limitReached,
           limitResetTime,
+          resultSummary: lastTextContent || null, // Issue #1263: Use last text content from JSON output stream
         };
       }
 
       await log('\n\n✅ OpenCode command completed');
 
+      // Issue #1263: Log if result summary was captured
+      if (lastTextContent) {
+        await log('📝 Captured result summary from OpenCode output', { verbose: true });
+      }
+
       return {
         success: true,
         sessionId,
         limitReached,
         limitResetTime,
+        resultSummary: lastTextContent || null, // Issue #1263: Use last text content from JSON output stream
       };
     } catch (error) {
       // Clean up the opencode.json config file even on error
@@ -441,6 +518,7 @@ export const executeOpenCodeCommand = async params => {
         sessionId: null,
         limitReached: false,
         limitResetTime: null,
+        resultSummary: null, // Issue #1263: No result summary available on error
       };
     }
   };

package/src/solve.config.lib.mjs

@@ -359,6 +359,16 @@ export const SOLVE_OPTION_DEFINITIONS = {
     description: 'Guide Claude to use agent-commander CLI (start-agent) instead of native Task tool for subagent delegation. Allows using any supported agent type (claude, opencode, codex, agent) with unified API. Only works with --tool claude and requires agent-commander to be installed.',
     default: false,
   },
+  'attach-solution-summary': {
+    type: 'boolean',
+    description: 'Attach the AI solution summary (from the result field) as a comment to the PR/issue after completion. The summary is extracted from the AI tool JSON output and posted under a "Solution summary" header.',
+    default: false,
+  },
+  'auto-attach-solution-summary': {
+    type: 'boolean',
+    description: 'Automatically attach solution summary only if the AI did not create any comments during the session. This provides visible feedback when the AI completes silently.',
+    default: false,
+  },
 };
 
 // Function to create yargs configuration - avoids duplication

package/src/solve.mjs

@@ -58,7 +58,7 @@ const { processAutoContinueForIssue } = autoContinue;
 const repository = await import('./solve.repository.lib.mjs');
 const { setupTempDirectory, cleanupTempDirectory } = repository;
 const results = await import('./solve.results.lib.mjs');
-const { cleanupClaudeFile, showSessionSummary, verifyResults, buildClaudeResumeCommand } = results;
+const { cleanupClaudeFile, showSessionSummary, verifyResults, buildClaudeResumeCommand, checkForAiCreatedComments, attachSolutionSummary } = results;
 const claudeLib = await import('./claude.lib.mjs');
 const { executeClaude } = claudeLib;
 
@@ -910,6 +910,7 @@ try {
   let publicPricingEstimate = toolResult.publicPricingEstimate; // Used by agent tool
   let pricingInfo = toolResult.pricingInfo; // Used by agent tool for detailed pricing
   let errorDuringExecution = toolResult.errorDuringExecution || false; // Issue #1088: Track error_during_execution
+  let resultSummary = toolResult.resultSummary || null; // Issue #1263: Capture result summary for --attach-solution-summary
   limitReached = toolResult.limitReached;
   cleanupContext.limitReached = limitReached;
 
@@ -1183,6 +1184,41 @@ try {
   // Show summary of session and log file
   await showSessionSummary(sessionId, limitReached, argv, issueUrl, tempDir, shouldAttachLogs);
 
+  // Issue #1263: Handle solution summary attachment
+  // --attach-solution-summary: Always attach if result summary is available
+  // --auto-attach-solution-summary: Only attach if AI didn't create any comments during session
+  if (success && resultSummary && (argv.attachSolutionSummary || argv.autoAttachSolutionSummary)) {
+    let shouldAttachSummary = false;
+
+    if (argv.attachSolutionSummary) {
+      // Explicit flag - always attach
+      shouldAttachSummary = true;
+      await log('📝 --attach-solution-summary enabled, attaching result summary...');
+    } else if (argv.autoAttachSolutionSummary) {
+      // Auto mode - only attach if AI didn't create comments
+      await log('🔍 Checking if AI created any comments during session (--auto-attach-solution-summary)...');
+      const aiCreatedComments = await checkForAiCreatedComments(referenceTime, owner, repo, prNumber, issueNumber);
+      if (aiCreatedComments) {
+        await log('ℹ️  AI created comments during session, skipping solution summary attachment');
+      } else {
+        shouldAttachSummary = true;
+        await log('📝 No AI comments detected, attaching solution summary...');
+      }
+    }
+
+    if (shouldAttachSummary) {
+      await attachSolutionSummary({
+        resultSummary,
+        prNumber,
+        issueNumber,
+        owner,
+        repo,
+      });
+    }
+  } else if ((argv.attachSolutionSummary || argv.autoAttachSolutionSummary) && !resultSummary) {
+    await log('ℹ️  No solution summary available from AI tool output', { verbose: true });
+  }
+
   // Search for newly created pull requests and comments
   // Pass shouldRestart to prevent early exit when auto-restart is needed
   // Include agent tool pricing data when available (publicPricingEstimate, pricingInfo)

package/src/solve.results.lib.mjs

@@ -876,3 +876,133 @@ export const handleExecutionError = async (error, shouldAttachLogs, owner, repo,
 
   await safeExit(1, 'Execution error');
 };
+
+/**
+ * Check if new comments were created by the AI during the session.
+ * This is used by --auto-attach-solution-summary to determine if the AI
+ * already provided feedback.
+ *
+ * Issue #1263: Support for --attach-solution-summary and --auto-attach-solution-summary
+ *
+ * @param {Date} referenceTime - The timestamp before tool execution
+ * @param {string} owner - Repository owner
+ * @param {string} repo - Repository name
+ * @param {number} prNumber - Pull request number (null if working on issue only)
+ * @param {number} issueNumber - Issue number
+ * @returns {Promise<boolean>} - True if AI created comments during the session
+ */
+export const checkForAiCreatedComments = async (referenceTime, owner, repo, prNumber, issueNumber) => {
+  try {
+    // Get the current user's GitHub username
+    const userResult = await $`gh api user --jq .login`;
+    if (userResult.code !== 0) {
+      return false; // Cannot determine, default to not attaching
+    }
+    const currentUser = userResult.stdout.toString().trim();
+    if (!currentUser) {
+      return false;
+    }
+
+    // Check comments on the PR first (if we have a PR)
+    if (prNumber) {
+      // Check PR conversation comments
+      const prCommentsResult = await $`gh api repos/${owner}/${repo}/issues/${prNumber}/comments --paginate`;
+      if (prCommentsResult.code === 0) {
+        const prComments = JSON.parse(prCommentsResult.stdout.toString().trim() || '[]');
+        const newPrComments = prComments.filter(comment => comment.user.login === currentUser && new Date(comment.created_at) > referenceTime);
+        if (newPrComments.length > 0) {
+          return true;
+        }
+      }
+
+      // Check PR review comments (inline code comments)
+      const reviewCommentsResult = await $`gh api repos/${owner}/${repo}/pulls/${prNumber}/comments --paginate`;
+      if (reviewCommentsResult.code === 0) {
+        const reviewComments = JSON.parse(reviewCommentsResult.stdout.toString().trim() || '[]');
+        const newReviewComments = reviewComments.filter(comment => comment.user.login === currentUser && new Date(comment.created_at) > referenceTime);
+        if (newReviewComments.length > 0) {
+          return true;
+        }
+      }
+    }
+
+    // Check issue comments (if different from PR number or no PR)
+    if (issueNumber && issueNumber !== prNumber) {
+      const issueCommentsResult = await $`gh api repos/${owner}/${repo}/issues/${issueNumber}/comments --paginate`;
+      if (issueCommentsResult.code === 0) {
+        const issueComments = JSON.parse(issueCommentsResult.stdout.toString().trim() || '[]');
+        const newIssueComments = issueComments.filter(comment => comment.user.login === currentUser && new Date(comment.created_at) > referenceTime);
+        if (newIssueComments.length > 0) {
+          return true;
+        }
+      }
+    }
+
+    return false;
+  } catch (error) {
+    // On error, default to not attaching (safer choice)
+    await log(`⚠️  Could not check for AI comments: ${error.message}`, { verbose: true });
+    return false;
+  }
+};
+
+/**
+ * Attach the AI's solution summary as a comment to the PR or issue.
+ * The summary is extracted from the tool's result field and posted
+ * with a "Solution summary" header.
+ *
+ * Issue #1263: Support for --attach-solution-summary and --auto-attach-solution-summary
+ *
+ * @param {Object} options - Options object
+ * @param {string} options.resultSummary - The AI's result summary text
+ * @param {number} options.prNumber - Pull request number (null if posting to issue)
+ * @param {number} options.issueNumber - Issue number
+ * @param {string} options.owner - Repository owner
+ * @param {string} options.repo - Repository name
+ * @returns {Promise<boolean>} - True if comment was posted successfully
+ */
+export const attachSolutionSummary = async ({ resultSummary, prNumber, issueNumber, owner, repo }) => {
+  if (!resultSummary || typeof resultSummary !== 'string') {
+    await log('⚠️  No solution summary available to attach', { verbose: true });
+    return false;
+  }
+
+  const targetNumber = prNumber || issueNumber;
+  const targetType = prNumber ? 'pr' : 'issue';
+  const ghCommand = prNumber ? 'pr' : 'issue';
+
+  if (!targetNumber) {
+    await log('⚠️  No PR or issue number to attach solution summary to', { verbose: true });
+    return false;
+  }
+
+  try {
+    const comment = `## Solution summary
+
+${resultSummary}
+
+---
+*This summary was automatically extracted from the AI working session output.*`;
+
+    const result = await $`gh ${ghCommand} comment ${targetNumber} --repo ${owner}/${repo} --body ${comment}`;
+
+    if (result.code === 0) {
+      await log(`✅ Solution summary attached to ${targetType} #${targetNumber}`);
+      return true;
+    } else {
+      await log(`⚠️  Failed to attach solution summary: ${result.stderr?.toString() || 'Unknown error'}`, {
+        level: 'warning',
+      });
+      return false;
+    }
+  } catch (error) {
+    reportError(error, {
+      context: 'attach_solution_summary',
+      targetType,
+      targetNumber,
+      operation: 'post_solution_summary_comment',
+    });
+    await log(`⚠️  Error attaching solution summary: ${error.message}`, { level: 'warning' });
+    return false;
+  }
+};