bluera-knowledge 0.11.5 → 0.11.7

package/src/plugin/commands.ts

@@ -4,6 +4,17 @@ import { DependencyUsageAnalyzer } from '../analysis/dependency-usage-analyzer.j
 import { RepoUrlResolver } from '../analysis/repo-url-resolver.js';
 import { createServices } from '../services/index.js';
 
+/**
+ * Options passed from CLI global options to plugin command handlers.
+ */
+export interface CommandOptions {
+  config?: string | undefined;
+  dataDir?: string | undefined;
+  projectRoot?: string | undefined;
+  format?: 'json' | 'table' | 'plain' | undefined;
+  quiet?: boolean | undefined;
+}
+
 export async function handleSearch(args: {
   query: string;
   stores?: string;
@@ -45,13 +56,19 @@ export async function handleSearch(args: {
   }
 }
 
-export async function handleAddRepo(args: {
-  url: string;
-  name?: string;
-  branch?: string;
-}): Promise<void> {
-  // PWD is set by Claude Code to user's project directory
-  const services = await createServices(undefined, undefined, process.env['PWD']);
+export async function handleAddRepo(
+  args: {
+    url: string;
+    name?: string;
+    branch?: string;
+  },
+  options: CommandOptions = {}
+): Promise<void> {
+  const services = await createServices(
+    options.config,
+    options.dataDir,
+    options.projectRoot ?? process.env['PWD']
+  );
   const storeName = args.name ?? extractRepoName(args.url);
 
   console.log(`Cloning ${args.url}...`);
@@ -84,9 +101,15 @@ export async function handleAddRepo(args: {
   }
 }
 
-export async function handleAddFolder(args: { path: string; name?: string }): Promise<void> {
-  // PWD is set by Claude Code to user's project directory
-  const services = await createServices(undefined, undefined, process.env['PWD']);
+export async function handleAddFolder(
+  args: { path: string; name?: string },
+  options: CommandOptions = {}
+): Promise<void> {
+  const services = await createServices(
+    options.config,
+    options.dataDir,
+    options.projectRoot ?? process.env['PWD']
+  );
   const { basename } = await import('node:path');
   const storeName = args.name ?? basename(args.path);
 
@@ -142,9 +165,12 @@ export async function handleIndex(args: { store: string }): Promise<void> {
   }
 }
 
-export async function handleStores(): Promise<void> {
-  // PWD is set by Claude Code to user's project directory
-  const services = await createServices(undefined, undefined, process.env['PWD']);
+export async function handleStores(options: CommandOptions = {}): Promise<void> {
+  const services = await createServices(
+    options.config,
+    options.dataDir,
+    options.projectRoot ?? process.env['PWD']
+  );
   const stores = await services.store.list();
 
   if (stores.length === 0) {
@@ -176,14 +202,13 @@ export async function handleStores(): Promise<void> {
   }
 }
 
-export async function handleSuggest(): Promise<void> {
-  // PWD is set by Claude Code to user's project directory
-  const projectRoot = process.env['PWD'] ?? process.cwd();
+export async function handleSuggest(options: CommandOptions = {}): Promise<void> {
+  const projectRoot = options.projectRoot ?? process.env['PWD'] ?? process.cwd();
 
   console.log('Analyzing project dependencies...\n');
 
   // Create analyzer instance
-  const services = await createServices(undefined, undefined, projectRoot);
+  const services = await createServices(options.config, options.dataDir, projectRoot);
   const analyzer = new DependencyUsageAnalyzer();
   const resolver = new RepoUrlResolver();
 

package/src/scripts/validate-npm-release.test.ts

@@ -0,0 +1,34 @@
+import { describe, it, expect } from 'vitest';
+import { readFileSync } from 'fs';
+import { join } from 'path';
+
+/**
+ * Tests for validate-npm-release.sh script.
+ * Ensures the script follows best practices for cross-platform compatibility.
+ */
+describe('validate-npm-release.sh', () => {
+  const scriptPath = join(process.cwd(), 'scripts/validate-npm-release.sh');
+  const scriptContent = readFileSync(scriptPath, 'utf-8');
+
+  it('does not use hardcoded /tmp paths', () => {
+    // Script should use mktemp -d for temporary directories, not hardcoded /tmp
+    // This ensures better cross-platform compatibility and avoids path collisions
+
+    // Check for hardcoded /tmp assignments (excluding comments)
+    const lines = scriptContent.split('\n').filter((line) => !line.trim().startsWith('#'));
+    const hasHardcodedTmp = lines.some((line) => /=["']?\/tmp\//.test(line));
+
+    expect(hasHardcodedTmp).toBe(false);
+  });
+
+  it('uses mktemp for temporary directories', () => {
+    // Should use mktemp -d for creating temporary directories
+    expect(scriptContent).toContain('mktemp -d');
+  });
+
+  it('cleans up temporary directories on exit', () => {
+    // Should have a trap to clean up on exit
+    expect(scriptContent).toContain('trap');
+    expect(scriptContent).toMatch(/rm -rf/);
+  });
+});

package/.mcp.json

@@ -1,13 +0,0 @@
-{
-  "mcpServers": {
-    "bluera-knowledge": {
-      "command": "node",
-      "args": ["./dist/mcp/server.js"],
-      "env": {
-        "PROJECT_ROOT": "${PWD}",
-        "DATA_DIR": ".bluera/bluera-knowledge/data",
-        "CONFIG_PATH": ".bluera/bluera-knowledge/config.json"
-      }
-    }
-  }
-}

package/src/mcp/mcp-config.test.ts

@@ -1,43 +0,0 @@
-import { describe, it, expect } from 'vitest';
-import { readFileSync } from 'fs';
-import { join } from 'path';
-
-/**
- * Tests to verify .mcp.json is correctly configured for project-level use.
- * The MCP server should work when running from the project directory,
- * not just when loaded as a Claude Code plugin.
- */
-describe('MCP Configuration (.mcp.json)', () => {
-  const configPath = join(process.cwd(), '.mcp.json');
-  const config = JSON.parse(readFileSync(configPath, 'utf-8'));
-
-  it('does not use CLAUDE_PLUGIN_ROOT (only available in plugin mode)', () => {
-    const serverConfig = config.mcpServers['bluera-knowledge'];
-    const argsString = JSON.stringify(serverConfig.args);
-
-    // CLAUDE_PLUGIN_ROOT is only set when running as a Claude Code plugin
-    // .mcp.json is for project-level config, so it should use relative paths
-    expect(argsString).not.toContain('CLAUDE_PLUGIN_ROOT');
-  });
-
-  it('uses relative path for server entry point', () => {
-    const serverConfig = config.mcpServers['bluera-knowledge'];
-
-    // Should use ./dist/mcp/server.js or similar relative path
-    expect(serverConfig.args).toContain('./dist/mcp/server.js');
-  });
-
-  it('sets PROJECT_ROOT environment variable', () => {
-    const serverConfig = config.mcpServers['bluera-knowledge'];
-
-    // MCP server requires PROJECT_ROOT to function
-    expect(serverConfig.env).toHaveProperty('PROJECT_ROOT');
-  });
-
-  it('uses PWD for PROJECT_ROOT', () => {
-    const serverConfig = config.mcpServers['bluera-knowledge'];
-
-    // PROJECT_ROOT should be set to PWD (current working directory)
-    expect(serverConfig.env['PROJECT_ROOT']).toBe('${PWD}');
-  });
-});