@hominis/fireforge 0.45.0 → 0.46.1

package/dist/src/commands/export-shared.d.ts

@@ -2,29 +2,43 @@ import type { PatchesManifest, PatchLintIssue } from '../types/commands/index.js
 import type { ExportOptions, PatchCategory } from '../types/commands/index.js';
 import type { FireForgeConfig } from '../types/config.js';
 import type { SpinnerHandle } from '../utils/logger.js';
+export interface RunPatchLintOptions {
+    /** Engine root directory. */
+    engineDir: string;
+    /** Files touched by the patch. */
+    filesAffected: string[];
+    /** Raw unified diff string. */
+    diffContent: string;
+    /** Project configuration. */
+    config: FireForgeConfig;
+    /** If true, downgrade errors to warnings. */
+    skipLint?: boolean | undefined;
+    /** Optional cross-patch context for ownership resolution. */
+    patchQueueCtx?: import('../core/patch-lint-cross.js').PatchQueueContext | undefined;
+    /**
+     * Optional per-patch set of `check` IDs to suppress (threaded from
+     * `PatchMetadata.lintIgnore`). Surgical alternative to `--skip-lint` when
+     * exactly one advisory rule does not apply to a specific patch, e.g.
+     * `large-patch-lines` on a cohesive branding bundle that genuinely cannot
+     * be split.
+     */
+    ignoreChecks?: ReadonlySet<string> | undefined;
+    /**
+     * Optional explicit tier override (threaded from `PatchMetadata.tier`).
+     * Forces the branding-tier thresholds when set, independent of the
+     * auto-detect allowlist. When the branding tier is applied, a single
+     * `info()` line surfaces the choice so the tier decision is visible rather
+     * than silent.
+     */
+    patchTier?: 'branding' | undefined;
+}
 /**
  * Runs the full patch lint pipeline and reports results. Warnings are always
  * displayed. Errors block the export unless skipLint is true.
  *
- * @param engineDir - Engine root directory
- * @param filesAffected - Files touched by the patch
- * @param diffContent - Raw unified diff string
- * @param config - Project configuration
- * @param skipLint - If true, downgrade errors to warnings
- * @param patchQueueCtx - Optional cross-patch context for ownership
- *   resolution
- * @param ignoreChecks - Optional per-patch set of `check` IDs to suppress
- *   (threaded from `PatchMetadata.lintIgnore`). Surgical alternative to
- *   `--skip-lint` when exactly one advisory rule does not apply to a
- *   specific patch — e.g. `large-patch-lines` on a cohesive branding bundle
- *   that genuinely cannot be split.
- * @param patchTier - Optional explicit tier override (threaded from
- *   `PatchMetadata.tier`). Forces the branding-tier thresholds when set,
- *   independent of the auto-detect allowlist. When the branding tier is
- *   applied, a single `info()` line surfaces the choice so the tier decision
- *   is visible rather than silent.
+ * @param options - See {@link RunPatchLintOptions}
  */
-export declare function runPatchLint(engineDir: string, filesAffected: string[], diffContent: string, config: FireForgeConfig, skipLint?: boolean, patchQueueCtx?: import('../core/patch-lint-cross.js').PatchQueueContext, ignoreChecks?: ReadonlySet<string>, patchTier?: 'branding'): Promise<void>;
+export declare function runPatchLint(options: RunPatchLintOptions): Promise<void>;
 /**
  * Prints a patch-lint issue list with `runPatchLint`'s exact severity
  * buckets and skip-lint semantics, throwing on errors unless `skipLint`.
@@ -32,17 +46,22 @@ export declare function runPatchLint(engineDir: string, filesAffected: string[],
  * stored issues through the identical reporter without re-linting.
  */
 export declare function reportPatchLintOutcome(issues: PatchLintIssue[], skipLint?: boolean): void;
+/** Patch identity resolved for an export, from flags or interactive prompts. */
+export interface ExportPatchMetadata {
+    /** Normalized patch name (without order prefix or extension). */
+    patchName: string;
+    /** Category the patch is filed under. */
+    selectedCategory: PatchCategory;
+    /** Human-readable patch description. */
+    description: string;
+}
 /**
  * Resolves patch metadata interactively or from flags, with shared validation.
  * @param options - Export command options
  * @param isInteractive - Whether interactive prompts are allowed
  * @param commandName - Command name for error/help text
  */
-export declare function promptExportPatchMetadata(options: ExportOptions, isInteractive: boolean, commandName: 'export' | 'export-all', config?: FireForgeConfig): Promise<{
-    patchName: string;
-    selectedCategory: PatchCategory;
-    description: string;
-} | null>;
+export declare function promptExportPatchMetadata(options: ExportOptions, isInteractive: boolean, commandName: 'export' | 'export-all', config?: FireForgeConfig): Promise<ExportPatchMetadata | null>;
 /**
  * Confirms whether an export may supersede existing patches.
  * @param patchesDir - Patches directory
@@ -56,14 +75,14 @@ export declare function confirmSupersedePatches(patchesDir: string, filesAffecte
  * Detects new files missing license headers and offers to add them.
  *
  * In interactive mode the user is prompted before any files are modified. In
- * non-interactive mode the function is a no-op — the existing lint error
+ * non-interactive mode the function is a no-op. The existing lint error
  * blocks the export instead.
  *
  * @param engineDir - Absolute path to engine directory
  * @param diffContent - Current unified diff
  * @param config - Project configuration
  * @param isInteractive - Whether interactive prompts are available
- * @param dryRun - When true, only REPORT missing headers, never prompt or
+ * @param dryRun - When true, only report missing headers, never prompt or
  *   write. Dry-run must stay read-only: without this flag an interactive
  *   `export --dry-run` prompts (default Yes), writes license headers into
  *   engine/ files, and then closes with "no changes made".
@@ -77,23 +96,23 @@ export declare function autoFixLicenseHeaders(engineDir: string, diffContent: st
  *
  * Returns an empty map when no overlap exists. Used by the overlap gate in
  * `export` and `export-all` to refuse a default-mode export that would
- * silently create cross-patch ownership conflicts — the same class of
+ * silently create cross-patch ownership conflicts, the same class of
  * conflict `verify` immediately fails with.
  */
 export declare function findPartialOwnershipOverlap(manifest: PatchesManifest, filesAffected: string[], excludeFilenames: ReadonlySet<string>): Map<string, string[]>;
 /**
  * Gate that refuses the default export path when the new patch would
  * silently claim files already tracked by other non-superseded patches.
- * `findAllPatchesForFiles` catches the full-coverage supersede case; this
- * fills the gap for PARTIAL overlap — two patches both claiming e.g.
+ * `findAllPatchesForFiles` catches the full-coverage supersede case. This
+ * fills the gap for partial overlap: two patches both claiming e.g.
  * `browser/themes/shared/jar.inc.mn` after a second export with `--before`.
  *
  * Proceeds silently when there is no overlap, or when the caller passed
  * `--allow-overlap`. In interactive mode the caller is prompted to
  * acknowledge the overlap (the proper fix path is `re-export --files` to
  * repartition ownership, so the prompt surfaces that pointer). In
- * non-interactive mode it throws — better to fail fast than let the queue
- * fall out of sync with verify.
+ * non-interactive mode it throws, since failing fast beats letting the
+ * queue fall out of sync with verify.
  */
 export declare function guardOwnershipOverlap(args: {
     patchesDir: string;
@@ -108,7 +127,7 @@ export declare function guardOwnershipOverlap(args: {
  * order: the supersede confirmation, then the cross-patch ownership overlap
  * guard. The overlap guard receives the filenames of the patches the export
  * would fully supersede so it does not flag a file claimed by a patch that
- * is about to be removed. Catching only FULL-coverage supersedes leaves a
+ * is about to be removed. Catching only full-coverage supersedes leaves a
  * second export targeting a shared file creating a queue where two patches
  * both claim it and `verify` fails immediately.
  *
@@ -119,7 +138,7 @@ export declare function guardOwnershipOverlap(args: {
  * @param args.allowOverlap - The command's `--allow-overlap` flag
  * @param args.isInteractive - Whether prompting the operator is possible
  * @param args.s - Active spinner, stopped before any prompt
- * @returns `true` when both gates passed; `false` when the operator declined
+ * @returns `true` when both gates passed, `false` when the operator declined
  *   (the caller returns without committing)
  */
 export declare function runSupersedeAndOverlapGates(args: {

package/dist/src/commands/export-shared.js

@@ -15,27 +15,12 @@ import { describePatchNameProblem, normalizePatchDisplayName, PATCH_CATEGORIES,
  * Runs the full patch lint pipeline and reports results. Warnings are always
  * displayed. Errors block the export unless skipLint is true.
  *
- * @param engineDir - Engine root directory
- * @param filesAffected - Files touched by the patch
- * @param diffContent - Raw unified diff string
- * @param config - Project configuration
- * @param skipLint - If true, downgrade errors to warnings
- * @param patchQueueCtx - Optional cross-patch context for ownership
- *   resolution
- * @param ignoreChecks - Optional per-patch set of `check` IDs to suppress
- *   (threaded from `PatchMetadata.lintIgnore`). Surgical alternative to
- *   `--skip-lint` when exactly one advisory rule does not apply to a
- *   specific patch — e.g. `large-patch-lines` on a cohesive branding bundle
- *   that genuinely cannot be split.
- * @param patchTier - Optional explicit tier override (threaded from
- *   `PatchMetadata.tier`). Forces the branding-tier thresholds when set,
- *   independent of the auto-detect allowlist. When the branding tier is
- *   applied, a single `info()` line surfaces the choice so the tier decision
- *   is visible rather than silent.
+ * @param options - See {@link RunPatchLintOptions}
  */
-export async function runPatchLint(engineDir, filesAffected, diffContent, config, skipLint, patchQueueCtx, ignoreChecks, patchTier) {
+export async function runPatchLint(options) {
+    const { engineDir, filesAffected, diffContent, config, skipLint, patchQueueCtx, ignoreChecks, patchTier, } = options;
     // Compute the tier decision independently of the lint pipeline so it can
-    // be surfaced even when the rule body emitted no issues — a branding patch
+    // be surfaced even when the rule body emitted no issues. A branding patch
     // under the soft threshold still benefits from operators knowing which
     // tier governed the run. The same helper is reused inside `lintPatchSize`,
     // so the surfaced tier and the tier that actually drove the thresholds
@@ -115,8 +100,8 @@ export function reportPatchLintOutcome(issues, skipLint) {
  */
 export async function promptExportPatchMetadata(options, isInteractive, commandName, config) {
     const categories = config !== undefined ? getPatchPolicyCategories(config) : [...PATCH_CATEGORIES];
-    // A filename-shaped `--name foo.patch` must not die on the dot rule —
-    // strip the extension before validation so the one-step flow works for
+    // A filename-shaped `--name foo.patch` must not die on the dot rule, so
+    // strip the extension before validation and the one-step flow works for
     // both name shapes.
     let patchName = options.name?.trim().replace(/\.patch$/i, '');
     if (patchName) {
@@ -173,7 +158,7 @@ export async function promptExportPatchMetadata(options, isInteractive, commandN
             description = descResult;
         }
     }
-    // The filename slug pipeline strips redundant category prefixes; the
+    // The filename slug pipeline strips redundant category prefixes. The
     // manifest display name must agree, or every `export --name ui-foo`
     // needs a follow-up `patch rename` to satisfy bare-slug naming policy.
     const normalized = normalizePatchDisplayName(patchName, category);
@@ -223,14 +208,14 @@ export async function confirmSupersedePatches(patchesDir, filesAffected, superse
  * Detects new files missing license headers and offers to add them.
  *
  * In interactive mode the user is prompted before any files are modified. In
- * non-interactive mode the function is a no-op — the existing lint error
+ * non-interactive mode the function is a no-op. The existing lint error
  * blocks the export instead.
  *
  * @param engineDir - Absolute path to engine directory
  * @param diffContent - Current unified diff
  * @param config - Project configuration
  * @param isInteractive - Whether interactive prompts are available
- * @param dryRun - When true, only REPORT missing headers, never prompt or
+ * @param dryRun - When true, only report missing headers, never prompt or
  *   write. Dry-run must stay read-only: without this flag an interactive
  *   `export --dry-run` prompts (default Yes), writes license headers into
  *   engine/ files, and then closes with "no changes made".
@@ -308,7 +293,7 @@ export async function autoFixLicenseHeaders(engineDir, diffContent, config, isIn
  *
  * Returns an empty map when no overlap exists. Used by the overlap gate in
  * `export` and `export-all` to refuse a default-mode export that would
- * silently create cross-patch ownership conflicts — the same class of
+ * silently create cross-patch ownership conflicts, the same class of
  * conflict `verify` immediately fails with.
  */
 export function findPartialOwnershipOverlap(manifest, filesAffected, excludeFilenames) {
@@ -330,16 +315,16 @@ export function findPartialOwnershipOverlap(manifest, filesAffected, excludeFile
 /**
  * Gate that refuses the default export path when the new patch would
  * silently claim files already tracked by other non-superseded patches.
- * `findAllPatchesForFiles` catches the full-coverage supersede case; this
- * fills the gap for PARTIAL overlap — two patches both claiming e.g.
+ * `findAllPatchesForFiles` catches the full-coverage supersede case. This
+ * fills the gap for partial overlap: two patches both claiming e.g.
  * `browser/themes/shared/jar.inc.mn` after a second export with `--before`.
  *
  * Proceeds silently when there is no overlap, or when the caller passed
  * `--allow-overlap`. In interactive mode the caller is prompted to
  * acknowledge the overlap (the proper fix path is `re-export --files` to
  * repartition ownership, so the prompt surfaces that pointer). In
- * non-interactive mode it throws — better to fail fast than let the queue
- * fall out of sync with verify.
+ * non-interactive mode it throws, since failing fast beats letting the
+ * queue fall out of sync with verify.
  */
 export async function guardOwnershipOverlap(args) {
     const { patchesDir, filesAffected, supersedingFilenames, allowOverlap, isInteractive, s } = args;
@@ -353,7 +338,7 @@ export async function guardOwnershipOverlap(args) {
         return true;
     s.stop();
     const entries = [...overlap.entries()].sort(([a], [b]) => a.localeCompare(b));
-    warn(`This export would create cross-patch ownership overlap on ${String(entries.length)} file${entries.length === 1 ? '' : 's'}:`);
+    warn(`This export would create cross-patch ownership overlap on ${entries.length} file${entries.length === 1 ? '' : 's'}:`);
     for (const [file, owners] of entries) {
         warn(`  - ${file} already claimed by: ${owners.join(', ')}`);
     }
@@ -379,7 +364,7 @@ export async function guardOwnershipOverlap(args) {
  * order: the supersede confirmation, then the cross-patch ownership overlap
  * guard. The overlap guard receives the filenames of the patches the export
  * would fully supersede so it does not flag a file claimed by a patch that
- * is about to be removed. Catching only FULL-coverage supersedes leaves a
+ * is about to be removed. Catching only full-coverage supersedes leaves a
  * second export targeting a shared file creating a queue where two patches
  * both claim it and `verify` fails immediately.
  *
@@ -390,7 +375,7 @@ export async function guardOwnershipOverlap(args) {
  * @param args.allowOverlap - The command's `--allow-overlap` flag
  * @param args.isInteractive - Whether prompting the operator is possible
  * @param args.s - Active spinner, stopped before any prompt
- * @returns `true` when both gates passed; `false` when the operator declined
+ * @returns `true` when both gates passed, `false` when the operator declined
  *   (the caller returns without committing)
  */
 export async function runSupersedeAndOverlapGates(args) {

package/dist/src/commands/export.js

@@ -88,11 +88,11 @@ async function collectExportFiles(paths, files) {
     return { files: [...collectedFiles].sort(), fromDirectory };
 }
 /**
- * Auto-excludes directory-derived files already owned by OTHER patches.
+ * Auto-excludes directory-derived files already owned by other patches.
  *
  * A directory export otherwise plans files owned by earlier patches into the
  * new patch, and the duplicate-new-file-creation refusal surfaces only at
- * placement lint, suggesting `--force-unsafe` — the wrong tool for "leave
+ * placement lint, suggesting `--force-unsafe`, the wrong tool for "leave
  * that file with its owner". Explicitly named files are never excluded (the
  * overlap gates still confront the operator with those). Prints one notice
  * per exclusion.
@@ -217,7 +217,7 @@ async function prepareExport(projectRoot, files, options) {
     }
     const config = await loadConfig(projectRoot);
     const isInteractive = stdioIsInteractive();
-    // Auto-fix missing license headers on new files (interactive only;
+    // Auto-fix missing license headers on new files (interactive only, and
     // report-only under --dry-run so the preview never mutates engine/)
     const headersAdded = await autoFixLicenseHeaders(paths.engine, diff, config, isInteractive, options.dryRun ?? false);
     if (headersAdded) {
@@ -258,9 +258,18 @@ export async function exportCommand(projectRoot, files, options) {
         const patchQueueCtx = (await pathExists(paths.patches))
             ? await buildPatchQueueContext(paths.patches)
             : undefined;
-        await runPatchLint(paths.engine, filesAffected, diff, config, options.skipLint, patchQueueCtx, exportIgnoreChecks, options.tier);
+        await runPatchLint({
+            engineDir: paths.engine,
+            filesAffected,
+            diffContent: diff,
+            config,
+            skipLint: options.skipLint,
+            patchQueueCtx,
+            ignoreChecks: exportIgnoreChecks,
+            patchTier: options.tier,
+        });
         // Resolve placement (if any flag was given). Placement is mutually
-        // exclusive with supersede — the semantics overlap confusingly.
+        // exclusive with supersede, because the semantics overlap confusingly.
         let placementPlan = null;
         if (placementFlagCount > 0) {
             const gated = await gatePlacementPlan({

package/dist/src/commands/furnace/apply.js

@@ -29,7 +29,7 @@ async function snapshotWatchedChecksums(watchDirs) {
             }
         }
         catch {
-            // Directory may have been removed between iterations — ignore.
+            // Directory may have been removed between iterations, so ignore.
         }
     }
     return combined;
@@ -46,7 +46,7 @@ function checksumMapsEqual(a, b) {
 /**
  * Builds a watch-loop apply-failure message tailored to the error class so
  * transient filesystem errors (EACCES, ENOSPC, lock timeout) look different
- * from genuine apply-level failures; the previous generic "Apply failed: ..."
+ * from genuine apply-level failures. The previous generic "Apply failed: ..."
  * collapsed all causes into one string and made diagnosis difficult.
  */
 function classifyWatchApplyError(err) {
@@ -74,8 +74,8 @@ function classifyWatchApplyError(err) {
 }
 async function runWatchLoop(projectRoot) {
     const furnacePaths = getFurnacePaths(projectRoot);
-    // Both categories are eligible targets. The set is fixed; only existence
-    // varies over time — a component dir created AFTER watch started (the user
+    // Both categories are eligible targets. The set is fixed. Only existence
+    // varies over time: a component dir created after watch started (the user
     // running `furnace create` in another terminal) must be picked up without
     // restarting watch. A one-shot `pathExists` check at startup captures only
     // the dirs that existed then, leaving any later creation invisible.
@@ -115,8 +115,8 @@ async function runWatchLoop(projectRoot) {
             // for changes that are already reflected in the engine.
             lastChecksums = await snapshotWatchedChecksums(watchDirs);
         }
-        // Another change arrived while we were applying — run again so the edit
-        // is not silently absorbed into the post-apply checksum bump.
+        // Another change arrived while we were applying, so run again and the
+        // edit is not silently absorbed into the post-apply checksum bump.
         if (pendingChange) {
             pendingChange = false;
             await runApplyCycle();
@@ -124,7 +124,7 @@ async function runWatchLoop(projectRoot) {
     };
     const triggerApply = () => {
         if (applyInFlight) {
-            // An apply is already running; record the change so runApplyCycle
+            // An apply is already running. Record the change so runApplyCycle
             // re-runs after it completes. Do not schedule a new debounce: the
             // in-flight apply will observe this flag when it finishes.
             pendingChange = true;
@@ -136,7 +136,7 @@ async function runWatchLoop(projectRoot) {
             debounceTimer = null;
             if (applyInFlight) {
                 // Race with a change that started its own apply between the debounce
-                // scheduling and the timer firing. Record the pending change; the
+                // scheduling and the timer firing. Record the pending change. The
                 // in-flight apply will pick it up.
                 pendingChange = true;
                 return;
@@ -188,7 +188,7 @@ async function runWatchLoop(projectRoot) {
         }
         return added;
     };
-    // Register signal-driven cleanup BEFORE creating watchers so there is no
+    // Register signal-driven cleanup before creating watchers so there is no
     // race window where a SIGINT could arrive after watchers exist but before
     // cleanup handlers are registered.
     const cleanup = () => {
@@ -229,7 +229,7 @@ async function runWatchLoop(projectRoot) {
                 }
             }
             catch {
-                // Best effort — errors here are transient filesystem issues.
+                // Best effort: errors here are transient filesystem issues.
             }
         })();
     }, WATCH_POLL_INTERVAL_MS);
@@ -277,14 +277,14 @@ export async function furnaceApplyCommand(projectRoot, name, options = {}) {
     }
     const applySpinner = dryRun ? undefined : spinner('Applying components to engine...');
     const result = await runFurnaceMutation(projectRoot, 'apply-rollback', async (ctx) => {
-        // `persistState: false` for a NAMED apply is load-bearing: the batch
-        // persist path replaces `appliedChecksums` wholesale with only this
-        // run's entries, and the batch loops filter to the named component —
-        // so routing a named apply through it persists a state file containing
-        // ONLY that component, wiping every other component's checksums.
+        // `persistState: false` for a named apply matters: the batch persist
+        // path replaces `appliedChecksums` wholesale with only this run's
+        // entries, and the batch loops filter to the named component, so
+        // routing a named apply through it persists a state file containing
+        // only that component and wipes every other component's checksums.
         // Orphan detection and deleted-file undeploy both key on that state,
         // so the wiped components' stale engine files become invisible to
-        // apply AND to `furnace validate`. Named apply merges per-component
+        // apply and to `furnace validate`. Named apply merges per-component
         // state below, exactly like `furnace deploy <name>`.
         const applyResult = await applyAllComponents(projectRoot, dryRun, {
             operationContext: ctx,

package/dist/src/commands/furnace/authoring-config.d.ts

@@ -0,0 +1,13 @@
+import type { FurnaceConfig } from '../../types/index.js';
+/**
+ * Loads `furnace.json` for an authoring command, defaulting to an empty
+ * configuration when the project has none yet.
+ *
+ * `furnace create` and `furnace override` both need this: they are the two
+ * commands that may run before any furnace configuration exists, and both must
+ * treat "no file" as "nothing registered yet" rather than an error. They had a
+ * byte-identical private copy each.
+ * @param projectRoot - Project root to load from.
+ * @returns The parsed configuration, or a fresh default.
+ */
+export declare function loadAuthoringFurnaceConfig(projectRoot: string): Promise<FurnaceConfig>;

package/dist/src/commands/furnace/authoring-config.js

@@ -0,0 +1,19 @@
+// SPDX-License-Identifier: EUPL-1.2
+import { createDefaultFurnaceConfig, furnaceConfigExists, loadFurnaceConfig, } from '../../core/furnace-config.js';
+/**
+ * Loads `furnace.json` for an authoring command, defaulting to an empty
+ * configuration when the project has none yet.
+ *
+ * `furnace create` and `furnace override` both need this: they are the two
+ * commands that may run before any furnace configuration exists, and both must
+ * treat "no file" as "nothing registered yet" rather than an error. They had a
+ * byte-identical private copy each.
+ * @param projectRoot - Project root to load from.
+ * @returns The parsed configuration, or a fresh default.
+ */
+export async function loadAuthoringFurnaceConfig(projectRoot) {
+    if (await furnaceConfigExists(projectRoot)) {
+        return loadFurnaceConfig(projectRoot);
+    }
+    return createDefaultFurnaceConfig();
+}

package/dist/src/commands/furnace/chrome-doc-remove.d.ts

@@ -1,5 +1,5 @@
 /**
- * `fireforge furnace chrome-doc remove <name>` — removes the files and
+ * `fireforge furnace chrome-doc remove <name>` removes the files and
  * registrations created by `furnace chrome-doc create`.
  */
 /** Options for `furnace chrome-doc remove`. */

package/dist/src/commands/furnace/chrome-doc-remove.js

@@ -1,6 +1,6 @@
 // SPDX-License-Identifier: EUPL-1.2
 /**
- * `fireforge furnace chrome-doc remove <name>` — removes the files and
+ * `fireforge furnace chrome-doc remove <name>` removes the files and
  * registrations created by `furnace chrome-doc create`.
  */
 import { readdir } from 'node:fs/promises';
@@ -10,7 +10,7 @@ import { loadConfig } from '../../core/config.js';
 import { stdioIsInteractive } from '../../core/destructive.js';
 import { completeJournalRollback, runFurnaceMutation, } from '../../core/furnace-operation.js';
 import { assertFurnaceEngineReady } from '../../core/furnace-precondition.js';
-import { createRollbackJournal, snapshotDir, snapshotFile } from '../../core/furnace-rollback.js';
+import { createRollbackJournal, snapshotDir, snapshotFile, } from '../../core/furnace-rollback.js';
 import { FurnaceError } from '../../errors/furnace.js';
 import { pathExists, readText, removeDir, removeFile, writeText } from '../../utils/fs.js';
 import { cancel, info, intro, isCancel, note, outro } from '../../utils/logger.js';

package/dist/src/commands/furnace/chrome-doc-templates.d.ts

@@ -41,21 +41,21 @@
  */
 export declare function generateChromeDocXhtml(name: string, withTitlebar: boolean, license: string): string;
 /**
- * browser.xhtml-like scaffold for the document that ships as the fork's MAIN
- * BROWSER WINDOW.
+ * browser.xhtml-like scaffold for the document that ships as the fork's main
+ * browser window.
  *
  * The generic dialog-shaped `<window>` scaffold is wrong for the
  * `tokenHostDocuments[0]` / BROWSER_CHROME_URL target: platform C++ reads
- * the root element BEFORE any script runs, and expects the `browser.xhtml`
+ * the root element before any script runs, and expects the `browser.xhtml`
  * shape:
  *
- * - `<html id="main-window">` root (not `<window id="<name>-window">`) —
+ * - `<html id="main-window">` root (not `<window id="<name>-window">`):
  *   upstream code from nsXULWindow sizing to session restore looks up
- *   `main-window` by id;
+ *   `main-window` by id.
  * - `windowtype="navigator:browser"`, `chromehidden=""` and the geometry
- *   `persist` allowlist declared as ROOT ATTRIBUTES so the platform's
+ *   `persist` allowlist declared as root attributes so the platform's
  *   pre-script pass (window tracking, XULStore geometry, chrome flags) sees
- *   them;
+ *   them.
  * - the same head/bootstrap wiring (customElements.js, per-doc subscript,
  *   CSS + Fluent links) and `data-furnace-chrome-doc` sentinel as the
  *   generic scaffold, so jar.mn registration and the platform-module guard
@@ -89,7 +89,7 @@ export declare function generateChromeDocJs(name: string, licenseHeader: string)
  * the platform window controls that `global.css` applies by default.
  */
 export declare function generateChromeDocCss(name: string, withTitlebar: boolean, licenseHeader: string): string;
-/** Fluent stub — one placeholder message keyed to the window title. */
+/** Fluent stub with one placeholder message keyed to the window title. */
 export declare function generateChromeDocFtl(name: string, licenseHeader: string): string;
 /**
  * Single-line jar.mn entry that registers an xhtml + js pair under
@@ -110,7 +110,7 @@ export declare function jarMnEntriesForChromeDoc(name: string): string[];
  * The source path is `../shared/<name>-chrome.css` because `jar.inc.mn` is
  * included from each theme-specific manifest (`browser/themes/osx/jar.mn`,
  * `.../linux/`, `.../windows/`), and every existing entry in those manifests
- * resolves paths relative to the INCLUDING manifest's directory. A bare
+ * resolves paths relative to the including manifest's directory. A bare
  * `(shared/…)` path resolves to
  * `obj-.../browser/themes/osx/shared/<name>-chrome.css`, which does not
  * exist.
@@ -120,7 +120,7 @@ export declare function jarIncMnEntryForChromeDoc(name: string): string;
  * locales/jar.mn entry that registers the `.ftl` under the browser locale
  * bundle. The source path is resolved by mach-locale-jar relative to the
  * per-locale root (e.g. `engine/browser/locales/en-US/`), and the FTL file
- * is scaffolded at `browser/${name}.ftl` under that root — the `%` prefix
+ * is scaffolded at `browser/${name}.ftl` under that root. The `%` prefix
  * means "per-locale content" and the `browser/` subdirectory matches where
  * the scaffolder writes. A bare `(%${name}.ftl)` points at
  * `en-US/${name}.ftl` and breaks the first post-scaffold build with
@@ -132,11 +132,11 @@ export declare function localeJarMnEntryForChromeDoc(name: string): string;
  * wildcard rooted at `%browser/` whose pattern would already pick up a
  * scaffolded `browser/<name>.ftl` file. Recognises:
  *
- *   - `(%browser/**\/*.ftl)` — recursive (the upstream shape).
- *   - `(%browser/*.ftl)` — flat.
+ *   - `(%browser/**\/*.ftl)`: recursive (the upstream shape).
+ *   - `(%browser/*.ftl)`: flat.
  *
  * Forks that have migrated entirely to `[localization]` wildcards typically
- * keep no per-file `locale/...` entries for FTL at all; appending one
+ * keep no per-file `locale/...` entries for FTL at all. Appending one
  * there is dead weight at best, and an outright build break when the fork
  * has also dropped the `% locale browser …` registration. The chrome-doc
  * scaffolder consults this predicate before its locales/jar.mn append and
@@ -146,7 +146,7 @@ export declare function localeJarMnEntryForChromeDoc(name: string): string;
  * Conservative by design: only wildcards rooted at `%browser/` count, and
  * a `(%browser/foo.ftl)`-style explicit reference (no `*`) is not treated
  * as a capture. A fork with a narrower wildcard (e.g. `(%browser/about/*.ftl)`)
- * is correctly NOT captured by this predicate, because that wildcard would
+ * is correctly not captured by this predicate, because that wildcard would
  * not pick up the top-level `browser/<name>.ftl` the scaffold writes.
  */
 export declare function localesFtlWildcardCapturesScaffoldedName(jarMnContents: string): boolean;