@eldrforge/kodrdriv 1.2.132 → 1.2.134

package/dist/commands/release.js

@@ -4,8 +4,8 @@ import 'dotenv/config';
 import { DEFAULT_TO_COMMIT_ALIAS, DEFAULT_MAX_DIFF_BYTES, DEFAULT_EXCLUDED_PATTERNS, DEFAULT_OUTPUT_DIRECTORY } from '../constants.js';
 import { getCurrentBranch, getDefaultFromRef, safeJsonParse } from '@eldrforge/git-tools';
 import { create } from '../content/log.js';
-import { create as create$1, truncateDiffByFiles } from '../content/diff.js';
-import { runAgenticRelease, requireTTY, createReleasePrompt, createCompletionWithRetry, generateReflectionReport, getUserChoice, STANDARD_CHOICES, getLLMFeedbackInEditor, editContentInEditor } from '@eldrforge/ai-service';
+import { create as create$1 } from '../content/diff.js';
+import { runAgenticRelease, requireTTY, generateReflectionReport, getUserChoice, STANDARD_CHOICES, getLLMFeedbackInEditor, editContentInEditor, createCompletionWithRetry, createReleasePrompt } from '@eldrforge/ai-service';
 import { improveContentWithLLM } from '../util/interactive.js';
 import { toAIConfig } from '../util/aiAdapter.js';
 import { createStorageAdapter } from '../util/storageAdapter.js';
@@ -17,6 +17,24 @@ import { validateReleaseSummary } from '../util/validation.js';
 import * as GitHub from '@eldrforge/github-tools';
 import { filterContent } from '../util/stopContext.js';
 
+// Helper function to read context files
+async function readContextFiles(contextFiles, logger) {
+    if (!contextFiles || contextFiles.length === 0) {
+        return '';
+    }
+    const storage = createStorage();
+    const contextParts = [];
+    for (const filePath of contextFiles){
+        try {
+            const content = await storage.readFile(filePath, 'utf8');
+            contextParts.push(`## Context from ${filePath}\n\n${content}\n`);
+            logger.debug(`Read context from file: ${filePath}`);
+        } catch (error) {
+            logger.warn(`Failed to read context file ${filePath}: ${error.message}`);
+        }
+    }
+    return contextParts.join('\n---\n\n');
+}
 // Helper function to edit release notes using editor
 async function editReleaseNotesInteractively(releaseSummary) {
     const templateLines = [
@@ -65,7 +83,7 @@ Please revise the release notes according to the user's feedback while maintaini
         callLLM: async (request, runConfig, outputDirectory)=>{
             var _aiConfig_commands_release, _aiConfig_commands, _aiConfig_commands_release1, _aiConfig_commands1;
             const aiConfig = toAIConfig(runConfig);
-            const aiStorageAdapter = createStorageAdapter();
+            const aiStorageAdapter = createStorageAdapter(outputDirectory);
             const aiLogger = createLoggerAdapter(false);
             const modelToUse = ((_aiConfig_commands = aiConfig.commands) === null || _aiConfig_commands === void 0 ? void 0 : (_aiConfig_commands_release = _aiConfig_commands.release) === null || _aiConfig_commands_release === void 0 ? void 0 : _aiConfig_commands_release.model) || aiConfig.model || 'gpt-4o-mini';
             const openaiReasoning = ((_aiConfig_commands1 = aiConfig.commands) === null || _aiConfig_commands1 === void 0 ? void 0 : (_aiConfig_commands_release1 = _aiConfig_commands1.release) === null || _aiConfig_commands_release1 === void 0 ? void 0 : _aiConfig_commands_release1.reasoning) || aiConfig.reasoning;
@@ -103,10 +121,8 @@ async function generateSelfReflection(agenticResult, outputDirectory, storage, l
             releaseNotes: agenticResult.releaseNotes,
             logger
         });
-        // Save the report
-        const storageAdapter = createStorageAdapter();
-        const filename = `agentic-reflection-release-${timestamp}.md`;
-        await storageAdapter.writeOutput(filename, report);
+        // Save the report to output directory
+        await storage.writeFile(reflectionPath, report, 'utf8');
         logger.info('');
         logger.info('═'.repeat(80));
         logger.info('📊 SELF-REFLECTION REPORT GENERATED');
@@ -188,7 +204,7 @@ async function handleInteractiveReleaseFeedback(releaseSummary, runConfig, promp
     }
 }
 const execute = async (runConfig)=>{
-    var _runConfig_release, _runConfig_release1, _runConfig_release2, _runConfig_release3, _runConfig_release4, _runConfig_release5, _runConfig_release6, _runConfig_release7, _runConfig_release8, _runConfig_release9, _aiConfig_commands_release, _aiConfig_commands, _aiConfig_commands_release1, _aiConfig_commands1, _runConfig_release10;
+    var _runConfig_release, _runConfig_release1, _runConfig_release2, _runConfig_release3, _runConfig_release4, _runConfig_release5, _runConfig_release6, _runConfig_release7, _runConfig_release8, _runConfig_release9, _aiConfig_commands_release, _aiConfig_commands, _runConfig_release10, _aiConfig_commands_release1, _aiConfig_commands1, _runConfig_release11, _runConfig_release12;
     const isDryRun = runConfig.dryRun || false;
     const logger = getDryRunLogger(isDryRun);
     // Get current branch to help determine best tag comparison
@@ -275,154 +291,61 @@ const execute = async (runConfig)=>{
     } else {
         logger.debug('Milestone integration disabled via --no-milestones');
     }
-    // Create adapters for ai-service
-    const aiConfig = toAIConfig(runConfig);
-    const aiStorageAdapter = createStorageAdapter();
-    const aiLogger = createLoggerAdapter(isDryRun);
     // Always ensure output directory exists for request/response files
     const outputDirectory = runConfig.outputDirectory || DEFAULT_OUTPUT_DIRECTORY;
     const storage = createStorage();
     await storage.ensureDirectory(outputDirectory);
-    // Check if agentic mode is enabled
-    if ((_runConfig_release7 = runConfig.release) === null || _runConfig_release7 === void 0 ? void 0 : _runConfig_release7.agentic) {
-        var _runConfig_release11, _runConfig_release12, _aiConfig_commands_release2, _aiConfig_commands2, _runConfig_release13, _aiConfig_commands_release3, _aiConfig_commands3, _runConfig_release14, _runConfig_release15;
-        logger.info('🤖 Using agentic mode for release notes generation');
-        // Run agentic release notes generation
-        const agenticResult = await runAgenticRelease({
-            fromRef,
-            toRef,
-            logContent,
-            diffContent,
-            milestoneIssues: milestoneIssuesContent,
-            releaseFocus: (_runConfig_release11 = runConfig.release) === null || _runConfig_release11 === void 0 ? void 0 : _runConfig_release11.focus,
-            userContext: (_runConfig_release12 = runConfig.release) === null || _runConfig_release12 === void 0 ? void 0 : _runConfig_release12.context,
-            model: ((_aiConfig_commands2 = aiConfig.commands) === null || _aiConfig_commands2 === void 0 ? void 0 : (_aiConfig_commands_release2 = _aiConfig_commands2.release) === null || _aiConfig_commands_release2 === void 0 ? void 0 : _aiConfig_commands_release2.model) || aiConfig.model || 'gpt-4o',
-            maxIterations: ((_runConfig_release13 = runConfig.release) === null || _runConfig_release13 === void 0 ? void 0 : _runConfig_release13.maxAgenticIterations) || 30,
-            debug: runConfig.debug,
-            debugRequestFile: getOutputPath(outputDirectory, getTimestampedRequestFilename('release-agentic')),
-            debugResponseFile: getOutputPath(outputDirectory, getTimestampedResponseFilename('release-agentic')),
-            storage: aiStorageAdapter,
-            logger: aiLogger,
-            openaiReasoning: ((_aiConfig_commands3 = aiConfig.commands) === null || _aiConfig_commands3 === void 0 ? void 0 : (_aiConfig_commands_release3 = _aiConfig_commands3.release) === null || _aiConfig_commands_release3 === void 0 ? void 0 : _aiConfig_commands_release3.reasoning) || aiConfig.reasoning
-        });
-        const iterations = agenticResult.iterations || 0;
-        const toolCalls = agenticResult.toolCallsExecuted || 0;
-        logger.info(`🔍 Agentic analysis complete: ${iterations} iterations, ${toolCalls} tool calls`);
-        // Generate self-reflection output if enabled
-        if ((_runConfig_release14 = runConfig.release) === null || _runConfig_release14 === void 0 ? void 0 : _runConfig_release14.selfReflection) {
-            await generateSelfReflection(agenticResult, outputDirectory, storage, logger);
-        }
-        // Apply stop-context filtering to release notes
-        const titleFilterResult = filterContent(agenticResult.releaseNotes.title, runConfig.stopContext);
-        const bodyFilterResult = filterContent(agenticResult.releaseNotes.body, runConfig.stopContext);
-        let releaseSummary = {
-            title: titleFilterResult.filtered,
-            body: bodyFilterResult.filtered
-        };
-        // Handle interactive mode
-        if (((_runConfig_release15 = runConfig.release) === null || _runConfig_release15 === void 0 ? void 0 : _runConfig_release15.interactive) && !isDryRun) {
-            var _runConfig_release16;
-            requireTTY('Interactive mode requires a terminal. Use --dry-run instead.');
-            const interactivePromptContext = {
-                context: (_runConfig_release16 = runConfig.release) === null || _runConfig_release16 === void 0 ? void 0 : _runConfig_release16.context,
-                directories: runConfig.contextDirectories
-            };
-            const interactiveResult = await handleInteractiveReleaseFeedback(releaseSummary, runConfig, promptConfig, interactivePromptContext, outputDirectory, storage, logContent, diffContent);
-            if (interactiveResult.action === 'skip') {
-                logger.info('RELEASE_ABORTED: Release notes generation aborted by user | Reason: User choice | Status: cancelled');
-            } else {
-                logger.info('RELEASE_FINALIZED: Release notes finalized and accepted | Status: ready | Next: Create release or save');
-            }
-            releaseSummary = interactiveResult.finalSummary;
-        }
-        // Save timestamped copy of release notes to output directory
-        try {
-            const timestampedFilename = getTimestampedReleaseNotesFilename();
-            const outputPath = getOutputPath(outputDirectory, timestampedFilename);
-            // Format the release notes as markdown
-            const releaseNotesContent = `# ${releaseSummary.title}\n\n${releaseSummary.body}`;
-            await storage.writeFile(outputPath, releaseNotesContent, 'utf-8');
-            logger.debug('Saved timestamped release notes: %s', outputPath);
-        } catch (error) {
-            logger.warn('RELEASE_SAVE_FAILED: Failed to save timestamped release notes | Error: %s | Impact: Notes not persisted to file', error.message);
-        }
-        if (isDryRun) {
-            logger.info('RELEASE_SUMMARY_COMPLETE: Generated release summary successfully | Status: completed');
-            logger.info('RELEASE_SUMMARY_TITLE: %s', releaseSummary.title);
-            logger.info('RELEASE_SUMMARY_BODY: %s', releaseSummary.body);
-        }
-        return releaseSummary;
-    }
-    // Non-agentic mode: use traditional prompt-based approach
-    const promptContent = {
+    // Create adapters for ai-service
+    const aiConfig = toAIConfig(runConfig);
+    const aiStorageAdapter = createStorageAdapter(outputDirectory);
+    const aiLogger = createLoggerAdapter(isDryRun);
+    // Read context from files if provided
+    const contextFromFiles = await readContextFiles((_runConfig_release7 = runConfig.release) === null || _runConfig_release7 === void 0 ? void 0 : _runConfig_release7.contextFiles, logger);
+    // Combine file context with existing context
+    const combinedContext = [
+        (_runConfig_release8 = runConfig.release) === null || _runConfig_release8 === void 0 ? void 0 : _runConfig_release8.context,
+        contextFromFiles
+    ].filter(Boolean).join('\n\n---\n\n');
+    // Run agentic release notes generation
+    const agenticResult = await runAgenticRelease({
+        fromRef,
+        toRef,
         logContent,
         diffContent,
-        releaseFocus: (_runConfig_release8 = runConfig.release) === null || _runConfig_release8 === void 0 ? void 0 : _runConfig_release8.focus,
-        milestoneIssues: milestoneIssuesContent
-    };
-    const promptContext = {
-        context: (_runConfig_release9 = runConfig.release) === null || _runConfig_release9 === void 0 ? void 0 : _runConfig_release9.context,
-        directories: runConfig.contextDirectories
-    };
-    const promptResult = await createReleasePrompt(promptConfig, promptContent, promptContext);
-    const modelToUse = ((_aiConfig_commands = aiConfig.commands) === null || _aiConfig_commands === void 0 ? void 0 : (_aiConfig_commands_release = _aiConfig_commands.release) === null || _aiConfig_commands_release === void 0 ? void 0 : _aiConfig_commands_release.model) || aiConfig.model || 'gpt-4o-mini';
-    const request = Formatter.create({
-        logger
-    }).formatPrompt(modelToUse, promptResult.prompt);
-    logger.debug('Release analysis: isLargeRelease=%s, maxTokens=%d', promptResult.isLargeRelease, promptResult.maxTokens);
-    // Create retry callback that reduces diff size on token limit errors
-    const createRetryCallback = (originalDiffContent, originalLogContent)=>async (attempt)=>{
-            var _runConfig_release, _runConfig_release1;
-            logger.info('RELEASE_RETRY: Retrying with reduced diff size | Attempt: %d | Strategy: Truncate diff | Reason: Previous attempt failed', attempt);
-            // Progressively reduce the diff size on retries
-            const reductionFactor = Math.pow(0.5, attempt - 1); // 50% reduction per retry
-            const reducedMaxDiffBytes = Math.max(512, Math.floor(maxDiffBytes * reductionFactor));
-            logger.debug('Reducing maxDiffBytes from %d to %d for retry', maxDiffBytes, reducedMaxDiffBytes);
-            // Re-truncate the diff with smaller limits
-            const reducedDiffContent = originalDiffContent.length > reducedMaxDiffBytes ? truncateDiffByFiles(originalDiffContent, reducedMaxDiffBytes) : originalDiffContent;
-            // Rebuild the prompt with the reduced diff
-            const reducedPromptContent = {
-                logContent: originalLogContent,
-                diffContent: reducedDiffContent,
-                releaseFocus: (_runConfig_release = runConfig.release) === null || _runConfig_release === void 0 ? void 0 : _runConfig_release.focus,
-                milestoneIssues: milestoneIssuesContent
-            };
-            const reducedPromptContext = {
-                context: (_runConfig_release1 = runConfig.release) === null || _runConfig_release1 === void 0 ? void 0 : _runConfig_release1.context,
-                directories: runConfig.contextDirectories
-            };
-            const retryPromptResult = await createReleasePrompt(promptConfig, reducedPromptContent, reducedPromptContext);
-            const retryRequest = Formatter.create({
-                logger
-            }).formatPrompt(modelToUse, retryPromptResult.prompt);
-            return retryRequest.messages;
-        };
-    const summary = await createCompletionWithRetry(request.messages, {
-        model: modelToUse,
-        openaiReasoning: ((_aiConfig_commands1 = aiConfig.commands) === null || _aiConfig_commands1 === void 0 ? void 0 : (_aiConfig_commands_release1 = _aiConfig_commands1.release) === null || _aiConfig_commands_release1 === void 0 ? void 0 : _aiConfig_commands_release1.reasoning) || aiConfig.reasoning,
-        maxTokens: promptResult.maxTokens,
-        responseFormat: {
-            type: 'json_object'
-        },
+        milestoneIssues: milestoneIssuesContent,
+        releaseFocus: (_runConfig_release9 = runConfig.release) === null || _runConfig_release9 === void 0 ? void 0 : _runConfig_release9.focus,
+        userContext: combinedContext || undefined,
+        model: ((_aiConfig_commands = aiConfig.commands) === null || _aiConfig_commands === void 0 ? void 0 : (_aiConfig_commands_release = _aiConfig_commands.release) === null || _aiConfig_commands_release === void 0 ? void 0 : _aiConfig_commands_release.model) || aiConfig.model || 'gpt-4o',
+        maxIterations: ((_runConfig_release10 = runConfig.release) === null || _runConfig_release10 === void 0 ? void 0 : _runConfig_release10.maxAgenticIterations) || 30,
         debug: runConfig.debug,
         debugRequestFile: getOutputPath(outputDirectory, getTimestampedRequestFilename('release')),
         debugResponseFile: getOutputPath(outputDirectory, getTimestampedResponseFilename('release')),
         storage: aiStorageAdapter,
-        logger: aiLogger
-    }, createRetryCallback(diffContent, logContent));
-    // Validate and safely cast the response
-    const rawReleaseSummary = validateReleaseSummary(summary);
+        logger: aiLogger,
+        openaiReasoning: ((_aiConfig_commands1 = aiConfig.commands) === null || _aiConfig_commands1 === void 0 ? void 0 : (_aiConfig_commands_release1 = _aiConfig_commands1.release) === null || _aiConfig_commands_release1 === void 0 ? void 0 : _aiConfig_commands_release1.reasoning) || aiConfig.reasoning
+    });
+    const iterations = agenticResult.iterations || 0;
+    const toolCalls = agenticResult.toolCallsExecuted || 0;
+    logger.info(`🔍 Analysis complete: ${iterations} iterations, ${toolCalls} tool calls`);
+    // Generate self-reflection output if enabled
+    if ((_runConfig_release11 = runConfig.release) === null || _runConfig_release11 === void 0 ? void 0 : _runConfig_release11.selfReflection) {
+        await generateSelfReflection(agenticResult, outputDirectory, storage, logger);
+    }
     // Apply stop-context filtering to release notes
-    const titleFilterResult = filterContent(rawReleaseSummary.title, runConfig.stopContext);
-    const bodyFilterResult = filterContent(rawReleaseSummary.body, runConfig.stopContext);
+    const titleFilterResult = filterContent(agenticResult.releaseNotes.title, runConfig.stopContext);
+    const bodyFilterResult = filterContent(agenticResult.releaseNotes.body, runConfig.stopContext);
     let releaseSummary = {
         title: titleFilterResult.filtered,
         body: bodyFilterResult.filtered
     };
     // Handle interactive mode
-    if (((_runConfig_release10 = runConfig.release) === null || _runConfig_release10 === void 0 ? void 0 : _runConfig_release10.interactive) && !isDryRun) {
+    if (((_runConfig_release12 = runConfig.release) === null || _runConfig_release12 === void 0 ? void 0 : _runConfig_release12.interactive) && !isDryRun) {
         requireTTY('Interactive mode requires a terminal. Use --dry-run instead.');
-        const interactiveResult = await handleInteractiveReleaseFeedback(releaseSummary, runConfig, promptConfig, promptContext, outputDirectory, storage, logContent, diffContent);
+        const interactivePromptContext = {
+            context: combinedContext || undefined,
+            directories: runConfig.contextDirectories
+        };
+        const interactiveResult = await handleInteractiveReleaseFeedback(releaseSummary, runConfig, promptConfig, interactivePromptContext, outputDirectory, storage, logContent, diffContent);
         if (interactiveResult.action === 'skip') {
             logger.info('RELEASE_ABORTED: Release notes generation aborted by user | Reason: User choice | Status: cancelled');
         } else {