specvector 0.1.7 → 0.1.9

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "specvector",
-  "version": "0.1.7",
+  "version": "0.1.9",
   "description": "Context-aware AI code review using Model Context Protocol (MCP)",
   "type": "module",
   "main": "src/index.ts",

package/src/context/linear.ts

@@ -1,10 +1,9 @@
 /**
  * Linear Context Provider - Fetches ticket context for reviews.
  *
- * Extracts ticket ID from branch name/PR and fetches details via MCP.
+ * Extracts ticket ID from branch name/PR and fetches details via Linear GraphQL API.
  */
 
-import { createMCPClient } from "../mcp";
 import type { Result } from "../types/result";
 import { ok, err } from "../types/result";
 
@@ -13,12 +12,12 @@ import { ok, err } from "../types/result";
 // ============================================================================
 
 export interface LinearTicket {
-  id: string;
-  title: string;
-  description: string;
-  state: string;
-  priority: number;
-  labels: string[];
+  identifier?: string;
+  title?: string;
+  description?: string;
+  state?: { name?: string };
+  priority?: number;
+  labels?: { nodes?: Array<{ name?: string }> };
 }
 
 export interface LinearContextError {
@@ -76,7 +75,7 @@ export function extractTicketId(
 /**
  * Fetch Linear ticket context for a review.
  *
- * Returns formatted context string to inject into review prompt.
+ * Uses Linear's GraphQL API directly - simpler and more reliable than MCP.
  */
 export async function fetchLinearContext(
   ticketId: string
@@ -90,67 +89,85 @@ export async function fetchLinearContext(
     });
   }
 
-  // Create MCP client
-  const clientResult = await createMCPClient({
-    name: "linear",
-    command: "npx",
-    args: ["linear-mcp-server"],
-    env: {
-      LINEAR_API_KEY: apiToken,
-    },
-    timeout: 30000,
-  });
-
-  if (!clientResult.ok) {
-    return err({
-      code: "CONNECTION_FAILED",
-      message: clientResult.error.message,
-    });
-  }
-
-  const client = clientResult.value;
-
   try {
-    // Fetch ticket details
-    const ticketResult = await client.callTool("get_ticket", { ticket_id: ticketId });
+    // Use Linear GraphQL API directly
+    const response = await fetch("https://api.linear.app/graphql", {
+      method: "POST",
+      headers: {
+        "Content-Type": "application/json",
+        Authorization: apiToken,
+      },
+      body: JSON.stringify({
+        query: `
+          query GetIssue($id: String!) {
+            issue(id: $id) {
+              identifier
+              title
+              description
+              state { name }
+              priority
+              labels { nodes { name } }
+            }
+          }
+        `,
+        variables: { id: ticketId },
+      }),
+    });
 
-    if (!ticketResult.ok) {
+    if (!response.ok) {
       return err({
         code: "FETCH_FAILED",
-        message: ticketResult.error.message,
+        message: `Linear API error: ${response.status}`,
       });
     }
 
-    // Parse the response
-    const content = ticketResult.value.content;
-    const textContent = content.find(c => c.type === "text");
+    const data = await response.json() as { data?: { issue?: LinearTicket | null }; errors?: Array<{ message: string }> };
     
-    if (!textContent?.text) {
+    if (data.errors?.length) {
+      return err({
+        code: "FETCH_FAILED",
+        message: data.errors[0]?.message ?? "GraphQL error",
+      });
+    }
+
+    const issue = data.data?.issue;
+    if (!issue) {
       return err({
         code: "FETCH_FAILED",
-        message: "No ticket data returned",
+        message: `Issue ${ticketId} not found`,
       });
     }
 
     // Format the context for the review prompt
-    const contextBlock = formatLinearContext(ticketId, textContent.text);
+    const contextBlock = formatLinearContext(ticketId, issue);
     return ok(contextBlock);
 
-  } finally {
-    await client.close();
+  } catch (error) {
+    return err({
+      code: "FETCH_FAILED",
+      message: error instanceof Error ? error.message : "Unknown error",
+    });
   }
 }
 
 /**
- * Format Linear ticket data as context for review prompt.
+ * Format Linear issue data as context for review prompt.
  */
-function formatLinearContext(ticketId: string, ticketData: string): string {
+function formatLinearContext(ticketId: string, issue: { title?: string; description?: string; state?: { name?: string }; priority?: number; labels?: { nodes?: Array<{ name?: string }> } }): string {
+  const labels = issue.labels?.nodes?.map(l => l.name).filter(Boolean).join(", ") || "none";
+  const priority = issue.priority ?? 0;
+  const priorityLabels = ["", "Urgent", "High", "Medium", "Low", "No Priority"];
+
   return `
 ## Linear Ticket Context: ${ticketId}
 
-The following is the context from the linked Linear ticket. Use this to understand the business requirements and acceptance criteria for this PR.
+**Title:** ${issue.title ?? "Untitled"}
+**Status:** ${issue.state?.name ?? "Unknown"}
+**Priority:** ${priorityLabels[priority] ?? "Unknown"}
+**Labels:** ${labels}
 
-${ticketData}
+### Description
+${issue.description ?? "No description provided."}
 
 ---
 When reviewing, check that the PR changes align with the ticket requirements above.