@hominis/fireforge 0.46.0 → 0.46.1

package/dist/src/commands/doctor-working-tree.js

@@ -4,11 +4,11 @@
  *
  * Partitions engine-tree dirtiness into `branding`, `patch-backed`,
  * `furnace`, `conflict`, and `unmanaged` buckets, and only warns on the
- * last two — everything else is tool-managed state that the operator
+ * last two. Everything else is tool-managed state that the operator
  * did not author directly.
  *
  * Split out of `doctor.ts` so that file stays under the per-file LOC
- * budget; see the call site in `runEngineGitChecks`.
+ * budget. See the call site in `runEngineGitChecks`.
  */
 import { collectFurnaceManagedPrefixes } from '../core/furnace-config.js';
 import { expandUntrackedDirectoryEntries, getWorkingTreeStatus } from '../core/git-status.js';
@@ -30,7 +30,7 @@ function formatManagedDetail(counts) {
 /**
  * Inspects the engine working tree and returns a single `DoctorCheck`.
  * Ownership-aware: patch-backed / branding / furnace rows are reported as OK
- * with an ownership summary; unmanaged drift warns; cross-patch conflicts
+ * with an ownership summary. Unmanaged drift warns. Cross-patch conflicts
  * warn loudly with a pointer at `fireforge status --ownership` +
  * `fireforge verify`.
  *

package/dist/src/commands/doctor.d.ts

@@ -10,7 +10,7 @@ import type { DoctorCheckDefinition } from './doctor-check-core.js';
  * context-population bug at runtime.
  *
  * Exported so tests can exercise the forward-only invariant against
- * fixtures — the real DOCTOR_CHECKS list is also validated at import
+ * fixtures. The real DOCTOR_CHECKS list is also validated at import
  * time, but a targeted unit test makes the contract explicit and
  * prevents regressions if the validator is ever relaxed.
  */
@@ -22,7 +22,7 @@ export declare function validateCheckDependencies(checks: readonly DoctorCheckDe
  * also update this list, which gives us a single place to notice and
  * think through the consequences.
  *
- * @internal Exported only so tests can reach it; not part of the public surface.
+ * @internal Exported only so tests can reach it. Not part of the public surface.
  */
 export declare const DOCTOR_CHECK_ORDER: readonly string[];
 /**

package/dist/src/commands/doctor.js

@@ -90,8 +90,8 @@ async function runEngineGitChecks(ctx) {
         branch === undefined &&
         currentHead === undefined &&
         !state.baseCommit) {
-        // Unborn repository with no recorded baseline — the earlier failure row
-        // explains recovery; avoid adding a second near-identical row.
+        // Unborn repository with no recorded baseline. The earlier failure row
+        // explains recovery. Avoid adding a second near-identical row.
     }
     else if (!canValidateBranch) {
         rows.push(warning('Engine branch', 'Skipped branch validation because the baseline commit is missing.', 'Finish recreating the engine baseline with "fireforge download --force".'));
@@ -114,7 +114,7 @@ async function runEngineGitChecks(ctx) {
  * context-population bug at runtime.
  *
  * Exported so tests can exercise the forward-only invariant against
- * fixtures — the real DOCTOR_CHECKS list is also validated at import
+ * fixtures. The real DOCTOR_CHECKS list is also validated at import
  * time, but a targeted unit test makes the contract explicit and
  * prevents regressions if the validator is ever relaxed.
  */
@@ -136,7 +136,7 @@ export function validateCheckDependencies(checks) {
 /**
  * The declarative doctor check registry. The order of entries here is the
  * order checks appear in the report. Adding a new check is a one-entry
- * edit; each check only contains its own inspection logic.
+ * edit. Each check only contains its own inspection logic.
  *
  * ## Ordering dependency chain
  *
@@ -214,7 +214,7 @@ const DOCTOR_CHECKS = [
         name: 'Engine is git repository',
         skipIf: (ctx) => !ctx.engineExists,
         // runEngineGitChecks consults ctx.config for ownership-aware
-        // working-tree classification; declare the dependency so a future
+        // working-tree classification. Declare the dependency so a future
         // reorder doesn't silently regress the doctor back to the
         // count-only fallback.
         dependsOn: ['fireforge.json is valid'],
@@ -248,7 +248,7 @@ const DOCTOR_CHECKS = [
             // Resolve the absolute path so the OK row names what doctor actually
             // found. A PATH-export discrepancy between the operator's interactive
             // shell and the spawned subprocess otherwise reads as doctor printing
-            // "OK" for a binary `which watchman` cannot find; surfacing the
+            // "OK" for a binary `which watchman` cannot find. Surfacing the
             // resolved path makes the discrepancy visible without a verbose flag.
             const path = await findExecutable('watchman');
             if (path) {
@@ -295,7 +295,7 @@ const DOCTOR_CHECKS = [
     POST_REBASE_AUDIT_CHECK,
     // Furnace checks live in a sibling module so this file stays under the
     // max-lines threshold. Splicing them in as an array preserves the
-    // declarative registry contract — each entry remains a single
+    // declarative registry contract: each entry remains a single
     // `DoctorCheckDefinition` with its own skipIf/run/fix, and the order
     // here is the order they appear in the report.
     ...FURNACE_DOCTOR_CHECKS,
@@ -320,7 +320,7 @@ validateCheckDependencies(DOCTOR_CHECKS);
  * also update this list, which gives us a single place to notice and
  * think through the consequences.
  *
- * @internal Exported only so tests can reach it; not part of the public surface.
+ * @internal Exported only so tests can reach it. Not part of the public surface.
  */
 export const DOCTOR_CHECK_ORDER = DOCTOR_CHECKS.map((check) => check.name);
 /**
@@ -382,7 +382,7 @@ export function reportDoctorResults(checks, mutations = []) {
  * Rejects repair-flag combinations that cannot mean what they say.
  *
  * The two manifest repairs are different operations on the same file, and
- * `--dry-run` with nothing to project is a flag that silently does nothing —
+ * `--dry-run` with nothing to project is a flag that silently does nothing,
  * the shape most likely to be mistaken for a preview that reported "no
  * changes".
  */

package/dist/src/commands/download.js

@@ -78,7 +78,7 @@ async function cleanPatchTouchedFiles(engineDir, patchesDir, preExistingDirty) {
  * Prints a one-line nudge pointing at `fireforge import` when the project
  * carries a non-empty patch queue but the just-downloaded engine has not yet
  * had any patches applied. The post-download spinner closes with
- * "Patch-touched files already match baseline" because a fresh tree IS at
+ * "Patch-touched files already match baseline" because a fresh tree is at
  * baseline, which reads as "patches are restored" and invites skipping the
  * import step. Suppressed when patches/ is missing or the manifest is empty
  * so unconfigured projects stay quiet.
@@ -94,10 +94,11 @@ async function noteUnappliedPatches(patchesDir) {
 }
 /**
  * Stops `restoreSpinner` with a message that reflects what actually
- * happened. Three branches: empty queue → explicit no-op; queue present but
- * nothing dirty → "already clean"; queue with dirty files → the usual
- * "Patch-touched files restored" success line. Always closing with the third
- * claims restore work that did not happen on a project with zero patches.
+ * happened. Three branches. An empty queue gives an explicit no-op. A queue
+ * that is present but has nothing dirty gives "already clean". A queue with
+ * dirty files gives the usual "Patch-touched files restored" success line.
+ * Always closing with the third claims restore work that did not happen on a
+ * project with zero patches.
  */
 function closeRestoreSpinner(restoreSpinner, result) {
     if (!result.hadQueue) {
@@ -194,7 +195,7 @@ async function downloadAndExtractFirefox(args) {
 }
 /**
  * Prints the major-version-hop toolchain nudge when this download moved the
- * engine across a Firefox MAJOR version — the first post-hop build otherwise
+ * engine across a Firefox major version. The first post-hop build otherwise
  * dies in `mach configure` on a moved cbindgen minimum with nothing in the
  * download output suggesting `fireforge bootstrap`. Quiet on first downloads
  * and same-major re-downloads.
@@ -276,7 +277,7 @@ export async function downloadCommand(projectRoot, options) {
     intro('FireForge Download');
     const config = await loadConfig(projectRoot), version = config.firefox.version;
     const paths = getProjectPaths(projectRoot);
-    // Captured BEFORE any state update so the post-download major-hop
+    // Captured before any state update so the post-download major-hop
     // notice compares against what was actually on disk until now.
     const previousVersion = (await loadState(projectRoot)).downloadedVersion;
     info(`Firefox version: ${version}`);
@@ -284,7 +285,7 @@ export async function downloadCommand(projectRoot, options) {
     // A legitimate holder of this lock runs for 10+ minutes (download +
     // extract + git indexing). A 30 s timeout fails the second invocation with
     // "remove the lock directory if it is stale" advice while the lock is
-    // actively held — dangerous guidance mid-extraction. Wait generously
+    // actively held, which is dangerous guidance mid-extraction. Wait generously
     // instead: a dead holder is reaped within seconds by the PID-based stale
     // probe, so a long timeout only ever waits on real work.
     const downloadLockOptions = {
@@ -297,7 +298,7 @@ export async function downloadCommand(projectRoot, options) {
         // Reclaim multi-GB partial trees left behind by interrupted runs.
         // Safe under the download lock: any `.tmp-*`/`.replacement-*` dir
         // present now is orphaned, since live ones only exist while a
-        // download holds this lock. `.backup-*` dirs are NOT swept — a
+        // download holds this lock. `.backup-*` dirs are not swept: a
         // backup can hold the previous engine after a failed forced
         // replacement and is the operator's recovery copy.
         const sweptDirs = await sweepOrphanedEngineWorkDirs(paths.engine);
@@ -317,7 +318,8 @@ export async function downloadCommand(projectRoot, options) {
                     }
                     catch (error) {
                         if (isMissingHeadError(error)) {
-                            // Partial init detected — attempt to resume instead of requiring --force
+                            // Partial init detected. Attempt to resume instead of
+                            // requiring --force.
                             info('Detected partially initialized engine. Attempting to resume...');
                             // Snapshot patch-touched files that are already dirty so we
                             // can preserve them after the resume commit.
@@ -340,7 +342,7 @@ export async function downloadCommand(projectRoot, options) {
                                 });
                                 const baseCommit = await getHead(paths.engine);
                                 resumeSpinner.stop('Git repository resumed successfully');
-                                // Restore patch-touched files BEFORE stamping state. If this
+                                // Restore patch-touched files before stamping state. If this
                                 // step fails (disk full, permission denied, git object issue),
                                 // state.json keeps the previous downloadedVersion so the
                                 // invariant "state.downloadedVersion matches a clean engine"
@@ -359,7 +361,7 @@ export async function downloadCommand(projectRoot, options) {
                             }
                             catch (error) {
                                 resumeSpinner.error('Resume failed');
-                                // Preserve the underlying cause so the operator sees *why*
+                                // Preserve the underlying cause so the operator sees why
                                 // the resume failed (timeout, permission denied, corrupted
                                 // object, disk full) instead of only the generic "partial
                                 // engine exists" story. The cause reaches them through the
@@ -372,7 +374,7 @@ export async function downloadCommand(projectRoot, options) {
                         // Re-throw unexpected git errors (corrupted objects, permission
                         // denied, …) wrapped in PartialEngineExistsError so the user sees
                         // both narratives: "we detected a partial engine and attempted
-                        // resume" AND the underlying git failure. Without the wrap the
+                        // resume" and the underlying git failure. Without the wrap the
                         // raw git error loses the context that resume was in flight.
                         throw new PartialEngineExistsError(paths.engine, toError(error));
                     }

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

@@ -56,7 +56,7 @@ async function resolveFurnaceExclusionPolicy(paths, projectRoot, excludeFurnace)
     if (prefixes.size === 0)
         return new Set();
     // Expand collapsed `?? dir/` entries before matching against Furnace
-    // prefixes — otherwise a Furnace-introduced directory slips past the
+    // prefixes, otherwise a Furnace-introduced directory slips past the
     // filter and later lands in the non-Furnace path list that feeds the
     // aggregate diff, where `getDiffForFilesAgainstHead` crashes with EISDIR.
     const rawStatus = await getWorkingTreeStatus(paths.engine);
@@ -81,7 +81,7 @@ async function resolveFurnaceExclusionPolicy(paths, projectRoot, excludeFurnace)
  * `export-all --exclude-furnace` after `furnace create --localized
  * --with-tests` otherwise produces a patch that adds
  * `toolkit/content/widgets/<tag>/*` via jar.mn / customElements.js / locale
- * jar.mn while excluding the component source files themselves — a
+ * jar.mn while excluding the component source files themselves, a
  * structurally broken queue that `fireforge verify` stays silent about. The
  * operator is asked to either include the component sources (skip
  * `--exclude-furnace`) or revert the furnace changes before exporting.
@@ -134,7 +134,7 @@ async function checkDanglingFurnaceRegistrations(projectRoot, diff, furnaceExclu
  * path that some existing patch in the queue already creates. `verify`
  * detects this post-hoc via `collectNewFileCreatorsByPath`, but by the time
  * `verify` runs the operator has already landed a patch that irreversibly
- * sits atop the queue — resolving the conflict from there requires a
+ * sits atop the queue, and resolving the conflict from there requires a
  * `patch delete` or hand-surgery on `re-export --files`. Catching it here,
  * pre-write, keeps the queue clean and gives the operator a specific path
  * to narrow with `export` + explicit file scoping (which lets them drop
@@ -232,12 +232,12 @@ export async function exportAllCommand(projectRoot, options = {}) {
     await checkDuplicateNewFileCreations(paths, diff);
     // Dangling-furnace-registration preflight. Runs after the diff is
     // assembled so it can inspect the exact hunks the operator is about to
-    // land, and BEFORE any write so a refusal leaves the patches directory
+    // land, and before any write so a refusal leaves the patches directory
     // untouched.
     await checkDanglingFurnaceRegistrations(projectRoot, diff, furnaceExcluded);
     // Check for non-interactive mode
     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, isDryRun);
     if (headersAdded) {
@@ -251,7 +251,7 @@ export async function exportAllCommand(projectRoot, options = {}) {
         return;
     const { patchName, selectedCategory, description } = metadata;
     // Ensure patches directory exists. Skip during a dry-run so the command
-    // is purely read-only — `--dry-run` callers should be safe to invoke
+    // is purely read-only. `--dry-run` callers should be safe to invoke
     // against a project that has never exported a patch without leaving the
     // empty `patches/` directory behind.
     if (!isDryRun) {
@@ -275,8 +275,8 @@ export async function exportAllCommand(projectRoot, options = {}) {
         // Dry-run: enumerate filename, metadata, and supersede coverage without
         // writing. Mirrors `fireforge export --dry-run` so the same preview
         // surface is available for both targeted and aggregate exports. Runs
-        // AFTER lint so the operator sees the same lint output they would on
-        // a real run; runs BEFORE the supersede confirmation prompt because
+        // after lint so the operator sees the same lint output they would on
+        // a real run. It runs before the supersede confirmation prompt because
         // confirming a dry-run is meaningless.
         if (isDryRun) {
             s.stop('Plan ready');

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

@@ -20,7 +20,7 @@ export interface PlacementPlan {
     renameMap: Map<string, PatchRenameEntry>;
 }
 /**
- * Structural equality for placement plans — used by placement-mode export
+ * Structural equality for placement plans. Used by placement-mode export
  * and `patch split` to verify the queue did not change between the
  * confirmed preview and the under-lock commit.
  */
@@ -40,14 +40,14 @@ export declare function computeExactPlacementPlan(manifestPatches: PatchMetadata
 /**
  * Resolves a placement plan from CLI flags against the current manifest.
  * When `config` is provided, positional plans (`--before`/`--after`) are
- * refused up front if their renumber would touch a reserved range — one
- * error for the whole shift instead of per-patch policy findings. The
+ * refused up front if their renumber would touch a reserved range, giving
+ * one error for the whole shift instead of per-patch policy findings. The
  * exact `--order` branch never renames, so it bypasses that gate.
  */
 export declare function resolvePlacementPlan(patchesDir: string, options: ExportOptions, category: PatchCategory, name: string, config?: FireForgeConfig): Promise<PlacementPlan>;
 /**
  * Projects the placement through cross-patch lint to detect forward-imports
- * the renumber would introduce *or* that the new patch itself would
+ * the renumber would introduce, or that the new patch itself would
  * introduce by landing earlier than one of its dependencies. Returns null
  * when the projection is clean.
  */
@@ -79,7 +79,7 @@ export interface CommitPlacementExportInput {
      * Optional post-commit hook that runs inside the patch directory lock,
      * after the mutation has succeeded but before the lock is released.
      * Intended for the caller's history-log append so the audit record
-     * lands atomically with the mutation — a crash between mutation and
+     * lands atomically with the mutation. A crash between mutation and
      * hook leaves no room for another process's history record to sneak
      * in first.
      *

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

@@ -36,7 +36,7 @@ function prefixWidthForPatches(manifestPatches, requestedOrder) {
     }, 3);
 }
 /**
- * Structural equality for placement plans — used by placement-mode export
+ * Structural equality for placement plans. Used by placement-mode export
  * and `patch split` to verify the queue did not change between the
  * confirmed preview and the under-lock commit.
  */
@@ -62,7 +62,7 @@ export function computePlacementPlan(manifestPatches, newPatchCategory, newPatch
     const renameMap = new Map();
     const prefixWidth = prefixWidthForPatches(sorted, requestedOrder);
     // Patches at requestedOrder or later shift up by one, but only until the
-    // cascade reaches a free ordinal — a gap absorbs the insert, so the tail
+    // cascade reaches a free ordinal. A gap absorbs the insert, so the tail
     // beyond it (including any reserved block) keeps its numbering.
     let cursor = requestedOrder;
     for (const patch of sorted) {
@@ -109,18 +109,18 @@ export function computeExactPlacementPlan(manifestPatches, newPatchCategory, new
 /**
  * Resolves a placement plan from CLI flags against the current manifest.
  * When `config` is provided, positional plans (`--before`/`--after`) are
- * refused up front if their renumber would touch a reserved range — one
- * error for the whole shift instead of per-patch policy findings. The
+ * refused up front if their renumber would touch a reserved range, giving
+ * one error for the whole shift instead of per-patch policy findings. The
  * exact `--order` branch never renames, so it bypasses that gate.
  */
 export async function resolvePlacementPlan(patchesDir, options, category, name, config) {
-    // ForWrite: placement planning feeds a manifest rewrite; a corrupt
+    // ForWrite: placement planning feeds a manifest rewrite. A corrupt
     // manifest read as empty would allocate colliding orders.
     const manifest = await loadPatchesManifestForWrite(patchesDir);
     const existingPatches = manifest?.patches ?? [];
     let targetOrder;
     if (options.order !== undefined) {
-        // Defense-in-depth — argParser covers the CLI path, but this
+        // Defense-in-depth: argParser covers the CLI path, but this
         // function is called directly from the command body which could
         // reach here with a NaN/0/negative value passed in via test harness.
         if (!Number.isInteger(options.order) || options.order <= 0) {
@@ -152,7 +152,7 @@ export async function resolvePlacementPlan(patchesDir, options, category, name,
 }
 /**
  * Projects the placement through cross-patch lint to detect forward-imports
- * the renumber would introduce *or* that the new patch itself would
+ * the renumber would introduce, or that the new patch itself would
  * introduce by landing earlier than one of its dependencies. Returns null
  * when the projection is clean.
  */
@@ -165,8 +165,8 @@ export async function projectPlacementForLint(patchesDir, plan, diff, config) {
         return { ...entry, filename: rename.newFilename, order: rename.newOrder };
     });
     // Synthetic entry for the pending patch, populated with both its new-file
-    // content AND its added-line content for files it modifies, so the
-    // forward-import rule can inspect imports the patch *itself* authors —
+    // content and its added-line content for files it modifies, so the
+    // forward-import rule can inspect imports the patch itself authors,
     // whether they live in a brand-new file or are added to an existing one.
     // Leaving either map empty lets a patch land before one of its own
     // dependencies and still pass the gate.
@@ -184,9 +184,9 @@ export async function projectPlacementForLint(patchesDir, plan, diff, config) {
     }).filter((i) => i.severity === 'error');
     if (projectedIssues.length === 0)
         return null;
-    // Baseline lint over the UN-renumbered queue: an error present here is
-    // pre-existing breakage, not a consequence of this placement — calling
-    // it one would misattribute. Entries are already in memory,
+    // Baseline lint over the un-renumbered queue: an error present here is
+    // pre-existing breakage rather than a consequence of this placement, and
+    // calling it one would misattribute. Entries are already in memory,
     // so this second pass costs no additional IO.
     const baselineIssues = lintPatchQueue({
         entries: baseCtx.entries,
@@ -274,7 +274,7 @@ export async function commitPlacementExport(input) {
                 await renumberPatchesInManifest(input.patchesDir, currentPlan.renameMap);
                 renumberApplied = true;
             }
-            // Normalize identically to commitExportedPatch — the two export
+            // Normalize identically to commitExportedPatch: the two export
             // paths must produce one artifact contract for the same diff.
             await writeText(patchPath, input.diff);
             await addPatchToManifest(input.patchesDir, {
@@ -289,7 +289,7 @@ export async function commitPlacementExport(input) {
                 catch (hookError) {
                     // Mutation has already committed and is not reversible. Warn
                     // so operators know the audit trail has a gap, but do not
-                    // re-throw — that would look like the export itself failed.
+                    // re-throw, which would look like the export itself failed.
                     warn(`History log append failed after export committed (export-order, ${currentPlan.newFilename}): ` +
                         toError(hookError).message);
                 }

package/dist/src/commands/export-placement-conflicts.d.ts

@@ -7,9 +7,9 @@ export interface PlacementAttributionPlan {
     renameMap: Map<string, PatchRenameEntry>;
 }
 /**
- * Partitions the projected placement errors into three labeled groups —
- * defects in the exported content, consequences of the renumbering, and
- * errors the queue already had — rendered as indented detail lines for
+ * Partitions the projected placement errors into three labeled groups
+ * (defects in the exported content, consequences of the renumbering, and
+ * errors the queue already had), rendered as indented detail lines for
  * {@link ConflictReport.details}. Grouping travels through the plain
  * `details: string[]`, so both refusal sites and confirmDestructive's
  * conflict printer show it without changes.

package/dist/src/commands/export-placement-conflicts.js

@@ -3,31 +3,31 @@
  * Attribution of projected placement lint errors, split out of
  * export-flow.ts for the line budget.
  *
- * A placement projection lints one merged queue — the renumbered existing
- * patches plus a synthetic entry for the pending patch — so a flat error
- * list reads as though the NEW patch caused every one of them. A renumbering
+ * A placement projection lints one merged queue (the renumbered existing
+ * patches plus a synthetic entry for the pending patch), so a flat error
+ * list reads as though the new patch caused every one of them. A renumbering
  * that re-evaluates pre-existing staged relationships (patch 455 importing a
  * module created by 456, legal only at their current ordinals) then sends
  * the operator auditing a perfectly fine export.
  *
- * Consumed by export-flow.ts; no top-level registrar is exported and none is
+ * Consumed by export-flow.ts. No top-level registrar is exported and none is
  * wanted.
  */
 import { formatPatchLintIssue } from '../core/patch-lint.js';
 /**
  * Identity for baseline comparison: check + site file + implicated
- * patches, with projected filenames mapped BACK through the rename map so
+ * patches, with projected filenames mapped back through the rename map so
  * a pre-existing error keeps its identity across the renumber. Never
- * parses the fingerprint — it is documented rename-sensitive.
+ * parses the fingerprint. It is documented rename-sensitive.
  */
 function issueKey(issue, mapBackToOldName) {
     const patches = (issue.patches ?? []).map(mapBackToOldName).sort((a, b) => a.localeCompare(b));
     return `${issue.check}|${issue.file}|${patches.join(',')}`;
 }
 /**
- * Partitions the projected placement errors into three labeled groups —
- * defects in the exported content, consequences of the renumbering, and
- * errors the queue already had — rendered as indented detail lines for
+ * Partitions the projected placement errors into three labeled groups
+ * (defects in the exported content, consequences of the renumbering, and
+ * errors the queue already had), rendered as indented detail lines for
  * {@link ConflictReport.details}. Grouping travels through the plain
  * `details: string[]`, so both refusal sites and confirmDestructive's
  * conflict printer show it without changes.

package/dist/src/commands/export-placement-gate.d.ts

@@ -11,7 +11,7 @@ import { type PlacementPlan } from './export-flow.js';
 /**
  * Spreadable optional metadata (`tier`, `lintIgnore`) derived from the
  * export flags. Every manifest-row construction site in this command
- * shares this shape; with exactOptionalPropertyTypes the keys must be
+ * shares this shape. With exactOptionalPropertyTypes the keys must be
  * omitted entirely (not set to undefined) when the flags are absent.
  */
 export declare function patchMetadataExtras(options: ExportOptions): Partial<Pick<PatchMetadata, 'tier' | 'lintIgnore'>>;
@@ -22,7 +22,7 @@ export declare function patchMetadataExtras(options: ExportOptions): Partial<Pic
  * projected manifest, and routes destructive renumbers (or dry-runs)
  * through `confirmDestructive`. Returns the plan to commit, or `'stop'`
  * when the command should end here (dry-run rendered or operator
- * cancelled — the corresponding outro has already been printed).
+ * cancelled, and the corresponding outro has already been printed).
  */
 export declare function gatePlacementPlan(args: {
     patchesDir: string;

package/dist/src/commands/export-placement-gate.js

@@ -15,7 +15,7 @@ import { placementSummary, projectPlacementForLint, resolvePlacementPlan, } from
 /**
  * Spreadable optional metadata (`tier`, `lintIgnore`) derived from the
  * export flags. Every manifest-row construction site in this command
- * shares this shape; with exactOptionalPropertyTypes the keys must be
+ * shares this shape. With exactOptionalPropertyTypes the keys must be
  * omitted entirely (not set to undefined) when the flags are absent.
  */
 export function patchMetadataExtras(options) {
@@ -33,7 +33,7 @@ export function patchMetadataExtras(options) {
  * projected manifest, and routes destructive renumbers (or dry-runs)
  * through `confirmDestructive`. Returns the plan to commit, or `'stop'`
  * when the command should end here (dry-run rendered or operator
- * cancelled — the corresponding outro has already been printed).
+ * cancelled, and the corresponding outro has already been printed).
  */
 export async function gatePlacementPlan(args) {
     const { patchesDir, options, selectedCategory, patchName, description, filesAffected, diff, config, isDryRun, s, } = args;
@@ -41,7 +41,7 @@ export async function gatePlacementPlan(args) {
         throw new InvalidArgumentError('Placement flags (--order/--before/--after) cannot be combined with --supersede.', 'export placement');
     }
     // resolvePlacementPlan runs the reserved-range gate itself when config
-    // is passed — one up-front error per run instead of per-patch findings.
+    // is passed: one up-front error per run instead of per-patch findings.
     const placementPlan = await resolvePlacementPlan(patchesDir, options, selectedCategory, patchName, config);
     const currentManifest = await loadPatchesManifest(patchesDir);
     const conflicts = await projectPlacementForLint(patchesDir, placementPlan, diff, config);
@@ -70,9 +70,9 @@ export async function gatePlacementPlan(args) {
     const summary = placementSummary(placementPlan);
     const renameCount = placementPlan.renameMap.size;
     // Route through confirmDestructive when the operation is destructive
-    // enough to warrant a prompt (more than one rename) OR when the user asked
+    // enough to warrant a prompt (more than one rename) or when the user asked
     // for a dry-run. The dry-run branch must always print the placement
-    // summary; otherwise single-rename and no-rename dry-runs exit silently
+    // summary. Otherwise single-rename and no-rename dry-runs exit silently
     // with no filename or projected layout.
     if (renameCount > 1 || isDryRun) {
         s.stop();

package/dist/src/commands/export-placement-policy.d.ts

@@ -12,7 +12,7 @@ export interface PlacementPolicyPlan {
 }
 /**
  * Refuses positional placement plans whose renumber would touch a
- * `patchPolicy.reservedRanges` block — either by moving a patch INTO a
+ * `patchPolicy.reservedRanges` block, either by moving a patch into a
  * reserved range or by moving a patch that currently sits inside one.
  * Throws a single up-front error for the first reserved range hit (the
  * per-patch alternative surfaced one confusing finding per shifted

package/dist/src/commands/export-placement-policy.js

@@ -23,7 +23,7 @@ function firstFreeOrderBelowReservedRange(patches, range) {
 }
 /**
  * Refuses positional placement plans whose renumber would touch a
- * `patchPolicy.reservedRanges` block — either by moving a patch INTO a
+ * `patchPolicy.reservedRanges` block, either by moving a patch into a
  * reserved range or by moving a patch that currently sits inside one.
  * Throws a single up-front error for the first reserved range hit (the
  * per-patch alternative surfaced one confusing finding per shifted

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

@@ -18,7 +18,7 @@ export interface RunPatchLintOptions {
     /**
      * 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.
+     * 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.
      */
@@ -75,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".
@@ -96,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;
@@ -127,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.
  *
@@ -138,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: {