@unifiedmemory/cli 1.3.15 → 1.3.16

package/.env.example

@@ -17,17 +17,17 @@
 # ============================================
 # Clerk OAuth Configuration
 # ============================================
-# Production defaults are included in the CLI
-# Only override these if using a custom Clerk instance
-# CLERK_CLIENT_ID=custom_clerk_client_id
-# CLERK_DOMAIN=custom-app.clerk.accounts.dev
+# Production defaults are included in the CLI.
+# For local development, uncomment and use dev Clerk values:
+# CLERK_CLIENT_ID=nULlnomaKB9rRGP2
+# CLERK_DOMAIN=clear-caiman-45.clerk.accounts.dev
 
 # ============================================
 # API Configuration
 # ============================================
-# Production default is included in the CLI
-# Only override this for local testing or custom deployments
-# API_ENDPOINT=https://custom-api-gateway.zuplo.dev
+# Production default is included in the CLI.
+# For local development, uncomment and use dev gateway:
+# API_ENDPOINT=https://rose-asp-main-1c0b114.d2.zuplo.dev
 
 # ============================================
 # OAuth Flow Configuration

package/lib/config.js

@@ -10,12 +10,14 @@ dotenvConfig({ path: join(__dirname, '..', '.env') });
 
 export const config = {
   // Clerk OAuth configuration (production defaults, can be overridden via env vars)
-  clerkClientId: process.env.CLERK_CLIENT_ID || 'nULlnomaKB9rRGP2',
+  // TODO: Replace clerkClientId with production Clerk OAuth client ID once created
+  clerkClientId: process.env.CLERK_CLIENT_ID || 'bNpTWw0hP3V7ueqN',
   clerkClientSecret: process.env.CLERK_CLIENT_SECRET, // Optional for PKCE flow
-  clerkDomain: process.env.CLERK_DOMAIN || 'clear-caiman-45.clerk.accounts.dev',
+  clerkDomain: process.env.CLERK_DOMAIN || 'clerk.unifiedmemory.ai',
 
   // API configuration (production default, can be overridden via env var)
-  apiEndpoint: process.env.API_ENDPOINT || 'https://rose-asp-main-1c0b114.d2.zuplo.dev',
+  // TODO: Replace with production Zuplo gateway URL once deployed
+  apiEndpoint: process.env.API_ENDPOINT || 'https://api.unifiedmemory.ai',
 
   // OAuth flow configuration (localhost defaults for callback server)
   redirectUri: process.env.REDIRECT_URI || 'http://localhost:3333/callback',

package/lib/mcp-proxy.js

@@ -2,7 +2,9 @@
  * MCP Proxy Client - Forwards MCP requests to the gateway HTTP endpoint
  */
 
-const GATEWAY_MCP_URL = 'https://rose-asp-main-1c0b114.d2.zuplo.dev/mcp';
+import { config } from './config.js';
+
+const GATEWAY_MCP_URL = `${config.apiEndpoint}/mcp`;
 
 /**
  * Transform tool schema to hide context parameters (org, proj, user)

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@unifiedmemory/cli",
-  "version": "1.3.15",
+  "version": "1.3.16",
   "description": "UnifiedMemory CLI - AI code assistant integration",
   "main": "index.js",
   "type": "module",

package/tests/unit/config.test.js

@@ -8,6 +8,11 @@
 
 import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
 
+// Mock dotenv so the local .env file doesn't interfere with unit tests
+vi.mock('dotenv', () => ({
+  config: vi.fn()
+}));
+
 describe('config', () => {
   // Store original env vars
   const originalEnv = { ...process.env };
@@ -35,9 +40,9 @@ describe('config', () => {
       // Fresh import to get defaults
       const { config } = await import('../../lib/config.js');
 
-      expect(config.clerkClientId).toBe('nULlnomaKB9rRGP2');
-      expect(config.clerkDomain).toBe('clear-caiman-45.clerk.accounts.dev');
-      expect(config.apiEndpoint).toBe('https://rose-asp-main-1c0b114.d2.zuplo.dev');
+      expect(config.clerkClientId).toBe('bNpTWw0hP3V7ueqN');
+      expect(config.clerkDomain).toBe('clerk.unifiedmemory.ai');
+      expect(config.apiEndpoint).toBe('https://api.unifiedmemory.ai');
       expect(config.redirectUri).toBe('http://localhost:3333/callback');
       expect(config.port).toBe(3333);
     });
@@ -123,7 +128,7 @@ describe('config', () => {
       const { config, validateConfig } = await import('../../lib/config.js');
 
       // Empty string is falsy, so it falls back to default
-      expect(config.clerkDomain).toBe('clear-caiman-45.clerk.accounts.dev');
+      expect(config.clerkDomain).toBe('clerk.unifiedmemory.ai');
       expect(validateConfig()).toBe(true);
     });
 
@@ -142,7 +147,7 @@ describe('config', () => {
       const { config, validateConfig } = await import('../../lib/config.js');
 
       // Empty string is falsy, so it falls back to default
-      expect(config.clerkClientId).toBe('nULlnomaKB9rRGP2');
+      expect(config.clerkClientId).toBe('bNpTWw0hP3V7ueqN');
       expect(validateConfig()).toBe(true);
     });
 

package/tests/unit/mcp-proxy.test.js

@@ -8,11 +8,17 @@
  * are tested indirectly through the exported functions.
  */
 
-import { describe, it, expect, vi, beforeAll, afterAll, afterEach } from 'vitest';
+import { describe, it, expect, vi, beforeAll, beforeEach, afterAll, afterEach } from 'vitest';
 import { setupServer } from 'msw/node';
 import { http, HttpResponse } from 'msw';
 
-const GATEWAY_MCP_URL = 'https://rose-asp-main-1c0b114.d2.zuplo.dev/mcp';
+// Mock dotenv so the local .env file doesn't interfere with unit tests
+vi.mock('dotenv', () => ({
+  config: vi.fn()
+}));
+
+// Must match the fallback default in lib/config.js (used when no .env is present, e.g. CI)
+const GATEWAY_MCP_URL = 'https://api.unifiedmemory.ai/mcp';
 
 // Sample tool with pathParams and headers that should be transformed
 const sampleToolWithContext = {
@@ -59,6 +65,11 @@ const sampleToolNoContext = {
 const server = setupServer();
 
 beforeAll(() => server.listen({ onUnhandledRequest: 'bypass' }));
+beforeEach(() => {
+  vi.resetModules();
+  // Clear API_ENDPOINT so config.js uses the hardcoded default matching GATEWAY_MCP_URL above
+  delete process.env.API_ENDPOINT;
+});
 afterEach(() => server.resetHandlers());
 afterAll(() => server.close());