@makeitvisible/cli 0.1.0 → 0.2.0

package/dist/index.d.ts

@@ -1,66 +1,7 @@
+import { A as ArtifactSource, a as ArtifactContext } from './types-BmNeVNFe.js';
+export { b as Artifact, c as ArtifactStatus, C as CreateArtifactInput, d as CreateArtifactResponse } from './types-BmNeVNFe.js';
 import { ChatCompletionTool } from 'openai/resources/chat/completions';
 
-/**
- * Shared Types for Visible CLI
- *
- * Types related to artifacts generated from code analysis.
- * (Vendored from @visible/shared for standalone package)
- */
-/**
- * Represents an artifact generated from code analysis.
- * An artifact contains high-level descriptions and guidelines
- * that serve as input for final material generation.
- */
-interface Artifact {
-    id: string;
-    title: string;
-    description: string;
-    source: ArtifactSource;
-    context: ArtifactContext;
-    guidelines: string[];
-    createdAt: Date;
-    updatedAt: Date;
-}
-/**
- * The source of the artifact (PR, commit, code section, etc.)
- */
-interface ArtifactSource {
-    type: "pr" | "commit" | "code-section" | "repository";
-    reference: string;
-    url?: string;
-}
-/**
- * Context extracted from the code analysis
- */
-interface ArtifactContext {
-    summary: string;
-    technicalDetails: string[];
-    affectedComponents: string[];
-    breakingChanges: boolean;
-    keywords: string[];
-}
-/**
- * Status of an artifact
- */
-type ArtifactStatus = "draft" | "ready" | "generating" | "completed";
-/**
- * Input for creating a new artifact (from CLI)
- */
-interface CreateArtifactInput {
-    title: string;
-    description: string;
-    source: ArtifactSource;
-    context: ArtifactContext;
-    guidelines: string[];
-}
-/**
- * Response after creating an artifact
- */
-interface CreateArtifactResponse {
-    id: string;
-    url: string;
-}
-
 /**
  * Git Analyzer
  *
@@ -73,12 +14,25 @@ interface CreateArtifactResponse {
  * - Identify changed files and their context
  */
 
+/**
+ * Code snippet extracted from diff
+ */
+interface DiffCodeSnippet {
+    file: string;
+    code: string;
+    startLine?: number;
+    language?: string;
+    description?: string;
+    changeType: "added" | "deleted" | "modified" | "context";
+}
 interface AnalysisResult {
     title: string;
     description: string;
     source: ArtifactSource;
     context: ArtifactContext;
     guidelines: string[];
+    /** Code snippets extracted from the diff */
+    codeSnippets?: DiffCodeSnippet[];
 }
 /**
  * Analyze a git diff range and produce an artifact
@@ -228,6 +182,7 @@ interface DetectiveOptions {
     projectRoot: string;
     verbose?: boolean;
     maxIterations?: number;
+    maxToolCalls?: number;
     onToolCall?: (toolName: string, args: unknown) => void;
     onToolResult?: (toolName: string, result: ToolResult) => void;
     onThinking?: (thought: string) => void;
@@ -289,4 +244,4 @@ declare function investigateFeature(query: string, options: Omit<DetectiveOption
     projectRoot?: string;
 }): Promise<InvestigationResult>;
 
-export { AGENT_TOOLS, type AnalysisResult, type Artifact, type ArtifactContext, type ArtifactSource, type ArtifactStatus, type ContextInput, type CreateArtifactInput, type CreateArtifactResponse, DetectiveAgent, type DetectiveOptions, type InvestigationResult, type PROptions, type ToolContext, type ToolName, type ToolResult, analyzeDiff, analyzePR, artifactGenerator, detectBreakingChangesFromDiff, executeTool, extractContextFromPaths, findDefinition, findReferences, generateTechnicalSummary, getFileStructure, investigateFeature, listDirectory, mergeContexts, readFile, searchFiles, searchTypes };
+export { AGENT_TOOLS, type AnalysisResult, ArtifactContext, ArtifactSource, type ContextInput, DetectiveAgent, type DetectiveOptions, type DiffCodeSnippet, type InvestigationResult, type PROptions, type ToolContext, type ToolName, type ToolResult, analyzeDiff, analyzePR, artifactGenerator, detectBreakingChangesFromDiff, executeTool, extractContextFromPaths, findDefinition, findReferences, generateTechnicalSummary, getFileStructure, investigateFeature, listDirectory, mergeContexts, readFile, searchFiles, searchTypes };

package/dist/index.js

@@ -63,6 +63,92 @@ function getCommits(range) {
 function getDiffContent(range) {
   return git(`diff ${range}`);
 }
+function extractCodeSnippets(diffContent, maxSnippets = 6) {
+  const snippets = [];
+  const lines = diffContent.split("\n");
+  let currentFile = "";
+  let currentHunk = [];
+  let currentStartLine = 0;
+  let inHunk = false;
+  for (const line of lines) {
+    if (line.startsWith("diff --git")) {
+      if (currentHunk.length > 0 && currentFile) {
+        snippets.push(createSnippetFromHunk(currentFile, currentHunk, currentStartLine));
+        if (snippets.length >= maxSnippets) break;
+      }
+      currentHunk = [];
+      inHunk = false;
+      continue;
+    }
+    if (line.startsWith("+++ b/")) {
+      currentFile = line.slice(6);
+      continue;
+    }
+    if (line.startsWith("@@")) {
+      if (currentHunk.length > 0 && currentFile) {
+        snippets.push(createSnippetFromHunk(currentFile, currentHunk, currentStartLine));
+        if (snippets.length >= maxSnippets) break;
+      }
+      const match = line.match(/@@ -\d+(?:,\d+)? \+(\d+)/);
+      currentStartLine = match ? parseInt(match[1], 10) : 1;
+      currentHunk = [];
+      inHunk = true;
+      continue;
+    }
+    if (inHunk && currentHunk.length < 15) {
+      currentHunk.push(line);
+    }
+  }
+  if (currentHunk.length > 0 && currentFile && snippets.length < maxSnippets) {
+    snippets.push(createSnippetFromHunk(currentFile, currentHunk, currentStartLine));
+  }
+  return snippets;
+}
+function createSnippetFromHunk(file, hunkLines, startLine) {
+  const hasAdditions = hunkLines.some((l) => l.startsWith("+"));
+  const hasDeletions = hunkLines.some((l) => l.startsWith("-"));
+  let changeType;
+  if (hasAdditions && hasDeletions) {
+    changeType = "modified";
+  } else if (hasAdditions) {
+    changeType = "added";
+  } else if (hasDeletions) {
+    changeType = "deleted";
+  } else {
+    changeType = "context";
+  }
+  const code = hunkLines.map((line) => {
+    if (line.startsWith("+") || line.startsWith("-") || line.startsWith(" ")) {
+      return line.slice(1);
+    }
+    return line;
+  }).join("\n");
+  const ext = file.split(".").pop()?.toLowerCase() || "";
+  const langMap = {
+    ts: "typescript",
+    tsx: "typescript",
+    js: "javascript",
+    jsx: "javascript",
+    vue: "vue",
+    py: "python",
+    rb: "ruby",
+    go: "go",
+    rs: "rust",
+    java: "java",
+    css: "css",
+    scss: "scss",
+    html: "html",
+    json: "json"
+  };
+  return {
+    file,
+    code,
+    startLine,
+    language: langMap[ext] || "text",
+    description: `${changeType === "added" ? "Added" : changeType === "deleted" ? "Removed" : "Changed"} in ${file}`,
+    changeType
+  };
+}
 function detectBreakingChanges(commits, diffContent) {
   const breakingPatterns = [
     /BREAKING CHANGE/i,
@@ -213,12 +299,14 @@ async function analyzeDiff(range) {
     breakingChanges,
     keywords
   };
+  const codeSnippets = extractCodeSnippets(diffContent);
   return {
     title,
     description,
     source,
     context,
-    guidelines
+    guidelines,
+    codeSnippets
   };
 }
 async function analyzePR(options) {
@@ -1167,6 +1255,14 @@ Your mission is to investigate a codebase based on a user's query and produce a
 
 6. **Document Technical Details**: Note any important patterns, validations, error handling, or edge cases.
 
+## Efficiency Rules (Important)
+
+- Be efficient: use as few tools as possible (aim for 8-12 tool calls).
+- Prefer targeted \`search_files\` in content or filename over broad directory listing.
+- Read only the most relevant files; avoid reading the same file multiple times.
+- If you have enough information to explain the feature, **complete the investigation** even if some sections are partial.
+- Use empty arrays for unknown sections rather than continuing to search.
+
 ## Tool Usage Tips
 
 - Use \`search_files\` with searchIn="filename" first for broad discovery
@@ -1210,6 +1306,7 @@ var DetectiveAgent = class {
     this.context = { projectRoot: options.projectRoot };
     this.options = {
       maxIterations: 25,
+      maxToolCalls: 18,
       ...options
     };
     this.messages = [{ role: "system", content: SYSTEM_PROMPT }];
@@ -1228,14 +1325,23 @@ Use the available tools to explore the codebase, understand the feature, and the
     });
     let iterations = 0;
     const maxIterations = this.options.maxIterations;
+    const maxToolCalls = this.options.maxToolCalls;
     while (iterations < maxIterations) {
       iterations++;
       try {
+        const shouldForceCompletion = this.toolCallCount >= maxToolCalls || iterations === maxIterations;
+        if (shouldForceCompletion) {
+          this.messages.push({
+            role: "user",
+            content: "Finalize the investigation now using ONLY the information already gathered. Do not call any tools except complete_investigation. Use empty arrays for any section you could not verify."
+          });
+        }
+        const tools = shouldForceCompletion ? AGENT_TOOLS.filter((tool) => tool.function.name === "complete_investigation") : AGENT_TOOLS;
         const response = await this.openai.chat.completions.create({
-          model: "gpt-4o",
+          model: "gpt-5.2",
           messages: this.messages,
-          tools: AGENT_TOOLS,
-          tool_choice: "auto",
+          tools,
+          tool_choice: shouldForceCompletion ? "required" : "auto",
           temperature: 0.1
         });
         const message = response.choices[0].message;
@@ -1243,15 +1349,41 @@ Use the available tools to explore the codebase, understand the feature, and the
         if (message.content && this.options.onThinking) {
           this.options.onThinking(message.content);
         }
-        if (!message.tool_calls || message.tool_calls.length === 0) {
-          return {
-            success: false,
-            error: "Agent finished without completing investigation",
-            toolCalls: this.toolCallCount
-          };
+        let toolCalls = message.tool_calls;
+        if (!toolCalls || toolCalls.length === 0) {
+          if (!shouldForceCompletion) {
+            this.messages.push({
+              role: "user",
+              content: "Complete the investigation now using the current context. Call complete_investigation with best-effort details and empty arrays where needed."
+            });
+            const forced = await this.openai.chat.completions.create({
+              model: "gpt-5.2",
+              messages: this.messages,
+              tools: AGENT_TOOLS.filter((tool) => tool.function.name === "complete_investigation"),
+              tool_choice: "required",
+              temperature: 0.1
+            });
+            const forcedMessage = forced.choices[0].message;
+            this.messages.push(forcedMessage);
+            if (forcedMessage.tool_calls && forcedMessage.tool_calls.length > 0) {
+              toolCalls = forcedMessage.tool_calls;
+            } else {
+              return {
+                success: false,
+                error: forcedMessage.content ? `Agent finished without tool calls: ${forcedMessage.content}` : "Agent finished without completing investigation",
+                toolCalls: this.toolCallCount
+              };
+            }
+          } else {
+            return {
+              success: false,
+              error: message.content ? `Agent finished without tool calls: ${message.content}` : "Agent finished without completing investigation",
+              toolCalls: this.toolCallCount
+            };
+          }
         }
         const toolResults = [];
-        for (const toolCall of message.tool_calls) {
+        for (const toolCall of toolCalls ?? []) {
           const toolName = toolCall.function.name;
           let args;
           try {