View your CI Pipeline Execution ↗ for commit 9417b61

☁️ Nx Cloud last updated this comment at 2026-01-15 20:28:33 UTC

Thanks for this @Tofandel, were you able to confirm macOS still ok with dirname in that?

Some build errors here:

  Compiling TypeScript files for project "vite"...
  packages/vite/helpers/project.ts:11:33 - error TS2448: Block-scoped variable '__dirname' used before its declaration.
  
  11 export const __dirname = typeof __dirname !== 'undefined' ? __dirname : import.meta.dirname;

You can npx nx build vite --skip-nx-cache to confirm locally.

Ah yeah typescript doesn't like that both the global and local __dirname are used, since vite doesn't have a CJS build, we can remove it anyways

I don't have mac OS, but since it just uses an ESM variable instead, I see no reason why it wouldn't work on any platform. This is node 20+ only though; if we need to support less than node 20, then we should use

import { dirname } from 'node:path';
import { fileURLToPath } from 'node:url';
    
export const __dirname = dirname(fileURLToPath(import.meta.url));

Since vite 7 is already node 20.19+, I guess no need for that either

https://github.com/vitejs/vite/blob/main/packages/vite/package.json#L49

Nice thanks yes, v9 cli is node 22+ required so 👍