Created
March 17, 2025 19:54
-
-
Save JoshuaCWebDeveloper/de2da9b1baf9a9a249f3124a1ddad737 to your computer and use it in GitHub Desktop.
Fixes to node.impl.ts for https://github.com/nrwl/nx/issues/22999
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
import { ChildProcess, fork } from "child_process"; | |
import { randomUUID } from "crypto"; | |
import * as path from "path"; | |
import { join } from "path"; | |
import { | |
ExecutorContext, | |
isDaemonEnabled, | |
joinPathFragments, | |
logger, | |
parseTargetString, | |
ProjectGraphProjectNode, | |
readTargetOptions, | |
runExecutor, | |
} from "@nx/devkit"; | |
import { createAsyncIterable } from "@nx/devkit/src/utils/async-iterable"; | |
import { killTree } from "@nx/js/src/executors/node/lib/kill-tree"; | |
import { calculateProjectBuildableDependencies } from "@nx/js/src/utils/buildable-libs-utils"; | |
import { getRelativeDirectoryToProjectRoot } from "@nx/js/src/utils/get-main-file-dir"; | |
import * as chalk from "chalk"; | |
import { daemonClient } from "nx/src/daemon/client/client"; | |
import { interpolate } from "nx/src/tasks-runner/utils"; | |
import { fileExists } from "nx/src/utils/fileutils"; | |
import { InspectType, NodeExecutorOptions } from "./schema"; | |
const TERMINAL_RESET = "\x1b[0m"; | |
interface ActiveTask { | |
id: string; | |
killed: boolean; | |
promise: Promise<{ success: boolean }>; | |
childProcess: null | ChildProcess; | |
start: () => Promise<void>; | |
stop: (signal: NodeJS.Signals) => Promise<void>; | |
} | |
function debounce<T>(fn: () => Promise<T>, wait: number): () => Promise<T> { | |
let timeoutId: NodeJS.Timeout | null = null; | |
let pendingPromise: Promise<T> | null = null; | |
return () => { | |
if (timeoutId) { | |
return pendingPromise; | |
} | |
pendingPromise = new Promise<T>((resolve, reject) => { | |
timeoutId = setTimeout(() => { | |
fn().then(resolve, reject); | |
timeoutId = null; | |
pendingPromise = null; | |
}, wait); | |
}); | |
return pendingPromise; | |
}; | |
} | |
export async function* nodeExecutor( | |
options: NodeExecutorOptions, | |
context: ExecutorContext, | |
) { | |
process.env.NODE_ENV ??= context?.configurationName ?? "development"; | |
const project = context.projectGraph.nodes[context.projectName]; | |
const buildTarget = parseTargetString(options.buildTarget, context); | |
if (!project.data.targets[buildTarget.target]) { | |
throw new Error( | |
`Cannot find build target ${chalk.bold( | |
options.buildTarget, | |
)} for project ${chalk.bold(context.projectName)}`, | |
); | |
} | |
const buildTargetExecutor = | |
project.data.targets[buildTarget.target]?.executor; | |
if (buildTargetExecutor === "nx:run-commands") { | |
// Run commands does not emit build event, so we have to switch to run entire build through Nx CLI. | |
options.runBuildTargetDependencies = true; | |
} | |
const buildOptions: Record<string, any> = { | |
...readTargetOptions(buildTarget, context), | |
...options.buildTargetOptions, | |
target: buildTarget.target, | |
}; | |
if (options.waitUntilTargets && options.waitUntilTargets.length > 0) { | |
const results = await runWaitUntilTargets(options, context); | |
for (const [i, result] of results.entries()) { | |
if (!result.success) { | |
throw new Error( | |
`Wait until target failed: ${options.waitUntilTargets[i]}.`, | |
); | |
} | |
} | |
} | |
// Re-map buildable workspace projects to their output directory. | |
const mappings = calculateResolveMappings(context, options); | |
const fileToRun = getFileToRun( | |
context, | |
project, | |
buildOptions, | |
buildTargetExecutor, | |
); | |
let additionalExitHandler: null | (() => void) = null; | |
let currentTask: ActiveTask = null; | |
const tasks: ActiveTask[] = []; | |
yield* createAsyncIterable<{ | |
success: boolean; | |
options?: Record<string, any>; | |
}>(async ({ done, next, error, registerCleanup }) => { | |
const processQueue = async () => { | |
if (tasks.length === 0) return; | |
const previousTask = currentTask; | |
const task = tasks.shift(); | |
currentTask = task; | |
await previousTask?.stop("SIGTERM"); | |
await task.start(); | |
}; | |
const debouncedProcessQueue = debounce( | |
processQueue, | |
options.debounce ?? 1_000, | |
); | |
const addToQueue = async ( | |
createBuildProcess: (() => ChildProcess) | ActiveTask["promise"], | |
) => { | |
const task: ActiveTask = { | |
id: randomUUID(), | |
killed: false, | |
childProcess: null, | |
promise: null, | |
start: async () => { | |
if (typeof createBuildProcess === "function") { | |
// Create and execute the build process. | |
task.promise = new Promise(async (resolve) => { | |
task.childProcess = createBuildProcess(); | |
task.childProcess.once("exit", (code) => { | |
if (code === 0) resolve({ success: true }); | |
// If process is killed due to current task being killed, then resolve with success. | |
else resolve({ success: !!currentTask?.killed }); | |
}); | |
}); | |
} else { | |
task.promise = createBuildProcess; | |
} | |
// Wait for build to finish. | |
const result = await task.promise; | |
if (result && !result.success) { | |
// If in watch-mode, don't throw or else the process exits. | |
if (options.watch) { | |
if (!task.killed) { | |
// Only log build error if task was not killed by a new change. | |
logger.error(`Build failed, waiting for changes to restart...`); | |
} | |
return; | |
} else { | |
throw new Error(`Build failed. See above for errors.`); | |
} | |
} | |
// Before running the program, check if the task has been killed (by a new change during watch). | |
if (task.killed) return; | |
// Run the program | |
task.promise = new Promise((resolve, reject) => { | |
task.childProcess = fork( | |
joinPathFragments(__dirname, "node-with-require-overrides"), | |
options.args ?? [], | |
{ | |
execArgv: getExecArgv(options), | |
stdio: [0, "pipe", "pipe", "ipc"], | |
env: { | |
...process.env, | |
NX_FILE_TO_RUN: fileToRunCorrectPath(fileToRun), | |
NX_MAPPINGS: JSON.stringify(mappings), | |
}, | |
}, | |
); | |
// Handle stdout with careful buffering | |
const handleStdOut = (data) => { | |
if (options.watch && task.killed) return; | |
// Just pipe the data directly | |
process.stdout.write(data); | |
}; | |
task.childProcess.stdout.on("data", handleStdOut); | |
// Handle stderr similarly | |
const handleStdErr = (data) => { | |
// Don't log out error if task is killed and new one has started. | |
// This could happen if a new build is triggered while new process is starting, since the operation is not atomic. | |
// Log the error in normal mode | |
if (!options.watch || !task.killed) { | |
logger.error(data.toString()); | |
} | |
}; | |
task.childProcess.stderr.on("data", handleStdErr); | |
task.childProcess.once("exit", (code) => { | |
task.childProcess.stdout.removeListener("data", handleStdOut); | |
task.childProcess.stderr.removeListener("data", handleStdErr); | |
if (options.watch && !task.killed) { | |
logger.info( | |
`NX Process exited with code ${code}, waiting for changes to restart...`, | |
); | |
} | |
if (!options.watch) { | |
if (code !== 0) { | |
error(new Error(`Process exited with code ${code}`)); | |
} else { | |
done(); | |
} | |
} | |
resolve({ success: true }); | |
}); | |
next({ success: true, options: buildOptions }); | |
}); | |
}, | |
stop: async (signal = "SIGTERM") => { | |
task.killed = true; | |
// Request termination and wait for process to finish gracefully. | |
// NOTE: `childProcess` may not have been set yet if the task did not have a chance to start. | |
// e.g. multiple file change events in a short time (like git checkout). | |
if (task.childProcess) { | |
// Pause streams before killing | |
if (task.childProcess.stdout) task.childProcess.stdout.pause(); | |
if (task.childProcess.stderr) task.childProcess.stderr.pause(); | |
// Reset terminal state | |
process.stdout.write(TERMINAL_RESET); | |
await killTree(task.childProcess.pid, signal); | |
} | |
try { | |
await task.promise; | |
} catch { | |
// Doesn't matter if task fails, we just need to wait until it finishes. | |
} | |
// Small delay to ensure terminal processing is complete | |
await new Promise((resolve) => setTimeout(resolve, 500)); | |
// Reset terminal state again | |
process.stdout.write(TERMINAL_RESET); | |
}, | |
}; | |
tasks.push(task); | |
}; | |
const stopAllTasks = async (signal: NodeJS.Signals = "SIGTERM") => { | |
additionalExitHandler?.(); | |
await currentTask?.stop(signal); | |
for (const task of tasks) { | |
await task.stop(signal); | |
} | |
}; | |
process.on("SIGTERM", async () => { | |
await stopAllTasks("SIGTERM"); | |
process.exit(128 + 15); | |
}); | |
process.on("SIGINT", async () => { | |
await stopAllTasks("SIGINT"); | |
process.exit(128 + 2); | |
}); | |
process.on("SIGHUP", async () => { | |
await stopAllTasks("SIGHUP"); | |
process.exit(128 + 1); | |
}); | |
registerCleanup(async () => { | |
await stopAllTasks("SIGTERM"); | |
}); | |
if (options.runBuildTargetDependencies) { | |
// If a all dependencies need to be rebuild on changes, then register with watcher | |
// and run through CLI, otherwise only the current project will rebuild. | |
const runBuild = async () => { | |
await addToQueue(() => | |
fork( | |
require.resolve("node_modules/nx"), | |
[ | |
"run", | |
`${context.projectName}:${buildTarget.target}${buildTarget.configuration ? `:${buildTarget.configuration}` : ""}`, | |
], | |
{ | |
cwd: context.root, | |
stdio: "inherit", | |
}, | |
), | |
); | |
await debouncedProcessQueue(); | |
}; | |
if (isDaemonEnabled()) { | |
additionalExitHandler = await daemonClient.registerFileWatcher( | |
{ | |
watchProjects: [context.projectName], | |
includeDependentProjects: true, | |
}, | |
async (err, data) => { | |
if (err === "closed") { | |
logger.error(`Watch error: Daemon closed the connection`); | |
process.exit(1); | |
} else if (err) { | |
logger.error(`Watch error: ${err?.message ?? "Unknown"}`); | |
} else { | |
if (options.watch) { | |
logger.info(`NX File change detected. Restarting...`); | |
await runBuild(); | |
} | |
} | |
}, | |
); | |
} else { | |
logger.warn( | |
`NX Daemon is not running. Node process will not restart automatically after file changes.`, | |
); | |
} | |
await runBuild(); // run first build | |
} else { | |
// Otherwise, run the build executor, which will not run task dependencies. | |
// This is mostly fine for bundlers like webpack that should already watch for dependency libs. | |
// For tsc/swc or custom build commands, consider using `runBuildTargetDependencies` instead. | |
const output = await runExecutor( | |
buildTarget, | |
{ | |
...options.buildTargetOptions, | |
watch: options.watch, | |
}, | |
context, | |
); | |
while (true) { | |
const event = await output.next(); | |
await addToQueue(Promise.resolve(event.value)); | |
await debouncedProcessQueue(); | |
if (event.done || !options.watch) { | |
break; | |
} | |
} | |
} | |
}); | |
} | |
function getExecArgv(options: NodeExecutorOptions) { | |
const args = (options.runtimeArgs ??= []); | |
args.push("-r", require.resolve("source-map-support/register")); | |
if (options.inspect === true) { | |
options.inspect = InspectType.Inspect; | |
} | |
if (options.inspect) { | |
args.push(`--${options.inspect}=${options.host}:${options.port}`); | |
} | |
return args; | |
} | |
function calculateResolveMappings( | |
context: ExecutorContext, | |
options: NodeExecutorOptions, | |
) { | |
const parsed = parseTargetString(options.buildTarget, context); | |
const { dependencies } = calculateProjectBuildableDependencies( | |
context.taskGraph, | |
context.projectGraph, | |
context.root, | |
parsed.project, | |
parsed.target, | |
parsed.configuration, | |
); | |
return dependencies.reduce((m, c) => { | |
if (c.node.type !== "npm" && c.outputs[0] != null) { | |
m[c.name] = joinPathFragments(context.root, c.outputs[0]); | |
} | |
return m; | |
}, {}); | |
} | |
function runWaitUntilTargets( | |
options: NodeExecutorOptions, | |
context: ExecutorContext, | |
): Promise<{ success: boolean }[]> { | |
return Promise.all( | |
options.waitUntilTargets.map(async (waitUntilTarget) => { | |
const target = parseTargetString(waitUntilTarget, context); | |
const output = await runExecutor(target, {}, context); | |
return new Promise<{ success: boolean }>(async (resolve) => { | |
let event = await output.next(); | |
// Resolve after first event | |
resolve(event.value as { success: boolean }); | |
// Continue iterating | |
while (!event.done) { | |
event = await output.next(); | |
} | |
}); | |
}), | |
); | |
} | |
function getFileToRun( | |
context: ExecutorContext, | |
project: ProjectGraphProjectNode, | |
buildOptions: Record<string, any>, | |
buildTargetExecutor: string, | |
): string { | |
// If using run-commands or another custom executor, then user should set | |
// outputFileName, but we can try the default value that we use. | |
if (!buildOptions?.outputPath && !buildOptions?.outputFileName) { | |
// If we are using crystal for inferring the target, we can use the output path from the target. | |
// Since the output path has a token for the project name, we need to interpolate it. | |
// {workspaceRoot}/dist/{projectRoot} -> dist/my-app | |
const outputPath = project.data.targets[buildOptions.target]?.outputs?.[0]; | |
if (outputPath) { | |
const outputFilePath = interpolate(outputPath, { | |
projectName: project.name, | |
projectRoot: project.data.root, | |
workspaceRoot: context.root, | |
}); | |
return path.join(outputFilePath, "main.js"); | |
} | |
const fallbackFile = path.join("dist", project.data.root, "main.js"); | |
logger.warn( | |
`Build option ${chalk.bold("outputFileName")} not set for ${chalk.bold( | |
project.name, | |
)}. Using fallback value of ${chalk.bold(fallbackFile)}.`, | |
); | |
return join(context.root, fallbackFile); | |
} | |
let outputFileName = buildOptions.outputFileName; | |
if (!outputFileName) { | |
const fileName = `${path.parse(buildOptions.main).name}.js`; | |
if ( | |
buildTargetExecutor === "@nx/js:tsc" || | |
buildTargetExecutor === "@nx/js:swc" | |
) { | |
outputFileName = path.join( | |
getRelativeDirectoryToProjectRoot(buildOptions.main, project.data.root), | |
fileName, | |
); | |
} else { | |
outputFileName = fileName; | |
} | |
} | |
return join(context.root, buildOptions.outputPath, outputFileName); | |
} | |
function fileToRunCorrectPath(fileToRun: string): string { | |
if (fileExists(fileToRun)) return fileToRun; | |
const extensionsToTry = [".cjs", ".mjs", ".cjs.js", ".esm.js"]; | |
for (const ext of extensionsToTry) { | |
const file = fileToRun.replace(/\.js$/, ext); | |
if (fileExists(file)) return file; | |
} | |
throw new Error( | |
`Could not find ${fileToRun}. Make sure your build succeeded.`, | |
); | |
} | |
export default nodeExecutor; |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment