Add regression test for #226 (snapshot-ui CLI crash)

src/utils/responses/__tests__/next-steps-renderer.test.ts

@@ -223,6 +223,30 @@ describe('next-steps-renderer', () => {
       expect(result).toContain('1. Take a look at the screenshot');
       expect(result).toContain('2. Open simulator: xcodebuildmcp open-sim');
     });
+
+    it('should not throw when cliTool is absent and tool name contains underscores (regression for #226)', () => {
+      // In v2.0.7, snapshot_ui next steps referenced 'tap_coordinate' and 'take_screenshot',
+      // which had no catalog entries, so enrichNextStepsForCli left cliTool undefined.
+      // The renderer then threw: "Next step for tool 'tap_coordinate' is missing cliTool".
+      // The fix: fall back to toKebabCase(tool) instead of throwing.
+      const steps: NextStep[] = [
+        {
+          tool: 'tap_coordinate',
+          label: 'Tap on element',
+          params: { simulatorId: 'ABC', x: 0, y: 0 },
+        },
+        {
+          tool: 'take_screenshot',
+          label: 'Take screenshot for verification',
+          params: { simulatorId: 'ABC' },
+        },
+      ];
+
+      expect(() => renderNextStepsSection(steps, 'cli')).not.toThrow();
+      const result = renderNextStepsSection(steps, 'cli');
+      expect(result).toContain('xcodebuildmcp tap-coordinate');
+      expect(result).toContain('xcodebuildmcp take-screenshot');
+    });
   });
 
   describe('processToolResponse', () => {