refactor: improve text parser robustness and test coverage

- Replace fixed-width indentation (\s{4}) with flexible indentation (\s+)
- Capture unavailable suffix in regex group instead of line.includes()
- Add test for merge logic with text-only devices (core bug fix scenario)
- Update resource test to handle new fallback behavior

Addresses CodeRabbit review feedback on PR #118

Author: andreyz

src/mcp/resources/__tests__/simulators.test.ts

@@ -65,16 +65,36 @@ describe('simulators resource', () => {
       expect(result.contents[0].text).toContain('Command failed');
     });
 
-    it('should handle JSON parsing errors', async () => {
-      const mockExecutor = createMockExecutor({
-        success: true,
-        output: 'invalid json',
-      });
+    it('should handle JSON parsing errors and fall back to text parsing', async () => {
+      const mockTextOutput = `== Devices ==
+-- iOS 17.0 --
+    iPhone 15 (test-uuid-123) (Shutdown)`;
+
+      const mockExecutor = async (command: string[]) => {
+        // JSON command returns invalid JSON
+        if (command.includes('--json')) {
+          return {
+            success: true,
+            output: 'invalid json',
+            error: undefined,
+            process: { pid: 12345 },
+          };
+        }
+
+        // Text command returns valid text output
+        return {
+          success: true,
+          output: mockTextOutput,
+          error: undefined,
+          process: { pid: 12345 },
+        };
+      };
 
       const result = await simulatorsResourceLogic(mockExecutor);
 
       expect(result.contents).toHaveLength(1);
-      expect(result.contents[0].text).toBe('invalid json');
+      expect(result.contents[0].text).toContain('iPhone 15 (test-uuid-123)');
+      expect(result.contents[0].text).toContain('iOS 17.0');
     });
 
     it('should handle spawn errors', async () => {

src/mcp/tools/simulator/__tests__/list_sims.test.ts

@@ -176,6 +176,68 @@ Next Steps:
 iOS 17.0:
 - iPhone 15 (test-uuid-123) [Booted]
 
+Next Steps:
+1. Boot a simulator: boot_sim({ simulatorUuid: 'UUID_FROM_ABOVE' })
+2. Open the simulator UI: open_sim({})
+3. Build for simulator: build_sim({ scheme: 'YOUR_SCHEME', simulatorId: 'UUID_FROM_ABOVE' })
+4. Get app path: get_sim_app_path({ scheme: 'YOUR_SCHEME', platform: 'iOS Simulator', simulatorId: 'UUID_FROM_ABOVE' })`,
+          },
+        ],
+      });
+    });
+
+    it('should merge devices from text that are missing from JSON', async () => {
+      const mockJsonOutput = JSON.stringify({
+        devices: {
+          'iOS 18.6': [
+            {
+              name: 'iPhone 15',
+              udid: 'json-uuid-123',
+              isAvailable: true,
+              state: 'Shutdown',
+            },
+          ],
+        },
+      });
+
+      const mockTextOutput = `== Devices ==
+-- iOS 18.6 --
+    iPhone 15 (json-uuid-123) (Shutdown)
+-- iOS 26.0 --
+    iPhone 17 Pro (text-uuid-456) (Shutdown)`;
+
+      const mockExecutor = async (command: string[]) => {
+        if (command.includes('--json')) {
+          return {
+            success: true,
+            output: mockJsonOutput,
+            error: undefined,
+            process: { pid: 12345 },
+          };
+        }
+        return {
+          success: true,
+          output: mockTextOutput,
+          error: undefined,
+          process: { pid: 12345 },
+        };
+      };
+
+      const result = await list_simsLogic({ enabled: true }, mockExecutor);
+
+      // Should contain both iOS 18.6 from JSON and iOS 26.0 from text
+      expect(result).toEqual({
+        content: [
+          {
+            type: 'text',
+            text: `Available iOS Simulators:
+
+iOS 18.6:
+- iPhone 15 (json-uuid-123)
+
+iOS 26.0:
+- iPhone 17 Pro (text-uuid-456)
+
 Next Steps:
 1. Boot a simulator: boot_sim({ simulatorUuid: 'UUID_FROM_ABOVE' })
 2. Open the simulator UI: open_sim({})

src/mcp/tools/simulator/list_sims.ts

@@ -42,11 +42,11 @@ function parseTextOutput(textOutput: string): SimulatorDevice[] {
     // Match device lines like "    iPhone 17 Pro (UUID) (Booted)"
     // UUID pattern is flexible to handle test UUIDs like "test-uuid-123"
     const deviceMatch = line.match(
-      /^\s{4}(.+?)\s+\(([^)]+)\)\s+\((Booted|Shutdown|Booting|Shutting Down)\)(?:\s+\(unavailable.*\))?$/i,
+      /^\s+(.+?)\s+\(([^)]+)\)\s+\((Booted|Shutdown|Booting|Shutting Down)\)(\s+\(unavailable.*\))?$/i,
     );
     if (deviceMatch && currentRuntime) {
-      const [, name, udid, state] = deviceMatch;
-      const isUnavailable = line.includes('unavailable');
+      const [, name, udid, state, unavailableSuffix] = deviceMatch;
+      const isUnavailable = Boolean(unavailableSuffix);
       if (!isUnavailable) {
         devices.push({
           name: name.trim(),