@hominis/fireforge 0.45.0 → 0.46.0

package/dist/src/commands/furnace/create-browser-test.js

@@ -16,6 +16,7 @@ import { toError } from '../../utils/errors.js';
 import { ensureDir, pathExists, readText, writeText } from '../../utils/fs.js';
 import { info, success, warn } from '../../utils/logger.js';
 import { stripEnginePrefix } from '../../utils/paths.js';
+import { browserTestFileName, deriveTestStem } from './test-file-name.js';
 /**
  * Validates the `--test-dir` option against the resolved test style before
  * any writes. Mochikit lives in the upstream toolkit/content/tests/widgets
@@ -65,15 +66,9 @@ export function resolveTestDirOverride(raw) {
  * the harness will try to run.
  */
 export async function scaffoldTestFiles(componentName, license, forgeConfig, paths, journal, testDirOverride) {
-    const strippedName = componentName.startsWith('moz-') ? componentName.slice(4) : componentName;
-    // Avoid double-prefixing: strip binaryName prefix since the default test
-    // dir name already uses it
     const binaryName = forgeConfig.binaryName;
-    const withoutBinaryPrefix = strippedName.startsWith(binaryName + '-')
-        ? strippedName.slice(binaryName.length + 1)
-        : strippedName;
-    const underscored = withoutBinaryPrefix.replace(/-/g, '_');
-    const testFileName = `browser_${binaryName}_${underscored}.js`;
+    const underscored = deriveTestStem(componentName, binaryName);
+    const testFileName = browserTestFileName(componentName, binaryName);
     // --test-dir redirects the scaffold: the hardcoded
     // `.../test/<binaryName>/` target can collide with a test suite owned by a
     // different patch. The manifest-registration name is the path below

package/dist/src/commands/furnace/create-dry-run.js

@@ -8,6 +8,7 @@
  * independently of the mutation plumbing.
  */
 import { BROWSER_TEST_SCAFFOLD_ROOT, resolveBrowserChromeTestDir, resolveXpcshellTestDir, } from '../../core/furnace-constants.js';
+import { deriveTestStem } from './test-file-name.js';
 /**
  * Builds the test-section fragment of the dry-run plan for a given
  * harness choice. Kept separate from the top-level formatter so the
@@ -18,11 +19,7 @@ function formatTestSection(args) {
     const { testStyle, componentName, binaryName, testDir } = args;
     if (testStyle === 'none')
         return '';
-    const strippedName = componentName.startsWith('moz-') ? componentName.slice(4) : componentName;
-    const withoutBinaryPrefix = strippedName.startsWith(binaryName + '-')
-        ? strippedName.slice(binaryName.length + 1)
-        : strippedName;
-    const underscored = withoutBinaryPrefix.replace(/-/g, '_');
+    const underscored = deriveTestStem(componentName, binaryName);
     if (testStyle === 'browser-chrome') {
         // Same resolver as scaffoldTestFiles: the registration name is the
         // path below browser/base/content/test/, which under --test-dir is

package/dist/src/commands/furnace/create-templates.d.ts

@@ -3,6 +3,22 @@
  * command entrypoint so the generator is unit-testable in isolation and the
  * command file stays under the per-file LOC budget.
  */
+export interface GenerateMjsContentOptions {
+    /** Custom element tag name. */
+    name: string;
+    /** Generated component class name. */
+    className: string;
+    /** Human-readable component description. */
+    description: string;
+    /** Whether to emit the Fluent wiring. */
+    localized: boolean;
+    /** License header prepended to the module. */
+    header: string;
+    /** chrome:// sub-path for the generated FTL reference, when localized. */
+    ftlChromeSubPath: string | undefined;
+    /** Explicit shared FTL path, used verbatim when supplied. */
+    sharedFtl: string | undefined;
+}
 /**
  * Generates the .mjs file content for a custom component.
  *
@@ -23,7 +39,7 @@
  *      matching the locale jar.mn entry that `furnace apply` writes.
  *   3. `<name>.ftl` — fallback when no chrome sub-path was resolvable.
  */
-export declare function generateMjsContent(name: string, className: string, description: string, localized: boolean, header: string, ftlChromeSubPath: string | undefined, sharedFtl: string | undefined): string;
+export declare function generateMjsContent(options: GenerateMjsContentOptions): string;
 /** Generates the .css file content for a custom component. */
 export declare function generateCssContent(header: string): string;
 /** Generates the .ftl file content for a custom component. */

package/dist/src/commands/furnace/create-templates.js

@@ -24,7 +24,8 @@
  *      matching the locale jar.mn entry that `furnace apply` writes.
  *   3. `<name>.ftl` — fallback when no chrome sub-path was resolvable.
  */
-export function generateMjsContent(name, className, description, localized, header, ftlChromeSubPath, sharedFtl) {
+export function generateMjsContent(options) {
+    const { name, className, description, localized, header, ftlChromeSubPath, sharedFtl } = options;
     // A `*/` in the operator-supplied description would close the JSDoc block
     // early and leave the remainder as stray code. Escaped here rather than
     // rejected at input, because the same text is also rendered by

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

@@ -3,7 +3,7 @@ import { join } from 'node:path';
 import { text } from '@clack/prompts';
 import { getProjectPaths, loadConfig } from '../../core/config.js';
 import { stdioIsInteractive } from '../../core/destructive.js';
-import { createDefaultFurnaceConfig, furnaceConfigExists, getFurnacePaths, loadFurnaceConfig, writeFurnaceConfig, } from '../../core/furnace-config.js';
+import { getFurnacePaths, writeFurnaceConfig } from '../../core/furnace-config.js';
 import { resolveFtlChromeSubPath, tagNameToClassName, WIDGETS_DIR, } from '../../core/furnace-constants.js';
 import { completeJournalRollback, runFurnaceMutation, } from '../../core/furnace-operation.js';
 import { CUSTOM_ELEMENT_TAG_PATTERN, CUSTOM_ELEMENT_TAG_RULES, describeTagNameProblem, } from '../../core/furnace-registration-validate.js';
@@ -15,6 +15,7 @@ import { InvalidArgumentError } from '../../errors/base.js';
 import { FurnaceError } from '../../errors/furnace.js';
 import { ensureDir, pathExists, writeText } from '../../utils/fs.js';
 import { cancel, intro, isCancel, note, outro } from '../../utils/logger.js';
+import { loadAuthoringFurnaceConfig } from './authoring-config.js';
 import { resolveValidatedTestDir, scaffoldTestFiles } from './create-browser-test.js';
 import { formatDryRunPlan, formatSuccessNote } from './create-dry-run.js';
 import { resolveCreateFeatures } from './create-features.js';
@@ -23,12 +24,6 @@ import { assertCustomEntryPersisted } from './create-readback.js';
 import { generateCssContent, generateFtlContent, generateMjsContent } from './create-templates.js';
 import { validateCreateAgainstConfig } from './create-validation.js';
 import { scaffoldXpcshellTestFiles } from './create-xpcshell.js';
-async function loadAuthoringFurnaceConfig(projectRoot) {
-    if (await furnaceConfigExists(projectRoot)) {
-        return loadFurnaceConfig(projectRoot);
-    }
-    return createDefaultFurnaceConfig();
-}
 function knownComponentSet(config) {
     return new Set([
         ...config.stock,
@@ -64,24 +59,25 @@ function validateTagName(name) {
 }
 /**
  * Writes the scaffolded component source files to disk.
- * @param componentDir - Destination component directory
- * @param componentName - Custom element tag name
- * @param className - Generated component class name
- * @param description - Human-readable component description
- * @param localized - Whether to include a Fluent file
- * @param license - Project license used for generated headers
- * @param ftlChromeSubPath - chrome:// sub-path for the generated FTL reference, when localized
- * @param sharedFtl - Explicit shared FTL path from `--shared-ftl`, when supplied
- * @param journal - Optional rollback journal that snapshots files before writes
+ * @param options - See {@link WriteComponentFilesOptions}
  * @returns Relative filenames written for the component
  */
-async function writeComponentFiles(componentDir, componentName, className, description, localized, license, ftlChromeSubPath, sharedFtl, journal) {
+async function writeComponentFiles(options) {
+    const { componentDir, componentName, className, description, localized, license, ftlChromeSubPath, sharedFtl, journal, } = options;
     await ensureDir(componentDir);
     const files = [`${componentName}.mjs`, `${componentName}.css`];
     const mjsPath = join(componentDir, `${componentName}.mjs`);
     if (journal)
         await snapshotFile(journal, mjsPath);
-    const mjsContent = generateMjsContent(componentName, className, description, localized, getLicenseHeader(license, 'js'), ftlChromeSubPath, sharedFtl);
+    const mjsContent = generateMjsContent({
+        name: componentName,
+        className,
+        description,
+        localized,
+        header: getLicenseHeader(license, 'js'),
+        ftlChromeSubPath,
+        sharedFtl,
+    });
     await writeText(mjsPath, mjsContent);
     const cssPath = join(componentDir, `${componentName}.css`);
     if (journal)
@@ -165,7 +161,17 @@ async function performCreateMutations(args) {
         // so signal-driven rollback can clean it up even if writeComponentFiles
         // is interrupted mid-ensureDir.
         recordCreatedDir(journal, args.componentDir);
-        files = await writeComponentFiles(args.componentDir, args.componentName, args.className, args.description, args.localized, args.license, resolveFtlChromeSubPath(freshConfig.ftlBasePath), args.sharedFtl, journal);
+        files = await writeComponentFiles({
+            componentDir: args.componentDir,
+            componentName: args.componentName,
+            className: args.className,
+            description: args.description,
+            localized: args.localized,
+            license: args.license,
+            ftlChromeSubPath: resolveFtlChromeSubPath(freshConfig.ftlBasePath),
+            sharedFtl: args.sharedFtl,
+            journal,
+        });
         const customEntry = {
             description: args.description,
             targetPath: `${WIDGETS_DIR}/${args.componentName}`,

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

@@ -2,7 +2,7 @@
 import { loadConfig } from '../../core/config.js';
 import { applyAllComponents } from '../../core/furnace-apply.js';
 import { logApplyResult } from '../../core/furnace-apply-output.js';
-import { getFurnacePaths, loadFurnaceConfig } from '../../core/furnace-config.js';
+import { getFurnacePaths } from '../../core/furnace-config.js';
 import { reportJsconfigPathsSync } from '../../core/furnace-jsconfig.js';
 import { runFurnaceMutation, waitLockMutationOptions, } from '../../core/furnace-operation.js';
 import { assertFurnaceReady } from '../../core/furnace-precondition.js';
@@ -203,7 +203,16 @@ export async function furnaceDeployCommand(projectRoot, name, options = {}) {
     }
     const validateSpinner = spinner(isDryRun ? 'Validating (read-only)...' : 'Validating...');
     const failedComponents = getFailedComponentNames(result);
-    const validation = await runDeployValidation(validateSpinner, name, config, furnacePaths, failedComponents, isDryRun, projectRoot, result.actions);
+    const validation = await runDeployValidation({
+        validateSpinner,
+        name,
+        config,
+        furnacePaths,
+        failedComponents,
+        isDryRun,
+        projectRoot,
+        dryRunActions: result.actions,
+    });
     if (validation.done)
         return;
     const { totalErrors, totalWarnings, componentCount, skippedValidationCount } = validation;

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

@@ -13,6 +13,7 @@ import { FurnaceError } from '../../errors/furnace.js';
 import { getNodeErrorCode, toError } from '../../utils/errors.js';
 import { ensureDir, pathExists, writeText } from '../../utils/fs.js';
 import { cancel, info, intro, isCancel, note, outro, success, warn } from '../../utils/logger.js';
+import { normalizePathSlashes } from '../../utils/paths.js';
 /**
  * File extensions that are definitely FTL resources (not locale
  * directories). A value ending in one of these is almost certainly the
@@ -64,7 +65,7 @@ async function validateFtlBasePath(value, engineDir) {
     // Normalize with the POSIX rules regardless of host: `normalize` on Windows
     // emits backslashes, so a `../`-prefix test against the platform `normalize`
     // silently passes every traversal attempt there.
-    const normalized = posix.normalize(value.replace(/\\/g, '/'));
+    const normalized = posix.normalize(normalizePathSlashes(value));
     if (normalized === '..' || normalized.startsWith('../')) {
         throw new FurnaceError(`ftlBasePath "${value}" must not escape the engine checkout via parent-directory segments.`);
     }

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

@@ -4,7 +4,7 @@ import { dirname, join } from 'node:path';
 import { select, text } from '@clack/prompts';
 import { getProjectPaths, loadConfig, loadState } from '../../core/config.js';
 import { stdioIsInteractive } from '../../core/destructive.js';
-import { createDefaultFurnaceConfig, furnaceConfigExists, getFurnacePaths, loadFurnaceConfig, writeFurnaceConfig, } from '../../core/furnace-config.js';
+import { getFurnacePaths, writeFurnaceConfig } from '../../core/furnace-config.js';
 import { resolveFtlDir } from '../../core/furnace-constants.js';
 import { completeJournalRollback, runFurnaceMutation } from '../../core/furnace-operation.js';
 import { assertFurnaceEngineReady } from '../../core/furnace-precondition.js';
@@ -16,25 +16,14 @@ import { FurnaceError } from '../../errors/furnace.js';
 import { toError } from '../../utils/errors.js';
 import { copyFile, ensureDir, pathExists, writeJson } from '../../utils/fs.js';
 import { cancel, info, intro, isCancel, note, outro, warn } from '../../utils/logger.js';
-async function loadAuthoringFurnaceConfig(projectRoot) {
-    if (await furnaceConfigExists(projectRoot)) {
-        return loadFurnaceConfig(projectRoot);
-    }
-    return createDefaultFurnaceConfig();
-}
+import { loadAuthoringFurnaceConfig } from './authoring-config.js';
 /**
  * Copies the source files needed for a new override into the workspace.
- * @param engineDir - Absolute path to the engine checkout
- * @param srcDir - Original component directory in the engine checkout
- * @param destDir - Destination override directory in the workspace
- * @param componentName - Custom element tag name being overridden
- * @param hasFTL - Whether the component ships a localized `.ftl` file
- * @param overrideType - Requested override mode
- * @param ftlDir - Engine-relative directory the shared `.ftl` deploys into
- * @param journal - Rollback journal every write is recorded in
+ * @param options - See {@link CopyOverrideFilesOptions}
  * @returns Filenames copied into the override directory
  */
-async function copyOverrideFiles(engineDir, srcDir, destDir, componentName, hasFTL, overrideType, ftlDir, journal) {
+async function copyOverrideFiles(options) {
+    const { engineDir, srcDir, destDir, componentName, hasFTL, overrideType, ftlDir, journal } = options;
     await ensureDir(destDir);
     const entries = await readdir(srcDir, { withFileTypes: true });
     const copiedFiles = [];
@@ -93,7 +82,8 @@ async function copyOverrideFiles(engineDir, srcDir, destDir, componentName, hasF
  * cycles into a single surviving entry — otherwise both report success and
  * furnace.json keeps only the second writer's.
  */
-async function saveOverrideConfig(projectRoot, destDir, componentName, overrideType, description, details, firefoxVersion, journal, baseCommit) {
+async function saveOverrideConfig(options) {
+    const { projectRoot, destDir, componentName, overrideType, description, details, firefoxVersion, journal, baseCommit, } = options;
     const overrideJson = {
         type: overrideType,
         description,
@@ -138,9 +128,28 @@ async function performOverrideMutations(args) {
         ctx.registerJournal(journal);
         recordCreatedDir(journal, args.destDir);
         try {
-            const filesCopied = await copyOverrideFiles(args.engineDir, args.srcDir, args.destDir, args.componentName, args.details.hasFTL, args.overrideType, args.ftlDir, journal);
+            const filesCopied = await copyOverrideFiles({
+                engineDir: args.engineDir,
+                srcDir: args.srcDir,
+                destDir: args.destDir,
+                componentName: args.componentName,
+                hasFTL: args.details.hasFTL,
+                overrideType: args.overrideType,
+                ftlDir: args.ftlDir,
+                journal,
+            });
             await snapshotFile(journal, args.furnacePaths.furnaceConfig);
-            await saveOverrideConfig(args.projectRoot, args.destDir, args.componentName, args.overrideType, args.description, args.details, args.firefoxVersion, journal, args.baseCommit);
+            await saveOverrideConfig({
+                projectRoot: args.projectRoot,
+                destDir: args.destDir,
+                componentName: args.componentName,
+                overrideType: args.overrideType,
+                description: args.description,
+                details: args.details,
+                firefoxVersion: args.firefoxVersion,
+                journal,
+                baseCommit: args.baseCommit,
+            });
             return filesCopied;
         }
         catch (error) {

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

@@ -126,7 +126,15 @@ async function refreshSingleOverride(projectRoot, name, options = {}) {
             for (const entry of overrideFiles) {
                 const overridePath = join(overrideDir, entry.name);
                 const engineRelPath = getOverrideEngineTargetPath(paths.engine, overrideConfig, entry.name, ftlDir).slice(paths.engine.length + 1);
-                const result = await refreshOverrideFile(paths.engine, overridePath, engineRelPath, baseCommit, entry.name, dryRun, strategy);
+                const result = await refreshOverrideFile({
+                    engineDir: paths.engine,
+                    overridePath,
+                    engineRelPath,
+                    baseCommit,
+                    fileName: entry.name,
+                    dryRun,
+                    strategy,
+                });
                 fileResults.push(result);
             }
             // Update baseVersion and baseCommit on clean merge (not dry-run, no conflicts)

package/dist/src/commands/furnace/remove-mutations.d.ts

@@ -1,7 +1,6 @@
-import type { loadFurnaceConfig } from '../../core/furnace-config.js';
-import { getFurnacePaths } from '../../core/furnace-config.js';
+import type { FurnacePaths } from '../../core/furnace-config.js';
 import type { RollbackJournal } from '../../core/furnace-rollback.js';
-import type { FurnaceState } from '../../types/furnace.js';
+import type { FurnaceConfig, FurnaceState } from '../../types/furnace.js';
 /**
  * Engine-side and workspace-side removal for an OVERRIDE component.
  *
@@ -14,8 +13,8 @@ export declare function performOverrideRemovalMutations(args: {
     paths: {
         engine: string;
     };
-    furnacePaths: ReturnType<typeof getFurnacePaths>;
-    freshConfig: Awaited<ReturnType<typeof loadFurnaceConfig>>;
+    furnacePaths: FurnacePaths;
+    freshConfig: FurnaceConfig;
     freshState: FurnaceState;
     ftlDir: string;
     journal: RollbackJournal;
@@ -30,8 +29,8 @@ export declare function performCustomRemovalMutations(args: {
     paths: {
         engine: string;
     };
-    furnacePaths: ReturnType<typeof getFurnacePaths>;
-    freshConfig: Awaited<ReturnType<typeof loadFurnaceConfig>>;
+    furnacePaths: FurnacePaths;
+    freshConfig: FurnaceConfig;
     ftlDir: string;
     journal: RollbackJournal;
 }): Promise<void>;

package/dist/src/commands/furnace/remove-mutations.js

@@ -16,7 +16,6 @@ import { join } from 'node:path';
 import { removeCustomFtlJarMnEntry } from '../../core/furnace-apply-ftl.js';
 import { getOverrideEngineTargetPath, isOverrideCopyCandidate, restoreOverrideFileToBaseline, } from '../../core/furnace-apply-helpers.js';
 import { extractComponentChecksums } from '../../core/furnace-checksum-utils.js';
-import { getFurnacePaths } from '../../core/furnace-config.js';
 import { removeCustomElementRegistration, removeJarMnEntries, } from '../../core/furnace-registration.js';
 import { snapshotDir, snapshotFile } from '../../core/furnace-rollback.js';
 import { isGitRepository } from '../../core/git.js';

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

@@ -16,6 +16,7 @@ import { pathExists, readText, removeDir, writeText } from '../../utils/fs.js';
 import { cancel, info, intro, isCancel, outro, warn } from '../../utils/logger.js';
 import { performCustomRemovalMutations, performOverrideRemovalMutations, } from './remove-mutations.js';
 import { dropChecksumsByPrefix } from './remove-state.js';
+import { browserTestFileName } from './test-file-name.js';
 /**
  * Removes an entire TOML section (header + body lines) for a given test file.
  * Matches from `["filename"]` up to the next section header `[` or end-of-file,
@@ -94,12 +95,7 @@ async function cleanupCustomTestFiles(name, projectRoot, journal) {
     }
     const paths = getProjectPaths(projectRoot);
     const binaryName = forgeConfig.binaryName;
-    const strippedName = name.startsWith('moz-') ? name.slice(4) : name;
-    const withoutBinaryPrefix = strippedName.startsWith(binaryName + '-')
-        ? strippedName.slice(binaryName.length + 1)
-        : strippedName;
-    const underscored = withoutBinaryPrefix.replace(/-/g, '_');
-    const testFileName = `browser_${binaryName}_${underscored}.js`;
+    const testFileName = browserTestFileName(name, binaryName);
     const testDir = join(paths.engine, 'browser/base/content/test', binaryName);
     if (!(await pathExists(testDir)))
         return { partialFailures };

package/dist/src/commands/furnace/rename-browser-test.js

@@ -1,13 +1,7 @@
 // SPDX-License-Identifier: EUPL-1.2
 import { tagNameToClassName } from '../../core/furnace-constants.js';
 import { escapeRegex } from '../../utils/regex.js';
-function deriveTestStem(componentName, binaryName) {
-    const strippedName = componentName.startsWith('moz-') ? componentName.slice(4) : componentName;
-    const withoutBinaryPrefix = strippedName.startsWith(binaryName + '-')
-        ? strippedName.slice(binaryName.length + 1)
-        : strippedName;
-    return withoutBinaryPrefix.replace(/-/g, '_');
-}
+import { deriveTestStem } from './test-file-name.js';
 /** Rewrites scaffolded browser-chrome test literals after a component rename. */
 export function updateBrowserChromeTestContent(content, oldName, newName, binaryName) {
     const oldClassName = tagNameToClassName(oldName);

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

@@ -18,18 +18,7 @@ import { escapeRegex } from '../../utils/regex.js';
 import { updateBrowserChromeTestContent } from './rename-browser-test.js';
 import { rekeyStateChecksums, renameComponentFileName, updateConfigForCustomRename, updateConfigForOverrideRename, } from './rename-helpers.js';
 import { renameXpcshellTestFiles } from './rename-xpcshell.js';
-/**
- * Derives the test file name for a component, matching the convention used by
- * `furnace create --with-tests`.
- */
-function deriveTestFileName(componentName, binaryName) {
-    const strippedName = componentName.startsWith('moz-') ? componentName.slice(4) : componentName;
-    const withoutBinaryPrefix = strippedName.startsWith(binaryName + '-')
-        ? strippedName.slice(binaryName.length + 1)
-        : strippedName;
-    const underscored = withoutBinaryPrefix.replace(/-/g, '_');
-    return `browser_${binaryName}_${underscored}.js`;
-}
+import { browserTestFileName } from './test-file-name.js';
 /**
  * Renames test files created by `furnace create --with-tests` in the engine
  * test directory. Best-effort: failures are logged as warnings but do not
@@ -44,8 +33,8 @@ async function renameTestFiles(engineDir, projectRoot, oldName, newName, journal
         return; // Cannot determine test paths without config.
     }
     const binaryName = forgeConfig.binaryName;
-    const oldTestFileName = deriveTestFileName(oldName, binaryName);
-    const newTestFileName = deriveTestFileName(newName, binaryName);
+    const oldTestFileName = browserTestFileName(oldName, binaryName);
+    const newTestFileName = browserTestFileName(newName, binaryName);
     const testDir = join(engineDir, 'browser/base/content/test', binaryName);
     if (!(await pathExists(testDir)))
         return;
@@ -259,7 +248,15 @@ async function performRenameMutations(args) {
             if (isCustom && config.custom[newName]?.register && (await pathExists(args.engineDir))) {
                 const ftlDir = resolveFtlDir(config.ftlBasePath);
                 const isLocalized = config.custom[newName].localized;
-                await updateEngineRegistrations(args.engineDir, oldName, newName, newDir, ftlDir, isLocalized, journal);
+                await updateEngineRegistrations({
+                    engineDir: args.engineDir,
+                    oldName,
+                    newName,
+                    newDir,
+                    ftlDir,
+                    isLocalized,
+                    journal,
+                });
             }
             // 4. Re-key furnace-state.json checksums from old name to new name
             await rekeyStateChecksums(args.projectRoot, componentType, oldName, newName);
@@ -323,7 +320,14 @@ async function performRenameMutations(args) {
         }
     });
 }
-async function updateEngineRegistrations(engineDir, oldName, newName, newDir, ftlDir, isLocalized, journal) {
+/**
+ * Re-points the engine-side registrations (customElements.js, jar.mn, and
+ * the localized `.ftl` entries) from the old tag name to the new one.
+ *
+ * @param options - See {@link UpdateEngineRegistrationsOptions}
+ */
+async function updateEngineRegistrations(options) {
+    const { engineDir, oldName, newName, newDir, ftlDir, isLocalized, journal } = options;
     const customElementsPath = join(engineDir, 'toolkit/content/customElements.js');
     const jarMnPath = join(engineDir, 'toolkit/content/jar.mn');
     if (await pathExists(customElementsPath)) {
@@ -396,15 +400,15 @@ export async function furnaceRenameCommand(projectRoot, oldName, newName) {
     if (newName in config.custom || newName in config.overrides || config.stock.includes(newName)) {
         throw new FurnaceError(`A component named "${newName}" already exists in furnace.json.`, newName);
     }
-    const componentType = isCustom ? 'custom' : 'override';
-    // `componentType` is the furnace-state key (singular: `custom` /
-    // `override`); the on-disk directory label differs — custom components
-    // live under `components/custom/` (singular) while overrides live under
-    // `components/overrides/` (plural). Appending an `s` to `componentType`
-    // produces the wrong label `components/customs/` for custom components
-    // and is correct for overrides only by coincidence. `componentDirLabel`
-    // centralises the pick so every operator-facing string names the
-    // directory that actually exists on disk.
+    // The furnace-state key is singular (`custom` / `override`, derived
+    // inside performRenameMutations); the on-disk directory label differs —
+    // custom components live under `components/custom/` (singular) while
+    // overrides live under `components/overrides/` (plural). Appending an
+    // `s` to the state key produces the wrong label `components/customs/`
+    // for custom components and is correct for overrides only by
+    // coincidence. `componentDirLabel` centralises the pick so every
+    // operator-facing string names the directory that actually exists on
+    // disk.
     const componentDirLabel = isCustom ? 'custom' : 'overrides';
     const baseDir = isCustom ? furnacePaths.customDir : furnacePaths.overridesDir;
     const oldDir = join(baseDir, oldName);
@@ -419,11 +423,7 @@ export async function furnaceRenameCommand(projectRoot, oldName, newName) {
         projectRoot,
         oldName,
         newName,
-        oldDir,
         newDir,
-        isCustom,
-        componentType,
-        config,
         furnaceConfigPath: furnacePaths.furnaceConfig,
         furnacePaths,
         engineDir: paths.engine,

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

@@ -11,15 +11,10 @@ import { pathExists } from '../../utils/fs.js';
 import { info, intro, note, outro, warn } from '../../utils/logger.js';
 /**
  * Displays detailed status for a single Furnace component, including registration drift.
- * @param name - Component tag name to inspect
- * @param config - Loaded Furnace configuration
- * @param state - Loaded furnace state, for applied/engine checksums
- * @param projectRoot - Root directory of the project
- * @param paths - Resolved project paths
- * @param furnacePaths - Resolved Furnace-specific paths
- * @param ftlDir - Engine-relative directory localized components deploy their `.ftl` into
+ * @param options - See {@link DetailedComponentStatusOptions}
  */
-async function showDetailedComponentStatus(name, config, state, projectRoot, paths, furnacePaths, ftlDir) {
+async function showDetailedComponentStatus(options) {
+    const { name, config, state, projectRoot, paths, furnacePaths, ftlDir } = options;
     const customConfig = config.custom[name];
     const overrideConfig = config.overrides[name];
     if (!customConfig && !overrideConfig && !config.stock.includes(name)) {
@@ -113,7 +108,15 @@ export async function furnaceStatusCommand(projectRoot, name) {
     const furnacePaths = getFurnacePaths(projectRoot);
     const ftlDir = resolveFtlDir(config.ftlBasePath);
     if (name) {
-        await showDetailedComponentStatus(name, config, state, projectRoot, paths, furnacePaths, ftlDir);
+        await showDetailedComponentStatus({
+            name,
+            config,
+            state,
+            projectRoot,
+            paths,
+            furnacePaths,
+            ftlDir,
+        });
         return;
     }
     // Surface a pendingRepair marker before the normal summary so it cannot

package/dist/src/commands/furnace/test-file-name.d.ts

@@ -0,0 +1,28 @@
+/**
+ * Derives the underscored stem a component's generated test files are named
+ * after.
+ *
+ * Five modules (`create-browser-test`, `create-dry-run`, `remove`, `rename`,
+ * `rename-browser-test`) had each inlined this three-step transform. The steps
+ * are not arbitrary and must agree exactly, or `furnace rename` and
+ * `furnace remove` compute a different filename than `furnace create` wrote
+ * and silently leave the old test behind:
+ *
+ * 1. drop the upstream `moz-` custom-element prefix, which never appears in a
+ *    test filename;
+ * 2. drop a leading `<binaryName>-`, because the test directory is already
+ *    named after the binary and the prefix would be doubled;
+ * 3. underscore the remaining dashes, matching mozilla-central's
+ *    `browser_*.js` convention.
+ * @param componentName - Custom-element name, e.g. `moz-mybrowser-sidebar`.
+ * @param binaryName - Configured binary name, e.g. `mybrowser`.
+ * @returns The stem, e.g. `sidebar`.
+ */
+export declare function deriveTestStem(componentName: string, binaryName: string): string;
+/**
+ * Builds the browser-chrome test filename for a component.
+ * @param componentName - Custom-element name.
+ * @param binaryName - Configured binary name.
+ * @returns The filename, e.g. `browser_mybrowser_sidebar.js`.
+ */
+export declare function browserTestFileName(componentName: string, binaryName: string): string;

package/dist/src/commands/furnace/test-file-name.js

@@ -0,0 +1,39 @@
+// SPDX-License-Identifier: EUPL-1.2
+/**
+ * Derives the underscored stem a component's generated test files are named
+ * after.
+ *
+ * Five modules (`create-browser-test`, `create-dry-run`, `remove`, `rename`,
+ * `rename-browser-test`) had each inlined this three-step transform. The steps
+ * are not arbitrary and must agree exactly, or `furnace rename` and
+ * `furnace remove` compute a different filename than `furnace create` wrote
+ * and silently leave the old test behind:
+ *
+ * 1. drop the upstream `moz-` custom-element prefix, which never appears in a
+ *    test filename;
+ * 2. drop a leading `<binaryName>-`, because the test directory is already
+ *    named after the binary and the prefix would be doubled;
+ * 3. underscore the remaining dashes, matching mozilla-central's
+ *    `browser_*.js` convention.
+ * @param componentName - Custom-element name, e.g. `moz-mybrowser-sidebar`.
+ * @param binaryName - Configured binary name, e.g. `mybrowser`.
+ * @returns The stem, e.g. `sidebar`.
+ */
+export function deriveTestStem(componentName, binaryName) {
+    const withoutMozPrefix = componentName.startsWith('moz-')
+        ? componentName.slice(4)
+        : componentName;
+    const withoutBinaryPrefix = withoutMozPrefix.startsWith(`${binaryName}-`)
+        ? withoutMozPrefix.slice(binaryName.length + 1)
+        : withoutMozPrefix;
+    return withoutBinaryPrefix.replace(/-/g, '_');
+}
+/**
+ * Builds the browser-chrome test filename for a component.
+ * @param componentName - Custom-element name.
+ * @param binaryName - Configured binary name.
+ * @returns The filename, e.g. `browser_mybrowser_sidebar.js`.
+ */
+export function browserTestFileName(componentName, binaryName) {
+    return `browser_${binaryName}_${deriveTestStem(componentName, binaryName)}.js`;
+}