cloudflare/vinext

Normalize internal paths to forward slashes for Windows support

Open

#1,605 创建于 2026年5月26日

在 GitHub 查看
 (1 评论) (0 反应) (0 负责人)TypeScript (329 fork)github user discovery
good first issuehelp wanted

仓库指标

Star
 (8,120 star)
PR 合并指标
 (平均合并 1天 1小时) (30 天内合并 462 个 PR)

描述

Background

Vite normalizes all internal paths to forward slashes. On Windows, Node.js APIs (path.join, path.relative, fs.glob, etc.) return backslash paths. When vinext stores or compares these paths without normalization, things break — cache misses, failed route lookups, wrong startsWith/endsWith results, etc.

#1578 fixed the first case (StaticFileCache). #1604 extracted the fix into a reusable toSlash() utility in utils/path.ts. The same problem exists in other parts of the codebase.

Known affected areas

  • RoutingpagePath, layoutPath, errorPath from app-route-graph.ts are OS-native paths. Downstream forward-slash comparisons silently mismatch on Windows.
  • File scannerscanWithExtensions yields fs.glob results which use backslashes on Windows.
  • Entry generatorsentries/ files apply normalizePathSeparators() per-callsite; should migrate to the shared utils/path.ts.
  • Build & deploypath.relative() in prerender output, asset prefix, deploy packaging.
  • Tests — Hardcoded forward-slash assertions and NTFS-illegal fixture filenames (*, :) cause failures on Windows.

Proposal

  1. Normalize at filesystem boundaries — apply toSlash() wherever path.join, path.relative, or fs.glob produces a path for internal use (#1604 provides the utility). For constructing internal-only paths that never touch the OS filesystem, prefer path.posix.join to produce forward slashes directly without a normalize step.

  2. Add Windows CI on main only — catch regressions post-merge without blocking the current fast-paced PR workflow.

References

  • #1578 — StaticFileCache inline fix
  • #1604 — utils/path.ts (slash / toSlash)

贡献者指南