starred/servers-modelcontextprotocol
1
0
Fork 0
mirror of https://github.com/modelcontextprotocol/servers.git synced 2026-04-17 23:53:24 +02:00
Code Issues Packages Projects Releases Wiki Activity

Remove only trailing slashes (#3183)

Browse Source
This commit is contained in:
tomonacci
2026-01-05 13:42:10 -05:00
committed by GitHub
parent 862e717ff7
commit 1ae217d74d
2 changed files with 7 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

6
src/filesystem/__tests__/path-utils.test.ts
View File

@@ -70,6 +70,12 @@ describe('Path Utilities', () => {
.toBe('/home/user/some path'); .toBe('/home/user/some path');
expect(normalizePath('"/usr/local/some app/"')) expect(normalizePath('"/usr/local/some app/"'))
.toBe('/usr/local/some app'); .toBe('/usr/local/some app');
expect(normalizePath('/usr/local//bin/app///'))
.toBe('/usr/local/bin/app');
expect(normalizePath('/'))
.toBe('/');
expect(normalizePath('///'))
.toBe('/');
}); });
it('removes surrounding quotes', () => { it('removes surrounding quotes', () => {

2
src/filesystem/path-utils.ts
View File

@@ -55,7 +55,7 @@ export function normalizePath(p: string): string {
if (isUnixPath) { if (isUnixPath) {
// For Unix paths, just normalize without converting to Windows format // For Unix paths, just normalize without converting to Windows format
// Replace double slashes with single slashes and remove trailing slashes // Replace double slashes with single slashes and remove trailing slashes
return p.replace(/\/+/g, '/').replace(/\/+$/, ''); return p.replace(/\/+/g, '/').replace(/(?<!^)\/$/, '');
} }
// Convert Unix-style Windows paths (/c/, /d/) to Windows format if on Windows // Convert Unix-style Windows paths (/c/, /d/) to Windows format if on Windows