@heroiclands/package-build 23.1.1 → 23.2.0

package/engine/wikilinks.mjs

@@ -36,7 +36,6 @@ import { readCanonicalKey } from "./content-address.mjs";
 // is stated there and read here — the anchor and the label are this module's,
 // and the tuple inside them is not.
 import {
-    ITEM_DOC_PREFIX,
     readQualifier,
     resolveItemDocType,
     parseAddress,
@@ -197,16 +196,7 @@ export function buildWikilinkIndex(
     for (const v of foreignByKey.values()) {
         if (v.type) foreignTypes.push(norm(v.type));
     }
-    // A legacy index may contain `doc<type>` records. Keep the authored type
-    // vocabulary separate so the input alias can resolve to its base type.
-    for (const t of foreignTypes) {
-        if (!t.startsWith(ITEM_DOC_PREFIX)) types.add(t);
-    }
-    for (const t of foreignTypes) {
-        if (!t.startsWith(ITEM_DOC_PREFIX)) continue;
-        if (types.has(t.slice(ITEM_DOC_PREFIX.length))) continue;
-        types.add(t);
-    }
+    for (const t of foreignTypes) types.add(t);
 
     // Every package an address may name: this one, plus every package a
     // vendored manifest speaks for. What lets `thalorna-creature-grkrahk` be
@@ -274,7 +264,7 @@ function foreignHits(index, read) {
     if (!read || read.reason || !index.foreign?.size) return [];
     const tuple = completeAddress(read, { package: index.contentPackage, system: NOTE_SYSTEM });
     const wanted = norm(tuple.type);
-    const shortcode = norm(read.shortcode);
+    const shortcode = read.shortcode;
 
     // **An omitted package means this package**, so a short form
     // addresses nothing foreign and never reaches a dependency's index. A link
@@ -300,6 +290,26 @@ function foreignHits(index, read) {
     return hits;
 }
 
+/**
+ * What a `shortcode-not-lowercase` finding carries: the shortcode as written,
+ * and the lowercase spelling when a local note or a fetched dependency declares
+ * it.
+ *
+ * @param {object} index - From {@link buildWikilinkIndex}.
+ * @param {{shortcode: string, lowercase: object}} problem - The reader's verdict.
+ * @returns {{shortcode: string, declared?: string}} The finding's fields.
+ */
+function shortcodeCase(index, problem) {
+    const read = problem.lowercase;
+    const local =
+        (read.package ?? index.contentPackage) === index.contentPackage &&
+        index.byShortcode?.has(
+            `${resolveItemDocType(read.type, index.types) ?? read.type}/${read.shortcode}`,
+        );
+    const declared = local || foreignHits(index, read).length > 0;
+    return { shortcode: problem.shortcode, ...(declared ? { declared: read.shortcode } : {}) };
+}
+
 /**
  * How an **unresolved** link renders.
  *
@@ -335,8 +345,7 @@ function unresolvedLink(text, target) {
  * **The wrapper carries the cue and nothing else.** The link itself is
  * untouched — Foundry enriches inside HTML, so the `@UUID` still becomes a live
  * content link, and the note is in the packs, the manifest and the index
- * exactly as any other. Nothing here resembles the retired `draft:` field,
- * which moved a note from published to unresolvable without saying so.
+ * exactly as any other.
  *
  * The appearance lives in `scss/components/_draft-link.scss`, beside the
  * unresolved-link partial, not here.
@@ -409,8 +418,7 @@ export function convertWikilinks(markdown, { type, id, pack, docPack, index, cap
         const slug = parsed.anchor || null;
 
         // **Every link carries a label**. Without one there is nothing
-        // to resolve against: the alias namespace a bare `[[Text]]` was looked
-        // up in is retired, and a shortcode is an address rather than prose, so
+        // to resolve against: a shortcode is an address rather than prose, so
         // the link has neither a target this build can find nor text to show.
         // Reported before anything else, including the same-page form, because
         // it is a statement about how the link is *written* — `[[#slug]]` needs
@@ -452,6 +460,9 @@ export function convertWikilinks(markdown, { type, id, pack, docPack, index, cap
                     target,
                     offset,
                     reason: qualified?.reason ?? "not-an-address",
+                    ...(qualified?.reason === "shortcode-not-lowercase" ?
+                        shortcodeCase(index, qualified)
+                    :   {}),
                     addressed: true,
                 });
                 return unresolvedLink(text || target, target);

package/githooks/attribution.mjs

@@ -1,5 +1,13 @@
 /*
+ * This file is part of the Song of Heroic Lands (SoHL) system for Foundry VTT.
  * Copyright (c) 2024-2026 Tom Rodriguez ("Toasty") — <toasty@heroiclands.org>
+ *
+ * This work is licensed under the GNU General Public License v3.0 (GPLv3).
+ * You may copy, modify, and distribute it under the terms of that license.
+ *
+ * For full terms, see the LICENSE.md file in the project root or visit:
+ * https://www.gnu.org/licenses/gpl-3.0.html
+ *
  * SPDX-License-Identifier: GPL-3.0-or-later
  */
 

package/githooks/commit-message.mjs

@@ -1,7 +1,16 @@
 /*
+ * This file is part of the Song of Heroic Lands (SoHL) system for Foundry VTT.
  * Copyright (c) 2024-2026 Tom Rodriguez ("Toasty") — <toasty@heroiclands.org>
+ *
+ * This work is licensed under the GNU General Public License v3.0 (GPLv3).
+ * You may copy, modify, and distribute it under the terms of that license.
+ *
+ * For full terms, see the LICENSE.md file in the project root or visit:
+ * https://www.gnu.org/licenses/gpl-3.0.html
+ *
  * SPDX-License-Identifier: GPL-3.0-or-later
  */
+
 import { readFileSync } from "node:fs";
 import { relative } from "node:path";
 import { attribution } from "./attribution.mjs";

package/hm3/actors.mjs

@@ -46,20 +46,13 @@
  */
 
 import { resolveName } from "../engine/helpers.mjs";
-import { buildFromFields, retiredTopLevelKey, STRING } from "../engine/field-spec.mjs";
+import { buildFromFields, STRING } from "../engine/field-spec.mjs";
 import { SystemActorCompiler } from "../engine/actor-compiler.mjs";
 import { renderSection } from "../engine/anchored-sections.mjs";
 import { withDraftNotice } from "../engine/draft-notice.mjs";
 import { documentSubtype } from "../engine/document-subtypes.mjs";
 import { HM3_DOCUMENT_SUBTYPES } from "./document-subtypes.mjs";
 import { templateFlags } from "./template-priority.mjs";
-// The retirement window's reports, shared with the frontmatter lint so the
-// two cannot say different things about the same key.
-import {
-    legacyKeyMessage,
-    locateFrontmatterKey,
-    retiredTopLevelMessage,
-} from "../engine/retired-fields.mjs";
 // The note-level `hm3:` block: `hm3.system` onto the document's `system`
 // verbatim, and `hm3.img` / `hm3.items` / `hm3.effects` / `hm3.flags`
 // overriding their shared top-level forms for this system alone.
@@ -92,15 +85,13 @@ export const ACTOR_FIELDS = Object.freeze([
 /**
  * The two `data:` facts HM3 declares on a `character` and not on a `creature`.
  *
- * Both name their shared source and their legacy in-block key, for the reason
- * {@link ACTOR_FIELDS} does.
+ * Both name a shared source in `data:`, which is the only place they are authored.
  *
  * @type {readonly import("../engine/field-spec.mjs").FieldSpec[]}
  */
 export const CHARACTER_FIELDS = Object.freeze([
     {
         name: "data.gender",
-        legacyKey: "gender",
         to: "gender",
         ...STRING,
         default: "",
@@ -108,7 +99,6 @@ export const CHARACTER_FIELDS = Object.freeze([
     },
     {
         name: "data.occupation",
-        legacyKey: "occupation",
         to: "occupation",
         ...STRING,
         default: "",
@@ -126,8 +116,8 @@ export const CHARACTER_FIELDS = Object.freeze([
  * @type {Readonly<Record<string, string>>}
  */
 const DEFAULT_IMG = Object.freeze({
-    character: "systems/hm3/images/svg/knight-silhouette.svg",
-    creature: "systems/hm3/images/svg/monster-silhouette.svg",
+    character: "systems/hm3/assets/images/knightsilhouette.svg",
+    creature: "systems/hm3/assets/images/monstersilhouette.svg",
 });
 
 /**
@@ -176,9 +166,9 @@ function defaultActorImg(subType) {
  * @type {Readonly<Record<string, string>>}
  */
 const EMBEDDED_ITEM_ART = Object.freeze({
-    invocation: "systems/hm3/images/icons/svg/circle.svg",
-    psionic: "systems/hm3/images/icons/svg/psionics.svg",
-    spell: "systems/hm3/images/icons/svg/pentacle.svg",
+    invocation: "systems/hm3/assets/icons/circle.svg",
+    psionic: "systems/hm3/assets/icons/psionics.svg",
+    spell: "systems/hm3/assets/icons/pentacle.svg",
 });
 
 /**
@@ -297,38 +287,9 @@ export class Hm3Actors extends SystemActorCompiler {
 
         const items = this.buildEmbeddedItems(itemsMap, id, fm, ctx);
 
-        // The sweep's progress signal, reported where an author meets it
-        // soonest — every consumer runs the compile, and not every one runs the
-        // lint. A warning: the note compiles to the correct document
-        // either way, and refusing the position comes once no tree writes it.
-        const onLegacyKey = (field) =>
-            this.noteWarn(
-                legacyKeyMessage(block, field),
-                locateFrontmatterKey(this.currentNote?.absPath, field.legacyKey),
-            );
-
-        // The shared level's own retiring position — the top-level key `data:`
-        // gathered the field off. Same signal, same severity, separate
-        // callback: a note may have moved one position and not the other.
-        // Anchored at column 1, because the two spellings coincide — `portrait`
-        // names a block key and a top-level one — and a locator that took the
-        // first match would point at the wrong line.
-        const onRetiredTopLevel = (field) =>
-            this.noteWarn(
-                retiredTopLevelMessage(field),
-                locateFrontmatterKey(
-                    this.currentNote?.absPath,
-                    retiredTopLevelKey(field),
-                    undefined,
-                    { topLevel: true },
-                ),
-            );
-        const reports = { block, onLegacyKey, onRetiredTopLevel };
+        const reports = { block };
 
-        // One spelling, as everywhere else: `packFolder` names a folder note
-        // by its address. This pass once read only the Foundry id, so an HM3
-        // tree could not file an actor by address at all — which its own sweep
-        // needs; the id spelling is retired outright.
+        // `packFolder` names a folder note by its address.
         const folder = this.folderResolver(blockDataProperty(fm, block, "packFolder", null), {
             isAddress: true,
         });

package/hm3/default-item-art.mjs

@@ -41,15 +41,15 @@
  * @type {Readonly<Record<string, string>>}
  */
 export const HM3_DEFAULT_ITEM_ART = Object.freeze({
-    armorgear: "hm3/assets/images/icons/svg/armor.svg",
-    armorlocation: "hm3/assets/images/icons/svg/anatomy.svg",
-    containergear: "hm3/assets/images/icons/svg/sack.svg",
-    miscgear: "hm3/assets/images/icons/svg/miscgear.svg",
-    mysticalability: "hm3/assets/images/icons/svg/psionics.svg",
-    projectilegear: "hm3/assets/images/icons/svg/arrow.svg",
-    skill: "hm3/assets/images/icons/svg/skills.svg",
-    trauma: "hm3/assets/images/icons/svg/injury.svg",
-    weapongear: "hm3/assets/images/icons/svg/sword.svg",
+    armorgear: "hm3/assets/icons/armor.svg",
+    armorlocation: "hm3/assets/icons/anatomy.svg",
+    containergear: "hm3/assets/icons/sack.svg",
+    miscgear: "hm3/assets/icons/miscgear.svg",
+    mysticalability: "hm3/assets/icons/psionics.svg",
+    projectilegear: "hm3/assets/icons/arrow.svg",
+    skill: "hm3/assets/icons/skills.svg",
+    trauma: "hm3/assets/icons/injury.svg",
+    weapongear: "hm3/assets/icons/sword.svg",
 });
 
 /**

package/hm3/item-fields.mjs

@@ -77,7 +77,6 @@ import { AS_AUTHORED, NUMBER } from "../engine/field-spec.mjs";
 const GEAR_COMMON = Object.freeze([
     {
         name: "data.weight",
-        legacyKey: "weight",
         to: "weight",
         ...AS_AUTHORED,
         kind: "number",
@@ -86,7 +85,6 @@ const GEAR_COMMON = Object.freeze([
     },
     {
         name: "data.value",
-        legacyKey: "value",
         to: "value",
         ...AS_AUTHORED,
         kind: "number",
@@ -107,7 +105,6 @@ const GEAR_COMMON = Object.freeze([
  */
 const QUANTITY = Object.freeze({
     name: "data.quantity",
-    legacyKey: "quantity",
     to: "quantity",
     ...NUMBER,
     default: 1,
@@ -127,10 +124,8 @@ const QUANTITY = Object.freeze({
  * a registry is addressed by what a note calls itself. What the document is
  * called is the map's answer, and only the map's.
  *
- * Each field names both of its authored positions, as
- * {@link module:hm3/actors} does and for the same reason: `name` is the shared
- * source the content format's mapping tables state, and `legacyKey` is the key
- * the `hm3:` block still carries.
+ * A field whose `name` is a path into `data:` is authored there and nowhere
+ * else: `name` is the shared source the content format's mapping tables state.
  *
  * @type {Readonly<Record<string, readonly import("../engine/field-spec.mjs").FieldSpec[]>>}
  */
@@ -148,7 +143,6 @@ export const HM3_ITEM_FIELDS = Object.freeze({
         ...GEAR_COMMON,
         {
             name: "data.capacity",
-            legacyKey: "capacity",
             to: "capacity.max",
             ...NUMBER,
             default: 0,
@@ -171,7 +165,6 @@ export const HM3_ITEM_FIELDS = Object.freeze({
     skill: Object.freeze([
         {
             name: "data.masteryLevel",
-            legacyKey: "masteryLevel",
             to: "masteryLevel",
             ...NUMBER,
             default: 0,

package/hm3/template-priority.mjs

@@ -49,13 +49,12 @@ import { blockProperty } from "../engine/system-block.mjs";
  * than a `null` nothing reads.
  *
  * **Read through the shared resolver, not a field declaration**. A
- * `FieldSpec`'s shared source is a single position, and this value has five:
- * `data:`, this block, the top level, and the retiring `archetype` spelling in
- * the latter two. The resolver is the single implementation of what a note said,
- * so the two systems cannot disagree about it. It is read against **this**
- * block: the `sohl:` block is not a source for an HM3 document, so a tree that
- * still states the priority there (`harn-ensemble`, on 2,502 notes) writes no
- * HM3 flag until it sweeps to `data:`.
+ * `FieldSpec`'s shared source is a single position, and this value has three:
+ * `data:`, this block, and the top level. The resolver is the single
+ * implementation of what a note said, so the two systems cannot disagree about
+ * it. It is read against **this** block: the `sohl:` block is not a source for
+ * an HM3 document, so a note that states the priority only there writes no HM3
+ * flag.
  *
  * @param {object} fm - The note's frontmatter.
  * @param {string} block - This pass's system block.

package/hugo-theme/data/banner-kinds.yaml

@@ -0,0 +1,78 @@
+# Which banner a page of each kind opens with.
+#
+# `partials/banner-url.html` reads a page's note `type` and `subType` here and
+# resolves the result against `banners.yaml`. A type maps to one banner name for
+# every subType, or to a table keyed by every subType the note vocabulary
+# declares for it. `default` is a deliberate choice of the default banner; a
+# kind absent from this file resolves to its own name and warns when that name
+# is not declared.
+#
+# `npm run lint:banners` and the test suite fail on a mapped name that
+# `banners.yaml` does not declare, and on a type or subType missing here.
+
+# Types whose banner is named by the page's `category`.
+categoryTypes:
+    - doc
+
+kinds:
+    affiliation:
+        arcanetradition: organization
+        criminal: organization
+        fellowship: organization
+        faithtradition: faith
+        governmental: organization
+        guild: organization
+        lineage: organization
+        order: organization
+        polity: organization
+        spirittradition: organization
+        venture: organization
+    affliction: affliction
+    armorgear: armor-clothing
+    armorlocation: default
+    attribute: skills-attributes
+    being:
+        character: character
+        creature: creature
+        npc: character
+    bundle: default
+    concoctiongear: default
+    containergear: containers
+    folder: default
+    homepage: default
+    lore:
+        arcana: default
+        bestiary: default
+        calendar: default
+        cosmology: default
+        culture: people
+        custom: default
+        deity: default
+        economy: default
+        folk: default
+        gathering: default
+        history: default
+        law: default
+        literature: default
+        material: default
+        spirit: default
+        theology: default
+    macro: default
+    map: default
+    miscgear: misc-gear
+    mystery: default
+    mysticalability: mystical-abilities
+    place:
+        celestial: default
+        feature: default
+        region: region
+        settlement: settlement
+        site: default
+        structure: default
+        world: default
+    projectilegear: projectiles
+    scenario: default
+    skill: skills-attributes
+    trauma: trauma
+    vehicle: default
+    weapongear: weapons

package/hugo-theme/data/banners.yaml

@@ -4,10 +4,9 @@
 # `images/banners/`, and that path is emitted into a `background-image` with no
 # way to test it: the artwork is served from the consuming site's
 # `params.cdnBaseURL`, and Hugo cannot ask a remote host whether a URL exists.
-# So a banner that was never drawn used to 404 in silence — the band rendered
+# So a banner that was never drawn would 404 in silence — the band renders
 # with its title, palette and gradient intact and a dead image URL behind it,
-# and nothing failed: not the build, not Hugo, not the deploy guard
-# (issue #36).
+# and nothing fails: not the build, not Hugo, not the deploy guard.
 #
 # This file is the declaration that replaces that missing test. A name listed
 # here is asserted to exist; a resolved banner whose name is absent falls back

package/hugo-theme/layouts/404.html

@@ -6,7 +6,7 @@
   the deployment does not publish. Without it, Pages falls back to the site
   root, so an unknown path answers 200 with the landing page: a soft-404 that
   makes every "does this URL resolve?" check report success, and lets search
-  engines index retired content as live (issue #1416).
+  engines index withdrawn content as live.
 
   Every consumer needs exactly this, so it lives here rather than being copied
   into each one (issue #1454). The theme carries layout, not addresses, so the
@@ -41,8 +41,8 @@
   <div class="single-content">
     <div class="single-body">
       <p>
-        The page may have been renamed, moved to another part of the
-        {{ $noun }}, or retired along with the concept it described. Links
+        The page may have been moved to another part of the
+        {{ $noun }}, or withdrawn along with the concept it described. Links
         from older discussions, issues, and search results can outlive the
         pages they point at.
       </p>

package/hugo-theme/layouts/partials/banner-url.html

@@ -13,8 +13,12 @@
     2. `banner` — a hand-authored page's value. `none` (or `false`) declines an
        image and returns "". A full URL is used as-is; anything else is a
        fragment under `images/`.
-    3. `images/banners/{subtype}.webp`, when neither is set and a `subtype` is
-       supplied. A caller that passes no subtype gets "" instead, which is
+    3. `images/banners/{name}.webp`, when neither is set. The name is the
+       `subtype` supplied, or, when the caller passes the page's `type`, the
+       banner `data/banner-kinds.yaml` maps that `type` and `subType` to — a
+       `category` for the types it lists as taking theirs from there. A type
+       absent from the mapping keeps its own name. A caller that supplies
+       neither gets "" instead, which is
        what a card wants: a section with no banner of its own renders as the
        text card it rendered before.
     4. `images/banners/{fallback}` — whenever the name picked by 2 or 3 is not
@@ -44,8 +48,13 @@
                the content build resolved. Omit for a page that carries none.
     banner   — the raw `banner:` value; may be unset, `none`/`false`, a
                fragment or an absolute URL.
-    subtype  — optional. The name to default to when `banner` is unset. Omit
-               for "no banner means no image".
+    subtype  — optional. The name to default to when `banner` is unset and no
+               `type` is given. Omit for "no banner means no image".
+    type     — optional. The page's note `type`; with `subType` and `category`
+               it selects the default name through `data/banner-kinds.yaml`.
+    subType  — optional. The page's note `subType`.
+    category — optional. The page's `category`, for the types that take their
+               banner name from it.
     context  — optional. A page address, named in the missing-banner warning so
                the report says where the want came from.
 */ -}}
@@ -54,6 +63,22 @@
 {{- $context := .context | default "" -}}
 {{- $data := .data | default dict -}}
 
+{{- /* The default name for a page that states its `type`. A type maps to one
+       name or to a table keyed by subType; one that the mapping does not know
+       keeps its own name, so the missing-banner warning below still reports
+       it. */ -}}
+{{- with .type -}}
+  {{- $kinds := index (hugo.Data | default dict) "banner-kinds" | default dict -}}
+  {{- $entry := index ($kinds.kinds | default dict) . -}}
+  {{- if in ($kinds.categoryTypes | default slice) . -}}
+    {{- $subtype = $.category | default "default" -}}
+  {{- else if reflect.IsMap $entry -}}
+    {{- $subtype = (index $entry (lower ($.subType | default ""))) | default . -}}
+  {{- else -}}
+    {{- $subtype = $entry | default . -}}
+  {{- end -}}
+{{- end -}}
+
 {{- $banners := hugo.Data.banners | default dict -}}
 {{- $dir := $banners.dir | default "images/banners" -}}
 {{- $fallback := $banners.fallback | default "default.webp" -}}
@@ -68,11 +93,10 @@
 {{- $declined := or (eq (lower $raw) "none") (eq $raw "false") -}}
 
 {{- /* A note's resolved banner, formatted for the same reason the decline
-       above is: an unset key and a deliberate blank are both falsy, and only
+       above is: an unset key and an empty string are both falsy, and only
        formatting tells them apart. `<nil>` covers both an absent key and an
        explicit `null`, which the note format reads as *unset* — so the
-       resolution carries on to the subtype default. `""` is the format's
-       deliberate blank: a note naming no hero image, which declines one. */ -}}
+       resolution carries on to the subtype default. */ -}}
 {{- $resolved := printf "%v" (index $data "banner") -}}
 {{- $fromBuild := ne $resolved "<nil>" -}}
 

package/hugo-theme/layouts/partials/hero-banner.html

@@ -23,9 +23,10 @@
                                      see "Declining" below. A full URL is used
                                      as-is; anything else is treated as a
                                      fragment under images/.
-    3. images/banners/{subtype}.webp — subtype-specific default, where
-                                     subtype = category for type=doc pages,
-                                     otherwise type
+    3. images/banners/{name}.webp  — the default for the page's kind: the
+                                     banner data/banner-kinds.yaml maps its
+                                     type and subType to, or its category for
+                                     type=doc pages
     4. images/banners/default.webp — used when the page has no type, *and*
                                      whenever the name picked by 2 or 3 is not
                                      in the declared inventory (see below).
@@ -58,22 +59,20 @@
   considered answer was "none", so declining has to be sayable.
 */ -}}
 
-{{- /* For narrative content (type=doc) the meaningful kind of page lives in
-       `category` — continent, region, polity, faith, lore, etc. — so prefer
-       that when picking a default banner. */ -}}
+{{- /* The order above, the `none` decision, the type-to-banner mapping in
+       `data/banner-kinds.yaml` and the declared-existence check all live in
+       `banner-url.html`, which a derived landing card calls too. A hero band
+       always wants the default for its page's kind; a card never does. A page
+       with no type gets `default`. */ -}}
 {{- $type := .Params.type | default "" -}}
-{{- $subtype := cond (eq $type "doc") (.Params.category | default "") $type -}}
-{{- if not $subtype }}{{ $subtype = "default" }}{{ end -}}
-
-{{- /* The order above, the `none` decision and the declared-existence check
-       all live in `banner-url.html`, which a derived landing card calls too
-       (issue #41). A hero band always wants the subtype default; a card never
-       does. */ -}}
 {{- $bannerURL := partial "banner-url.html" (dict
     "site" .Site
     "data" .Params.data
     "banner" .Params.banner
-    "subtype" $subtype
+    "type" $type
+    "subType" .Params.subType
+    "category" .Params.category
+    "subtype" (cond (eq $type "") "default" "")
     "context" .RelPermalink) -}}
 
 <section class="hero hero-with-image"{{ with $bannerURL }} style="background-image: url('{{ . }}');"{{ end }}>

package/hugo-theme/layouts/partials/related.html

@@ -25,7 +25,7 @@
   computed first and the <section> is rendered only when there is something
   to put in it.
 
-  `return` cannot be used to bail out early here. It sets the partial's
+  `return` cannot bail out early here. It sets the partial's
   return value; it does not discard template text the partial has already
   emitted, so a guard written that way still prints the empty shell. Hence
   the emptiness test wraps the markup instead of preceding it.

package/hugo-theme/layouts/partials/warn-missing-banner.html

@@ -1,3 +1,3 @@
 {{- warnidf (printf "hero-banner-missing-%s" .name)
-    "hero-banner: no banner named %q in the shared artwork set; falling back to %s/%s. Draw it and add %q to data/banners.yaml, or name a hero image on the page — `data.banner` on a note, `banner:` on a hand-authored page. A blank `data.banner`, or `banner: none`, declines a hero image. First seen on %s."
+    "hero-banner: no banner named %q in the shared artwork set; falling back to %s/%s. Draw it and add %q to data/banners.yaml, or name a hero image with `data.banner`. Write `data.banner: none` to decline a hero image. First seen on %s."
     .name .dir .fallback .name .context -}}

package/hugo-theme/static/css/style.css

@@ -771,7 +771,7 @@ img.door-image {
   margin: 0;
 }
 
-/* One card per row once two no longer fit beside each other; the fixed width
+/* One card per row once two do not fit beside each other; the fixed width
    would otherwise overflow a narrow viewport. */
 @media (max-width: 400px) {
   .door {

package/manifest.mjs

@@ -31,8 +31,8 @@
  *   `descriptionHtml` is the one exception, folded into the derived
  *   `description` below rather than surviving under its own name.
  * - **Derived** — the identity, the description (from `descriptionHtml`), the
- *   version, the release addresses, the compatibility ranges and the pack
- *   list. Declaring `description` directly is an error rather than an
+ *   version, the authors, the release addresses, the compatibility ranges and
+ *   the pack list. Declaring `description` directly is an error rather than an
  *   override: the authored copy would be silently overwritten.
  * - **Computed** — namespaced `flags` a repository works out for itself.
  *
@@ -478,8 +478,9 @@ function requireManifestTitle(config) {
  *   a key Foundry adds later needs no release of this package. The one
  *   exception is `descriptionHtml`, folded into the description below rather
  *   than surviving under its own name.
- * - **Derived** — the identity, the description, the release addresses, the
- *   version, the Foundry and system compatibility ranges, and the pack list.
+ * - **Derived** — the identity, the description, the authors (the top-level
+ *   `authors` list without its roles), the release addresses, the version, the
+ *   Foundry and system compatibility ranges, and the pack list.
  *   These are refused if also declared (`description` directly; `descriptionHtml`
  *   is how it is authored) and the two would disagree with nothing to say so.
  * - **Computed** — namespaced `flags` a repository works out for itself, merged
@@ -519,6 +520,12 @@ export function buildManifest({ config, packageJson, artifact, flags }) {
     // in it. Set only when the repository actually declares one.
     if (descriptionHtml !== undefined) derived.description = descriptionHtml;
     if (config.compatibility) derived.compatibility = config.compatibility;
+    // Foundry's author schema is name, email, url, discord and flags; a role
+    // is the book's, printed on its title page, and the manifest ships what
+    // it is given.
+    if (config.authors?.length) {
+        derived.authors = config.authors.map(({ role: _role, ...person }) => ({ ...person }));
+    }
 
     // `requiresSystem` is the gate half of the declare/require split. It
     // emits the `relationships.systems` entry Foundry's `supportsSystem` reads,

package/markdownlint-config.mjs

@@ -12,7 +12,7 @@
  */
 
 /**
- * The shared markdownlint rules, as a markdownlint-cli2 options module.
+ * The shared markdownlint rules, as a `.markdownlint-cli2.mjs` options module.
  *
  * `package-build markdown` applies these without a consumer declaring
  * anything; this is for an editor's markdownlint extension, and for a consumer