Skip to content

Commit 69353c1

Browse files
claude[bot]Dhravya
andcommitted
fix: improve Zed support with cross-platform paths and documentation
- Add Zed to supported clients list in README.md - Fix Zed config path to use cross-platform baseDir instead of hardcoded Unix path - Update tests to include Zed and Goose in expected clients list - Increase minimum client count test from 11 to 13 This addresses the code review feedback to make Zed support production-ready with proper Windows/macOS/Linux compatibility and complete documentation. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-authored-by: Dhravya Shah <Dhravya@users.noreply.github.com>
1 parent 6116278 commit 69353c1

File tree

4 files changed

+7
-4
lines changed

4 files changed

+7
-4
lines changed

README.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -108,6 +108,7 @@ The `--client` flag specifies which MCP client you're installing for:
108108
- `gemini-cli`
109109
- `claude-code`
110110
- `goose`
111+
- `zed`
111112
- `warp` (outputs config to copy/paste into Warp's cloud-based settings)
112113

113114
## License

src/client-config.test.ts

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -55,12 +55,14 @@ describe('client-config', () => {
5555
'gemini-cli',
5656
'vscode',
5757
'claude-code',
58+
'goose',
59+
'zed',
5860
]),
5961
)
6062
})
6163

62-
it('should have at least 11 clients', () => {
63-
expect(clientNames.length).toBeGreaterThanOrEqual(11)
64+
it('should have at least 13 clients', () => {
65+
expect(clientNames.length).toBeGreaterThanOrEqual(13)
6466
})
6567
})
6668

src/client-config.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -127,7 +127,7 @@ function getClientPaths(): { [key: string]: ClientInstallTarget } {
127127
},
128128
zed: {
129129
type: 'file',
130-
path: path.join(homeDir, '.config', 'zed', 'settings.json'),
130+
path: path.join(baseDir, 'zed', 'settings.json'),
131131
configKey: 'context_servers',
132132
},
133133
}

src/commands/install.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@ describe('install command', () => {
2020
beforeEach(() => {
2121
jest.clearAllMocks()
2222
Object.defineProperty(mockClientConfig, 'clientNames', {
23-
value: ['claude', 'cline', 'cursor', 'vscode', 'warp'],
23+
value: ['claude', 'cline', 'cursor', 'vscode', 'warp', 'goose', 'zed'],
2424
writable: true,
2525
})
2626
mockClientConfig.readConfig.mockReturnValue({ mcpServers: {} })

0 commit comments

Comments
 (0)