ctxman 3.0.0 → 3.1.1

package/index.js

@@ -27,41 +27,65 @@ import { Logger, getLogger, createLogger } from './lib/utils/logger.js'; // v2.3
 import Updater from './lib/utils/updater.js'; // v2.3.6+
 import GitUtils from './lib/utils/git-utils.js'; // v2.3.6+
 
+// Pull-request review context
+import { buildPrContext } from './lib/pr/pr-context.js';
+
 // Orchestrator functions
 import { generateDigestFromReport, generateDigestFromContext } from './ctxman.js';
 
+// Repository map (`ctxman map`)
+import { buildRepoGraph, rankFiles, renderRepoMap } from './lib/map/index.js';
+
+// Code history explanation (`ctxman why`)
+import { explainHistory } from './lib/why/index.js';
+// File contents in one document (`ctxman pack`)
+import { packContext } from './lib/pack/index.js';
+
 export {
-    // Analyzers
-    TokenCalculator,
-    MethodAnalyzer,
-
-    // Parsers
-    GitIgnoreParser,
-    MethodFilterParser,
-
-    // Formatters
-    GitIngestFormatter,
-    ToonFormatter,
-    FormatRegistry,
-
-    // Utils
-    TokenUtils,
-    FileUtils,
-    ClipboardUtils,
-    ConfigUtils,
-    FormatConverter,
-    ErrorHandler,
-
-    // v2.3.6+ Utils
-    Logger,
-    getLogger,
-    createLogger,
-    Updater,
-    GitUtils,
-
-    // Functions
-    generateDigestFromReport,
-    generateDigestFromContext
+  // Analyzers
+  TokenCalculator,
+  MethodAnalyzer,
+
+  // Parsers
+  GitIgnoreParser,
+  MethodFilterParser,
+
+  // Formatters
+  GitIngestFormatter,
+  ToonFormatter,
+  FormatRegistry,
+
+  // Utils
+  TokenUtils,
+  FileUtils,
+  ClipboardUtils,
+  ConfigUtils,
+  FormatConverter,
+  ErrorHandler,
+
+  // v2.3.6+ Utils
+  Logger,
+  getLogger,
+  createLogger,
+  Updater,
+  GitUtils,
+
+  // Functions
+  generateDigestFromReport,
+  generateDigestFromContext,
+
+  // Repository map
+  buildRepoGraph,
+  rankFiles,
+  renderRepoMap,
+
+  // Pull-request context
+  buildPrContext,
+
+  // Code history explanation (`ctxman why`)
+  explainHistory,
+  // Packed file contents
+  packContext,
 };
 
 // Alias for backward compatibility

package/lib/agents/pr-failure-agent.js

@@ -1,19 +1,22 @@
 import { ContextBuilder } from '../core/ContextBuilder.js';
 import { Scanner } from '../core/Scanner.js';
 import { Analyzer } from '../core/Analyzer.js';
+import { DEFAULT_TARGET_MODEL } from '../utils/llm-detector.js';
 
 export class PRFailureAgent {
   constructor(projectRoot, options = {}) {
     this.projectRoot = projectRoot;
     this.options = options;
-    
+
     // Initialize core components or use injected ones
     this.scanner = options.scanner || new Scanner(projectRoot);
     this.analyzer = options.analyzer || new Analyzer({ methodLevel: true }); // Enable method level for better context
-    this.contextBuilder = options.contextBuilder || new ContextBuilder({
-      targetModel: 'gpt-4', // Default to high capacity model
-      ...options
-    });
+    this.contextBuilder =
+      options.contextBuilder ||
+      new ContextBuilder({
+        targetModel: DEFAULT_TARGET_MODEL,
+        ...options,
+      });
   }
 
   /**
@@ -25,42 +28,42 @@ export class PRFailureAgent {
   async run(ciLog, diff) {
     // 1. Extract error from log
     const error = this.extractError(ciLog);
-    
+
     // 2. Identify relevant files
     const relevantFiles = this.identifyRelevantFiles(error, diff);
-    
+
     // 3. Scan codebase
     const allFiles = this.scanner.scan();
-    
+
     // Filter for relevant files
     // If no specific files identified, we might fallback to all (or risk context overflow)
     // For POC, we'll try to fuzzy match the relevant files against scanned files
     let targetFiles = allFiles;
-    
+
     if (relevantFiles.length > 0) {
-      targetFiles = allFiles.filter(file => {
-        return relevantFiles.some(relevant => file.relativePath.endsWith(relevant));
+      targetFiles = allFiles.filter((file) => {
+        return relevantFiles.some((relevant) => file.relativePath.endsWith(relevant));
       });
     }
 
     // 4. Analyze relevant files
     const analysis = await this.analyzer.analyze(targetFiles);
-    
+
     // 5. Build Context
     const context = this.contextBuilder.build(analysis);
-    
+
     // 6. Construct Prompt
     const prompt = this.constructPrompt(error, diff, context);
-    
+
     // 7. Generate Fix
     const fix = await this.generateFix(prompt);
-    
+
     return fix;
   }
 
   /**
    * Extract error message from CI log
-   * @param {string} log 
+   * @param {string} log
    * @returns {string}
    */
   extractError(log) {
@@ -68,32 +71,32 @@ export class PRFailureAgent {
     // Matches "Error:", "FAIL", "Exception" and the rest of the line
     const errorRegex = /(?:Error:|FAIL|Exception:|Caused by:).*/gi;
     const matches = log.match(errorRegex);
-    
+
     if (matches && matches.length > 0) {
       // Return the unique errors found, joined
       return [...new Set(matches)].join('\n');
     }
-    
-    return "Could not extract specific error message from log.";
+
+    return 'Could not extract specific error message from log.';
   }
 
   /**
    * Identify relevant files from error and diff
-   * @param {string} error 
-   * @param {string} diff 
+   * @param {string} error
+   * @param {string} diff
    * @returns {string[]} - Array of file paths/names
    */
   identifyRelevantFiles(error, diff) {
     const files = new Set();
-    
+
     // Regex for common source file extensions, allowing paths
     // Order matters: longer extensions first to avoid partial matches (e.g., .ts matching .tsx)
     const fileExtRegex = /[\w\-./]+\.(tsx|jsx|ts|js|py|java|rb|go|c|cpp|hpp|h|rs|php)/gi;
-    
+
     // Find files in error message
     const errorFiles = error.match(fileExtRegex) || [];
-    errorFiles.forEach(f => files.add(f));
-    
+    errorFiles.forEach((f) => files.add(f));
+
     // Find files in diff
     // Look for lines like "diff --git a/path/to/file.js b/path/to/file.js"
     // or "+++ b/path/to/file.js"
@@ -102,7 +105,7 @@ export class PRFailureAgent {
     while ((match = diffFileRegex.exec(diff)) !== null) {
       const fullPath = match[1].trim();
       // extracting just the filename might be safer for fuzzy matching if paths differ
-      // but let's try to keep the path if possible. 
+      // but let's try to keep the path if possible.
       // scanner returns relativePath, so if we capture relative path here it's good.
       files.add(fullPath);
     }
@@ -112,29 +115,29 @@ export class PRFailureAgent {
 
   /**
    * Construct the prompt for the LLM
-   * @param {string} error 
-   * @param {string} diff 
-   * @param {object} context 
+   * @param {string} error
+   * @param {string} diff
+   * @param {object} context
    * @returns {string}
    */
   constructPrompt(error, diff, context) {
     // In a real agent, we might use a template engine or more sophisticated structure
-    
+
     const contextSummary = this.contextBuilder.getSummary(context);
-    
+
     // Serialize files from context
     // context.files is an object where keys are directories, values are arrays of file objects
     // We need the content, but ContextBuilder (in the version I saw) returns metadata.
     // Wait, Analyzer reads the content to count tokens, but does it return it?
     // Checking Analyzer.js: it returns { path, tokens, ... } but NOT content.
-    // 
-    // Issue: The ContextBuilder builds metadata/structure but might not include full content 
+    //
+    // Issue: The ContextBuilder builds metadata/structure but might not include full content
     // depending on how it was implemented or intended.
-    // 
+    //
     // Re-reading Analyzer.js: "const content = fs.readFileSync..." -> calculates tokens -> returns analysis object.
     // The analysis object does NOT contain 'content'.
     //
-    // So for the prompt, I actually need the CONTENT. 
+    // So for the prompt, I actually need the CONTENT.
     // The current ContextBuilder seems to be for *sizing* or *listing* context, or maybe I missed where content is stored.
     //
     // Let's re-read ContextBuilder build() method.
@@ -146,15 +149,15 @@ export class PRFailureAgent {
     // Or maybe it relies on the user to read the files based on the list?
     //
     // To make a functional POC, I should probably read the content of the relevant files.
-    // 
+    //
     // I will add a step to read content for the prompt.
-    
-    let fileContentsSection = "";
-    
+
+    let _fileContentsSection = '';
+
     // Since I can't easily change the Analyzer/ContextBuilder right now without Scope creep,
-    // I will just list the files and Mock the content reading for the prompt construction 
+    // I will just list the files and Mock the content reading for the prompt construction
     // or import fs to read them.
-    
+
     return `
 You are a PR Failure Fixing Agent.
 
@@ -177,13 +180,13 @@ Please analyze the error and the code, and provide a fix.
 
   /**
    * Mock LLM Generation
-   * @param {string} prompt 
+   * @param {string} prompt
    * @returns {Promise<string>}
    */
-  async generateFix(prompt) {
+  async generateFix(_prompt) {
     // Simulate LLM delay
-    await new Promise(resolve => setTimeout(resolve, 100));
-    
+    await new Promise((resolve) => setTimeout(resolve, 100));
+
     return `
 Here is the fix for the reported error: