@donkeylabs/server 0.5.1 → 0.6.3

package/docs/router.md

@@ -304,6 +304,99 @@ router.route("getUser").typed({
 
 ---
 
+## Type Generation
+
+When using `donkeylabs generate` to create a typed API client, **you must provide explicit `output` schemas if your route returns data**.
+
+### Output Schema Rules
+
+- **No `output` schema** → Generated type is `void` (handler should return nothing)
+- **With `output` schema** → Generated type matches the schema
+
+This enforces explicitness: if you want to return data, you must declare what you're returning.
+
+**Without `output` schema (returns void):**
+```ts
+// Handler should NOT return anything
+router.route("delete").typed({
+  input: z.object({ id: z.string() }),
+  handle: async (input, ctx) => {
+    await ctx.plugins.recordings.delete(input.id);
+    // No return - this is correct for void output
+  },
+});
+```
+
+**With `output` schema (returns data):**
+```ts
+// ✅ Generated type will be: Output = Expand<{ recordings: Recording[]; total: number; }>
+router.route("list").typed({
+  input: z.object({ page: z.number() }),
+  output: z.object({
+    recordings: z.array(RecordingSchema),
+    total: z.number(),
+  }),
+  handle: async (input, ctx) => {
+    return ctx.plugins.recordings.list(input);
+  },
+});
+```
+
+### Best Practice: Always Define Output Schemas
+
+For proper type safety in generated clients:
+
+1. **Define Zod schemas for outputs**:
+```ts
+// schemas.ts
+export const RecordingSchema = z.object({
+  id: z.string(),
+  name: z.string(),
+  duration: z.number(),
+  createdAt: z.string(),
+});
+
+export const RecordingListOutput = z.object({
+  recordings: z.array(RecordingSchema),
+  total: z.number(),
+  page: z.number(),
+});
+```
+
+2. **Use them in routes**:
+```ts
+import { RecordingListOutput } from "./schemas";
+
+router.route("list").typed({
+  input: z.object({ page: z.number().default(1) }),
+  output: RecordingListOutput,
+  handle: async (input, ctx) => {
+    return ctx.plugins.recordings.list(input);
+  },
+});
+```
+
+3. **Run type generation**:
+```bash
+donkeylabs generate
+```
+
+The generated client will have properly typed methods:
+```ts
+// Generated API client
+api.recordings.list({ page: 1 })  // Returns Promise<{ recordings: Recording[]; total: number; page: number; }>
+```
+
+### Debugging Missing Types
+
+If your generated client shows `Output = Expand<void>` but your handler returns data:
+
+1. Add an explicit `output` Zod schema that matches your return type
+2. Run `donkeylabs generate` to regenerate the client
+3. Check the warning logs - routes without output schemas are listed
+
+---
+
 ## Real-World Examples
 
 ### CRUD Operations

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@donkeylabs/server",
-  "version": "0.5.1",
+  "version": "0.6.3",
   "type": "module",
   "description": "Type-safe plugin system for building RPC-style APIs with Bun",
   "main": "./src/index.ts",
@@ -75,7 +75,7 @@
   ],
   "repository": {
     "type": "git",
-    "url": "https://github.com/donkeylabs/server"
+    "url": "https://github.com/donkeylabs-io/donkeylabs"
   },
   "license": "MIT"
 }

package/src/core/index.ts

@@ -146,3 +146,27 @@ export {
   workflow,
   createWorkflows,
 } from "./workflows";
+
+export {
+  type Processes,
+  type ProcessesConfig,
+  type ProcessStatus,
+  type ProcessConfig,
+  type ProcessDefinition,
+  type ManagedProcess,
+  type SpawnOptions,
+  createProcesses,
+} from "./processes";
+
+export {
+  SqliteProcessAdapter,
+  type SqliteProcessAdapterConfig,
+  type ProcessAdapter,
+} from "./process-adapter-sqlite";
+
+export {
+  type ProcessSocketServer,
+  type ProcessMessage,
+  type ProcessSocketConfig,
+  createProcessSocketServer,
+} from "./process-socket";

package/src/core/process-adapter-sqlite.ts

@@ -0,0 +1,282 @@
+/**
+ * Built-in SQLite Process Adapter
+ *
+ * Provides automatic persistence for managed processes, enabling server restart resilience
+ * and orphan recovery without requiring user configuration.
+ */
+
+import { Database } from "bun:sqlite";
+import { mkdir } from "node:fs/promises";
+import { dirname } from "node:path";
+import type { ProcessStatus, ManagedProcess, ProcessConfig } from "./processes";
+
+export interface SqliteProcessAdapterConfig {
+  /** Path to SQLite database file (default: .donkeylabs/processes.db) */
+  path?: string;
+  /** Auto-cleanup stopped processes older than N days (default: 7, 0 to disable) */
+  cleanupDays?: number;
+  /** Cleanup interval in ms (default: 3600000 = 1 hour) */
+  cleanupInterval?: number;
+}
+
+export interface ProcessAdapter {
+  /** Create a new process record */
+  create(process: Omit<ManagedProcess, "id">): Promise<ManagedProcess>;
+  /** Get a process by ID */
+  get(processId: string): Promise<ManagedProcess | null>;
+  /** Update a process record */
+  update(processId: string, updates: Partial<ManagedProcess>): Promise<void>;
+  /** Delete a process record */
+  delete(processId: string): Promise<boolean>;
+  /** Get all processes by name */
+  getByName(name: string): Promise<ManagedProcess[]>;
+  /** Get all running processes */
+  getRunning(): Promise<ManagedProcess[]>;
+  /** Get orphaned processes (status="running" from before crash) */
+  getOrphaned(): Promise<ManagedProcess[]>;
+  /** Stop the adapter and cleanup timer */
+  stop(): void;
+}
+
+export class SqliteProcessAdapter implements ProcessAdapter {
+  private db: Database;
+  private initialized = false;
+  private cleanupTimer?: ReturnType<typeof setInterval>;
+  private cleanupDays: number;
+
+  constructor(config: SqliteProcessAdapterConfig = {}) {
+    const dbPath = config.path ?? ".donkeylabs/processes.db";
+    this.cleanupDays = config.cleanupDays ?? 7;
+
+    // Ensure directory exists
+    this.ensureDir(dbPath);
+
+    this.db = new Database(dbPath);
+    this.init();
+
+    // Start cleanup timer
+    if (this.cleanupDays > 0) {
+      const interval = config.cleanupInterval ?? 3600000; // 1 hour
+      this.cleanupTimer = setInterval(() => this.cleanup(), interval);
+      // Run cleanup on startup
+      this.cleanup();
+    }
+  }
+
+  private ensureDir(dbPath: string): void {
+    const dir = dirname(dbPath);
+    if (dir && dir !== ".") {
+      // Sync mkdir for constructor
+      try {
+        Bun.spawnSync(["mkdir", "-p", dir]);
+      } catch {
+        // Directory may already exist
+      }
+    }
+  }
+
+  private init(): void {
+    if (this.initialized) return;
+
+    this.db.run(`
+      CREATE TABLE IF NOT EXISTS processes (
+        id TEXT PRIMARY KEY,
+        name TEXT NOT NULL,
+        pid INTEGER,
+        socket_path TEXT,
+        tcp_port INTEGER,
+        status TEXT NOT NULL DEFAULT 'stopped',
+        config TEXT NOT NULL,
+        metadata TEXT,
+        created_at TEXT NOT NULL,
+        started_at TEXT,
+        stopped_at TEXT,
+        last_heartbeat TEXT,
+        restart_count INTEGER DEFAULT 0,
+        consecutive_failures INTEGER DEFAULT 0,
+        error TEXT
+      )
+    `);
+
+    // Indexes for efficient queries
+    this.db.run(`CREATE INDEX IF NOT EXISTS idx_processes_status ON processes(status)`);
+    this.db.run(`CREATE INDEX IF NOT EXISTS idx_processes_name ON processes(name)`);
+    this.db.run(`CREATE INDEX IF NOT EXISTS idx_processes_name_status ON processes(name, status)`);
+
+    this.initialized = true;
+  }
+
+  async create(process: Omit<ManagedProcess, "id">): Promise<ManagedProcess> {
+    const id = `proc_${Date.now()}_${Math.random().toString(36).slice(2, 9)}`;
+
+    this.db.run(
+      `INSERT INTO processes (
+        id, name, pid, socket_path, tcp_port, status, config, metadata,
+        created_at, started_at, stopped_at, last_heartbeat,
+        restart_count, consecutive_failures, error
+      ) VALUES (?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?, ?)`,
+      [
+        id,
+        process.name,
+        process.pid ?? null,
+        process.socketPath ?? null,
+        process.tcpPort ?? null,
+        process.status,
+        JSON.stringify(process.config),
+        process.metadata ? JSON.stringify(process.metadata) : null,
+        process.createdAt.toISOString(),
+        process.startedAt?.toISOString() ?? null,
+        process.stoppedAt?.toISOString() ?? null,
+        process.lastHeartbeat?.toISOString() ?? null,
+        process.restartCount ?? 0,
+        process.consecutiveFailures ?? 0,
+        process.error ?? null,
+      ]
+    );
+
+    return { ...process, id };
+  }
+
+  async get(processId: string): Promise<ManagedProcess | null> {
+    const row = this.db.query(`SELECT * FROM processes WHERE id = ?`).get(processId) as any;
+    if (!row) return null;
+    return this.rowToProcess(row);
+  }
+
+  async update(processId: string, updates: Partial<ManagedProcess>): Promise<void> {
+    const sets: string[] = [];
+    const values: any[] = [];
+
+    if (updates.pid !== undefined) {
+      sets.push("pid = ?");
+      values.push(updates.pid);
+    }
+    if (updates.socketPath !== undefined) {
+      sets.push("socket_path = ?");
+      values.push(updates.socketPath);
+    }
+    if (updates.tcpPort !== undefined) {
+      sets.push("tcp_port = ?");
+      values.push(updates.tcpPort);
+    }
+    if (updates.status !== undefined) {
+      sets.push("status = ?");
+      values.push(updates.status);
+    }
+    if (updates.config !== undefined) {
+      sets.push("config = ?");
+      values.push(JSON.stringify(updates.config));
+    }
+    if (updates.metadata !== undefined) {
+      sets.push("metadata = ?");
+      values.push(updates.metadata ? JSON.stringify(updates.metadata) : null);
+    }
+    if (updates.startedAt !== undefined) {
+      sets.push("started_at = ?");
+      values.push(updates.startedAt?.toISOString() ?? null);
+    }
+    if (updates.stoppedAt !== undefined) {
+      sets.push("stopped_at = ?");
+      values.push(updates.stoppedAt?.toISOString() ?? null);
+    }
+    if (updates.lastHeartbeat !== undefined) {
+      sets.push("last_heartbeat = ?");
+      values.push(updates.lastHeartbeat?.toISOString() ?? null);
+    }
+    if (updates.restartCount !== undefined) {
+      sets.push("restart_count = ?");
+      values.push(updates.restartCount);
+    }
+    if (updates.consecutiveFailures !== undefined) {
+      sets.push("consecutive_failures = ?");
+      values.push(updates.consecutiveFailures);
+    }
+    if (updates.error !== undefined) {
+      sets.push("error = ?");
+      values.push(updates.error);
+    }
+
+    if (sets.length === 0) return;
+
+    values.push(processId);
+    this.db.run(`UPDATE processes SET ${sets.join(", ")} WHERE id = ?`, values);
+  }
+
+  async delete(processId: string): Promise<boolean> {
+    const result = this.db.run(`DELETE FROM processes WHERE id = ?`, [processId]);
+    return result.changes > 0;
+  }
+
+  async getByName(name: string): Promise<ManagedProcess[]> {
+    const rows = this.db
+      .query(`SELECT * FROM processes WHERE name = ? ORDER BY created_at DESC`)
+      .all(name) as any[];
+    return rows.map((r) => this.rowToProcess(r));
+  }
+
+  async getRunning(): Promise<ManagedProcess[]> {
+    const rows = this.db
+      .query(`SELECT * FROM processes WHERE status = 'running' OR status = 'spawning'`)
+      .all() as any[];
+    return rows.map((r) => this.rowToProcess(r));
+  }
+
+  async getOrphaned(): Promise<ManagedProcess[]> {
+    // Get processes that were running or spawning when server died
+    const rows = this.db
+      .query(
+        `SELECT * FROM processes WHERE status IN ('running', 'spawning', 'orphaned')`
+      )
+      .all() as any[];
+    return rows.map((r) => this.rowToProcess(r));
+  }
+
+  private rowToProcess(row: any): ManagedProcess {
+    return {
+      id: row.id,
+      name: row.name,
+      pid: row.pid ?? undefined,
+      socketPath: row.socket_path ?? undefined,
+      tcpPort: row.tcp_port ?? undefined,
+      status: row.status as ProcessStatus,
+      config: JSON.parse(row.config),
+      metadata: row.metadata ? JSON.parse(row.metadata) : undefined,
+      createdAt: new Date(row.created_at),
+      startedAt: row.started_at ? new Date(row.started_at) : undefined,
+      stoppedAt: row.stopped_at ? new Date(row.stopped_at) : undefined,
+      lastHeartbeat: row.last_heartbeat ? new Date(row.last_heartbeat) : undefined,
+      restartCount: row.restart_count ?? 0,
+      consecutiveFailures: row.consecutive_failures ?? 0,
+      error: row.error ?? undefined,
+    };
+  }
+
+  /** Clean up old stopped/crashed processes */
+  private cleanup(): void {
+    if (this.cleanupDays <= 0) return;
+
+    try {
+      const cutoff = new Date();
+      cutoff.setDate(cutoff.getDate() - this.cleanupDays);
+
+      const result = this.db.run(
+        `DELETE FROM processes WHERE status IN ('stopped', 'crashed', 'dead') AND stopped_at < ?`,
+        [cutoff.toISOString()]
+      );
+
+      if (result.changes > 0) {
+        console.log(`[Processes] Cleaned up ${result.changes} old process records`);
+      }
+    } catch (err) {
+      console.error("[Processes] Cleanup error:", err);
+    }
+  }
+
+  /** Stop the adapter and cleanup timer */
+  stop(): void {
+    if (this.cleanupTimer) {
+      clearInterval(this.cleanupTimer);
+      this.cleanupTimer = undefined;
+    }
+  }
+}