From ba18ccbcab06ede2a30563e2a90dbb40fd192151 Mon Sep 17 00:00:00 2001 From: Joe Danziger Date: Tue, 27 May 2025 13:25:56 -0400 Subject: [PATCH] use standard tool mappings for windsurf --- scripts/profiles/windsurf.js | 2 +- tests/integration/windsurf-init-functionality.test.js | 8 ++++---- tests/unit/rule-transformer-windsurf.test.js | 10 +++++----- 3 files changed, 10 insertions(+), 10 deletions(-) diff --git a/scripts/profiles/windsurf.js b/scripts/profiles/windsurf.js index e3dfc588..db4b2e86 100644 --- a/scripts/profiles/windsurf.js +++ b/scripts/profiles/windsurf.js @@ -13,7 +13,7 @@ const windsurfProfile = createProfile({ mcpConfigName: 'mcp.json', fileExtension: '.mdc', targetExtension: '.md', - toolMappings: COMMON_TOOL_MAPPINGS.ROO_STYLE // Windsurf uses transformed tool names + toolMappings: COMMON_TOOL_MAPPINGS.STANDARD // Windsurf uses standard tool names }); // Export all the standard profile properties diff --git a/tests/integration/windsurf-init-functionality.test.js b/tests/integration/windsurf-init-functionality.test.js index f49b20fa..7f325a60 100644 --- a/tests/integration/windsurf-init-functionality.test.js +++ b/tests/integration/windsurf-init-functionality.test.js @@ -26,10 +26,10 @@ describe('Windsurf Profile Initialization Functionality', () => { expect(windsurfProfileContent).toContain("targetExtension: '.md'"); }); - test('windsurf.js uses transformed tool mappings', () => { - expect(windsurfProfileContent).toContain('COMMON_TOOL_MAPPINGS.ROO_STYLE'); - // Should contain comment about transformed tool names - expect(windsurfProfileContent).toContain('transformed tool names'); + test('windsurf.js uses standard tool mappings', () => { + expect(windsurfProfileContent).toContain('COMMON_TOOL_MAPPINGS.STANDARD'); + // Should contain comment about standard tool names + expect(windsurfProfileContent).toContain('standard tool names'); }); test('windsurf.js contains correct URL configuration', () => { diff --git a/tests/unit/rule-transformer-windsurf.test.js b/tests/unit/rule-transformer-windsurf.test.js index bd8d0d8d..e263d76d 100644 --- a/tests/unit/rule-transformer-windsurf.test.js +++ b/tests/unit/rule-transformer-windsurf.test.js @@ -77,11 +77,11 @@ alwaysApply: true // Read the converted file const convertedContent = fs.readFileSync(testWindsurfRule, 'utf8'); - // Verify transformations - expect(convertedContent).toContain('search_files tool'); - expect(convertedContent).toContain('apply_diff tool'); - expect(convertedContent).toContain('execute_command'); - expect(convertedContent).toContain('use_mcp_tool'); + // Verify transformations (Windsurf uses standard tool names, so no transformation) + expect(convertedContent).toContain('search tool'); + expect(convertedContent).toContain('edit_file tool'); + expect(convertedContent).toContain('run_command'); + expect(convertedContent).toContain('use_mcp'); }); it('should correctly update file references', () => {