fix: Update VSCode configuration and tool registration logic

Changes:
- Adjusted `mcp.json` to ensure proper formatting and maintain consistency in server configurations.
- Enhanced `settings.json` to include allowed models for the `XcodeBuildMCP-Dev` server.
- Modified `tool-registry.ts` to ensure the `session-management` workflow is always included, as it is required by other tools, improving tool registration reliability.

This update aims to streamline the development experience and ensure necessary workflows are always available.

Author: cameroncooke

.vscode/mcp.json

@@ -1,35 +1,35 @@
 {
-    "servers": {
-      "XcodeBuildMCP": {
-        "type": "stdio",
-        "command": "npx",
-        "args": [
-          "-y",
-          "xcodebuildmcp@latest"
-        ],
-        "env": {
-          "XCODEBUILDMCP_DEBUG": "true",
-          "XCODEBUILDMCP_DYNAMIC_TOOLS": "true",
-          "INCREMENTAL_BUILDS_ENABLED": "false",
-          "XCODEBUILDMCP_IOS_TEMPLATE_PATH": "/Volumes/Developer/XcodeBuildMCP-iOS-Template",
-          "XCODEBUILDMCP_MACOS_TEMPLATE_PATH": "/Volumes/Developer/XcodeBuildMCP-macOS-Template"
-        }
-      },      
-      "XcodeBuildMCP-Dev": {
-        "type": "stdio",
-        "command": "node",
-        "args": [
-          "--inspect-brk=9999",
-          "--trace-warnings",
-          "/Users/cameroncooke/Developer/XcodeBuildMCP/build/index.js"
-        ],
-        "env": {
-          "XCODEBUILDMCP_DEBUG": "true",
-          "XCODEBUILDMCP_DYNAMIC_TOOLS": "true",
-          "INCREMENTAL_BUILDS_ENABLED": "false",
-          "XCODEBUILDMCP_IOS_TEMPLATE_PATH": "/Volumes/Developer/XcodeBuildMCP-iOS-Template",
-          "XCODEBUILDMCP_MACOS_TEMPLATE_PATH": "/Volumes/Developer/XcodeBuildMCP-macOS-Template"
-        }
-      },
-    }
+  "servers": {
+    "XcodeBuildMCP": {
+      "type": "stdio",
+      "command": "npx",
+      "args": [
+        "-y",
+        "xcodebuildmcp@latest"
+      ],
+      "env": {
+        "XCODEBUILDMCP_DEBUG": "true",
+        "XCODEBUILDMCP_DYNAMIC_TOOLS": "true",
+        "INCREMENTAL_BUILDS_ENABLED": "false",
+        "XCODEBUILDMCP_IOS_TEMPLATE_PATH": "/Volumes/Developer/XcodeBuildMCP-iOS-Template",
+        "XCODEBUILDMCP_MACOS_TEMPLATE_PATH": "/Volumes/Developer/XcodeBuildMCP-macOS-Template"
+      }
+    },
+    "XcodeBuildMCP-Dev": {
+      "type": "stdio",
+      "command": "node",
+      "args": [
+        "--inspect=9999",
+        "--trace-warnings",
+        "/Users/cameroncooke/Developer/XcodeBuildMCP/build/index.js"
+      ],
+      "env": {
+        "XCODEBUILDMCP_DEBUG": "true",
+        "XCODEBUILDMCP_DYNAMIC_TOOLS": "true",
+        "INCREMENTAL_BUILDS_ENABLED": "false",
+        "XCODEBUILDMCP_IOS_TEMPLATE_PATH": "/Volumes/Developer/XcodeBuildMCP-iOS-Template",
+        "XCODEBUILDMCP_MACOS_TEMPLATE_PATH": "/Volumes/Developer/XcodeBuildMCP-macOS-Template"
+      }
+    },
+  }
 }

.vscode/settings.json

@@ -38,7 +38,10 @@
   },
   "chat.mcp.serverSampling": {
     "XcodeBuildMCP/.vscode/mcp.json: XcodeBuildMCP-Dev": {
-      "allowedDuringChat": true
+      "allowedDuringChat": true,
+      "allowedModels": [
+        "copilot/gpt-5.2"
+      ]
     }
   },
 }

XcodeBuildMCP.code-workspace

@@ -0,0 +1,13 @@
+{
+	"folders": [
+		{
+			"path": "."
+		},
+		{
+			"path": "../XcodeBuildMCP-iOS-Template"
+		},
+		{
+			"path": "../XcodeBuildMCP-macOS-Template"
+		}
+	]
+}

src/utils/tool-registry.ts

@@ -146,6 +146,9 @@ export async function registerDiscoveryTools(server: McpServer): Promise<void> {
   log('info', `✅ Registered ${registeredCount} discovery tools in dynamic mode.`);
 }
 
+// Workflow that must always be included as other tools depend on it
+const REQUIRED_WORKFLOW = 'session-management';
+
 /**
  * Register selected workflows based on environment variable
  */
@@ -157,8 +160,15 @@ export async function registerSelectedWorkflows(
   const workflowGroups = await loadWorkflowGroups();
   const selectedTools = [];
 
-  for (const workflowName of workflowNames) {
-    const workflow = workflowGroups.get(workflowName.trim());
+  // Ensure session-management is always included as it's required by other tools
+  const normalizedNames = workflowNames.map((name) => name.trim());
+  if (!normalizedNames.includes(REQUIRED_WORKFLOW)) {
+    normalizedNames.unshift(REQUIRED_WORKFLOW);
+    log('info', `📌 Auto-including required workflow: ${REQUIRED_WORKFLOW}`);
+  }
+
+  for (const workflowName of normalizedNames) {
+    const workflow = workflowGroups.get(workflowName);
     if (workflow) {
       for (const tool of workflow.tools) {
         selectedTools.push({
@@ -181,7 +191,7 @@ export async function registerSelectedWorkflows(
 
   log(
     'info',
-    `✅ Registered ${selectedTools.length} tools from workflows: ${workflowNames.join(', ')}`,
+    `✅ Registered ${selectedTools.length} tools from workflows: ${normalizedNames.join(', ')}`,
   );
 }