fix(app): shorten export filenames for long session titles (#60347)

Co-authored-by: [owner] <[owner]@gmail.com>
Co-authored-by: Daniel Smolsky <dannysmo@gmail.com>
This commit is contained in:
Craig Albrecht
2026-11-04 16:02:14 -06:00
committed by GitHub
Unverified
parent 4cb790123c
commit ded4c290d8
2 changed files with 5 additions and 0 deletions
@@ -14,6 +14,10 @@ describe("sessionExportFilename", () => {
])("names %o as %s", (session, filename) => {
expect(sessionExportFilename(session)).toBe(filename)
})
test("shortens long titles to fit the filename limit", () => {
expect(sessionExportFilename({ id: "ses_123", title: "a".repeat(300) })).toBe(`${"a".repeat(250)}.json`)
})
})
describe("fetchSessionExport", () => {
@@ -43,6 +43,7 @@ export function sessionExportFilename(session: { id: string; title?: string; slu
const clean = name
.toLowerCase()
.replace(/[^a-z0-9_-]+/gi, "-")
.slice(0, 250)
.replace(/^-+|-+$/g, "")
return `${clean || session.id}.json`