@agent-finops/core 0.4.4 → 0.5.1

package/dist/agentInventory.d.ts

@@ -35,6 +35,8 @@ export type InventoryItem = {
     /** "estimated_understated" when an MCP tool schema is unavailable. */
     weightConfidence: "estimated" | "estimated_understated";
     path?: string;
+    /** For project-scoped MCP servers: the project dirs that load this server. */
+    ownerDirs?: string[];
 };
 export type AgentInventoryOptions = {
     /** Default: ~/.claude */

package/dist/agentInventory.js

@@ -113,7 +113,7 @@ export async function loadAgentInventory(options = {}) {
     // --- MCP servers (from ~/.claude.json: top-level + per-project map) ---
     const config = await readJson(configPath);
     const serverScopes = collectMcpServers(config, projectDir, options.includeAllProjectMcp ?? false);
-    for (const { id, scope } of serverScopes) {
+    for (const { id, scope, ownerDirs } of serverScopes) {
         scanned.mcpServers += 1;
         // We almost never have tool schemas from config, so we can't measure the
         // real weight (full tool definitions). Use a conservative published-typical
@@ -128,7 +128,8 @@ export async function loadAgentInventory(options = {}) {
             group: id,
             alwaysLoadedTokens: MCP_SERVER_TOKEN_FLOOR,
             weightConfidence: "estimated_understated",
-            path: configPath
+            path: configPath,
+            ownerDirs
         });
     }
     return { items, scanned };
@@ -139,16 +140,19 @@ export async function loadAgentInventory(options = {}) {
 function collectMcpServers(config, projectDir, includeAllProjectMcp) {
     if (!isRecord(config))
         return [];
-    const out = [];
-    const seen = new Set();
-    const add = (id, scope) => {
+    const byKey = new Map();
+    const add = (id, scope, ownerDir) => {
         // Dedupe by id across all scopes so a server configured in several projects
-        // is counted once in the global view.
+        // is counted once in the global view — but keep EVERY owning project dir,
+        // because that's where `claude mcp remove` has to run.
         const key = includeAllProjectMcp ? id : `${scope}:${id}`;
-        if (seen.has(key))
+        const existing = byKey.get(key);
+        if (existing) {
+            if (ownerDir && !existing.ownerDirs.includes(ownerDir))
+                existing.ownerDirs.push(ownerDir);
             return;
-        seen.add(key);
-        out.push({ id, scope });
+        }
+        byKey.set(key, { id, scope, ownerDirs: ownerDir ? [ownerDir] : [] });
     };
     // Top-level mcpServers are user-scope (global).
     if (isRecord(config.mcpServers)) {
@@ -159,16 +163,16 @@ function collectMcpServers(config, projectDir, includeAllProjectMcp) {
     // Global view: collect every project's servers; otherwise just this project's.
     if (isRecord(config.projects)) {
         const entries = includeAllProjectMcp
-            ? Object.values(config.projects)
-            : [config.projects[projectDir]];
-        for (const projectEntry of entries) {
+            ? Object.entries(config.projects)
+            : [[projectDir, config.projects[projectDir]]];
+        for (const [dir, projectEntry] of entries) {
             if (isRecord(projectEntry) && isRecord(projectEntry.mcpServers)) {
                 for (const id of Object.keys(projectEntry.mcpServers))
-                    add(id, "project");
+                    add(id, "project", dir);
             }
         }
     }
-    return out;
+    return [...byKey.values()];
 }
 /**
  * Extract `name` and `description` from a leading `---` fenced YAML block.

package/dist/deadContext.d.ts

@@ -8,6 +8,8 @@ export type DeadContextItem = {
     weightConfidence: InventoryItem["weightConfidence"];
     /** Config file the item is loaded from — the place to remove it. */
     path?: string;
+    /** Project dirs that load this item (where `claude mcp remove` must run). */
+    ownerDirs?: string[];
 };
 export type DeadContextResult = {
     /** True when we found inventory + transcripts AND at least one dead item. */

package/dist/deadContext.js

@@ -84,7 +84,8 @@ export function computeDeadContext(items, invocations, config) {
             name: item.name,
             alwaysLoadedTokens: item.alwaysLoadedTokens,
             weightConfidence: item.weightConfidence,
-            path: item.path
+            path: item.path,
+            ownerDirs: item.ownerDirs
         });
     }
     dead.sort((a, b) => b.alwaysLoadedTokens - a.alwaysLoadedTokens);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@agent-finops/core",
-  "version": "0.4.4",
+  "version": "0.5.1",
   "type": "module",
   "main": "./dist/index.js",
   "types": "./dist/index.d.ts",