@yeseh/cortex-cli 0.6.6 → 0.6.8

package/src/tests/cli.integration.spec.ts

@@ -18,7 +18,7 @@ const diagnosticsEnabled = process.env.CORTEX_TEST_DIAGNOSTICS === '1';
 const formatCliFailure = (
     args: string[],
     result: CliResult,
-    extra?: Record<string, unknown>
+    extra?: Record<string, unknown>,
 ): string => {
     const extraLines = extra ? `\nextra: ${JSON.stringify(extra, null, 2)}` : '';
     return (
@@ -88,12 +88,19 @@ const runCortexCli = async (args: string[], options: CliOptions): Promise<CliRes
         // Preserve TERM if available (some help formatting can depend on it)
         if (process.env.TERM) env.TERM = process.env.TERM;
 
-        for (const [key, value] of Object.entries(options.env ?? {})) {
+        for (const [
+            key, value,
+        ] of Object.entries(options.env ?? {})) {
             if (typeof value === 'string') env[key] = value;
         }
 
         // Use Bun.spawn for cross-platform subprocess spawning
-        const proc = Bun.spawn(['bun', 'run', scriptPath, ...args], {
+        const proc = Bun.spawn([
+            'bun',
+            'run',
+            scriptPath,
+            ...args,
+        ], {
             cwd: options.cwd,
             env,
             stdin: options.stdin !== undefined ? 'pipe' : 'inherit',
@@ -109,7 +116,9 @@ const runCortexCli = async (args: string[], options: CliOptions): Promise<CliRes
             proc.stdin.end();
         }
 
-        const [stdout, stderr] = await Promise.all([
+        const [
+            stdout, stderr,
+        ] = await Promise.all([
             new Response(proc.stdout).text(),
             new Response(proc.stderr).text(),
         ]);
@@ -136,7 +145,8 @@ const runCortexCli = async (args: string[], options: CliOptions): Promise<CliRes
             resolvedConfigPath,
             resolvedStoreRoot,
         };
-    } catch (error) {
+    }
+    catch (error) {
         // Handle process errors
         const processError = error as { stdout?: Buffer; stderr?: Buffer; exitCode?: number };
         const stdoutTrimmed = processError.stdout?.toString().trim() ?? '';
@@ -207,7 +217,7 @@ const createMemoryFile = async (
         expiresAt?: Date;
         createdAt?: Date;
         updatedAt?: Date;
-    } = {}
+    } = {},
 ): Promise<void> => {
     const content = options.content ?? 'Test memory content.';
     const tags = options.tags ?? ['test'];
@@ -229,7 +239,7 @@ const createMemoryFile = async (
             expiresAt: options.expiresAt,
             citations: [],
         },
-        content
+        content,
     );
     if (!memoryResult.ok()) {
         throw new Error(`Failed to create memory: ${memoryResult.error.message}`);
@@ -237,7 +247,7 @@ const createMemoryFile = async (
 
     const adapter = new FilesystemStorageAdapter(
         new FilesystemConfigAdapter(join(storeRoot, 'config.yaml')),
-        { rootDirectory: storeRoot }
+        { rootDirectory: storeRoot },
     );
     const writeResult = await adapter.memories.save(memoryResult.value.path, memoryResult.value);
     if (!writeResult.ok()) {
@@ -270,7 +280,8 @@ const createCategory = async (categoryPath: string, storeRoot: string): Promise<
     const rootIndexPath = join(storeRoot, 'index.yaml');
     try {
         await fs.access(rootIndexPath);
-    } catch {
+    }
+    catch {
         await fs.writeFile(rootIndexPath, 'memories: []\nsubcategories: []', 'utf8');
     }
 };
@@ -284,14 +295,15 @@ const createCategoryIndex = async (
     storeRoot: string,
     categoryPath: string,
     memories: { path: string; tokenEstimate: number; summary?: string }[] = [],
-    subcategories: { path: string; memoryCount?: number }[] = []
+    subcategories: { path: string; memoryCount?: number }[] = [],
 ): Promise<void> => {
     const lines: string[] = [];
 
     // Memory section
     if (memories.length === 0) {
         lines.push('memories: []');
-    } else {
+    }
+    else {
         lines.push('memories:');
         for (const memory of memories) {
             lines.push('  -');
@@ -308,7 +320,8 @@ const createCategoryIndex = async (
     // Subcategories section
     if (subcategories.length === 0) {
         lines.push('subcategories: []');
-    } else {
+    }
+    else {
         lines.push('subcategories:');
         for (const sub of subcategories) {
             lines.push('  -');
@@ -330,7 +343,8 @@ const memoryExists = async (storeRoot: string, slugPath: string): Promise<boolea
     try {
         await fs.access(join(storeRoot, `${slugPath}.md`));
         return true;
-    } catch {
+    }
+    catch {
         return false;
     }
 };
@@ -341,7 +355,8 @@ const memoryExists = async (storeRoot: string, slugPath: string): Promise<boolea
 const readMemoryFile = async (storeRoot: string, slugPath: string): Promise<string | null> => {
     try {
         return await fs.readFile(join(storeRoot, `${slugPath}.md`), 'utf8');
-    } catch {
+    }
+    catch {
         return null;
     }
 };
@@ -365,13 +380,25 @@ describe('Cortex CLI Integration Tests', () => {
         it('should add a new memory with inline content', async () => {
             await createCategory('project', storeDir);
             const result = await runCortexCli(
-                ['memory', 'add', 'project/test-memory', '--content', 'This is test content.'],
-                { cwd: testProject }
+                [
+                    'memory',
+                    'add',
+                    'project/test-memory',
+                    '--content',
+                    'This is test content.',
+                ],
+                { cwd: testProject },
             );
 
             expectCliOk(
-                ['memory', 'add', 'project/test-memory', '--content', 'This is test content.'],
-                result
+                [
+                    'memory',
+                    'add',
+                    'project/test-memory',
+                    '--content',
+                    'This is test content.',
+                ],
+                result,
             );
             expect(result.exitCode).toBe(0);
             expect(result.stdout).toContain('Added memory');
@@ -396,7 +423,7 @@ describe('Cortex CLI Integration Tests', () => {
                     '--tags',
                     'tag1,tag2,tag3',
                 ],
-                { cwd: testProject }
+                { cwd: testProject },
             );
 
             expect(result.exitCode).toBe(0);
@@ -424,7 +451,7 @@ describe('Cortex CLI Integration Tests', () => {
                     '-t',
                     'third',
                 ],
-                { cwd: testProject }
+                { cwd: testProject },
             );
 
             expect(result.exitCode).toBe(0);
@@ -450,7 +477,7 @@ describe('Cortex CLI Integration Tests', () => {
                     '-t',
                     'gamma',
                 ],
-                { cwd: testProject }
+                { cwd: testProject },
             );
 
             expect(result.exitCode).toBe(0);
@@ -475,7 +502,7 @@ describe('Cortex CLI Integration Tests', () => {
                     '--expires-at',
                     expiryDate,
                 ],
-                { cwd: testProject }
+                { cwd: testProject },
             );
 
             expect(result.exitCode).toBe(0);
@@ -486,7 +513,12 @@ describe('Cortex CLI Integration Tests', () => {
         });
 
         it('should fail when memory path is missing', async () => {
-            const result = await runCortexCli(['memory', 'add', '--content', 'No path provided.'], {
+            const result = await runCortexCli([
+                'memory',
+                'add',
+                '--content',
+                'No path provided.',
+            ], {
                 cwd: testProject,
             });
 
@@ -497,8 +529,14 @@ describe('Cortex CLI Integration Tests', () => {
 
         it('should fail for invalid memory path format', async () => {
             const result = await runCortexCli(
-                ['memory', 'add', 'invalid-single-segment', '--content', 'Bad path.'],
-                { cwd: testProject }
+                [
+                    'memory',
+                    'add',
+                    'invalid-single-segment',
+                    '--content',
+                    'Bad path.',
+                ],
+                { cwd: testProject },
             );
 
             expect(result.exitCode).toBe(1);
@@ -506,10 +544,16 @@ describe('Cortex CLI Integration Tests', () => {
 
         it('should fail for unknown flags', async () => {
             const result = await runCortexCli(
-                ['memory', 'add', 'project/memory', '--unknown-flag', 'value'],
+                [
+                    'memory',
+                    'add',
+                    'project/memory',
+                    '--unknown-flag',
+                    'value',
+                ],
                 {
                     cwd: testProject,
-                }
+                },
             );
 
             expect(result.exitCode).toBe(1);
@@ -524,10 +568,16 @@ describe('Cortex CLI Integration Tests', () => {
             await fs.writeFile(contentFile, 'Content from file.', 'utf8');
 
             const result = await runCortexCli(
-                ['memory', 'add', 'project/file-memory', '--file', contentFile],
+                [
+                    'memory',
+                    'add',
+                    'project/file-memory',
+                    '--file',
+                    contentFile,
+                ],
                 {
                     cwd: testProject,
-                }
+                },
             );
 
             expect(result.exitCode).toBe(0);
@@ -546,7 +596,7 @@ describe('Cortex CLI Integration Tests', () => {
                     '--content',
                     'Deeply nested content.',
                 ],
-                { cwd: testProject }
+                { cwd: testProject },
             );
 
             expect(result.exitCode).toBe(0);
@@ -559,8 +609,14 @@ describe('Cortex CLI Integration Tests', () => {
             await createCategory('project', storeDir);
             const specialContent = 'Special chars: $HOME, `backticks`, "quotes", \'single\'';
             const result = await runCortexCli(
-                ['memory', 'add', 'project/special-memory', '--content', specialContent],
-                { cwd: testProject }
+                [
+                    'memory',
+                    'add',
+                    'project/special-memory',
+                    '--content',
+                    specialContent,
+                ],
+                { cwd: testProject },
             );
 
             expect(result.exitCode).toBe(0);
@@ -586,7 +642,9 @@ describe('Cortex CLI Integration Tests', () => {
         });
 
         it('should list all memories across categories', async () => {
-            const result = await runCortexCli(['memory', 'list'], { cwd: testProject });
+            const result = await runCortexCli([
+                'memory', 'list',
+            ], { cwd: testProject });
 
             expect(result.exitCode).toBe(0);
             expect(result.stdout).toContain('project/memory-one');
@@ -595,7 +653,11 @@ describe('Cortex CLI Integration Tests', () => {
         });
 
         it('should list memories in a specific category', async () => {
-            const result = await runCortexCli(['memory', 'list', 'project'], {
+            const result = await runCortexCli([
+                'memory',
+                'list',
+                'project',
+            ], {
                 cwd: testProject,
             });
 
@@ -606,7 +668,12 @@ describe('Cortex CLI Integration Tests', () => {
         });
 
         it('should output in JSON format', async () => {
-            const result = await runCortexCli(['memory', 'list', '--format', 'json'], {
+            const result = await runCortexCli([
+                'memory',
+                'list',
+                '--format',
+                'json',
+            ], {
                 cwd: testProject,
             });
 
@@ -624,7 +691,11 @@ describe('Cortex CLI Integration Tests', () => {
                 expiresAt: new Date('2020-01-01T00:00:00.000Z'),
             });
 
-            const result = await runCortexCli(['memory', 'list', 'project'], {
+            const result = await runCortexCli([
+                'memory',
+                'list',
+                'project',
+            ], {
                 cwd: testProject,
             });
 
@@ -640,7 +711,12 @@ describe('Cortex CLI Integration Tests', () => {
                 expiresAt: new Date('2020-01-01T00:00:00.000Z'),
             });
 
-            const result = await runCortexCli(['memory', 'list', 'project', '--include-expired'], {
+            const result = await runCortexCli([
+                'memory',
+                'list',
+                'project',
+                '--include-expired',
+            ], {
                 cwd: testProject,
             });
 
@@ -650,7 +726,11 @@ describe('Cortex CLI Integration Tests', () => {
         });
 
         it('should return empty list for non-existent category', async () => {
-            const result = await runCortexCli(['memory', 'list', 'nonexistent'], {
+            const result = await runCortexCli([
+                'memory',
+                'list',
+                'nonexistent',
+            ], {
                 cwd: testProject,
             });
 
@@ -661,7 +741,12 @@ describe('Cortex CLI Integration Tests', () => {
 
         it('should use yaml format for invalid format option', async () => {
             // Invalid formats fall back to default YAML formatting
-            const result = await runCortexCli(['memory', 'list', '--format', 'invalid'], {
+            const result = await runCortexCli([
+                'memory',
+                'list',
+                '--format',
+                'invalid',
+            ], {
                 cwd: testProject,
             });
 
@@ -681,8 +766,14 @@ describe('Cortex CLI Integration Tests', () => {
 
         it('should update memory content', async () => {
             const result = await runCortexCli(
-                ['memory', 'update', 'project/updatable', '--content', 'Updated content.'],
-                { cwd: testProject }
+                [
+                    'memory',
+                    'update',
+                    'project/updatable',
+                    '--content',
+                    'Updated content.',
+                ],
+                { cwd: testProject },
             );
 
             expect(result.exitCode).toBe(0);
@@ -695,8 +786,14 @@ describe('Cortex CLI Integration Tests', () => {
 
         it('should update memory tags', async () => {
             const result = await runCortexCli(
-                ['memory', 'update', 'project/updatable', '--tags', 'new-tag,updated'],
-                { cwd: testProject }
+                [
+                    'memory',
+                    'update',
+                    'project/updatable',
+                    '--tags',
+                    'new-tag,updated',
+                ],
+                { cwd: testProject },
             );
 
             expect(result.exitCode).toBe(0);
@@ -715,7 +812,7 @@ describe('Cortex CLI Integration Tests', () => {
                     '--expires-at',
                     '2030-01-01T00:00:00.000Z',
                 ],
-                { cwd: testProject }
+                { cwd: testProject },
             );
 
             expect(result.exitCode).toBe(0);
@@ -733,10 +830,15 @@ describe('Cortex CLI Integration Tests', () => {
             });
 
             const result = await runCortexCli(
-                ['memory', 'update', 'project/with-expiry', '--no-expires-at'],
+                [
+                    'memory',
+                    'update',
+                    'project/with-expiry',
+                    '--no-expires-at',
+                ],
                 {
                     cwd: testProject,
-                }
+                },
             );
 
             expect(result.exitCode).toBe(0);
@@ -747,8 +849,14 @@ describe('Cortex CLI Integration Tests', () => {
 
         it('should fail when memory does not exist', async () => {
             const result = await runCortexCli(
-                ['memory', 'update', 'project/nonexistent', '--content', 'New content.'],
-                { cwd: testProject }
+                [
+                    'memory',
+                    'update',
+                    'project/nonexistent',
+                    '--content',
+                    'New content.',
+                ],
+                { cwd: testProject },
             );
 
             expect(result.exitCode).toBe(1);
@@ -756,7 +864,11 @@ describe('Cortex CLI Integration Tests', () => {
         });
 
         it('should fail when no updates provided', async () => {
-            const result = await runCortexCli(['memory', 'update', 'project/updatable'], {
+            const result = await runCortexCli([
+                'memory',
+                'update',
+                'project/updatable',
+            ], {
                 cwd: testProject,
             });
 
@@ -769,10 +881,16 @@ describe('Cortex CLI Integration Tests', () => {
             await fs.writeFile(contentFile, 'Content from file update.', 'utf8');
 
             const result = await runCortexCli(
-                ['memory', 'update', 'project/updatable', '--file', contentFile],
+                [
+                    'memory',
+                    'update',
+                    'project/updatable',
+                    '--file',
+                    contentFile,
+                ],
                 {
                     cwd: testProject,
-                }
+                },
             );
 
             expect(result.exitCode).toBe(0);
@@ -783,10 +901,16 @@ describe('Cortex CLI Integration Tests', () => {
 
         it('should preserve original content when only updating tags', async () => {
             const result = await runCortexCli(
-                ['memory', 'update', 'project/updatable', '--tags', 'new-tag'],
+                [
+                    'memory',
+                    'update',
+                    'project/updatable',
+                    '--tags',
+                    'new-tag',
+                ],
                 {
                     cwd: testProject,
-                }
+                },
             );
 
             expect(result.exitCode).toBe(0);
@@ -809,7 +933,7 @@ describe('Cortex CLI Integration Tests', () => {
                     '-t',
                     'tag-c',
                 ],
-                { cwd: testProject }
+                { cwd: testProject },
             );
 
             expect(result.exitCode).toBe(0);
@@ -823,8 +947,16 @@ describe('Cortex CLI Integration Tests', () => {
 
         it('should update memory tags with mixed formats (comma-separated and multiple flags)', async () => {
             const result = await runCortexCli(
-                ['memory', 'update', 'project/updatable', '-t', 'x,y', '-t', 'z'],
-                { cwd: testProject }
+                [
+                    'memory',
+                    'update',
+                    'project/updatable',
+                    '-t',
+                    'x,y',
+                    '-t',
+                    'z',
+                ],
+                { cwd: testProject },
             );
 
             expect(result.exitCode).toBe(0);
@@ -855,7 +987,11 @@ describe('Cortex CLI Integration Tests', () => {
         });
 
         it('should report expired memories with --dry-run', async () => {
-            const result = await runCortexCli(['store', 'prune', '--dry-run'], {
+            const result = await runCortexCli([
+                'store',
+                'prune',
+                '--dry-run',
+            ], {
                 cwd: testProject,
             });
 
@@ -870,7 +1006,9 @@ describe('Cortex CLI Integration Tests', () => {
         });
 
         it('should delete expired memories without --dry-run', async () => {
-            const result = await runCortexCli(['store', 'prune'], {
+            const result = await runCortexCli([
+                'store', 'prune',
+            ], {
                 cwd: testProject,
             });
 
@@ -891,11 +1029,11 @@ describe('Cortex CLI Integration Tests', () => {
             await fs.rm(join(storeDir, 'project', 'expired-one.md'));
             await fs.rm(join(storeDir, 'project', 'expired-two.md'));
 
-            await createCategoryIndex(storeDir, 'project', [
-                { path: 'project/fresh-memory', tokenEstimate: 10 },
-            ]);
+            await createCategoryIndex(storeDir, 'project', [{ path: 'project/fresh-memory', tokenEstimate: 10 }]);
 
-            const result = await runCortexCli(['store', 'prune'], {
+            const result = await runCortexCli([
+                'store', 'prune',
+            ], {
                 cwd: testProject,
             });
 
@@ -916,7 +1054,9 @@ describe('Cortex CLI Integration Tests', () => {
         });
 
         it('should rebuild indexes', async () => {
-            const result = await runCortexCli(['store', 'reindex'], {
+            const result = await runCortexCli([
+                'store', 'reindex',
+            ], {
                 cwd: testProject,
             });
 
@@ -963,14 +1103,17 @@ describe('Cortex CLI Integration Tests', () => {
             await fs.mkdir(fakeCortex, { recursive: true });
             // Note: .cortex/memory doesn't exist, so resolution should fail
             try {
-                const result = await runCortexCli(['memory', 'list'], {
+                const result = await runCortexCli([
+                    'memory', 'list',
+                ], {
                     cwd: emptyDir,
                 });
 
                 // Either fails because no store found, or succeeds with empty list
                 // depending on whether global store exists
                 expect(result.exitCode).toBeGreaterThanOrEqual(0);
-            } finally {
+            }
+            finally {
                 await fs.rm(emptyDir, { recursive: true, force: true });
             }
         });
@@ -998,7 +1141,9 @@ describe('Cortex CLI Integration Tests', () => {
         });
 
         it('should show memory subcommand help', async () => {
-            const result = await runCortexCli(['memory', '--help'], {
+            const result = await runCortexCli([
+                'memory', '--help',
+            ], {
                 cwd: testProject,
             });
 
@@ -1010,7 +1155,9 @@ describe('Cortex CLI Integration Tests', () => {
         });
 
         it('should show store subcommand help', async () => {
-            const result = await runCortexCli(['store', '--help'], {
+            const result = await runCortexCli([
+                'store', '--help',
+            ], {
                 cwd: testProject,
             });
 
@@ -1034,14 +1181,28 @@ describe('Cortex CLI Integration Tests', () => {
 
             try {
                 const result = await runCortexCli(
-                    ['store', 'init', storePath, '--name', 'my-project', '--format', 'json'],
+                    [
+                        'store',
+                        'init',
+                        storePath,
+                        '--name',
+                        'my-project',
+                        '--format',
+                        'json',
+                    ],
                     {
                         cwd: freshDir,
                         env: { CORTEX_CONFIG_DIR: configDir },
-                    }
+                    },
                 );
 
-                expectCliOk(['store', 'init', storePath, '--name', 'my-project'], result);
+                expectCliOk([
+                    'store',
+                    'init',
+                    storePath,
+                    '--name',
+                    'my-project',
+                ], result);
                 expect(result.exitCode).toBe(0);
 
                 const parsed = JSON.parse(result.stdout) as {
@@ -1054,7 +1215,8 @@ describe('Cortex CLI Integration Tests', () => {
                 const configContent = await fs.readFile(join(configDir, 'config.yaml'), 'utf8');
                 expect(configContent).toContain('my-project');
                 expect(configContent).toContain(storePath);
-            } finally {
+            }
+            finally {
                 await fs.rm(freshDir, { recursive: true, force: true });
             }
         });
@@ -1070,19 +1232,34 @@ describe('Cortex CLI Integration Tests', () => {
 
             try {
                 const result = await runCortexCli(
-                    ['store', 'init', storePath, '--name', 'new-dir-store', '--format', 'json'],
+                    [
+                        'store',
+                        'init',
+                        storePath,
+                        '--name',
+                        'new-dir-store',
+                        '--format',
+                        'json',
+                    ],
                     {
                         cwd: freshDir,
                         env: { CORTEX_CONFIG_DIR: configDir },
-                    }
+                    },
                 );
 
-                expectCliOk(['store', 'init', storePath, '--name', 'new-dir-store'], result);
+                expectCliOk([
+                    'store',
+                    'init',
+                    storePath,
+                    '--name',
+                    'new-dir-store',
+                ], result);
 
                 // The directory must have been created by the init command
                 const dirStat = await fs.stat(storePath);
                 expect(dirStat.isDirectory()).toBe(true);
-            } finally {
+            }
+            finally {
                 await fs.rm(freshDir, { recursive: true, force: true });
             }
         });
@@ -1091,10 +1268,24 @@ describe('Cortex CLI Integration Tests', () => {
             // Register a store first using the existing testProject setup
             const storePath = join(testProject, '.cortex', 'memory2');
             const firstInit = await runCortexCli(
-                ['store', 'init', storePath, '--name', 'existing-store', '--format', 'json'],
-                { cwd: testProject }
+                [
+                    'store',
+                    'init',
+                    storePath,
+                    '--name',
+                    'existing-store',
+                    '--format',
+                    'json',
+                ],
+                { cwd: testProject },
             );
-            expectCliOk(['store', 'init', storePath, '--name', 'existing-store'], firstInit);
+            expectCliOk([
+                'store',
+                'init',
+                storePath,
+                '--name',
+                'existing-store',
+            ], firstInit);
 
             // Attempt to init the same name again — should fail
             const secondInit = await runCortexCli(
@@ -1105,7 +1296,7 @@ describe('Cortex CLI Integration Tests', () => {
                     '--name',
                     'existing-store',
                 ],
-                { cwd: testProject }
+                { cwd: testProject },
             );
 
             expect(secondInit.exitCode).toBe(1);