Skip to content

Comments

refactor: unify tarball resolution and optimize download latency#468

Open
aprendendofelipe wants to merge 2 commits intostackblitz-labs:mainfrom
aprendendofelipe:refactor/tarball-resolver
Open

refactor: unify tarball resolution and optimize download latency#468
aprendendofelipe wants to merge 2 commits intostackblitz-labs:mainfrom
aprendendofelipe:refactor/tarball-resolver

Conversation

@aprendendofelipe
Copy link
Contributor

  • Replace multiple specific endpoints with a global tarball-resolver middleware.
  • Remove 302 redirects.
  • Eliminate routing hacks required to prevent Nitro from hijacking Vue's / and /~/... routes.
  • Improve E2E testability by introducing x-commit-key and x-pkg-name-key headers.
  • Make analytics database writes non-blocking (event.waitUntil).

@aprendendofelipe
Copy link
Contributor Author

Marking as draft. I need to fix a bug when ref is a branch name containing @. I'll check this out later.

@aprendendofelipe
Copy link
Contributor Author

I updated the workflow_run.in_progress.json fixture to use a chaotic branch name (@test/@chaotic.branch/name-@v1.@) to stress-test the resolver. Interestingly, I noticed the current version of the service also fails to parse this payload correctly. This PR fixes that pre-existing bug!

@aprendendofelipe aprendendofelipe marked this pull request as ready for review February 23, 2026 22:37
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