SEA + cluster.fork on Windows prepends executable path into worker process.argv (breaks argv parsing)
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- JavaScript
- Estrellas
- 122k
- Forks
- 37.3k
- Merge medio
- 4 d 2 h
- PR fusionados (30 d)
- 283
Descripción
Version
v24.15.0
Platform
Microsoft Windows NT 10.0.26200.0 x64
Subsystem
single executable applications (SEA), cluster.fork
What steps will reproduce the bug?
Minimal reproduction idea
- Build a SEA executable whose entry:
- Prints
process.pid, role,process.execPath, andprocess.argv.slice(2) - If primary:
cluster.fork()one worker - If worker: print argv then exit
- Prints
- Run executable with user args, for example:
app.exe command --flag value - Compare primary vs worker argv
slice(2).
Primary shows:
["foo","bar"]
Worker shows:
["X:\\...\\app.exe","foo","bar"]
Node.js versions:
- v24.15.0 (reproducible)
- v22.16.0 (reproducible)
- Historically observed since Node 20 era in our project
How often does it reproduce? Is there a required condition?
100% reproducible under these conditions: (1) the application is built as a SEA executable, (2) the primary process uses cluster.fork() to spawn workers, (3) tested on Windows (Linux not verified). Running the same logic with plain node script.mjs does not reproduce the issue — the extra token only appears in SEA-built worker processes.
What is the expected behavior? Why is that the expected behavior?
Observed pattern:
- Primary process
process.argv.slice(2):
["command","--flag","value"] - Worker process
process.argv.slice(2):
["X:\\path\\to\\app.exe","command","--flag","value"]
Expected behavior
Worker processes in SEA mode should see the same user command arguments as primary for process.argv.slice(2), without an injected executable path token.
What do you see instead?
Actual behavior
Worker process argv includes an extra executable path token before actual user arguments.
Additional information
Control test
Running equivalent script without SEA (node script.mjs foo bar) does not show this divergence.
Impact
- Breaks multi-process startup paths in production for SEA apps using argv-based command routing.
- Particularly problematic when a process model uses cluster workers that execute the same CLI entrypoint.
Current workaround
Application-side argv normalization — filter any token in process.argv.slice(2) that equals process.execPath before handing off to the command parser:
import path from 'node:path';
function normalizePathname(p) {
return path.resolve(p).toLowerCase();
}
function isSelfExecutableArg(arg) {
if (typeof arg !== 'string' || arg.trim() === '' || arg.startsWith('-')) {
return false;
}
return normalizePathname(arg) === normalizePathname(process.execPath);
}
function normalizeArgv(rawArgv) {
return rawArgv.filter(arg => !isSelfExecutableArg(arg));
}
// Usage
const argv = normalizeArgv(process.argv.slice(2));
Key points:
- Case-insensitive path comparison is required on Windows (
path.resolve+.toLowerCase()). - The filter must skip tokens starting with
-to avoid accidentally dropping flags that happen to match. - The injected token can appear anywhere in
slice(2), not just at index 0, so a.filterover the full array is safer than removing index 0 unconditionally.
This avoids the crash but seems like a workaround for runtime behavior that may be unintended.
Additional notes
If needed, I can provide a complete minimal repro repository.
Signed-off-by
GitHub Copilot
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Empieza construyendo el ejecutable SEA descrito en la issue y compara process.argv.slice(2) en el primary y en el worker de cluster.fork() en Windows. Sigue la ruta de inicio del worker de SEA que construye process.argv; el trabajo estará terminado cuando el worker conserve los argumentos del usuario del primary sin una ruta de ejecutable inyectada y el caso de control que no es SEA siga pasando.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- javascript, nodejs
- Área
- backend, operating-systems
- Tipo de issue
- Error
- Dificultad
- 4/5
- Tiempo estimado
- 3-5 días
- Estado de actividad
- Tranquilo
- Claridad
- Bastante claro
- Aptitud para principiantes
- 48/100