@yeseh/cortex-cli 0.6.3 → 0.6.5

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,19 +88,12 @@ 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',
@@ -116,9 +109,7 @@ 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(),
         ]);
@@ -145,8 +136,7 @@ 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() ?? '';
@@ -217,7 +207,7 @@ const createMemoryFile = async (
         expiresAt?: Date;
         createdAt?: Date;
         updatedAt?: Date;
-    } = {},
+    } = {}
 ): Promise<void> => {
     const content = options.content ?? 'Test memory content.';
     const tags = options.tags ?? ['test'];
@@ -239,7 +229,7 @@ const createMemoryFile = async (
             expiresAt: options.expiresAt,
             citations: [],
         },
-        content,
+        content
     );
     if (!memoryResult.ok()) {
         throw new Error(`Failed to create memory: ${memoryResult.error.message}`);
@@ -247,7 +237,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()) {
@@ -280,8 +270,7 @@ 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');
     }
 };
@@ -295,15 +284,14 @@ 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('  -');
@@ -320,8 +308,7 @@ const createCategoryIndex = async (
     // Subcategories section
     if (subcategories.length === 0) {
         lines.push('subcategories: []');
-    }
-    else {
+    } else {
         lines.push('subcategories:');
         for (const sub of subcategories) {
             lines.push('  -');
@@ -343,8 +330,7 @@ const memoryExists = async (storeRoot: string, slugPath: string): Promise<boolea
     try {
         await fs.access(join(storeRoot, `${slugPath}.md`));
         return true;
-    }
-    catch {
+    } catch {
         return false;
     }
 };
@@ -355,8 +341,7 @@ 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;
     }
 };
@@ -380,25 +365,13 @@ 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');
@@ -423,7 +396,7 @@ describe('Cortex CLI Integration Tests', () => {
                     '--tags',
                     'tag1,tag2,tag3',
                 ],
-                { cwd: testProject },
+                { cwd: testProject }
             );
 
             expect(result.exitCode).toBe(0);
@@ -451,7 +424,7 @@ describe('Cortex CLI Integration Tests', () => {
                     '-t',
                     'third',
                 ],
-                { cwd: testProject },
+                { cwd: testProject }
             );
 
             expect(result.exitCode).toBe(0);
@@ -477,7 +450,7 @@ describe('Cortex CLI Integration Tests', () => {
                     '-t',
                     'gamma',
                 ],
-                { cwd: testProject },
+                { cwd: testProject }
             );
 
             expect(result.exitCode).toBe(0);
@@ -502,7 +475,7 @@ describe('Cortex CLI Integration Tests', () => {
                     '--expires-at',
                     expiryDate,
                 ],
-                { cwd: testProject },
+                { cwd: testProject }
             );
 
             expect(result.exitCode).toBe(0);
@@ -513,12 +486,7 @@ 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,
             });
 
@@ -529,14 +497,8 @@ 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);
@@ -544,16 +506,10 @@ 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);
@@ -568,16 +524,10 @@ 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);
@@ -596,7 +546,7 @@ describe('Cortex CLI Integration Tests', () => {
                     '--content',
                     'Deeply nested content.',
                 ],
-                { cwd: testProject },
+                { cwd: testProject }
             );
 
             expect(result.exitCode).toBe(0);
@@ -609,14 +559,8 @@ 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);
@@ -642,9 +586,7 @@ 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');
@@ -653,11 +595,7 @@ 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,
             });
 
@@ -668,12 +606,7 @@ 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,
             });
 
@@ -691,11 +624,7 @@ 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,
             });
 
@@ -711,12 +640,7 @@ 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,
             });
 
@@ -726,11 +650,7 @@ 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,
             });
 
@@ -741,12 +661,7 @@ 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,
             });
 
@@ -766,14 +681,8 @@ 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);
@@ -786,14 +695,8 @@ 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);
@@ -812,7 +715,7 @@ describe('Cortex CLI Integration Tests', () => {
                     '--expires-at',
                     '2030-01-01T00:00:00.000Z',
                 ],
-                { cwd: testProject },
+                { cwd: testProject }
             );
 
             expect(result.exitCode).toBe(0);
@@ -830,15 +733,10 @@ 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);
@@ -849,14 +747,8 @@ 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);
@@ -864,11 +756,7 @@ 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,
             });
 
@@ -881,16 +769,10 @@ 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);
@@ -901,16 +783,10 @@ 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);
@@ -933,7 +809,7 @@ describe('Cortex CLI Integration Tests', () => {
                     '-t',
                     'tag-c',
                 ],
-                { cwd: testProject },
+                { cwd: testProject }
             );
 
             expect(result.exitCode).toBe(0);
@@ -947,16 +823,8 @@ 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);
@@ -987,11 +855,7 @@ 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,
             });
 
@@ -1006,9 +870,7 @@ 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,
             });
 
@@ -1029,11 +891,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,
             });
 
@@ -1054,9 +916,7 @@ describe('Cortex CLI Integration Tests', () => {
         });
 
         it('should rebuild indexes', async () => {
-            const result = await runCortexCli([
-                'store', 'reindex',
-            ], {
+            const result = await runCortexCli(['store', 'reindex'], {
                 cwd: testProject,
             });
 
@@ -1103,17 +963,14 @@ 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 });
             }
         });
@@ -1141,9 +998,7 @@ 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,
             });
 
@@ -1155,9 +1010,7 @@ 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,
             });
 
@@ -1181,28 +1034,14 @@ 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 {
@@ -1215,8 +1054,7 @@ 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 });
             }
         });
@@ -1232,34 +1070,19 @@ 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 });
             }
         });
@@ -1268,24 +1091,10 @@ 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(
@@ -1296,7 +1105,7 @@ describe('Cortex CLI Integration Tests', () => {
                     '--name',
                     'existing-store',
                 ],
-                { cwd: testProject },
+                { cwd: testProject }
             );
 
             expect(secondInit.exitCode).toBe(1);