aws-runtime-bridge 1.2.0 → 1.3.0

package/dist/services/mcp-launch-binding-queue.test.js

@@ -4,7 +4,7 @@ describe('mcp launch binding queue', () => {
     afterEach(() => {
         clearMcpLaunchBindingQueues();
     });
-    it('claims same-workspace bindings only for the requested agent identity', () => {
+    it('claims same-workspace same-server bindings from the front of the queue', () => {
         const firstBinding = enqueueMcpLaunchBinding({
             agentId: 'agent-a',
             workspacePath: 'C:/repo/demo',
@@ -22,15 +22,11 @@ describe('mcp launch binding queue', () => {
             schedulerBaseUrl: 'http://server-a:7380',
         });
         expect(getMcpLaunchQueueSize('C:/repo/demo')).toBe(2);
-        const second = claimMcpLaunchBinding({
-            agentId: 'agent-b',
-            bindingId: secondBinding?.id,
+        const first = claimMcpLaunchBinding({
             workspacePath: 'C:/repo/demo',
             serverUrl: 'ws://server-a:7380/ws/agent',
         });
-        const first = claimMcpLaunchBinding({
-            agentId: 'agent-a',
-            bindingId: firstBinding?.id,
+        const second = claimMcpLaunchBinding({
             workspacePath: 'C:/repo/demo',
             serverUrl: 'ws://server-a:7380/ws/agent',
         });
@@ -40,68 +36,77 @@ describe('mcp launch binding queue', () => {
         expect(second?.runtimeAccessToken).toBe('token-b');
         expect(getMcpLaunchQueueSize('C:/repo/demo')).toBe(0);
     });
-    it('does not consume a binding when the requested agent id does not match', () => {
+    it('claims by workspace path and server URL regardless of agent id', () => {
         const binding = enqueueMcpLaunchBinding({
             agentId: 'agent-a',
             workspacePath: 'C:/repo/demo',
             serverUrl: 'ws://server:7380/ws/agent',
         });
         expect(claimMcpLaunchBinding({
-            agentId: 'agent-b',
-            bindingId: binding?.id,
-            workspacePath: 'C:/repo/demo',
-            serverUrl: 'ws://server:7380/ws/agent',
-        })).toBeNull();
-        expect(getMcpLaunchQueueSize('C:/repo/demo')).toBe(1);
-        expect(claimMcpLaunchBinding({
-            agentId: 'agent-a',
-            bindingId: binding?.id,
             workspacePath: 'C:/repo/demo',
             serverUrl: 'ws://server:7380/ws/agent',
         })?.agentId).toBe('agent-a');
+        expect(binding?.agentId).toBe('agent-a');
     });
-    it('does not consume a binding when the binding id is missing or wrong', () => {
+    it('ignores explicit binding id and claims the first matching queue entry', () => {
         const binding = enqueueMcpLaunchBinding({
             agentId: 'agent-a',
             workspacePath: 'C:/repo/demo',
             serverUrl: 'ws://server:7380/ws/agent',
         });
         expect(claimMcpLaunchBinding({
-            agentId: 'agent-a',
             workspacePath: 'C:/repo/demo',
             serverUrl: 'ws://server:7380/ws/agent',
-        })).toBeNull();
+        })?.agentId).toBe('agent-a');
+        expect(binding?.id).toBeTruthy();
+    });
+    it('matches by server URL as well as workspace path', () => {
+        const firstBinding = enqueueMcpLaunchBinding({ agentId: 'agent-a', workspacePath: 'C:/repo/demo', serverUrl: 'ws://server-a:7380/ws/agent' });
+        const secondBinding = enqueueMcpLaunchBinding({ agentId: 'agent-b', workspacePath: 'C:/repo/demo', serverUrl: 'ws://server-b:7380/ws/agent' });
+        expect(claimMcpLaunchBinding({ workspacePath: 'C:/repo/demo', serverUrl: 'ws://server-b:7380/ws/agent' })?.agentId).toBe('agent-b');
+        expect(claimMcpLaunchBinding({ workspacePath: 'C:/repo/demo', serverUrl: 'ws://server-a:7380/ws/agent' })?.agentId).toBe('agent-a');
+    });
+    it('matches http scheduler URLs and websocket agent URLs by scheduler origin', () => {
+        enqueueMcpLaunchBinding({ agentId: 'agent-a', workspacePath: 'C:/repo/demo', schedulerBaseUrl: 'http://server-a:7380' });
         expect(claimMcpLaunchBinding({
+            workspacePath: 'C:/repo/demo',
+            serverUrl: 'ws://server-a:7380/ws/agent',
+        })?.agentId).toBe('agent-a');
+    });
+    it('repairs duplicated scheduler ports when enqueuing launch bindings', () => {
+        const binding = enqueueMcpLaunchBinding({
             agentId: 'agent-a',
-            bindingId: 'wrong-binding',
             workspacePath: 'C:/repo/demo',
-            serverUrl: 'ws://server:7380/ws/agent',
-        })).toBeNull();
-        expect(getMcpLaunchQueueSize('C:/repo/demo')).toBe(1);
+            schedulerBaseUrl: 'http://127.0.0.1:8080:8080',
+        });
+        expect(binding?.serverUrl).toBe('http://127.0.0.1:8080');
+        expect(binding?.schedulerBaseUrl).toBe('http://127.0.0.1:8080');
         expect(claimMcpLaunchBinding({
+            workspacePath: 'C:/repo/demo',
+            serverUrl: 'ws://127.0.0.1:8080/ws/agent',
+        })?.schedulerBaseUrl).toBe('http://127.0.0.1:8080');
+    });
+    it('falls back to the only same-workspace binding when scheduler URLs differ', () => {
+        enqueueMcpLaunchBinding({
             agentId: 'agent-a',
-            bindingId: binding?.id,
             workspacePath: 'C:/repo/demo',
-            serverUrl: 'ws://server:7380/ws/agent',
+            serverUrl: 'ws://127.0.0.1:7380/ws/agent',
+        });
+        expect(claimMcpLaunchBinding({
+            workspacePath: 'C:/repo/demo',
+            serverUrl: 'ws://101.42.200.84:7380/ws/agent',
         })?.agentId).toBe('agent-a');
     });
-    it('matches by server URL as well as workspace path', () => {
-        const firstBinding = enqueueMcpLaunchBinding({ agentId: 'agent-a', workspacePath: 'C:/repo/demo', serverUrl: 'ws://server-a:7380/ws/agent' });
-        const secondBinding = enqueueMcpLaunchBinding({ agentId: 'agent-b', workspacePath: 'C:/repo/demo', serverUrl: 'ws://server-b:7380/ws/agent' });
-        expect(claimMcpLaunchBinding({ agentId: 'agent-b', bindingId: secondBinding?.id, workspacePath: 'C:/repo/demo', serverUrl: 'ws://server-b:7380/ws/agent' })?.agentId).toBe('agent-b');
-        expect(claimMcpLaunchBinding({ agentId: 'agent-a', bindingId: firstBinding?.id, workspacePath: 'C:/repo/demo', serverUrl: 'ws://server-a:7380/ws/agent' })?.agentId).toBe('agent-a');
-    });
     it('does not mix different workspace paths', () => {
         const firstBinding = enqueueMcpLaunchBinding({ agentId: 'agent-a', workspacePath: 'C:/repo/a', serverUrl: 'ws://server:7380/ws/agent' });
         const secondBinding = enqueueMcpLaunchBinding({ agentId: 'agent-b', workspacePath: 'C:/repo/b', serverUrl: 'ws://server:7380/ws/agent' });
-        expect(claimMcpLaunchBinding({ agentId: 'agent-b', bindingId: secondBinding?.id, workspacePath: 'C:/repo/b', serverUrl: 'ws://server:7380/ws/agent' })?.agentId).toBe('agent-b');
-        expect(claimMcpLaunchBinding({ agentId: 'agent-a', bindingId: firstBinding?.id, workspacePath: 'C:/repo/a', serverUrl: 'ws://server:7380/ws/agent' })?.agentId).toBe('agent-a');
+        expect(claimMcpLaunchBinding({ workspacePath: 'C:/repo/b', serverUrl: 'ws://server:7380/ws/agent' })?.agentId).toBe('agent-b');
+        expect(claimMcpLaunchBinding({ workspacePath: 'C:/repo/a', serverUrl: 'ws://server:7380/ws/agent' })?.agentId).toBe('agent-a');
     });
     it('returns null for missing or invalid workspaces', () => {
         expect(enqueueMcpLaunchBinding({ agentId: '', workspacePath: 'C:/repo/demo' })).toBeNull();
         expect(enqueueMcpLaunchBinding({ agentId: 'agent-a', workspacePath: '' })).toBeNull();
-        expect(claimMcpLaunchBinding({ agentId: 'agent-a', workspacePath: 'C:/repo/missing', serverUrl: 'ws://server:7380/ws/agent' })).toBeNull();
-        expect(claimMcpLaunchBinding({ agentId: '', workspacePath: 'C:/repo/demo', serverUrl: 'ws://server:7380/ws/agent' })).toBeNull();
-        expect(claimMcpLaunchBinding({ agentId: 'agent-a', bindingId: 'binding-a', workspacePath: 'C:/repo/demo' })).toBeNull();
+        expect(claimMcpLaunchBinding({ workspacePath: 'C:/repo/missing', serverUrl: 'ws://server:7380/ws/agent' })).toBeNull();
+        expect(claimMcpLaunchBinding({ workspacePath: 'C:/repo/demo' })).toBeNull();
     });
 });

package/dist/services/runtime-binding.d.ts

@@ -14,6 +14,7 @@ export type RuntimeBindingState = {
     updatedAt?: string;
     revokedAt?: string;
 };
+export declare function normalizeSchedulerBaseUrl(value: unknown): string | undefined;
 export declare function getRuntimePairingCode(): string;
 export declare function getRuntimeBindingFilePath(): string;
 export declare function getRuntimeTokenStoreFilePath(): string;

package/dist/services/runtime-binding.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"runtime-binding.d.ts","sourceRoot":"","sources":["../../src/services/runtime-binding.ts"],"names":[],"mappings":"AASA,MAAM,MAAM,mBAAmB,GAAG;IAChC,MAAM,EAAE,UAAU,GAAG,QAAQ,CAAC;IAC9B,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,gBAAgB,CAAC,EAAE,MAAM,CAAC;IAC1B;;;;OAIG;IACH,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,SAAS,CAAC,EAAE,MAAM,CAAC;CACpB,CAAC;AAoDF,wBAAgB,qBAAqB,IAAI,MAAM,CAE9C;AAED,wBAAgB,yBAAyB,IAAI,MAAM,CAElD;AAED,wBAAgB,4BAA4B,IAAI,MAAM,CAErD;AAoBD,wBAAgB,oBAAoB,CAClC,MAAM,EAAE,OAAO,EACf,aAAa,EAAE,OAAO,GACrB,MAAM,CAYR;AAsCD,wBAAgB,4BAA4B,CAAC,KAAK,EAAE;IAClD,MAAM,EAAE,MAAM,CAAC;IACf,aAAa,EAAE,MAAM,CAAC;IACtB,WAAW,EAAE,MAAM,CAAC;CACrB,GAAG,MAAM,CAWT;AAED,wBAAgB,2BAA2B,CACzC,MAAM,EAAE,OAAO,EACf,aAAa,EAAE,OAAO,GACrB,MAAM,GAAG,SAAS,CAUpB;AAcD,wBAAgB,kBAAkB,IAAI,mBAAmB,CAWxD;AAED,wBAAgB,4BAA4B,IAAI,IAAI,CAClD,mBAAmB,EACnB,WAAW,GAAG,aAAa,CAC5B,GAAG;IAAE,MAAM,EAAE,OAAO,CAAA;CAAE,CAWtB;AAED,wBAAgB,iBAAiB,IAAI,OAAO,CAG3C;AAED,wBAAgB,2BAA2B,CAAC,KAAK,EAAE,OAAO,GAAG,OAAO,CAcnE;AAED,wBAAgB,qBAAqB,CACnC,MAAM,CAAC,EAAE,OAAO,EAChB,aAAa,CAAC,EAAE,OAAO,GACtB,MAAM,GAAG,SAAS,CAWpB;AAED,wBAAgB,0BAA0B,CAAC,IAAI,EAAE,OAAO,GAAG,OAAO,CAEjE;AAED,wBAAgB,kBAAkB,CAAC,KAAK,EAAE;IACxC,WAAW,EAAE,MAAM,CAAC;IACpB,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,gBAAgB,CAAC,EAAE,MAAM,CAAC;CAC3B,GAAG,mBAAmB,CAuCtB;AAED,wBAAgB,mBAAmB,IAAI,IAAI,CAkB1C"}
+{"version":3,"file":"runtime-binding.d.ts","sourceRoot":"","sources":["../../src/services/runtime-binding.ts"],"names":[],"mappings":"AASA,MAAM,MAAM,mBAAmB,GAAG;IAChC,MAAM,EAAE,UAAU,GAAG,QAAQ,CAAC;IAC9B,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,gBAAgB,CAAC,EAAE,MAAM,CAAC;IAC1B;;;;OAIG;IACH,WAAW,CAAC,EAAE,MAAM,CAAC;IACrB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,SAAS,CAAC,EAAE,MAAM,CAAC;IACnB,SAAS,CAAC,EAAE,MAAM,CAAC;CACpB,CAAC;AA2DF,wBAAgB,yBAAyB,CAAC,KAAK,EAAE,OAAO,GAAG,MAAM,GAAG,SAAS,CAqB5E;AAED,wBAAgB,qBAAqB,IAAI,MAAM,CAE9C;AAED,wBAAgB,yBAAyB,IAAI,MAAM,CAElD;AAED,wBAAgB,4BAA4B,IAAI,MAAM,CAErD;AAoBD,wBAAgB,oBAAoB,CAClC,MAAM,EAAE,OAAO,EACf,aAAa,EAAE,OAAO,GACrB,MAAM,CAYR;AAsCD,wBAAgB,4BAA4B,CAAC,KAAK,EAAE;IAClD,MAAM,EAAE,MAAM,CAAC;IACf,aAAa,EAAE,MAAM,CAAC;IACtB,WAAW,EAAE,MAAM,CAAC;CACrB,GAAG,MAAM,CAWT;AAED,wBAAgB,2BAA2B,CACzC,MAAM,EAAE,OAAO,EACf,aAAa,EAAE,OAAO,GACrB,MAAM,GAAG,SAAS,CAUpB;AAcD,wBAAgB,kBAAkB,IAAI,mBAAmB,CAgBxD;AAED,wBAAgB,4BAA4B,IAAI,IAAI,CAClD,mBAAmB,EACnB,WAAW,GAAG,aAAa,CAC5B,GAAG;IAAE,MAAM,EAAE,OAAO,CAAA;CAAE,CAWtB;AAED,wBAAgB,iBAAiB,IAAI,OAAO,CAG3C;AAED,wBAAgB,2BAA2B,CAAC,KAAK,EAAE,OAAO,GAAG,OAAO,CAcnE;AAED,wBAAgB,qBAAqB,CACnC,MAAM,CAAC,EAAE,OAAO,EAChB,aAAa,CAAC,EAAE,OAAO,GACtB,MAAM,GAAG,SAAS,CAepB;AAED,wBAAgB,0BAA0B,CAAC,IAAI,EAAE,OAAO,GAAG,OAAO,CAEjE;AAED,wBAAgB,kBAAkB,CAAC,KAAK,EAAE;IACxC,WAAW,EAAE,MAAM,CAAC;IACpB,UAAU,CAAC,EAAE,MAAM,CAAC;IACpB,MAAM,CAAC,EAAE,MAAM,CAAC;IAChB,gBAAgB,CAAC,EAAE,MAAM,CAAC;CAC3B,GAAG,mBAAmB,CAwCtB;AAED,wBAAgB,mBAAmB,IAAI,IAAI,CAkB1C"}

package/dist/services/runtime-binding.js

@@ -46,6 +46,33 @@ function safeTextEqual(a, b) {
 function normalizeToken(token) {
     return String(token || "").trim();
 }
+function repairDuplicatePort(raw) {
+    return raw.replace(/^((?:https?|wss?):\/\/(?:\[[^\]]+\]|[^/:?#]+):\d+):\d+(?=\/|$)/i, "$1");
+}
+export function normalizeSchedulerBaseUrl(value) {
+    const raw = normalizeToken(value);
+    if (!raw) {
+        return undefined;
+    }
+    const repaired = repairDuplicatePort(raw);
+    try {
+        const url = new URL(repaired);
+        if (url.protocol === "ws:") {
+            url.protocol = "http:";
+        }
+        else if (url.protocol === "wss:") {
+            url.protocol = "https:";
+        }
+        else {
+            url.protocol = url.protocol.toLowerCase();
+        }
+        url.hostname = url.hostname.toLowerCase();
+        return url.origin;
+    }
+    catch {
+        return repaired.replace(/\/+$/, "");
+    }
+}
 export function getRuntimePairingCode() {
     return pairingCode;
 }
@@ -56,7 +83,7 @@ export function getRuntimeTokenStoreFilePath() {
     return tokenStoreFile();
 }
 function normalizeServerIp(serverBaseUrl) {
-    const raw = String(serverBaseUrl || "").trim();
+    const raw = normalizeSchedulerBaseUrl(serverBaseUrl) || String(serverBaseUrl || "").trim();
     if (!raw) {
         return "";
     }
@@ -157,7 +184,11 @@ export function loadRuntimeBinding() {
         const raw = fs.readFileSync(bindingFile(), "utf8");
         const parsed = JSON.parse(raw);
         if (parsed?.status === "paired" && parsed.tokenHash) {
-            return parsed;
+            return {
+                ...parsed,
+                schedulerBaseUrl: normalizeSchedulerBaseUrl(parsed.schedulerBaseUrl) ||
+                    parsed.schedulerBaseUrl,
+            };
         }
     }
     catch {
@@ -195,6 +226,9 @@ export function getRuntimeAccessToken(userId, serverBaseUrl) {
     if (scopedToken) {
         return scopedToken;
     }
+    if (serverBaseUrl) {
+        return undefined;
+    }
     const state = loadRuntimeBinding();
     if (state.status !== "paired") {
         return undefined;
@@ -210,6 +244,8 @@ export function saveRuntimeBinding(input) {
         throw new Error("accessToken must be at least 16 characters");
     }
     const previous = loadRuntimeBinding();
+    const schedulerBaseUrl = normalizeSchedulerBaseUrl(input.schedulerBaseUrl) ||
+        normalizeSchedulerBaseUrl(previous.schedulerBaseUrl);
     const now = new Date().toISOString();
     const next = {
         status: "paired",
@@ -217,9 +253,7 @@ export function saveRuntimeBinding(input) {
             ? String(input.instanceId)
             : previous.instanceId,
         userId: input.userId ? String(input.userId) : previous.userId,
-        schedulerBaseUrl: input.schedulerBaseUrl
-            ? String(input.schedulerBaseUrl)
-            : previous.schedulerBaseUrl,
+        schedulerBaseUrl,
         accessToken,
         tokenHash: hashToken(accessToken),
         createdAt: previous.createdAt || now,

package/dist/services/runtime-binding.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=runtime-binding.test.d.ts.map

package/dist/services/runtime-binding.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"runtime-binding.test.d.ts","sourceRoot":"","sources":["../../src/services/runtime-binding.test.ts"],"names":[],"mappings":""}

package/dist/services/runtime-binding.test.js

@@ -0,0 +1,11 @@
+import { describe, expect, it } from 'vitest';
+import { normalizeSchedulerBaseUrl } from './runtime-binding.js';
+describe('runtime binding scheduler URL normalization', () => {
+    it('repairs duplicated scheduler ports', () => {
+        expect(normalizeSchedulerBaseUrl('http://127.0.0.1:8080:8080')).toBe('http://127.0.0.1:8080');
+    });
+    it('normalizes websocket agent URLs to scheduler HTTP origins', () => {
+        expect(normalizeSchedulerBaseUrl('ws://127.0.0.1:8080/ws/agent')).toBe('http://127.0.0.1:8080');
+        expect(normalizeSchedulerBaseUrl('wss://example.com/ws/agent')).toBe('https://example.com');
+    });
+});

package/dist/utils/yaml-utils.test.js

@@ -12,95 +12,95 @@ describe('yaml-utils', () => {
             expect(hasMatchingAgentPath('', 'test')).toBeNull();
         });
         it('should return null when no agent key exists', () => {
-            const yaml = `
-other:
-  - name: test
+            const yaml = `
+other:
+  - name: test
 `;
             expect(hasMatchingAgentPath(yaml, 'test')).toBeNull();
         });
         it('should return null when agent is not an array', () => {
-            const yaml = `
-agent:
-  name: single
+            const yaml = `
+agent:
+  name: single
 `;
             expect(hasMatchingAgentPath(yaml, 'test')).toBeNull();
         });
         it('should return null when agent has no path', () => {
-            const yaml = `
-agent:
-  - name: test
-    description: no path
+            const yaml = `
+agent:
+  - name: test
+    description: no path
 `;
             expect(hasMatchingAgentPath(yaml, 'test')).toBeNull();
         });
         it('should match exact path', () => {
-            const yaml = `
-agent:
-  - name: test
-    path: src/modules
+            const yaml = `
+agent:
+  - name: test
+    path: src/modules
 `;
             expect(hasMatchingAgentPath(yaml, 'src/modules')).toBe('src/modules');
         });
         it('should match path with different separators (backslash)', () => {
-            const yaml = `
-agent:
-  - name: test
-    path: src/modules
+            const yaml = `
+agent:
+  - name: test
+    path: src/modules
 `;
             expect(hasMatchingAgentPath(yaml, 'src\\modules')).toBe('src/modules');
         });
         it('should match path with forward slash when yaml has backslash', () => {
-            const yaml = `
-agent:
-  - name: test
-    path: src\\modules
+            const yaml = `
+agent:
+  - name: test
+    path: src\\modules
 `;
             expect(hasMatchingAgentPath(yaml, 'src/modules')).toBe('src\\modules');
         });
         it('should match current directory "." path', () => {
-            const yaml = `
-agent:
-  - name: test
-    path: .
+            const yaml = `
+agent:
+  - name: test
+    path: .
 `;
             expect(hasMatchingAgentPath(yaml, '.')).toBe('.');
         });
         it('should match empty path when target is "."', () => {
-            const yaml = `
-agent:
-  - name: test
-    path: ''
+            const yaml = `
+agent:
+  - name: test
+    path: ''
 `;
             expect(hasMatchingAgentPath(yaml, '.')).toBe('');
         });
         it('should return null when no matching path found', () => {
-            const yaml = `
-agent:
-  - name: test
-    path: other/path
+            const yaml = `
+agent:
+  - name: test
+    path: other/path
 `;
             expect(hasMatchingAgentPath(yaml, 'different/path')).toBeNull();
         });
         it('should return first matching path from multiple agents', () => {
-            const yaml = `
-agent:
-  - name: first
-    path: first/path
-  - name: second
-    path: second/path
+            const yaml = `
+agent:
+  - name: first
+    path: first/path
+  - name: second
+    path: second/path
 `;
             expect(hasMatchingAgentPath(yaml, 'first/path')).toBe('first/path');
             expect(hasMatchingAgentPath(yaml, 'second/path')).toBe('second/path');
         });
         it('should handle complex YAML with nested objects', () => {
-            const yaml = `
-agent:
-  - name: complex
-    path: complex/path
-    project: my-project
-    prompt: |
-      This is a multi-line
-      prompt
+            const yaml = `
+agent:
+  - name: complex
+    path: complex/path
+    project: my-project
+    prompt: |
+      This is a multi-line
+      prompt
 `;
             expect(hasMatchingAgentPath(yaml, 'complex/path')).toBe('complex/path');
         });
@@ -113,12 +113,12 @@ agent:
             expect(result.isTemplate).toBe(false);
         });
         it('should return matching agent config', () => {
-            const yaml = `
-agent:
-  - name: my-agent
-    path: src/modules
-    project: my-project
-    prompt: test prompt
+            const yaml = `
+agent:
+  - name: my-agent
+    path: src/modules
+    project: my-project
+    prompt: test prompt
 `;
             const result = extractAgentConfigOrTemplate(yaml, 'src/modules');
             expect(result.isTemplate).toBe(false);
@@ -127,11 +127,11 @@ agent:
             expect(result.content).toContain('my-project');
         });
         it('should generate template when no matching agent found', () => {
-            const yaml = `
-agent:
-  - name: other-agent
-    path: other/path
-    project: existing-project
+            const yaml = `
+agent:
+  - name: other-agent
+    path: other/path
+    project: existing-project
 `;
             const result = extractAgentConfigOrTemplate(yaml, 'new/path');
             expect(result.isTemplate).toBe(true);
@@ -139,10 +139,10 @@ agent:
             expect(result.content).toContain('new/path');
         });
         it('should generate template with empty project when no agents have project', () => {
-            const yaml = `
-agent:
-  - name: agent1
-    path: path1
+            const yaml = `
+agent:
+  - name: agent1
+    path: path1
 `;
             const result = extractAgentConfigOrTemplate(yaml, 'new/path');
             expect(result.isTemplate).toBe(true);
@@ -150,21 +150,21 @@ agent:
             expect(result.content).toContain('project: ""');
         });
         it('should match current directory "." path', () => {
-            const yaml = `
-agent:
-  - name: root-agent
-    path: .
-    project: root-project
+            const yaml = `
+agent:
+  - name: root-agent
+    path: .
+    project: root-project
 `;
             const result = extractAgentConfigOrTemplate(yaml, '.');
             expect(result.isTemplate).toBe(false);
             expect(result.content).toContain('root-agent');
         });
         it('should normalize path separators in output', () => {
-            const yaml = `
-agent:
-  - name: test
-    path: src\\modules
+            const yaml = `
+agent:
+  - name: test
+    path: src\\modules
 `;
             const result = extractAgentConfigOrTemplate(yaml, 'src/modules');
             expect(result.isTemplate).toBe(false);
@@ -192,78 +192,78 @@ agent:
             expect(autoFillPathInYml(invalidYaml, 'test/path')).toBe(invalidYaml);
         });
         it('should return original content when no agent key', () => {
-            const yaml = `
-other:
-  - name: test
+            const yaml = `
+other:
+  - name: test
 `;
             expect(autoFillPathInYml(yaml, 'test/path')).toBe(yaml);
         });
         it('should fill empty path', () => {
-            const yaml = `
-agent:
-  - name: test
-    path: ''
-    project: my-project
+            const yaml = `
+agent:
+  - name: test
+    path: ''
+    project: my-project
 `;
             const result = autoFillPathInYml(yaml, 'new/path');
             expect(result).toContain('new/path');
             expect(result).not.toContain("path: ''");
         });
         it('should fill undefined path', () => {
-            const yaml = `
-agent:
-  - name: test
-    project: my-project
+            const yaml = `
+agent:
+  - name: test
+    project: my-project
 `;
             const result = autoFillPathInYml(yaml, 'new/path');
             expect(result).toContain('new/path');
         });
         it('should fill null path', () => {
-            const yaml = `
-agent:
-  - name: test
-    path: null
+            const yaml = `
+agent:
+  - name: test
+    path: null
 `;
             const result = autoFillPathInYml(yaml, 'new/path');
             expect(result).toContain('new/path');
         });
         it('should fill whitespace-only path', () => {
-            const yaml = `
-agent:
-  - name: test
-    path: '   '
+            const yaml = `
+agent:
+  - name: test
+    path: '   '
 `;
             const result = autoFillPathInYml(yaml, 'new/path');
             expect(result).toContain('new/path');
         });
         it('should not modify existing non-empty path', () => {
-            const yaml = `
-agent:
-  - name: test
-    path: existing/path
+            const yaml = `
+agent:
+  - name: test
+    path: existing/path
 `;
             const result = autoFillPathInYml(yaml, 'new/path');
             expect(result).toContain('existing/path');
             expect(result).not.toContain('new/path');
         });
         it('should handle multiple agents', () => {
-            const yaml = `
-agent:
-  - name: agent1
-    path: ''
-  - name: agent2
-    path: existing/path
-  - name: agent3
+            const yaml = `
+agent:
+  - name: agent1
+    path: ''
+  - name: agent2
+    path: existing/path
+  - name: agent3
 `;
             const result = autoFillPathInYml(yaml, 'new/path');
             // agent1 should have path filled
             expect(result).toContain('new/path');
         });
         it('should return original content when no modification needed', () => {
-            const yaml = `
-agent:
-  - name: test
-    path: existing/path
+            const yaml = `
+agent:
+  - name: test
+    path: existing/path
 `;
             const result = autoFillPathInYml(yaml, 'new/path');
             expect(result).toBe(yaml);
@@ -296,11 +296,11 @@ agent:
         it('should load config from workspace directory with matching path "."', async () => {
             const configPath = path.join(workspacePath, relativePath);
             await fs.mkdir(path.dirname(configPath), { recursive: true });
-            await fs.writeFile(configPath, `
-agent:
-  - name: local-agent
-    path: .
-    project: local-project
+            await fs.writeFile(configPath, `
+agent:
+  - name: local-agent
+    path: .
+    project: local-project
 `, 'utf-8');
             const result = await loadYmlWithUpwardSearch(workspacePath, relativePath);
             expect(result.content).not.toBeNull();
@@ -312,11 +312,11 @@ agent:
             // 在 project/module 目录创建配置，匹配 src 路径
             const parentConfigPath = path.join(tempRoot, 'project', 'module', relativePath);
             await fs.mkdir(path.dirname(parentConfigPath), { recursive: true });
-            await fs.writeFile(parentConfigPath, `
-agent:
-  - name: module-agent
-    path: src
-    project: module-project
+            await fs.writeFile(parentConfigPath, `
+agent:
+  - name: module-agent
+    path: src
+    project: module-project
 `, 'utf-8');
             const result = await loadYmlWithUpwardSearch(workspacePath, relativePath);
             expect(result.content).not.toBeNull();
@@ -328,10 +328,10 @@ agent:
             // 在 project/module 目录创建配置，但不匹配 src 路径
             const parentConfigPath = path.join(tempRoot, 'project', 'module', relativePath);
             await fs.mkdir(path.dirname(parentConfigPath), { recursive: true });
-            await fs.writeFile(parentConfigPath, `
-agent:
-  - name: other-agent
-    path: other/path
+            await fs.writeFile(parentConfigPath, `
+agent:
+  - name: other-agent
+    path: other/path
 `, 'utf-8');
             const result = await loadYmlWithUpwardSearch(workspacePath, relativePath);
             expect(result.content).not.toBeNull();
@@ -342,17 +342,17 @@ agent:
             // 在 workspace 目录创建匹配的配置
             const localConfigPath = path.join(workspacePath, relativePath);
             await fs.mkdir(path.dirname(localConfigPath), { recursive: true });
-            await fs.writeFile(localConfigPath, `
-agent:
-  - name: local-agent
-    path: .
+            await fs.writeFile(localConfigPath, `
+agent:
+  - name: local-agent
+    path: .
 `, 'utf-8');
             // 在 parent 目录也创建配置
             const parentConfigPath = path.join(tempRoot, 'project', 'module', relativePath);
-            await fs.writeFile(parentConfigPath, `
-agent:
-  - name: parent-agent
-    path: src
+            await fs.writeFile(parentConfigPath, `
+agent:
+  - name: parent-agent
+    path: src
 `, 'utf-8');
             const result = await loadYmlWithUpwardSearch(workspacePath, relativePath);
             // 应该使用 local 的配置，因为更近且匹配