pi-local-agents-only 0.1.4 → 0.1.6

package/CHANGELOG.md

@@ -0,0 +1,17 @@
+# Changelog
+
+## 0.1.6 - 2026-04-07
+
+- add dev-time static checking with `tsc --noEmit`
+- add a local `check` script that runs tests plus typechecking
+- keep pi core typing support in `devDependencies` only so this stays a lightweight published package
+- annotate the extension with `// @ts-check` and JSDoc types for earlier local error detection
+
+## 0.1.5 - 2026-04-07
+
+- strip already-loaded global `AGENTS.md` / `CLAUDE.md` context reliably instead of rereading live files from disk
+- remove the now-empty `# Project Context` section when only global context was loaded
+- handle prompts whose custom prompt text also mentions the `# Project Context` heading
+- make `/local-agents-only off` report when the repo is still enabled via the global allowlist or `PI_LOCAL_AGENTS_ONLY`
+- document the repo-marker-only behavior of `/local-agents-only off`
+- add integration tests against pi's real system prompt builder and command UX regressions

package/README.md

@@ -30,6 +30,8 @@ Disable for the current repo:
 /local-agents-only off
 ```
 
+`/local-agents-only off` clears the repo marker only. If the repo is still enabled via `/local-agents-only global-on` or `PI_LOCAL_AGENTS_ONLY=1`, it remains enabled until you also run `/local-agents-only global-off` or unset the env var.
+
 Enable or disable via the global allowlist:
 
 ```bash

package/extensions/local-agents-only.js

@@ -1,3 +1,5 @@
+// @ts-check
+
 /**
  * Purpose: Strip pi's global AGENTS.md and CLAUDE.md blocks from the effective prompt for opted-in projects.
  * Responsibilities: Detect repo and worktree opt-in state, manage repo and global toggles, add a local-only guardrail, and remove matching global context blocks before model calls.
@@ -11,12 +13,25 @@ import { existsSync, mkdirSync, readFileSync, rmSync, writeFileSync } from "node
 import { homedir } from "node:os";
 import { dirname, join, resolve } from "node:path";
 
+/** @typedef {import("@mariozechner/pi-coding-agent").ExtensionAPI} ExtensionAPI */
+/** @typedef {import("@mariozechner/pi-coding-agent").ExtensionContext} ExtensionContext */
+/** @typedef {{ projects: string[]; repositories: string[] }} LocalAgentsOnlyConfig */
+/** @typedef {{ start: string; projectRoot: string; repoId: string; worktreeRoots: string[] }} ProjectState */
+/** @typedef {{ enabled: boolean; source: "env" | "marker" | "global-config" | "default" }} Mode */
+/** @typedef {{ path: string; start: number; end: number }} ContextBlock */
+/** @typedef {{ prompt: string; removedPaths: string[] }} StripResult */
+
 const COMMAND = "local-agents-only";
 const MARKER = join(".pi", COMMAND);
 const GLOBAL_CONTEXT_FILES = ["AGENTS.md", "CLAUDE.md"];
 const ENV_TRUE = ["1", "true", "yes", "on"];
 const ENV_FALSE = ["0", "false", "no", "off"];
+const PROJECT_CONTEXT_HEADER = "\n\n# Project Context\n\nProject-specific instructions and guidelines:\n\n";
+const SKILLS_HEADER = "\n\nThe following skills provide specialized instructions for specific tasks.";
+const DATE_HEADER = "\nCurrent date:";
+const CONTEXT_BLOCK_HEADER = /^## ([^\n]+(?:AGENTS|CLAUDE)\.md)\n\n/gm;
 
+/** @returns {string} */
 const getAgentDir = () => {
 	const env = process.env.PI_CODING_AGENT_DIR;
 	if (env === "~") {
@@ -27,10 +42,24 @@ const getAgentDir = () => {
 	}
 	return env || join(homedir(), ".pi", "agent");
 };
+
+/** @param {string} path */
 const normalizePath = (path) => resolve(path).replace(/\\/g, "/");
+
+/** @returns {string} */
 const CONFIG = () => join(getAgentDir(), `${COMMAND}.json`);
+
+/** @param {string} projectRoot */
 const getMarkerPath = (projectRoot) => join(projectRoot, MARKER);
+
+/** @param {string[]} values */
 const uniqueSorted = (values) => [...new Set(values.map(normalizePath))].sort();
+
+/**
+ * @param {string} start
+ * @param {(dir: string) => boolean} predicate
+ * @returns {string | undefined}
+ */
 const walkUp = (start, predicate) => {
 	let current = resolve(start);
 	while (true) {
@@ -39,11 +68,17 @@ const walkUp = (start, predicate) => {
 		}
 		const parent = dirname(current);
 		if (parent === current) {
-			return;
+			return undefined;
 		}
 		current = parent;
 	}
 };
+
+/**
+ * @param {string} start
+ * @param {string[]} args
+ * @returns {string | undefined}
+ */
 const runGit = (start, args) => {
 	try {
 		return execFileSync("git", args, {
@@ -52,20 +87,33 @@ const runGit = (start, args) => {
 			stdio: ["ignore", "pipe", "ignore"],
 		}).trim();
 	} catch {
-		return;
+		return undefined;
 	}
 };
+
+/**
+ * @param {string} [configPath]
+ * @returns {LocalAgentsOnlyConfig}
+ */
 const readConfig = (configPath = CONFIG()) => {
 	try {
-		const { projects = [], repositories = [] } = JSON.parse(readFileSync(configPath, "utf8"));
+		const parsed = /** @type {{ projects?: unknown; repositories?: unknown }} */ (
+			JSON.parse(readFileSync(configPath, "utf8"))
+		);
+		const { projects = [], repositories = [] } = parsed;
 		return {
-			projects: Array.isArray(projects) ? projects.map(normalizePath) : [],
-			repositories: Array.isArray(repositories) ? repositories.map(normalizePath) : [],
+			projects: Array.isArray(projects) ? projects.map((value) => normalizePath(String(value))) : [],
+			repositories: Array.isArray(repositories) ? repositories.map((value) => normalizePath(String(value))) : [],
 		};
 	} catch {
 		return { projects: [], repositories: [] };
 	}
 };
+
+/**
+ * @param {LocalAgentsOnlyConfig} config
+ * @param {string} [configPath]
+ */
 const writeConfig = ({ projects, repositories }, configPath = CONFIG()) => {
 	mkdirSync(dirname(configPath), { recursive: true });
 	writeFileSync(
@@ -80,6 +128,11 @@ const writeConfig = ({ projects, repositories }, configPath = CONFIG()) => {
 		) + "\n",
 	);
 };
+
+/**
+ * @param {string | undefined} [value]
+ * @returns {boolean | undefined}
+ */
 const getEnvToggle = (value = process.env.PI_LOCAL_AGENTS_ONLY) => {
 	const toggle = `${value ?? ""}`.trim().toLowerCase();
 	if (ENV_TRUE.includes(toggle)) {
@@ -88,19 +141,107 @@ const getEnvToggle = (value = process.env.PI_LOCAL_AGENTS_ONLY) => {
 	if (ENV_FALSE.includes(toggle)) {
 		return false;
 	}
+	return undefined;
+};
+
+/** @param {string} [agentDir] */
+const getGlobalContextPaths = (agentDir = getAgentDir()) => GLOBAL_CONTEXT_FILES.map((name) => join(agentDir, name));
+
+/** @param {string} [agentDir] */
+const getExistingGlobalContextPaths = (agentDir = getAgentDir()) =>
+	getGlobalContextPaths(agentDir).filter((path) => existsSync(path));
+
+/**
+ * @param {string} prompt
+ * @param {number} offset
+ * @returns {number}
+ */
+const getContextSectionEnd = (prompt, offset) => {
+	const candidates = [prompt.indexOf(SKILLS_HEADER, offset), prompt.indexOf(DATE_HEADER, offset)].filter(
+		(index) => index !== -1,
+	);
+	return candidates.length > 0 ? Math.min(...candidates) : prompt.length;
+};
+
+/**
+ * @param {string} contextSection
+ * @returns {ContextBlock[]}
+ */
+const getContextBlocks = (contextSection) => {
+	const matches = [...contextSection.matchAll(CONTEXT_BLOCK_HEADER)];
+	return matches.map((match, index) => ({
+		path: match[1],
+		start: match.index ?? 0,
+		end: index + 1 < matches.length ? (matches[index + 1].index ?? contextSection.length) : contextSection.length,
+	}));
+};
+
+/**
+ * @param {string} prompt
+ * @param {string[]} [globalPaths]
+ * @returns {StripResult}
+ */
+const stripGlobalContext = (prompt, globalPaths = getGlobalContextPaths()) => {
+	const sectionStart = prompt.lastIndexOf(PROJECT_CONTEXT_HEADER);
+	if (sectionStart === -1) {
+		return { prompt, removedPaths: [] };
+	}
+	const contextStart = sectionStart + PROJECT_CONTEXT_HEADER.length;
+	const sectionEnd = getContextSectionEnd(prompt, contextStart);
+	const contextSection = prompt.slice(contextStart, sectionEnd);
+	const blocks = getContextBlocks(contextSection);
+	if (blocks.length === 0) {
+		return { prompt, removedPaths: [] };
+	}
+	const globalPathKeys = new Set(globalPaths.map(normalizePath));
+	/** @type {string[]} */
+	const keptBlocks = [];
+	/** @type {string[]} */
+	const removedPaths = [];
+	for (const block of blocks) {
+		const blockText = contextSection.slice(block.start, block.end);
+		if (globalPathKeys.has(normalizePath(block.path))) {
+			removedPaths.push(block.path);
+		} else {
+			keptBlocks.push(blockText);
+		}
+	}
+	if (removedPaths.length === 0) {
+		return { prompt, removedPaths: [] };
+	}
+	const prefix = prompt.slice(0, sectionStart);
+	const suffix = prompt.slice(sectionEnd);
+	if (keptBlocks.length === 0) {
+		return { prompt: `${prefix}${suffix}`, removedPaths: uniqueSorted(removedPaths) };
+	}
+	return {
+		prompt: `${prefix}${PROJECT_CONTEXT_HEADER}${keptBlocks.join("")}${suffix}`,
+		removedPaths: uniqueSorted(removedPaths),
+	};
 };
-const getGlobalContextPaths = (agentDir = getAgentDir()) =>
-	GLOBAL_CONTEXT_FILES.map((name) => join(agentDir, name)).filter((path) => existsSync(path));
-const getGlobalBlocks = (agentDir = getAgentDir()) =>
-	getGlobalContextPaths(agentDir).map((path) => `## ${path}\n\n${readFileSync(path, "utf8")}\n\n`);
+
+/**
+ * @param {string} start
+ * @returns {string | undefined}
+ */
 const getGitTopLevel = (start) => {
 	const topLevel = runGit(start, ["rev-parse", "--show-toplevel"]);
 	return topLevel ? normalizePath(topLevel) : undefined;
 };
+
+/**
+ * @param {string} start
+ * @returns {string | undefined}
+ */
 const getGitCommonDir = (start) => {
 	const commonDir = runGit(start, ["rev-parse", "--git-common-dir"]);
 	return commonDir ? normalizePath(resolve(start, commonDir)) : undefined;
 };
+
+/**
+ * @param {string} start
+ * @returns {string[]}
+ */
 const getWorktreeRoots = (start) => {
 	const list = runGit(start, ["worktree", "list", "--porcelain"]);
 	if (!list) {
@@ -113,6 +254,11 @@ const getWorktreeRoots = (start) => {
 			.map((line) => line.slice("worktree ".length)),
 	);
 };
+
+/**
+ * @param {string} [start]
+ * @returns {ProjectState}
+ */
 const getProjectState = (start = process.cwd()) => {
 	const normalizedStart = normalizePath(start);
 	const gitTopLevel = getGitTopLevel(normalizedStart);
@@ -130,21 +276,33 @@ const getProjectState = (start = process.cwd()) => {
 			worktreeRoots.length > 0 ? uniqueSorted([projectRoot, ...worktreeRoots]) : [normalizePath(projectRoot)],
 	};
 };
+
+/** @param {ProjectState} state */
 const getMarkerRoots = (state) => uniqueSorted([state.projectRoot, ...state.worktreeRoots]);
+
+/** @param {ProjectState} state */
 const hasMarker = (state) => getMarkerRoots(state).some((root) => existsSync(getMarkerPath(root)));
+
+/** @param {ProjectState} state */
 const writeMarkers = (state) => {
 	for (const root of getMarkerRoots(state)) {
 		mkdirSync(dirname(getMarkerPath(root)), { recursive: true });
 		writeFileSync(getMarkerPath(root), "\n");
 	}
 };
+
+/** @param {ProjectState} state */
 const clearMarkers = (state) => {
 	for (const root of getMarkerRoots(state)) {
 		rmSync(getMarkerPath(root), { force: true });
 	}
 };
-const buildLocalOnlyNotice = (agentDir = getAgentDir()) => {
-	const paths = getGlobalContextPaths(agentDir);
+
+/**
+ * @param {string[]} [paths]
+ * @returns {string}
+ */
+const buildLocalOnlyNotice = (paths = getExistingGlobalContextPaths(getAgentDir())) => {
 	if (paths.length === 0) {
 		return "";
 	}
@@ -152,15 +310,25 @@ const buildLocalOnlyNotice = (agentDir = getAgentDir()) => {
 		"# Local Context Mode",
 		"This repo is in local-agents-only mode.",
 		"Ignore instructions from these global context files even if they appear in older session messages, summaries, or retries:",
-		...paths.map((path) => `- ${path}`),
+		...uniqueSorted(paths).map((path) => `- ${path}`),
 		"Follow only repo-local AGENTS.md or CLAUDE.md guidance for this project.",
 	].join("\n");
 };
+
+/**
+ * @param {string} prompt
+ * @param {string} [agentDir]
+ * @returns {string}
+ */
 const applyLocalOnlyPrompt = (prompt, agentDir = getAgentDir()) => {
-	const stripped = stripGlobalBlocks(prompt, getGlobalBlocks(agentDir));
-	const notice = buildLocalOnlyNotice(agentDir);
+	const { prompt: stripped, removedPaths } = stripGlobalContext(prompt, getGlobalContextPaths(agentDir));
+	const notice = buildLocalOnlyNotice(
+		removedPaths.length > 0 ? removedPaths : getExistingGlobalContextPaths(agentDir),
+	);
 	return notice ? `${stripped}\n\n${notice}` : stripped;
 };
+
+/** @param {ExtensionContext} ctx */
 const setStatus = (ctx) => {
 	if (!ctx.hasUI) {
 		return;
@@ -169,10 +337,45 @@ const setStatus = (ctx) => {
 	ctx.ui.setStatus(COMMAND, mode.enabled ? `AGENTS: local-only (${mode.source})` : undefined);
 };
 
+/**
+ * @param {ProjectState} state
+ * @returns {string}
+ */
+const getProjectTarget = (state) =>
+	state.worktreeRoots.length > 1 ? `${state.projectRoot} and linked worktrees` : state.projectRoot;
+
+/**
+ * @param {ProjectState} state
+ * @returns {string}
+ */
+const getOffNotification = (state) => {
+	const mode = getMode(state);
+	if (!mode.enabled) {
+		return `Disabled for ${getProjectTarget(state)}`;
+	}
+	if (mode.source === "global-config") {
+		return `Repo marker cleared for ${getProjectTarget(state)}, but local-agents-only is still enabled via global allowlist. Use /local-agents-only global-off to fully disable it.`;
+	}
+	if (mode.source === "env") {
+		return `Repo marker cleared for ${getProjectTarget(state)}, but local-agents-only is still enabled via PI_LOCAL_AGENTS_ONLY.`;
+	}
+	return `Repo marker cleared for ${getProjectTarget(state)}, but local-agents-only is still enabled via ${mode.source}.`;
+};
+
+/**
+ * @param {string} [start]
+ * @returns {string}
+ */
 export function findProjectRoot(start = process.cwd()) {
 	return getProjectState(start).projectRoot;
 }
 
+/**
+ * @param {string | ProjectState} [start]
+ * @param {string | undefined} [envValue]
+ * @param {string} [configPath]
+ * @returns {Mode}
+ */
 export function getMode(start = process.cwd(), envValue = process.env.PI_LOCAL_AGENTS_ONLY, configPath = CONFIG()) {
 	const state = typeof start === "string" ? getProjectState(start) : start;
 	const envToggle = getEnvToggle(envValue);
@@ -192,10 +395,16 @@ export function getMode(start = process.cwd(), envValue = process.env.PI_LOCAL_A
 	return { enabled: false, source: "default" };
 }
 
-export function stripGlobalBlocks(prompt, blocks = getGlobalBlocks()) {
-	return blocks.reduce((nextPrompt, block) => nextPrompt.replace(block, ""), prompt);
+/**
+ * @param {string} prompt
+ * @param {string[]} [globalPaths]
+ * @returns {string}
+ */
+export function stripGlobalBlocks(prompt, globalPaths = getGlobalContextPaths()) {
+	return stripGlobalContext(prompt, globalPaths).prompt;
 }
 
+/** @param {ExtensionAPI} pi */
 export default function localAgentsOnly(pi) {
 	pi.registerCommand(COMMAND, {
 		description: "Use only repo-local AGENTS prompt context",
@@ -205,12 +414,15 @@ export default function localAgentsOnly(pi) {
 				case "on":
 					writeMarkers(state);
 					setStatus(ctx);
-					ctx.ui.notify(`Enabled for ${state.projectRoot}${state.worktreeRoots.length > 1 ? ` across ${state.worktreeRoots.length} worktrees` : ""}`, "info");
+					ctx.ui.notify(
+						`Enabled for ${state.projectRoot}${state.worktreeRoots.length > 1 ? ` across ${state.worktreeRoots.length} worktrees` : ""}`,
+						"info",
+					);
 					return;
 				case "off":
 					clearMarkers(state);
 					setStatus(ctx);
-					ctx.ui.notify(`Disabled for ${state.projectRoot}${state.worktreeRoots.length > 1 ? ` and linked worktrees` : ""}`, "info");
+					ctx.ui.notify(getOffNotification(state), "info");
 					return;
 				case "global-on": {
 					const config = readConfig();

package/package.json

@@ -1,11 +1,16 @@
 {
   "name": "pi-local-agents-only",
-  "version": "0.1.4",
+  "version": "0.1.6",
   "description": "Pi extension that strips global AGENTS.md and CLAUDE.md from the effective prompt for selected projects.",
   "author": "Mitch Fultz (https://github.com/fitchmultz)",
   "license": "MIT",
   "type": "module",
-  "keywords": ["pi-package", "pi", "pi-extension", "extension"],
+  "keywords": [
+    "pi-package",
+    "pi",
+    "pi-extension",
+    "extension"
+  ],
   "repository": {
     "type": "git",
     "url": "git+https://github.com/fitchmultz/pi-local-agents-only.git"
@@ -14,11 +19,25 @@
     "url": "https://github.com/fitchmultz/pi-local-agents-only/issues"
   },
   "homepage": "https://github.com/fitchmultz/pi-local-agents-only#readme",
-  "files": ["extensions", "README.md", "LICENSE"],
+  "files": [
+    "extensions",
+    "README.md",
+    "CHANGELOG.md",
+    "LICENSE"
+  ],
   "pi": {
-    "extensions": ["./extensions"]
+    "extensions": [
+      "./extensions"
+    ]
   },
   "scripts": {
-    "test": "node --test"
+    "test": "node --test",
+    "typecheck": "tsc --noEmit",
+    "check": "npm test && npm run typecheck"
+  },
+  "devDependencies": {
+    "@mariozechner/pi-coding-agent": "^0.65.2",
+    "@types/node": "^24.3.0",
+    "typescript": "^5.7.3"
   }
 }