Fix asymmetric workflow normalization in tool registry

Normalize requestedWorkflows before passing to selectWorkflowsForMcp to ensure consistent matching against workflow IDs. Previously, mixed-case workflow names would pass the unknown workflow check but fail to match during selection, resulting in silent no-ops.

Applied via @cursor push command

Author: cursoragent

src/utils/tool-registry.ts

@@ -187,12 +187,17 @@ export async function applyWorkflowSelectionFromManifest(
   }
   const allWorkflows = [...manifest.workflows.values(), ...customSelection.workflows];
 
+  // Normalize requested workflows for consistent matching
+  const normalizedRequestedWorkflows = requestedWorkflows
+    ?.map(normalizeName)
+    .filter((name) => name.length > 0);
+
   // Select workflows using manifest-driven rules
-  const selectedWorkflows = selectWorkflowsForMcp(allWorkflows, requestedWorkflows, ctx);
+  const selectedWorkflows = selectWorkflowsForMcp(allWorkflows, normalizedRequestedWorkflows, ctx);
   const knownWorkflowIds = new Set(allWorkflows.map((workflow) => workflow.id));
-  const unknownRequestedWorkflows = (requestedWorkflows ?? [])
-    .map(normalizeName)
-    .filter((workflowName) => workflowName.length > 0 && !knownWorkflowIds.has(workflowName));
+  const unknownRequestedWorkflows = (normalizedRequestedWorkflows ?? []).filter(
+    (workflowName) => !knownWorkflowIds.has(workflowName),
+  );
   if (unknownRequestedWorkflows.length > 0) {
     const uniqueUnknownRequestedWorkflows = [...new Set(unknownRequestedWorkflows)];
     log(