gameforge-cli 0.2.0 → 0.2.1

package/dist/index.js

@@ -31,7 +31,18 @@ const Orchestrator_1 = require("./core/Orchestrator");
 const debugLogger_1 = require("./utils/debugLogger");
 const issueReviewer_1 = require("./utils/issueReviewer");
 const Modifier_1 = require("./agents/core/Modifier");
+const DocumentUpdater_1 = require("./agents/core/DocumentUpdater");
 dotenv_1.default.config();
+/**
+ * Creates a clickable file link for terminals that support OSC 8 hyperlinks.
+ * Falls back to plain text in unsupported terminals.
+ */
+function fileLink(filePath, displayText) {
+    const absolutePath = path_1.default.resolve(filePath);
+    const fileUrl = `file://${absolutePath}`;
+    const text = displayText || filePath;
+    return `\x1b]8;;${fileUrl}\x1b\\${text}\x1b]8;;\x1b\\`;
+}
 const program = new commander_1.Command();
 program
     .name('gameforge')
@@ -168,11 +179,6 @@ async function runCreationFlow(options) {
         console.log(picocolors_1.default.cyan('\n📊 Concept Review\n'));
         stateMachine.transition(StateMachine_1.GameForgePhase.REVIEW_CONCEPT);
         showConceptBrief(transcript);
-        const proceed1 = await confirmGate('Proceed to architecture phase?');
-        if (!proceed1) {
-            console.log(picocolors_1.default.yellow('Exiting...'));
-            return;
-        }
         await sessionManager.saveCheckpoint(sessionId, stateMachine.getState(), 'after_discovery');
         // Phase 2: Architecture
         console.log(picocolors_1.default.cyan('\n🏗️  Phase 2: Architecture\n'));
@@ -180,8 +186,26 @@ async function runCreationFlow(options) {
         costTracker.startPhase('architecture');
         const spinner = (0, ora_1.default)('Generating Game Bible...').start();
         const architect = new Architect_1.Architect(apiKey, costTracker, modelSelector);
+        // Provide spinner control so the agent can pause during user prompts
+        let spinnerPaused = false;
+        architect.setSpinnerControl({
+            pause: () => {
+                if (!spinnerPaused) {
+                    spinner.stop();
+                    spinnerPaused = true;
+                }
+            },
+            resume: (text) => {
+                if (spinnerPaused) {
+                    spinner.start(text || spinner.text);
+                    spinnerPaused = false;
+                }
+            }
+        });
         let bible = await architect.execute(transcript, (message) => {
-            spinner.text = `Generating Game Bible: ${message}`;
+            if (!spinnerPaused) {
+                spinner.text = `Generating Game Bible: ${message}`;
+            }
         });
         // Merge template features if a template was selected
         if (selectedTemplate && selectedTemplate.partialBible.features) {
@@ -202,11 +226,6 @@ async function runCreationFlow(options) {
         stateMachine.transition(StateMachine_1.GameForgePhase.REVIEW_DATA);
         console.log(picocolors_1.default.green(`✓ Generated ${bible.features.length} features`));
         console.log(picocolors_1.default.green(`✓ Generated ${bible.gameObjects.length} game objects`));
-        const proceed2 = await confirmGate('Proceed to production phase?');
-        if (!proceed2) {
-            console.log(picocolors_1.default.yellow('Exiting...'));
-            return;
-        }
         // Phase 3: Production (Specialists)
         console.log(picocolors_1.default.cyan('\n🎨 Phase 3: Production\n'));
         stateMachine.transition(StateMachine_1.GameForgePhase.PRODUCTION);
@@ -236,8 +255,26 @@ async function runCreationFlow(options) {
         // Run chaos analysis
         let chaosSpinner = (0, ora_1.default)('Running Chaos Agent...').start();
         const chaosAgent = new Chaos_1.Chaos(apiKey, costTracker, modelSelector);
+        // Provide spinner control so the agent can pause during user prompts
+        let chaosSpinnerPaused = false;
+        chaosAgent.setSpinnerControl({
+            pause: () => {
+                if (!chaosSpinnerPaused) {
+                    chaosSpinner.stop();
+                    chaosSpinnerPaused = true;
+                }
+            },
+            resume: (text) => {
+                if (chaosSpinnerPaused) {
+                    chaosSpinner.start(text || chaosSpinner.text);
+                    chaosSpinnerPaused = false;
+                }
+            }
+        });
         let currentChaosIssues = await chaosAgent.execute(bible, (message) => {
-            chaosSpinner.text = `Chaos Agent: ${message}`;
+            if (!chaosSpinnerPaused) {
+                chaosSpinner.text = `Chaos Agent: ${message}`;
+            }
         });
         chaosSpinner.succeed(`Found ${currentChaosIssues.length} design issues`);
         // End validation phase and show cost
@@ -248,6 +285,8 @@ async function runCreationFlow(options) {
         // Gate 3: Review Issues One by One
         stateMachine.transition(StateMachine_1.GameForgePhase.REVIEW_ISSUES);
         let fixesApplied = false;
+        let gapAnalysisConsistencyIssues = currentConsistencyIssues;
+        let gapAnalysisChaosIssues = currentChaosIssues;
         const totalIssues = currentConsistencyIssues.length + currentChaosIssues.length;
         if (totalIssues > 0) {
             console.log(picocolors_1.default.yellow(`\n⚠️  ${totalIssues} issues found. Let's review them one by one.\n`));
@@ -289,6 +328,10 @@ async function runCreationFlow(options) {
                         // Update bible
                         bible = result.updatedBible;
                         stateMachine.updateBible(bible);
+                        // Filter issues for gap analysis: only show unfixable issues
+                        const filtered = (0, issueReviewer_1.filterIssuesForGapAnalysis)(currentConsistencyIssues, currentChaosIssues, reviewed, result.unfixableIssues);
+                        gapAnalysisConsistencyIssues = filtered.consistencyIssues;
+                        gapAnalysisChaosIssues = filtered.chaosIssues;
                     }
                     catch (error) {
                         remediationSpinner.fail('Remediation failed');
@@ -298,23 +341,36 @@ async function runCreationFlow(options) {
                             console.log(picocolors_1.default.yellow('Exiting...'));
                             return;
                         }
+                        // Remediation failed - show all issues that were marked for fixing as unfixable
+                        const filtered = (0, issueReviewer_1.filterIssuesForGapAnalysis)(currentConsistencyIssues, currentChaosIssues, reviewed, [] // Treat all toFix issues as unfixable since remediation failed
+                        );
+                        // Show the issues that were marked to fix since they remain unfixed
+                        gapAnalysisConsistencyIssues = reviewed.toFix
+                            .filter(r => r.issueType === 'consistency')
+                            .map(r => r.issue);
+                        gapAnalysisChaosIssues = reviewed.toFix
+                            .filter(r => r.issueType === 'chaos')
+                            .map(r => r.issue);
                     }
                 }
+                else {
+                    // User declined to run remediation - show issues marked to fix as they remain unfixed
+                    gapAnalysisConsistencyIssues = reviewed.toFix
+                        .filter(r => r.issueType === 'consistency')
+                        .map(r => r.issue);
+                    gapAnalysisChaosIssues = reviewed.toFix
+                        .filter(r => r.issueType === 'chaos')
+                        .map(r => r.issue);
+                }
             }
-            // Final confirmation to save
-            const saveConfirm = await confirmGate('Save and export GDD?');
-            if (!saveConfirm) {
-                console.log(picocolors_1.default.yellow('Exiting...'));
-                return;
+            else {
+                // All issues were ignored, don't show any in gap analysis
+                gapAnalysisConsistencyIssues = [];
+                gapAnalysisChaosIssues = [];
             }
         }
         else {
             console.log(picocolors_1.default.green('\n✓ No issues found!'));
-            const proceed = await confirmGate('Save and export GDD?');
-            if (!proceed) {
-                console.log(picocolors_1.default.yellow('Exiting...'));
-                return;
-            }
         }
         // If fixes were applied, re-run specialists to regenerate markdown
         if (fixesApplied) {
@@ -324,16 +380,12 @@ async function runCreationFlow(options) {
         // Save all outputs
         console.log(picocolors_1.default.cyan('\n💾 Saving outputs...\n'));
         const projectDir = await fileManager.initialize(projectId);
+        // Save game bible JSON
         await fileManager.saveGameBible(projectDir, bible);
-        await fileManager.saveCoverPage(projectDir, bible);
-        for (const [name, content] of markdownSections) {
-            const filename = name.replace(/\s+/g, '_') + '.md';
-            await fileManager.saveMarkdown(projectDir, filename, content);
-        }
-        await fileManager.saveGapAnalysis(projectDir, currentChaosIssues, currentConsistencyIssues);
-        // Save combined GDD
-        await fileManager.saveCombinedGDD(projectDir, bible, markdownSections, currentChaosIssues, currentConsistencyIssues);
-        console.log(picocolors_1.default.green('✓ Saved combined Game Design Document'));
+        console.log(picocolors_1.default.green('✓ Saved game_bible.json'));
+        // Save combined GDD (markdown)
+        await fileManager.saveCombinedGDD(projectDir, bible, markdownSections, gapAnalysisChaosIssues, gapAnalysisConsistencyIssues);
+        console.log(picocolors_1.default.green('✓ Saved Game_Design_Document.md'));
         // Complete
         stateMachine.transition(StateMachine_1.GameForgePhase.COMPLETE);
         // Save final checkpoint
@@ -343,15 +395,31 @@ async function runCreationFlow(options) {
         console.log(picocolors_1.default.green('\n✅ Complete!\n'));
         console.log(picocolors_1.default.dim(`Session ID: ${sessionId}`));
         console.log(picocolors_1.default.dim(`Output directory: ${projectDir}`));
+        console.log(picocolors_1.default.cyan('\n📄 Generated Files:'));
+        console.log(picocolors_1.default.green(`  → ${fileLink(path_1.default.join(projectDir, 'game_bible.json'))}`));
+        console.log(picocolors_1.default.green(`  → ${fileLink(path_1.default.join(projectDir, 'Game_Design_Document.md'))}`));
         // Show per-phase cost breakdown
         costTracker.printFinalBreakdown();
         console.log(picocolors_1.default.dim(`Total cost: $${report.total.toFixed(4)} (${report.operations} operations)`));
         console.log(picocolors_1.default.dim(`Budget remaining: $${report.remaining.toFixed(2)}\n`));
+        // Ask if user wants to make changes
+        const wantChanges = await confirmGate('Would you like to make any changes to the design?');
+        if (wantChanges) {
+            try {
+                await runModifyFlowWithBible(sessionId, bible, costTracker, modelSelector, fileManager, sessionManager, stateMachine, options);
+            }
+            catch (modifyError) {
+                console.error(picocolors_1.default.red('Error during modification:'), modifyError);
+                if (debugLogger_1.debugLogger.isEnabled()) {
+                    debugLogger_1.debugLogger.logError('ModifyFlowFromCreate', modifyError);
+                }
+            }
+        }
         if (debugLogger_1.debugLogger.isEnabled()) {
             console.log(picocolors_1.default.yellow(`🐛 Debug log saved to: ${debugLogger_1.debugLogger.getLogFilePath()}\n`));
             debugLogger_1.debugLogger.disable();
         }
-        console.log(picocolors_1.default.dim('View this session anytime with:'));
+        console.log(picocolors_1.default.dim('\nView this session anytime with:'));
         console.log(picocolors_1.default.dim(`  gameforge session ${sessionId}\n`));
     }
     catch (error) {
@@ -625,7 +693,27 @@ async function resumeFlow(idOrCheckpoint, options = {}) {
                 }
                 const architect = new Architect_1.Architect(apiKey, costTracker, modelSelector);
                 const spinner = (0, ora_1.default)('Generating Game Bible...').start();
-                bible = await architect.execute(savedState.transcript);
+                // Provide spinner control so the agent can pause during user prompts
+                let spinnerPaused = false;
+                architect.setSpinnerControl({
+                    pause: () => {
+                        if (!spinnerPaused) {
+                            spinner.stop();
+                            spinnerPaused = true;
+                        }
+                    },
+                    resume: (text) => {
+                        if (spinnerPaused) {
+                            spinner.start(text || spinner.text);
+                            spinnerPaused = false;
+                        }
+                    }
+                });
+                bible = await architect.execute(savedState.transcript, (message) => {
+                    if (!spinnerPaused) {
+                        spinner.text = `Generating Game Bible: ${message}`;
+                    }
+                });
                 spinner.succeed('Game Bible generated');
             }
             stateMachine.updateBible(bible);
@@ -703,7 +791,27 @@ async function continueToProduction(stateMachine, bible, apiKey, costTracker, mo
     console.log(picocolors_1.default.yellow(`Found ${currentConsistencyIssues.length} consistency issues`));
     let chaosSpinner = (0, ora_1.default)('Running Chaos Agent...').start();
     const chaosAgent = new Chaos_1.Chaos(apiKey, costTracker, modelSelector);
-    let currentChaosIssues = await chaosAgent.execute(bible);
+    // Provide spinner control so the agent can pause during user prompts
+    let chaosSpinnerPaused = false;
+    chaosAgent.setSpinnerControl({
+        pause: () => {
+            if (!chaosSpinnerPaused) {
+                chaosSpinner.stop();
+                chaosSpinnerPaused = true;
+            }
+        },
+        resume: (text) => {
+            if (chaosSpinnerPaused) {
+                chaosSpinner.start(text || chaosSpinner.text);
+                chaosSpinnerPaused = false;
+            }
+        }
+    });
+    let currentChaosIssues = await chaosAgent.execute(bible, (message) => {
+        if (!chaosSpinnerPaused) {
+            chaosSpinner.text = `Chaos Agent: ${message}`;
+        }
+    });
     chaosSpinner.succeed(`Found ${currentChaosIssues.length} design issues`);
     // End validation phase and show cost
     const validationCost = costTracker.endPhase();
@@ -789,15 +897,12 @@ async function continueToProduction(stateMachine, bible, apiKey, costTracker, mo
     // Save all outputs
     console.log(picocolors_1.default.cyan('\n💾 Saving outputs...\n'));
     const projectDir = await fileManager.initialize(projectId);
+    // Save game bible JSON
     await fileManager.saveGameBible(projectDir, bible);
-    await fileManager.saveCoverPage(projectDir, bible);
-    for (const [name, content] of markdownSections) {
-        const filename = name.replace(/\s+/g, '_') + '.md';
-        await fileManager.saveMarkdown(projectDir, filename, content);
-    }
-    await fileManager.saveGapAnalysis(projectDir, currentChaosIssues, currentConsistencyIssues);
-    // Save combined GDD
+    console.log(picocolors_1.default.green('✓ Saved game_bible.json'));
+    // Save combined GDD (markdown)
     await fileManager.saveCombinedGDD(projectDir, bible, markdownSections, currentChaosIssues, currentConsistencyIssues);
+    console.log(picocolors_1.default.green('✓ Saved Game_Design_Document.md'));
     stateMachine.transition(StateMachine_1.GameForgePhase.COMPLETE);
     // Save final checkpoint
     await sessionManager.saveCheckpoint(sessionId, stateMachine.getState(), 'complete');
@@ -844,7 +949,7 @@ async function runModifyFlow(sessionId, options) {
             console.log(picocolors_1.default.dim('\nList all sessions with: gameforge sessions\n'));
             return;
         }
-        let bible = latest.state.bible;
+        const bible = latest.state.bible;
         console.log(picocolors_1.default.green(`✓ Loaded GDD: ${bible.meta?.title || 'Untitled'}`));
         console.log(picocolors_1.default.dim(`  Genre: ${bible.meta?.genre || 'Unknown'}`));
         console.log(picocolors_1.default.dim(`  Features: ${bible.features?.length || 0}`));
@@ -853,54 +958,103 @@ async function runModifyFlow(sessionId, options) {
         const budgetLimit = latest.state.budgetLimit || 5.0;
         const costTracker = new costTracker_1.CostTracker(budgetLimit);
         const modelSelector = new modelSelector_1.ModelSelector();
-        // Prompt for modification description
-        const response = await enquirer_1.default.prompt({
-            type: 'input',
-            name: 'modification',
-            message: 'Describe the changes you want to make:'
-        });
-        const modification = response.modification.trim();
-        if (!modification) {
-            console.log(picocolors_1.default.yellow('\nNo modification provided. Exiting.'));
-            return;
+        // Create state machine for checkpoints
+        const stateMachine = new StateMachine_1.StateMachine(budgetLimit);
+        stateMachine.restoreState(latest.state);
+        // Run the modification loop
+        await runModifyFlowWithBible(sessionId, bible, costTracker, modelSelector, fileManager, sessionManager, stateMachine, options);
+    }
+    catch (error) {
+        console.error(picocolors_1.default.red('Error during modification:'), error);
+        if (debugLogger_1.debugLogger.isEnabled()) {
+            debugLogger_1.debugLogger.log('Modification flow failed: ' + String(error));
+            debugLogger_1.debugLogger.disable();
         }
-        // Apply modification via Modifier agent
-        console.log(picocolors_1.default.cyan('\n🔧 Applying modifications...\n'));
-        costTracker.startPhase('remediation'); // Use 'remediation' as closest phase type
-        const modifier = new Modifier_1.Modifier(apiKey, costTracker, modelSelector);
-        const modifySpinner = (0, ora_1.default)('Processing modification request...').start();
-        const result = await modifier.execute(bible, modification, (message) => {
-            modifySpinner.text = `Modifier: ${message}`;
+    }
+}
+async function runModifyFlowWithBible(sessionId, bible, costTracker, modelSelector, fileManager, sessionManager, stateMachine, options) {
+    const apiKey = process.env.ANTHROPIC_API_KEY;
+    // Prompt for modification description
+    const response = await enquirer_1.default.prompt({
+        type: 'input',
+        name: 'modification',
+        message: 'Describe the changes you want to make:'
+    });
+    const modification = response.modification.trim();
+    if (!modification) {
+        console.log(picocolors_1.default.yellow('\nNo modification provided.'));
+        showFinalSummary(costTracker, sessionId, options);
+        return;
+    }
+    // Apply modification via Modifier agent
+    console.log(picocolors_1.default.cyan('\n🔧 Applying modifications...\n'));
+    costTracker.startPhase('remediation');
+    const modifier = new Modifier_1.Modifier(apiKey, costTracker, modelSelector);
+    const modifySpinner = (0, ora_1.default)('Processing modification request...').start();
+    const result = await modifier.execute(bible, modification, (message) => {
+        modifySpinner.text = `Modifier: ${message}`;
+    });
+    modifySpinner.succeed('Modifications applied');
+    // End modification phase and show cost
+    const modificationCost = costTracker.endPhase();
+    if (modificationCost) {
+        costTracker.printPhaseSummary(modificationCost);
+    }
+    // Show summary of changes
+    if (result.changesSummary.length > 0) {
+        console.log(picocolors_1.default.green('\nChanges made:'));
+        result.changesSummary.forEach((change) => {
+            console.log(picocolors_1.default.dim(`  ✓ ${change}`));
         });
-        modifySpinner.succeed('Modifications applied');
-        // End modification phase and show cost
-        const modificationCost = costTracker.endPhase();
-        if (modificationCost) {
-            costTracker.printPhaseSummary(modificationCost);
-        }
-        // Show summary of changes
-        if (result.changesSummary.length > 0) {
-            console.log(picocolors_1.default.green('\nChanges made:'));
-            result.changesSummary.forEach((change) => {
-                console.log(picocolors_1.default.dim(`  ✓ ${change}`));
-            });
-        }
-        if (result.errors.length > 0) {
-            console.log(picocolors_1.default.yellow('\nWarnings/Errors:'));
-            result.errors.forEach((err) => {
-                console.log(picocolors_1.default.dim(`  ⚠ ${err}`));
-            });
+    }
+    if (result.errors.length > 0) {
+        console.log(picocolors_1.default.yellow('\nWarnings/Errors:'));
+        result.errors.forEach((err) => {
+            console.log(picocolors_1.default.dim(`  ⚠ ${err}`));
+        });
+    }
+    const updatedBible = result.updatedBible;
+    // Try to use targeted document updates instead of full regeneration
+    console.log(picocolors_1.default.cyan('\n🎨 Updating documentation...\n'));
+    costTracker.startPhase('production');
+    const projectDir = await fileManager.initialize(sessionId);
+    const existingGDD = await fileManager.readGDD(projectDir);
+    let markdownSections = null;
+    let updatedGDD = null;
+    if (existingGDD && result.changesSummary.length > 0) {
+        // Use targeted document updates - preserve existing content
+        console.log(picocolors_1.default.dim('  Using targeted updates to preserve existing content...\n'));
+        const documentUpdater = new DocumentUpdater_1.DocumentUpdater(apiKey, costTracker, modelSelector);
+        const updateSpinner = (0, ora_1.default)('Applying targeted document updates...').start();
+        const updateResult = await documentUpdater.execute(existingGDD, result.changesSummary, updatedBible, (message) => {
+            updateSpinner.text = `Document Updater: ${message}`;
+        });
+        // Require both: no errors AND document was actually modified
+        const updateSuccessful = updateResult.errors.length === 0 && updateResult.updatedDocument !== existingGDD;
+        if (updateSuccessful) {
+            updateSpinner.succeed(`Updated ${updateResult.sectionsModified.length} sections`);
+            updatedGDD = updateResult.updatedDocument;
+            if (updateResult.sectionsModified.length > 0) {
+                console.log(picocolors_1.default.dim('  Modified sections:'));
+                updateResult.sectionsModified.forEach(section => {
+                    console.log(picocolors_1.default.dim(`    • ${section}`));
+                });
+            }
         }
-        bible = result.updatedBible;
-        // Confirm and regenerate documentation
-        const proceed = await confirmGate('\nRegenerate documentation with these changes?');
-        if (!proceed) {
-            console.log(picocolors_1.default.yellow('Changes discarded.'));
-            return;
+        else {
+            // Log specific reason for fallback
+            if (updateResult.errors.length > 0) {
+                updateSpinner.warn(`Targeted update failed: ${updateResult.errors[0]}`);
+            }
+            else {
+                updateSpinner.warn('Targeted update produced no changes');
+            }
+            console.log(picocolors_1.default.yellow('  Falling back to full document regeneration...\n'));
         }
-        // Run specialists to regenerate markdown
-        console.log(picocolors_1.default.cyan('\n🎨 Regenerating documentation...\n'));
-        costTracker.startPhase('production');
+    }
+    // Fall back to full regeneration if targeted update failed or no existing GDD
+    if (!updatedGDD) {
+        console.log(picocolors_1.default.dim('  Regenerating full documentation...\n'));
         const orchestrator = new Orchestrator_1.SpecialistOrchestrator();
         const specialists = [
             { name: 'Creative Direction', agent: new CreativeSpecialist_1.CreativeSpecialist(apiKey, costTracker, modelSelector) },
@@ -908,63 +1062,87 @@ async function runModifyFlow(sessionId, options) {
             { name: 'Entity Specifications', agent: new EntitySpecialist_1.EntitySpecialist(apiKey, costTracker, modelSelector) },
             { name: 'Technical Specifications', agent: new TechSpecialist_1.TechSpecialist(apiKey, costTracker, modelSelector) }
         ];
-        const markdownSections = await orchestrator.runAllSpecialists(bible, specialists, {
+        markdownSections = await orchestrator.runAllSpecialists(updatedBible, specialists, {
             skipDisambiguation: true
         });
-        // End specialists phase
-        const specialistsCost = costTracker.endPhase();
-        if (specialistsCost) {
-            costTracker.printPhaseSummary(specialistsCost);
-        }
-        // Run validation for gap analysis
-        console.log(picocolors_1.default.cyan('\n🔍 Running validation...\n'));
-        costTracker.startPhase('validation');
-        const consistencyAgent = new Consistency_1.ConsistencyAgent();
-        const consistencyIssues = consistencyAgent.validate(bible);
-        const chaosSpinner = (0, ora_1.default)('Running Chaos Agent...').start();
-        const chaosAgent = new Chaos_1.Chaos(apiKey, costTracker, modelSelector);
-        const chaosIssues = await chaosAgent.execute(bible, (message) => {
-            chaosSpinner.text = `Chaos Agent: ${message}`;
-        });
-        chaosSpinner.succeed(`Found ${chaosIssues.length} design issues`);
-        const validationCost = costTracker.endPhase();
-        if (validationCost) {
-            costTracker.printPhaseSummary(validationCost);
-        }
-        // Save outputs
-        console.log(picocolors_1.default.cyan('\n💾 Saving outputs...\n'));
-        const projectDir = await fileManager.initialize(sessionId);
-        await fileManager.saveGameBible(projectDir, bible);
-        await fileManager.saveCoverPage(projectDir, bible);
-        for (const [sectionName, content] of markdownSections) {
-            await fileManager.saveMarkdown(projectDir, sectionName, content);
+    }
+    // End production phase
+    const productionCost = costTracker.endPhase();
+    if (productionCost) {
+        costTracker.printPhaseSummary(productionCost);
+    }
+    // Run validation for gap analysis
+    console.log(picocolors_1.default.cyan('\n🔍 Running validation...\n'));
+    costTracker.startPhase('validation');
+    const consistencyAgent = new Consistency_1.ConsistencyAgent();
+    const consistencyIssues = consistencyAgent.validate(updatedBible);
+    const chaosSpinner = (0, ora_1.default)('Running Chaos Agent...').start();
+    const chaosAgent = new Chaos_1.Chaos(apiKey, costTracker, modelSelector);
+    const chaosIssues = await chaosAgent.execute(updatedBible, (message) => {
+        chaosSpinner.text = `Chaos Agent: ${message}`;
+    });
+    chaosSpinner.succeed(`Found ${chaosIssues.length} design issues`);
+    const validationCost = costTracker.endPhase();
+    if (validationCost) {
+        costTracker.printPhaseSummary(validationCost);
+    }
+    // Save outputs
+    console.log(picocolors_1.default.cyan('\n💾 Saving outputs...\n'));
+    // Save game bible JSON
+    await fileManager.saveGameBible(projectDir, updatedBible);
+    console.log(picocolors_1.default.green('✓ Saved game_bible.json'));
+    // Save GDD - either the updated version or regenerated
+    if (updatedGDD) {
+        // Update gap analysis section in the targeted-update GDD before saving
+        const gddWithGapAnalysis = fileManager.updateGapAnalysisInDocument(updatedGDD, chaosIssues, consistencyIssues, sessionId);
+        await fileManager.saveUpdatedGDD(projectDir, gddWithGapAnalysis);
+        console.log(picocolors_1.default.green('✓ Saved Game_Design_Document.md (targeted update)'));
+    }
+    else if (markdownSections) {
+        // Save the fully regenerated GDD
+        await fileManager.saveCombinedGDD(projectDir, updatedBible, markdownSections, chaosIssues, consistencyIssues);
+        console.log(picocolors_1.default.green('✓ Saved Game_Design_Document.md (full regeneration)'));
+    }
+    // Save checkpoint
+    stateMachine.updateBible(updatedBible);
+    await sessionManager.saveCheckpoint(sessionId, stateMachine.getState(), 'after_modification');
+    // Show iteration summary (without final breakdown)
+    const report = costTracker.getReport();
+    console.log(picocolors_1.default.green('\n✅ Modification complete!\n'));
+    console.log(picocolors_1.default.dim(`Session ID: ${sessionId}`));
+    console.log(picocolors_1.default.dim(`Output directory: ${projectDir}`));
+    console.log(picocolors_1.default.cyan('\n📄 Generated Files:'));
+    console.log(picocolors_1.default.green(`  → ${fileLink(path_1.default.join(projectDir, 'game_bible.json'))}`));
+    console.log(picocolors_1.default.green(`  → ${fileLink(path_1.default.join(projectDir, 'Game_Design_Document.md'))}`));
+    console.log(picocolors_1.default.dim(`\nIteration cost: $${report.total.toFixed(4)} (${report.operations} operations)`));
+    console.log(picocolors_1.default.dim(`Budget remaining: $${report.remaining.toFixed(2)}\n`));
+    // Ask if user wants to make more changes
+    const moreChanges = await confirmGate('Would you like to make more changes?');
+    if (moreChanges) {
+        // Loop back for more modifications with try-catch
+        try {
+            await runModifyFlowWithBible(sessionId, updatedBible, costTracker, modelSelector, fileManager, sessionManager, stateMachine, options);
         }
-        await fileManager.saveGapAnalysis(projectDir, chaosIssues, consistencyIssues);
-        await fileManager.saveCombinedGDD(projectDir, bible, markdownSections, chaosIssues, consistencyIssues);
-        // Save checkpoint
-        const stateMachine = new StateMachine_1.StateMachine(budgetLimit);
-        stateMachine.restoreState(latest.state);
-        stateMachine.updateBible(bible);
-        await sessionManager.saveCheckpoint(sessionId, stateMachine.getState(), 'after_modification');
-        // Final output
-        const report = costTracker.getReport();
-        console.log(picocolors_1.default.green('\n✅ Modification complete!\n'));
-        console.log(picocolors_1.default.dim(`Session ID: ${sessionId}`));
-        console.log(picocolors_1.default.dim(`Output directory: ${projectDir}`));
-        costTracker.printFinalBreakdown();
-        console.log(picocolors_1.default.dim(`Total cost: $${report.total.toFixed(4)} (${report.operations} operations)`));
-        console.log(picocolors_1.default.dim(`Budget remaining: $${report.remaining.toFixed(2)}\n`));
-        if (debugLogger_1.debugLogger.isEnabled()) {
-            console.log(picocolors_1.default.yellow(`🐛 Debug log saved to: ${debugLogger_1.debugLogger.getLogFilePath()}\n`));
-            debugLogger_1.debugLogger.disable();
+        catch (error) {
+            console.error(picocolors_1.default.red('Error during modification:'), error);
+            if (debugLogger_1.debugLogger.isEnabled()) {
+                debugLogger_1.debugLogger.logError('ModifyFlowRecursive', error);
+            }
+            showFinalSummary(costTracker, sessionId, options);
         }
     }
-    catch (error) {
-        console.error(picocolors_1.default.red('Error during modification:'), error);
-        if (debugLogger_1.debugLogger.isEnabled()) {
-            debugLogger_1.debugLogger.log('Modification flow failed: ' + String(error));
-            debugLogger_1.debugLogger.disable();
-        }
+    else {
+        showFinalSummary(costTracker, sessionId, options);
+    }
+}
+function showFinalSummary(costTracker, sessionId, options) {
+    // Show final cost breakdown only once when exiting the loop
+    costTracker.printFinalBreakdown();
+    if (debugLogger_1.debugLogger.isEnabled()) {
+        console.log(picocolors_1.default.yellow(`🐛 Debug log saved to: ${debugLogger_1.debugLogger.getLogFilePath()}\n`));
+        debugLogger_1.debugLogger.disable();
     }
+    console.log(picocolors_1.default.dim('\nView this session anytime with:'));
+    console.log(picocolors_1.default.dim(`  gameforge session ${sessionId}\n`));
 }
 //# sourceMappingURL=index.js.map