pi-repoprompt-mcp 0.5.4 → 0.6.0

package/README.md

@@ -53,7 +53,7 @@ Forked sessions inherit the parent session-plus-node's window, tab, and auto-sel
   <img width="270" height="936" alt="Collapsed call/result summaries" src="https://raw.githubusercontent.com/w-winter/dot314/main/packages/pi-repoprompt-mcp/docs/images/collapsed-summaries.png" />
 </p>
 
-- RepoPrompt `apply_edits` calls are forwarded with `verbose: true` by default (unless `raw: true`), while the returned diff is normalized into `details.diff` and presented to the agent as a terse summary.  The same is done for `file_actions create/delete` outputs, so you see all edited/created/deleted LOC with rich rendering but the extension prevents the context window from getting bloated by round-tripping tool I/O tokens
+- RepoPrompt `apply_edits` calls are forwarded with `verbose: true` by default, while the returned diff is normalized into `details.diff` and presented to the agent as a terse summary.  The same is done for `file_actions create/delete` outputs, so you see all edited/created/deleted LOC with rich rendering but the extension prevents the context window from getting bloated by round-tripping tool I/O tokens
 - Adaptive diff rendering for RepoPrompt `git` and `apply_edits` outputs by default (`diffViewMode: "auto"` picks split, unified, compact, or summary at render time based on pane width).  This uses the active Pi theme's `toolDiffAdded`, `toolDiffRemoved`, and `toolDiffContext` colors (typically mapped to chosen hues for green and red), and its visual design and rendering logic are indebted to [MasuRii/pi-tool-display](https://github.com/MasuRii/pi-tool-display).  Two different examples at different pane widths:
 
 <p align="center">
@@ -207,7 +207,6 @@ but this is best-effort
 
 ## Readcache gotchas
 
-- `raw: true` disables readcache (and rendering). Don't use unless debugging
 - Need full content? use `bypass_cache: true` in `read_file` args
 - Multi-root: use absolute or specific relative paths (MCP `read_file` has no `RootName:` disambiguation)
 

package/extensions/repoprompt-mcp/src/binding.ts

@@ -754,7 +754,7 @@ function parseTabFromJson(raw: unknown): RpTab | null {
   }
 
   const obj = raw as Record<string, unknown>;
-  const idRaw = obj.id ?? obj.tabId ?? obj.tab_id ?? obj.uuid;
+  const idRaw = obj.contextId ?? obj.context_id ?? obj.id ?? obj.tabId ?? obj.tab_id ?? obj.uuid;
   if (typeof idRaw !== "string" || !idRaw.trim()) {
     return null;
   }
@@ -786,7 +786,20 @@ function collectTabsFromJson(raw: unknown): RpTab[] {
   }
 
   const obj = raw as Record<string, unknown>;
-  const containers = [obj.tabs, obj.tab, obj.createdTab, obj.created_tab, obj.selectedTab, obj.selected_tab];
+  const containers = [
+    obj.tabs,
+    obj.contexts,
+    obj.tab,
+    obj.context,
+    obj.createdTab,
+    obj.created_tab,
+    obj.createdContext,
+    obj.created_context,
+    obj.selectedTab,
+    obj.selected_tab,
+    obj.selectedContext,
+    obj.selected_context,
+  ];
 
   for (const candidate of containers) {
     const parsed = collectTabsFromJson(candidate);
@@ -835,7 +848,26 @@ function dedupeTabs(tabs: RpTab[]): RpTab[] {
 
 function parseTabLine(line: string): RpTab | null {
   const trimmed = line.trim();
-  if (!trimmed || !trimmed.includes("`")) {
+  if (!trimmed) {
+    return null;
+  }
+
+  const contextMatch = trimmed.match(/^[\u2022-]\s*(.+?)(?:\s+\[([^\]]+)\])?\s+—\s+context_id:\s*`([^`]+)`/i);
+  if (contextMatch) {
+    const state = contextMatch[2] ?? "";
+    return {
+      id: contextMatch[3].trim(),
+      name: stripTrailingTabStateAnnotations(contextMatch[1].trim()) || contextMatch[3].trim(),
+      isActive: /\bactive\b|\bin-focus\b/i.test(state)
+        ? true
+        : /\bout-of-focus\b/i.test(state)
+          ? false
+          : undefined,
+      isBound: /\bbound\b/i.test(state) ? true : undefined,
+    };
+  }
+
+  if (!trimmed.includes("`")) {
     return null;
   }
 
@@ -869,6 +901,17 @@ function parseTabLine(line: string): RpTab | null {
   };
 }
 
+function parseTabsFromBindContextText(text: string): RpTab[] {
+  return dedupeTabs(
+    text
+      .split("\n")
+      .map((line) => line.trim())
+      .filter((line) => /—\s+context_id:\s*`[^`]+`/i.test(line))
+      .map(parseTabLine)
+      .filter((tab): tab is RpTab => tab !== null)
+  );
+}
+
 export function parseTabList(text: string): RpTab[] {
   const tabs: RpTab[] = [];
   let lastTab: RpTab | null = null;
@@ -899,132 +942,129 @@ function parseTabsFromJson(value: unknown): RpTab[] | null {
   return tabs.length > 0 ? dedupeTabs(tabs) : null;
 }
 
-function parseChatCountFromJson(value: unknown): number | undefined {
-  if (!value) {
-    return undefined;
-  }
-
-  if (Array.isArray(value)) {
-    return value.length;
+function findLiveTab(tabs: RpTab[], reference: string | undefined): RpTab | null {
+  if (!reference) {
+    return null;
   }
 
-  if (typeof value !== "object") {
-    return undefined;
-  }
+  return tabs.find((tab) => tab.id === reference || tab.name === reference) ?? null;
+}
 
-  const obj = value as Record<string, unknown>;
-  const directCount = parseCountMaybe(
-    obj.count ?? obj.chatCount ?? obj.chat_count ?? obj.total ?? obj.totalCount ?? obj.total_count
-  );
-  if (directCount !== undefined) {
-    return directCount;
-  }
+function isExplicitlyEmptyTab(tab: RpTab): boolean {
+  return tab.selectedFileCount === 0;
+}
 
-  for (const key of ["chats", "sessions", "items", "results"]) {
-    const candidate = obj[key];
-    if (Array.isArray(candidate)) {
-      return candidate.length;
-    }
-  }
+function orderReusableTabCandidates(tabs: RpTab[]): RpTab[] {
+  const ordered = [
+    ...tabs.filter((tab) => tab.isBound === true),
+    ...tabs.filter((tab) => tab.isBound !== true && tab.isActive === true),
+    ...tabs.filter((tab) => tab.isBound !== true && tab.isActive !== true),
+  ];
 
-  return undefined;
+  return ordered.filter((tab, index) => ordered.findIndex((candidate) => candidate.id === tab.id) === index);
 }
 
-function parseChatCountFromText(text: string): number | undefined {
-  const countMatch = text.match(/\bCount\b[^\d]*([\d,]+)/i);
-  if (countMatch?.[1]) {
-    return parseCountMaybe(countMatch[1]);
-  }
+function parseOracleSessionTabPrefixes(text: string): Set<string> {
+  const prefixes = new Set<string>();
 
-  if (/\bNo chats\b/i.test(text)) {
-    return 0;
+  for (const match of text.matchAll(/\btab=([A-F0-9-]{6,})(?:…|\b)/gi)) {
+    const prefix = match[1]?.trim().toUpperCase();
+    if (prefix) {
+      prefixes.add(prefix);
+    }
   }
 
-  const sessionCount = text
-    .split("\n")
-    .map((line) => line.trim())
-    .filter((line) => /^•\s*\[[^\]]+\]/.test(line)).length;
-
-  return sessionCount > 0 ? sessionCount : undefined;
+  return prefixes;
 }
 
-async function fetchTabChatCount(
-  tabId: string,
+async function fetchOracleSessionTabPrefixes(
+  windowId: number,
   client: ReturnType<typeof getRpClient> = getRpClient()
-): Promise<number | undefined> {
-  if (!client.isConnected) {
-    return undefined;
-  }
-
-  const chatsToolName = resolveToolName(client.tools, "chats");
-  if (!chatsToolName) {
-    return undefined;
+): Promise<Set<string> | null> {
+  const oracleUtilsToolName = resolveToolName(client.tools, "oracle_utils");
+  if (!oracleUtilsToolName) {
+    return null;
   }
 
-  const result = await client.callTool(chatsToolName, {
-    action: "list",
-    scope: "tab",
-    tab_id: tabId,
-    limit: 1,
+  const result = await client.callTool(oracleUtilsToolName, {
+    op: "sessions",
+    limit: 200,
+    _windowID: windowId,
   });
 
   if (result.isError) {
-    return undefined;
-  }
-
-  const countFromJson = parseChatCountFromJson(extractJsonContent(result.content));
-  if (countFromJson !== undefined) {
-    return countFromJson;
+    return null;
   }
 
-  return parseChatCountFromText(extractTextContent(result.content));
+  return parseOracleSessionTabPrefixes(extractTextContent(result.content));
 }
 
-function findLiveTab(tabs: RpTab[], reference: string | undefined): RpTab | null {
-  if (!reference) {
-    return null;
+function tabHasOracleHistory(tabId: string, sessionTabPrefixes: Set<string> | null): boolean {
+  if (!sessionTabPrefixes || sessionTabPrefixes.size === 0) {
+    return false;
   }
 
-  return tabs.find((tab) => tab.id === reference || tab.name === reference) ?? null;
-}
+  const normalizedTabId = tabId.trim().toUpperCase();
+  for (const prefix of sessionTabPrefixes) {
+    if (normalizedTabId.startsWith(prefix)) {
+      return true;
+    }
+  }
 
-function isExplicitlyEmptyTab(tab: RpTab): boolean {
-  return tab.selectedFileCount === 0;
+  return false;
 }
 
-async function isSafeReusableTab(
+async function hasEmptySelection(
+  windowId: number,
   tab: RpTab,
   client: ReturnType<typeof getRpClient> = getRpClient()
 ): Promise<boolean> {
-  if (!isExplicitlyEmptyTab(tab)) {
+  if (isExplicitlyEmptyTab(tab)) {
+    return true;
+  }
+
+  const manageSelectionToolName = resolveToolName(client.tools, "manage_selection");
+  if (!manageSelectionToolName) {
     return false;
   }
 
-  const chatCount = await fetchTabChatCount(tab.id, client);
-  return chatCount === 0;
-}
+  const result = await client.callTool(manageSelectionToolName, {
+    op: "get",
+    view: "summary",
+    _windowID: windowId,
+    context_id: tab.id,
+  });
 
-function orderReusableEmptyTabs(tabs: RpTab[]): RpTab[] {
-  const emptyTabs = tabs.filter(isExplicitlyEmptyTab);
-  if (emptyTabs.length === 0) {
-    return [];
+  if (result.isError) {
+    return false;
   }
 
-  const ordered = [
-    ...emptyTabs.filter((tab) => tab.isBound === true),
-    ...emptyTabs.filter((tab) => tab.isBound !== true && tab.isActive === true),
-    ...emptyTabs.filter((tab) => tab.isBound !== true && tab.isActive !== true),
-  ];
+  const text = extractTextContent(result.content);
+  return /\b0 total tokens\b/i.test(text);
+}
 
-  return ordered.filter((tab, index) => ordered.findIndex((candidate) => candidate.id === tab.id) === index);
+async function isSafeReusableTab(
+  windowId: number,
+  tab: RpTab,
+  sessionTabPrefixes: Set<string> | null,
+  client: ReturnType<typeof getRpClient> = getRpClient()
+): Promise<boolean> {
+  if (tabHasOracleHistory(tab.id, sessionTabPrefixes)) {
+    return false;
+  }
+
+  return await hasEmptySelection(windowId, tab, client);
 }
 
 async function findReusableSafeTab(
+  windowId: number,
   tabs: RpTab[],
   client: ReturnType<typeof getRpClient> = getRpClient()
 ): Promise<RpTab | null> {
-  for (const tab of orderReusableEmptyTabs(tabs)) {
-    if (await isSafeReusableTab(tab, client)) {
+  const sessionTabPrefixes = await fetchOracleSessionTabPrefixes(windowId, client);
+
+  for (const tab of orderReusableTabCandidates(tabs)) {
+    if (await isSafeReusableTab(windowId, tab, sessionTabPrefixes, client)) {
       return tab;
     }
   }
@@ -1066,14 +1106,14 @@ export async function fetchWindowTabs(
     throw new Error("Not connected to RepoPrompt");
   }
 
-  const manageWorkspacesToolName = resolveToolName(client.tools, "manage_workspaces");
-  if (!manageWorkspacesToolName) {
+  const bindContextToolName = resolveToolName(client.tools, "bind_context");
+  if (!bindContextToolName) {
     return [];
   }
 
-  const result = await client.callTool(manageWorkspacesToolName, {
-    action: "list_tabs",
-    ...bindingWindowArgs(windowId),
+  const result = await client.callTool(bindContextToolName, {
+    op: "list",
+    window_id: windowId,
   });
 
   if (result.isError) {
@@ -1086,7 +1126,7 @@ export async function fetchWindowTabs(
     return tabsFromJson;
   }
 
-  return parseTabList(extractTextContent(result.content));
+  return parseTabsFromBindContextText(extractTextContent(result.content));
 }
 
 async function selectTab(
@@ -1094,16 +1134,15 @@ async function selectTab(
   tabId: string,
   client: ReturnType<typeof getRpClient> = getRpClient()
 ): Promise<void> {
-  const manageWorkspacesToolName = resolveToolName(client.tools, "manage_workspaces");
-  if (!manageWorkspacesToolName) {
+  const bindContextToolName = resolveToolName(client.tools, "bind_context");
+  if (!bindContextToolName) {
     return;
   }
 
-  const result = await client.callTool(manageWorkspacesToolName, {
-    action: "select_tab",
-    tab: tabId,
-    focus: false,
-    ...bindingWindowArgs(windowId),
+  const result = await client.callTool(bindContextToolName, {
+    op: "bind",
+    window_id: windowId,
+    context_id: tabId,
   });
 
   if (result.isError) {
@@ -1290,7 +1329,7 @@ export async function ensureBindingHasTab(
   }
 
   if (!binding.tab || reuseSoleEmptyTab || recoverIfMissing) {
-    const reusableSafeTab = await findReusableSafeTab(liveTabs, client);
+    const reusableSafeTab = await findReusableSafeTab(binding.windowId, liveTabs, client);
     if (reusableSafeTab) {
       return await adoptTab(reusableSafeTab, true);
     }
@@ -1498,7 +1537,7 @@ export function getBindingArgs(): Record<string, unknown> {
   };
 
   if (currentBinding.tab) {
-    args._tabID = currentBinding.tab;
+    args.context_id = currentBinding.tab;
   }
 
   return args;

package/extensions/repoprompt-mcp/src/index.ts

@@ -546,9 +546,6 @@ const RpToolSchema = Type.Object({
   confirmEdits: Type.Optional(
     Type.Boolean({ description: "Confirm edit-like operations (required when confirmEdits is enabled)" })
   ),
-
-  // Formatting
-  raw: Type.Optional(Type.Boolean({ description: "Return raw output without formatting" })),
 });
 
 // ─────────────────────────────────────────────────────────────────────────────
@@ -816,7 +813,7 @@ export default function repopromptMcp(pi: ExtensionAPI) {
   function bindingArgsForAutoSelectionState(state: AutoSelectionEntryData): Record<string, unknown> {
     return {
       _windowID: state.windowId,
-      ...(state.tab ? { _tabID: state.tab } : {}),
+      ...(state.tab ? { context_id: state.tab } : {}),
     };
   }
 
@@ -1585,9 +1582,9 @@ export default function repopromptMcp(pi: ExtensionAPI) {
           try {
             await ensureTabScopedBinding(ctx, "RepoPrompt binding has no tab. Use /rp bind or /rp tab new first.");
 
-            const chatSendToolName = resolveToolName(client.tools, "chat_send");
-            if (!chatSendToolName) {
-              ctx.ui.notify("RepoPrompt tool 'chat_send' not available", "error");
+            const oracleSendToolName = resolveToolName(client.tools, "oracle_send");
+            if (!oracleSendToolName) {
+              ctx.ui.notify("RepoPrompt tool 'oracle_send' not available", "error");
               return;
             }
 
@@ -1601,7 +1598,7 @@ export default function repopromptMcp(pi: ExtensionAPI) {
             if (chatName) callArgs.chat_name = chatName;
             if (chatId) callArgs.chat_id = chatId;
 
-            const result = await client.callTool(chatSendToolName, callArgs);
+            const result = await client.callTool(oracleSendToolName, callArgs);
 
             const text = extractTextContent(result.content);
 
@@ -1783,10 +1780,6 @@ Mode priority: call > describe > search > windows > bind > status`,
         return new Text(theme.fg("error", "↳ " + textContent), 0, 0);
       }
 
-      if (details.raw) {
-        return new Text(textContent, 0, 0);
-      }
-
       const successPrefix = theme.fg("success", "↳ ");
       const collapsedMaxLines = config.collapsedMaxLines ?? 15;
       const normalizedToolName = typeof details.tool === "string" ? normalizeToolName(details.tool) : undefined;
@@ -2577,14 +2570,12 @@ Mode priority: call > describe > search > windows > bind > status`,
     const forwardedUserArgs = buildForwardedUserArgs({
       toolName: normalizedTool,
       userArgs,
-      raw: params.raw,
     });
 
     const mergedArgs = { ...forwardedUserArgs, ...bindingArgs };
 
     const fileActionDeleteSnapshot = normalizedTool === "file_actions"
       && userArgs.action === "delete"
-      && params.raw !== true
       && typeof userArgs.path === "string"
       ? (() => {
         try {
@@ -2611,7 +2602,6 @@ Mode priority: call > describe > search > windows > bind > status`,
 
       const shouldReadcache =
         config.readcacheReadFile === true &&
-        params.raw !== true &&
         normalizedTool === "read_file" &&
         typeof userArgs.path === "string" &&
         ctx !== undefined;
@@ -2668,9 +2658,8 @@ Mode priority: call > describe > search > windows > bind > status`,
         : normalizeToolResultText({
           toolName: normalizedTool,
           text: textContent,
-          raw: params.raw,
         });
-      const normalizedFileActionResult = result.isError || params.raw === true
+      const normalizedFileActionResult = result.isError
         ? null
         : normalizeFileActionResult({
           action: userArgs.action,
@@ -2733,7 +2722,6 @@ Mode priority: call > describe > search > windows > bind > status`,
           warning: guardResult.warning,
           editNoop,
           rpReadcache: rpReadcache ?? undefined,
-          raw: params.raw,
           ...(normalizedTextResult ? normalizedTextResult.details : {}),
           ...(normalizedFileActionResult ?? {}),
         },

package/extensions/repoprompt-mcp/src/result-normalization.ts

@@ -39,9 +39,8 @@ function normalizeDiffBlockCode(code: string): string {
 export function normalizeToolResultText(args: {
   toolName: string | undefined;
   text: string;
-  raw: boolean | undefined;
 }): ToolResultNormalization | null {
-  if (args.toolName !== "apply_edits" || args.raw === true || !args.text.includes("```diff")) {
+  if (args.toolName !== "apply_edits" || !args.text.includes("```diff")) {
     return null;
   }
 

package/extensions/repoprompt-mcp/src/tool-forwarding-policy.ts

@@ -1,7 +1,6 @@
 export function buildForwardedUserArgs(args: {
   toolName: string | undefined;
   userArgs: Record<string, unknown>;
-  raw: boolean | undefined;
 }): Record<string, unknown> {
   const forwardedUserArgs: Record<string, unknown> = { ...args.userArgs };
 
@@ -9,7 +8,7 @@ export function buildForwardedUserArgs(args: {
     delete forwardedUserArgs.bypass_cache;
   }
 
-  if (args.toolName === "apply_edits" && args.raw !== true) {
+  if (args.toolName === "apply_edits") {
     forwardedUserArgs.verbose = true;
   }
 

package/extensions/repoprompt-mcp/src/types.ts

@@ -152,9 +152,6 @@ export interface RpToolParams {
   // Safety overrides
   allowDelete?: boolean;           // Allow delete operations
   confirmEdits?: boolean;          // Confirm edit-like operations when confirmEdits is enabled
-  
-  // Formatting
-  raw?: boolean;                   // Return raw output without formatting
 }
 
 // ─────────────────────────────────────────────────────────────────────────────

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "pi-repoprompt-mcp",
-  "version": "0.5.4",
+  "version": "0.6.0",
   "description": "A token-efficient RepoPrompt integration for Pi with automated and branch-safe workspace management",
   "keywords": ["pi-package", "pi", "pi-coding-agent", "repoprompt", "mcp"],
   "license": "MIT",