fix: address review feedback across remaining tools migration

- Restore dropped coverage calculation assertions in tests
- Wrap runCommand 'process interrupt' in enqueue for thread safety
- Add mapError to swift_package_test withErrorHandling to prevent
  double header emission on error
- Wrap doctor runDoctor body in try/finally to restore env variable

Author: cameroncooke

src/mcp/tools/coverage/__tests__/get_coverage_report.test.ts

@@ -181,6 +181,8 @@ describe('get_coverage_report', () => {
       expect(result.isError).toBeUndefined();
       expect(result.content.length).toBeGreaterThanOrEqual(1);
       const text = allText(result);
+      expect(text).toContain('Overall: 24.7%');
+      expect(text).toContain('180/730 lines');
       const coreIdx = text.indexOf('Core');
       const appIdx = text.indexOf('MyApp.app');
       const testIdx = text.indexOf('MyAppTests.xctest');
@@ -218,6 +220,9 @@ describe('get_coverage_report', () => {
 
       expect(result.isError).toBeUndefined();
       expect(result.content.length).toBeGreaterThan(0);
+      const text = allText(result);
+      expect(text).toContain('Core: 10.0%');
+      expect(text).toContain('MyApp.app: 50.0%');
     });
   });
 

src/mcp/tools/doctor/doctor.ts

@@ -167,6 +167,7 @@ export async function runDoctor(params: DoctorParams, deps: DoctorDependencies)
   const prevSilence = process.env.XCODEBUILDMCP_SILENCE_LOGS;
   process.env.XCODEBUILDMCP_SILENCE_LOGS = 'true';
   log('info', `${LOG_PREFIX}: Running doctor tool`);
+  try {
 
   const xcodemakeEnabled = deps.features.isXcodemakeEnabled();
   const requiredBinaries = ['axe', 'mise', ...(xcodemakeEnabled ? ['xcodemake'] : [])];
@@ -475,11 +476,6 @@ export async function runDoctor(params: DoctorParams, deps: DoctorDependencies)
 
   events.push(statusLine('success', 'Doctor diagnostics complete'));
 
-  if (prevSilence === undefined) {
-    delete process.env.XCODEBUILDMCP_SILENCE_LOGS;
-  } else {
-    process.env.XCODEBUILDMCP_SILENCE_LOGS = prevSilence;
-  }
   const rendered = renderEvents(events, 'text');
   const hasError = events.some(
     (e) =>
@@ -491,6 +487,14 @@ export async function runDoctor(params: DoctorParams, deps: DoctorDependencies)
     isError: hasError || undefined,
     _meta: { events: [...events] },
   };
+
+  } finally {
+    if (prevSilence === undefined) {
+      delete process.env.XCODEBUILDMCP_SILENCE_LOGS;
+    } else {
+      process.env.XCODEBUILDMCP_SILENCE_LOGS = prevSilence;
+    }
+  }
 }
 
 export async function doctorLogic(params: DoctorParams, executor: CommandExecutor) {

src/mcp/tools/swift-package/swift_package_test.ts

@@ -110,6 +110,12 @@ export async function swift_package_testLogic(
       header: headerEvent,
       errorMessage: ({ message }) => `Failed to execute swift test: ${message}`,
       logMessage: ({ message }) => `Swift package test failed: ${message}`,
+      mapError: ({ message, headerEvent: hdr, emit }) => {
+        if (emit) {
+          emit(hdr);
+          emit(statusLine('error', `Failed to execute swift test: ${message}`));
+        }
+      },
     },
   );
 }

src/utils/debugger/backends/dap-backend.ts

@@ -137,8 +137,10 @@ class DapBackend implements DebuggerBackend {
 
     const normalizedCommand = command.trim().toLowerCase();
     if (normalizedCommand === 'process interrupt') {
-      await this.pauseExecution(opts?.timeoutMs);
-      return 'Process interrupted.';
+      return this.enqueue(async () => {
+        await this.pauseExecution(opts?.timeoutMs);
+        return 'Process interrupted.';
+      });
     }
 
     try {