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

Offen
  • Stimmt etwas nicht?
#164 1 Kommentar 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 1 Tag

Dieses Issue hat noch niemand übernommen.

Bewertung

Schwierigkeit
3/5
Geschätzter Aufwand
1-2 Tage
Anfängerfreundlichkeit
55/100
Issue-Typ
Bug
Klarheit
Klar beschrieben
Aktivitätsstatus
Aktiv
Tech-Stack
typescript, vite
Bereich
build-system, ci-cd

Rechercherichtung

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.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

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.

Vorherrschende Sprache
TypeScript
Sterne
565
Forks
27
Ø Merge
10 Std. 27 Min.
Gemergte PRs (30 T.)
40

Entwicklungsumgebung

Dieses Projekt bietet weder Dev-Container noch Dockerfile noch Beitragsleitfaden – die Einrichtung liegt bei Ihnen. Beginnen Sie mit der README; die allgemeinen Schritte stehen in unserem Leitfaden für den ersten Beitrag.

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus TanStack/redact

Alle Issues in TanStack/redact

Ähnliche Issues

Weitere Issues zu TypeScript

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.