@heroiclands/package-build 22.21.0 → 22.23.0

package/hm3/actors.mjs

@@ -78,7 +78,7 @@ import {
  *
  * @type {readonly import("../engine/field-spec.mjs").FieldSpec[]}
  */
-const ACTOR_FIELDS = Object.freeze([
+export const ACTOR_FIELDS = Object.freeze([
     {
         name: "species",
         to: "species",
@@ -97,7 +97,7 @@ const ACTOR_FIELDS = Object.freeze([
  *
  * @type {readonly import("../engine/field-spec.mjs").FieldSpec[]}
  */
-const CHARACTER_FIELDS = Object.freeze([
+export const CHARACTER_FIELDS = Object.freeze([
     {
         name: "data.gender",
         legacyKey: "gender",
@@ -227,9 +227,9 @@ export class Hm3Actors extends SystemActorCompiler {
     /**
      * Build every embedded item an HM3 actor carries, from `hm3.items`.
      *
-     * One list, not two: HM3 keeps a character's abilities in `system.abilities`
-     * rather than as embedded documents, so there is no attributes map to
-     * expand the way SoHL's pass expands `sohl.attributes`.
+     * HM3 keeps a character's abilities in `system.abilities` rather than as
+     * embedded documents, so an ability reaches the document through the
+     * `hm3.system` passthrough and never through this list.
      *
      * @param {Map<string, object>} itemsMap - The predefined items, by address.
      * @param {string} actorId - The owning actor's id, seeding embedded ids.
@@ -333,6 +333,9 @@ export class Hm3Actors extends SystemActorCompiler {
             isAddress: true,
         });
 
+        this.checkAnchoredSection(body || "", "appearance", ctx);
+        this.checkAnchoredSection(body || "", "dossier", ctx);
+
         const system = {
             // `system.bioImage` is **not** written here, for the reason SoHL's
             // `system.portrait` is not: a being's portrait is the lead image of

package/manifest.mjs

@@ -48,10 +48,12 @@
  * @module
  */
 
+import { RELATIONSHIP_KEYS } from "./content-config.mjs";
 import fs from "node:fs/promises";
 import fsSync from "node:fs";
 import path from "node:path";
 
+import { withBaseStyle } from "./engine/base-styles.mjs";
 import { emitDiagnostic, formatDiagnostic, positionOfYamlPath } from "./engine/diagnostics.mjs";
 import { metadataFileName } from "./engine/metadata-index.mjs";
 
@@ -389,33 +391,38 @@ export function packFolderFindings({ packFolders, packs = [] }) {
 }
 
 /**
- * Relationship keys that direct the **build**, rather than describe the
- * package.
- *
- * `relationships` is the one manifest block with a second reader.
- * `@heroiclands/package-build` consumes it too, and v1.8.0 added
- * `itemCatalog: true` as an opt-in on a declared dependency: it selects that
- * package's Item packs as a resolution
- * source for the actors pass. That is an instruction to the build, not a fact
- * about the shipped package — Foundry's relationship schema does not define
- * it, and someone reading a published manifest cannot tell a build directive
- * from a declaration about what the package needs.
- *
- * So the block is filtered rather than copied whole. The rule is the
- * distinction, not the name: a key listed here answers *how is this built?*,
- * and every key that survives answers *what does this package depend on?*.
- * `itemCatalog` is the first build-time key to land on a relationship and is
- * unlikely to be the last.
- *
- * A list is enough, and needs no prefix agreed between the two packages,
- * because the input is already closed: package-build normalises a relationship
- * to `id`, `type`, `manifest`, `compatibility` and its own build keys, and
- * rejects anything else at configuration time. A key that reaches here is one
- * the toolchain itself put there.
+ * The relationship keys Foundry's own schema names.
+ *
+ * `RelatedPackage` declares exactly these (`common/packages/base-package.mjs`),
+ * so a key outside them is one the build put there for itself.
+ *
+ * @type {readonly string[]}
+ */
+export const FOUNDRY_RELATIONSHIP_KEYS = Object.freeze([
+    "id",
+    "type",
+    "manifest",
+    "compatibility",
+    "reason",
+]);
+
+/**
+ * The relationship keys the published manifest drops.
+ *
+ * Derived rather than listed: every key this package accepts on a relationship
+ * that Foundry's schema does not name is one the build reads for itself, so a
+ * key added to the configuration is dropped from the manifest without a second
+ * list being kept in step.
+ *
+ * The filter stays a denylist for the reason the pass-through below gives — a
+ * key Foundry adds later reaches the manifest unread rather than being stripped
+ * by an allowlist that has not heard of it.
  *
  * @type {readonly string[]}
  */
-export const BUILD_ONLY_RELATIONSHIP_KEYS = Object.freeze(["itemCatalog"]);
+export const BUILD_ONLY_RELATIONSHIP_KEYS = Object.freeze(
+    RELATIONSHIP_KEYS.filter((key) => !FOUNDRY_RELATIONSHIP_KEYS.includes(key)),
+);
 
 /**
  * The `relationships` block as published — every declared dependency, with the
@@ -548,6 +555,18 @@ export function buildManifest({ config, packageJson, artifact, flags }) {
 
     const merged = { ...declared, ...derived };
 
+    // The shared base stylesheet, named ahead of whatever the package declares.
+    // Derived rather than declared, because every package needs it and four of
+    // the five that do have no stylesheet of their own to name it from. See
+    // `engine/base-styles.mjs` for the layer it loads into and why that is what
+    // lets a package override it.
+    const styles = withBaseStyle({
+        declared: declared.styles,
+        artifact: /** @type {"system"|"module"} */ (artifact),
+        enabled: config.packageBuild?.baseStyles !== false,
+    });
+    if (styles !== undefined) merged.styles = styles;
+
     // The index every consumer resolves this package's addresses through.
     // Written unconditionally, because a package that publishes no
     // index is one nothing can link into — and the failure of an absent one is

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@heroiclands/package-build",
-    "version": "22.21.0",
+    "version": "22.23.0",
     "description": "Shared toolchain for building and shipping a HeroicLands Foundry VTT package — content compilation, manifest, localization, staging, bundle, release and deployment.",
     "license": "GPL-3.0-or-later",
     "type": "module",

package/packagebuild-metadata.jsonl

@@ -1,13 +1,13 @@
-{"address":{"canonical":"packagebuild-none-image-afflictionbnr"},"asset":{"ai":"","attribution":"Tom Rodriguez","license":"CC-BY-SA-4.0","notes":"","path":"images/banners/afflictionbnr.webp","source":""},"package":"packagebuild","shortcode":"afflictionbnr","type":"image"}
-{"address":{"canonical":"packagebuild-none-image-armorclothingbnr"},"asset":{"ai":"","attribution":"Tom Rodriguez","license":"CC-BY-SA-4.0","notes":"","path":"images/banners/armorclothingbnr.webp","source":""},"package":"packagebuild","shortcode":"armorclothingbnr","type":"image"}
-{"address":{"canonical":"packagebuild-none-image-attributebnr"},"asset":{"ai":"","attribution":"Tom Rodriguez","license":"CC-BY-SA-4.0","notes":"","path":"images/banners/attributebnr.webp","source":""},"package":"packagebuild","shortcode":"attributebnr","type":"image"}
-{"address":{"canonical":"packagebuild-none-image-containerbnr"},"asset":{"ai":"","attribution":"Tom Rodriguez","license":"CC-BY-SA-4.0","notes":"","path":"images/banners/containerbnr.webp","source":""},"package":"packagebuild","shortcode":"containerbnr","type":"image"}
-{"address":{"canonical":"packagebuild-none-image-devdocsbnr"},"asset":{"ai":"","attribution":"Tom Rodriguez","license":"CC-BY-SA-4.0","notes":"","path":"images/banners/devdocsbnr.webp","source":""},"package":"packagebuild","shortcode":"devdocsbnr","type":"image"}
-{"address":{"canonical":"packagebuild-none-image-miscgearbnr"},"asset":{"ai":"","attribution":"Tom Rodriguez","license":"CC-BY-SA-4.0","notes":"","path":"images/banners/miscgearbnr.webp","source":""},"package":"packagebuild","shortcode":"miscgearbnr","type":"image"}
-{"address":{"canonical":"packagebuild-none-image-mysticalabilitybnr"},"asset":{"ai":"","attribution":"Tom Rodriguez","license":"CC-BY-SA-4.0","notes":"","path":"images/banners/mysticalabilitybnr.webp","source":""},"package":"packagebuild","shortcode":"mysticalabilitybnr","type":"image"}
-{"address":{"canonical":"packagebuild-none-image-projectilebnr"},"asset":{"ai":"","attribution":"Tom Rodriguez","license":"CC-BY-SA-4.0","notes":"","path":"images/banners/projectilebnr.webp","source":""},"package":"packagebuild","shortcode":"projectilebnr","type":"image"}
-{"address":{"canonical":"packagebuild-none-image-rulesbnr"},"asset":{"ai":"","attribution":"Tom Rodriguez","license":"CC-BY-SA-4.0","notes":"","path":"images/banners/rulesbnr.webp","source":""},"package":"packagebuild","shortcode":"rulesbnr","type":"image"}
-{"address":{"canonical":"packagebuild-none-image-skillbnr"},"asset":{"ai":"","attribution":"Tom Rodriguez","license":"CC-BY-SA-4.0","notes":"","path":"images/banners/skillbnr.webp","source":""},"package":"packagebuild","shortcode":"skillbnr","type":"image"}
-{"address":{"canonical":"packagebuild-none-image-traumabnr"},"asset":{"ai":"","attribution":"Tom Rodriguez","license":"CC-BY-SA-4.0","notes":"","path":"images/banners/traumabnr.webp","source":""},"package":"packagebuild","shortcode":"traumabnr","type":"image"}
-{"address":{"canonical":"packagebuild-none-image-userguidebnr"},"asset":{"ai":"","attribution":"Tom Rodriguez","license":"CC-BY-SA-4.0","notes":"","path":"images/banners/userguidebnr.webp","source":""},"package":"packagebuild","shortcode":"userguidebnr","type":"image"}
-{"address":{"canonical":"packagebuild-none-image-weapongearbnr"},"asset":{"ai":"","attribution":"Tom Rodriguez","license":"CC-BY-SA-4.0","notes":"","path":"images/banners/weapongearbnr.webp","source":""},"package":"packagebuild","shortcode":"weapongearbnr","type":"image"}
+{"address":{"canonical":"packagebuild-none-image-afflictionbnr"},"asset":{"ai":false,"attribution":"Tom Rodriguez","height":768,"license":"CC-BY-SA-4.0","notes":"","path":"images/banners/afflictionbnr.webp","role":"","source":"","width":1792},"package":"packagebuild","shortcode":"afflictionbnr","type":"image"}
+{"address":{"canonical":"packagebuild-none-image-armorclothingbnr"},"asset":{"ai":false,"attribution":"Tom Rodriguez","height":768,"license":"CC-BY-SA-4.0","notes":"","path":"images/banners/armorclothingbnr.webp","role":"","source":"","width":1792},"package":"packagebuild","shortcode":"armorclothingbnr","type":"image"}
+{"address":{"canonical":"packagebuild-none-image-attributebnr"},"asset":{"ai":false,"attribution":"Tom Rodriguez","height":768,"license":"CC-BY-SA-4.0","notes":"","path":"images/banners/attributebnr.webp","role":"","source":"","width":1792},"package":"packagebuild","shortcode":"attributebnr","type":"image"}
+{"address":{"canonical":"packagebuild-none-image-containerbnr"},"asset":{"ai":false,"attribution":"Tom Rodriguez","height":768,"license":"CC-BY-SA-4.0","notes":"","path":"images/banners/containerbnr.webp","role":"","source":"","width":1792},"package":"packagebuild","shortcode":"containerbnr","type":"image"}
+{"address":{"canonical":"packagebuild-none-image-devdocsbnr"},"asset":{"ai":false,"attribution":"Tom Rodriguez","height":768,"license":"CC-BY-SA-4.0","notes":"","path":"images/banners/devdocsbnr.webp","role":"","source":"","width":1792},"package":"packagebuild","shortcode":"devdocsbnr","type":"image"}
+{"address":{"canonical":"packagebuild-none-image-miscgearbnr"},"asset":{"ai":false,"attribution":"Tom Rodriguez","height":768,"license":"CC-BY-SA-4.0","notes":"","path":"images/banners/miscgearbnr.webp","role":"","source":"","width":1792},"package":"packagebuild","shortcode":"miscgearbnr","type":"image"}
+{"address":{"canonical":"packagebuild-none-image-mysticalabilitybnr"},"asset":{"ai":false,"attribution":"Tom Rodriguez","height":768,"license":"CC-BY-SA-4.0","notes":"","path":"images/banners/mysticalabilitybnr.webp","role":"","source":"","width":1792},"package":"packagebuild","shortcode":"mysticalabilitybnr","type":"image"}
+{"address":{"canonical":"packagebuild-none-image-projectilebnr"},"asset":{"ai":false,"attribution":"Tom Rodriguez","height":768,"license":"CC-BY-SA-4.0","notes":"","path":"images/banners/projectilebnr.webp","role":"","source":"","width":1792},"package":"packagebuild","shortcode":"projectilebnr","type":"image"}
+{"address":{"canonical":"packagebuild-none-image-rulesbnr"},"asset":{"ai":false,"attribution":"Tom Rodriguez","height":768,"license":"CC-BY-SA-4.0","notes":"","path":"images/banners/rulesbnr.webp","role":"","source":"","width":1792},"package":"packagebuild","shortcode":"rulesbnr","type":"image"}
+{"address":{"canonical":"packagebuild-none-image-skillbnr"},"asset":{"ai":false,"attribution":"Tom Rodriguez","height":768,"license":"CC-BY-SA-4.0","notes":"","path":"images/banners/skillbnr.webp","role":"","source":"","width":1792},"package":"packagebuild","shortcode":"skillbnr","type":"image"}
+{"address":{"canonical":"packagebuild-none-image-traumabnr"},"asset":{"ai":false,"attribution":"Tom Rodriguez","height":768,"license":"CC-BY-SA-4.0","notes":"","path":"images/banners/traumabnr.webp","role":"","source":"","width":1792},"package":"packagebuild","shortcode":"traumabnr","type":"image"}
+{"address":{"canonical":"packagebuild-none-image-userguidebnr"},"asset":{"ai":false,"attribution":"Tom Rodriguez","height":768,"license":"CC-BY-SA-4.0","notes":"","path":"images/banners/userguidebnr.webp","role":"","source":"","width":1792},"package":"packagebuild","shortcode":"userguidebnr","type":"image"}
+{"address":{"canonical":"packagebuild-none-image-weapongearbnr"},"asset":{"ai":false,"attribution":"Tom Rodriguez","height":768,"license":"CC-BY-SA-4.0","notes":"","path":"images/banners/weapongearbnr.webp","role":"","source":"","width":1792},"package":"packagebuild","shortcode":"weapongearbnr","type":"image"}

package/sohl/actors.mjs

@@ -18,10 +18,9 @@
  * reference translation through the system's map, embedding and its stable ids,
  * the anchored prose sections — lives in
  * {@link module:engine/actor-compiler}, where a second system reaches it.
- * What is left here is SoHL's data model: the body structure and its
- * movement profiles, the attributes-and-items frontmatter that becomes embedded
- * documents, the opening mastery level a skill is baked with, and the `system`
- * block itself.
+ * What is left here is SoHL's data model: the `sohl.items` frontmatter that
+ * becomes embedded documents, the opening mastery level a skill is baked with,
+ * and the `system` block itself.
  *
  * One content type today, and the actor subtype it produces is **declared**
  * rather than assumed to be the same word: `sohl/document-subtypes.mjs` maps
@@ -29,10 +28,16 @@
  * `ACTOR_VAULT_TYPE = "being"` here and emitted `type: "being"` several hundred
  * lines below, which made the two vocabularies agree by coincidence.
  *
- * Attributes (`sohl.attributes` map) become embedded attribute items with
- * `scoreBase` set from the map value. Each entry in `sohl.items` is similarly
- * resolved by `(type, shortcode)` and deep-merged with the entry's other
- * properties. `sohl.skills` is ignored.
+ * Each entry in `sohl.items` is resolved by `(type, shortcode)` and deep-merged
+ * with the entry's other properties, so an attribute carries its `scoreBase`
+ * and a skill its `masteryLevelBase` as that entry's own `system` overlay.
+ *
+ * **A being's own document fields are authored under `sohl.system`**, which
+ * {@link module:engine/system-block.mergeSystemData} writes at the DataModel's
+ * own paths: a being's body is `sohl.system.body`, with
+ * `sohl.system.currentMoveMedium` and `sohl.system.movementProfiles` beside it.
+ * So this pass writes no body itself, and an incorporeal being authors none and
+ * keeps the schema's empty one.
  *
  * **A membership is derived, never authored here.** `data.affiliations` names
  * the bodies a being belongs to and the standing it holds in each, and each
@@ -105,60 +110,12 @@ function defaultActorImg(subType) {
     return img;
 }
 
-/**
- * Normalize a being's persisted `system.body` from a `sohl.body` block. The
- * authoring frontmatter mirrors the schema field-for-field: `sohl.body` nests
- * `structure` / `weight` / `reachBase` / `bodyScaleBase` / `personalFatigue`,
- * exactly like `system.body`.
- */
-function normalizeBody(bodyObj) {
-    const b = bodyObj && typeof bodyObj === "object" ? bodyObj : {};
-    const weight = b.weight || {};
-    return {
-        structure: b.structure ?? { parts: [], adjacent: [] },
-        weight: {
-            base: weight.base == null ? null : Number(weight.base),
-            calc: String(weight.calc ?? "0"),
-        },
-        reachBase: Number(b.reachBase ?? 0) || 0,
-        bodyScaleBase: Number(b.bodyScaleBase ?? 1) || 1,
-        personalFatigue: String(b.personalFatigue ?? "enc"),
-    };
-}
-
-/** Normalize per-medium movement profiles from a `sohl.movementProfiles` list. */
-function normalizeMovementProfiles(list) {
-    return (Array.isArray(list) ? list : []).map((p) => ({
-        medium: String(p.medium ?? "terrestrial"),
-        feetPerRound: Number(p.feetPerRound ?? 0) || 0,
-        leaguesPerWatch: Number(p.leaguesPerWatch ?? 0) || 0,
-        encumbrance: String(p.encumbrance ?? "0"),
-        strMod: String(p.strMod ?? "0"),
-        disabled: Boolean(p.disabled ?? false),
-    }));
-}
-
-/**
- * Extract a being's body (+ its movement) from a `sohl` block that mirrors the
- * schema: `sohl.body` (nested → `system.body`) and the flat
- * `sohl.currentMoveMedium` / `sohl.movementProfiles` (→ the base-actor movement
- * fields; movement is a universal actor capability, not part of the body).
- */
-function extractBodyAndMovement(fm) {
-    return {
-        body: normalizeBody(sohlField(fm, "body", {})),
-        currentMoveMedium: String(sohlField(fm, "currentMoveMedium", "none")),
-        movementProfiles: normalizeMovementProfiles(sohlField(fm, "movementProfiles", [])),
-    };
-}
-
 /**
  * SoHL's Actor compile pass.
  *
  * Declares SoHL's note-type → document-subtype map and builds a `being` note's
- * document: the body structure and its movement profiles, the embedded
- * attribute and item documents the frontmatter names, and the `system` block.
- * Everything else is {@link module:engine/actor-compiler}'s.
+ * document: the embedded item documents the frontmatter names, and the `system`
+ * block. Everything else is {@link module:engine/actor-compiler}'s.
  */
 export class Actors extends SystemActorCompiler {
     /**
@@ -184,30 +141,12 @@ export class Actors extends SystemActorCompiler {
     }
 
     /**
-     * Build all embedded items for an actor: one per `sohl.attributes`
-     * entry plus one per `sohl.items` entry. `sohl.skills` is ignored.
+     * Build all embedded items for an actor: one per `sohl.items` entry, plus
+     * one per body the being belongs to.
      */
     buildEmbeddedItems(itemsMap, actorId, fm, ctx) {
         const items = [];
 
-        const attributes = sohlField(fm, "attributes", null);
-        if (attributes && typeof attributes === "object") {
-            for (const [shortcode, value] of Object.entries(attributes)) {
-                const overlay = { system: { scoreBase: Number(value) || 0 } };
-                const embedded = this.resolveEmbedded(
-                    itemsMap,
-                    actorId,
-                    "attribute",
-                    shortcode,
-                    overlay,
-                    `attr:${shortcode}`,
-                    ctx,
-                    { fmKey: "attributes" },
-                );
-                if (embedded) items.push(embedded);
-            }
-        }
-
         const sohlItems = sohlField(fm, "items", null);
         if (Array.isArray(sohlItems)) {
             sohlItems.forEach((entry, index) => {
@@ -333,8 +272,8 @@ export class Actors extends SystemActorCompiler {
      *
      * **The scores used are the ones just written.** `SkillLogic` resolves
      * `attr.<code>` to an attribute's *effective* score, after active effects;
-     * all this pass has is the `scoreBase` it set from `sohl.attributes`. For a
-     * compiled being carrying no attribute-altering effects the two agree,
+     * all this pass has is the `scoreBase` on each embedded attribute item. For
+     * a compiled being carrying no attribute-altering effects the two agree,
      * which is every being in content today. One that did carry such an effect
      * would bake a Skill Base its client then disagrees with — that is the
      * limit of doing this at build time, and the point to revisit if it bites.
@@ -387,6 +326,9 @@ export class Actors extends SystemActorCompiler {
         const { value: authoredFolder, isAddress } = folderField(fm);
         const folder = this.folderResolver(authoredFolder, { isAddress });
 
+        this.checkAnchoredSection(body || "", "appearance", ctx);
+        this.checkAnchoredSection(body || "", "dossier", ctx);
+
         const system = {
             // The frontmatter shortcode is the actor's stable `(type, shortcode)`
             // key — and, for a being that is an archetype, its archetype
@@ -411,25 +353,6 @@ export class Actors extends SystemActorCompiler {
             dossier: withDraftNotice(fm, renderSection(body || "", "dossier")),
         };
 
-        // Fill `system.body` (+ the base-actor movement fields) from the being's
-        // frontmatter, rather than embedding a corpus item. The `sohl`
-        // block mirrors `system` field-for-field: `sohl.body` nests the body
-        // (`structure` / `weight` / …), with `currentMoveMedium` /
-        // `movementProfiles` flat alongside it. An **incorporeal** being omits
-        // `sohl.body` and keeps the schema's empty body.
-        const bodyField = sohlField(fm, "body", null);
-        if (bodyField && typeof bodyField === "object") {
-            const bodyData = extractBodyAndMovement(fm);
-            system.body = bodyData.body;
-            system.currentMoveMedium = bodyData.currentMoveMedium;
-            system.movementProfiles = bodyData.movementProfiles;
-        } else if (bodyField != null) {
-            this.noteError(
-                `${ctx}: sohl.body must be an inline object (structure/weight/…), got ${typeof bodyField}`,
-            );
-            this.errorCount++;
-        }
-
         // Being-only combat grouping (mirrors `system.defaultCombatGroup`).
         const defaultCombatGroup = sohlField(fm, "defaultCombatGroup", undefined);
         if (defaultCombatGroup !== undefined) {

package/sohl/item-fields.mjs

@@ -391,11 +391,11 @@ export const ITEM_FIELDS = Object.freeze({
             ...STRING,
             default: "",
             topLevelMeans:
-                "the note's own title — the heading its page is published under, " +
-                "which has nothing to do with the style of address an office " +
-                "carries. Author this on the membership instead: `sohl.system.title` " +
-                "on the item, or the `system.title` of the entry in a being's " +
-                "`sohl.items`.",
+                "A top-level `title` is refused outright, since an office's " +
+                "style of address belongs to the membership that holds the " +
+                "office and nothing outside the item supplies it. Author it " +
+                "at `sohl.system.title` on the item, or at the " +
+                "`system.title` of the entry in a being's `sohl.items`.",
             describe: "The style of address the office carries.",
         },
         {

package/sohl/items.mjs

@@ -93,7 +93,7 @@ export class Items extends SystemItemCompiler {
             // it — an undeclared `system` key is discarded at construction
             // without a warning.
             templatePriority: systemTemplatePriority(fm, label),
-            actionDefs: Array.isArray(fm.actionDefs) ? fm.actionDefs : [],
+            actionDefs: [],
             notes: "",
             docHtml: description || "",
         };

package/sohl/note-schemas.mjs

@@ -99,37 +99,27 @@ const MACRO_FIELDS = Object.freeze([
 const VEHICLE_FIELDS = Object.freeze([]);
 
 /**
- * A `being` note — an actor, with its body, movement and embedded items.
+ * A `being` note — an actor, with its embedded items and its combat grouping.
+ *
+ * **A being's own document fields are authored under `sohl.system`**, which the
+ * passthrough writes at the DataModel's own paths — a body and the movement that
+ * goes with it are `sohl.system.body`, `sohl.system.currentMoveMedium` and
+ * `sohl.system.movementProfiles`, written together — so this declaration states
+ * none of them. **Attributes and skills are embedded items**, each named by the
+ * catalogue entry it copies, so both are `sohl.items` entries with the score or
+ * the mastery level as that entry's own `system` overlay.
+ *
+ * What is left is the two keys the pass reads off the block itself.
  *
  * @type {readonly import("../engine/field-spec.mjs").FieldSpec[]}
  */
 const BEING_FIELDS = Object.freeze([
-    {
-        name: "body",
-        ...MAP,
-        describe: "The being's body structure, by part shortcode.",
-    },
-    {
-        name: "attributes",
-        ...MAP,
-        describe: "Attribute scores, by attribute shortcode.",
-    },
     {
         name: "items",
         ...LIST,
         describe:
             "Items embedded on the being, each addressed by (type, shortcode) or carrying enough fields to stand alone.",
     },
-    {
-        name: "currentMoveMedium",
-        ...STRING,
-        describe: "Which medium the being is currently moving through.",
-    },
-    {
-        name: "movementProfiles",
-        ...LIST,
-        describe: "Movement rates, one profile per medium.",
-    },
     {
         name: "defaultCombatGroup",
         ...AS_AUTHORED,

package/types/config.d.mts

@@ -102,6 +102,9 @@ export const DERIVED_MANIFEST_KEYS: Readonly<Record<string, string>>;
  * @property {string} stageDir      The staged package root, relative to
  *                                   `rootDir`. Every asset `to:` lands under it.
  * @property {readonly Readonly<AssetSpec>[]} assets
+ * @property {boolean} baseStyles    Whether the package takes the shared base
+ *                                   stylesheet, staged into `stageDir` and
+ *                                   named first in the manifest's `styles`.
  * @property {Readonly<Record<string, unknown>>} manifest  The manifest
  *                                   specification, emitted as declared.
  * @property {string|null} manifestFlags  Module to load a `flags` function
@@ -235,6 +238,12 @@ export type PackageBuildConfig = {
      */
     stageDir: string;
     assets: readonly Readonly<AssetSpec>[];
+    /**
+     * Whether the package takes the shared base
+     * stylesheet, staged into `stageDir` and
+     * named first in the manifest's `styles`.
+     */
+    baseStyles: boolean;
     /**
      * The manifest
      * specification, emitted as declared.

package/types/content-config.d.mts

@@ -126,6 +126,16 @@ export const RETIRED_ADDRESS_KEYS: Readonly<Record<string, string>>;
 export const DEFAULT_ADDRESS_SCHEME: Readonly<{
     prefix: "";
 }>;
+/**
+ * Every key a declared relationship may carry.
+ *
+ * Read by the manifest writer as well as by the configuration check: the keys
+ * Foundry's own relationship schema does not name are the build's own, and the
+ * published manifest carries none of them.
+ *
+ * @type {readonly string[]}
+ */
+export const RELATIONSHIP_KEYS: readonly string[];
 /**
  * How the loader hands {@link defineConfig} the system version it resolved.
  *
@@ -274,7 +284,8 @@ export type PathsInput = {
     unpack?: string | undefined;
     /**
      * Where a dependency declaring
-     *      `itemCatalog: true` is unpacked.
+     *      `itemCatalog: true` or
+     *      `assetArchive: true` is unpacked.
      *      Inbound, and fetched rather than
      *      committed.
      */
@@ -438,6 +449,21 @@ export type RelationshipSpec = {
      * `_stats.systemVersion` is stamped from.
      */
     compatibility?: CompatibilitySpec | undefined;
+    /**
+     * Whether `deps fetch` unpacks this
+     *  dependency's release archive and extracts
+     *  its Item packs, so the actors pass can
+     *  resolve embedded items this repository
+     *  does not hold. Needs a `manifest`.
+     */
+    itemCatalog?: boolean | undefined;
+    /**
+     * Whether `deps fetch` unpacks this
+     * dependency's release archive for its
+     * asset bytes alone, building no item
+     * catalogue from it. Needs a `manifest`.
+     */
+    assetArchive?: boolean | undefined;
     /**
      * Whether `deps fetch` fetches this
      * dependency's content index. Default
@@ -449,6 +475,16 @@ export type RelationshipSpec = {
      * index.
      */
     contentIndex?: boolean | undefined;
+    /**
+     * Whether this dependency's asset
+     * tree answers this package's own asset
+     * addresses ahead of its local record.
+     * Default `false`. Refuses `contentIndex:
+     * false` on the same entry, since a
+     * replacement with no fetched index to
+     * check against resolves nothing.
+     */
+    assetReplacement?: boolean | undefined;
 };
 /**
  * How a generated documentation page is framed in the repository publishing it.
@@ -628,6 +664,12 @@ export type ContentBuildConfigInput = {
      *    `Templates`). Default `[]`.
      */
     skipDirectories?: string[] | undefined;
+    /**
+     * Whether `lint` refuses an asset
+     *  record of this package carrying
+     *  `ai: true`. Default `false`.
+     */
+    forbidGeneratedArt?: boolean | undefined;
     /**
      * Reserved for
      * `@heroiclands/package-build`, which
@@ -764,6 +806,11 @@ export type ContentBuildConfig = {
      */
     docEntryTypes: ReadonlySet<string>;
     skipDirectories: readonly string[];
+    /**
+     * Whether `lint` refuses an asset
+     * record of this package carrying `ai: true`.
+     */
+    forbidGeneratedArt: boolean;
     /**
      * The
      * fonts this package ships and the names it

package/types/engine/actor-compiler.d.mts

@@ -248,6 +248,17 @@ export class SystemActorCompiler extends BasePackCompiler {
         img: string;
         token: string;
     };
+    /**
+     * Report an `{#appearance}` or `{#dossier}` anchor on a heading
+     * {@link extractAnchorSection} cannot read, rather than letting the field
+     * it feeds compile blank with nothing to say why.
+     *
+     * @param {string} body - The note body.
+     * @param {string} anchorId - `"appearance"` or `"dossier"`.
+     * @param {string} ctx - The actor's label, for the message.
+     * @returns {void}
+     */
+    checkAnchoredSection(body: string, anchorId: string, ctx: string): void;
     /** @type {readonly string[]} */
     itemsSourceDirs: readonly string[];
     foreignSourceDirs: any;

package/types/engine/anchored-sections.d.mts

@@ -6,6 +6,11 @@
  * heading exists. Fenced code blocks are respected so `# foo` inside
  * ``` blocks does not trigger a match.
  *
+ * An anchor declared on a lower heading is found by {@link collectAnchors}
+ * but not by the `#\s+` match below, so it extracts as absent rather than as
+ * wrong — {@link misplacedAnchorSection} is the check a caller runs first to
+ * tell the two apart.
+ *
  * @param {string} body - The note body.
  * @param {string} anchorId - The anchor to find.
  * @returns {string} The section's markdown, or "".
@@ -19,3 +24,22 @@ export function extractAnchorSection(body: string, anchorId: string): string;
  * @returns {string} The rendered HTML, or "".
  */
 export function renderSection(body: string, anchorId: string): string;
+/**
+ * Where an `{#appearance}` or `{#dossier}` anchor sits on a heading below the
+ * top level, which {@link extractAnchorSection} cannot read.
+ *
+ * {@link module:engine/anchors.collectAnchors} finds the anchor at any
+ * level, because a lower heading legitimately starts a journal page. It does
+ * not feed an actor field: an H1-only extractor reads nothing from one, and a
+ * caller that trusts the empty result as "no such section" rather than
+ * "found, but unreadable" compiles a blank field with nothing to say why.
+ *
+ * @param {string} body - The note body.
+ * @param {string} anchorId - The anchor to check, case-insensitively.
+ * @param {number} [bodyLine=1] - The 1-based file line the body starts on.
+ * @returns {{line: number}|null} The misplaced anchor's line, or `null` when
+ *   it is absent or already on an H1.
+ */
+export function misplacedAnchorSection(body: string, anchorId: string, bodyLine?: number): {
+    line: number;
+} | null;

package/types/engine/anchors.d.mts

@@ -1,17 +1,22 @@
 /**
- * The `{#slug}` and `:::caption {#slug}` anchors a note declares.
+ * The `{#slug}` and `:::figure {#slug}` anchors a note declares.
  *
- * A bare `#` heading starts a journal page without declaring a slug. Captioned
- * blocks declare a slug and start an addressable journal page.
+ * A bare `#` heading starts a journal page without declaring a slug, and so
+ * does a figure fence that declares no id. A fence that declares one declares a
+ * slug and starts an addressable journal page.
  *
  * @param {string} body - The note's markdown body, frontmatter already removed.
  * @param {number} [bodyLine] - The 1-based file line the body starts on, from
  *   `parseMarkdownFile`. Anchors are reported at their position in the **file**,
  *   so an editor can jump straight to one; passing nothing numbers from the body.
+ * @param {(address: string) => string|undefined} [resolveRole] - From a
+ *   picture's address to the role its asset declares, so a figure anchor's
+ *   `name` reads `Map 1` rather than `Figure 1` where it is due — see
+ *   {@link module:engine/content-figures.scanFigures}.
  * @returns {Array<{slug: string, name: string, level: number, line: number}>}
  *   In document order.
  */
-export function collectAnchors(body: string, bodyLine?: number): Array<{
+export function collectAnchors(body: string, bodyLine?: number, resolveRole?: (address: string) => string | undefined): Array<{
     slug: string;
     name: string;
     level: number;