fix: support long macOS app names

Author: cameroncooke

src/mcp/tools/macos/__tests__/stop_mac_app.test.ts

@@ -88,8 +88,7 @@ describe('stop_mac_app plugin', () => {
       );
 
       expect(calls).toHaveLength(1);
-      expect(calls[0]).toEqual(['pkill', '-x', '--', 'Brimday']);
-      expect(calls[0]).not.toContain('-f');
+      expect(calls[0]).toEqual(['pkill', '-f', '--', '^(.*/)?Brimday( |$)']);
     });
 
     it('should preserve long app executable names', async () => {
@@ -107,7 +106,7 @@ describe('stop_mac_app plugin', () => {
         ),
       );
 
-      expect(calls[0]).toEqual(['pkill', '-x', '--', 'ThisIsAVeryLongApplicationName']);
+      expect(calls[0]).toEqual(['pkill', '-f', '--', '^(.*/)?ThisIsAVeryLongApplicationName( |$)']);
     });
 
     it('should treat app names as literal process names', async () => {
@@ -125,7 +124,7 @@ describe('stop_mac_app plugin', () => {
         ),
       );
 
-      expect(calls[0]).toEqual(['pkill', '-x', '--', '-Example\\.\\*\\[Test\\]']);
+      expect(calls[0]).toEqual(['pkill', '-f', '--', '^(.*/)?-Example\\.\\*\\[Test\\]( |$)']);
     });
 
     it('should prioritize processId over appName', async () => {

src/mcp/tools/macos/stop_mac_app.ts

@@ -83,7 +83,12 @@ export function createStopMacAppExecutor(
       const command =
         params.processId !== undefined
           ? ['kill', String(params.processId)]
-          : ['pkill', '-x', '--', escapeExtendedRegularExpression(params.appName!)];
+          : [
+              'pkill',
+              '-f',
+              '--',
+              `^(.*/)?${escapeExtendedRegularExpression(params.appName!)}( |$)`,
+            ];
       const result = await executor(command, 'Stop macOS App');
 
       if (!result.success) {