pennyrouter 0.2.29 → 0.2.31

package/README.md

@@ -56,9 +56,9 @@ the user explicitly changes it. Close running Claude Code or Codex sessions only
 removing global bare-command routing. A Penny-only install does not require closing native sessions.
 
 The launcher uses a private config, a Claude settings overlay, and a secret-free Codex profile.
-The installer adds one marked PATH block to zsh or bash startup files on macOS/Linux, or a single
-user-PATH entry on Windows, when needed; `--no-path-update` skips that edit. Open a new terminal
-after a PATH update.
+The installer adds marked PATH blocks to the interactive and login startup files for zsh or bash
+on macOS/Linux, or a single user-PATH entry on Windows, when needed; `--no-path-update` skips that
+edit. Open a new terminal after a PATH update.
 
 In Claude Code sessions launched with `penny claude`, `/penny help` is an alias for the existing
 `@penny help` syntax. Both forms remain supported. Codex continues to use `@penny` because Codex
@@ -112,8 +112,8 @@ forward. Use `install` instead when the intent is to change those choices.
 
 Because the managed tool config is rewritten only for tools whose bare command already routes
 through PennyRouter, a Penny-only install leaves files such as `~/.claude/settings.json`
-untouched and needs no session shutdown. Updating a tool that owns global routing requires
-closing its running sessions, or passing `--allow-running-harnesses`. Unrelated keys and user
+untouched and needs no session shutdown. Updates do not inspect or require stopping running
+harness processes. Unrelated keys and user
 settings inside managed config files are preserved, and re-running `update` is idempotent.
 
 Restart Claude Code or Codex afterwards to pick up the refreshed configuration.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pennyrouter",
-  "version": "0.2.29",
+  "version": "0.2.31",
   "description": "Install and manage PennyRouter local coding-agent integrations.",
   "homepage": "https://pennyrouter.com",
   "bugs": {

package/src/cli.js

@@ -364,7 +364,6 @@ async function authProvider(flags) {
   const ownsGlobalRouting = existingRecord
     ? existingRecord.default_routing !== false
     : Boolean(await readClaudeCodePennyRouterKey());
-  if (ownsGlobalRouting) assertNoRunningHarness("claude-code", flags);
   await storeAnthropicToken({ gatewayBaseUrl, apiKey: pennyKey, token });
   await probeClaudeCodeGateway({ gatewayBaseUrl, apiKey: pennyKey });
   await installPennyRuntime({ updatePath: !flags.noPathUpdate });
@@ -633,9 +632,6 @@ export async function install(flags = {}) {
       const supportsPennyCommand = ["claude-code", "codex", "opencode"].includes(harness.id);
       const makeDefault = defaultRouting.get(harness.id) ?? true;
       const hadGlobalRouting = Boolean(existingRecord && existingRecord.default_routing !== false);
-      if (["claude-code", "codex"].includes(harness.id) && (makeDefault || hadGlobalRouting)) {
-        assertNoRunningHarness(harness.id, flags);
-      }
 
       const harnessNeededAuth = needsAuth.includes(harness);
       const codexSubscriptionAuth = harness.id === "codex"
@@ -730,7 +726,7 @@ export async function install(flags = {}) {
       console.log(color(`How to Run ${harnessName}`, "bold"));
       console.log("");
       console.log(`  ${color(command.padEnd(commandWidth), "bold")}→ ${harnessName} with PennyRouter`);
-      console.log(`  ${color(native.padEnd(commandWidth), "bold")}→ ${record.default_routing ? `${harnessName} with PennyRouter (default routing enabled)` : `${harnessName} native`}`);
+      console.log(`  ${color(native.padEnd(commandWidth), "bold")}→ ${record.default_routing ? `${harnessName}` : `${harnessName} native`}`);
     } else {
       console.log(`Installed on ${harness?.name || record.harness}${installNoteForRecord(record)}.`);
     }
@@ -947,7 +943,6 @@ async function update(flags) {
       // rewrite — i.e. the bare command still routes through the gateway. In penny-only
       // mode the harness config belongs to the user and we touch just the launch profile.
       const ownsGlobalRouting = !record.launch_available || record.default_routing !== false;
-      if (ownsGlobalRouting) assertNoRunningHarness(record.harness, flags);
 
       const claudeSubscriptionAuth = record.harness === "claude-code"
         && Boolean(record.anthropic_token_stored);
@@ -2032,53 +2027,12 @@ function openBrowser(url) {
   spawnSync(command, args, { stdio: "ignore", detached: true });
 }
 
-function runningClaudeCodeProcesses() {
-  if (process.platform === "win32") {
-    const result = spawnSync("tasklist", ["/FO", "CSV", "/NH"], { encoding: "utf8" });
-    if (result.status !== 0) return [];
-    return String(result.stdout || "").split(/\r?\n/)
-      .filter((line) => /^"claude(?:\.exe)?"/i.test(line.trim()));
-  }
-  const result = spawnSync("ps", ["-ax", "-o", "pid=,command="], { encoding: "utf8" });
-  if (result.status !== 0) return [];
-  return String(result.stdout || "").split(/\r?\n/)
-    .map((line) => line.trim())
-    // Case-sensitive on purpose: match the Claude Code executable, not Claude.app.
-    .filter((line) => /(^|[\/\s])claude(?:\s|$)/.test(line));
-}
-
-function runningCodexProcesses() {
-  if (process.platform === "win32") return [];
-  const result = spawnSync("ps", ["-ax", "-o", "pid=,command="], { encoding: "utf8" });
-  if (result.status !== 0) return [];
-  return String(result.stdout || "").split(/\r?\n/)
-    .map((line) => line.trim())
-    .filter((line) => /(^|[\/\s])codex(?:\s|$)/.test(line));
-}
-
-function assertNoRunningHarness(harnessId, flags) {
-  if (flags.allowRunningHarnesses || (harnessId === "claude-code" && flags.allowRunningClaude)) {
-    return;
-  }
-  const running = harnessId === "claude-code"
-    ? runningClaudeCodeProcesses()
-    : runningCodexProcesses();
-  if (running.length === 0) return;
-  const name = harnessId === "claude-code" ? "Claude Code" : "Codex";
-  throw new Error(
-    `${name} is currently running. Close all ${name} sessions before changing its global routing. `
-    + "Installing only the explicit `penny` command does not require this. If you intentionally "
-    + "accept the risk, pass --allow-running-harnesses.\nDetected:\n  "
-    + running.slice(0, 5).join("\n  "),
-  );
-}
-
 function printHelp() {
   console.log(`PennyRouter CLI
 
 Usage:
-  pennyrouter install [--harness ${SUPPORTED_HARNESS_IDS}] [--all] [--yes] [--dry-run] [--make-default|--penny-only] [--no-path-update] [--no-mcp] [--anthropic-auth|--no-anthropic-auth] [--anthropic-api-key KEY] [--openai-api-key KEY] [--openrouter-api-key KEY] [--proxy-base-url URL --proxy-api-key KEY] [--local[=URL]|--prod] [--config PATH] [--allow-running-harnesses]
-  pennyrouter update [--harness ${SUPPORTED_HARNESS_IDS}] [--all] [--dry-run] [--penny-key pr-...] [--gateway-base-url URL|--local[=URL]|--prod] [--no-mcp] [--allow-running-harnesses]
+  pennyrouter install [--harness ${SUPPORTED_HARNESS_IDS}] [--all] [--yes] [--dry-run] [--make-default|--penny-only] [--no-path-update] [--no-mcp] [--anthropic-auth|--no-anthropic-auth] [--anthropic-api-key KEY] [--openai-api-key KEY] [--openrouter-api-key KEY] [--proxy-base-url URL --proxy-api-key KEY] [--local[=URL]|--prod] [--config PATH]
+  pennyrouter update [--harness ${SUPPORTED_HARNESS_IDS}] [--all] [--dry-run] [--penny-key pr-...] [--gateway-base-url URL|--local[=URL]|--prod] [--no-mcp]
   pennyrouter auth anthropic [--penny-key pr-...] [--token oauth-token] [--token-command CMD] [--gateway-base-url URL] [--forget]
   pennyrouter disable [--harness ${SUPPORTED_HARNESS_IDS}] [--all] [--yes]
   pennyrouter enable [--harness ${SUPPORTED_HARNESS_IDS}] [--all] [--yes]

package/src/launch.js

@@ -197,26 +197,34 @@ async function ensurePennyOnWindowsPath({ updatePath }) {
   return { available: true, changed: true, path_managed: true };
 }
 
-function shellRcPath() {
+function shellRcPaths() {
   const shell = basename(process.env.SHELL || "");
-  if (shell === "zsh") return join(homeDir(), ".zshrc");
-  if (shell === "bash") return join(homeDir(), ".bashrc");
-  return null;
+  if (shell === "zsh") return [join(homeDir(), ".zshrc"), join(homeDir(), ".zprofile")];
+  if (shell === "bash") return [join(homeDir(), ".bashrc"), join(homeDir(), ".bash_profile")];
+  return [];
 }
 
 export async function ensurePennyOnPath({ updatePath = true } = {}) {
   if (process.platform === "win32") return ensurePennyOnWindowsPath({ updatePath });
   if (pathContains(pennyBinDir())) return { available: true, changed: false, rc_path: null };
   if (!updatePath) return { available: false, changed: false, rc_path: null };
-  const rcPath = shellRcPath();
-  if (!rcPath) return { available: false, changed: false, rc_path: null };
-  const current = await readText(rcPath, "");
-  if (!current.includes(PATH_START)) {
-    const block = `${PATH_START}\nexport PATH="$HOME/.local/bin:$PATH"\n${PATH_END}`;
+  const rcPaths = shellRcPaths();
+  if (rcPaths.length === 0) return { available: false, changed: false, rc_path: null, rc_paths: [] };
+  const block = `${PATH_START}\nexport PATH="$HOME/.local/bin:$PATH"\n${PATH_END}`;
+  const changedPaths = [];
+  for (const rcPath of rcPaths) {
+    const current = await readText(rcPath, "");
+    if (current.includes(PATH_START)) continue;
     const prefix = current && !current.endsWith("\n") ? "\n" : "";
     await atomicWrite(rcPath, `${current}${prefix}${block}\n`);
+    changedPaths.push(rcPath);
   }
-  return { available: true, changed: !current.includes(PATH_START), rc_path: rcPath };
+  return {
+    available: true,
+    changed: changedPaths.length > 0,
+    rc_path: rcPaths[0],
+    rc_paths: rcPaths,
+  };
 }
 
 export async function installPennyRuntime({ updatePath = true } = {}) {
@@ -227,8 +235,9 @@ export async function installPennyRuntime({ updatePath = true } = {}) {
   await writeShim();
   const path = await ensurePennyOnPath({ updatePath });
   const config = await loadPennyConfig();
-  if (path.rc_path) {
+  if (path.rc_paths?.length > 0) {
     config.path_rc = path.rc_path;
+    config.path_rcs = path.rc_paths;
     await savePennyConfig(config);
   }
   if (process.platform === "win32" && path.path_managed) {
@@ -493,7 +502,8 @@ export async function removePennyRuntime() {
   if (process.platform === "win32") {
     if (config.windows_path_managed) removeWindowsPathEntry();
   } else {
-    await removeManagedPathBlock(config.path_rc);
+    const paths = config.path_rcs || [config.path_rc];
+    for (const rcPath of paths) await removeManagedPathBlock(rcPath);
   }
   await unlink(pennyShimPath()).catch((error) => {
     if (error.code !== "ENOENT") throw error;

package/src/launch.test.js

@@ -96,6 +96,7 @@ async function main() {
 
     const config = await loadPennyConfig();
     assert.equal(config.api_key, "pr-super-secret");
+    assert.equal(config.harnesses.opencode.enabled, true);
     const codexProfile = fs.readFileSync(codexLaunchProfilePath(), "utf8");
     assert.ok(codexProfile.includes('env_http_headers = { "x-pennyrouter-key" = "PENNYROUTER_API_KEY" }'));
     assert.ok(codexProfile.includes("[tui]\nterminal_title = []"));
@@ -122,6 +123,7 @@ async function main() {
     assert.deepEqual(codex.args, ["--profile", "pennyrouter", "resume", "--last"]);
     assert.equal(codex.env.PENNYROUTER_API_KEY, "pr-super-secret");
     const opencode = buildPennyInvocation("opencode", ["--continue"], config);
+    assert.equal(opencode.env.PENNYROUTER_API_KEY, "pr-super-secret");
     assert.deepEqual(opencode, {
       command: "opencode",
       args: ["--continue"],
@@ -158,6 +160,7 @@ async function main() {
     assert.equal(fs.readFileSync(claudeUserSkillPath(), "utf8"), "user-owned command\n");
     assert.ok(fs.existsSync(codexLaunchProfilePath()));
     assert.ok(fs.existsSync(pennyShimPath()));
+    assert.ok(fs.readFileSync(path.join(home, ".zprofile"), "utf8").includes(">>> pennyrouter PATH >>>"));
     await removePennyHarness("codex");
     await removePennyHarness("opencode");
     await removePennyRuntime();
@@ -165,6 +168,7 @@ async function main() {
     assert.ok(!fs.existsSync(pennyRuntimeDir()));
     assert.ok(!fs.existsSync(pennyConfigPath()));
     assert.equal(fs.readFileSync(path.join(home, ".zshrc"), "utf8"), "# user setting\n");
+    assert.equal(fs.readFileSync(path.join(home, ".zprofile"), "utf8"), "");
 
     const paid = renderCodexLaunchProfile({
       gatewayBaseUrl: "https://gateway.example/", subscriptionAuth: false,