## Summary Add an opt-in source-development loop that restarts the dashboard and engine when runtime TypeScript or JSON changes. Use `pnpm dev:watch`; `pnpm dev:hmr` now combines Vite UI HMR with the same supervised API/engine restart path. The watcher filters tests, fixtures, generated declarations, build output, and task state. It coalesces bursts with a two-second maximum wait, waits for the child to acknowledge its IPC listener, and rebuilds runtime dist artifacts before a source-triggered respawn. ## Safety model - Close scheduler, triage, heartbeat, mission, routine, self-healing, and merge admission before checking for active work. - Let already-running agents reach a safe boundary; do not mutate durable pause settings. - Enter the existing graceful exit-code-86 shutdown and supervised respawn path. - Retry failed liveness reads and declined restart requests instead of dropping the pending change. - Keep ordinary `pnpm dev` behavior unchanged; inherited watch state does not break nested non-dashboard development commands. A development restart intentionally replaces the dashboard process, so transient dashboard connections and project dev-server children reconnect or restart with it. Agent work is the protected boundary. ## Validation - `pnpm lint` - `pnpm test:gate` (753 tests passed across engine, core, PostgreSQL gate, and CI-shape suites) - Focused CLI watcher/restart/supervision suites: 40 tests passed - Focused engine drain/manager suites: 52 tests passed - `pnpm --filter @runfusion/fusion typecheck` - `pnpm --filter @fusion/engine typecheck` - `pnpm verify:fast` (13 steps passed, including CLI build and real health boot smoke) - Manual unsupported-command probe confirms explicit `--watch` fails clearly outside the dashboard command ## Post-Deploy Monitoring & Validation - Watch for `[fusion:dev] source changed`, `source restart deferred`, `active work drained`, and `restart requested` logs during the first watched development session. - Healthy behavior is one exit-86 respawn per edit batch, no interrupted active agents, refreshed dist artifacts, and a healthy dashboard after respawn. - Investigate repeated restart loops, watcher attachment warnings, declined restart retries, or liveness-read failures. - Immediate mitigation is to use ordinary `pnpm dev` without `--watch`; no production runtime behavior or durable setting needs rollback. - Validation owner: Fusion maintainers during the first source edit after merge. --- [](https://github.com/EveryInc/compound-engineering-plugin) <!-- This is an auto-generated comment: release notes by coderabbit.ai --> ## Summary by CodeRabbit * **New Features** * Added `pnpm dev:watch` to automatically restart development runtime processes when source files change. * Development restarts now wait for active work to finish, preventing new work from starting during the transition. * Enhanced `pnpm dev:hmr` with graceful runtime source restarts while keeping the dashboard available. * Rapid source changes are grouped to avoid unnecessary restarts. * **Documentation** * Updated development setup and contribution guides with the new watch workflow. <!-- end of auto-generated comment: release notes by coderabbit.ai -->
228 lines
8.4 KiB
JavaScript
228 lines
8.4 KiB
JavaScript
#!/usr/bin/env node
|
|
/**
|
|
* Memory-aware development entrypoint for Fusion.
|
|
*
|
|
* This script increases the Node.js heap size to prevent memory pressure
|
|
* during the optional prebuild/start sequence, while preserving argument
|
|
* pass-through for documented invocations like `pnpm dev dashboard`.
|
|
*
|
|
* Cross-platform: Works on Windows, macOS, and Linux.
|
|
*/
|
|
import {
|
|
buildForwardedDevArgs,
|
|
buildDevNodeArgs,
|
|
createDevWatchRestartCoordinator,
|
|
getPrebuildCommand,
|
|
parseDevWrapperArgs,
|
|
resolvePrebuildMode,
|
|
} from "./dev-with-memory-lib.mjs";
|
|
import { createDevSourceWatcher } from "./lib/dev-source-watch.mjs";
|
|
|
|
// Set increased heap size (8GB) to prevent OOM during initial build/start
|
|
const MEMORY_MB = process.env.FUSION_DEV_MEMORY_MB || "8192";
|
|
|
|
// Spawn the actual dev command with all arguments passed through
|
|
const { spawn } = await import("child_process");
|
|
const rawArgs = process.argv.slice(2);
|
|
let parsedArgs;
|
|
try {
|
|
parsedArgs = parseDevWrapperArgs(rawArgs);
|
|
} catch (error) {
|
|
console.error(error instanceof Error ? error.message : String(error));
|
|
process.exit(1);
|
|
}
|
|
const { inspectFlags, args, requestedPrebuild, watchSourceFromFlag } = parsedArgs;
|
|
let { watchSource } = parsedArgs;
|
|
|
|
// NODE_OPTIONS is shared with every spawned node process (build + run +
|
|
// agents). Heap size belongs here. Inspector flags do NOT — see comment above.
|
|
const nodeOptions = `--max-old-space-size=${MEMORY_MB} ${process.env.NODE_OPTIONS || ""}`.trim();
|
|
process.env.NODE_OPTIONS = nodeOptions;
|
|
|
|
// In dev we bind the dashboard to 0.0.0.0 so the server is reachable from
|
|
// mobile devices and other machines on the LAN for testing. Production
|
|
// builds default to 127.0.0.1; this override only applies when starting
|
|
// the dashboard via `pnpm dev dashboard` and only if no --host was passed.
|
|
const forwardedArgs = buildForwardedDevArgs(args);
|
|
if (watchSource && forwardedArgs[0] !== "dashboard") {
|
|
if (watchSourceFromFlag) {
|
|
console.error("[fusion:dev] --watch is supported for the dashboard engine process only");
|
|
process.exit(1);
|
|
}
|
|
watchSource = false;
|
|
}
|
|
const prebuildMode = resolvePrebuildMode(requestedPrebuild, forwardedArgs);
|
|
const prebuildCommand = getPrebuildCommand(prebuildMode);
|
|
|
|
// Resolve absolute paths to tsx loader so they survive shell quoting.
|
|
// Use Node's resolver instead of hardcoding the pnpm version-specific path.
|
|
const { createRequire } = await import("node:module");
|
|
const path = await import("node:path");
|
|
const require = createRequire(import.meta.url);
|
|
const tsxPkgJson = require.resolve("tsx/package.json");
|
|
const tsxDir = path.dirname(tsxPkgJson);
|
|
const PRELOAD = path.join(tsxDir, "dist", "preflight.cjs");
|
|
const LOADER = path.join(tsxDir, "dist", "loader.mjs");
|
|
const ENTRY = path.resolve(process.cwd(), "packages/cli/src/bin.ts");
|
|
|
|
// Spawn node directly (no shell) so the inspector attaches to the real app
|
|
// process and there's no parent/child wrapper consuming --inspect.
|
|
// Inspector flags are CLI args here so they apply only to this process and
|
|
// don't propagate to grandchildren via NODE_OPTIONS.
|
|
/*
|
|
FNXC:SystemPanel 2026-07-12-10:45:
|
|
This wrapper is the supervising parent for `pnpm dev` / `pnpm start`, so it is
|
|
where the dashboard System panel's "Restart"/"Rebuild & restart" actions land:
|
|
the child exits with FUSION_RESTART_EXIT_CODE (86 — keep in sync with
|
|
packages/core/src/process-supervisor.ts) and we respawn the same command
|
|
immediately, keeping the same terminal/TTY so the TUI comes back seamlessly.
|
|
FUSION_RESTART_SUPERVISED=1 tells the child a respawning parent exists, which
|
|
is what makes the dashboard advertise restart support. Any other exit code
|
|
propagates unchanged (no crash-restart loop here — `--supervise` owns that).
|
|
*/
|
|
const RESTART_EXIT_CODE = 86;
|
|
let appChild;
|
|
let sourceWatcher;
|
|
const watchRestart = createDevWatchRestartCoordinator();
|
|
|
|
function ensureSourceWatcher() {
|
|
if (!watchSource || sourceWatcher) return;
|
|
sourceWatcher = createDevSourceWatcher({
|
|
rootDir: process.cwd(),
|
|
onRestart: (paths) => watchRestart.request(paths),
|
|
});
|
|
console.log(`[fusion:dev] source watch active (${sourceWatcher.watchedPaths.join(", ")})`);
|
|
}
|
|
|
|
function runApp(extraArgs) {
|
|
const tsx = spawn(process.execPath, buildDevNodeArgs({
|
|
inspectFlags,
|
|
preload: PRELOAD,
|
|
loader: LOADER,
|
|
entry: ENTRY,
|
|
args: extraArgs,
|
|
}), {
|
|
stdio: watchSource ? ["inherit", "inherit", "inherit", "ipc"] : "inherit",
|
|
// FNXC:SystemPanel 2026-07-25-10:05: stamp the supervisor pid alongside the
|
|
// flag so the child can tell a real supervising parent from an inherited
|
|
// copy of the variable (see hasLiveSupervisingParent in commands/dashboard.ts).
|
|
env: {
|
|
...process.env,
|
|
FUSION_RESTART_SUPERVISED: "1",
|
|
FUSION_SUPERVISOR_PID: String(process.pid),
|
|
...(watchSource ? { FUSION_DEV_WATCH: "1" } : {}),
|
|
},
|
|
});
|
|
appChild = tsx;
|
|
watchRestart.attach(tsx);
|
|
tsx.on("message", (message) => watchRestart.onMessage(message));
|
|
ensureSourceWatcher();
|
|
tsx.on("close", (c) => {
|
|
const sourceRestart = watchRestart.detach(tsx);
|
|
if (appChild === tsx) appChild = undefined;
|
|
if (c === RESTART_EXIT_CODE) {
|
|
console.log("[fusion:dev] restart requested — restarting…");
|
|
if (sourceRestart && prebuildCommand) {
|
|
runPrebuild(() => runApp(extraArgs));
|
|
} else {
|
|
runApp(extraArgs);
|
|
}
|
|
return;
|
|
}
|
|
process.exit(c ?? 1);
|
|
});
|
|
}
|
|
|
|
function runPrebuild(onSuccess) {
|
|
console.log(`[fusion] Running ${prebuildCommand.label} (${prebuildMode}) before source startup...`);
|
|
const build = spawn(prebuildCommand.command, prebuildCommand.args, { stdio: "inherit", shell: true });
|
|
build.on("close", (code) => {
|
|
if (code !== 0) process.exit(code ?? 1);
|
|
onSuccess();
|
|
});
|
|
}
|
|
|
|
async function warnIfSourceVersionBehind() {
|
|
if (process.env.FUSION_SKIP_STARTUP_UPDATE_PREFLIGHT === "1") {
|
|
return;
|
|
}
|
|
|
|
let currentVersion;
|
|
try {
|
|
const { readFile } = await import("node:fs/promises");
|
|
const pkg = JSON.parse(await readFile(path.resolve(process.cwd(), "packages/cli/package.json"), "utf8"));
|
|
currentVersion = typeof pkg.version === "string" ? pkg.version : undefined;
|
|
} catch {
|
|
return;
|
|
}
|
|
|
|
if (!currentVersion) return;
|
|
|
|
try {
|
|
const controller = new AbortController();
|
|
const timeout = setTimeout(() => controller.abort(), 1_500);
|
|
let payload;
|
|
try {
|
|
const response = await fetch("https://registry.npmjs.org/@runfusion%2Ffusion", {
|
|
signal: controller.signal,
|
|
});
|
|
payload = await response.json();
|
|
} finally {
|
|
clearTimeout(timeout);
|
|
}
|
|
const latestVersion = payload?.["dist-tags"]?.latest;
|
|
if (typeof latestVersion !== "string") return;
|
|
|
|
const currentParts = currentVersion.split(".").map((part) => Number.parseInt(part, 10) || 0);
|
|
const latestParts = latestVersion.split(".").map((part) => Number.parseInt(part, 10) || 0);
|
|
let latestIsNewer = false;
|
|
for (let i = 0; i < Math.max(currentParts.length, latestParts.length, 3); i += 1) {
|
|
const latest = latestParts[i] ?? 0;
|
|
const current = currentParts[i] ?? 0;
|
|
if (latest > current) {
|
|
latestIsNewer = true;
|
|
break;
|
|
}
|
|
if (latest < current) {
|
|
break;
|
|
}
|
|
}
|
|
|
|
if (latestIsNewer) {
|
|
console.warn(
|
|
`\n[fusion] This source checkout is v${currentVersion}, but npm latest is v${latestVersion}. ` +
|
|
"If you meant to run the latest Fusion, pull/switch branches before startup.\n",
|
|
);
|
|
}
|
|
} catch {
|
|
// Best-effort only. Startup must not depend on the registry.
|
|
}
|
|
}
|
|
|
|
await warnIfSourceVersionBehind();
|
|
|
|
// FNXC:DevWorkflow 2026-06-18-16:50:
|
|
// FN-6638 stale-dist guard. Warn (loudly, best-effort) when built dist/ is older
|
|
// than src/ so a never-rebuilt/never-restarted process does not silently run
|
|
// phantom-old code. When a prebuild is about to run it will refresh dist, so the
|
|
// check is informational there; for --prebuild none / dist-resolving consumers
|
|
// it is the safety net. Never let the check break startup.
|
|
async function warnIfDistStale() {
|
|
if (process.env.FUSION_SKIP_DIST_FRESHNESS_CHECK === "1") return;
|
|
try {
|
|
const { computeDistStaleness, formatDistStalenessWarning } = await import("./lib/dist-freshness.mjs");
|
|
const warning = formatDistStalenessWarning(computeDistStaleness({ rootDir: process.cwd() }));
|
|
if (warning) console.warn(warning);
|
|
} catch {
|
|
// Best-effort only. Startup must not depend on the freshness check.
|
|
}
|
|
}
|
|
|
|
await warnIfDistStale();
|
|
|
|
if (!prebuildCommand) {
|
|
runApp(forwardedArgs);
|
|
} else {
|
|
runPrebuild(() => runApp(forwardedArgs));
|
|
}
|