@pi-unipi/mcp 2.6.0 → 2.6.2

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pi-unipi/mcp",
-  "version": "2.6.0",
+  "version": "2.6.2",
   "description": "MCP server management extension for Pi coding agent — browse, add, configure, and use MCP servers",
   "type": "module",
   "main": "src/index.ts",
@@ -30,7 +30,7 @@
     "test": "npx tsx --test tests/**/*.test.ts"
   },
   "dependencies": {
-    "@pi-unipi/core": "2.6.0"
+    "@pi-unipi/core": "2.6.1"
   },
   "peerDependencies": {
     "@earendil-works/pi-coding-agent": "^0.80.0",
@@ -41,8 +41,12 @@
     "@types/node": "^25.6.0"
   },
   "pi": {
-    "extensions": [],
-    "skills": [],
+    "extensions": [
+      "./src/index.ts"
+    ],
+    "skills": [
+      "./skills"
+    ],
     "prompts": [],
     "themes": []
   }

package/src/bridge/client.ts

@@ -47,8 +47,6 @@ interface PendingRequest {
 export interface McpClientOptions {
   /** Per-request timeout in ms (default: MCP_DEFAULTS.STARTUP_TIMEOUT_MS) */
   timeoutMs?: number;
-  /** Working directory for the spawned process */
-  cwd?: string;
 }
 
 /**
@@ -65,11 +63,9 @@ export class McpClient {
   private connected = false;
   private stderrBuffer = "";
   private readonly timeoutMs: number;
-  private readonly cwd?: string;
 
   constructor(options?: McpClientOptions) {
     this.timeoutMs = options?.timeoutMs ?? MCP_DEFAULTS.STARTUP_TIMEOUT_MS;
-    this.cwd = options?.cwd;
   }
 
   /**
@@ -91,7 +87,6 @@ export class McpClient {
         this.process = spawn(command, args, {
           stdio: ["pipe", "pipe", "pipe"],
           env: mergedEnv,
-          cwd: this.cwd,
         });
 
         this.process.on("error", (err) => {
@@ -230,21 +225,11 @@ export class McpClient {
     this.cleanup();
   }
 
-  /** Whether the client is currently connected */
-  get isConnected(): boolean {
-    return this.connected;
-  }
-
   /** Process ID of the spawned MCP server, or undefined if not connected */
   get pid(): number | undefined {
     return this.process?.pid;
   }
 
-  /** Captured stderr output */
-  get stderr(): string {
-    return this.stderrBuffer;
-  }
-
   // ── Internal methods ────────────────────────────────────────────
 
   private ensureConnected(): void {

package/src/config/manager.ts

@@ -11,7 +11,6 @@ import * as os from "node:os";
 import type {
   McpConfig,
   McpMetadata,
-  McpAuth,
   ResolvedServer,
   ServerSource,
 } from "../types.js";
@@ -123,15 +122,6 @@ export function loadMetadata(dir: string): McpMetadata {
   };
 }
 
-/**
- * Load auth data (auth.json) from a directory.
- * Returns empty object if file doesn't exist.
- */
-export function loadAuth(dir: string): McpAuth {
-  const filePath = path.join(dir, "auth.json");
-  return readJsonFile<McpAuth>(filePath) ?? {};
-}
-
 // ── Config saving ─────────────────────────────────────────────────
 
 /**
@@ -150,14 +140,6 @@ export function saveMetadata(dir: string, meta: McpMetadata): void {
   writeJsonFile(filePath, meta);
 }
 
-/**
- * Save auth data (auth.json) with chmod 600.
- */
-export function saveAuth(dir: string, auth: McpAuth): void {
-  const filePath = path.join(dir, "auth.json");
-  writeJsonFile(filePath, auth, 0o600);
-}
-
 // ── Config merging ────────────────────────────────────────────────
 
 /**
@@ -220,26 +202,6 @@ export function resolveServers(
   }));
 }
 
-/**
- * Merge auth.json env vars into a server definition at spawn time.
- * Auth env vars are added to the server's env, but don't override
- * explicitly set values in mcp-config.json.
- */
-export function mergeEnvWithAuth(
-  serverDef: McpConfig["mcpServers"][string],
-  auth: Record<string, string>,
-): McpConfig["mcpServers"][string] {
-  if (Object.keys(auth).length === 0) return serverDef;
-
-  return {
-    ...serverDef,
-    env: {
-      ...auth,
-      ...(serverDef.env ?? {}),
-    },
-  };
-}
-
 /**
  * Load both global and project configs and resolve servers.
  * Convenience wrapper around the individual load + resolve functions.

package/src/config/schema.ts

@@ -85,30 +85,3 @@ export function validateMcpConfig(config: unknown): ValidationResult {
 
   return { valid: errors.length === 0, errors };
 }
-
-/**
- * Create a minimal server definition template for a new server.
- */
-export function createServerTemplate(
-  name: string,
-  command: string,
-  args: string[],
-  envVars?: string[],
-): McpConfig {
-  const env: Record<string, string> = {};
-  if (envVars) {
-    for (const v of envVars) {
-      env[v] = "";
-    }
-  }
-
-  return {
-    mcpServers: {
-      [name]: {
-        command,
-        args,
-        env: Object.keys(env).length > 0 ? env : undefined,
-      },
-    },
-  };
-}

package/src/config/sync.ts

@@ -319,15 +319,6 @@ function enrichWithInstallInfo(entries: CatalogEntry[]): CatalogEntry[] {
   });
 }
 
-/**
- * Check if enough time has passed since last sync.
- */
-function shouldSync(lastSyncAt: string | null, intervalMs: number): boolean {
-  if (!lastSyncAt) return true;
-  const elapsed = Date.now() - new Date(lastSyncAt).getTime();
-  return elapsed >= intervalMs;
-}
-
 /**
  * Sync the MCP server catalog from GitHub.
  * Parses the awesome-mcp-servers README, enriches with install info,
@@ -394,23 +385,3 @@ export function loadCatalog(): CatalogData {
     };
   }
 }
-
-/**
- * Sync if enough time has passed since last sync.
- * Returns null if no sync was needed, or the catalog data if synced.
- */
-export async function syncIfNeeded(): Promise<CatalogData | null> {
-  const configDir = getGlobalConfigDir();
-  const meta = loadMetadata(configDir);
-
-  if (!shouldSync(meta.sync.lastSyncAt, meta.sync.syncIntervalMs)) {
-    return null;
-  }
-
-  try {
-    return await syncCatalog();
-  } catch {
-    // Sync failed — return null, will use cached/seed data
-    return null;
-  }
-}

package/src/index.ts

@@ -16,7 +16,7 @@ import {
 } from "@pi-unipi/core";
 import type { ResolvedServer } from "./types.js";
 import { loadAndResolve, getGlobalConfigDir } from "./config/manager.js";
-import { syncCatalog, loadCatalog } from "./config/sync.js";
+import { syncCatalog } from "./config/sync.js";
 import { ServerRegistry } from "./bridge/registry.js";
 import { compareCodeUnits } from "./bridge/translator.js";
 import { renderMcpAddOverlay } from "./tui/add-overlay.js";

package/src/tui/add-overlay.ts

@@ -353,7 +353,7 @@ export function renderMcpAddOverlay(params?: {
         return;
       }
       // Arrow keys + j/k navigate list while searching
-      if (matchesKey(data, Key.down) || data === "\x1b[B" || data === "j") {
+      if (matchesKey(data, Key.down) || data === "j") {
         if (state.selectedIndex < state.filteredServers.length - 1) {
           state.selectedIndex++;
           ensureVisible();
@@ -361,7 +361,7 @@ export function renderMcpAddOverlay(params?: {
         }
         return;
       }
-      if (matchesKey(data, Key.up) || data === "\x1b[A" || data === "k") {
+      if (matchesKey(data, Key.up) || data === "k") {
         if (state.selectedIndex > 0) {
           state.selectedIndex--;
           ensureVisible();
@@ -370,13 +370,13 @@ export function renderMcpAddOverlay(params?: {
         return;
       }
       // PgUp/PgDn also work in search
-      if (data === "\x1b[6~") {
+      if (matchesKey(data, Key.pageDown)) {
         state.selectedIndex = Math.min(state.filteredServers.length - 1, state.selectedIndex + lastListHeight);
         ensureVisible();
         refresh();
         return;
       }
-      if (data === "\x1b[5~") {
+      if (matchesKey(data, Key.pageUp)) {
         state.selectedIndex = Math.max(0, state.selectedIndex - lastListHeight);
         ensureVisible();
         refresh();
@@ -463,13 +463,13 @@ export function renderMcpAddOverlay(params?: {
         return;
       }
       // PageDown / PageUp
-      if (data === "\x1b[6~") {
+      if (matchesKey(data, Key.pageDown)) {
         state.selectedIndex = Math.min(listLen - 1, state.selectedIndex + lastListHeight);
         ensureVisible();
         refresh();
         return;
       }
-      if (data === "\x1b[5~") {
+      if (matchesKey(data, Key.pageUp)) {
         state.selectedIndex = Math.max(0, state.selectedIndex - lastListHeight);
         ensureVisible();
         refresh();

package/src/types.ts

@@ -46,12 +46,6 @@ export interface McpMetadata {
   sync: SyncConfig;
 }
 
-/** Per-server auth data (auth.json) */
-export interface McpAuth {
-  /** Map of server name → environment variable key-value pairs */
-  [serverName: string]: Record<string, string>;
-}
-
 /** Single entry from the MCP server catalog */
 export interface CatalogEntry {
   /** Unique identifier (e.g. "github/github-mcp-server") */