From 143feda0644d2289954b694f3ce4cee2fc74b808 Mon Sep 17 00:00:00 2001
From: Craig Jennings <c@cjennings.net>
Date: Sat, 30 May 2026 21:48:13 -0500
Subject: feat(session-context): resolve the active path per AI_AGENT_ID

A single .ai/session-context.org races when two agents share a project: each agent's writes clobber the other's session log. I added .ai/scripts/session-context-path, which resolves the active path from AI_AGENT_ID: unset gives the legacy .ai/session-context.org singleton (so every existing one-agent session is unchanged), set gives .ai/session-context.d/<id>.org with the id sanitized to filename-safe characters. This is Codex's Phase 1 slice from the runtime-neutral spec: the race fix on its own, no broader refactor.

startup.org's existence check and wrap-it-up.org's rename now resolve through the helper, each with a singleton fallback so older checkouts that haven't synced the script still work. Wrap folds the agent id into the archive name so two agents wrapping in the same minute don't collide. protocols.org documents the rule. Verified with 5 bats cases and a two-agent simulation showing distinct paths per id.
---
 .ai/scripts/tests/session-context-path.bats | 40 +++++++++++++++++++++++++++++
 1 file changed, 40 insertions(+)
 create mode 100644 .ai/scripts/tests/session-context-path.bats

(limited to '.ai/scripts/tests/session-context-path.bats')

diff --git a/.ai/scripts/tests/session-context-path.bats b/.ai/scripts/tests/session-context-path.bats
new file mode 100644
index 0000000..ea8937d
--- /dev/null
+++ b/.ai/scripts/tests/session-context-path.bats
@@ -0,0 +1,40 @@
+#!/usr/bin/env bats
+# Tests for the session-context-path helper: resolve the active session-context
+# path from AI_AGENT_ID, defaulting to the legacy singleton.
+
+setup() {
+    SCRIPT_DIR="$(cd "$(dirname "$BATS_TEST_FILENAME")/.." && pwd)"
+    SCP="$SCRIPT_DIR/session-context-path"
+}
+
+@test "session-context-path: no AI_AGENT_ID gives the legacy singleton" {
+    run env -u AI_AGENT_ID "$SCP"
+    [ "$status" -eq 0 ]
+    [ "$output" = ".ai/session-context.org" ]
+}
+
+@test "session-context-path: empty AI_AGENT_ID gives the legacy singleton" {
+    AI_AGENT_ID="" run "$SCP"
+    [ "$status" -eq 0 ]
+    [ "$output" = ".ai/session-context.org" ]
+}
+
+@test "session-context-path: a set AI_AGENT_ID gives a per-agent file" {
+    AI_AGENT_ID="pearl.org-drill.claude.a83f" run "$SCP"
+    [ "$status" -eq 0 ]
+    [ "$output" = ".ai/session-context.d/pearl.org-drill.claude.a83f.org" ]
+}
+
+@test "session-context-path: two distinct ids resolve to distinct files" {
+    a=$(AI_AGENT_ID="claude" "$SCP")
+    b=$(AI_AGENT_ID="local-qwen" "$SCP")
+    [ "$a" = ".ai/session-context.d/claude.org" ]
+    [ "$b" = ".ai/session-context.d/local-qwen.org" ]
+    [ "$a" != "$b" ]
+}
+
+@test "session-context-path: unsafe id characters are sanitized" {
+    AI_AGENT_ID="a b/c" run "$SCP"
+    [ "$status" -eq 0 ]
+    [ "$output" = ".ai/session-context.d/a_b_c.org" ]
+}
-- 
cgit v1.2.3