@hominis/fireforge 0.19.2 → 0.19.4

package/dist/src/core/furnace-config-array-utils.d.ts

@@ -0,0 +1,11 @@
+/**
+ * Shared string-array parsing for furnace.json validation. Lives in its own
+ * module so `furnace-config-custom.ts` and `furnace-config-tokens.ts` can use
+ * it without importing `furnace-config.ts`, which would create import cycles.
+ */
+/**
+ * Parses a JSON array-of-strings field from raw furnace config data.
+ * @param value - Raw value from JSON
+ * @param fieldName - Field label for error messages
+ */
+export declare function parseStringArray(value: unknown, fieldName: string): string[];

package/dist/src/core/furnace-config-array-utils.js

@@ -0,0 +1,27 @@
+// SPDX-License-Identifier: EUPL-1.2
+/**
+ * Shared string-array parsing for furnace.json validation. Lives in its own
+ * module so `furnace-config-custom.ts` and `furnace-config-tokens.ts` can use
+ * it without importing `furnace-config.ts`, which would create import cycles.
+ */
+import { FurnaceError } from '../errors/furnace.js';
+import { isArray, isString } from '../utils/validation.js';
+/**
+ * Parses a JSON array-of-strings field from raw furnace config data.
+ * @param value - Raw value from JSON
+ * @param fieldName - Field label for error messages
+ */
+export function parseStringArray(value, fieldName) {
+    if (!isArray(value)) {
+        throw new FurnaceError(`Furnace config: "${fieldName}" must be an array`);
+    }
+    const items = [];
+    for (const item of value) {
+        if (!isString(item)) {
+            throw new FurnaceError(`Furnace config: "${fieldName}" array must contain only strings`);
+        }
+        items.push(item);
+    }
+    return items;
+}
+//# sourceMappingURL=furnace-config-array-utils.js.map

package/dist/src/core/furnace-config-custom.js

@@ -9,7 +9,7 @@
 import { FurnaceError } from '../errors/furnace.js';
 import { isExplicitAbsolutePath } from '../utils/paths.js';
 import { isBoolean, isString } from '../utils/validation.js';
-import { parseStringArray } from './furnace-config.js';
+import { parseStringArray } from './furnace-config-array-utils.js';
 import { validateSharedFtl } from './shared-ftl.js';
 /**
  * Validates a custom component config object.

package/dist/src/core/furnace-config-tokens.js

@@ -6,7 +6,7 @@
  */
 import { FurnaceError } from '../errors/furnace.js';
 import { isContainedRelativePath } from '../utils/paths.js';
-import { parseStringArray } from './furnace-config.js';
+import { parseStringArray } from './furnace-config-array-utils.js';
 /**
  * Validates a `tokenHostDocuments` raw value. Each entry must be a non-empty
  * relative path contained in the engine tree. Throws `FurnaceError` on

package/dist/src/core/furnace-config.d.ts

@@ -38,12 +38,6 @@ export declare function getFurnacePaths(root: string): FurnacePaths;
  * @returns True if furnace.json exists
  */
 export declare function furnaceConfigExists(root: string): Promise<boolean>;
-/**
- * Validates an override component config object.
- * @param data - Raw data to validate
- * @param name - Component name for error messages
- */
-export declare function parseStringArray(value: unknown, fieldName: string): string[];
 /**
  * Migrates a furnace config from an older schema version to the current one.
  * Returns the data unchanged if it is already at the current version.

package/dist/src/core/furnace-config.js

@@ -4,8 +4,9 @@ import { FurnaceError } from '../errors/furnace.js';
 import { toError } from '../utils/errors.js';
 import { pathExists, readJson, writeJson } from '../utils/fs.js';
 import { warn } from '../utils/logger.js';
-import { isArray, isObject, isString } from '../utils/validation.js';
+import { isObject, isString } from '../utils/validation.js';
 import { FIREFORGE_DIR } from './config.js';
+import { parseStringArray } from './furnace-config-array-utils.js';
 import { parseCustomConfig } from './furnace-config-custom.js';
 import { validateRuntimeVariables, validateTokenHostDocuments } from './furnace-config-tokens.js';
 import { resolveFtlDir } from './furnace-constants.js';
@@ -51,19 +52,6 @@ export async function furnaceConfigExists(root) {
  * @param data - Raw data to validate
  * @param name - Component name for error messages
  */
-export function parseStringArray(value, fieldName) {
-    if (!isArray(value)) {
-        throw new FurnaceError(`Furnace config: "${fieldName}" must be an array`);
-    }
-    const items = [];
-    for (const item of value) {
-        if (!isString(item)) {
-            throw new FurnaceError(`Furnace config: "${fieldName}" array must contain only strings`);
-        }
-        items.push(item);
-    }
-    return items;
-}
 function parseOverrideConfig(data, name) {
     const validTypes = ['css-only', 'full'];
     if (!isString(data['type']) || !validTypes.includes(data['type'])) {

package/dist/src/core/furnace-refresh.d.ts

@@ -3,10 +3,6 @@ export interface RefreshFileResult {
     status: 'merged' | 'conflict' | 'unchanged' | 'new-file';
     conflictMarkers?: number;
 }
-export interface RefreshResult {
-    files: RefreshFileResult[];
-    newBaseVersion: string;
-}
 /**
  * Refreshes a single override file against the current engine HEAD.
  *

package/dist/src/core/furnace-registration-ast.d.ts

@@ -15,7 +15,6 @@ export interface RegistrationWriteOptions {
     markerComment?: string;
 }
 export { removeCustomElementRegistration } from './furnace-registration-remove.js';
-export { CUSTOM_ELEMENTS_JS, JAR_MN } from './furnace-constants.js';
 /**
  * Adds a custom element registration entry to customElements.js.
  *

package/dist/src/core/furnace-registration-ast.js

@@ -49,7 +49,6 @@ function formatMarkerSuffix(markerComment) {
 // Re-export from split modules so existing import sites continue working
 export { removeCustomElementRegistration } from './furnace-registration-remove.js';
 // Re-export constants so existing import sites continue working
-export { CUSTOM_ELEMENTS_JS, JAR_MN } from './furnace-constants.js';
 // ---------------------------------------------------------------------------
 // Helpers
 // ---------------------------------------------------------------------------

package/dist/src/core/furnace-registration.d.ts

@@ -1,4 +1,3 @@
-export { CUSTOM_ELEMENTS_JS, JAR_MN } from './furnace-constants.js';
 export { addCustomElementRegistration, removeCustomElementRegistration, validateCustomElementRegistration, } from './furnace-registration-ast.js';
 /**
  * Adds jar.mn entries that map chrome:// URIs to on-disk paths for a
@@ -27,7 +26,7 @@ export { addCustomElementRegistration, removeCustomElementRegistration, validate
  *
  * If Firefox upstream changes the jar.mn section ordering or switches to a
  * different resource registration mechanism, the preflight validation in
- * `validateJarMnEntries` will catch the format mismatch before any writes
+ * `validateJarMnInsertionForFiles` will catch the format mismatch before any writes
  * occur.
  *
  * @param engineDir - Path to the Firefox engine source root
@@ -76,4 +75,4 @@ export declare function removeJarMnEntries(engineDir: string, tagName: string):
  * Validates that jar.mn entries *could* be added without writing anything.
  * Used by dry-run to surface structural problems early.
  */
-export declare function validateJarMnEntries(engineDir: string, tagName: string, files: string[]): Promise<void>;
+export declare function validateJarMnInsertionForFiles(engineDir: string, tagName: string, files: string[]): Promise<void>;

package/dist/src/core/furnace-registration.js

@@ -21,7 +21,6 @@ function detectJarMnIndent(lines) {
     return '   ';
 }
 // Re-export everything from the AST module so existing imports keep working
-export { CUSTOM_ELEMENTS_JS, JAR_MN } from './furnace-constants.js';
 export { addCustomElementRegistration, removeCustomElementRegistration, validateCustomElementRegistration, } from './furnace-registration-ast.js';
 import { JAR_MN } from './furnace-constants.js';
 /**
@@ -51,7 +50,7 @@ import { JAR_MN } from './furnace-constants.js';
  *
  * If Firefox upstream changes the jar.mn section ordering or switches to a
  * different resource registration mechanism, the preflight validation in
- * `validateJarMnEntries` will catch the format mismatch before any writes
+ * `validateJarMnInsertionForFiles` will catch the format mismatch before any writes
  * occur.
  *
  * @param engineDir - Path to the Firefox engine source root
@@ -237,7 +236,7 @@ export async function removeJarMnEntries(engineDir, tagName) {
  * Validates that jar.mn entries *could* be added without writing anything.
  * Used by dry-run to surface structural problems early.
  */
-export async function validateJarMnEntries(engineDir, tagName, files) {
+export async function validateJarMnInsertionForFiles(engineDir, tagName, files) {
     const filePath = join(engineDir, JAR_MN);
     if (!(await pathExists(filePath))) {
         throw new FurnaceError('jar.mn not found in engine', tagName);

package/dist/src/core/git-file-ops.d.ts

@@ -1,10 +1,4 @@
 import type { GitStatusEntry } from './git-base.js';
-/**
- * Discards changes to a specific file.
- * @param repoDir - Repository directory
- * @param filePath - Path to the file (relative to repo)
- */
-export declare function discardFile(repoDir: string, filePath: string): Promise<void>;
 /**
  * Restores a tracked path from HEAD, including staged changes.
  * @param repoDir - Repository directory

package/dist/src/core/git-file-ops.js

@@ -5,14 +5,6 @@ import { GitError } from '../errors/git.js';
 import { removeFile } from '../utils/fs.js';
 import { exec } from '../utils/process.js';
 import { ensureGit, git } from './git-base.js';
-/**
- * Discards changes to a specific file.
- * @param repoDir - Repository directory
- * @param filePath - Path to the file (relative to repo)
- */
-export async function discardFile(repoDir, filePath) {
-    await restoreTrackedPath(repoDir, filePath);
-}
 /**
  * Restores a tracked path from HEAD, including staged changes.
  * @param repoDir - Repository directory

package/dist/src/core/git-status.d.ts

@@ -20,12 +20,6 @@ export declare function getWorkingTreeStatus(repoDir: string): Promise<GitStatus
  * @returns Status entries with untracked directories expanded to individual files
  */
 export declare function expandUntrackedDirectoryEntries(repoDir: string, entries: GitStatusEntry[]): Promise<GitStatusEntry[]>;
-/**
- * Gets the list of modified files.
- * @param repoDir - Repository directory
- * @returns List of modified file paths
- */
-export declare function getModifiedFiles(repoDir: string): Promise<string[]>;
 /**
  * Gets all untracked files (including files inside untracked directories).
  * @param repoDir - Repository directory
@@ -55,11 +49,3 @@ export declare function getModifiedFilesInDir(repoDir: string, dir: string): Pro
  * @returns List of dirty file paths
  */
 export declare function getDirtyFiles(repoDir: string, files: string[]): Promise<string[]>;
-/**
- * Lists all files in a directory (tracked and untracked, respecting .gitignore).
- * Combines git ls-files for tracked files and --others for untracked files.
- * @param repoDir - Repository directory
- * @param dir - Directory path (relative to repo root)
- * @returns List of file paths relative to repo root
- */
-export declare function listAllFilesInDir(repoDir: string, dir: string): Promise<string[]>;

package/dist/src/core/git-status.js

@@ -72,15 +72,6 @@ export async function expandUntrackedDirectoryEntries(repoDir, entries) {
     }
     return expanded;
 }
-/**
- * Gets the list of modified files.
- * @param repoDir - Repository directory
- * @returns List of modified file paths
- */
-export async function getModifiedFiles(repoDir) {
-    const entries = await getWorkingTreeStatus(repoDir);
-    return entries.map((entry) => entry.file);
-}
 /**
  * Gets all untracked files (including files inside untracked directories).
  * @param repoDir - Repository directory
@@ -134,19 +125,4 @@ export async function getDirtyFiles(repoDir, files) {
     const untracked = untrackedOutput.split('\n').filter((line) => line.trim().length > 0);
     return [...new Set([...tracked, ...untracked])].sort();
 }
-/**
- * Lists all files in a directory (tracked and untracked, respecting .gitignore).
- * Combines git ls-files for tracked files and --others for untracked files.
- * @param repoDir - Repository directory
- * @param dir - Directory path (relative to repo root)
- * @returns List of file paths relative to repo root
- */
-export async function listAllFilesInDir(repoDir, dir) {
-    await ensureGit();
-    const trackedOutput = await git(['ls-files', '--', dir], repoDir);
-    const trackedFiles = trackedOutput.split('\n').filter((line) => line.trim().length > 0);
-    const untrackedOutput = await git(['ls-files', '--others', '--exclude-standard', '--', dir], repoDir);
-    const untrackedFiles = untrackedOutput.split('\n').filter((line) => line.trim().length > 0);
-    return [...new Set([...trackedFiles, ...untrackedFiles])].sort();
-}
 //# sourceMappingURL=git-status.js.map

package/dist/src/core/git.d.ts

@@ -1,4 +1,3 @@
-export type { GitStatusEntry } from './git-base.js';
 /**
  * Checks if a directory is a git repository.
  * @param dir - Directory to check

package/dist/src/core/mach.d.ts

@@ -1,6 +1,6 @@
 import { type SmokeLineCallback, type SmokeRunResult } from '../utils/process.js';
-export { attemptMozinfoRewrite, type BuildArtifactCheck, buildArtifactMismatchMessage, hasBuildArtifacts, hasRunnableBundle, type MozinfoRewriteResult, type RunnableBundleCheck, } from './mach-build-artifacts.js';
-export { generateMozconfig, type MozconfigVariables } from './mach-mozconfig.js';
+export { attemptMozinfoRewrite, buildArtifactMismatchMessage, hasBuildArtifacts, hasRunnableBundle, } from './mach-build-artifacts.js';
+export { generateMozconfig } from './mach-mozconfig.js';
 export { ensurePython, resetResolvedPython } from './mach-python.js';
 /**
  * Ensures mach is available in the engine directory.

package/dist/src/core/marionette-port.d.ts

@@ -64,13 +64,31 @@ export declare function assertMarionettePortAvailable(port?: number, options?: {
  *   `undefined`.
  */
 export declare function extractForwardedMarionettePort(machArgs: string[]): number | undefined;
+/**
+ * True when forwarded mach args explicitly select the xpcshell harness.
+ * Used so `--marionette-port` auto-forward skips `--setpref=marionette.port`
+ * for runs where the pref is ignored anyway.
+ */
+export declare function hasExplicitXpcshellFlavor(machArgs: string[]): boolean;
+/**
+ * Whether `fireforge test` should append `--setpref=marionette.port=<n>` when
+ * the operator passed `--marionette-port`. Forwards for every harness except
+ * an explicit `--flavor=xpcshell` / `xpcshell-tests` (toolkit widget mochitests
+ * under `toolkit/content/tests/` do not match the older path-only heuristic
+ * but still launch a Marionette-driven browser).
+ */
+export declare function shouldAutoForwardMarionettePortToMach(machArgs: string[]): boolean;
 /**
  * Heuristic: do the test paths or forwarded mach args indicate a flavour
  * that actually launches a Marionette-driven browser? Browser-chrome and
- * mochitest do; xpcshell does not. Used to decide whether to auto-forward
- * `--setpref=marionette.port=<n>` to mach when the operator passed
- * `--marionette-port`. A no-paths invocation (the default "run all tests"
- * shape) is treated as marionette-relevant since it includes browser-chrome.
+ * mochitest do; xpcshell does not. A no-paths invocation (the default "run
+ * all tests" shape) is treated as marionette-relevant since it includes
+ * browser-chrome.
+ *
+ * Note: `fireforge test` auto-forward of `--marionette-port` to mach uses
+ * {@link shouldAutoForwardMarionettePortToMach} (mach-arg flavor gate) rather
+ * than this function alone, so toolkit paths without `/mochitest/` still get
+ * the pref when appropriate.
  *
  * @param testPaths - Engine-relative paths after `stripEnginePrefix`.
  * @param machArgs - Forwarded mach args (post-`--mach-arg`).

package/dist/src/core/marionette-port.js

@@ -260,23 +260,47 @@ export function extractForwardedMarionettePort(machArgs) {
     }
     return undefined;
 }
+/**
+ * True when forwarded mach args explicitly select the xpcshell harness.
+ * Used so `--marionette-port` auto-forward skips `--setpref=marionette.port`
+ * for runs where the pref is ignored anyway.
+ */
+export function hasExplicitXpcshellFlavor(machArgs) {
+    for (const arg of machArgs) {
+        if (/^--flavor=xpcshell\b/.test(arg) || arg === '--flavor=xpcshell-tests')
+            return true;
+    }
+    return false;
+}
+/**
+ * Whether `fireforge test` should append `--setpref=marionette.port=<n>` when
+ * the operator passed `--marionette-port`. Forwards for every harness except
+ * an explicit `--flavor=xpcshell` / `xpcshell-tests` (toolkit widget mochitests
+ * under `toolkit/content/tests/` do not match the older path-only heuristic
+ * but still launch a Marionette-driven browser).
+ */
+export function shouldAutoForwardMarionettePortToMach(machArgs) {
+    return !hasExplicitXpcshellFlavor(machArgs);
+}
 /**
  * Heuristic: do the test paths or forwarded mach args indicate a flavour
  * that actually launches a Marionette-driven browser? Browser-chrome and
- * mochitest do; xpcshell does not. Used to decide whether to auto-forward
- * `--setpref=marionette.port=<n>` to mach when the operator passed
- * `--marionette-port`. A no-paths invocation (the default "run all tests"
- * shape) is treated as marionette-relevant since it includes browser-chrome.
+ * mochitest do; xpcshell does not. A no-paths invocation (the default "run
+ * all tests" shape) is treated as marionette-relevant since it includes
+ * browser-chrome.
+ *
+ * Note: `fireforge test` auto-forward of `--marionette-port` to mach uses
+ * {@link shouldAutoForwardMarionettePortToMach} (mach-arg flavor gate) rather
+ * than this function alone, so toolkit paths without `/mochitest/` still get
+ * the pref when appropriate.
  *
  * @param testPaths - Engine-relative paths after `stripEnginePrefix`.
  * @param machArgs - Forwarded mach args (post-`--mach-arg`).
  * @returns `true` when the run is likely to bind a Marionette listener.
  */
 export function isMarionetteFlavor(testPaths, machArgs) {
-    for (const arg of machArgs) {
-        if (/^--flavor=xpcshell\b/.test(arg) || arg === '--flavor=xpcshell-tests')
-            return false;
-    }
+    if (hasExplicitXpcshellFlavor(machArgs))
+        return false;
     for (const arg of machArgs) {
         if (/^--flavor=(browser-chrome|mochitest|chrome|a11y)\b/.test(arg))
             return true;
@@ -291,6 +315,9 @@ export function isMarionetteFlavor(testPaths, machArgs) {
             return true;
         if (path.includes('/browser-chrome/') || path.startsWith('browser-chrome/'))
             return true;
+        if (path.includes('toolkit/content/tests/') && !path.includes('/tests/xpcshell/')) {
+            return true;
+        }
     }
     return false;
 }

package/dist/src/core/patch-apply.d.ts

@@ -6,8 +6,8 @@ import type { ImportSummary, PatchResult } from '../types/commands/index.js';
 export { PatchError } from '../errors/patch.js';
 export { countPatches, discoverPatches, getAllTargetFilesFromPatch, getTargetFileFromPatch, isNewFilePatch, } from './patch-files.js';
 export { withPatchDirectoryLock } from './patch-lock.js';
-export { extractAffectedFiles, extractOrder, isNewFileInPatch, parseHunksForFile, } from './patch-parse.js';
-export { applyPatchToContent, extractNewFileContent, extractNewFileContentFromDiff, } from './patch-transform.js';
+export { extractAffectedFiles, extractOrder, isNewFileInPatch } from './patch-parse.js';
+export { applyPatchToContent, extractNewFileContent } from './patch-transform.js';
 /**
  * Applies all patches in order. Rolls back all successfully applied
  * patches when one fails so the engine directory stays clean.

package/dist/src/core/patch-apply.js

@@ -21,8 +21,8 @@ import { applyPatchToContent, extractNewFileContent } from './patch-transform.js
 export { PatchError } from '../errors/patch.js';
 export { countPatches, discoverPatches, getAllTargetFilesFromPatch, getTargetFileFromPatch, isNewFilePatch, } from './patch-files.js';
 export { withPatchDirectoryLock } from './patch-lock.js';
-export { extractAffectedFiles, extractOrder, isNewFileInPatch, parseHunksForFile, } from './patch-parse.js';
-export { applyPatchToContent, extractNewFileContent, extractNewFileContentFromDiff, } from './patch-transform.js';
+export { extractAffectedFiles, extractOrder, isNewFileInPatch } from './patch-parse.js';
+export { applyPatchToContent, extractNewFileContent } from './patch-transform.js';
 /**
  * Applies a single patch.
  * @param patch - Patch info

package/dist/src/core/patch-lint-reexports.d.ts

@@ -2,8 +2,5 @@
  * Public re-exports for {@link ./patch-lint.ts}. Split out so the
  * orchestrator stays within the ESLint `max-lines` budget.
  */
-export { runCheckJs } from './patch-lint-checkjs.js';
-export { buildPatchQueueContext, collectNewFileCreatorsByPath, type ExtractedSpecifier, extractImportSpecifiers, extractImportSpecifiersWithLines, findForwardImportIgnoreLines, FORWARD_IMPORT_IGNORE_MARKER, isForwardImportableFile, lintPatchQueue, lintPatchQueueDuplicateCreations, lintPatchQueueForwardImports, type PatchQueueContext, type PatchQueueEntry, } from './patch-lint-cross.js';
+export { buildPatchQueueContext, collectNewFileCreatorsByPath, extractImportSpecifiers, extractImportSpecifiersWithLines, findForwardImportIgnoreLines, FORWARD_IMPORT_IGNORE_MARKER, isForwardImportableFile, lintPatchQueue, lintPatchQueueDuplicateCreations, lintPatchQueueForwardImports, type PatchQueueContext, type PatchQueueEntry, } from './patch-lint-cross.js';
 export { buildModifiedFileAdditionsFromDiff, detectNewFilesInDiff } from './patch-lint-diff.js';
-export { type JsDocCheck, type JsDocIssue, validateExportJsDoc } from './patch-lint-jsdoc.js';
-export { resolvePatchOwnedChromeScripts, resolvePatchOwnedSysMjs } from './patch-lint-ownership.js';

package/dist/src/core/patch-lint-reexports.js

@@ -3,9 +3,6 @@
  * Public re-exports for {@link ./patch-lint.ts}. Split out so the
  * orchestrator stays within the ESLint `max-lines` budget.
  */
-export { runCheckJs } from './patch-lint-checkjs.js';
 export { buildPatchQueueContext, collectNewFileCreatorsByPath, extractImportSpecifiers, extractImportSpecifiersWithLines, findForwardImportIgnoreLines, FORWARD_IMPORT_IGNORE_MARKER, isForwardImportableFile, lintPatchQueue, lintPatchQueueDuplicateCreations, lintPatchQueueForwardImports, } from './patch-lint-cross.js';
 export { buildModifiedFileAdditionsFromDiff, detectNewFilesInDiff } from './patch-lint-diff.js';
-export { validateExportJsDoc } from './patch-lint-jsdoc.js';
-export { resolvePatchOwnedChromeScripts, resolvePatchOwnedSysMjs } from './patch-lint-ownership.js';
 //# sourceMappingURL=patch-lint-reexports.js.map

package/dist/src/core/patch-manifest.d.ts

@@ -4,9 +4,8 @@
  * Callers should continue to import from this module; the internal split
  * is an implementation detail.
  */
-export type { PatchManifestConsistencyIssue } from './patch-manifest-consistency.js';
 export { rebuildPatchesManifest, validatePatchesManifestConsistency, } from './patch-manifest-consistency.js';
-export { addPatchToManifest, loadPatchesManifest, PatchDeleteRollbackError, PATCHES_MANIFEST, type PatchRenameEntry, removePatchFileAndManifest, removePatchFromManifest, renumberPatchesInManifest, savePatchesManifest, } from './patch-manifest-io.js';
+export { addPatchToManifest, loadPatchesManifest, PATCHES_MANIFEST, type PatchRenameEntry, removePatchFileAndManifest, renumberPatchesInManifest, savePatchesManifest, } from './patch-manifest-io.js';
 export { checkVersionCompatibility, findPatchesAffectingFile, getClaimedFiles, stampPatchVersions, validatePatchIntegrity, } from './patch-manifest-query.js';
 export { resolvePatchIdentifier } from './patch-manifest-resolve.js';
 export { validatePatchesManifest } from './patch-manifest-validate.js';

package/dist/src/core/patch-manifest.js

@@ -6,7 +6,7 @@
  * is an implementation detail.
  */
 export { rebuildPatchesManifest, validatePatchesManifestConsistency, } from './patch-manifest-consistency.js';
-export { addPatchToManifest, loadPatchesManifest, PatchDeleteRollbackError, PATCHES_MANIFEST, removePatchFileAndManifest, removePatchFromManifest, renumberPatchesInManifest, savePatchesManifest, } from './patch-manifest-io.js';
+export { addPatchToManifest, loadPatchesManifest, PATCHES_MANIFEST, removePatchFileAndManifest, renumberPatchesInManifest, savePatchesManifest, } from './patch-manifest-io.js';
 export { checkVersionCompatibility, findPatchesAffectingFile, getClaimedFiles, stampPatchVersions, validatePatchIntegrity, } from './patch-manifest-query.js';
 export { resolvePatchIdentifier } from './patch-manifest-resolve.js';
 export { validatePatchesManifest } from './patch-manifest-validate.js';

package/dist/src/core/signal-critical.d.ts

@@ -29,11 +29,6 @@
  * persist pair," not "postpone exit indefinitely."
  */
 export declare function runInSignalCriticalSection<T>(label: string, fn: () => Promise<T>): Promise<T>;
-/**
- * Returns true while any critical section is currently running. Used by the
- * bin entry point's signal handler to decide whether to await before exit.
- */
-export declare function hasActiveCriticalSection(): boolean;
 /**
  * Waits for every active critical section to complete or for `timeoutMs` to
  * elapse, whichever comes first. Never rejects: a section that throws still
@@ -41,9 +36,3 @@ export declare function hasActiveCriticalSection(): boolean;
  * cleans up in `finally`.
  */
 export declare function waitForActiveCriticalSections(timeoutMs: number): Promise<void>;
-/**
- * Test-only helper: clears the critical-section registry. Production code
- * must never call this — it voids the exit-ordering guarantee for any
- * section still in flight.
- */
-export declare function resetCriticalSectionsForTests(): void;

package/dist/src/core/signal-critical.js

@@ -47,13 +47,6 @@ export async function runInSignalCriticalSection(label, fn) {
         resolver();
     }
 }
-/**
- * Returns true while any critical section is currently running. Used by the
- * bin entry point's signal handler to decide whether to await before exit.
- */
-export function hasActiveCriticalSection() {
-    return activeSections.size > 0;
-}
 /**
  * Waits for every active critical section to complete or for `timeoutMs` to
  * elapse, whichever comes first. Never rejects: a section that throws still
@@ -69,12 +62,4 @@ export async function waitForActiveCriticalSections(timeoutMs) {
         new Promise((resolve) => setTimeout(resolve, timeoutMs)),
     ]);
 }
-/**
- * Test-only helper: clears the critical-section registry. Production code
- * must never call this — it voids the exit-ordering guarantee for any
- * section still in flight.
- */
-export function resetCriticalSectionsForTests() {
-    activeSections.clear();
-}
 //# sourceMappingURL=signal-critical.js.map

package/dist/src/core/wire-targets.d.ts

@@ -1,7 +1,7 @@
 /**
  * Wire targets barrel — re-exports all wiring target modules.
  */
-export { addDestroyAST, addDestroyToBrowserInit, legacyAddDestroy } from './wire-destroy.js';
-export { addDomFragment, addDomFragmentTokenized, legacyAddDomFragment, } from './wire-dom-fragment.js';
-export { addInitAST, addInitToBrowserInit, legacyAddInit } from './wire-init.js';
-export { addSubscriptAST, addSubscriptToBrowserMain, legacyAddSubscript, } from './wire-subscript.js';
+export { addDestroyToBrowserInit } from './wire-destroy.js';
+export { addDomFragment } from './wire-dom-fragment.js';
+export { addInitToBrowserInit } from './wire-init.js';
+export { addSubscriptToBrowserMain } from './wire-subscript.js';

package/dist/src/core/wire-targets.js

@@ -2,8 +2,8 @@
 /**
  * Wire targets barrel — re-exports all wiring target modules.
  */
-export { addDestroyAST, addDestroyToBrowserInit, legacyAddDestroy } from './wire-destroy.js';
-export { addDomFragment, addDomFragmentTokenized, legacyAddDomFragment, } from './wire-dom-fragment.js';
-export { addInitAST, addInitToBrowserInit, legacyAddInit } from './wire-init.js';
-export { addSubscriptAST, addSubscriptToBrowserMain, legacyAddSubscript, } from './wire-subscript.js';
+export { addDestroyToBrowserInit } from './wire-destroy.js';
+export { addDomFragment } from './wire-dom-fragment.js';
+export { addInitToBrowserInit } from './wire-init.js';
+export { addSubscriptToBrowserMain } from './wire-subscript.js';
 //# sourceMappingURL=wire-targets.js.map

package/dist/src/types/commands/options.d.ts

@@ -319,9 +319,10 @@ export interface TestOptions {
     /**
      * Override the Marionette control port (default 2828) used by the
      * stale-browser probe, the `--doctor` preflight, and the auto-forwarded
-     * `--setpref=marionette.port=<n>` arg passed to mach. Set this when a
-     * stale process holds the default port and `kill` is not an option, or
-     * when a CI runner reserves a different port for parallel test runs.
+     * `--setpref=marionette.port=<n>` arg passed to mach (omitted when mach
+     * args explicitly set `--flavor=xpcshell` / `xpcshell-tests`). Set this
+     * when a stale process holds the default port and `kill` is not an option,
+     * or when a CI runner reserves a different port for parallel test runs.
      */
     marionettePort?: number;
 }

package/dist/src/utils/logger.d.ts

@@ -3,11 +3,6 @@
  * @param enabled - Whether to enable verbose output
  */
 export declare function setVerbose(enabled: boolean): void;
-/**
- * Checks if verbose mode is enabled.
- * @returns True if verbose mode is enabled
- */
-export declare function isVerbose(): boolean;
 /**
  * Displays a verbose/debug message (only shown if verbose mode is enabled).
  * @param message - Message to display

package/dist/src/utils/logger.js

@@ -14,7 +14,7 @@ export function setVerbose(enabled) {
  * Checks if verbose mode is enabled.
  * @returns True if verbose mode is enabled
  */
-export function isVerbose() {
+function isVerbose() {
     return verboseMode;
 }
 /**
@@ -22,7 +22,7 @@ export function isVerbose() {
  * @param message - Message to display
  */
 export function verbose(message) {
-    if (verboseMode) {
+    if (isVerbose()) {
         p.log.info(`[debug] ${message}`);
     }
 }

package/dist/src/utils/package-root.d.ts

@@ -14,8 +14,6 @@ interface PackageMetadata {
  * within a process.
  */
 export declare function getPackageRoot(): string;
-/** Clears the cached package root for testing. */
-export declare function resetPackageRootCacheForTests(): void;
 /** @internal */
 export declare function isFireForgePackageMetadata(pkg: PackageMetadata): boolean;
 /** Reads the current package version from the repository root package manifest. */

package/dist/src/utils/package-root.js

@@ -53,10 +53,6 @@ export function getPackageRoot() {
         current = parent;
     }
 }
-/** Clears the cached package root for testing. */
-export function resetPackageRootCacheForTests() {
-    cachedPackageRoot = undefined;
-}
 /** @internal */
 export function isFireForgePackageMetadata(pkg) {
     if (typeof pkg.bin !== 'object' || pkg.bin === null || Array.isArray(pkg.bin)) {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@hominis/fireforge",
-  "version": "0.19.2",
+  "version": "0.19.4",
   "description": "FireForge — a build tool for customizing Firefox",
   "type": "module",
   "main": "./dist/src/index.js",