Established single source of truth for state directory name

Author: aponamarev

plugins/code/README.md

@@ -276,7 +276,7 @@ Cleans up session-level artifacts: removes the session workdir mapping file, cle
 
 Runs when any subagent starts. Performs three tasks:
 
-1. **Loop agent state creation**: If the agent type appears in `loop-agents.json`, creates the initial state file in `{WORKDIR}/.closedloop/` if it does not already exist.
+1. **Loop agent state creation**: If the agent type appears in `loop-agents.json`, creates the initial state file in `{WORKDIR}/.closedloop-ai/` if it does not already exist.
 2. **Agent type tracking**: Writes the agent type, short name, and start timestamp to `.agent-types/{agent_id}` so the stop hook can track timing and type.
 3. **Learning injection**: Reads `~/.closedloop-ai/learnings/org-patterns.toon`, filters patterns matching the agent's name, sorts by category priority (mistake > convention > pattern > insight) and confidence, and injects up to 15 patterns into the agent's context via `additionalContext`. Also injects environment variables (`CLOSEDLOOP_WORKDIR`, `CLAUDE_PLUGIN_ROOT`, etc.) into every agent's context.
 
@@ -293,7 +293,7 @@ Runs when any subagent exits. Performs:
 
 Implements the validation loop for agents registered in `loop-agents.json`. When an agent exits:
 
-1. Reads the loop state file (`{WORKDIR}/.closedloop/{state_file_suffix}`)
+1. Reads the loop state file (`{WORKDIR}/.closedloop-ai/{state_file_suffix}`)
 2. Checks whether the agent output contains the expected completion promise (e.g., `<promise>PLAN_VALIDATED</promise>`)
 3. If the promise is present, optionally runs a validation script (e.g., `validate-plan.sh`)
 4. If validation passes, allows the agent to exit (returns nothing)
@@ -346,7 +346,7 @@ Defines the structure of `code-map.json` produced by the `pre-explorer` agent. R
 
 ### `setup-closedloop.sh`
 
-Initializes a ClosedLoop session. Parses arguments (`--prd`, `--max-iterations`, `--prompt`, positional workdir), auto-detects the PRD file by checking common patterns (`prd.md`, `prd.pdf`, `requirements.md`, etc.), establishes the session-to-workdir mapping, validates the prompt name, and writes `{WORKDIR}/.closedloop/config.env` with all environment variables.
+Initializes a ClosedLoop session. Parses arguments (`--prd`, `--max-iterations`, `--prompt`, positional workdir), auto-detects the PRD file by checking common patterns (`prd.md`, `prd.pdf`, `requirements.md`, etc.), establishes the session-to-workdir mapping, validates the prompt name, and writes `{WORKDIR}/.closedloop-ai/config.env` with all environment variables.
 
 ### `run-loop.sh`
 
@@ -485,7 +485,7 @@ After a full run, the work directory will contain:
   log.md                     # Change log appended each phase
   perf.jsonl                 # Agent timing events
   reviews/                   # Critic review files (*.review.json)
-  .closedloop/config.env     # Session environment variables
+  .closedloop-ai/config.env   # Session environment variables
   .learnings/                # Self-learning artifacts
     pending/                 # Unprocessed learning JSON files
     outcomes.log             # Pattern application outcomes

plugins/code/hooks/loop-stop-hook.sh

@@ -6,6 +6,9 @@
 
 set -euo pipefail
 
+# Single source of truth for the state directory name
+CLOSEDLOOP_STATE_DIR=".closedloop-ai"
+
 # Debug logging (redirected to WORKDIR once discovered)
 DEBUG_LOG="/dev/null"
 
@@ -39,15 +42,15 @@ SESSION_ID=$(echo "$HOOK_INPUT" | jq -r '.session_id // empty')
 # Discover WORKDIR via session_id mapping (created by setup-closedloop.sh)
 CLOSEDLOOP_WORKDIR=""
 if [[ -n "$SESSION_ID" ]]; then
-  WORKDIR_FILE="$CWD/.closedloop-ai/session-$SESSION_ID.workdir"
+  WORKDIR_FILE="$CWD/$CLOSEDLOOP_STATE_DIR/session-$SESSION_ID.workdir"
   if [[ -f "$WORKDIR_FILE" ]]; then
     CLOSEDLOOP_WORKDIR=$(cat "$WORKDIR_FILE")
   fi
 fi
 
 # Source closedloop config from WORKDIR if found
 if [[ -n "$CLOSEDLOOP_WORKDIR" ]]; then
-  CLOSEDLOOP_CONFIG="$CLOSEDLOOP_WORKDIR/.closedloop-ai/config.env"
+  CLOSEDLOOP_CONFIG="$CLOSEDLOOP_WORKDIR/$CLOSEDLOOP_STATE_DIR/config.env"
   if [[ -f "$CLOSEDLOOP_CONFIG" ]]; then
     source "$CLOSEDLOOP_CONFIG"
   fi
@@ -90,14 +93,14 @@ STATE_FILE_SUFFIX=$(echo "$AGENT_CONFIG" | jq -r '.state_file_suffix // "loop.lo
 
 echo "$(date): Agent config - validation=$VALIDATION_SCRIPT, max_iter=$MAX_ITERATIONS_DEFAULT, promise=$PROMISE, state_suffix=$STATE_FILE_SUFFIX" >> "$DEBUG_LOG"
 
-# Build state file path (in CLOSEDLOOP_WORKDIR/.closedloop-ai/)
+# Build state file path (in CLOSEDLOOP_WORKDIR/$CLOSEDLOOP_STATE_DIR/)
 # Exit early if CLOSEDLOOP_WORKDIR is not set - no loop context
 if [[ -z "$CLOSEDLOOP_WORKDIR" ]]; then
   echo "$(date): No CLOSEDLOOP_WORKDIR, exiting loop-stop-hook" >> "$DEBUG_LOG"
   exit 0
 fi
 
-STATE_FILE="$CLOSEDLOOP_WORKDIR/.closedloop-ai/$STATE_FILE_SUFFIX"
+STATE_FILE="$CLOSEDLOOP_WORKDIR/$CLOSEDLOOP_STATE_DIR/$STATE_FILE_SUFFIX"
 
 if [[ ! -f "$STATE_FILE" ]]; then
   echo "$(date): No active loop - state file not found: $STATE_FILE" >> "$DEBUG_LOG"

plugins/code/hooks/plan-review.sh

@@ -3,13 +3,16 @@
 # Triggers when Claude exits plan mode to get a second opinion on the plan
 # The plan content is available directly from tool_response.plan
 
+# Single source of truth for the state directory name
+CLOSEDLOOP_STATE_DIR=".closedloop-ai"
+
 # Read hook input from stdin (must happen before CWD extraction)
 INPUT=$(cat)
 
-# Debug logging — keeps at most 15 log files in .closedloop-ai/plan-review-logs/
+# Debug logging — keeps at most 15 log files in $CLOSEDLOOP_STATE_DIR/plan-review-logs/
 CWD=$(echo "$INPUT" | jq -r '.cwd // empty' 2>/dev/null)
 CWD="${CWD:-$PWD}"
-LOG_DIR="$CWD/.closedloop-ai/plan-review-logs"
+LOG_DIR="$CWD/$CLOSEDLOOP_STATE_DIR/plan-review-logs"
 mkdir -p "$LOG_DIR"
 LOG_FILE="$LOG_DIR/$(date +%Y%m%d-%H%M%S).log"
 

plugins/code/hooks/pretooluse-hook.sh

@@ -8,6 +8,9 @@
 
 set -e
 
+# Single source of truth for the state directory name
+CLOSEDLOOP_STATE_DIR=".closedloop-ai"
+
 # Debug logging (redirected to WORKDIR once discovered)
 DEBUG_LOG="/dev/null"
 
@@ -122,7 +125,7 @@ esac
 # Discover WORKDIR via session_id mapping (same pattern as subagent-start-hook.sh)
 CLOSEDLOOP_WORKDIR=""
 if [[ -n "$SESSION_ID" ]]; then
-    WORKDIR_FILE="$CWD/.closedloop-ai/session-$SESSION_ID.workdir"
+    WORKDIR_FILE="$CWD/$CLOSEDLOOP_STATE_DIR/session-$SESSION_ID.workdir"
     if [[ -f "$WORKDIR_FILE" ]]; then
         CLOSEDLOOP_WORKDIR=$(cat "$WORKDIR_FILE")
     fi
@@ -139,7 +142,7 @@ DEBUG_LOG="$CLOSEDLOOP_WORKDIR/.learnings/pretooluse-hook-debug.log"
 echo "$(date): PreToolUse hook started, tool=$TOOL_NAME" >> "$DEBUG_LOG"
 
 # Source closedloop config and skip learning injection if disabled
-CLOSEDLOOP_CONFIG="$CLOSEDLOOP_WORKDIR/.closedloop-ai/config.env"
+CLOSEDLOOP_CONFIG="$CLOSEDLOOP_WORKDIR/$CLOSEDLOOP_STATE_DIR/config.env"
 if [[ -f "$CLOSEDLOOP_CONFIG" ]]; then
     source "$CLOSEDLOOP_CONFIG"
 fi
@@ -148,7 +151,7 @@ if [[ "${CLOSEDLOOP_SELF_LEARNING:-false}" != "true" ]]; then
 fi
 
 # Path to org-patterns.toon
-PATTERNS_FILE="$HOME/.closedloop-ai/learnings/org-patterns.toon"
+PATTERNS_FILE="$HOME/$CLOSEDLOOP_STATE_DIR/learnings/org-patterns.toon"
 
 if [[ ! -f "$PATTERNS_FILE" ]]; then
     echo "$(date): No patterns file found, exiting" >> "$DEBUG_LOG"

plugins/code/hooks/session-end-hook.sh

@@ -4,6 +4,9 @@
 
 set -e
 
+# Single source of truth for the state directory name
+CLOSEDLOOP_STATE_DIR=".closedloop-ai"
+
 # Debug logging (redirected once CWD is known)
 DEBUG_LOG="/dev/null"
 
@@ -17,8 +20,8 @@ REASON=$(echo "$INPUT" | jq -r '.reason // empty')
 
 # Redirect debug logs into project dir (not shared /tmp)
 if [[ -n "$CWD" ]]; then
-    mkdir -p "$CWD/.closedloop-ai"
-    DEBUG_LOG="$CWD/.closedloop-ai/session-end-hook-debug.log"
+    mkdir -p "$CWD/$CLOSEDLOOP_STATE_DIR"
+    DEBUG_LOG="$CWD/$CLOSEDLOOP_STATE_DIR/session-end-hook-debug.log"
 fi
 echo "$(date): Session end hook started, session=$SESSION_ID, reason=$REASON" >> "$DEBUG_LOG"
 
@@ -27,7 +30,7 @@ echo "$(date): Session end hook started, session=$SESSION_ID, reason=$REASON" >>
 # Remove session-specific files from .closedloop-ai/
 # ============================================================================
 
-CLOSEDLOOP_DIR="$CWD/.closedloop-ai"
+CLOSEDLOOP_DIR="$CWD/$CLOSEDLOOP_STATE_DIR"
 
 # Discover CLOSEDLOOP_WORKDIR before cleaning up mappings
 CLOSEDLOOP_WORKDIR=""

plugins/code/hooks/session-start-hook.sh

@@ -5,6 +5,9 @@
 
 set -e
 
+# Single source of truth for the state directory name
+CLOSEDLOOP_STATE_DIR=".closedloop-ai"
+
 # Debug logging (redirected once CWD is known)
 DEBUG_LOG="/dev/null"
 
@@ -19,16 +22,16 @@ if [[ -z "$SESSION_ID" ]] || [[ -z "$CWD" ]]; then
     exit 0
 fi
 
-# Create .closedloop-ai directory at project root
-mkdir -p "$CWD/.closedloop-ai"
+# Create state directory at project root
+mkdir -p "$CWD/$CLOSEDLOOP_STATE_DIR"
 
 # Redirect debug logs into project dir (not shared /tmp)
-DEBUG_LOG="$CWD/.closedloop-ai/session-start-hook-debug.log"
+DEBUG_LOG="$CWD/$CLOSEDLOOP_STATE_DIR/session-start-hook-debug.log"
 echo "$(date): SessionStart hook started, PPID=$PPID" >> "$DEBUG_LOG"
 
 # Write PID -> session_id mapping using Claude Code's PID (our PPID)
 # ! commands will walk up their process tree to find this
-echo "$SESSION_ID" > "$CWD/.closedloop-ai/pid-$PPID.session"
+echo "$SESSION_ID" > "$CWD/$CLOSEDLOOP_STATE_DIR/pid-$PPID.session"
 echo "$(date): Wrote session mapping: pid-$PPID.session -> $SESSION_ID" >> "$DEBUG_LOG"
 
 exit 0

plugins/code/hooks/subagent-start-hook.sh

@@ -4,6 +4,9 @@
 
 set -e
 
+# Single source of truth for the state directory name
+CLOSEDLOOP_STATE_DIR=".closedloop-ai"
+
 # Debug logging (redirected to WORKDIR once discovered)
 DEBUG_LOG="/dev/null"
 
@@ -17,14 +20,14 @@ CWD=$(echo "$INPUT" | jq -r '.cwd // empty')
 SESSION_ID=$(echo "$INPUT" | jq -r '.session_id // empty')
 
 # Early debug log (before WORKDIR discovery) to catch all SubagentStart events
-EARLY_DEBUG_LOG="${CWD:-.}/.closedloop-ai/subagent-start-hook-debug.log"
+EARLY_DEBUG_LOG="${CWD:-.}/$CLOSEDLOOP_STATE_DIR/subagent-start-hook-debug.log"
 mkdir -p "$(dirname "$EARLY_DEBUG_LOG")" 2>/dev/null
 echo "$(date): SubagentStart hook fired — agent_type=$AGENT_TYPE agent_id=$AGENT_ID session_id=$SESSION_ID" >> "$EARLY_DEBUG_LOG"
 
 # Discover WORKDIR via session_id mapping (created by setup-closedloop.sh)
 CLOSEDLOOP_WORKDIR=""
 if [[ -n "$SESSION_ID" ]]; then
-    WORKDIR_FILE="$CWD/.closedloop-ai/session-$SESSION_ID.workdir"
+    WORKDIR_FILE="$CWD/$CLOSEDLOOP_STATE_DIR/session-$SESSION_ID.workdir"
     if [[ -f "$WORKDIR_FILE" ]]; then
         CLOSEDLOOP_WORKDIR=$(cat "$WORKDIR_FILE")
         echo "$(date): Found WORKDIR=$CLOSEDLOOP_WORKDIR from session mapping" >> "$DEBUG_LOG"
@@ -35,7 +38,7 @@ fi
 
 # Source closedloop config from WORKDIR if found
 if [[ -n "$CLOSEDLOOP_WORKDIR" ]]; then
-    CLOSEDLOOP_CONFIG="$CLOSEDLOOP_WORKDIR/.closedloop-ai/config.env"
+    CLOSEDLOOP_CONFIG="$CLOSEDLOOP_WORKDIR/$CLOSEDLOOP_STATE_DIR/config.env"
     if [[ -f "$CLOSEDLOOP_CONFIG" ]]; then
         source "$CLOSEDLOOP_CONFIG"
     fi
@@ -72,13 +75,13 @@ if [[ -f "$LOOP_CONFIG" ]] && [[ -n "$AGENT_TYPE" ]]; then
         MAX_ITERATIONS="${CLOSEDLOOP_MAX_ITERATIONS:-$CONFIG_MAX_ITERATIONS}"
         PRD_FILE="${CLOSEDLOOP_PRD_FILE:-}"
         WORKDIR="${CLOSEDLOOP_WORKDIR:-$CWD}"
-        STATE_FILE="$WORKDIR/.closedloop-ai/$STATE_FILE_SUFFIX"
+        STATE_FILE="$WORKDIR/$CLOSEDLOOP_STATE_DIR/$STATE_FILE_SUFFIX"
 
         echo "$(date): Loop agent detected: $AGENT_TYPE, state_file=$STATE_FILE" >> "$DEBUG_LOG"
 
         # Only create if state file doesn't exist (idempotent)
         if [[ ! -f "$STATE_FILE" ]] && [[ -n "$WORKDIR" ]]; then
-            mkdir -p "$WORKDIR/.closedloop-ai"
+            mkdir -p "$WORKDIR/$CLOSEDLOOP_STATE_DIR"
 
             PROMPT="Create a comprehensive implementation plan for the requirements in @${PRD_FILE}.
 
@@ -124,8 +127,8 @@ if [[ -z "$CLOSEDLOOP_WORKDIR" ]]; then
 fi
 
 # Write base environment (same for all agents, only write once)
-mkdir -p "$CWD/.closedloop-ai"
-BASE_ENV_FILE="$CWD/.closedloop-ai/env"
+mkdir -p "$CWD/$CLOSEDLOOP_STATE_DIR"
+BASE_ENV_FILE="$CWD/$CLOSEDLOOP_STATE_DIR/env"
 if [[ ! -f "$BASE_ENV_FILE" ]]; then
     cat > "$BASE_ENV_FILE" << EOF
 CLOSEDLOOP_WORKDIR=$CLOSEDLOOP_WORKDIR
@@ -176,7 +179,7 @@ fi
 AGENT_NAME="$AGENT_NAME_ONLY"
 
 # Path to org-patterns.toon
-PATTERNS_FILE="$HOME/.closedloop-ai/learnings/org-patterns.toon"
+PATTERNS_FILE="$HOME/$CLOSEDLOOP_STATE_DIR/learnings/org-patterns.toon"
 
 # Only process learnings if we have agent name and patterns file
 if [[ -z "$AGENT_NAME" ]] || [[ ! -f "$PATTERNS_FILE" ]]; then
@@ -424,7 +427,7 @@ if [[ -n "$LEARNINGS" ]]; then
 
 $LEARNINGS"
     # Write learnings to agent-specific file
-    LEARNINGS_FILE="$CWD/.closedloop-ai/learnings-$AGENT_NAME_LOWER"
+    LEARNINGS_FILE="$CWD/$CLOSEDLOOP_STATE_DIR/learnings-$AGENT_NAME_LOWER"
     echo "$LEARNINGS" > "$LEARNINGS_FILE"
     echo "$(date): Wrote learnings to $LEARNINGS_FILE" >> "$DEBUG_LOG"
 fi

plugins/code/hooks/subagent-stop-hook.sh

@@ -5,6 +5,9 @@
 
 set -e
 
+# Single source of truth for the state directory name
+CLOSEDLOOP_STATE_DIR=".closedloop-ai"
+
 # Debug logging (redirected to WORKDIR once discovered)
 DEBUG_LOG="/dev/null"
 
@@ -25,7 +28,7 @@ SESSION_ID=$(echo "$INPUT" | jq -r '.session_id // empty')
 # Discover WORKDIR via session_id mapping (created by setup-closedloop.sh)
 CLOSEDLOOP_WORKDIR=""
 if [[ -n "$SESSION_ID" ]]; then
-    WORKDIR_FILE="$CWD/.closedloop-ai/session-$SESSION_ID.workdir"
+    WORKDIR_FILE="$CWD/$CLOSEDLOOP_STATE_DIR/session-$SESSION_ID.workdir"
     if [[ -f "$WORKDIR_FILE" ]]; then
         CLOSEDLOOP_WORKDIR=$(cat "$WORKDIR_FILE")
         echo "$(date): Found WORKDIR=$CLOSEDLOOP_WORKDIR from session mapping" >> "$DEBUG_LOG"
@@ -36,7 +39,7 @@ fi
 
 # Source closedloop config from WORKDIR if found
 if [[ -n "$CLOSEDLOOP_WORKDIR" ]]; then
-    CLOSEDLOOP_CONFIG="$CLOSEDLOOP_WORKDIR/.closedloop-ai/config.env"
+    CLOSEDLOOP_CONFIG="$CLOSEDLOOP_WORKDIR/$CLOSEDLOOP_STATE_DIR/config.env"
     if [[ -f "$CLOSEDLOOP_CONFIG" ]]; then
         source "$CLOSEDLOOP_CONFIG"
     fi
@@ -179,7 +182,7 @@ if [[ "${CLOSEDLOOP_SELF_LEARNING:-false}" == "true" ]]; then
     # This ensures compute_success_rates.py always has data to work with.
     # ============================================================================
     AGENT_NAME_LOWER=$(echo "$AGENT_NAME" | tr '[:upper:]' '[:lower:]')
-    LEARNINGS_FILE="$CWD/.closedloop-ai/learnings-$AGENT_NAME_LOWER"
+    LEARNINGS_FILE="$CWD/$CLOSEDLOOP_STATE_DIR/learnings-$AGENT_NAME_LOWER"
 
     if [[ -f "$LEARNINGS_FILE" ]]; then
         echo "$(date): Reading injected patterns from $LEARNINGS_FILE" >> "$DEBUG_LOG"

plugins/code/prompts/overlays/README.md

@@ -13,7 +13,7 @@ this order:
 1. If `prompts/<name>.md` exists, use it directly (backward compatible).
 2. Else if `prompts/overlays/<name>.overlay.md` exists, assemble
    `prompts/prompt.md` + blank line + overlay into
-   `$CLOSEDLOOP_WORKDIR/.closedloop/prompt-assembled.md` and point
+   `$CLOSEDLOOP_WORKDIR/.closedloop-ai/prompt-assembled.md` and point
    `CLOSEDLOOP_PROMPT_FILE` at that file.
 3. Else, fail loud with "prompt not found".
 
@@ -51,7 +51,7 @@ does not pass `--prompt` explicitly.
 ## Debugging
 
 - Inspect the assembled file at
-  `$CLOSEDLOOP_WORKDIR/.closedloop/prompt-assembled.md` after a run
+  `$CLOSEDLOOP_WORKDIR/.closedloop-ai/prompt-assembled.md` after a run
   starts.
 - To bypass the overlay, pass `--prompt prompt` — the base is used
   unchanged.

plugins/code/scripts/discover-repos.sh

@@ -4,11 +4,15 @@
 # Output: JSON to stdout
 
 set -e
+
+# Single source of truth for the state directory name
+CLOSEDLOOP_STATE_DIR=".closedloop-ai"
+
 PROJECT_ROOT="${1:-$PWD}"
 PROJECT_ROOT=$(cd "$PROJECT_ROOT" && pwd)
 
 # Read current repo's identity
-CURRENT_IDENTITY="$PROJECT_ROOT/.closedloop-ai/.repo-identity.json"
+CURRENT_IDENTITY="$PROJECT_ROOT/$CLOSEDLOOP_STATE_DIR/.repo-identity.json"
 if [[ -f "$CURRENT_IDENTITY" ]]; then
     CURRENT_NAME=$(jq -r '.name // "unknown"' "$CURRENT_IDENTITY")
     CURRENT_TYPE=$(jq -r '.type // "unknown"' "$CURRENT_IDENTITY")
@@ -51,7 +55,7 @@ if [[ -n "${CLOSEDLOOP_ADD_DIRS:-}" ]]; then
         _mark_seen "$path"
 
         # Read identity if exists, fall back to basename
-        identity_file="$path/.closedloop-ai/.repo-identity.json"
+        identity_file="$path/$CLOSEDLOOP_STATE_DIR/.repo-identity.json"
         repo_name=""
         repo_type="unknown"
         if [[ -f "$identity_file" ]]; then
@@ -86,7 +90,7 @@ if [[ -n "$CLAUDE_WORKSPACE_REPOS" ]]; then
         _mark_seen "$path"
 
         # Read identity if exists
-        identity_file="$path/.closedloop-ai/.repo-identity.json"
+        identity_file="$path/$CLOSEDLOOP_STATE_DIR/.repo-identity.json"
         type="unknown"
         repo_name=""
         if [[ -f "$identity_file" ]]; then
@@ -108,7 +112,7 @@ if [[ -z "$CLAUDE_WORKSPACE_REPOS" ]]; then
         [[ "$sibling" == "$PROJECT_ROOT" ]] && continue
         [[ ! -d "$sibling" ]] && continue
 
-        identity_file="$sibling/.closedloop-ai/.repo-identity.json"
+        identity_file="$sibling/$CLOSEDLOOP_STATE_DIR/.repo-identity.json"
         if [[ -f "$identity_file" ]]; then
             name=$(jq -r '.name // "unknown"' "$identity_file")
             type=$(jq -r '.type // "unknown"' "$identity_file")