pi-landstrip 0.17.38 → 0.17.40

package/dist/index.ts

@@ -192,7 +192,7 @@ function normalizeOpenCode(value, path) {
     result.showLocalAgents = showLocalAgents;
   return result;
 }
-function readConfig(path) {
+function readConfig(path, isGlobal = false) {
   if (!existsSync(path))
     return {};
   let value;
@@ -209,6 +209,14 @@ function readConfig(path) {
       throw new Error(`${path}: unknown top-level field ${key}`);
     }
   }
+  if (!isGlobal) {
+    if ("maxSubagents" in value) {
+      throw new Error(`${path}: maxSubagents is only allowed in global subagents.json`);
+    }
+    if ("opencode" in value) {
+      throw new Error(`${path}: opencode is only allowed in global subagents.json`);
+    }
+  }
   const config = {};
   if ("maxSubagents" in value) {
     if (typeof value.maxSubagents !== "number") {
@@ -256,24 +264,26 @@ function getPiConfigPaths(cwd, fileName, agentDir = getAgentDir()) {
   };
 }
 async function setMaxSubagentsConfigForScope(cwd, maxSubagents, scope, agentDir = getAgentDir()) {
+  if (scope !== "global") {
+    throw new Error("maxSubagents is only allowed in global subagents.json");
+  }
   if (!Number.isInteger(maxSubagents) || maxSubagents < 0 || maxSubagents > MAX_SUBAGENTS) {
     throw new Error(`maxSubagents must be an integer from 0 to ${MAX_SUBAGENTS}`);
   }
-  const { globalPath, projectPath } = getPiConfigPaths(cwd, "subagents.json", agentDir);
-  const path = scope === "project" ? projectPath : globalPath;
-  await withFileMutationQueue(path, async () => {
-    const config = readConfig(path);
+  const { globalPath } = getPiConfigPaths(cwd, "subagents.json", agentDir);
+  await withFileMutationQueue(globalPath, async () => {
+    const config = readConfig(globalPath, true);
     config.maxSubagents = maxSubagents;
-    mkdirSync(dirname(path), { recursive: true });
-    writeFileSync(path, `${JSON.stringify(config, null, 2)}
+    mkdirSync(dirname(globalPath), { recursive: true });
+    writeFileSync(globalPath, `${JSON.stringify(config, null, 2)}
 `, { mode: 384 });
   });
 }
 function loadLandstripConfig(cwd, includeProject = true, agentDir = getAgentDir()) {
   const { globalPath, projectPath } = getPiConfigPaths(cwd, "subagents.json", agentDir);
-  const builtInConfig = readConfig(join2(packageDir, "subagents.json"));
-  const globalConfig = readConfig(globalPath);
-  const projectConfig = includeProject ? readConfig(projectPath) : undefined;
+  const builtInConfig = readConfig(join2(packageDir, "subagents.json"), false);
+  const globalConfig = readConfig(globalPath, true);
+  const projectConfig = includeProject ? readConfig(projectPath, false) : undefined;
   const agentSources = new Map;
   recordAgentSources(agentSources, builtInConfig, "built-in");
   recordAgentSources(agentSources, globalConfig, "global");
@@ -282,7 +292,8 @@ function loadLandstripConfig(cwd, includeProject = true, agentDir = getAgentDir(
   let config = mergeValue(builtInConfig, globalConfig);
   if (projectConfig)
     config = mergeValue(config, projectConfig);
-  if (!Number.isInteger(config.maxSubagents) || (config.maxSubagents ?? -1) < 0 || (config.maxSubagents ?? MAX_SUBAGENTS + 1) > MAX_SUBAGENTS) {
+  const maxSubagents = config.maxSubagents ?? 0;
+  if (!Number.isInteger(maxSubagents) || maxSubagents < 0 || maxSubagents > MAX_SUBAGENTS) {
     throw new Error(`maxSubagents must be an integer from 0 to ${MAX_SUBAGENTS}`);
   }
   if (!isRecord2(config.subagents))
@@ -291,7 +302,7 @@ function loadLandstripConfig(cwd, includeProject = true, agentDir = getAgentDir(
     ...DEFAULT_OPENCODE,
     ...config.opencode
   };
-  return { ...config, opencode, agentSources };
+  return { ...config, maxSubagents, opencode, agentSources };
 }
 
 // opencode-agents.ts

package/dist/subagents.json

@@ -1,5 +1,4 @@
 {
-  "maxSubagents": 0,
   "subagents": {
     "agent": {
       "build": {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-landstrip",
-  "version": "0.17.38",
+  "version": "0.17.40",
   "description": "Sandbox-aware Pi subagents and interactive command permissions",
   "keywords": [
     "landstrip",
@@ -40,7 +40,7 @@
     "ci:test": "npm run test"
   },
   "dependencies": {
-    "@landstrip/landstrip": "^0.17.38",
+    "@landstrip/landstrip": "^0.17.40",
     "minimatch": "^10.2.5",
     "yaml": "^2.9.0"
   },

package/subagents.json

@@ -1,5 +1,4 @@
 {
-  "maxSubagents": 0,
   "subagents": {
     "agent": {
       "build": {