@satori-sh/cli 0.0.6 → 0.0.8

package/tests/index.test.ts

@@ -1,322 +0,0 @@
-import { test, expect, mock, beforeEach, afterEach } from 'bun:test';
-import { main, searchMemories, addMemories, enhanceMessagesWithMemory } from '../src/index';
-import { getConfig, checkWriteAccess, saveApiKey } from '../src/config.js';
-
-let originalFetch: typeof global.fetch;
-let originalEnv: NodeJS.ProcessEnv;
-let originalArgv: string[];
-let originalPlatform: string;
-
-beforeEach(() => {
-  originalFetch = global.fetch;
-  originalEnv = { ...process.env };
-  originalArgv = [...process.argv];
-  originalPlatform = process.platform;
-});
-
-afterEach(() => {
-  global.fetch = originalFetch;
-  process.env = originalEnv;
-  process.argv = originalArgv;
-  (process as any).platform = originalPlatform;
-});
-
-
-
-test('error handling for invalid base URL', async () => {
-  process.env.SATORI_BASE_URL = 'invalid-url';
-  process.argv = ['node', 'index.js', 'query'];
-  const errorSpy = mock(() => {});
-  console.error = errorSpy;
-  const exitSpy = mock(() => { throw new Error('exit'); });
-  process.exit = exitSpy;
-
-  try {
-    await main();
-    expect(true).toBe(false); // should not reach
-  } catch (e) {
-    expect((e as Error).message).toBe('exit');
-  }
-});
-
-test('error handling for invalid base URL', async () => {
-  process.env.SATORI_API_KEY = 'test-key';
-  process.env.SATORI_BASE_URL = 'invalid-url';
-  process.argv = ['node', 'index.js', 'query'];
-  const errorSpy = mock(() => {});
-  console.error = errorSpy;
-  const exitSpy = mock(() => { throw new Error('exit'); });
-  process.exit = exitSpy;
-
-  try {
-    await main();
-    expect(true).toBe(false); // should not reach
-  } catch (e) {
-    expect((e as Error).message).toBe('exit');
-  }
-});
-
-test('search with empty query', async () => {
-  process.env.SATORI_API_KEY = 'satori-erQCxG3GYryoXt4JydgEsbRmg5LuJf6p';
-  const errorSpy = mock(() => {});
-  console.error = errorSpy;
-
-  await searchMemories('');
-
-  expect(errorSpy).toHaveBeenCalledWith('Query cannot be empty');
-});
-
-test('add with empty text', async () => {
-  process.env.SATORI_API_KEY = 'satori-erQCxG3GYryoXt4JydgEsbRmg5LuJf6p';
-  const errorSpy = mock(() => {});
-  console.error = errorSpy;
-
-  await addMemories('');
-
-  expect(errorSpy).toHaveBeenCalledWith('Text cannot be empty');
-});
-
-test('HTTP 401 error', async () => {
-   process.env.SATORI_API_KEY = 'test-key';
-   const fetchMock = Object.assign(
-     mock(() => Promise.resolve(new Response('Unauthorized', { status: 401, statusText: 'Unauthorized' }))),
-     { preconnect: mock(() => {}) }
-   ) as typeof fetch;
-   globalThis.fetch = fetchMock;
-  const errorSpy = mock(() => {});
-  console.error = errorSpy;
-
-  await searchMemories('test');
-
-  expect(errorSpy).toHaveBeenCalledWith('HTTP error: 401 Unauthorized');
-});
-
-test('HTTP 500 error', async () => {
-   process.env.SATORI_API_KEY = 'test-key';
-   const fetchMock = Object.assign(
-     mock(() => Promise.resolve(new Response('Internal Server Error', { status: 500, statusText: 'Internal Server Error' }))),
-     { preconnect: mock(() => {}) }
-   ) as typeof fetch;
-   globalThis.fetch = fetchMock;
-  const errorSpy = mock(() => {});
-  console.error = errorSpy;
-
-  await searchMemories('test');
-
-  expect(errorSpy).toHaveBeenCalledWith('HTTP error: 500 Internal Server Error');
-});
-
-test('network error', async () => {
-   process.env.SATORI_API_KEY = 'test-key';
-   const fetchMock = Object.assign(
-     mock(() => { throw new Error('network error'); }),
-     { preconnect: mock(() => {}) }
-   ) as typeof fetch;
-   globalThis.fetch = fetchMock;
-  const errorSpy = mock(() => {});
-  console.error = errorSpy;
-
-  await searchMemories('test');
-
-  expect(errorSpy).toHaveBeenCalledWith('Error searching memories: network error');
-});
-
-test('malformed JSON response', async () => {
-   process.env.SATORI_API_KEY = 'test-key';
-   const fetchMock = Object.assign(
-     mock(() => Promise.resolve(new Response('invalid json', { status: 200 }))),
-     { preconnect: mock(() => {}) }
-   ) as typeof fetch;
-   (globalThis as any).fetch = fetchMock;
-  const errorSpy = mock(() => {});
-  console.error = errorSpy;
-
-  await searchMemories('test');
-
-  expect(errorSpy).toHaveBeenCalledWith(expect.stringContaining('Error searching memories:'));
-});
-
-test('enhanceMessagesWithMemory adds context', () => {
-  const messages = [{ role: 'user' as const, content: 'hello' }];
-  const memoryContext = { results: [{ id: '1', memory: 'test memory' }] };
-  const enhanced = enhanceMessagesWithMemory(messages, memoryContext);
-
-  expect(enhanced).toHaveLength(2);
-  expect(enhanced[0].role).toBe('system');
-  expect(enhanced[0].content).toContain('Relevant context from memory:\ntest memory');
-  expect(enhanced[1]).toEqual(messages[0]);
-});
-
-test('enhanceMessagesWithMemory handles empty memories', () => {
-  const messages = [{ role: 'user' as const, content: 'hello' }];
-  const memoryContext = { results: [] };
-  const enhanced = enhanceMessagesWithMemory(messages, memoryContext);
-
-  expect(enhanced).toEqual(messages);
-});
-
-
-
-test('getConfig throws on invalid base URL', async () => {
-  process.env.SATORI_BASE_URL = 'invalid-url';
-  try {
-    await getConfig();
-    expect(true).toBe(false); // should throw
-  } catch (error) {
-    expect((error as Error).message).toBe('Invalid SATORI_BASE_URL format');
-  } finally {
-    delete process.env.SATORI_BASE_URL;
-  }
-});
-
-test('getConfig throws on non-darwin platform', async () => {
-  (process as any).platform = 'win32';
-  try {
-    await getConfig();
-    expect(true).toBe(false); // should throw
-  } catch (error) {
-    expect((error as Error).message).toBe('We do not currently support Windows yet, email support@satori.sh to request Windows support');
-  }
-});
-
-test('getConfig generates apiKey on successful fetch', async () => {
-   delete process.env.SATORI_API_KEY;
-   const fetchMock = Object.assign(
-     mock(() => Promise.resolve(new Response(JSON.stringify({ api_key: 'generated-key' }), { status: 200 }))),
-     { preconnect: mock(() => {}) }
-   ) as typeof fetch;
-   globalThis.fetch = fetchMock;
-
-  const config = await getConfig();
-  expect(config.apiKey).toBe('generated-key');
-
-  globalThis.fetch = originalFetch;
-});
-
-test('getConfig throws on network error during generation', async () => {
-  delete process.env.SATORI_API_KEY;
-  mock.module('node:fs', () => ({
-    promises: {
-      mkdir: mock(() => Promise.resolve()),
-      writeFile: mock(() => Promise.resolve()),
-      unlink: mock(() => Promise.resolve())
-    }
-  }));
-  const mockFile = {
-    exists: mock(() => Promise.resolve(false))
-  };
-  const originalBun = (globalThis as any).Bun;
-  (globalThis as any).Bun = { file: mock(() => mockFile) };
-   const fetchMock = Object.assign(
-     mock(() => Promise.reject(new Error('network error'))),
-     { preconnect: mock(() => {}) }
-   ) as typeof fetch;
-   globalThis.fetch = fetchMock;
-
-  try {
-    await getConfig();
-    expect(true).toBe(false); // should throw
-  } catch (error) {
-    expect((error as Error).message).toBe('Failed to generate API key: network error');
-  }
-
-  globalThis.fetch = originalFetch;
-  (globalThis as any).Bun = originalBun;
-});
-
-test('getConfig throws on invalid response during generation', async () => {
-  delete process.env.SATORI_API_KEY;
-  mock.module('node:fs', () => ({
-    promises: {
-      mkdir: mock(() => Promise.resolve()),
-      writeFile: mock(() => Promise.resolve()),
-      unlink: mock(() => Promise.resolve())
-    }
-  }));
-  const mockFile = {
-    exists: mock(() => Promise.resolve(false))
-  };
-  const originalBun = (globalThis as any).Bun;
-  (globalThis as any).Bun = { file: mock(() => mockFile) };
-   const fetchMock = Object.assign(
-     mock(() => Promise.resolve(new Response(JSON.stringify({}), { status: 200 }))),
-     { preconnect: mock(() => {}) }
-   ) as typeof fetch;
-   globalThis.fetch = fetchMock;
-
-  try {
-    await getConfig();
-    expect(true).toBe(false); // should throw
-  } catch (error) {
-    expect((error as Error).message).toBe('Failed to generate API key: Invalid response: missing api_key');
-  }
-
-  globalThis.fetch = originalFetch;
-  (globalThis as any).Bun = originalBun;
-});
-
-test('checkWriteAccess succeeds on successful write', async () => {
-  mock.module('node:fs', () => ({
-    promises: {
-      mkdir: mock(() => Promise.resolve()),
-      writeFile: mock(() => Promise.resolve()),
-      unlink: mock(() => Promise.resolve())
-    }
-  }));
-
-  await checkWriteAccess();
-  // No throw means success
-});
-
-test('checkWriteAccess throws on permission denied', async () => {
-  mock.module('node:fs', () => ({
-    promises: {
-      mkdir: mock(() => Promise.resolve()),
-      writeFile: mock(() => { throw new Error('EACCES: permission denied'); }),
-      unlink: mock(() => Promise.resolve())
-    }
-  }));
-
-  try {
-    await checkWriteAccess();
-    expect(true).toBe(false); // should throw
-  } catch (error) {
-    expect((error as Error).message).toBe('Cannot write to config directory: EACCES: permission denied');
-  }
-});
-
-test('saveApiKey saves successfully', async () => {
-  const mockWriteFile = mock(() => Promise.resolve());
-  mock.module('node:fs', () => ({
-    promises: {
-      mkdir: mock(() => Promise.resolve()),
-      writeFile: mockWriteFile,
-      unlink: mock(() => Promise.resolve())
-    }
-  }));
-
-  await saveApiKey('test-key');
-
-  expect(mockWriteFile).toHaveBeenCalledWith(expect.stringContaining('satori.json'), JSON.stringify({ api_key: 'test-key' }, null, 2));
-});
-
-test('saveApiKey throws on write error', async () => {
-  const mockWriteFile = mock(() => { throw new Error('write error'); });
-  mock.module('node:fs', () => ({
-    promises: {
-      mkdir: mock(() => Promise.resolve()),
-      writeFile: mockWriteFile,
-      unlink: mock(() => Promise.resolve())
-    }
-  }));
-
-  try {
-    await saveApiKey('test-key');
-    expect(true).toBe(false); // should throw
-  } catch (error) {
-    expect((error as Error).message).toBe('Cannot write to config directory: write error');
-  }
-});
-
-// Note: Additional unit tests for file reading scenarios would require mocking Bun.file,
-// which is readonly. The existing test covers the missing file case.

package/tsconfig.json

@@ -1,33 +0,0 @@
-{
-  "compilerOptions": {
-    "target": "ES2022",
-    "lib": [
-      "ES2022"
-    ],
-    "types": ["bun-types"],
-    "allowJs": true,
-    "skipLibCheck": true,
-    "esModuleInterop": true,
-    "allowSyntheticDefaultImports": true,
-    "strict": true,
-    "forceConsistentCasingInFileNames": true,
-    "noFallthroughCasesInSwitch": true,
-    "module": "ESNext",
-    "moduleResolution": "bundler",
-    "resolveJsonModule": true,
-    "isolatedModules": true,
-    "noEmit": false,
-    "outDir": "./dist",
-    "declaration": true,
-    "declarationMap": true,
-    "sourceMap": true
-  },
-  "include": [
-    "src/**/*",
-    "tests/**/*"
-  ],
-  "exclude": [
-    "node_modules",
-    "dist"
-  ]
-}

package/tsup.config.js

@@ -1,7 +0,0 @@
-import { defineConfig } from 'tsup'
-
-export default defineConfig({
-  entry: ['src/index.ts'],
-  format: ['esm'],
-  clean: true
-})