@hominis/fireforge 0.54.0 → 0.60.0

package/dist/src/commands/build.js

@@ -1,441 +1,37 @@
 // SPDX-License-Identifier: EUPL-1.2
-import { join } from 'node:path';
-import { Command, InvalidArgumentError as CommanderInvalidArgumentError } from 'commander';
-import { validateBrandOverride } from '../core/brand-validation.js';
-import { auditBuildArtifacts } from '../core/build-audit.js';
-import { readBuildBaseline, writeBuildBaseline } from '../core/build-baseline.js';
-import { describeSignalShapedExit, prepareBuildEnvironment } from '../core/build-prepare.js';
-import { getProjectPaths, loadConfig } from '../core/config.js';
-import { assertEngineExists } from '../core/engine-precondition.js';
-import { withEngineSessionLock } from '../core/engine-session-lock.js';
-import { attemptMozinfoRewrite, build, buildArtifactMismatchMessage, buildUI, hasBuildArtifacts, hasRunnableBundle, runMach, withBuildLock, } from '../core/mach.js';
-import { findObjdirRelocationViolation } from '../core/mach-build-artifacts.js';
-import { ensureLaunchableBrowserNotRunning } from '../core/marionette-port.js';
-import { assertHealthyPartialConfig, pruneDanglingTestLinks } from '../core/objdir-maintenance.js';
-import { waitForPreflight } from '../core/preflight-wait.js';
-import { closeActiveRunLog, getActiveRunLogPath, keepRunLogCopy, openRunLog, setActiveRunLog, } from '../core/run-log.js';
-import { buildHarnessCrashMessage } from '../core/test-harness-crash.js';
-import { formatToolchainMismatchMessage, runToolchainPreflight, } from '../core/toolchain-preflight.js';
-import { GeneralError, InvalidArgumentError } from '../errors/base.js';
-import { AmbiguousBuildArtifactsError, BuildError } from '../errors/build.js';
-import { elapsedSince } from '../utils/elapsed.js';
-import { toError } from '../utils/errors.js';
-import { checkDiskSpace } from '../utils/fs.js';
-import { error, info, intro, notice, outro, spinner, verbose, warn } from '../utils/logger.js';
-import { addWaitLockOption, commanderArgParser, pickDefined, resolveWaitLockSeconds, } from '../utils/options.js';
-import { isPositiveInteger } from '../utils/validation.js';
-import { waitForBuildBrowserBeforeLock } from './build-prelock.js';
-function parseJobCount(value) {
-    const parsed = Number(value);
-    if (!Number.isSafeInteger(parsed) || parsed < 1) {
-        throw new CommanderInvalidArgumentError('jobs must be a positive integer');
-    }
-    return parsed;
-}
-/**
- * Patches `mozinfo.json` under the active obj-* directory so its recorded
- * paths match the current engine checkout, then runs `mach configure` to
- * regenerate every other generated path alongside it. Throws with the
- * original mismatch guidance when the rewriter refuses to proceed. That
- * covers the unsafe cases, so the operator still gets the correct fallback
- * recovery instruction.
- */
-async function rewriteAndReconfigure(engineDir, objDir, mismatchMessage) {
-    const rewriteSpinner = spinner('Rewriting mozinfo.json paths...');
-    let rewrite;
-    try {
-        rewrite = await attemptMozinfoRewrite(engineDir, objDir);
-    }
-    catch (rewriteError) {
-        rewriteSpinner.error('mozinfo rewrite failed');
-        throw new GeneralError(`${mismatchMessage}\n\nmozinfo rewrite failed: ${toError(rewriteError).message}`);
-    }
-    if (!rewrite.rewritten) {
-        rewriteSpinner.error('mozinfo rewrite refused');
-        throw new GeneralError(`${mismatchMessage}\n\nmozinfo rewrite refused: ${rewrite.reason ?? 'unspecified reason'}`);
-    }
-    rewriteSpinner.stop(`mozinfo.json patched (topsrcdir → ${rewrite.newTopsrcdir})`);
-    const configureSpinner = spinner('Running mach configure against rewritten mozinfo.json...');
-    let exitCode;
-    try {
-        exitCode = await runMach(['configure'], engineDir);
-    }
-    catch (configureError) {
-        configureSpinner.error('mach configure failed');
-        throw new BuildError('mach configure failed after mozinfo rewrite', 'mach configure', configureError instanceof Error ? configureError : undefined);
-    }
-    if (exitCode !== 0) {
-        configureSpinner.error('mach configure exited non-zero');
-        throw new BuildError(`mach configure exited non-zero (${exitCode}) after mozinfo rewrite; a clean rebuild is required.`, 'mach configure');
-    }
-    configureSpinner.stop('mach configure regenerated the backend');
-    const violation = await findObjdirRelocationViolation({
-        engineDir,
-        objDir,
-        forbiddenDir: rewrite.oldTopsrcdir ?? '',
-    });
-    if (violation)
-        throw new GeneralError(`Relocation incomplete: ${violation}`);
-    info('Backend path-rewrite complete; continuing with the build.');
-}
-function resolveJobCount(options, configJobs) {
-    const jobs = options.jobs ?? configJobs;
-    if (jobs === undefined) {
-        return undefined;
-    }
-    if (!isPositiveInteger(jobs)) {
-        throw new InvalidArgumentError('Build jobs must be a positive integer', '--jobs');
-    }
-    return jobs;
-}
-function tailLines(text, maxLines) {
-    const lines = text
-        .split(/\r?\n/)
-        .map((line) => line.trimEnd())
-        .filter((line) => line.length > 0);
-    return lines.slice(-maxLines).join('\n');
-}
-function extractLastMakeError(captured) {
-    const lines = captured.split(/\r?\n/).filter((line) => /\bg?make(?:\[\d+\])?: \*\*\*/.test(line));
-    const actionable = lines.filter((line) => !/\[\s*(?:all|build|default)\s*\]\s+Error\s+\d+/i.test(line));
-    return (actionable.at(-1) ?? lines.at(-1))?.trim();
-}
-function isWarningOnlyLine(line) {
-    if (/\b(?:error|failed|fatal)\b/i.test(line))
-        return false;
-    return (/\bwarning\b/i.test(line) ||
-        /urllib3|LibreSSL|NotOpenSSLWarning|InsecurePlatformWarning/i.test(line));
-}
-function extractRecentMakeContext(captured) {
-    const lines = captured
-        .split(/\r?\n/)
-        .map((line) => line.trimEnd())
-        .filter((line) => line.trim().length > 0);
-    const makeLines = lines.filter((line) => /\b(?:g?make)(?:\[\d+\])?:/.test(line));
-    if (makeLines.length === 0)
-        return undefined;
-    return makeLines.slice(-6).join('\n');
-}
-function extractLikelyFailingCommand(captured) {
-    const lines = captured
-        .split(/\r?\n/)
-        .map((line) => line.trim())
-        .filter((line) => line.length > 0);
-    let lastMakeErrorIndex = -1;
-    for (let index = lines.length - 1; index >= 0; index--) {
-        if (/\bg?make(?:\[\d+\])?: \*\*\*/.test(lines[index] ?? '')) {
-            lastMakeErrorIndex = index;
-            break;
-        }
-    }
-    const startIndex = lastMakeErrorIndex > 0 ? lastMakeErrorIndex - 1 : lines.length - 1;
-    for (let index = startIndex; index >= 0; index--) {
-        const line = lines[index];
-        if (!line)
-            continue;
-        if (isWarningOnlyLine(line))
-            continue;
-        if (/^g?make(?:\[\d+\])?:/.test(line))
-            continue;
-        if (/^Error running mach:/.test(line))
-            continue;
-        const comparable = line.replace(/^\d+:\d+\.\d+\s+/, '');
-        if (/\b(?:cp|clang|clang\+\+|rustc|python|node|make|install_name_tool)\b/.test(comparable)) {
-            return line;
-        }
-    }
-    return undefined;
-}
-function buildFailureDiagnostics(result, engineDir, objDir, machCommand) {
-    const captured = `${result.stderr}\n${result.stdout}`;
-    const stderrTail = tailLines(result.stderr, 20);
-    const combinedTail = tailLines(captured, 30);
-    const makeError = extractLastMakeError(captured);
-    const makeContext = extractRecentMakeContext(captured);
-    const failingCommand = extractLikelyFailingCommand(captured);
-    const logHint = objDir
-        ? `engine/${objDir}/ (inspect build logs, warnings, and generated make targets under this objdir)`
-        : 'engine/obj-* (inspect the active objdir for build logs, warnings, and generated make targets)';
-    const verboseRerun = objDir
-        ? `cd ${engineDir} && ./mach build -v; if a make target is named above, retry it with: make -C ${objDir} <target> V=1`
-        : `cd ${engineDir} && ./mach build -v`;
-    return [
-        `Build failed with exit code ${result.exitCode}.`,
-        describeSignalShapedExit(result.exitCode),
-        `Mach phase: ${machCommand}`,
-        makeError ? `Last make error: ${makeError}` : undefined,
-        makeContext ? `Recent make context:\n${makeContext}` : undefined,
-        failingCommand ? `Final failing command/error line: ${failingCommand}` : undefined,
-        stderrTail ? `Captured stderr tail:\n${stderrTail}` : undefined,
-        `Captured output tail:\n${combinedTail}`,
-        `Logs/profile/warnings: ${logHint}`,
-        `Verbose rerun: ${verboseRerun}`,
-    ]
-        .filter((part) => part !== undefined && part.length > 0)
-        .join('\n\n');
-}
-/**
- * Runs the build command.
- * @param projectRoot - Root directory of the project
- * @param options - Build options
- * @param underLock - Runs the build under the engine session lock; a
- *   `--wait-browser` budget is spent before it (see build-prelock.ts)
- */
-export async function buildCommand(projectRoot, options, underLock = (body) => body()) {
-    const buildType = options.ui ? 'UI-only' : 'Full';
-    const brandInfo = options.brand ? ` [${options.brand}]` : '';
-    intro(`FireForge Build (${buildType}${brandInfo})`);
-    // A build's own complete output, kept independently of whatever the
-    // operator piped the run through. `build` prints no FIREFORGE-VERDICT
-    // line, so the path is announced directly at the end of the run.
-    setActiveRunLog(await keepRunLogCopy(await openRunLog(projectRoot, 'build'), options.keepLog, 'build'));
-    try {
-        const lockedOptions = await waitForBuildBrowserBeforeLock(projectRoot, options);
-        await underLock(() => runBuildCommandBody(projectRoot, lockedOptions));
-    }
-    finally {
-        const logPath = getActiveRunLogPath();
-        await closeActiveRunLog();
-        if (logPath !== undefined)
-            info(`Full build output: ${logPath}`);
-    }
-}
-/** Checks objdir health and prevents deployments under a running browser. */
-async function preflightBuildObjdir(engineDir, binaryName, objDir, options) {
-    if (!objDir)
-        return;
-    await assertHealthyPartialConfig(engineDir, objDir);
-    const bundle = await hasRunnableBundle(engineDir, binaryName, objDir);
-    const expectedPath = bundle.expectedPath;
-    if (bundle.runnable && expectedPath) {
-        await waitForPreflight(() => ensureLaunchableBrowserNotRunning(join(engineDir, expectedPath)), options.waitBrowser === undefined ? undefined : resolveWaitLockSeconds(options.waitBrowser));
-    }
-    const pruned = await pruneDanglingTestLinks(engineDir, objDir);
-    if (pruned > 0)
-        notice(`Pruned ${pruned} dangling _tests symlink(s).`);
-}
-async function runBuildCommandBody(projectRoot, options) {
-    const buildType = options.ui ? 'UI-only' : 'Full';
-    // Load configuration
-    const config = await loadConfig(projectRoot);
-    const paths = getProjectPaths(projectRoot);
-    validateBrandOverride(config.binaryName, options.brand);
-    // Disk space pre-flight: a full Firefox build can be ~20 GB
-    await checkDiskSpace(projectRoot, 20 * 1024 * 1024 * 1024, warn);
-    // Check if engine exists
-    await assertEngineExists(paths.engine);
-    // Toolchain preflight: compare the minimums the tree itself declares
-    // (cbindgen / rust) against the host binaries mach configure will resolve,
-    // and fail fast naming `fireforge bootstrap` instead of dying ~8s into
-    // configure with mach's "./mach bootstrap" remediation text. Fail-soft by
-    // design: only a definitively parsed minimum vs a definitively probed host
-    // version can fail here. Anything uncertain proceeds to mach, where the
-    // mach-error-hints translator still names the right remedy.
-    const toolchainMismatches = await runToolchainPreflight(paths.engine);
-    if (toolchainMismatches.length > 0) {
-        throw new BuildError(formatToolchainMismatchMessage(toolchainMismatches), 'toolchain preflight');
-    }
-    const buildCheck = await hasBuildArtifacts(paths.engine);
-    if (buildCheck.ambiguous && buildCheck.objDirs && buildCheck.objDirs.length > 0) {
-        throw new AmbiguousBuildArtifactsError(buildCheck.objDirs);
-    }
-    const mismatchMessage = buildArtifactMismatchMessage(paths.engine, buildCheck, 'Build');
-    if (mismatchMessage) {
-        if (options.rewriteMozinfo && buildCheck.objDir) {
-            // Safe-relocation rewrite path: patch mozinfo.json paths in place so
-            // `mach configure` can regenerate the backend without scrubbing the
-            // whole obj tree. The rewriter refuses anything that is not a pure
-            // prefix-move. On refusal we surface the refusal reason alongside
-            // the original mismatch guidance and abort.
-            await rewriteAndReconfigure(paths.engine, buildCheck.objDir, mismatchMessage);
-        }
-        else {
-            throw new GeneralError(mismatchMessage);
-        }
-    }
-    if (options.ui) {
-        if (!buildCheck.exists || !buildCheck.objDir) {
-            const detail = buildCheck.objDir
-                ? `Build artifacts incomplete in ${buildCheck.objDir}/`
-                : 'No completed obj-* build artifacts found.';
-            throw new GeneralError(`UI-only builds require a completed full build first. ${detail}\n\n` +
-                'Run "fireforge build" and let it finish, then retry "fireforge build --ui".');
-        }
-        const bundleCheck = await hasRunnableBundle(paths.engine, config.binaryName, buildCheck.objDir);
-        if (!bundleCheck.runnable) {
-            const expectedSuffix = bundleCheck.expectedPath
-                ? ` Expected launchable binary at engine/${bundleCheck.expectedPath}.`
-                : '';
-            throw new GeneralError(`UI-only builds require a completed full build first.${expectedSuffix}\n\n` +
-                'Freshly imported or partially built trees cannot use `mach build faster` yet. ' +
-                'Run "fireforge build" and let it finish, then retry "fireforge build --ui".');
-        }
-    }
-    // Log brand info if specified
-    if (options.brand) {
-        verbose(`Building with brand: ${options.brand}`);
-        // Future: Load brand-specific config from fireforge.json brands section
-        info(`Brand: ${options.brand}`);
-    }
-    // Read the previous build baseline BEFORE prepareBuildEnvironment so the
-    // auto-configure step there can detect moz.build-family changes since the
-    // last successful build. The post-build audit below reuses the same
-    // baseline to diff engine changes against dist artifacts.
-    await preflightBuildObjdir(paths.engine, config.binaryName, buildCheck.objDir, options);
-    const previousBaseline = await readBuildBaseline(projectRoot);
-    // Shared pre-flight: branding, Furnace, mozconfig, auto-configure
-    await prepareBuildEnvironment(projectRoot, paths, config, {
-        previousBaseline,
-        ...(options.refuseUnexportedDrift !== undefined
-            ? { refuseUnexportedDrift: options.refuseUnexportedDrift }
-            : {}),
-    });
-    const jobs = resolveJobCount(options, config.build?.jobs);
-    // Run build
-    info(`Starting ${buildType.toLowerCase()} build...`);
-    if (jobs !== undefined) {
-        info(`Using ${jobs} parallel jobs`);
-    }
-    info(''); // Empty line before build output
-    const startTime = Date.now();
-    let result;
-    try {
-        // Hold the per-project build lock across the mach invocation so two
-        // overlapping `fireforge build` / `fireforge build --ui` commands
-        // against the same engine tree serialise instead of racing through the
-        // same obj-*. A `build --ui` launched during an in-progress full build
-        // otherwise hits `No rule to make target 'XUL'` in mach, the
-        // downstream consequence of an incomplete backend, with no clue that a
-        // concurrent build caused it. The lock turns the second invocation's
-        // failure into an explicit refusal naming the holder PID.
-        result = await withBuildLock(projectRoot, async () => {
-            if (options.ui) {
-                return buildUI(paths.engine);
-            }
-            return build(paths.engine, jobs);
+import { defineCommand, lockWaitMs, opt } from '../cli/command.js';
+import { build as buildWorkspace } from '../engine/index.js';
+import { formatDuration } from '../kernel/time.js';
+import { deployComponents, withWorkspace } from '../mode/index.js';
+import { loadProject } from '../project/index.js';
+import { workspaceOptions } from './workspace.js';
+export const build = defineCommand({
+    name: 'build',
+    summary: 'Build the browser in the workspace',
+    description: 'Deploys the components, then runs mach build. --ui rebuilds the front end only and becomes ' +
+        'a full build when a compiled source or build definition changed since the last one.',
+    group: 'Daily',
+    options: {
+        ui: opt.flag('Rebuild the front end only (mach build faster)'),
+        jobs: opt.integer('Parallel compile jobs (default build.jobs)', { short: 'j', min: 1 }),
+        ...workspaceOptions,
+    },
+    json: true,
+    run: async (ctx, _args, opts) => {
+        const project = await loadProject(ctx.cwd, ctx.env);
+        const waitMs = lockWaitMs(opts.waitLock);
+        const { workspace, result } = await withWorkspace(project, { command: 'build', workspace: opts.workspace, waitMs }, async (leased) => {
+            await deployComponents(leased, { waitMs });
+            const built = await buildWorkspace(leased, { ui: opts.ui, jobs: opts.jobs, waitMs });
+            return { workspace: leased.name, result: built };
         });
-    }
-    catch (error) {
-        throw new BuildError('Build process failed to start', options.ui ? 'mach build faster' : 'mach build', error instanceof Error ? error : undefined);
-    }
-    const timeStr = elapsedSince(startTime);
-    if (result.exitCode !== 0) {
-        error(`Build failed after ${timeStr}`);
-        const machCommand = options.ui ? 'mach build faster' : 'mach build';
-        // When the protected dispatch exhausted its recognized-crash retry
-        // budget, lead with the environmental-crash explanation so the
-        // operator does not read a resource-monitor traceback as a build
-        // regression.
-        const crashPreamble = result.crashSignature
-            ? `${buildHarnessCrashMessage(result.crashSignature, result.attempts, machCommand)}\n\n`
-            : '';
-        throw new BuildError(crashPreamble + buildFailureDiagnostics(result, paths.engine, buildCheck.objDir, machCommand), machCommand);
-    }
-    // Tool-managed branding edits that land on `browser/moz.configure` before
-    // the build cause mach's post-build guard to print one of two banners that
-    // read like build failures even though the build completed cleanly:
-    //
-    //   1) "config.status is out of date with respect to ..."
-    //   2) "Config object not found by mach. / Configure complete! /
-    //      Be sure to run |mach build| to pick up any changes."
-    //
-    // Matching only (1) leaves operators on the (2) path seeing mach's own
-    // "Configure complete!" and "run |mach build|" lines unexplained between
-    // mach's "Your build was successful!" and FireForge's "Build completed in
-    // Xm Ys" outro, a contradictory tail. Both shapes route through the same
-    // annotation, emitted before FireForge's outro so the operator's last
-    // terminal line is the explanation instead of the confusing mach guard
-    // text.
-    const staleConfigurePatterns = [
-        /config\.status is out of date/i,
-        /Config object not found by mach\.[\s\S]*Configure complete!/i,
-    ];
-    const captured = `${result.stdout}\n${result.stderr}`;
-    if (staleConfigurePatterns.some((p) => p.test(captured))) {
-        info('Note: mach printed a post-build "Configure complete!" / "config.status is out of date" ' +
-            'banner. That is a known side effect of tool-managed branding edits applied before the ' +
-            'build and does not mean the build is stale or that you need to rerun mach — the FireForge ' +
-            'exit code is authoritative.');
-    }
-    // Warn-only post-build audit: surfaces silent packaging drops (files
-    // edited in engine/ but never registered for packaging) against the
-    // previous-build baseline. Never fails the build. The worst case is a
-    // warning an operator chooses to investigate.
-    try {
-        await auditBuildArtifacts(projectRoot, paths.engine, previousBaseline, {
-            ...(config.buildAudit?.unpackaged ? { unpackaged: config.buildAudit.unpackaged } : {}),
+        ctx.out.line(`${result.kind === 'ui' ? 'UI build' : 'Full build'} of workspace "${workspace}" finished in ${formatDuration(result.durationMs)}.`);
+        ctx.out.json({
+            workspace,
+            kind: result.kind,
+            escalated: result.escalated ?? null,
+            durationMs: result.durationMs,
+            unpackaged: result.unpackaged,
         });
-    }
-    catch (auditError) {
-        verbose(`Audit skipped: ${toError(auditError).message}`);
-    }
-    // Record a fresh baseline only on clean success so the next run audits
-    // against this build's HEAD. A failed build keeps the prior baseline so
-    // the next attempt still catches long-standing packaging drops.
-    await recordFullBuildBaseline(projectRoot, paths.engine, config.binaryName, options.ui === true, previousBaseline);
-    outro(`Build completed in ${timeStr}!`);
-}
-/**
- * Persists the full-coverage baseline. A failed write never fails the build.
- * `--ui` is a `mach build faster`, so it is recorded as such: its `jar.mn`
- * fingerprints are carried forward from the previous baseline rather than
- * refreshed, which is why that baseline is threaded through.
- */
-async function recordFullBuildBaseline(projectRoot, engineDir, binaryName, ui, previousBaseline) {
-    try {
-        await writeBuildBaseline({
-            projectRoot,
-            engineDir,
-            binaryName,
-            testPackagingCoverage: 'full',
-            previousBaseline,
-            recordedBy: ui ? 'fireforge build --ui' : 'fireforge build',
-            staticComponentsHandling: 'auto',
-            buildKind: ui ? 'faster' : 'full',
-            // A full build recompiles a negative control's restored source, so
-            // it drops the control-built stamp; `--ui` compiles nothing and
-            // keeps it.
-            controlBuild: ui ? previousBaseline?.controlBuild : undefined,
-        });
-    }
-    catch (baselineError) {
-        verbose(`Could not persist build baseline: ${toError(baselineError).message}`);
-    }
-}
-/** Registers the build command on the CLI program. */
-export function registerBuild(program, { getProjectRoot, withErrorHandling }) {
-    const build = program
-        .command('build')
-        .description('Build the browser (auto-applies Furnace components first)')
-        .option('--wait-browser [seconds]', 'Wait for this objdir browser to exit before changing its build (default 60)', commanderArgParser((raw) => resolveWaitLockSeconds(raw)))
-        .option('--keep-log <path>', "Also write this run's log to <path> (a directory receives it under its own name), outside .fireforge/logs/ retention. FIREFORGE_LOG_KEEP sets a session default.")
-        .option('--ui', 'Fast UI-only rebuild')
-        .option('-j, --jobs <n>', 'Number of parallel jobs', parseJobCount)
-        .option('--brand <name>', 'Build specific brand')
-        .option('--rewrite-mozinfo', 'On a mozinfo path mismatch, patch mozinfo.json paths in place and run mach configure instead of aborting with a clean-rebuild instruction. Refuses anything that is not a pure prefix-move.')
-        .option('--refuse-unexported-drift', 'Refuse (instead of warning) when the build would overwrite engine content recorded in neither a patch body nor the pristine baseline — the multi-session hazard where an unexported edit is silently reverted and a later re-export captures a hybrid. A deployed Furnace file that is a deployment of its own component source is allowed')
-        .addHelpText('after', [
-        '',
-        'Furnace apply runs automatically before the build step, so edits in',
-        'components/custom/ and components/overrides/ are propagated to the',
-        'engine/ tree every time. The command prints a banner listing the',
-        'components synced during the current invocation.',
-        '',
-        'If you want to preview the engine state without triggering a build,',
-        'run `fireforge furnace apply` directly. For source-change-driven',
-        'rebuild loops during development, use `fireforge watch`.',
-        '',
-        '--rewrite-mozinfo: when a workspace is moved to a new path, mozinfo.json',
-        'still records the old topsrcdir/topobjdir and the build aborts with a',
-        'delete-and-rebuild instruction. This flag patches those paths in place',
-        'and runs mach configure — preserving up to ~20 GB of obj-* artefacts on',
-        'a relocation. The rewriter refuses any change that is not a pure prefix',
-        'move, in which case a clean rebuild is still required.',
-    ].join('\n'));
-    addWaitLockOption(build).action(withErrorHandling(async (options) => {
-        const projectRoot = getProjectRoot();
-        await buildCommand(projectRoot, pickDefined(options), (body) => withEngineSessionLock(projectRoot, 'build', body, {
-            waitLockSeconds: resolveWaitLockSeconds(options.waitLock),
-        }));
-    }));
-}
+    },
+});