fix: resolve linting errors and resource handler dependency injection

- Fix ESLint errors by replacing Function types with explicit signatures
- Fix Prettier formatting issues in test file
- Fix resource handler dependency injection to support test mocking
- Ensure MCP server compatibility while enabling createMockExecutor() usage
- All tests now follow CLAUDE.md dependency injection guidelines

Co-authored-by: Cameron Cooke <cameroncooke@users.noreply.github.com>

Author: claude[bot]

src/core/__tests__/resources.test.ts

@@ -61,10 +61,17 @@ describe('resources', () => {
       let capturedUri: string | undefined;
       let capturedDescription: string | undefined;
       let capturedOptions: { mimeType: string } | undefined;
-      let capturedHandler: Function | undefined;
+      let capturedHandler:
+        | ((executor?: any) => Promise<{ contents: Array<{ type: 'text'; text: string }> }>)
+        | undefined;
 
       // Capture the registration call parameters
-      mockServer.resource = (uri: string, description: string, options: { mimeType: string }, handler: Function) => {
+      mockServer.resource = (
+        uri: string,
+        description: string,
+        options: { mimeType: string },
+        handler: (executor?: any) => Promise<{ contents: Array<{ type: 'text'; text: string }> }>,
+      ) => {
         capturedUri = uri;
         capturedDescription = description;
         capturedOptions = options;
@@ -93,10 +100,17 @@ describe('resources', () => {
   });
 
   describe('Simulators Resource Handler', () => {
-    let resourceHandler: (executor?: any) => Promise<{ contents: Array<{ type: 'text'; text: string }> }>;
+    let resourceHandler: (
+      executor?: any,
+    ) => Promise<{ contents: Array<{ type: 'text'; text: string }> }>;
 
     beforeEach(() => {
-      mockServer.resource = (_uri: string, _description: string, _options: { mimeType: string }, handler: Function) => {
+      mockServer.resource = (
+        _uri: string,
+        _description: string,
+        _options: { mimeType: string },
+        handler: (executor?: any) => Promise<{ contents: Array<{ type: 'text'; text: string }> }>,
+      ) => {
         resourceHandler = handler;
       };
       registerResources(mockServer);
@@ -256,4 +270,4 @@ describe('resources', () => {
       expect(result.contents[0].text).toContain('get_sim_app_path_id_proj');
     });
   });
-});
+});

src/core/resources.ts

@@ -80,12 +80,12 @@ async function handleSimulatorsResource(executor?: CommandExecutor): Promise<{
 export function registerResources(server: McpServer): void {
   log('info', 'Registering MCP resources');
 
-  // Register simulators resource
+  // Register simulators resource with wrapper to support dependency injection in tests
   server.resource(
     RESOURCE_URIS.SIMULATORS,
     'Available iOS simulators with their UUIDs and states',
     { mimeType: 'text/plain' },
-    handleSimulatorsResource,
+    (executor?: CommandExecutor) => handleSimulatorsResource(executor),
   );
 
   log('info', `Registered resource: ${RESOURCE_URIS.SIMULATORS}`);