@brainjar/cli 0.6.3 → 0.6.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@brainjar/cli",
-  "version": "0.6.3",
+  "version": "0.6.4",
   "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

@@ -279,6 +279,18 @@ export async function upgradeServer(): Promise<{ version: string; alreadyLatest:
   const pid = await readPid(localContext(config).pid_file)
   if (pid !== null && isAlive(pid)) {
     await stop()
+
+    // Verify process is actually dead before replacing binary
+    const deadline = Date.now() + 3000
+    while (Date.now() < deadline && isAlive(pid)) {
+      await new Promise(r => setTimeout(r, 100))
+    }
+    if (isAlive(pid)) {
+      throw createError(ErrorCode.SERVER_START_FAILED, {
+        message: `Server process (PID ${pid}) is still running. Cannot replace binary.`,
+        hint: `Kill it manually: kill -9 ${pid}`,
+      })
+    }
   }
 
   const versionBase = `${DIST_BASE}/${version}`
@@ -351,7 +363,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 +382,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 }
 }