@celilo/cli 0.3.0 → 0.3.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@celilo/cli",
-  "version": "0.3.0",
+  "version": "0.3.1",
   "description": "Celilo — home lab orchestration CLI",
   "type": "module",
   "bin": {

package/src/cli/commands/events.test.ts

@@ -22,10 +22,10 @@ describe('celilo events command handlers', () => {
   beforeEach(() => {
     dir = mkdtempSync(join(tmpdir(), 'events-cmd-test-'));
     dbPath = join(dir, 'events.db');
-    process.env.CELILO_EVENT_BUS_PATH = dbPath;
+    process.env.EVENT_BUS_DB = dbPath;
   });
   afterEach(() => {
-    process.env.CELILO_EVENT_BUS_PATH = undefined;
+    process.env.EVENT_BUS_DB = undefined;
     try {
       rmSync(dir, { recursive: true, force: true });
     } catch {

package/src/config/paths.ts

@@ -96,7 +96,8 @@ export function getDbPath(): string {
  * Get the SQLite event-bus database file path.
  *
  * Priority:
- * 1. CELILO_EVENT_BUS_PATH environment variable (explicit override)
+ * 1. EVENT_BUS_DB environment variable (the bus library's native name —
+ *    used by the event-bus CLI, defineHandler, and any standalone tool)
  * 2. <data-dir>/events.db (platform-specific)
  *
  * Kept separate from the main celilo.db so the bus library remains
@@ -105,8 +106,8 @@ export function getDbPath(): string {
  * @returns Absolute path to event-bus database
  */
 export function getEventBusPath(): string {
-  if (process.env.CELILO_EVENT_BUS_PATH) {
-    return process.env.CELILO_EVENT_BUS_PATH;
+  if (process.env.EVENT_BUS_DB) {
+    return process.env.EVENT_BUS_DB;
   }
   return join(getDataDir(), 'events.db');
 }

package/src/module/import.ts

@@ -23,6 +23,25 @@ import {
 import { parseJsonWithValidation } from '../validation/schemas';
 import { cleanupTempDir, extractPackage, verifyPackageIntegrity } from './packaging/extract';
 
+/**
+ * Phase-timing helper for `celilo module import`. Set CELILO_IMPORT_DEBUG=1
+ * to get a phase-by-phase breakdown to stderr — useful when an import is
+ * inexplicably slow (e.g. a Docker bind-mount fsync amplification, a stale
+ * lockfile triggering a fresh bun install, etc.).
+ *
+ * Zero overhead when disabled: the wrapper just calls the inner fn.
+ */
+async function timedPhase<T>(label: string, fn: () => Promise<T> | T): Promise<T> {
+  if (!process.env.CELILO_IMPORT_DEBUG) return await fn();
+  const start = Date.now();
+  try {
+    return await fn();
+  } finally {
+    const ms = Date.now() - start;
+    process.stderr.write(`[import-timing] ${label.padEnd(36)} ${ms}ms\n`);
+  }
+}
+
 /**
  * Module import options
  */
@@ -547,7 +566,7 @@ export async function importModule(options: ModuleImportOptions): Promise<Module
 
     // Execution: Copy files
     try {
-      await copyModuleFiles(actualSourcePath, targetPath);
+      await timedPhase('copyModuleFiles', () => copyModuleFiles(actualSourcePath, targetPath));
     } catch (error) {
       if (tempDir) await cleanupTempDir(tempDir);
       return {
@@ -563,7 +582,9 @@ export async function importModule(options: ModuleImportOptions): Promise<Module
     // If no package.json exists but hook scripts do, auto-generate one with
     // just the framework dep — smooths migration for existing modules.
     try {
-      await installScriptDependencies(targetPath, manifest);
+      await timedPhase('installScriptDependencies', () =>
+        installScriptDependencies(targetPath, manifest),
+      );
     } catch (error) {
       // Non-fatal: the module is importable without deps, but hooks
       // will fail at runtime. Warn and continue.
@@ -574,7 +595,7 @@ export async function importModule(options: ModuleImportOptions): Promise<Module
 
     // Execution: Insert to database
     try {
-      await insertModuleToDb(manifest, targetPath, db);
+      await timedPhase('insertModuleToDb', () => insertModuleToDb(manifest, targetPath, db));
     } catch (error) {
       if (tempDir) await cleanupTempDir(tempDir);
       return {
@@ -586,8 +607,10 @@ export async function importModule(options: ModuleImportOptions): Promise<Module
 
     // Execution: Register capabilities if module provides them
     if (manifest.provides?.capabilities && manifest.provides.capabilities.length > 0) {
-      const { registerModuleCapabilities } = await import('../capabilities/registration');
-      const capResult = await registerModuleCapabilities(manifest.id, manifest, db.$client, flags);
+      const capResult = await timedPhase('registerModuleCapabilities', async () => {
+        const { registerModuleCapabilities } = await import('../capabilities/registration');
+        return registerModuleCapabilities(manifest.id, manifest, db.$client, flags);
+      });
 
       if (!capResult.success) {
         if (tempDir) await cleanupTempDir(tempDir);
@@ -604,8 +627,10 @@ export async function importModule(options: ModuleImportOptions): Promise<Module
     // can re-run the import after fixing the bus, and bus.subscribe is
     // idempotent.
     try {
-      const { registerModuleSubscriptions } = await import('../services/module-subscriptions');
-      registerModuleSubscriptions(manifest, targetPath);
+      await timedPhase('registerModuleSubscriptions', async () => {
+        const { registerModuleSubscriptions } = await import('../services/module-subscriptions');
+        registerModuleSubscriptions(manifest, targetPath);
+      });
     } catch (error) {
       const msg = error instanceof Error ? error.message : String(error);
       log.warn(`Failed to register event-bus subscriptions: ${msg}`);

package/src/services/celilo-events.test.ts

@@ -17,10 +17,10 @@ describe('celilo lifecycle events', () => {
   beforeEach(() => {
     dir = mkdtempSync(join(tmpdir(), 'celilo-events-test-'));
     dbPath = join(dir, 'events.db');
-    process.env.CELILO_EVENT_BUS_PATH = dbPath;
+    process.env.EVENT_BUS_DB = dbPath;
   });
   afterEach(() => {
-    process.env.CELILO_EVENT_BUS_PATH = undefined;
+    process.env.EVENT_BUS_DB = undefined;
     try {
       rmSync(dir, { recursive: true, force: true });
     } catch {
@@ -92,7 +92,7 @@ describe('celilo lifecycle events', () => {
   });
 
   it('does not throw when the bus path is unwritable', () => {
-    process.env.CELILO_EVENT_BUS_PATH = '/proc/no/such/place/events.db';
+    process.env.EVENT_BUS_DB = '/proc/no/such/place/events.db';
     expect(() => emitDeployStarted({ module: 'x', startedAt: 0 })).not.toThrow();
   });
 });

package/src/services/events-daemon.test.ts

@@ -23,7 +23,7 @@ describe('renderSystemdUnit', () => {
     expect(out).toContain(
       'ExecStart=/usr/local/bin/celilo events run --poll-ms 1000 --concurrency 4',
     );
-    expect(out).toContain('Environment=CELILO_EVENT_BUS_PATH=/var/lib/celilo/events.db');
+    expect(out).toContain('Environment=EVENT_BUS_DB=/var/lib/celilo/events.db');
     expect(out).toContain('Restart=on-failure');
     expect(out).toContain('WantedBy=default.target');
   });
@@ -141,9 +141,9 @@ describe('installDaemon / uninstallDaemon roundtrip', () => {
       pollMs: 500,
     });
     const written = readFileSync(second.unitPath, 'utf-8');
-    expect(written).toContain('CELILO_EVENT_BUS_PATH=/db2');
+    expect(written).toContain('EVENT_BUS_DB=/db2');
     expect(written).toContain('--poll-ms 500');
-    expect(written).not.toContain('CELILO_EVENT_BUS_PATH=/db1');
+    expect(written).not.toContain('EVENT_BUS_DB=/db1');
   });
 
   it('uninstall on a missing unit reports not-removed', () => {

package/src/services/events-daemon.ts

@@ -119,7 +119,7 @@ Type=simple
 ExecStart=${input.celiloPath} events run --poll-ms ${input.pollMs} --concurrency ${input.concurrency}
 Restart=on-failure
 RestartSec=10s
-Environment=CELILO_EVENT_BUS_PATH=${input.busDbPath}
+Environment=EVENT_BUS_DB=${input.busDbPath}
 # stdout/stderr are captured by journalctl --user -u celilo-events.service.
 StandardOutput=journal
 StandardError=journal
@@ -151,7 +151,7 @@ export function renderLaunchdPlist(input: UnitInputs): string {
   </array>
   <key>EnvironmentVariables</key>
   <dict>
-    <key>CELILO_EVENT_BUS_PATH</key>
+    <key>EVENT_BUS_DB</key>
     <string>${input.busDbPath}</string>
   </dict>
   <key>RunAtLoad</key>

package/src/services/module-subscriptions.test.ts

@@ -78,10 +78,10 @@ describe('register / unregister roundtrip', () => {
   beforeEach(() => {
     dir = mkdtempSync(join(tmpdir(), 'modsubs-test-'));
     dbPath = join(dir, 'events.db');
-    process.env.CELILO_EVENT_BUS_PATH = dbPath;
+    process.env.EVENT_BUS_DB = dbPath;
   });
   afterEach(() => {
-    process.env.CELILO_EVENT_BUS_PATH = undefined;
+    process.env.EVENT_BUS_DB = undefined;
     try {
       rmSync(dir, { recursive: true, force: true });
     } catch {