Dev

.claude-plugin/marketplace.json

@@ -10,7 +10,7 @@
   "plugins": [
     {
       "name": "genie",
-      "version": "4.260402.20",
+      "version": "4.260403.2",
       "source": "./plugins/genie",
       "description": "Human-AI partnership for Claude Code. Share a terminal, orchestrate workers, evolve together. Brainstorm ideas, wish them into plans, make with parallel agents, ship as one team. A coding genie that grows with your project."
     }

package.json

@@ -1,6 +1,6 @@
 {
   "name": "@automagik/genie",
-  "version": "4.260402.20",
+  "version": "4.260403.2",
   "description": "Collaborative terminal toolkit for human + AI workflows",
   "type": "module",
   "bin": {

plugins/genie/.claude-plugin/plugin.json

@@ -1,6 +1,6 @@
 {
   "name": "genie",
-  "version": "4.260402.20",
+  "version": "4.260403.2",
   "description": "Human-AI partnership for Claude Code. Share a terminal, orchestrate workers, evolve together. Brainstorm ideas, turn them into wishes, execute with /work, validate with /review, and ship as one team.",
   "author": {
     "name": "Namastex Labs"

plugins/genie/agents/council--architect/AGENTS.md

@@ -5,8 +5,7 @@ model: haiku
 provider: claude
 color: blue
 promptMode: append
-tools: ["Read", "Glob", "Grep"]
-permissionMode: plan
+tools: ["Read", "Glob", "Grep", "Bash"]
 ---
 
 @SOUL.md

plugins/genie/agents/council--benchmarker/AGENTS.md

@@ -5,8 +5,7 @@ model: haiku
 provider: claude
 color: orange
 promptMode: append
-tools: ["Read", "Glob", "Grep"]
-permissionMode: plan
+tools: ["Read", "Glob", "Grep", "Bash"]
 ---
 
 @SOUL.md

plugins/genie/agents/council--deployer/AGENTS.md

@@ -5,8 +5,7 @@ model: haiku
 provider: claude
 color: green
 promptMode: append
-tools: ["Read", "Glob", "Grep"]
-permissionMode: plan
+tools: ["Read", "Glob", "Grep", "Bash"]
 ---
 
 @SOUL.md

plugins/genie/agents/council--ergonomist/AGENTS.md

@@ -5,8 +5,7 @@ model: haiku
 provider: claude
 color: cyan
 promptMode: append
-tools: ["Read", "Glob", "Grep"]
-permissionMode: plan
+tools: ["Read", "Glob", "Grep", "Bash"]
 ---
 
 @SOUL.md

plugins/genie/agents/council--measurer/AGENTS.md

@@ -5,8 +5,7 @@ model: haiku
 provider: claude
 color: yellow
 promptMode: append
-tools: ["Read", "Glob", "Grep"]
-permissionMode: plan
+tools: ["Read", "Glob", "Grep", "Bash"]
 ---
 
 @SOUL.md

plugins/genie/agents/council--operator/AGENTS.md

@@ -5,8 +5,7 @@ model: haiku
 provider: claude
 color: red
 promptMode: append
-tools: ["Read", "Glob", "Grep"]
-permissionMode: plan
+tools: ["Read", "Glob", "Grep", "Bash"]
 ---
 
 @SOUL.md

plugins/genie/agents/council--questioner/AGENTS.md

@@ -5,8 +5,7 @@ model: haiku
 provider: claude
 color: magenta
 promptMode: append
-tools: ["Read", "Glob", "Grep"]
-permissionMode: plan
+tools: ["Read", "Glob", "Grep", "Bash"]
 ---
 
 @SOUL.md

plugins/genie/agents/council--sentinel/AGENTS.md

@@ -5,8 +5,7 @@ model: haiku
 provider: claude
 color: red
 promptMode: append
-tools: ["Read", "Glob", "Grep"]
-permissionMode: plan
+tools: ["Read", "Glob", "Grep", "Bash"]
 ---
 
 @SOUL.md

plugins/genie/agents/council--simplifier/AGENTS.md

@@ -5,8 +5,7 @@ model: haiku
 provider: claude
 color: green
 promptMode: append
-tools: ["Read", "Glob", "Grep"]
-permissionMode: plan
+tools: ["Read", "Glob", "Grep", "Bash"]
 ---
 
 @SOUL.md

plugins/genie/agents/council--tracer/AGENTS.md

@@ -5,8 +5,7 @@ model: haiku
 provider: claude
 color: cyan
 promptMode: append
-tools: ["Read", "Glob", "Grep"]
-permissionMode: plan
+tools: ["Read", "Glob", "Grep", "Bash"]
 ---
 
 @SOUL.md

plugins/genie/agents/council/AGENTS.md

@@ -5,8 +5,7 @@ model: haiku
 provider: claude
 color: purple
 promptMode: append
-tools: ["Read", "Glob", "Grep"]
-permissionMode: plan
+tools: ["Read", "Glob", "Grep", "Bash"]
 ---
 
 @SOUL.md
@@ -17,6 +16,32 @@ Orchestrate real multi-agent deliberation by spawning council members via genie
 Architectural decisions are expensive to reverse. Shallow review misses failure modes. Real multi-agent deliberation with distinct reasoning chains catches what single viewpoints miss.
 </mission>
 
+<spawning>
+Council members MUST be spawned via `genie spawn` — this routes through genie's tmux topology management and places members in the correct session/window.
+
+**Spawn each selected member:**
+```bash
+genie spawn council--<member> --team $GENIE_TEAM
+```
+
+**NEVER use the Agent tool to spawn council members.** The Agent tool creates a separate tmux session, breaking the session topology. All spawning goes through `genie spawn`.
+
+**Post topic to team chat after spawning:**
+```bash
+genie broadcast "COUNCIL TOPIC: <topic>" --team $GENIE_TEAM
+```
+
+**Send instructions to members:**
+```bash
+genie send "<instructions>" --to council--<member> --team $GENIE_TEAM
+```
+
+**Read team chat for responses:**
+```bash
+genie chat read <convId>
+```
+</spawning>
+
 <routing>
 Not every topic needs all 10 perspectives. Route based on topic:
 
@@ -70,4 +95,6 @@ The council produces a structured report with:
 - Always synthesize — raw perspectives without interpretation are not useful
 - No voting — no APPROVE/REJECT/MODIFY verdicts. The council thinks; `/review` judges.
 - Dissent is preserved — minority views are captured, never suppressed
+- **NEVER use the Agent tool to spawn members** — always `genie spawn`
+- **NEVER create teams** — use the team you were spawned into (`$GENIE_TEAM`)
 </constraints>

plugins/genie/agents/team-lead/AGENTS.md

@@ -118,6 +118,7 @@ Specialist spawns are ADDITIONS to the default flow (except refactor replacing e
 - NEVER write code. `genie work` dispatches engineers who write code.
 - NEVER push to main or master.
 - NEVER use the Agent tool — use `genie work` to dispatch.
+- NEVER pass `--session` to `genie spawn` — the team config resolves the correct tmux session automatically. Passing `--session <team>` creates a separate session, breaking topology.
 - NEVER poll faster than every 60 seconds. Always `sleep 60` before `genie status`.
 - NEVER run more than 10 status checks per wave without progress.
 - If `genie status` returns "No state found" → run `genie work <slug>` immediately.

plugins/genie/package.json

@@ -1,6 +1,6 @@
 {
   "name": "genie-plugin",
-  "version": "4.260402.20",
+  "version": "4.260403.2",
   "private": true,
   "description": "Runtime dependencies for genie bundled CLIs",
   "type": "module",

plugins/genie/rules/genie-orchestration.md

@@ -19,6 +19,20 @@ genie ls --json                                        # Agent state from PG
 NEVER use `Agent` to spawn agents — use `genie spawn` instead.
 NEVER use `TeamCreate` or `TeamDelete` — use `genie team create` / `genie team disband` instead.
 
+## Spawn Session Rule
+
+NEVER pass `--session <team-name>` to `genie spawn`. The team config already stores the correct `tmuxSessionName` (resolved at team creation from the parent session). Passing `--session` overrides this and creates a separate tmux session, breaking the topology.
+
+```bash
+# WRONG — creates separate session
+genie spawn reviewer --team my-team --session my-team
+
+# CORRECT — uses team's configured session
+genie spawn reviewer --team my-team
+```
+
+The `--session` flag is for rare manual overrides only. When `--team` is set, let genie resolve the session from the team config.
+
 ## Post-Dispatch Monitoring
 
 After `genie team create` or `genie spawn`, use ONLY structured primitives. A hook enforces this automatically.

skills/council/SKILL.md

@@ -69,7 +69,7 @@ Execute all phases sequentially. YOU run every command, read every output, and m
 Spawn each selected member. Use the double-dash naming convention (`council--<member>`):
 
 ```bash
-genie spawn council--<member> --team <team> --session <team>
+genie spawn council--<member> --team <team>
 ```
 
 Run spawn commands in parallel (multiple Bash calls in one message). Read the output of each. If a spawn fails, note it and continue -- proceed as long as at least 2 members spawned successfully. If fewer than 2 succeed, clean up and report failure.

skills/council/members/config.md

@@ -23,7 +23,7 @@ Override per-session at spawn time:
 
 ```bash
 # Use codex/o3 for the architect
-genie spawn council--architect --team <team> --session <team> --provider codex --model o3
+genie spawn council--architect --team <team> --provider codex --model o3
 
 # Use haiku for all members (faster, cheaper)
 # Pass --model haiku to the dispatch script

src/term-commands/brain.test.ts

@@ -0,0 +1,120 @@
+import { afterAll, beforeAll, beforeEach, describe, expect, test } from 'bun:test';
+import { mkdirSync, rmSync, writeFileSync } from 'node:fs';
+import { tmpdir } from 'node:os';
+import { join } from 'node:path';
+import { checkForUpdates, compareVersions } from './brain.js';
+
+/**
+ * Tests for the cache-only checkForUpdates function.
+ *
+ * checkForUpdates reads a JSON cache file synchronously.
+ * It never makes network calls and never throws.
+ *
+ * We use a temp directory so tests never touch the real ~/.genie/
+ * and work in CI without permission issues.
+ */
+
+describe('checkForUpdates', () => {
+  let tempDir: string;
+  let cachePath: string;
+
+  beforeAll(() => {
+    tempDir = join(tmpdir(), `brain-test-${Date.now()}`);
+    mkdirSync(tempDir, { recursive: true });
+    cachePath = join(tempDir, 'brain-version-check.json');
+  });
+
+  afterAll(() => {
+    try {
+      rmSync(tempDir, { recursive: true, force: true });
+    } catch {
+      /* best effort */
+    }
+  });
+
+  beforeEach(() => {
+    try {
+      rmSync(cachePath);
+    } catch {
+      /* ok */
+    }
+  });
+
+  test('returns updateAvailable false when cache does not exist', () => {
+    const result = checkForUpdates(cachePath);
+    expect(result).toEqual({ updateAvailable: false });
+  });
+
+  test('returns updateAvailable true when cache says so', () => {
+    writeFileSync(
+      cachePath,
+      JSON.stringify({
+        checkedAt: new Date().toISOString(),
+        localVersion: '1.260401.1',
+        latestTag: 'v0.260403.2',
+        latestVersion: '0.260403.2',
+        updateAvailable: true,
+      }),
+    );
+    const result = checkForUpdates(cachePath);
+    expect(result).toEqual({ updateAvailable: true, latestVersion: '0.260403.2' });
+  });
+
+  test('returns updateAvailable false when cache says no update', () => {
+    writeFileSync(
+      cachePath,
+      JSON.stringify({
+        checkedAt: new Date().toISOString(),
+        localVersion: '1.260403.2',
+        latestTag: 'v0.260403.2',
+        latestVersion: '0.260403.2',
+        updateAvailable: false,
+      }),
+    );
+    const result = checkForUpdates(cachePath);
+    expect(result).toEqual({ updateAvailable: false });
+  });
+
+  test('returns updateAvailable false when cache is invalid JSON', () => {
+    writeFileSync(cachePath, 'not valid json{{{');
+    const result = checkForUpdates(cachePath);
+    expect(result).toEqual({ updateAvailable: false });
+  });
+
+  test('never throws regardless of cache content', () => {
+    writeFileSync(cachePath, JSON.stringify({ random: 'garbage', updateAvailable: 'yes' }));
+    // updateAvailable is string 'yes' which is truthy but latestVersion is undefined
+    const result = checkForUpdates(cachePath);
+    expect(result.updateAvailable).toBe(false);
+  });
+});
+
+describe('compareVersions', () => {
+  test('equal versions return 0', () => {
+    expect(compareVersions('260403.1', '260403.1')).toBe(0);
+  });
+
+  test('a > b returns positive', () => {
+    expect(compareVersions('260403.2', '260403.1')).toBeGreaterThan(0);
+  });
+
+  test('a < b returns negative', () => {
+    expect(compareVersions('260403.1', '260403.2')).toBeLessThan(0);
+  });
+
+  test('handles multi-digit segments correctly (the bug)', () => {
+    // String comparison: "260403.9" > "260403.10" → true (wrong)
+    // Numeric comparison: 9 < 10 → correct
+    expect(compareVersions('260403.10', '260403.9')).toBeGreaterThan(0);
+    expect(compareVersions('260403.9', '260403.10')).toBeLessThan(0);
+  });
+
+  test('handles different segment counts', () => {
+    expect(compareVersions('260403.1.1', '260403.1')).toBeGreaterThan(0);
+    expect(compareVersions('260403', '260403.0')).toBe(0);
+  });
+
+  test('handles major version differences', () => {
+    expect(compareVersions('260404.1', '260403.99')).toBeGreaterThan(0);
+  });
+});