bluera-knowledge 0.14.0 → 0.14.1

package/scripts/validate-npm-release.sh

@@ -1,314 +0,0 @@
-#!/usr/bin/env bash
-#
-# validate-npm-release.sh
-#
-# Post-release validation script for bluera-knowledge npm module.
-# Installs the latest version from npm and exercises all CLI commands.
-#
-# Usage:
-#   ./scripts/validate-npm-release.sh
-#
-# Output:
-#   Logs written to: <repo>/logs/validation/npm-validation-YYYYMMDD-HHMMSS.log
-#   Exit code: 0 if all tests pass, 1 if any fail
-#
-
-set -euo pipefail
-
-# Configuration
-SCRIPT_DIR="$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)"
-REPO_ROOT="$(cd "$SCRIPT_DIR/.." && pwd)"
-RESULTS_DIR="$REPO_ROOT/logs/validation"
-TIMESTAMP="$(date +%Y%m%d-%H%M%S)"
-LOG_FILE="$RESULTS_DIR/npm-validation-$TIMESTAMP.log"
-
-# Test configuration
-TEST_STORE="npm-validation-test-$TIMESTAMP"
-TEST_FOLDER="$(mktemp -d)"
-DATA_DIR="$(mktemp -d)"
-
-# Counters
-TESTS_RUN=0
-TESTS_PASSED=0
-TESTS_FAILED=0
-
-# Setup
-mkdir -p "$RESULTS_DIR"
-echo "Test content for validation" > "$TEST_FOLDER/test.txt"
-echo "Another test file" > "$TEST_FOLDER/test2.md"
-
-# Logging functions
-log() {
-    echo "[$(date +%H:%M:%S)] $*" | tee -a "$LOG_FILE"
-}
-
-log_header() {
-    echo "" | tee -a "$LOG_FILE"
-    echo "========================================" | tee -a "$LOG_FILE"
-    echo "$*" | tee -a "$LOG_FILE"
-    echo "========================================" | tee -a "$LOG_FILE"
-}
-
-pass() {
-    TESTS_RUN=$((TESTS_RUN + 1))
-    TESTS_PASSED=$((TESTS_PASSED + 1))
-    log "✓ PASS: $*"
-}
-
-fail() {
-    TESTS_RUN=$((TESTS_RUN + 1))
-    TESTS_FAILED=$((TESTS_FAILED + 1))
-    log "✗ FAIL: $*"
-}
-
-# Run a command and check exit code
-run_test() {
-    local name="$1"
-    shift
-    local cmd="$*"
-
-    log "Running: $cmd"
-    if eval "$cmd" 2>&1 | tee -a "$LOG_FILE"; then
-        pass "$name"
-        return 0
-    else
-        fail "$name (exit code: ${PIPESTATUS[0]})"
-        return 1
-    fi
-}
-
-# Run a command and check output contains expected string
-run_test_contains() {
-    local name="$1"
-    local expected="$2"
-    shift 2
-    local cmd="$*"
-
-    log "Running: $cmd"
-    local output
-    # Capture output while also showing it on terminal via tee
-    if output=$(eval "$cmd" 2>&1 | tee -a "$LOG_FILE"); then
-        if echo "$output" | grep -q "$expected"; then
-            pass "$name"
-            return 0
-        else
-            fail "$name (output missing: $expected)"
-            return 1
-        fi
-    else
-        fail "$name (command failed)"
-        return 1
-    fi
-}
-
-# Cleanup function
-cleanup() {
-    log_header "Cleanup"
-
-    # Delete test store if it exists
-    log "Deleting test store: $TEST_STORE"
-    bluera-knowledge store delete "$TEST_STORE" --force -d "$DATA_DIR" 2>/dev/null || true
-
-    # Remove test folder
-    log "Removing test folder: $TEST_FOLDER"
-    rm -rf "$TEST_FOLDER"
-
-    # Remove data directory
-    log "Removing data directory: $DATA_DIR"
-    rm -rf "$DATA_DIR"
-
-    log "Cleanup complete"
-}
-
-# Set trap for cleanup on exit
-trap cleanup EXIT
-
-# Start validation
-log_header "NPM Module Validation - $TIMESTAMP"
-log "Log file: $LOG_FILE"
-log "Test store: $TEST_STORE"
-log "Test folder: $TEST_FOLDER"
-log "Data directory: $DATA_DIR"
-
-# Get expected version from package.json
-EXPECTED_VERSION=$(node -p "require('$REPO_ROOT/package.json').version")
-log "Expected version: $EXPECTED_VERSION"
-
-# Install latest from npm
-log_header "Installing Latest from npm"
-log "Installing bluera-knowledge@latest globally..."
-if npm install -g bluera-knowledge@latest >> "$LOG_FILE" 2>&1; then
-    pass "npm install -g bluera-knowledge@latest"
-else
-    fail "npm install -g bluera-knowledge@latest"
-    log "ERROR: Failed to install package. Aborting."
-    exit 1
-fi
-
-# Verify installation
-log_header "Verifying Installation"
-
-# Capture and display the installed version
-INSTALLED_VERSION=$(bluera-knowledge --version 2>&1 | head -1)
-log "Installed version: $INSTALLED_VERSION"
-VERSION_TEXT="Installed: bluera-knowledge@$INSTALLED_VERSION"
-VERSION_LEN=${#VERSION_TEXT}
-PADDING=$((VERSION_LEN + 4))
-echo ""
-printf "  ╔"; printf '═%.0s' $(seq 1 $PADDING); printf "╗\n"
-printf "  ║  %s  ║\n" "$VERSION_TEXT"
-printf "  ╚"; printf '═%.0s' $(seq 1 $PADDING); printf "╝\n"
-echo ""
-
-if [ "$INSTALLED_VERSION" != "$EXPECTED_VERSION" ]; then
-    log "WARNING: Version mismatch! Expected $EXPECTED_VERSION but got $INSTALLED_VERSION"
-    log "This may indicate npm cache issues. Try: npm cache clean --force"
-fi
-
-run_test_contains "bluera-knowledge --version" "$INSTALLED_VERSION" "bluera-knowledge --version"
-
-run_test_contains "bluera-knowledge --help" "CLI tool for managing knowledge stores" "bluera-knowledge --help"
-
-# Test stores list (should work even if empty)
-log_header "Testing Store Operations"
-
-run_test "bluera-knowledge stores (initial list)" "bluera-knowledge stores -d '$DATA_DIR' -f json"
-
-# Create a store via add-folder
-log_header "Testing add-folder"
-
-run_test "bluera-knowledge add-folder" "bluera-knowledge add-folder '$TEST_FOLDER' --name '$TEST_STORE' -d '$DATA_DIR'"
-
-# Verify store was created
-run_test_contains "Store appears in list" "$TEST_STORE" "bluera-knowledge stores -d '$DATA_DIR'"
-
-# Test store info
-log_header "Testing store info"
-
-run_test_contains "bluera-knowledge store info" "$TEST_STORE" "bluera-knowledge store info '$TEST_STORE' -d '$DATA_DIR'"
-
-# Test search (may return no results, but should not error)
-log_header "Testing search"
-
-run_test "bluera-knowledge search" "bluera-knowledge search 'test content' --stores '$TEST_STORE' -d '$DATA_DIR' -f json"
-
-# Test index command (re-index)
-log_header "Testing index"
-
-run_test "bluera-knowledge index" "bluera-knowledge index '$TEST_STORE' -d '$DATA_DIR'"
-
-# Test search again after re-indexing
-run_test "bluera-knowledge search (after reindex)" "bluera-knowledge search 'validation' --stores '$TEST_STORE' -d '$DATA_DIR' -f json"
-
-# Test store delete
-log_header "Testing store delete"
-
-run_test "bluera-knowledge store delete" "bluera-knowledge store delete '$TEST_STORE' --force -d '$DATA_DIR'"
-
-# Verify store was deleted
-log "Verifying store was deleted..."
-if bluera-knowledge stores -d "$DATA_DIR" -f json 2>&1 | grep -q "$TEST_STORE"; then
-    fail "Store still exists after delete"
-else
-    pass "Store successfully deleted"
-fi
-
-# Test suggest command
-log_header "Testing suggest"
-
-# Create a minimal package.json for suggest to find
-echo '{"dependencies": {"lodash": "^4.0.0"}}' > "$TEST_FOLDER/package.json"
-run_test "bluera-knowledge suggest" "bluera-knowledge suggest -p '$TEST_FOLDER' -d '$DATA_DIR'"
-
-# Test sync command (should work with no definitions - returns empty result)
-log_header "Testing sync"
-
-run_test_contains "bluera-knowledge sync (no definitions)" "Sync completed" "bluera-knowledge sync -p '$TEST_FOLDER' -d '$DATA_DIR'"
-
-# Test sync with a definitions file
-mkdir -p "$TEST_FOLDER/.bluera/bluera-knowledge"
-cat > "$TEST_FOLDER/.bluera/bluera-knowledge/stores.config.json" << EOF
-{
-  "version": 1,
-  "stores": [
-    {
-      "name": "sync-test-store",
-      "type": "file",
-      "path": "."
-    }
-  ]
-}
-EOF
-run_test "bluera-knowledge sync (with definitions)" "bluera-knowledge sync -p '$TEST_FOLDER' -d '$DATA_DIR'"
-
-# Verify sync created the store
-run_test_contains "Sync created store" "sync-test-store" "bluera-knowledge stores -d '$DATA_DIR'"
-
-# Clean up sync test store
-bluera-knowledge store delete "sync-test-store" --force -d "$DATA_DIR" 2>/dev/null || true
-
-# Test serve command (start, test, stop)
-log_header "Testing serve"
-
-SERVE_PORT=19876
-SERVE_PID=""
-
-# Start server in background
-log "Starting serve on port $SERVE_PORT..."
-bluera-knowledge serve --port $SERVE_PORT -d "$DATA_DIR" &
-SERVE_PID=$!
-
-# Give it time to start
-sleep 2
-
-# Check if server is running
-if kill -0 $SERVE_PID 2>/dev/null; then
-    log "Server started with PID $SERVE_PID"
-
-    # Test health endpoint
-    if curl -s "http://localhost:$SERVE_PORT/health" | grep -q "ok"; then
-        pass "bluera-knowledge serve (health endpoint)"
-    else
-        fail "bluera-knowledge serve (health endpoint not responding)"
-    fi
-
-    # Stop server
-    kill $SERVE_PID 2>/dev/null || true
-    wait $SERVE_PID 2>/dev/null || true
-    log "Server stopped"
-else
-    fail "bluera-knowledge serve (failed to start)"
-fi
-
-# Test mcp command (just verify it starts and outputs JSON-RPC)
-log_header "Testing mcp"
-
-MCP_OUTPUT=$(timeout 2 bluera-knowledge mcp -d "$DATA_DIR" 2>&1 || true)
-if echo "$MCP_OUTPUT" | grep -qE "(jsonrpc|ready|listening|MCP)" 2>/dev/null || [ $? -eq 124 ]; then
-    # Timeout (124) is expected - MCP keeps running until killed
-    pass "bluera-knowledge mcp (starts without error)"
-else
-    # Even if it times out or produces no output, as long as it didn't crash
-    if [ -z "$MCP_OUTPUT" ]; then
-        pass "bluera-knowledge mcp (starts without error)"
-    else
-        log "MCP output: $MCP_OUTPUT"
-        fail "bluera-knowledge mcp (unexpected output)"
-    fi
-fi
-
-# Summary
-log_header "Validation Summary"
-log "Tests run:    $TESTS_RUN"
-log "Tests passed: $TESTS_PASSED"
-log "Tests failed: $TESTS_FAILED"
-log ""
-log "Log file: $LOG_FILE"
-
-if [ "$TESTS_FAILED" -gt 0 ]; then
-    log "VALIDATION FAILED"
-    exit 1
-else
-    log "VALIDATION PASSED"
-    exit 0
-fi

package/src/analysis/adapter-registry.test.ts

@@ -1,211 +0,0 @@
-import { describe, it, expect, beforeEach } from 'vitest';
-import { AdapterRegistry } from './adapter-registry.js';
-import type { LanguageAdapter } from './language-adapter.js';
-import type { CodeNode, ImportInfo } from './ast-parser.js';
-import type { GraphEdge } from './code-graph.js';
-
-/**
- * Mock adapter for testing
- */
-function createMockAdapter(overrides: Partial<LanguageAdapter> = {}): LanguageAdapter {
-  return {
-    languageId: 'test-lang',
-    extensions: ['.test'],
-    displayName: 'Test Language',
-    parse: (): CodeNode[] => [],
-    extractImports: (): ImportInfo[] => [],
-    ...overrides,
-  };
-}
-
-describe('AdapterRegistry', () => {
-  beforeEach(() => {
-    // Reset singleton for each test
-    AdapterRegistry.resetInstance();
-  });
-
-  describe('getInstance', () => {
-    it('should return the same instance on multiple calls', () => {
-      const instance1 = AdapterRegistry.getInstance();
-      const instance2 = AdapterRegistry.getInstance();
-      expect(instance1).toBe(instance2);
-    });
-  });
-
-  describe('register', () => {
-    it('should register an adapter', () => {
-      const registry = AdapterRegistry.getInstance();
-      const adapter = createMockAdapter();
-
-      registry.register(adapter);
-
-      expect(registry.getByLanguageId('test-lang')).toBe(adapter);
-    });
-
-    it('should be idempotent when registering same languageId', () => {
-      const registry = AdapterRegistry.getInstance();
-      const adapter1 = createMockAdapter();
-      const adapter2 = createMockAdapter();
-
-      registry.register(adapter1);
-      registry.register(adapter2); // Should not throw, just skip
-
-      // First adapter should still be registered
-      expect(registry.getByLanguageId('test-lang')).toBe(adapter1);
-      expect(registry.getAllAdapters()).toHaveLength(1);
-    });
-
-    it('should throw if extension is already registered by another adapter', () => {
-      const registry = AdapterRegistry.getInstance();
-      const adapter1 = createMockAdapter({ languageId: 'lang1', extensions: ['.test'] });
-      const adapter2 = createMockAdapter({ languageId: 'lang2', extensions: ['.test'] });
-
-      registry.register(adapter1);
-
-      expect(() => registry.register(adapter2)).toThrow(
-        'Extension ".test" is already registered by adapter "lang1"'
-      );
-    });
-  });
-
-  describe('getByExtension', () => {
-    it('should return adapter by extension', () => {
-      const registry = AdapterRegistry.getInstance();
-      const adapter = createMockAdapter({ extensions: ['.zil', '.mud'] });
-
-      registry.register(adapter);
-
-      expect(registry.getByExtension('.zil')).toBe(adapter);
-      expect(registry.getByExtension('.mud')).toBe(adapter);
-    });
-
-    it('should return undefined for unregistered extension', () => {
-      const registry = AdapterRegistry.getInstance();
-
-      expect(registry.getByExtension('.unknown')).toBeUndefined();
-    });
-
-    it('should normalize extension with leading dot', () => {
-      const registry = AdapterRegistry.getInstance();
-      const adapter = createMockAdapter({ extensions: ['.zil'] });
-
-      registry.register(adapter);
-
-      // Both with and without dot should work
-      expect(registry.getByExtension('.zil')).toBe(adapter);
-      expect(registry.getByExtension('zil')).toBe(adapter);
-    });
-  });
-
-  describe('getByLanguageId', () => {
-    it('should return adapter by language ID', () => {
-      const registry = AdapterRegistry.getInstance();
-      const adapter = createMockAdapter({ languageId: 'zil' });
-
-      registry.register(adapter);
-
-      expect(registry.getByLanguageId('zil')).toBe(adapter);
-    });
-
-    it('should return undefined for unregistered language ID', () => {
-      const registry = AdapterRegistry.getInstance();
-
-      expect(registry.getByLanguageId('unknown')).toBeUndefined();
-    });
-  });
-
-  describe('getAllAdapters', () => {
-    it('should return empty array when no adapters registered', () => {
-      const registry = AdapterRegistry.getInstance();
-
-      expect(registry.getAllAdapters()).toEqual([]);
-    });
-
-    it('should return all registered adapters', () => {
-      const registry = AdapterRegistry.getInstance();
-      const adapter1 = createMockAdapter({ languageId: 'lang1', extensions: ['.l1'] });
-      const adapter2 = createMockAdapter({ languageId: 'lang2', extensions: ['.l2'] });
-
-      registry.register(adapter1);
-      registry.register(adapter2);
-
-      const adapters = registry.getAllAdapters();
-      expect(adapters).toHaveLength(2);
-      expect(adapters).toContain(adapter1);
-      expect(adapters).toContain(adapter2);
-    });
-  });
-
-  describe('unregister', () => {
-    it('should unregister an adapter by language ID', () => {
-      const registry = AdapterRegistry.getInstance();
-      const adapter = createMockAdapter({ languageId: 'zil', extensions: ['.zil'] });
-
-      registry.register(adapter);
-      expect(registry.getByLanguageId('zil')).toBe(adapter);
-
-      const result = registry.unregister('zil');
-      expect(result).toBe(true);
-      expect(registry.getByLanguageId('zil')).toBeUndefined();
-      expect(registry.getByExtension('.zil')).toBeUndefined();
-    });
-
-    it('should return false when unregistering non-existent adapter', () => {
-      const registry = AdapterRegistry.getInstance();
-
-      const result = registry.unregister('nonexistent');
-      expect(result).toBe(false);
-    });
-  });
-
-  describe('hasExtension', () => {
-    it('should return true for registered extension', () => {
-      const registry = AdapterRegistry.getInstance();
-      const adapter = createMockAdapter({ extensions: ['.zil'] });
-
-      registry.register(adapter);
-
-      expect(registry.hasExtension('.zil')).toBe(true);
-    });
-
-    it('should return false for unregistered extension', () => {
-      const registry = AdapterRegistry.getInstance();
-
-      expect(registry.hasExtension('.unknown')).toBe(false);
-    });
-  });
-
-  describe('adapter with optional methods', () => {
-    it('should support adapters with chunk method', () => {
-      const registry = AdapterRegistry.getInstance();
-      const adapter = createMockAdapter({
-        chunk: () => [{ content: 'test', startLine: 1, endLine: 10 }],
-      });
-
-      registry.register(adapter);
-
-      const retrieved = registry.getByLanguageId('test-lang');
-      expect(retrieved?.chunk).toBeDefined();
-      expect(retrieved?.chunk?.('', '')).toEqual([{ content: 'test', startLine: 1, endLine: 10 }]);
-    });
-
-    it('should support adapters with analyzeCallRelationships method', () => {
-      const registry = AdapterRegistry.getInstance();
-      const mockEdge: GraphEdge = {
-        from: 'a',
-        to: 'b',
-        type: 'calls',
-        confidence: 1.0,
-      };
-      const adapter = createMockAdapter({
-        analyzeCallRelationships: () => [mockEdge],
-      });
-
-      registry.register(adapter);
-
-      const retrieved = registry.getByLanguageId('test-lang');
-      expect(retrieved?.analyzeCallRelationships).toBeDefined();
-      expect(retrieved?.analyzeCallRelationships?.('', '')).toEqual([mockEdge]);
-    });
-  });
-});

package/src/analysis/adapter-registry.ts

@@ -1,155 +0,0 @@
-/**
- * Adapter Registry
- *
- * Singleton registry for language adapters. Provides lookup by extension
- * or language ID.
- *
- * @example
- * ```typescript
- * // Register an adapter
- * const registry = AdapterRegistry.getInstance();
- * registry.register(zilAdapter);
- *
- * // Look up by extension
- * const adapter = registry.getByExtension('.zil');
- * if (adapter) {
- *   const nodes = adapter.parse(content, filePath);
- * }
- * ```
- */
-
-import type { LanguageAdapter } from './language-adapter.js';
-
-export class AdapterRegistry {
-  private static instance: AdapterRegistry | undefined;
-
-  /** Map from languageId to adapter */
-  private readonly adaptersByLanguageId = new Map<string, LanguageAdapter>();
-
-  /** Map from extension to adapter */
-  private readonly adaptersByExtension = new Map<string, LanguageAdapter>();
-
-  private constructor() {
-    // Private constructor for singleton
-  }
-
-  /**
-   * Get the singleton instance of the registry.
-   */
-  static getInstance(): AdapterRegistry {
-    AdapterRegistry.instance ??= new AdapterRegistry();
-    return AdapterRegistry.instance;
-  }
-
-  /**
-   * Reset the singleton instance (for testing).
-   */
-  static resetInstance(): void {
-    AdapterRegistry.instance = undefined;
-  }
-
-  /**
-   * Register a language adapter.
-   *
-   * @param adapter - The adapter to register
-   * @throws If a different adapter with the same extension is already registered
-   */
-  register(adapter: LanguageAdapter): void {
-    // Skip if already registered with same languageId (idempotent)
-    if (this.adaptersByLanguageId.has(adapter.languageId)) {
-      return;
-    }
-
-    // Check for extension conflicts with other adapters
-    for (const ext of adapter.extensions) {
-      const normalizedExt = this.normalizeExtension(ext);
-      const existingAdapter = this.adaptersByExtension.get(normalizedExt);
-      if (existingAdapter !== undefined) {
-        throw new Error(
-          `Extension "${normalizedExt}" is already registered by adapter "${existingAdapter.languageId}"`
-        );
-      }
-    }
-
-    // Register by languageId
-    this.adaptersByLanguageId.set(adapter.languageId, adapter);
-
-    // Register by each extension
-    for (const ext of adapter.extensions) {
-      const normalizedExt = this.normalizeExtension(ext);
-      this.adaptersByExtension.set(normalizedExt, adapter);
-    }
-  }
-
-  /**
-   * Unregister a language adapter by its language ID.
-   *
-   * @param languageId - The language ID to unregister
-   * @returns true if the adapter was found and removed, false otherwise
-   */
-  unregister(languageId: string): boolean {
-    const adapter = this.adaptersByLanguageId.get(languageId);
-    if (adapter === undefined) {
-      return false;
-    }
-
-    // Remove from languageId map
-    this.adaptersByLanguageId.delete(languageId);
-
-    // Remove from extension map
-    for (const ext of adapter.extensions) {
-      const normalizedExt = this.normalizeExtension(ext);
-      this.adaptersByExtension.delete(normalizedExt);
-    }
-
-    return true;
-  }
-
-  /**
-   * Get an adapter by file extension.
-   *
-   * @param ext - File extension (with or without leading dot)
-   * @returns The adapter if found, undefined otherwise
-   */
-  getByExtension(ext: string): LanguageAdapter | undefined {
-    const normalizedExt = this.normalizeExtension(ext);
-    return this.adaptersByExtension.get(normalizedExt);
-  }
-
-  /**
-   * Get an adapter by language ID.
-   *
-   * @param languageId - The unique language identifier
-   * @returns The adapter if found, undefined otherwise
-   */
-  getByLanguageId(languageId: string): LanguageAdapter | undefined {
-    return this.adaptersByLanguageId.get(languageId);
-  }
-
-  /**
-   * Get all registered adapters.
-   *
-   * @returns Array of all registered adapters
-   */
-  getAllAdapters(): LanguageAdapter[] {
-    return Array.from(this.adaptersByLanguageId.values());
-  }
-
-  /**
-   * Check if an extension is registered.
-   *
-   * @param ext - File extension (with or without leading dot)
-   * @returns true if the extension is registered
-   */
-  hasExtension(ext: string): boolean {
-    const normalizedExt = this.normalizeExtension(ext);
-    return this.adaptersByExtension.has(normalizedExt);
-  }
-
-  /**
-   * Normalize extension to always have a leading dot.
-   */
-  private normalizeExtension(ext: string): string {
-    return ext.startsWith('.') ? ext : `.${ext}`;
-  }
-}