Repository navigation
[node] ForkOptions omits windowsHide despite Node core passing it to fork() #75523
Unanswered
christopher-buss
asked this question in
Issues with a @types package
Replies: 2 comments
|
Thanks for the discussion about " Pinging the DT module owners: @microsoft, @jkomyno, @r3nya, @btoueg, @Touffy, @mohsen1, @galkin, @eps1lon, @WilcoBakker, @chyzwar, @trivikr, @yoursunny, @qwelias, @ExE-Boss, @peterblazejewicz, @addaleax, @victorperin, @nodejs, @LinusU, @wafuwafu13, @mcollina, @Semigradsky, @Renegade334, @anonrig. |
0 replies
Upstream documentation fixes are always welcome. The simplest solution here would be @@ -1136,7 +1136,7 @@ declare module "node:child_process" {
stderr: string | NonSharedBuffer;
}>;
}
- interface ForkOptions extends ProcessEnvOptions, MessagingOptions, Abortable {
+ interface ForkOptions extends CommonOptions, MessagingOptions, Abortable {
execPath?: string | undefined;
execArgv?: string[] | undefined;
silent?: boolean | undefined; |
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
child_process.ForkOptionsomitswindowsHide, so TypeScript rejects a fork option that Node forwards tospawn()and explicitly uses in its own cluster implementation.Reproduction
Verified with TypeScript 6.0.3 and
@types/node24.12.3. The currenttypes/node/child_process.d.tson master also omits the property.Running a strict, no-emit typecheck produces:
Runtime/API evidence
spawn(options.execPath, args, options). I also inspected this implementation in the local Node v26.5.0 runtime.fork()withwindowsHide: cluster.settings.windowsHide.windowsVerbatimArgumentsbut notwindowsHide.Expected declaration
Should
ForkOptionsincludewindowsHide?: boolean | undefined, with the same console-window meaning and default of false asSpawnOptions.windowsHide?This matters for tools that fork Node workers from Windows GUI hosts: callers need to suppress extra console windows, but currently must extend the options type locally, for example
ForkOptions & Pick<SpawnOptions, 'windowsHide'>.Node's fork API documentation also omits the property. I have not found a documented reason for that omission; if a Node documentation fix is required before updating the declaration, please confirm. This report concerns the missing property, not changing its runtime default.
I checked existing issues, PRs, and discussions for
windowsHide/ForkOptionsand did not find an existing type-definition report. Filed in this category as directed by the repository's issue chooser.Generated with Codex.
All reactions