fix: deduplicate tool catalog entries and correct screenshot device matching

- Deduplicate tools in buildToolCatalogFromManifest when the same tool
  appears in multiple workflows, preventing ambiguous resolution via
  kebab-case MCP name lookup and non-deterministic Map overwrites
- Fix screenshot window title matching to use the actual Simulator title
  separator (en-dash/hyphen) instead of bare space, preventing
  false-positives like "iPhone 15" matching "iPhone 15 Pro"
- Remove dead tool-visibility.ts superseded by the predicate system

Author: cameroncooke

src/mcp/tools/ui-automation/screenshot.ts

@@ -59,8 +59,8 @@ function escapeSwiftStringLiteral(value: string): string {
  */
 function getWindowDetectionSwiftCode(deviceName: string): string {
   const escapedDeviceName = escapeSwiftStringLiteral(deviceName);
-  // Use hasPrefix + boundary check to avoid matching "iPhone 15" when looking for "iPhone 15 Pro"
-  // Window titles are formatted like "iPhone 15 Pro – iOS 17.2"
+  // Match by title separator (en-dash) to avoid "iPhone 15" matching "iPhone 15 Pro"
+  // Window titles are formatted like "iPhone 15 Pro \u{2013} iOS 17.2"
   return `
 import Cocoa
 import CoreGraphics
@@ -71,8 +71,9 @@ if let wins = CGWindowListCopyWindowInfo(opts, kCGNullWindowID) as? [[String: An
     if let o = w[kCGWindowOwnerName as String] as? String, o == "Simulator",
        let b = w[kCGWindowBounds as String] as? [String: Any],
        let n = w[kCGWindowName as String] as? String {
-      // Check for exact match: name starts with deviceName followed by separator or end
-      let isMatch = n == deviceName || n.hasPrefix(deviceName + " ")
+      // Check for exact match: name equals deviceName or is followed by the title separator
+      // Window titles use en-dash: "iPhone 15 Pro \u{2013} iOS 17.2"
+      let isMatch = n == deviceName || n.hasPrefix(deviceName + " \\u{2013}") || n.hasPrefix(deviceName + " -")
       if isMatch {
         print("\\(b["Width"] as? Int ?? 0),\\(b["Height"] as? Int ?? 0)")
         break

src/runtime/tool-catalog.ts

@@ -130,9 +130,12 @@ export async function buildToolCatalogFromManifest(opts: {
   // Cache imported modules to avoid re-importing the same tool
   const moduleCache = new Map<string, Awaited<ReturnType<typeof importToolModule>>>();
   const tools: ToolDefinition[] = [];
+  const seenToolIds = new Set<string>();
 
   for (const workflow of filteredWorkflows) {
     for (const toolId of workflow.tools) {
+      if (seenToolIds.has(toolId)) continue;
+
       const toolManifest = manifest.tools.get(toolId);
       if (!toolManifest) continue;
 
@@ -154,6 +157,7 @@ export async function buildToolCatalogFromManifest(opts: {
         }
       }
 
+      seenToolIds.add(toolId);
       const cliName = getEffectiveCliName(toolManifest);
       tools.push({
         cliName,