feature(util/exec): enable process group handling on executionTimeout

[Gabriel-Ladzaretti]

What would you like Renovate to be able to do?

Blocked by

• feat(util/exec): use spawn instead of exec #16414

Once above is merged we should enable the PID range hack.
Context -

• feat(util/exec): use spawn instead of exec #16414 (comment)

If you have any ideas on how this should be implemented, please tell us here.

Once above is merged
In lib/util/exec/common.ts

uncomment -

/**
     * If `pid` is negative, but not `-1`, signal shall be sent to all processes
     * (excluding an unspecified set of system processes),
     * whose process group ID (pgid) is equal to the absolute value of pid,
     * and for which the process has permission to send a signal.
     */
    // process.kill(-(cp.pid as number), signal);

and remove -

// destroying stdio is needed for unref to work
    // https://nodejs.org/api/child_process.html#subprocessunref
    // https://github.com/nodejs/node/blob/4d5ff25a813fd18939c9f76b17e36291e3ea15c3/lib/child_process.js#L412-L426
    cp.stderr?.destroy();
    cp.stdout?.destroy();
    cp.unref();
    return cp.kill(signal);

Is this a feature you are interested in implementing yourself?

Yes

[renovate-release]

🎉 This issue has been resolved in version 32.181.0 🎉

The release is available on:

• GitHub release
• 32.181.0

Your semantic-release bot 📦🚀