@heroiclands/package-build 8.0.0 → 9.0.0

package/sohl/item-builders.mjs

@@ -108,7 +108,5 @@ function entryFor(type) {
  * @type {Readonly<Record<string, Readonly<{system: (fm: object) => object, img: string, fields: readonly object[]}>>>}
  */
 export const ITEM_BUILDERS = Object.freeze(
-    Object.fromEntries(
-        Object.keys(ITEM_FIELDS).map((type) => [type, entryFor(type)]),
-    ),
+    Object.fromEntries(Object.keys(ITEM_FIELDS).map((type) => [type, entryFor(type)])),
 );

package/sohl/item-fields.mjs

@@ -106,8 +106,7 @@ function noteContext(fm, type) {
 /** Aptitude weights per skill selector, validated as whole numbers. */
 const SKILL_APTITUDES = Object.freeze({
     shape: "map of skill selector → whole number",
-    read: (_raw, { fm }) =>
-        resolveSkillAptitudes(fm, noteContext(fm, "mystery")),
+    read: (_raw, { fm }) => resolveSkillAptitudes(fm, noteContext(fm, "mystery")),
 });
 
 /** Standings toward other affiliations, validated against the closed list. */
@@ -127,14 +126,10 @@ const STRIKE_MODES = Object.freeze({
         const seen = new Set();
         for (const { shortcode } of raw) {
             if (!shortcode) {
-                throw new Error(
-                    "weapongear strikeModes array element requires a 'shortcode'",
-                );
+                throw new Error("weapongear strikeModes array element requires a 'shortcode'");
             }
             if (seen.has(shortcode)) {
-                throw new Error(
-                    `weapongear has duplicate strike-mode shortcode "${shortcode}"`,
-                );
+                throw new Error(`weapongear has duplicate strike-mode shortcode "${shortcode}"`);
             }
             seen.add(shortcode);
         }
@@ -218,8 +213,7 @@ const GEAR_COMMON = Object.freeze([
     {
         to: "sharedWithCohortIds",
         value: () => [],
-        describe:
-            "Cohorts sharing the article. Possession state, never authored.",
+        describe: "Cohorts sharing the article. Possession state, never authored.",
     },
     {
         to: "containerId",
@@ -250,8 +244,7 @@ export const ITEM_FIELDS = Object.freeze({
             to: "subType",
             ...SUB_TYPE,
             required: true,
-            describe:
-                "Which kind of affiliation this is — the society's character.",
+            describe: "Which kind of affiliation this is — the society's character.",
         },
         {
             name: "society",
@@ -286,8 +279,7 @@ export const ITEM_FIELDS = Object.freeze({
             to: "relation",
             ...RELATION,
             default: {},
-            describe:
-                "How this society regards others: aligned, unaligned, rival or nemesis.",
+            describe: "How this society regards others: aligned, unaligned, rival or nemesis.",
         },
     ]),
 
@@ -371,16 +363,14 @@ export const ITEM_FIELDS = Object.freeze({
             to: "locations.flexible",
             ...BLANK_IS_DEFAULT,
             default: [],
-            describe:
-                "Body locations the article covers flexibly, by location shortcode.",
+            describe: "Body locations the article covers flexibly, by location shortcode.",
         },
         {
             name: "rigidloc",
             to: "locations.rigid",
             ...BLANK_IS_DEFAULT,
             default: [],
-            describe:
-                "Body locations the article covers rigidly, by location shortcode.",
+            describe: "Body locations the article covers rigidly, by location shortcode.",
         },
         {
             name: "facing",
@@ -463,16 +453,14 @@ export const ITEM_FIELDS = Object.freeze({
             to: "initDiceFormula",
             ...AS_AUTHORED,
             default: "",
-            describe:
-                "Dice expression rolled to generate the score during character creation.",
+            describe: "Dice expression rolled to generate the score during character creation.",
         },
         {
             name: "impairedByRoles",
             to: "impairedByRoles",
             ...AS_AUTHORED,
             default: [],
-            describe:
-                "Body-part roles whose impairment penalises tests against this attribute.",
+            describe: "Body-part roles whose impairment penalises tests against this attribute.",
         },
     ]),
 
@@ -701,24 +689,21 @@ export const ITEM_FIELDS = Object.freeze({
             ref: "skill",
             ...AS_AUTHORED,
             default: "",
-            describe:
-                "Shortcode of the skill this one specialises, for a specialisation.",
+            describe: "Shortcode of the skill this one specialises, for a specialisation.",
         },
         {
             name: "initSkillMult",
             to: "initSkillMult",
             ...NUMBER,
             default: 0,
-            describe:
-                "Multiplier applied to Skill Base when the skill opens on an actor.",
+            describe: "Multiplier applied to Skill Base when the skill opens on an actor.",
         },
         {
             name: "impairedByRoles",
             to: "impairedByRoles",
             ...AS_AUTHORED,
             default: [],
-            describe:
-                "Body-part roles whose impairment penalises tests against this skill.",
+            describe: "Body-part roles whose impairment penalises tests against this skill.",
         },
     ]),
 
@@ -742,8 +727,7 @@ export const ITEM_FIELDS = Object.freeze({
             to: "levelBase",
             ...NULLABLE_COUNT,
             default: null,
-            describe:
-                "Injury level. Unset on a descriptive condition, which has no level.",
+            describe: "Injury level. Unset on a descriptive condition, which has no level.",
         },
         {
             name: "healingRateBase",
@@ -757,16 +741,14 @@ export const ITEM_FIELDS = Object.freeze({
             to: "aspect",
             ...AS_AUTHORED,
             default: null,
-            describe:
-                "How the injury was inflicted. Unset on a descriptive condition.",
+            describe: "How the injury was inflicted. Unset on a descriptive condition.",
         },
         {
             name: "bodyLocationCode",
             to: "bodyLocationCode",
             ...AS_AUTHORED,
             default: null,
-            describe:
-                "Shortcode of the body location injured. Unset on a descriptive condition.",
+            describe: "Shortcode of the body location injured. Unset on a descriptive condition.",
         },
     ]),
 

package/sohl/items.mjs

@@ -189,8 +189,6 @@ export class Items extends BasePackCompiler {
 
     /** @inheritdoc */
     reportDetail(stats) {
-        log.debug(
-            `Skipped ${stats.skippedOther} non-item file(s) (no recognized type)`,
-        );
+        log.debug(`Skipped ${stats.skippedOther} non-item file(s) (no recognized type)`);
     }
 }

package/sohl/kb-passes.mjs

@@ -90,8 +90,7 @@ function readSymbolMap(file, repoRoot) {
         raw = fs.readFileSync(resolved, "utf8");
     } catch (err) {
         throw new Error(
-            `site.passOptions.symbolMap ${resolved} cannot be read: ` +
-                `${err.message}`,
+            `site.passOptions.symbolMap ${resolved} cannot be read: ` + `${err.message}`,
         );
     }
 
@@ -100,8 +99,7 @@ function readSymbolMap(file, repoRoot) {
         parsed = JSON.parse(raw);
     } catch (err) {
         throw new Error(
-            `site.passOptions.symbolMap ${resolved} is not valid JSON: ` +
-                `${err.message}`,
+            `site.passOptions.symbolMap ${resolved} is not valid JSON: ` + `${err.message}`,
         );
     }
 
@@ -140,39 +138,34 @@ function readSymbolMap(file, repoRoot) {
  * @returns {string} The body with every tag resolved.
  */
 export function resolveApiLinks(body, symbols, apiBase) {
-    return body.replace(
-        /\{@link(code|plain)?\s+([^}]+)\}/g,
-        (_m, kind, inner) => {
-            inner = inner.trim();
-            let target, text;
-            const pipe = inner.indexOf("|");
-            const space = inner.search(/\s/);
-            if (pipe !== -1) {
-                target = inner.slice(0, pipe).trim();
-                text = inner.slice(pipe + 1).trim();
-            } else if (space !== -1) {
-                target = inner.slice(0, space);
-                text = inner.slice(space + 1).trim();
-            } else {
-                target = inner;
-                text = "";
-            }
+    return body.replace(/\{@link(code|plain)?\s+([^}]+)\}/g, (_m, kind, inner) => {
+        inner = inner.trim();
+        let target, text;
+        const pipe = inner.indexOf("|");
+        const space = inner.search(/\s/);
+        if (pipe !== -1) {
+            target = inner.slice(0, pipe).trim();
+            text = inner.slice(pipe + 1).trim();
+        } else if (space !== -1) {
+            target = inner.slice(0, space);
+            text = inner.slice(space + 1).trim();
+        } else {
+            target = inner;
+            text = "";
+        }
 
-            if (/^https?:\/\//.test(target)) {
-                return `[${text || target}](${target})`;
-            }
+        if (/^https?:\/\//.test(target)) {
+            return `[${text || target}](${target})`;
+        }
 
-            const url = symbols[target];
-            const display = text || target.split(".").pop();
-            if (url) {
-                const href = apiBase + url;
-                return kind === "code" ?
-                        `[\`${display}\`](${href})`
-                    :   `[${display}](${href})`;
-            }
-            return kind === "plain" ? display : `\`${display}\``;
-        },
-    );
+        const url = symbols[target];
+        const display = text || target.split(".").pop();
+        if (url) {
+            const href = apiBase + url;
+            return kind === "code" ? `[\`${display}\`](${href})` : `[${display}](${href})`;
+        }
+        return kind === "plain" ? display : `\`${display}\``;
+    });
 }
 
 /**
@@ -218,10 +211,7 @@ export function rewriteRepoLinks(body, docRel, options) {
         let out;
         if (repoRel.startsWith(`${docsRel}/`) && repoRel.endsWith(".md")) {
             const rel2 = repoRel.slice(docsRel.length + 1, -3).toLowerCase();
-            const devPath =
-                path.basename(rel2) === "readme" ?
-                    path.posix.dirname(rel2)
-                :   rel2;
+            const devPath = path.basename(rel2) === "readme" ? path.posix.dirname(rel2) : rel2;
             out = `${route}${devPath === "." ? "" : `${devPath}/`}${anchor}`;
         } else {
             out = `${blob}${repoRel}${anchor}`;

package/sohl/skill-base.mjs

@@ -116,9 +116,7 @@ function evalNode(node, attrs) {
     switch (node.type) {
         case "Literal": {
             if (typeof node.value !== "number") {
-                throw new Error(
-                    `unsupported literal ${JSON.stringify(node.value)}`,
-                );
+                throw new Error(`unsupported literal ${JSON.stringify(node.value)}`);
             }
             return node.value;
         }
@@ -126,9 +124,7 @@ function evalNode(node, attrs) {
             // Only `attr.<code>` and `attr["<code>"]`. Any other object, and
             // any computed key that is not a plain string, is out of scope.
             if (node.object?.type !== "Identifier") {
-                throw new Error(
-                    "only `attr.<code>` member reads are supported",
-                );
+                throw new Error("only `attr.<code>` member reads are supported");
             }
             if (node.object.name !== "attr") {
                 throw new Error(
@@ -136,13 +132,8 @@ function evalNode(node, attrs) {
                 );
             }
             if (node.computed) {
-                if (
-                    node.property.type !== "Literal" ||
-                    typeof node.property.value !== "string"
-                ) {
-                    throw new Error(
-                        "a computed `attr[...]` read needs a literal string shortcode",
-                    );
+                if (node.property.type !== "Literal" || typeof node.property.value !== "string") {
+                    throw new Error("a computed `attr[...]` read needs a literal string shortcode");
                 }
                 return readAttr(attrs, node.property.value);
             }
@@ -150,15 +141,11 @@ function evalNode(node, attrs) {
         }
         case "CallExpression": {
             if (node.callee?.type !== "Identifier") {
-                throw new Error(
-                    "only direct calls to a named helper are supported",
-                );
+                throw new Error("only direct calls to a named helper are supported");
             }
             const helper = HELPERS[node.callee.name];
             if (!helper) {
-                throw new Error(
-                    `unknown helper "${node.callee.name}()" in a skill base formula`,
-                );
+                throw new Error(`unknown helper "${node.callee.name}()" in a skill base formula`);
             }
             return helper(...node.arguments.map((a) => evalNode(a, attrs)));
         }
@@ -200,10 +187,7 @@ export function evaluateSkillBase(formula, attrs = {}) {
     let node;
     try {
         const program = parse(source, { ecmaVersion: 2022 });
-        if (
-            program.body.length !== 1 ||
-            program.body[0].type !== "ExpressionStatement"
-        ) {
+        if (program.body.length !== 1 || program.body[0].type !== "ExpressionStatement") {
             return {
                 value: 0,
                 error: `skill base formula "${source}" is not a single expression`,

package/stage.mjs

@@ -46,12 +46,7 @@ import path from "node:path";
  * beneath `site/` — but these four are common to all of them because they come
  * from the shared toolchain rather than from any one package's layout.
  */
-export const BUILD_ARTIFACT_DIRS = Object.freeze([
-    "build",
-    ".vite",
-    ".vitepress",
-    ".rollup.cache",
-]);
+export const BUILD_ARTIFACT_DIRS = Object.freeze(["build", ".vite", ".vitepress", ".rollup.cache"]);
 
 /**
  * A source that does not exist, described for a human.
@@ -66,9 +61,7 @@ export const BUILD_ARTIFACT_DIRS = Object.freeze([
  * @returns {string[]} Every source that is absent, in the order listed.
  */
 export function missingSources(entries, cwd = process.cwd()) {
-    return entries
-        .map(([src]) => src)
-        .filter((src) => !fs.existsSync(path.resolve(cwd, src)));
+    return entries.map(([src]) => src).filter((src) => !fs.existsSync(path.resolve(cwd, src)));
 }
 
 /**
@@ -157,10 +150,7 @@ export function stageAssets(entries, { cwd = process.cwd(), transform } = {}) {
  *   `distclean` case.
  * @returns {string[]} The directories removed, as listed.
  */
-export function cleanBuildArtifacts(
-    root,
-    { extra = [], includeNodeModules = false } = {},
-) {
+export function cleanBuildArtifacts(root, { extra = [], includeNodeModules = false } = {}) {
     const dirs = [
         ...BUILD_ARTIFACT_DIRS,
         ...extra,

package/templates.mjs

@@ -107,10 +107,7 @@ function stripNonProse(source) {
  * @returns {boolean} Whether it should have been localized.
  */
 function isProse(text, allowed) {
-    return (
-        /[A-Za-z]{2}/.test(text.replace(/&[a-zA-Z]+;|&#\d+;/g, " ")) &&
-        !allowed.has(text)
-    );
+    return /[A-Za-z]{2}/.test(text.replace(/&[a-zA-Z]+;|&#\d+;/g, " ")) && !allowed.has(text);
 }
 
 /**
@@ -201,23 +198,15 @@ export function findTemplateSyntaxErrors(source) {
         // answer, in the only place this error carries it.
         const loc = err?.hash?.loc;
         const stated = /Parse error on line (\d+)/.exec(String(err?.message));
-        const line =
-            loc?.first_line ??
-            err?.lineNumber ??
-            (stated ? Number(stated[1]) : undefined);
+        const line = loc?.first_line ?? err?.lineNumber ?? (stated ? Number(stated[1]) : undefined);
         return [
             {
                 ...(typeof line === "number" ? { line } : {}),
-                ...((
-                    typeof line === "number" &&
-                    typeof loc?.first_column === "number"
-                ) ?
+                ...(typeof line === "number" && typeof loc?.first_column === "number" ?
                     { column: loc.first_column + 1 }
                 :   {}),
                 severity: "error",
-                message: `template does not compile: ${
-                    String(err).split("\n")[0]
-                }`,
+                message: `template does not compile: ${String(err).split("\n")[0]}`,
             },
         ];
     }

package/types/bundle.d.mts

@@ -45,7 +45,7 @@ export function globalDeclarations(source: string): Array<{
  *   declaredAs: "esmodules"|"scripts"|"both"|"neither"}} The findings, empty
  *   when the two agree, and how the entry was declared.
  */
-export function checkBundleLoading({ manifest, source, entry, manifestName, }: {
+export function checkBundleLoading({ manifest, source, entry, manifestName }: {
     manifest: object;
     source: string;
     entry: string;

package/types/container.d.mts

@@ -82,7 +82,7 @@ export function resolveStagePort(stage: string, { env, stages }?: {
  * @param {string} [opts.e2eStage] - Which stage the suite runs against.
  * @returns {string|null} The exact build, or `null`.
  */
-export function resolveFoundryVersion(stage: string, { env, stages, compatibilityMinimum, e2eStage, }?: {
+export function resolveFoundryVersion(stage: string, { env, stages, compatibilityMinimum, e2eStage }?: {
     env?: NodeJS.ProcessEnv | undefined;
     stages?: Record<string, ContainerStage> | undefined;
     compatibilityMinimum?: string | null | undefined;
@@ -305,7 +305,7 @@ export function removeContainer(name: string, log?: (message: string) => void):
  * @returns {number} The exit status.
  * @throws {Error} On an unknown action, or a stage with no usable data root.
  */
-export function containerAction({ action, stage, config, env, log, }: {
+export function containerAction({ action, stage, config, env, log }: {
     action: string;
     stage: string;
     config: object;

package/types/coverage.d.mts

@@ -77,7 +77,7 @@ export function mergeReferences(sets: Iterable<ReferenceSet>): ReferenceSet;
  *   questions: one says the package is broken, the other that it carries
  *   something nobody could see a use for.
  */
-export function analyzeCoverage({ langSource, langFile, references, retained, roots, }: {
+export function analyzeCoverage({ langSource, langFile, references, retained, roots }: {
     langSource: string;
     langFile: string;
     references: ReferenceSet;

package/types/e2e.d.mts

@@ -156,7 +156,7 @@ export function parseFastArgs(argv: string[], build: Record<string, {
  * @param {(message: string) => void} [opts.log] - Progress reporting.
  * @returns {Promise<{worldDir: string, world: E2EWorld}>} Where it landed.
  */
-export function seedTestWorld({ config, packageJson, env, log, }: {
+export function seedTestWorld({ config, packageJson, env, log }: {
     config: object;
     packageJson: object;
     env?: NodeJS.ProcessEnv | undefined;
@@ -181,7 +181,7 @@ export function seedTestWorld({ config, packageJson, env, log, }: {
  * @returns {Promise<void>} Resolves once the world is active.
  * @throws {Error} On a licence failure or a timeout.
  */
-export function waitForWorld({ url, container, stage, timeoutMs, log, }: {
+export function waitForWorld({ url, container, stage, timeoutMs, log }: {
     url: string;
     container: string;
     stage: string;
@@ -204,7 +204,7 @@ export function waitForWorld({ url, container, stage, timeoutMs, log, }: {
  * @param {(message: string) => void} [opts.log] - Progress reporting.
  * @returns {number} The suite's exit status.
  */
-export function runSuite({ command, args, cwd, env, log, }: {
+export function runSuite({ command, args, cwd, env, log }: {
     command: string[];
     args?: string[] | undefined;
     cwd: string;
@@ -265,7 +265,7 @@ export function e2eRun({ config, packageJson, mode, suiteArgs, env, log, }: {
  * @param {(message: string) => void} [opts.log] - Progress reporting.
  * @returns {Promise<number>} The suite's exit status.
  */
-export function e2eFast({ config, argv, env, log, }: {
+export function e2eFast({ config, argv, env, log }: {
     config: object;
     argv?: string[] | undefined;
     env?: NodeJS.ProcessEnv | undefined;

package/types/engine/generate.d.mts

@@ -98,7 +98,7 @@ export function emptyPassErrors(passes: Array<{
  * @throws {Error} If the configured Foundry package id has drifted from the
  *   shipped manifest's `id` (see `package-manifest.mjs`).
  */
-export function generatePacksJson({ only, config, }?: {
+export function generatePacksJson({ only, config }?: {
     only?: string | undefined;
     config?: object | undefined;
 }): Promise<number>;

package/types/engine/helpers.d.mts

@@ -276,7 +276,7 @@ export function buildContentLinkIndex(contentBase: string, router?: object): {
  *   carries `file` and `position`, so a caller reports it in the same form
  *   rather than re-deriving one.
  */
-export function convertNoteWikilinks(body: string, { type, id, pack, docPack, index, name, file, bodyLine, bodyColumn, lineMap, }: object): {
+export function convertNoteWikilinks(body: string, { type, id, pack, docPack, index, name, file, bodyLine, bodyColumn, lineMap }: object): {
     markdown: string;
     unresolved: Array<object>;
 };

package/types/engine/schema-extract.d.mts

@@ -52,7 +52,7 @@ export function fieldsForClass({ file, className, aliases, cache, rootDir }: {
  * @param {string} opts.version - The package version.
  * @returns {object} The artifact `schema-check.mjs` reads.
  */
-export function buildSchemaArtifact({ rootDir, registries, packageId, version, }: {
+export function buildSchemaArtifact({ rootDir, registries, packageId, version }: {
     rootDir: string;
     registries: object[];
     packageId: string;

package/types/engine/site-index.d.mts

@@ -37,7 +37,7 @@ export function buildSiteIndex(entries: readonly SiteEntry[], { foreignIndex }?:
  *   unresolved cross-package link rather than fail.
  * @returns {object} The resolver context.
  */
-export function wikiContext(built: SiteIndex, { src, type, errors, foreignIndex, manifestsComplete, }: {
+export function wikiContext(built: SiteIndex, { src, type, errors, foreignIndex, manifestsComplete }: {
     src: string;
     type?: string | null | undefined;
     errors: object[];

package/types/manifest.d.mts

@@ -182,7 +182,7 @@ export function buildManifest({ config, packageJson, artifact, flags }: {
  * @throws {Error} When a `packFolders` entry names a pack the package does not
  *   ship. Nothing is written in that case.
  */
-export function writeManifest({ config, packageJson, artifact, outDir, flags, configFile, }: {
+export function writeManifest({ config, packageJson, artifact, outDir, flags, configFile }: {
     config: object;
     packageJson: object;
     artifact: string;