ai-workspace-services/qmd
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: allow $ route filenames in handelize. Closes #162

Browse Source
This commit is contained in:
Claude 2026-02-13 19:19:19 -05:00 committed by Tobi Lütke
parent da79e77d34
commit 0eabfe73db
No known key found for this signature in database
2 changed files with 9 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
src/store.test.ts
View File

@ -378,6 +378,11 @@ describe("handelize", () => {
expect(handelize("(DRAFT) Proposal v1.md")).toBe("draft-proposal-v1.md"); expect(handelize("(DRAFT) Proposal v1.md")).toBe("draft-proposal-v1.md");
}); });
test("handles symbol-only route filenames", () => {
expect(handelize("routes/api/auth/$.ts")).toBe("routes/api/auth/$.ts");
expect(handelize("app/routes/$id.tsx")).toBe("app/routes/$id.tsx");
});
test("filters out empty segments", () => { test("filters out empty segments", () => {
expect(handelize("a//b/c.md")).toBe("a/b/c.md"); expect(handelize("a//b/c.md")).toBe("a/b/c.md");
expect(handelize("/a/b/")).toBe("a/b"); expect(handelize("/a/b/")).toBe("a/b");

9
src/store.ts
View File

@ -798,12 +798,11 @@ export function handelize(path: string): string {
throw new Error('handelize: path cannot be empty'); throw new Error('handelize: path cannot be empty');
} }
// Check for paths that are just extensions or only dots/special chars // Allow route-style "$" filenames while still rejecting paths with no usable content.
// A valid path must have at least one letter or digit (including Unicode)
const segments = path.split('/').filter(Boolean); const segments = path.split('/').filter(Boolean);
const lastSegment = segments[segments.length - 1] || ''; const lastSegment = segments[segments.length - 1] || '';
const filenameWithoutExt = lastSegment.replace(/\.[^.]+$/, ''); const filenameWithoutExt = lastSegment.replace(/\.[^.]+$/, '');
const hasValidContent = /[\p{L}\p{N}]/u.test(filenameWithoutExt); const hasValidContent = /[\p{L}\p{N}$]/u.test(filenameWithoutExt);
if (!hasValidContent) { if (!hasValidContent) {
throw new Error(`handelize: path "${path}" has no valid filename content`); throw new Error(`handelize: path "${path}" has no valid filename content`);
} }
@ -822,14 +821,14 @@ export function handelize(path: string): string {
const nameWithoutExt = ext ? segment.slice(0, -ext.length) : segment; const nameWithoutExt = ext ? segment.slice(0, -ext.length) : segment;
const cleanedName = nameWithoutExt const cleanedName = nameWithoutExt
.replace(/[^\p{L}\p{N}]+/gu, '-') // Replace non-letter/digit chars with dash .replace(/[^\p{L}\p{N}$]+/gu, '-') // Keep route marker "$", dash-separate other chars
.replace(/^-+|-+$/g, ''); // Remove leading/trailing dashes .replace(/^-+|-+$/g, ''); // Remove leading/trailing dashes
return cleanedName + ext; return cleanedName + ext;
} else { } else {
// For directories, just clean normally // For directories, just clean normally
return segment return segment
.replace(/[^\p{L}\p{N}]+/gu, '-') .replace(/[^\p{L}\p{N}$]+/gu, '-')
.replace(/^-+|-+$/g, ''); .replace(/^-+|-+$/g, '');
} }
}) })