Vinext 1.4.3/1.4.4 support: adapter gate rejects versions whose required corrections are already upstream

Aperta
#164 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
3/5
Tempo stimato
1-2 giorni
Idoneità per principianti
55/100
Tipo di issue
Bug
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
typescript, vite

Direzione di ricerca

Start by reading the /vinext adapter’s version gate and the two capability checks that guard its source corrections, then inspect the CI configuration for the existing Vinext matrix. Update support for Vinext 1.3.x and 1.4.x while keeping unknown source shapes fail closed, and add 1.4.3 and 1.4.4 to CI. Done means the demo build and CI cover the requested versions.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

The /vinext adapter currently rejects every Vinext version except exactly 1.3.0:

if (pkg.version !== '1.3.0') throw new Error(...)

Reproduction

In this repository:

pnpm --filter vinext-demo add [email protected]
pnpm --filter vinext-demo build
# Error: Redact Vinext integration supports vinext 1.3.0; found 1.4.3.

Compatibility evidence

The adapter's two source corrections are guarded by capability checks. I checked Vinext 1.4.3 and 1.4.4: both contain the corrected implementations, so Redact's transforms should be no-ops for those versions.

In a scratch install, I changed only the installed Redact /vinext adapter's version check to let the newer Vinext versions pass. With that local gate change, vinext-demo builds on 1.3.0, 1.4.3, and 1.4.4. These are build checks; the requested CI matrix will provide repeatable coverage.

Requested change

Accept Vinext 1.3.x and 1.4.x, while warning on other version families. Keep the source-shape checks as the safety boundary: skip corrections when the installed Vinext code already has the corrected form, and fail closed when its shape is unknown. Add the Vinext versions above to the CI matrix.

The client instrumentation API request is tracked separately in cloudflare/vinext#4579; it does not cover the app-rsc-handler correction or Flight timing. @cloudflare/vite-plugin 2.x remains out of scope.

I can open a PR with the gate change and CI coverage.

Lingua principale
TypeScript
Stelle
537
Fork
22
Merge medio
10h 27m
PR unite (30g)
40

Preparare l'ambiente

Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di TanStack/redact

Tutte le issue di TanStack/redact

Issue simili

Altre issue su TypeScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.