refactor(satellite): update caching logic for tools and resources

Author: Lasim

services/satellite/src/services/stdio-tool-discovery-manager.ts

@@ -214,8 +214,9 @@ export class StdioToolDiscoveryManager {
           ...(tool._meta ? { _meta: tool._meta } : {})
         };
 
-        this.toolCache.set(namespacedName, cachedTool);
-        toolSet.add(namespacedName);
+        const cacheKey = `${installationName}::${namespacedName}`;
+        this.toolCache.set(cacheKey, cachedTool);
+        toolSet.add(cacheKey);
         cachedTools.push(cachedTool);
       }
 
@@ -363,7 +364,10 @@ export class StdioToolDiscoveryManager {
    * Get a specific tool by namespaced name
    */
   getTool(namespacedName: string): CachedStdioTool | null {
-    return this.toolCache.get(namespacedName) || null;
+    for (const tool of this.toolCache.values()) {
+      if (tool.namespacedName === namespacedName) return tool;
+    }
+    return null;
   }
 
   /**

services/satellite/src/services/unified-resource-discovery-manager.ts

@@ -133,8 +133,9 @@ export class UnifiedResourceDiscoveryManager {
         discoveredAt
       };
 
-      this.resourceCache.set(namespacedUri, cached);
-      resourceUriSet.add(namespacedUri);
+      const cacheKey = `${installationName}::${namespacedUri}`;
+      this.resourceCache.set(cacheKey, cached);
+      resourceUriSet.add(cacheKey);
     }
     this.resourcesByServer.set(installationName, resourceUriSet);
 
@@ -157,8 +158,9 @@ export class UnifiedResourceDiscoveryManager {
         discoveredAt
       };
 
-      this.templateCache.set(namespacedUri, cached);
-      templateUriSet.add(namespacedUri);
+      const cacheKey = `${installationName}::${namespacedUri}`;
+      this.templateCache.set(cacheKey, cached);
+      templateUriSet.add(cacheKey);
     }
     this.templatesByServer.set(installationName, templateUriSet);
 
@@ -194,7 +196,10 @@ export class UnifiedResourceDiscoveryManager {
    * Get resource by namespaced URI
    */
   getResource(namespacedUri: string): UnifiedCachedResource | null {
-    return this.resourceCache.get(namespacedUri) || null;
+    for (const resource of this.resourceCache.values()) {
+      if (resource.namespacedUri === namespacedUri) return resource;
+    }
+    return null;
   }
 
   /**