pennyrouter 0.2.14 → 0.2.15

package/package.json

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

package/src/cli.js

@@ -690,7 +690,11 @@ async function serveMcp(flags) {
     || process.env.PENNYROUTER_GATEWAY_BASE_URL
     || "https://api.pennyrouter.com";
 
-  await runMcpServer({ gatewayBaseUrl, apiKey });
+  await runMcpServer({
+    gatewayBaseUrl,
+    apiKey,
+    harness: process.env.PENNYROUTER_MCP_HARNESS,
+  });
 }
 
 // Refresh the managed local files to this package version WITHOUT re-running any of the

package/src/mcp-register.js

@@ -30,8 +30,12 @@ const MANAGED_START = "# --- PennyRouter managed MCP server ---";
 const MANAGED_END = "# --- end PennyRouter managed MCP server ---";
 
 /** What we register. Shared by both harnesses so the two configs cannot drift apart. */
-export function serverEntry() {
-  return { command: "npx", args: ["-y", "pennyrouter", "mcp"] };
+export function serverEntry(harness = "claude-code") {
+  return {
+    command: "npx",
+    args: ["-y", "pennyrouter", "mcp"],
+    env: { PENNYROUTER_MCP_HARNESS: harness },
+  };
 }
 
 /** True when an entry is one of ours, so re-installing updates rather than duplicating and
@@ -59,7 +63,7 @@ export function applyClaudeRegistry(config, { remove = false } = {}) {
           "write. Rename it before installing so your configuration is not overwritten.",
       );
     }
-    servers[SERVER_NAME] = serverEntry();
+    servers[SERVER_NAME] = serverEntry("claude-code");
   }
 
   if (Object.keys(servers).length) next.mcpServers = servers;
@@ -96,12 +100,13 @@ export function renderCodexMcp(text, { remove = false } = {}) {
     );
   }
 
-  const entry = serverEntry();
+  const entry = serverEntry("codex");
   const block = [
     MANAGED_START,
     `[mcp_servers.${SERVER_NAME}]`,
     `command = "${entry.command}"`,
     `args = [${entry.args.map((a) => `"${a}"`).join(", ")}]`,
+    `env = { PENNYROUTER_MCP_HARNESS = "${entry.env.PENNYROUTER_MCP_HARNESS}" }`,
     MANAGED_END,
   ].join("\n");
 

package/src/mcp-register.test.js

@@ -17,6 +17,7 @@ import {
   const entry = serverEntry();
   assert.equal(entry.command, "npx");
   assert.deepEqual(entry.args, ["-y", "pennyrouter", "mcp"]);
+  assert.equal(entry.env.PENNYROUTER_MCP_HARNESS, "claude-code");
   assert.ok(!JSON.stringify(entry).includes("/"), "no absolute paths");
   assert.ok(!/pennyrouter@/.test(JSON.stringify(entry)), "not version-pinned");
 }
@@ -85,6 +86,7 @@ assert.equal(isManagedEntry({}), false);
   assert.ok(written.includes(`[mcp_servers.${SERVER_NAME}]`));
   assert.ok(written.includes('command = "npx"'));
   assert.ok(written.includes('args = ["-y", "pennyrouter", "mcp"]'));
+  assert.ok(written.includes('PENNYROUTER_MCP_HARNESS = "codex"'));
   assert.ok(written.includes("[mcp_servers.node_repl]"), "existing servers survive");
 
   // A TOML table header captures every key after it, so our block must be appended — splicing

package/src/mcp.js

@@ -30,6 +30,13 @@ import { writeSession, VERIFIED_MIN, VERIFIED_MAX } from "./thread-session.js";
 import { writeSession as writeCodexSession } from "./thread-session-codex.js";
 
 const PROTOCOL_VERSION = "2024-11-05";
+const SERVER_INSTRUCTIONS =
+  "When the user provides an AI share link and asks to resume, continue, import, or build from " +
+  "that conversation in a coding agent, call a PennyRouter thread tool immediately. For the exact " +
+  "command '@penny import session <link>', use thread_read when it is the first user turn so the " +
+  "conversation enters the current session; otherwise use thread_resume to create a fresh native " +
+  "session. Do not search the local workspace, fetch the link directly, or look for another resume " +
+  "utility first.";
 
 const TOOLS = [
   {
@@ -39,6 +46,8 @@ const TOOLS = [
       "return its transcript. Use when the user wants to discuss, summarize or ask about a " +
       "past conversation. The transcript enters your context — for long threads that are " +
       "meant to be continued as work, prefer thread_resume. " +
+      "For `@penny import session <link>` on the FIRST user turn, call this tool immediately: " +
+      "the desired result is to continue in the current session. " +
       "This tool is the only supported way to read a share link: those pages are " +
       "JavaScript-rendered shells that return markup, not conversation, to a plain fetch. Never " +
       "fall back to fetching the URL directly, and never guess at the contents. If this call " +
@@ -57,8 +66,13 @@ const TOOLS = [
     description:
       "Fetch an AI conversation from a share link and write it as a resumable coding-agent " +
       "session (Claude Code or Codex), so the user can resume into it and continue coding with " +
-      "that conversation as history. Use when the user wants to CONTINUE or BUILD FROM a past " +
-      "chat rather than talk about it. Returns a session id and path; the transcript is " +
+      "that conversation as history. When the user asks to RESUME, CONTINUE, IMPORT, or BUILD " +
+      "FROM a share link, call this tool immediately — do not search the workspace or look for " +
+      "another resume utility first. Use this instead of thread_read when they want to work from " +
+      "the conversation rather than discuss it. For `@penny import session <link>` AFTER the " +
+      "first user turn, call this tool: a new native session avoids injecting a large transcript " +
+      "into an active conversation. The configured harness is the default; do not guess the other " +
+      "one. Returns a session id and path; the transcript is " +
       "deliberately NOT returned, so importing a long thread costs you no context. That is by " +
       "design and is not a failure — do not read the thread another way to 'check' it, and do " +
       "not fetch the share URL directly. Report the resume command to the user. If this call " +
@@ -77,8 +91,8 @@ const TOOLS = [
           type: "string",
           enum: ["claude-code", "codex"],
           description:
-            "Which agent should be able to resume it. Defaults to claude-code. Use codex when " +
-            "the user asks to continue the conversation in Codex.",
+            "Which agent should be able to resume it. Defaults to the harness that launched this " +
+            "MCP server. Supply this only to override that active harness.",
         },
       },
       required: ["url"],
@@ -153,7 +167,7 @@ async function callTool(name, args, context) {
 
   if (name === "thread_resume") {
     const cwd = String(args?.directory || process.cwd());
-    const harness = String(args?.harness || "claude-code").toLowerCase();
+    const harness = String(args?.harness || context.harness || "claude-code").toLowerCase();
     if (harness !== "claude-code" && harness !== "codex") {
       throw new Error(`unknown harness "${harness}"; expected claude-code or codex`);
     }
@@ -204,7 +218,7 @@ function toolError(id, error) {
   });
 }
 
-export async function runMcpServer({ gatewayBaseUrl, apiKey }) {
+export async function runMcpServer({ gatewayBaseUrl, apiKey, harness = process.env.PENNYROUTER_MCP_HARNESS }) {
   if (!apiKey) {
     process.stderr.write(
       "pennyrouter mcp: no API key. Run `npx pennyrouter install` first.\n",
@@ -213,7 +227,7 @@ export async function runMcpServer({ gatewayBaseUrl, apiKey }) {
     return;
   }
 
-  const context = { gatewayBaseUrl, apiKey };
+  const context = { gatewayBaseUrl, apiKey, harness };
   const rl = createInterface({ input: process.stdin });
 
   for await (const line of rl) {
@@ -233,6 +247,7 @@ export async function runMcpServer({ gatewayBaseUrl, apiKey }) {
           protocolVersion: PROTOCOL_VERSION,
           capabilities: { tools: {} },
           serverInfo: { name: "pennyrouter", version: "1.0.0" },
+          instructions: SERVER_INSTRUCTIONS,
         });
       } else if (method === "tools/list") {
         reply(id, { tools: TOOLS });

package/src/mcp.test.js

@@ -41,12 +41,15 @@ function stubGateway(handler) {
 }
 
 /** Drive the server over stdio and collect the responses to `requests`. */
-function rpc(requests, { port, home }) {
+function rpc(requests, { port, home, harness }) {
   return new Promise((resolve, reject) => {
     const child = spawn(
       process.execPath,
       [binary, "mcp", "--penny-key", "pr-test", "--gateway-base-url", `http://127.0.0.1:${port}`],
-      { stdio: ["pipe", "pipe", "pipe"], env: { ...process.env, HOME: home } },
+      {
+        stdio: ["pipe", "pipe", "pipe"],
+        env: { ...process.env, HOME: home, ...(harness ? { PENNYROUTER_MCP_HARNESS: harness } : {}) },
+      },
     );
     let out = "";
     let err = "";
@@ -108,6 +111,8 @@ const workspace = mkdtempSync(join(tmpdir(), "pr-mcp-ws-"));
 
   assert.equal(byId.get(1).result.protocolVersion, "2024-11-05");
   assert.equal(byId.get(1).result.serverInfo.name, "pennyrouter");
+  assert.match(byId.get(1).result.instructions, /thread_read when it is the first user turn/i);
+  assert.match(byId.get(1).result.instructions, /otherwise use thread_resume/i);
 
   assert.deepEqual(
     byId.get(2).result.tools.map((t) => t.name),
@@ -122,6 +127,10 @@ const workspace = mkdtempSync(join(tmpdir(), "pr-mcp-ws-"));
     assert.match(tool.description, /not fetch|Never fall back to fetching/i, tool.name);
     assert.match(tool.description, /blocked/i, tool.name);
   }
+  assert.match(
+    byId.get(2).result.tools.find((tool) => tool.name === "thread_resume").description,
+    /do not search the workspace/i,
+  );
 
   // The URL is forwarded to the gateway under the caller's key; no provider logic runs here.
   assert.equal(JSON.parse(requestBody).url, "https://www.perplexity.ai/search/x");
@@ -161,7 +170,7 @@ const workspace = mkdtempSync(join(tmpdir(), "pr-mcp-ws-"));
         method: "tools/call",
         params: {
           name: "thread_resume",
-          arguments: { url: "https://x/y", directory: workspace, harness: "codex" },
+          arguments: { url: "https://x/y", directory: workspace },
         },
       },
       {
@@ -174,7 +183,7 @@ const workspace = mkdtempSync(join(tmpdir(), "pr-mcp-ws-"));
         },
       },
     ],
-    { port, home },
+    { port, home, harness: "codex" },
   );
   server.close();