Scope pull request uploads to the pull request
Los mantenedores suelen responder en 2 días
Nadie ha tomado este issue todavía.
Evaluación
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Aptitud para principiantes
- 35/100
- Tipo de issue
- Nueva funcionalidad
- Claridad
- Bastante claro
- Estado de actividad
- Activo
- Stack tecnológico
- typescript
Línea de trabajo
Start by reading src/auth.ts, which the issue identifies as enforcing the current upload policy, then trace how cache requests are routed and how vp run addresses cache scopes. The proposal leaves endpoint design, fallback behavior, and retention unresolved, so define those decisions before implementation. Done means same-repository PR runs can write and read their own scope with default-branch fallback, while default-branch builds cannot read PR scopes.
Escrito por el modelo de indexación a partir del texto del issue.
Descripción
Motivation
A pull request can't reuse cache entries from its own earlier runs. Only push jobs on the namespace's default branch can upload (src/auth.ts), so pull_request runs get 403 write_policy and can only restore what the default branch published. After each new push to a PR, tasks whose inputs differ from the default branch rebuild, even if the PR's previous run already built them.
Uploading PR results into the namespace itself would let unreviewed PR code publish entries that default-branch builds restore, so PR entries need their own scope.
Proposal
- A
pull_requestrun uploads to a scope for its PR, derived from the token'srefclaim (refs/pull/<n>/merge). - PR runs read their PR scope first, falling back to the default branch. Default-branch builds never read PR scopes.
- Default-branch publishing is unchanged.
This covers same-repository PRs only; fork PR runs in public repositories don't get id-token: write.
Open questions
- Endpoint: how a client addresses a PR scope (e.g.
/projects/<namespace>/pulls/<n>), with the write policy checking that the token'srefmatches it. - Fallback: the server could serve default-branch entries on a PR-scope miss, or
vp run(vite-task) could read both endpoints. - Retention: PR scopes should expire soon after the PR closes or goes idle, within the namespace's limits.
- Lenguaje dominante
- TypeScript
- Estrellas
- 8
- Forks
- 0
- Merge medio
- 2 d 15 h
- PR fusionados (30 d)
- 33
Preparar el entorno
Este proyecto no incluye contenedor de desarrollo, Dockerfile ni guía de contribución, así que la configuración corre por tu cuenta: empieza por su README y consulta nuestra guía para la primera contribución para los pasos generales.
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Más de voidzero-dev/vite-plus-remote-cache-cloudflare
-
Dependency DashboardAbierto
Dificultad 4/5 3-5 días Aptitud para principiantes 35/100
voidzero-dev/vite-plus-remote-cache-cloudflare#6 ·
Los mantenedores suelen responder en 2 días
Todos los issues de voidzero-dev/vite-plus-remote-cache-cloudflare
Issues similares
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
oxidecomputer/skepsis#102 · 1 comentario ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 62/100
voidzero-dev/void#677 ·
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 68/100
Los mantenedores suelen responder en 1 día
-
Dificultad 2/5 1-3 horas Aptitud para principiantes 78/100
kunchenguid/gh-axi#247 ·
Los mantenedores suelen responder en 1 día
-
bug
Dificultad 2/5 1-3 horas Aptitud para principiantes 72/100
lidge-jun/opencodex#8101 · 2 comentarios ·
Los mantenedores suelen responder en 1 día