cloudflare/vinext

Build: transitive externals resolve to wrong package version

Open

#1 503 ouverte le 22 mai 2026

Voir sur GitHub
 (0 commentaires) (0 réactions) (0 assignés)TypeScript (329 forks)github user discovery
adapter-api-e2ehelp wanted

Métriques du dépôt

Stars
 (8 120 stars)
Métriques de merge PR
 (Merge moyen 1j 1h) (462 PRs mergées en 30 j)

Description

This issue was created by an agent analysing CI failures from the Next.js Deploy Suite (vinext main vs Next.js v16.2.6, 2026-05-22).

Problem

When serverExternalPackages (or equivalent) is configured, transitive dependencies resolve to the wrong version. Test expects depB: 4.17.21, 4.17.21, 5.0.0 but got depB: 3.10.1, 4.17.21, 5.0.0 — the first depB instance is wrongly resolved to depA's version.

Transitive externals resolution picks wrong package version

Estimated Impact

~1 test failures across the deploy suite.

Affected Test Suites

  • test/e2e/externals-transitive/externals-transitive.test.ts

Recommendation

  1. Reproduce first in vinext's own test suite. Add a fixture with depA -> depB@4 and root -> depB@3, mark depA as external, and assert each module sees the correct depB version.

  2. Preserve transitive version resolution. Externalising a package must not collapse nested copies of a transitive dep to a single version.

Guide contributeur

Direction de recherche
Examinez le fixture de test dans `test/e2e/externals transitive/` pour comprendre le graphe de dépendances (depA > depB@4, root > depB@3). Tracez la logique de résolution des modules dans le plugin vite pour trouver où les externes transitifs réduisent les copies imbriquées.
Stack technique
typescriptnodejs
Domaine
backendbuild system
Type d'issue
Bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
Une demi journée
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
Active
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
Claire
Prérequis
Node.jsTypeScript
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
50