refactor: use PI_CODING_AGENT_DIR env var and remove unused subagent infrastructure (#52)

* refactor: use PI_CODING_AGENT_DIR env var for agent configuration paths

* chore: remove closed ticket references and update brainstorm skill documentation

* fixes

* docs: update changelog

Author: kostyay

.tickets/as-0673.md

@@ -90,7 +90,13 @@ All notable changes to agent-stuff are documented here.
 
 
 
-## fix/type-safety-and-race-conditions
+
+
+## refactor/use-agent-dir-env-var
+
+This refactor centralizes agent and session data storage configuration through the `PI_CODING_AGENT_DIR` environment variable, replacing hard-coded `~/.pi` paths across multiple extensions (#52). The change improves portability and allows users to customize storage locations via environment variable, with automatic fallback to `$HOME/.pi/agent` when unset. Documentation and control socket paths in `control.ts`, `sandbox/index.ts`, `session-breakdown.ts`, and `stash.ts` have been updated to reference the new variable. Additionally, closed ticket references and subagent infrastructure (unused agent definitions and test suites) have been removed to reduce codebase complexity.
+
+## [1.0.36](https://github.com/kostyay/agent-stuff/pull/51) - 2026-03-16
 
 Improved type safety and race condition handling across subagent and ticket extensions (#51). The ticket system now retries ID generation on lock conflicts and uses per-ticket locks during garbage collection to prevent concurrent mutation races, while subagent tool calls include explicit type assertions for content blocks. Enhanced tmux session uniqueness by incorporating a directory hash into session names to avoid collisions in multi-workspace scenarios. Renamed the `/plan` command to `/agentic-plan` to clarify its role in the skill-injection planning workflow. Removed approximately 1,300 lines of test scaffolding that was no longer actively maintained.
 

.tickets/as-5a68.md

@@ -3,7 +3,7 @@
  *
  * Enables inter-session communication via Unix domain sockets.  When enabled with
  * the `--session-control` flag, each pi session creates a control socket at
- * `~/.pi/session-control/<session-id>.sock` that accepts JSON-RPC commands.
+ * `$PI_CODING_AGENT_DIR/../session-control/<session-id>.sock` that accepts JSON-RPC commands.
  *
  * Features:
  * - Send messages to other running pi sessions (steer or follow-up mode)
@@ -59,7 +59,10 @@ const CONTROL_SEND_MESSAGE_FLAG = "send-session-message";
 const CONTROL_SEND_MODE_FLAG = "send-session-mode";
 const CONTROL_SEND_WAIT_FLAG = "send-session-wait";
 const CONTROL_SEND_INCLUDE_SENDER_FLAG = "send-session-include-sender-info";
-const CONTROL_DIR = path.join(os.homedir(), ".pi", "session-control");
+const CONTROL_DIR = path.join(
+	path.dirname(process.env.PI_CODING_AGENT_DIR ?? path.join(os.homedir(), ".pi", "agent")),
+	"session-control",
+);
 const SOCKET_SUFFIX = ".sock";
 const SESSION_MESSAGE_TYPE = "session-message";
 const SENDER_INFO_PATTERN = /<sender_info>[\s\S]*?<\/sender_info>/g;
@@ -983,7 +986,7 @@ function shouldRegisterControlTools(pi: ExtensionAPI): boolean {
 
 export default function (pi: ExtensionAPI) {
 	pi.registerFlag(CONTROL_FLAG, {
-		description: "Enable per-session control socket under ~/.pi/session-control",
+		description: "Enable per-session control socket under $PI_CODING_AGENT_DIR/../session-control",
 		type: "boolean",
 	});
 	pi.registerFlag(CONTROL_TARGET_FLAG, {

CHANGELOG.md

@@ -7,7 +7,7 @@
  * - Thinking-level border color derived from mode or model config
  * - Prompt history loaded from current and previous sessions
  *
- * Modes are stored globally (~/.pi/agent/modes.json) or per-project (.pi/modes.json).
+ * Modes are stored globally ($PI_CODING_AGENT_DIR/modes.json) or per-project (.pi/modes.json).
  * Multiple running pi sessions use file-locking to avoid clobbering each other.
  */
 

pi-extensions/control.ts

@@ -6,7 +6,7 @@
  * bubblewrap on Linux).
  *
  * Config files (merged, project takes precedence):
- * - ~/.pi/agent/sandbox.json   (global)
+ * - $PI_CODING_AGENT_DIR/sandbox.json   (global)
  * - <cwd>/.pi/sandbox.json     (project-local)
  *
  * Array fields (allowedDomains, denyRead, etc.) are unioned across layers,
@@ -131,7 +131,7 @@ function mergeConfig(base: SandboxConfig, overrides: Partial<SandboxConfig>): Sa
 }
 
 function loadConfig(cwd: string): SandboxConfig {
-	const globalPath = join(homedir(), ".pi", "agent", "sandbox.json");
+	const globalPath = join(getProfileDir(), "sandbox.json");
 	const projectPath = join(cwd, ".pi", "sandbox.json");
 
 	let config = { ...DEFAULT_CONFIG };
@@ -155,7 +155,7 @@ function loadConfig(cwd: string): SandboxConfig {
 
 /**
  * Resolve the profile directory from PI_CODING_AGENT_DIR env var.
- * Falls back to ~/.pi/agent when unset.
+ * Falls back to $HOME/.pi/agent when unset.
  */
 function getProfileDir(): string {
 	return process.env.PI_CODING_AGENT_DIR ?? join(homedir(), ".pi", "agent");

pi-extensions/prompt-editor.ts

@@ -1,7 +1,7 @@
 /**
  * /session-breakdown
  *
- * Interactive TUI that analyzes ~/.pi/agent/sessions (recursively, *.jsonl) and shows
+ * Interactive TUI that analyzes $PI_CODING_AGENT_DIR/sessions (recursively, *.jsonl) and shows
  * last 7/30/90 days of:
  * - sessions/day
  * - messages/day
@@ -111,7 +111,7 @@ interface BreakdownData {
 	};
 }
 
-const SESSION_ROOT = path.join(os.homedir(), ".pi", "agent", "sessions");
+const SESSION_ROOT = path.join(process.env.PI_CODING_AGENT_DIR ?? path.join(os.homedir(), ".pi", "agent"), "sessions");
 const RANGE_DAYS = [7, 30, 90] as const;
 
 type MeasurementMode = "sessions" | "messages" | "tokens";
@@ -1085,7 +1085,7 @@ class BreakdownComponent implements Component {
 
 export default function sessionBreakdownExtension(pi: ExtensionAPI) {
 	pi.registerCommand("session-breakdown", {
-		description: "Interactive breakdown of last 7/30/90 days of ~/.pi session usage (sessions/messages/tokens + cost by model)",
+		description: "Interactive breakdown of last 7/30/90 days of pi session usage (sessions/messages/tokens + cost by model)",
 		handler: async (_args, ctx: ExtensionContext) => {
 			if (!ctx.hasUI) {
 				// Non-interactive fallback: just notify.

pi-extensions/sandbox/index.ts

@@ -11,7 +11,7 @@
  *
  * Persistence:
  * - Stash survives session switches and restarts.
- * - Stored per-workspace in ~/.pi/stash/<encoded-cwd>.txt
+ * - Stored per-workspace in $PI_CODING_AGENT_DIR/stash/<encoded-cwd>.txt
  *
  * Shows a widget above the editor with a truncated preview of the stashed text.
  */
@@ -22,7 +22,7 @@ import { join } from "node:path";
 
 const WIDGET_ID = "stash";
 const MAX_PREVIEW_LENGTH = 60;
-const STASH_DIR = join(process.env.PI_CODING_AGENT_DIR ?? join(process.env.HOME ?? "", ".pi"), "stash");
+const STASH_DIR = join(process.env.PI_CODING_AGENT_DIR ?? join(process.env.HOME ?? "", ".pi", "agent"), "stash");
 
 /**
  * Build the file path for a workspace's stash file.