@brainjar/cli 0.6.2 → 0.6.4

package/package.json

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

@@ -167,7 +167,7 @@ export const persona = Cli.create('persona', {
     options: z.object({
       project: z.boolean().default(false).describe('Show project persona override (if any)'),
       short: z.boolean().default(false).describe('Print only the active persona name'),
-      version: z.number().optional().describe('Show a specific version from history'),
+      rev: z.number().optional().describe('Show a specific version from history'),
     }),
     async run(c) {
       const api = await getApi()
@@ -178,11 +178,11 @@ export const persona = Cli.create('persona', {
         return state.persona ?? 'none'
       }
 
-      if (c.options.version) {
+      if (c.options.rev) {
         const name = c.args.name
-        if (!name) throw createError(ErrorCode.MISSING_ARG, { message: 'Name is required when using --version' })
+        if (!name) throw createError(ErrorCode.MISSING_ARG, { message: 'Name is required when using --rev' })
         const slug = normalizeSlug(name, 'persona name')
-        const v = await api.get<ApiContentVersion>(`/api/v1/personas/${slug}/versions/${c.options.version}`)
+        const v = await api.get<ApiContentVersion>(`/api/v1/personas/${slug}/versions/${c.options.rev}`)
         return { name: slug, version: v.version, content: v.content, metadata: v.metadata, created_at: v.created_at }
       }
 
@@ -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

@@ -146,14 +146,14 @@ export const rules = Cli.create('rules', {
       name: z.string().describe('Rule name to show'),
     }),
     options: z.object({
-      version: z.number().optional().describe('Show a specific version from history'),
+      rev: z.number().optional().describe('Show a specific version from history'),
     }),
     async run(c) {
       const name = normalizeSlug(c.args.name, 'rule name')
       const api = await getApi()
 
-      if (c.options.version) {
-        const v = await api.get<ApiContentVersion>(`/api/v1/rules/${name}/versions/${c.options.version}`)
+      if (c.options.rev) {
+        const v = await api.get<ApiContentVersion>(`/api/v1/rules/${name}/versions/${c.options.rev}`)
         const entries = (v.metadata as { entries?: Array<{ sort_key: number; content: string }> })?.entries ?? []
         const content = entries.map(e => e.content.trim()).join('\n\n')
         return { name, version: v.version, content, created_at: v.created_at }
@@ -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

@@ -133,7 +133,7 @@ export const soul = Cli.create('soul', {
     options: z.object({
       project: z.boolean().default(false).describe('Show project soul override (if any)'),
       short: z.boolean().default(false).describe('Print only the active soul name'),
-      version: z.number().optional().describe('Show a specific version from history'),
+      rev: z.number().optional().describe('Show a specific version from history'),
     }),
     async run(c) {
       const api = await getApi()
@@ -144,11 +144,11 @@ export const soul = Cli.create('soul', {
         return state.soul ?? 'none'
       }
 
-      if (c.options.version) {
+      if (c.options.rev) {
         const name = c.args.name
-        if (!name) throw createError(ErrorCode.MISSING_ARG, { message: 'Name is required when using --version' })
+        if (!name) throw createError(ErrorCode.MISSING_ARG, { message: 'Name is required when using --rev' })
         const slug = normalizeSlug(name, 'soul name')
-        const v = await api.get<ApiContentVersion>(`/api/v1/souls/${slug}/versions/${c.options.version}`)
+        const v = await api.get<ApiContentVersion>(`/api/v1/souls/${slug}/versions/${c.options.rev}`)
         return { name: slug, version: v.version, content: v.content, created_at: v.created_at }
       }
 
@@ -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 }
 }