fix: cubic feedback

Author: RomneyDa

core/context/mcp/MCPManagerSingleton.ts

@@ -27,7 +27,11 @@ export class MCPManagerSingleton {
         conn.status = "not-connected";
         await this.refreshConnection(serverId);
       } else {
-        await conn.disconnect(true);
+        try {
+          await conn.disconnect(true);
+        } catch (e) {
+          console.error(`Error disconnecting from MCP server ${serverId}`, e);
+        }
       }
     }
   }

core/context/providers/RulesContextProvider.ts

@@ -22,7 +22,7 @@ class RulesContextProvider extends BaseContextProvider {
   // This is only used within this class. Worst case if there are exact duplicates is that one always calls the other, but this is an extreme edge case
   // Can eventually pull in more metadata, but this is experimental
   private getIdFromRule(rule: RuleWithSource): string {
-    return rule.sourceFile ?? rule.slug ?? rule.name ?? rule.rule;
+    return rule.slug ?? rule.sourceFile ?? rule.name ?? rule.rule;
   }
 
   private getNameFromRule(rule: RuleWithSource): string {

gui/src/components/mainInput/Lump/useEditBlock.ts

@@ -25,7 +25,7 @@ export function useEditBlock() {
         path: selectedProfile.uri,
       });
     } else if (
-      selectedProfile?.fullSlug.ownerSlug &&
+      selectedProfile?.fullSlug?.ownerSlug &&
       selectedProfile?.fullSlug.packageSlug
     ) {
       ideMessenger.post("controlPlane/openUrl", {

gui/src/pages/config/components/ToolPoliciesGroup.tsx

@@ -31,7 +31,7 @@ export function ToolPoliciesGroup({
   const availableTools = useAppSelector((state) => state.config.config.tools);
   const tools = useMemo(() => {
     return availableTools.filter((t) => t.group === groupName);
-  }, [availableTools]);
+  }, [availableTools, groupName]);
 
   const toolGroupSettings = useAppSelector(
     (state) => state.ui.toolGroupSettings,