Skip to content

fix: normalize all repeated path slashes - #133

Open
vagdotdev wants to merge 1 commit into
tscircuit:mainfrom
vagdotdev:bounty/normalize-all-slashes-5
Open

fix: normalize all repeated path slashes#133
vagdotdev wants to merge 1 commit into
tscircuit:mainfrom
vagdotdev:bounty/normalize-all-slashes-5

Conversation

@vagdotdev

@vagdotdev vagdotdev commented Jun 29, 2026

Copy link
Copy Markdown

Summary

  • collapse every repeated slash run during path normalization, not just the first one
  • keep download query lookups consistent for paths containing multiple duplicate slash groups
  • add regression coverage for /files/download?file_path=...
  • make test servers bind to OS-assigned ports so parallel CI runs do not collide

Validation

  • bun test tests/routes/files.test.ts
  • bun test && bun test && bun test
  • bunx tsc --noEmit
  • bun run format:check
  • bun run build
  • git diff --check

/claim #5

@vagdotdev
vagdotdev force-pushed the bounty/normalize-all-slashes-5 branch from cb1fb9f to b282218 Compare June 29, 2026 20:34
@vagdotdev

Copy link
Copy Markdown
Author

/claim #5

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant