@unifiedmemory/cli 1.3.14 → 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/lib/memory-instructions.js

@@ -1,82 +1,76 @@
 // Unique marker to detect if instructions already present
-export const MEMORY_INSTRUCTIONS_MARKER = '<!-- UNIFIEDMEMORY_INSTRUCTIONS_V2 -->';
+export const MEMORY_INSTRUCTIONS_MARKER = '<!-- UNIFIEDMEMORY_INSTRUCTIONS_V3 -->';
 
 export function getMemoryInstructions() {
   return `${MEMORY_INSTRUCTIONS_MARKER}
 
-# UnifiedMemory Integration (REQUIRED)
+# STOP - Read Before ANY Code Task
 
-This project uses UnifiedMemory for persistent knowledge management. The \`unifiedmemory\` MCP server is configured and MUST be used for all relevant queries.
+**MANDATORY**: Call \`search_notes\` BEFORE writing any code. Always. No exceptions.
 
-## Available Tools
+## Pre-Implementation Checklist
 
-The \`unifiedmemory\` MCP server provides these tools (call them like any MCP tool):
+Before ANY coding task, you MUST:
+- [ ] Call \`search_notes\` with relevant keywords
+- [ ] Review results for existing patterns/decisions
+- [ ] Mention what you found (or didn't) in your response
+- [ ] THEN start implementation
 
-| Tool | Purpose | Required Parameters |
-|------|---------|---------------------|
-| \`search_notes\` | **Primary tool** - Search project knowledge base | \`query\` (search text) |
-| \`create_note\` | Record decisions, plans, or completions | \`summary_text\`, \`topic\` |
-| \`list_notes\` | List all notes in current project | None |
-| \`get_note\` | Retrieve a specific note by ID | \`note_id\` |
-| \`list_topics\` | List available topics/categories | None |
-| \`create_topic\` | Create a new topic for organization | \`name\` |
-| \`list_projects\` | List available projects | None |
+---
 
-All tools automatically use the correct project context - no additional configuration needed.
+## WRONG
 
-## MANDATORY: Search Before Implementing
+\`\`\`
+User: "Create a calculator app"
+Assistant: [immediately writes code]  ← NEVER DO THIS
+\`\`\`
 
-**REQUIREMENT**: Before responding to ANY coding task, feature request, or implementation question, you MUST:
+## CORRECT
 
-1. Call the \`search_notes\` tool with the user's query or relevant keywords
-2. Review returned results for prior decisions, patterns, or context
-3. Incorporate relevant findings into your response
-4. Mention what context you found (or didn't find) in your response
+\`\`\`
+User: "Create a calculator app"
+Assistant: [calls search_notes first]
+Assistant: "No relevant patterns found in UnifiedMemory. Proceeding..."
+Assistant: [writes code]
+\`\`\`
 
-### When to Search
+---
 
-You MUST search before:
-- Implementing any feature (search for existing patterns)
-- Fixing any bug (search for related issues)
-- Answering architecture questions (search for design decisions)
-- Modifying any component (search for existing conventions)
-- ANY coding task, no exceptions
+## When to Search (ALWAYS)
 
-### Example Workflow
+- Before implementing ANY feature ← search first
+- Before fixing ANY bug ← search first
+- Before answering architecture questions ← search first
+- Before modifying ANY component ← search first
+- For "simple" tasks ← search first (context always helps)
 
-\`\`\`
-User: "Add authentication to the API"
-
-Step 1: Call search_notes with query "authentication API security"
-Step 2: Review results - found: "Decision: Use JWT with refresh tokens"
-Step 3: Implement following the established pattern
-Step 4: Respond: "Based on prior decisions in UnifiedMemory, I'll use JWT..."
-\`\`\`
+---
 
-## Do NOT Skip These Tools
+## MCP Tool Reference
 
-- ❌ NEVER start implementing without calling \`search_notes\` first
-- ❌ NEVER assume patterns without checking the knowledge base
-- ❌ NEVER skip search for "simple" tasks - context always helps
-- ✅ ALWAYS search even if you think you know the answer
-- ✅ ALWAYS mention search results (or lack thereof) in your response
-- ✅ ALWAYS use these tools when they can provide relevant context
+| Tool | Purpose | Parameters |
+|------|---------|------------|
+| \`search_notes\` | **Search knowledge base** | \`body: {q: "search text"}\` |
+| \`create_note\` | Record decisions/completions | \`body: {summary_text: "...", topic: "..."}\` |
+| \`list_notes\` | Browse all notes | None |
+| \`get_note\` | Get note by ID | \`body: {note_id: "..."}\` |
+| \`list_topics\` | List categories | None |
+| \`create_topic\` | Create category | \`body: {name: "..."}\` |
 
-## Recording Knowledge with create_note
+Project context is automatic - no extra config needed.
 
-After completing significant work, use \`create_note\` to record:
-- Implementation decisions and rationale
-- Completed features or fixes
-- Architectural patterns established
-- Gotchas or lessons learned
+---
 
-Keep notes concise (under 1000 characters) and focus on the "why" not the "what".
+## After Completing Work
 
-## Additional Tools
+Use \`create_note\` to record:
+- Decisions and rationale (the "why")
+- Patterns established
+- Gotchas or lessons learned
 
-Use \`list_topics\` to understand project organization, \`list_notes\` to browse available knowledge, and \`get_note\` to retrieve full details of relevant notes found via search.
+Keep notes under 1000 chars. Focus on "why" not "what".
 
 ---
-*UnifiedMemory CLI - Knowledge that persists*
+*UnifiedMemory - \`search_notes\` FIRST, code SECOND*
 `;
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@unifiedmemory/cli",
-  "version": "1.3.14",
+  "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());