fix(@angular/cli): normalize Windows drive-letter casing for process.cwd · angular/angular-cli@1d9122f · GitHub

angular / angular-cli Public

• Notifications You must be signed in to change notification settings
• Fork 11.8k
• Star 27k

• Code
• Issues 241
• Pull requests 19
• Actions
• Security and quality 5
• Insights

Additional navigation options

Commit 1d9122f

Browse filesBrowse the repository at this point in the historyBrowse files

clydin

authored and

alan-agius4

committed

fix(@angular/cli): normalize Windows drive-letter casing for process.cwd

Windows cmd.exe and VS Code terminals often preserve lowercase drive letters (e.g. c:\...), which causes ESM module duplication in Node.js (e.g. loading 'vitest' twice) and path lookup mismatches in tools like Vite and esbuild. Normalizing process.cwd() early in CLI startup ensures consistent uppercase drive letter casing across all builders, worker threads, and ESM module resolution on Windows. (cherry picked from commit 25034f9)

1 parent 4e54f3a commit 1d9122f
Copy full SHA for 1d9122f

1 file changed

+17Lines changed: 17 additions & 0 deletions

File tree

Expand file treeCollapse file tree

Open diff view settings

Filter options

• packages/angular/cli/lib
  • init.ts

Expand file treeCollapse file tree

Open diff view settings

Collapse file

‎packages/angular/cli/lib/init.ts‎

Copy file name to clipboardExpand all lines: packages/angular/cli/lib/init.ts

+17Lines changed: 17 additions & 0 deletions

Original file line number	Diff line number	Diff line change
@@ -24,6 +24,23 @@ import { VERSION } from '../src/utilities/version';
24	24	let forceExit = false;
25	25
26	26	(async (): Promise<typeof import('./cli').default | null> => {
	27	+ /**
	28	+ * On Windows, ensure process.cwd() uses uppercase drive letter casing.
	29	+ * Windows cmd.exe and VS Code terminals often preserve lowercase drive letters (e.g. c:\...),
	30	+ * which causes ESM module duplication in Node.js (e.g. loading "vitest" twice) and path
	31	+ * lookup mismatches in tools like Vite and esbuild.
	32	+ */
	33	+ if (process.platform === 'win32') {
	34	+ const cwd = process.cwd();
	35	+ if (/^[a-z]:/.test(cwd)) {
	36	+ try {
	37	+ process.chdir(cwd[0].toUpperCase() + cwd.slice(1));
	38	+ } catch {
	39	+ // Ignore failure to change directory
	40	+ }
	41	+ }
	42	+ }
	43	+
27	44	/**
28	45	* Disable Browserslist old data warning as otherwise with every release we'd need to update this dependency
29	46	* which is cumbersome considering we pin versions and the warning is not user actionable.

0 commit comments

Comments

0 (0)