codehere 0.1.1 → 0.2.2

package/dist/application/use-cases/ask-question-use-case.js

@@ -78,35 +78,29 @@ export class AskQuestionUseCase {
         let embeddingGenTime = 0;
         if (!queryEmbedding) {
             const genStart = Date.now();
-            try {
-                // Use embedQuery if available, fallback to embed for backward compatibility
-                if ('embedQuery' in this.aiService && typeof this.aiService.embedQuery === 'function') {
-                    const embedding = await this.aiService.embedQuery(request.query);
-                    queryEmbedding = Array.isArray(embedding) && !Array.isArray(embedding[0]) ? embedding : null;
-                }
-                else {
-                    const embedding = await this.aiService.embed(request.query);
-                    if (Array.isArray(embedding[0])) {
-                        throw new Error('Expected single embedding vector');
-                    }
-                    queryEmbedding = embedding;
-                }
-                embeddingGenTime = Date.now() - genStart;
-                if (queryEmbedding) {
-                    // Cache the embedding for future use
-                    this.embeddingCache.set(request.query, queryEmbedding);
+            // Use embedQuery if available, fallback to embed for backward compatibility
+            if ('embedQuery' in this.aiService && typeof this.aiService.embedQuery === 'function') {
+                const embedding = await this.aiService.embedQuery(request.query);
+                queryEmbedding = Array.isArray(embedding) && !Array.isArray(embedding[0]) ? embedding : null;
+            }
+            else {
+                const embedding = await this.aiService.embed(request.query);
+                if (Array.isArray(embedding[0])) {
+                    throw new Error('Expected single embedding vector');
                 }
+                queryEmbedding = embedding;
             }
-            catch (embeddingError) {
-                // Embedding failed - likely network issue or API error
-                // Fall through to direct context fallback below
-                const errorMessage = embeddingError instanceof Error ? embeddingError.message : String(embeddingError);
-                console.warn('[AskQuestionUseCase] Embedding generation failed:', errorMessage);
-                queryEmbedding = null;
+            embeddingGenTime = Date.now() - genStart;
+            if (!queryEmbedding) {
+                throw new Error('Failed to generate query embedding');
             }
+            // Cache the embedding for future use
+            this.embeddingCache.set(request.query, queryEmbedding);
+        }
+        // Ensure we have a valid embedding
+        if (!queryEmbedding) {
+            throw new Error('Failed to generate query embedding');
         }
-        // If embedding failed, fall through to direct context (quick mode) fallback
-        // This allows the tool to work even without embeddings/indexing
         // 2. PARALLEL EXECUTION: Search codebase and retrieve memories simultaneously
         // OPTIMIZATION: For simple queries, reduce operations and skip memory if not needed
         const isSimpleQuery = this.isSimpleQuery(request.query);
@@ -121,27 +115,9 @@ export class AskQuestionUseCase {
         let allMemories = [];
         let searchTime = 0; // Performance metrics
         let rerankTime = 0; // Performance metrics
-        // Show progress indicator for search (if not cached and operation >2s expected)
+        // Progress is owned by the caller (REPL spinner or CLI command)
         let searchProgress = null;
-        if (!cachedResults && !isSimpleQuery) {
-            try {
-                const { createEmbeddingSearchProgress } = await import('../../infrastructure/ux/progress-indicator.js');
-                // Estimate total chunks (we don't know exact count, so use spinner)
-                searchProgress = createEmbeddingSearchProgress(0); // 0 = use spinner
-                searchProgress.start({ message: 'Searching codebase...' });
-            }
-            catch {
-                // Progress indicator not critical, continue without it
-            }
-        }
-        // Skip embedding search if we don't have a valid embedding (network error, etc.)
-        // This will fall through to the direct context fallback below
-        if (!queryEmbedding) {
-            initialChunks = [];
-            allMemories = [];
-            console.warn('[AskQuestionUseCase] No embedding available, falling back to direct context mode');
-        }
-        else if (cachedResults) {
+        if (cachedResults) {
             // Cache hit - use cached results
             this.queryResultCache.recordHit();
             initialChunks = cachedResults;
@@ -174,83 +150,8 @@ export class AskQuestionUseCase {
             }
         }
         if (initialChunks.length === 0) {
-            // FAKE_AI mode: Generate fake response for demos/testing
-            if (process.env.CODEHERE_FAKE_AI === '1') {
-                const fakeAnswer = `[FAKE_AI Mode] This is a demo response for your query: "${request.query}"
-
-In a real scenario, Codehere would search your indexed codebase and provide relevant information about your code.
-
-To use Codehere with real AI:
-1. Set up an API key (COHERE_API_KEY, CODEHERE_OPENAI_API_KEY, etc.)
-2. Run 'codehere index' to index your codebase
-3. Ask your question again
-
-Demo mode is useful for testing the CLI flow without making API calls.`;
-                return {
-                    answer: fakeAnswer,
-                    chunks: [],
-                    memories: [],
-                };
-            }
-            // Try direct context fallback (quick mode)
-            try {
-                const { getDirectContext } = await import('../../infrastructure/context/direct-context-provider.js');
-                const directChunks = await getDirectContext(request.query, {
-                    maxFiles: 15,
-                    rootDir: process.cwd(),
-                });
-                if (directChunks.length > 0) {
-                    // Use direct context to answer the question
-                    const contextText = directChunks
-                        .slice(0, 8)
-                        .map(c => `// File: ${c.filepath}\n${c.content}`)
-                        .join('\n\n---\n\n');
-                    const chatStart = Date.now();
-                    const answer = await this.aiService.chat(`You are a helpful coding assistant. Answer the user's question based on the code context provided.
-
-Question: ${request.query}
-
-Code Context:
-${contextText}
-
-Provide a helpful, accurate answer based on the code shown. If the code doesn't contain enough information to fully answer, say so.`, []);
-                    const chatTime = Date.now() - chatStart;
-                    return {
-                        answer,
-                        chunks: directChunks.slice(0, 8),
-                        memories: [],
-                        performanceMetrics: {
-                            chatTime,
-                            totalTime: Date.now() - embeddingStart,
-                        },
-                    };
-                }
-            }
-            catch (directError) {
-                // Direct context failed - show actual error to user
-                const errorMessage = directError instanceof Error ? directError.message : String(directError);
-                console.warn('[AskQuestionUseCase] Direct context fallback failed:', errorMessage);
-                // Check for common error patterns and provide helpful messages
-                if (errorMessage.includes('fetch failed') || errorMessage.includes('network')) {
-                    return {
-                        answer: `AI service connection failed. This could be due to:\n  1. Network connectivity issues\n  2. API endpoint unreachable\n  3. Firewall blocking outbound requests\n\nTry:\n  - Check your internet connection\n  - Verify the API endpoint is accessible\n  - Use CODEHERE_FAKE_AI=1 for demo mode without network`,
-                        chunks: [],
-                    };
-                }
-                if (errorMessage.includes('API key') || errorMessage.includes('401') || errorMessage.includes('unauthorized')) {
-                    return {
-                        answer: `AI service authentication failed. Check your API key:\n  - Ensure the key is valid and not expired\n  - Verify the correct environment variable is set\n\nError: ${errorMessage}`,
-                        chunks: [],
-                    };
-                }
-                // Generic error with actual message
-                return {
-                    answer: `AI service error: ${errorMessage}\n\nTry:\n  1. Run "codehere index" to index your codebase, or\n  2. Use CODEHERE_FAKE_AI=1 for demo mode`,
-                    chunks: [],
-                };
-            }
             return {
-                answer: 'No relevant code found. Try:\n  1. Run "codehere index" to index your codebase, or\n  2. Use "codehere --quick" to read files directly without indexing',
+                answer: 'No relevant code found. Please run "codehere index" first.',
                 chunks: [],
             };
         }