@amodalai/amodal 0.2.10 → 0.3.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@amodalai/amodal",
-  "version": "0.2.10",
+  "version": "0.3.1",
   "description": "Amodal CLI",
   "license": "MIT",
   "repository": {
@@ -30,10 +30,11 @@
     "semver": "^7.6.0",
     "yargs": "^17.7.2",
     "zod": "^4.3.6",
-    "@amodalai/types": "0.2.10",
-    "@amodalai/core": "0.2.10",
-    "@amodalai/runtime": "0.2.10",
-    "@amodalai/runtime-app": "0.2.10"
+    "@amodalai/types": "0.3.1",
+    "@amodalai/core": "0.3.1",
+    "@amodalai/db": "0.3.1",
+    "@amodalai/runtime": "0.3.1",
+    "@amodalai/runtime-app": "0.3.1"
   },
   "devDependencies": {
     "@types/node": "^20.11.24",

package/src/commands/dev.test.ts

@@ -1,63 +1,126 @@
 /**
  * @license
- * Copyright 2025 Amodal Labs, Inc.
+ * Copyright 2026 Amodal Labs, Inc.
  * SPDX-License-Identifier: MIT
  */
 
-import {describe, it, expect, vi} from 'vitest';
-
-vi.mock('../shared/repo-discovery.js', () => ({
-  findRepoRoot: vi.fn(() => '/test/repo'),
-}));
+import {describe, it, expect, beforeEach, afterEach, vi} from 'vitest';
+import {mkdtempSync, writeFileSync, rmSync} from 'node:fs';
+import path from 'node:path';
+import os from 'node:os';
 
+// Mock runtime dependencies that we don't need for the DATABASE_URL check
 vi.mock('@amodalai/runtime', () => ({
   createLocalServer: vi.fn().mockResolvedValue({
     app: {},
     start: vi.fn().mockResolvedValue({}),
     stop: vi.fn().mockResolvedValue(undefined),
   }),
+  initLogLevel: vi.fn(),
+  interceptConsole: vi.fn(),
+  log: {
+    info: vi.fn(),
+    warn: vi.fn(),
+    error: vi.fn(),
+    debug: vi.fn(),
+  },
+}));
+
+vi.mock('@amodalai/db', () => ({
+  getDb: vi.fn(),
+  ensureSchema: vi.fn().mockResolvedValue(undefined),
+  closeDb: vi.fn().mockResolvedValue(undefined),
+}));
+
+vi.mock('@amodalai/core', () => ({
+  resolveAdminAgent: vi.fn().mockResolvedValue(null),
+}));
+
+vi.mock('../shared/connection-preflight.js', () => ({
+  runConnectionPreflight: vi.fn().mockResolvedValue({results: [], hasFailures: false}),
+  printPreflightTable: vi.fn(),
 }));
 
+/**
+ * Smoke test: `amodal dev` without DATABASE_URL should print instructions
+ * and exit with code 1.
+ */
 describe('dev command', () => {
-  it('should import without error', async () => {
-    const mod = await import('./dev.js');
-    expect(mod.runDev).toBeDefined();
-    expect(typeof mod.runDev).toBe('function');
+  let tmpDir: string;
+  let fakeHome: string;
+  let origHome: string;
+  let origDatabaseUrl: string | undefined;
+
+  beforeEach(() => {
+    tmpDir = mkdtempSync(path.join(os.tmpdir(), 'dev-test-'));
+    fakeHome = mkdtempSync(path.join(os.tmpdir(), 'dev-test-home-'));
+
+    // Create a minimal amodal.json so findRepoRoot succeeds
+    writeFileSync(path.join(tmpDir, 'amodal.json'), JSON.stringify({name: 'test-agent'}));
+
+    origHome = process.env['HOME'] ?? '';
+    origDatabaseUrl = process.env['DATABASE_URL'];
+    delete process.env['DATABASE_URL'];
+    process.env['HOME'] = fakeHome;
   });
 
-  it('should find repo root', async () => {
-    const {findRepoRoot} = await import('../shared/repo-discovery.js');
-    const root = findRepoRoot('/some/path');
-    expect(root).toBe('/test/repo');
+  afterEach(() => {
+    process.env['HOME'] = origHome;
+    if (origDatabaseUrl !== undefined) {
+      process.env['DATABASE_URL'] = origDatabaseUrl;
+    } else {
+      delete process.env['DATABASE_URL'];
+    }
+    rmSync(tmpDir, {recursive: true, force: true});
+    rmSync(fakeHome, {recursive: true, force: true});
+    vi.restoreAllMocks();
   });
 
-  it('should create repo server with correct config', async () => {
-    const {createLocalServer} = await import('@amodalai/runtime');
-    const mockExit = vi.spyOn(process, 'exit').mockImplementation(() => {
-      throw new Error('exit');
+  it('exits with code 1 and prints instructions when DATABASE_URL is missing', async () => {
+    // Capture stderr output
+    const stderrChunks: string[] = [];
+    vi.spyOn(process.stderr, 'write').mockImplementation((chunk: string | Uint8Array) => {
+      stderrChunks.push(String(chunk));
+      return true;
     });
 
-    // runDev calls process.exit on success (after server starts), so we catch that
+    // Mock process.exit to throw so we can catch it
+    vi.spyOn(process, 'exit').mockImplementation((code?: number | string | null | undefined) => {
+      throw new ExitError(typeof code === 'number' ? code : 1);
+    });
+
+    const {runDev} = await import('./dev.js');
+
     try {
-      const {runDev} = await import('./dev.js');
-      // Don't actually run it — just verify the module loads
-      expect(runDev).toBeDefined();
-    } catch {
-      // Expected
+      await runDev({cwd: tmpDir});
+      expect.unreachable('runDev should have called process.exit');
+    } catch (err: unknown) {
+      if (err instanceof ExitError) {
+        expect(err.code).toBe(1);
+      } else {
+        throw err;
+      }
     }
 
-    expect(createLocalServer).toBeDefined();
-    mockExit.mockRestore();
+    const output = stderrChunks.join('');
+    expect(output).toContain('DATABASE_URL is required');
+    expect(output).toContain('docker run');
+    expect(output).toContain('~/.amodal/.env');
   });
 
-  it('should use default port 3847', async () => {
-    const mod = await import('./dev.js');
-    // Just verify the export exists — actual server test is integration
-    expect(mod.runDev).toBeDefined();
-  });
-
-  it('should accept custom port', async () => {
+  it('should import without error', async () => {
     const mod = await import('./dev.js');
     expect(mod.runDev).toBeDefined();
+    expect(typeof mod.runDev).toBe('function');
   });
 });
+
+/** Sentinel error thrown by our process.exit mock. */
+class ExitError extends Error {
+  readonly code: number;
+  constructor(code: number) {
+    super(`process.exit(${String(code)})`);
+    this.name = 'ExitError';
+    this.code = code;
+  }
+}