specvector 0.1.2 → 0.1.4

package/LICENSE

@@ -0,0 +1,21 @@
+MIT License
+
+Copyright (c) 2026 Not Diamond
+
+Permission is hereby granted, free of charge, to any person obtaining a copy
+of this software and associated documentation files (the "Software"), to deal
+in the Software without restriction, including without limitation the rights
+to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
+copies of the Software, and to permit persons to whom the Software is
+furnished to do so, subject to the following conditions:
+
+The above copyright notice and this permission notice shall be included in all
+copies or substantial portions of the Software.
+
+THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
+IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
+FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
+AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
+LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
+OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
+SOFTWARE.

package/package.json

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

package/src/config/index.ts

@@ -24,6 +24,8 @@ export interface SpecVectorConfig {
   maxFileSize: number;
   /** Maximum iterations for agent */
   maxIterations: number;
+  /** Path to ADR directory (relative to project root), null to disable */
+  adrPath: string | null;
 }
 
 /** Default configuration */
@@ -47,6 +49,7 @@ export const DEFAULT_CONFIG: SpecVectorConfig = {
   strictness: "normal",
   maxFileSize: 100 * 1024, // 100KB
   maxIterations: 15,
+  adrPath: null, // Disabled by default, set to "docs/adr" to enable
 };
 
 /** Config file name */

package/src/context/adr.ts

@@ -0,0 +1,218 @@
+/**
+ * ADR Context Provider - Reads Architecture Decision Records for reviews.
+ *
+ * Scans a configurable directory for markdown files and includes them
+ * as context in the review prompt.
+ */
+
+import { readdir, readFile, stat } from "fs/promises";
+import { join, basename } from "path";
+import type { Result } from "../types/result";
+import { ok, err } from "../types/result";
+
+// ============================================================================
+// Types
+// ============================================================================
+
+export interface ADRFile {
+  /** Filename (e.g., "001-use-bun-runtime.md") */
+  name: string;
+  /** Full file content */
+  content: string;
+}
+
+export interface ADRContext {
+  /** List of ADR files found */
+  files: ADRFile[];
+  /** Number of files loaded */
+  count: number;
+  /** Path that was scanned */
+  path: string;
+}
+
+export interface ADRContextError {
+  code: "PATH_NOT_FOUND" | "READ_ERROR" | "NO_FILES";
+  message: string;
+}
+
+// ============================================================================
+// Constants
+// ============================================================================
+
+/** Maximum ADR file size (50KB) */
+const MAX_ADR_FILE_SIZE = 50 * 1024;
+
+/** Maximum number of ADR files to read */
+const MAX_ADR_FILES = 20;
+
+/** Supported file extensions */
+const ADR_EXTENSIONS = [".md", ".markdown"];
+
+/** Common paths to auto-detect (in priority order) */
+const AUTO_DETECT_PATHS = [
+  "_bmad-output/planning-artifacts",  // BMAD projects
+  "docs/adr",                          // Standard ADR location
+  "docs/architecture",                 // Common alternative
+  "adr",                               // Simple ADR folder
+];
+
+// ============================================================================
+// Main Function
+// ============================================================================
+
+/**
+ * Read ADR files from a directory.
+ *
+ * @param workingDir - The project root directory
+ * @param adrPath - Relative path to ADR directory (e.g., "docs/adr")
+ */
+export async function getADRContext(
+  workingDir: string,
+  adrPath: string
+): Promise<Result<ADRContext, ADRContextError>> {
+  const fullPath = join(workingDir, adrPath);
+
+  // Check if directory exists
+  try {
+    const stats = await stat(fullPath);
+    if (!stats.isDirectory()) {
+      return err({
+        code: "PATH_NOT_FOUND",
+        message: `ADR path is not a directory: ${adrPath}`,
+      });
+    }
+  } catch {
+    return err({
+      code: "PATH_NOT_FOUND",
+      message: `ADR directory not found: ${adrPath}`,
+    });
+  }
+
+  // Read directory contents
+  let entries: string[];
+  try {
+    entries = await readdir(fullPath);
+  } catch (error) {
+    return err({
+      code: "READ_ERROR",
+      message: `Failed to read ADR directory: ${error instanceof Error ? error.message : "Unknown error"}`,
+    });
+  }
+
+  // Filter to markdown files
+  const mdFiles = entries
+    .filter((f) => ADR_EXTENSIONS.some((ext) => f.toLowerCase().endsWith(ext)))
+    .sort() // Sort alphabetically (ADRs often numbered: 001-xxx.md)
+    .slice(0, MAX_ADR_FILES);
+
+  if (mdFiles.length === 0) {
+    return err({
+      code: "NO_FILES",
+      message: `No ADR markdown files found in: ${adrPath}`,
+    });
+  }
+
+  // Read each file
+  const files: ADRFile[] = [];
+  for (const filename of mdFiles) {
+    const filePath = join(fullPath, filename);
+    try {
+      const fileStats = await stat(filePath);
+      
+      // Skip files that are too large
+      if (fileStats.size > MAX_ADR_FILE_SIZE) {
+        console.log(`⚠️  Skipping large ADR: ${filename} (${Math.round(fileStats.size / 1024)}KB)`);
+        continue;
+      }
+
+      const content = await readFile(filePath, "utf-8");
+      files.push({
+        name: basename(filename),
+        content: content.trim(),
+      });
+    } catch {
+      // Skip files we can't read
+      console.log(`⚠️  Could not read ADR: ${filename}`);
+    }
+  }
+
+  if (files.length === 0) {
+    return err({
+      code: "NO_FILES",
+      message: `No readable ADR files in: ${adrPath}`,
+    });
+  }
+
+  return ok({
+    files,
+    count: files.length,
+    path: adrPath,
+  });
+}
+
+// ============================================================================
+// Formatting
+// ============================================================================
+
+/**
+ * Format ADR context for inclusion in review prompt.
+ */
+export function formatADRContext(context: ADRContext): string {
+  const lines: string[] = [
+    `## Architecture Decision Records (${context.count} files from ${context.path})`,
+    "",
+  ];
+
+  for (const file of context.files) {
+    lines.push(`### ${file.name}`);
+    lines.push("");
+    lines.push(file.content);
+    lines.push("");
+    lines.push("---");
+    lines.push("");
+  }
+
+  return lines.join("\n");
+}
+
+/**
+ * Get ADR context for a review, with graceful error handling.
+ * Returns null if ADRs are not available (no error thrown).
+ * 
+ * If adrPath is not set, auto-detects common paths:
+ * - _bmad-output/planning-artifacts (BMAD projects)
+ * - docs/adr (standard ADR location)
+ * - docs/architecture (common alternative)
+ * - adr (simple ADR folder)
+ */
+export async function getADRContextForReview(
+  workingDir: string,
+  adrPath: string | null | undefined
+): Promise<{ context: ADRContext; formatted: string } | null> {
+  // If path is explicitly set, use it
+  if (adrPath) {
+    const result = await getADRContext(workingDir, adrPath);
+    if (!result.ok) {
+      console.log(`⚠️  ADR context unavailable: ${result.error.message}`);
+      return null;
+    }
+    return {
+      context: result.value,
+      formatted: formatADRContext(result.value),
+    };
+  }
+
+  // Auto-detect: try common paths in priority order
+  for (const path of AUTO_DETECT_PATHS) {
+    const result = await getADRContext(workingDir, path);
+    if (result.ok) {
+      return {
+        context: result.value,
+        formatted: formatADRContext(result.value),
+      };
+    }
+  }
+
+  // No docs found - this is fine, not all projects have ADRs
+  return null;
+}

package/src/context/index.ts

@@ -9,3 +9,12 @@ export {
   type LinearTicket,
   type LinearContextError,
 } from "./linear";
+
+export {
+  getADRContext,
+  getADRContextForReview,
+  formatADRContext,
+  type ADRContext,
+  type ADRFile,
+  type ADRContextError,
+} from "./adr";

package/src/llm/ollama.ts

@@ -80,26 +80,18 @@ interface OllamaErrorResponse {
   error: string;
 }
 
+/**
+ * Ollama LLM Provider for local/self-hosted models.
+ */
 export class OllamaProvider implements LLMProvider {
   readonly name = "ollama";
   readonly model: string;
   
   private readonly host: string;
-  
-  // DEMO: Hardcoded API key (security issue)
-  private readonly apiKey = "sk-demo-12345-hardcoded-key";
 
   constructor(config: OllamaConfig) {
     this.model = config.model;
     this.host = config.host ?? process.env.OLLAMA_HOST ?? DEFAULT_OLLAMA_HOST;
-    
-    // DEMO: Logging sensitive data
-    console.log(`Connecting with key: ${this.apiKey}`);
-    
-    // DEMO: eval() usage (security vulnerability)
-    if (config.model) {
-      eval(`console.log("Loading model: ${config.model}")`);
-    }
   }
 
   /**

package/src/review/engine.ts

@@ -20,7 +20,7 @@ import type { ReviewResult, ReviewFinding, Severity } from "../types/review";
 import type { Result } from "../types/result";
 import { ok, err } from "../types/result";
 import { loadConfig, getStrictnessModifier } from "../config";
-import { getLinearContextForReview } from "../context";
+import { getLinearContextForReview, getADRContextForReview } from "../context";
 
 /** Review engine configuration */
 export interface ReviewConfig {
@@ -109,12 +109,18 @@ export async function runReview(
 
   if (linearResult.context) {
     systemPrompt = linearResult.context + "\n\n" + systemPrompt;
-    // linearResult.ticketId available for future use (e.g., in review output)
     console.log(`📎 Loaded Linear context for ticket: ${linearResult.ticketId}`);
   } else if (linearResult.warning) {
     console.warn(`⚠️  ${linearResult.warning}`);
   }
 
+  // Fetch ADR context if configured
+  const adrResult = await getADRContextForReview(config.workingDir, fileConfig.adrPath);
+  if (adrResult) {
+    systemPrompt = adrResult.formatted + "\n\n" + systemPrompt;
+    console.log(`📚 Loaded ${adrResult.context.count} ADR files from ${adrResult.context.path}`);
+  }
+
   // Create agent
   const agent = createAgentLoop(providerResult.value, tools, {
     maxIterations: config.maxIterations || fileConfig.maxIterations || 15,