@brainjar/cli 0.6.3 → 0.6.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@brainjar/cli",
-  "version": "0.6.3",
+  "version": "0.6.5",
   "description": "Shape how your AI thinks — composable soul, persona, and rules for AI agents",
   "type": "module",
   "license": "MIT",

package/src/client.ts

@@ -153,6 +153,7 @@ export async function createClient(options?: ClientOptions): Promise<BrainjarCli
       throw new IncurError({ code, message, hint })
     }
 
+    if (response.status === 204) return undefined as T
     return response.json() as Promise<T>
   }
 

package/src/commands/persona.ts

@@ -307,8 +307,9 @@ export const persona = Cli.create('persona', {
       const name = normalizeSlug(c.args.name, 'persona name')
       const api = await getApi()
 
+      let result: { affected_brains?: string[] } | undefined
       try {
-        await api.delete(`/api/v1/personas/${name}`)
+        result = await api.delete<{ affected_brains?: string[] }>(`/api/v1/personas/${name}`)
       } catch (e) {
         if (e instanceof IncurError && e.code === ErrorCode.NOT_FOUND) {
           throw createError(ErrorCode.PERSONA_NOT_FOUND, { params: [name] })
@@ -320,7 +321,9 @@ export const persona = Cli.create('persona', {
       const state = await getEffectiveState(api)
       if (state.persona === name) await sync({ api })
 
-      return { deleted: name }
+      const out: Record<string, unknown> = { deleted: name }
+      if (result?.affected_brains?.length) out.affected_brains = result.affected_brains
+      return out
     },
   })
   .command('drop', {

package/src/commands/rules.ts

@@ -247,8 +247,9 @@ export const rules = Cli.create('rules', {
       const name = normalizeSlug(c.args.name, 'rule name')
       const api = await getApi()
 
+      let result: { affected_brains?: string[] } | undefined
       try {
-        await api.delete(`/api/v1/rules/${name}`)
+        result = await api.delete<{ affected_brains?: string[] }>(`/api/v1/rules/${name}`)
       } catch (e) {
         if (e instanceof IncurError && e.code === ErrorCode.NOT_FOUND) {
           throw createError(ErrorCode.RULE_NOT_FOUND, { params: [name] })
@@ -260,7 +261,9 @@ export const rules = Cli.create('rules', {
       const state = await getEffectiveState(api)
       if (state.rules.includes(name)) await sync({ api })
 
-      return { deleted: name }
+      const out: Record<string, unknown> = { deleted: name }
+      if (result?.affected_brains?.length) out.affected_brains = result.affected_brains
+      return out
     },
   })
   .command('drop', {

package/src/commands/soul.ts

@@ -264,8 +264,9 @@ export const soul = Cli.create('soul', {
       const name = normalizeSlug(c.args.name, 'soul name')
       const api = await getApi()
 
+      let result: { affected_brains?: string[] } | undefined
       try {
-        await api.delete(`/api/v1/souls/${name}`)
+        result = await api.delete<{ affected_brains?: string[] }>(`/api/v1/souls/${name}`)
       } catch (e) {
         if (e instanceof IncurError && e.code === ErrorCode.NOT_FOUND) {
           throw createError(ErrorCode.SOUL_NOT_FOUND, { params: [name] })
@@ -277,7 +278,9 @@ export const soul = Cli.create('soul', {
       const state = await getEffectiveState(api)
       if (state.soul === name) await sync({ api })
 
-      return { deleted: name }
+      const out: Record<string, unknown> = { deleted: name }
+      if (result?.affected_brains?.length) out.affected_brains = result.affected_brains
+      return out
     },
   })
   .command('drop', {

package/src/daemon.ts

@@ -1,6 +1,6 @@
 import { spawn, execFileSync } from 'node:child_process'
 import { createHash } from 'node:crypto'
-import { readFile, writeFile, rm, access, open, chmod, mkdir, constants } from 'node:fs/promises'
+import { readFile, writeFile, rm, access, open, chmod, mkdir, rename, copyFile, constants } from 'node:fs/promises'
 import { dirname, join } from 'node:path'
 import { tmpdir } from 'node:os'
 import { Errors } from 'incur'
@@ -31,7 +31,7 @@ const { IncurError } = Errors
  * Minimum server version this CLI is compatible with.
  * Bump when the CLI depends on server features/API changes.
  */
-export const MIN_SERVER_VERSION = '0.2.4'
+export const MIN_SERVER_VERSION = '0.2.7'
 
 export interface HealthStatus {
   healthy: boolean
@@ -229,9 +229,17 @@ export async function downloadAndVerify(binPath: string, versionBase: string): P
       })
     }
 
-    const binContent = await readFile(extractedBin)
-    await writeFile(binPath, binContent)
-    await chmod(binPath, 0o755)
+    // Avoid ETXTBSY: unlink the old binary first (running process
+    // keeps its inode), then move the new one into place.
+    await chmod(extractedBin, 0o755)
+    await rm(binPath, { force: true })
+    try {
+      await rename(extractedBin, binPath)
+    } catch {
+      // Cross-device rename (tmpdir on different fs) — fall back to copy
+      await copyFile(extractedBin, binPath)
+      await chmod(binPath, 0o755)
+    }
   } finally {
     await rm(tmpDir, { recursive: true, force: true })
   }
@@ -275,15 +283,17 @@ export async function upgradeServer(): Promise<{ version: string; alreadyLatest:
     return { version, alreadyLatest: true }
   }
 
-  // Stop server before replacing binary to avoid ETXTBSY on Linux
-  const pid = await readPid(localContext(config).pid_file)
-  if (pid !== null && isAlive(pid)) {
-    await stop()
-  }
-
+  // Binary replacement uses rm + rename/copy which avoids ETXTBSY.
+  // The running server keeps its old inode; next restart picks up the new binary.
   const versionBase = `${DIST_BASE}/${version}`
   await downloadAndVerify(binPath, versionBase)
   await setInstalledServerVersion(version)
+
+  // Restart the server on the new binary
+  await stop()
+  await new Promise(r => setTimeout(r, 1000))
+  await start()
+
   return { version, alreadyLatest: false }
 }
 
@@ -351,7 +361,14 @@ export async function stop(): Promise<{ stopped: boolean }> {
     return { stopped: false }
   }
 
-  process.kill(pid, 'SIGTERM')
+  // Kill entire process group (negative pid) so child processes
+  // like embedded postgres are also terminated.
+  try {
+    process.kill(-pid, 'SIGTERM')
+  } catch {
+    // Process group doesn't exist — try single process
+    try { process.kill(pid, 'SIGTERM') } catch {}
+  }
 
   // Poll for exit, up to 5s
   const deadline = Date.now() + 5000
@@ -363,10 +380,15 @@ export async function stop(): Promise<{ stopped: boolean }> {
     }
   }
 
-  // Force kill
+  // Force kill entire process group
   try {
-    process.kill(pid, 'SIGKILL')
-  } catch {}
+    process.kill(-pid, 'SIGKILL')
+  } catch {
+    try { process.kill(pid, 'SIGKILL') } catch {}
+  }
+
+  // Wait briefly for SIGKILL to take effect
+  await new Promise(r => setTimeout(r, 500))
   await rm(pid_file, { force: true })
   return { stopped: true }
 }