opencode-codegraph 0.1.1 → 0.1.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "opencode-codegraph",
-  "version": "0.1.1",
+  "version": "0.1.2",
   "description": "OpenCode plugin for CodeGraph CPG-powered code analysis",
   "type": "module",
   "main": "src/index.ts",

package/src/api.ts

@@ -6,24 +6,31 @@
  */
 
 export class CodeGraphAPI {
-  private baseUrl: string
-
-  constructor(baseUrl: string) {
-    this.baseUrl = baseUrl.replace(/\/+$/, "")
+  private baseUrl: string
+
+  constructor(baseUrl: string) {
+    const trimmed = baseUrl.replace(/\/+$/, "")
+    this.baseUrl = trimmed.replace(/\/api\/v1$/i, "")
   }
 
   /**
    * Get project summary for system prompt injection.
    * Returns a markdown block with file count, top hotspots, and open security findings.
    */
-  async getProjectSummary(projectId: string): Promise<string | null> {
-    const params = projectId ? `?project_id=${encodeURIComponent(projectId)}` : ""
-
-    // Fetch multiple endpoints in parallel
-    const [statsRes, findingsRes] = await Promise.allSettled([
-      this.fetch(`/api/v1/cpg/stats${params}`),
-      this.fetch(`/api/v1/cpg/pattern-results${params}&category=security&limit=10`),
-    ])
+  async getProjectSummary(projectId: string): Promise<string | null> {
+    const params = new URLSearchParams()
+    if (projectId) params.set("project_id", projectId)
+    const statsPath = params.toString() ? `/api/v1/cpg/stats?${params}` : "/api/v1/cpg/stats"
+    const findingsParams = new URLSearchParams(params)
+    findingsParams.set("category", "security")
+    findingsParams.set("limit", "10")
+    const findingsPath = `/api/v1/cpg/pattern-results?${findingsParams}`
+
+    // Fetch multiple endpoints in parallel
+    const [statsRes, findingsRes] = await Promise.allSettled([
+      this.fetch(statsPath),
+      this.fetch(findingsPath),
+    ])
 
     const stats =
       statsRes.status === "fulfilled" ? statsRes.value : null
@@ -66,14 +73,16 @@ export class CodeGraphAPI {
    * Returns markdown with methods, callers, complexity, and security findings.
    */
   async getFileCPGContext(projectId: string, filePath: string): Promise<string | null> {
-    const params = new URLSearchParams()
-    if (projectId) params.set("project_id", projectId)
-    params.set("filename", filePath)
-
-    const [methodsRes, findingsRes] = await Promise.allSettled([
-      this.fetch(`/api/v1/cpg/methods?${params}`),
-      this.fetch(`/api/v1/cpg/pattern-results?${params}&category=security`),
-    ])
+    const params = new URLSearchParams()
+    if (projectId) params.set("project_id", projectId)
+    params.set("filename", filePath)
+    const findingsParams = new URLSearchParams(params)
+    findingsParams.set("category", "security")
+
+    const [methodsRes, findingsRes] = await Promise.allSettled([
+      this.fetch(`/api/v1/cpg/methods?${params}`),
+      this.fetch(`/api/v1/cpg/pattern-results?${findingsParams.toString()}`),
+    ])
 
     const methods =
       methodsRes.status === "fulfilled" ? methodsRes.value : null
@@ -161,14 +170,18 @@ export class CodeGraphAPI {
         lines.push("")
       }
 
-      if (result.complexity_changes?.length) {
-        lines.push("### Complexity Changes")
-        for (const c of result.complexity_changes) {
-          const delta = c.new_cc - c.old_cc
-          const sign = delta > 0 ? "+" : ""
-          lines.push(`- \`${c.method}\`: ${c.old_cc} -> ${c.new_cc} (${sign}${delta})`)
-        }
-      }
+      if (result.complexity_changes?.length) {
+        lines.push("### Complexity Changes")
+        for (const c of result.complexity_changes) {
+          if ((c.old_cc ?? 0) > 0) {
+            const delta = c.new_cc - c.old_cc
+            const sign = delta > 0 ? "+" : ""
+            lines.push(`- \`${c.method}\`: ${c.old_cc} -> ${c.new_cc} (${sign}${delta})`)
+          } else {
+            lines.push(`- \`${c.method}\`: current CC ${c.new_cc}`)
+          }
+        }
+      }
 
       return lines.length > 2 ? lines.join("\n") : "CodeGraph review: no issues found."
     } catch {
@@ -237,29 +250,60 @@ export class CodeGraphAPI {
     }
   }
 
-  // -------------------------------------------------------------------
-  // Private helpers
-  // -------------------------------------------------------------------
-
-  private async fetch(path: string, init?: RequestInit): Promise<any> {
-    const url = `${this.baseUrl}${path}`
-    const headers: Record<string, string> = {
-      "Content-Type": "application/json",
-      ...(init?.headers as Record<string, string> | undefined),
-    }
-
-    const response = await globalThis.fetch(url, {
-      ...init,
-      headers,
-    })
-
-    if (!response.ok) {
-      throw new Error(`CodeGraph API ${response.status}: ${response.statusText}`)
-    }
-
-    const text = await response.text()
-    if (!text) return null
-
-    return JSON.parse(text)
-  }
-}
+  // -------------------------------------------------------------------
+  // Private helpers
+  // -------------------------------------------------------------------
+
+  private resolveUrl(path: string): string {
+    const normalizedPath = path.startsWith("/") ? path : `/${path}`
+    return new URL(normalizedPath, `${this.baseUrl}/`).toString()
+  }
+
+  private async parseJsonBody(response: Response): Promise<any> {
+    const text = await response.text()
+    if (!text) return null
+    return JSON.parse(text)
+  }
+
+  private async fetch(path: string, init?: RequestInit): Promise<any> {
+    const headers: Record<string, string> = {
+      "Content-Type": "application/json",
+      ...(init?.headers as Record<string, string> | undefined),
+    }
+
+    const response = await globalThis.fetch(this.resolveUrl(path), {
+      ...init,
+      headers,
+    })
+
+    if (response.status === 404 && path.startsWith("/api/v1/cpg/pattern-results")) {
+      const fallbackPath = path.replace(
+        "/api/v1/cpg/pattern-results",
+        "/api/v1/patterns/findings",
+      )
+      const fallbackResponse = await globalThis.fetch(this.resolveUrl(fallbackPath), {
+        ...init,
+        headers,
+      })
+
+      if (fallbackResponse.ok) {
+        const payload = await this.parseJsonBody(fallbackResponse)
+        const findings = Array.isArray(payload?.findings) ? payload.findings : []
+        return {
+          results: findings.map((finding: any) => ({
+            ...finding,
+            line: finding.line ?? finding.line_number ?? 0,
+          })),
+        }
+      }
+
+      throw new Error(`CodeGraph API ${fallbackResponse.status}: ${fallbackResponse.statusText}`)
+    }
+
+    if (!response.ok) {
+      throw new Error(`CodeGraph API ${response.status}: ${response.statusText}`)
+    }
+
+    return this.parseJsonBody(response)
+  }
+}

package/src/index.ts

@@ -48,20 +48,20 @@ const codegraphPlugin: Plugin = async (input) => {
       const files = extractFileRefs(output.parts)
       if (files.length === 0) return
 
-      try {
-        for (const file of files.slice(0, 5)) {
-          const context = await api.getFileCPGContext(projectId, file)
-          if (context) {
-            output.parts.push({
-              type: "text",
-              text: context,
-            })
-          }
-        }
-      } catch {
-        // CodeGraph API not available — skip silently
-      }
-    },
+      try {
+        for (const file of files.slice(0, 5)) {
+          const context = await api.getFileCPGContext(projectId, file)
+          if (context) {
+            output.parts.push({
+              type: "text",
+              text: context,
+            } as any)
+          }
+        }
+      } catch {
+        // CodeGraph API not available — skip silently
+      }
+    },
 
     // -----------------------------------------------------------------
     // 3. Post-commit: trigger incremental CPG update
@@ -123,14 +123,15 @@ const codegraphPlugin: Plugin = async (input) => {
       }),
     },
 
-    // -----------------------------------------------------------------
-    // 5. Permissions: auto-allow codegraph_* MCP tools
-    // -----------------------------------------------------------------
-    "permission.ask": async (inp, output) => {
-      if (inp.tool?.startsWith("codegraph_")) {
-        output.status = "allow"
-      }
-    },
+    // -----------------------------------------------------------------
+    // 5. Permissions: auto-allow codegraph_* MCP tools
+    // -----------------------------------------------------------------
+    "permission.ask": async (inp, output) => {
+      const toolName = (inp as any)?.tool as string | undefined
+      if (toolName?.startsWith("codegraph_")) {
+        output.status = "allow"
+      }
+    },
   }
 }