@heroiclands/package-build 23.1.0 → 23.2.0

package/engine/art-slots.mjs

@@ -23,6 +23,9 @@
  *   `icon` while its accepted set also takes `image`. A value naming a type
  *   outside it is an error, against this set rather than against `type`.
  * @property {boolean} document - Whether the slot reaches a compiled document.
+ * @property {string} [declines] - The literal value that declines the slot's
+ *   art on purpose. It is never read as an Address: it resolves to no image and
+ *   is emitted as a blank, which a theme reads as "draw none".
  * @property {string} describe - One line, for the author-facing reference.
  */
 
@@ -76,6 +79,12 @@ export const ART_SLOTS = Object.freeze([
         // fitted to whatever room it has, so no slot but this one states a
         // size.
         size: Object.freeze({ width: 1792, height: 768 }),
-        describe: "The page's hero image, cut to 1792×768. Reaches no compiled document.",
+        // `none` is the one way to decline a hero image. It is a word rather
+        // than a blank because `""` is refused: a blank is how an unfinished
+        // note looks, and declining has to be said on purpose.
+        declines: "none",
+        describe:
+            "The page's hero image, cut to 1792×768; `none` declines it. " +
+            "Reaches no compiled document.",
     }),
 ]);

package/engine/asset-bindings.mjs

@@ -1,4 +1,15 @@
-/* SPDX-License-Identifier: GPL-3.0-or-later */
+/*
+ * 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 fs from "node:fs";
 import path from "node:path";

package/engine/base-compiler.mjs

@@ -92,20 +92,17 @@ import {
 import { isNoteRecord, noteFile } from "./index-records.mjs";
 import { reduceAddressFields } from "./address-fields.mjs";
 import { artPathname, artSlot, unacceptedArtMessage, unresolvedArtMessage } from "./art-fields.mjs";
-import { emitDiagnostic, positionOfFrontmatterPath, positionOfLiteral } from "./diagnostics.mjs";
-import { assertNoDeclaredPackage } from "./note-package.mjs";
-import { assertNoDeclaredFolder } from "./folder-notes.mjs";
 import {
-    assertNoAliasesField,
-    assertNoDraftField,
-    assertNoSectionField,
-    assertNoTraitsField,
-} from "./retired-fields.mjs";
-import { assertTypeNotRetired, packForType } from "./ids.mjs";
+    emitDiagnostic,
+    locateFrontmatterKey,
+    positionOfFrontmatterPath,
+    positionOfLiteral,
+} from "./diagnostics.mjs";
+import { assertNoDeclaredPackage } from "./note-package.mjs";
+import { packForType } from "./ids.mjs";
 import { resolveNoteId } from "./note-ids.mjs";
 import { carriesSystemBlock } from "./system-block.mjs";
 import { checkAuthoredSystemData, checkEmittedSystemData } from "./schema-check.mjs";
-import { locateFrontmatterKey } from "./retired-fields.mjs";
 // A being's computed `age`, filled in against the package's own declared
 // present before this pass reads `data:` for anything else — see
 // `engine/being-age.mjs`.
@@ -116,8 +113,8 @@ import { reckoningContext } from "./reckoning-markers.mjs";
  * The tallies one pass accumulates while walking the tree.
  *
  * `declined` and `skippedOther` are deliberately separate numbers. A declined
- * note is one this build **refused** — it declares a retired frontmatter field
- * — and it is an error; a skipped one legitimately belongs to another pass, and
+ * note is one this build **refused** — it declares `package:` — and it is an
+ * error; a skipped one legitimately belongs to another pass, and
  * there are thousands of those. Folding the first into the second is what let a
  * whole tree be filtered out in silence.
  *
@@ -125,9 +122,9 @@ import { reckoningContext } from "./reckoning-markers.mjs";
  * @property {number} compiled - Notes that became a document.
  * @property {number} skippedNoId - Notes with no `id`, where that is tolerated.
  * @property {number} skippedOther - Notes this pass does not claim.
- * @property {number} declined - Notes this pack **refused** — one declaring a
- *   retired frontmatter field, or one routed to a system pack whose system it
- *   says nothing about. Counted as errors, never as skips.
+ * @property {number} declined - Notes this pack **refused** — one declaring
+ *   `package:`, or one routed to a system pack whose system it says nothing
+ *   about. Counted as errors, never as skips.
  */
 
 /**
@@ -524,14 +521,14 @@ export class BasePackCompiler {
      * A refusal only this pass can make, because its subject is the note's
      * **type**.
      *
-     * The `assertNo*Field` family above it in the walk is type-agnostic by
-     * construction: it runs before `selects`, so that a note declaring a
-     * retired field is answered whichever pass would have claimed it. A rule
+     * The `assertNoDeclaredPackage` check above it in the walk is type-agnostic
+     * by construction: it runs before `selects`, so that a note declaring
+     * `package:` is answered whichever pass would have claimed it. A rule
      * about what a *`trauma`* may write cannot live there — it needs the type's
      * field declaration, which only the pass that compiles the type can reach.
      *
      * So it is a hook, called once the note is known to be this pass's, and its
-     * throw is counted and located exactly as the family's is: the note is
+     * throw is counted and located exactly as that check's is: the note is
      * declined rather than skipped, and the build fails naming the line.
      *
      * The default refuses nothing, which is the honest position for a pass
@@ -832,12 +829,10 @@ export class BasePackCompiler {
     /**
      * The Foundry path one authored art address names, or `null`.
      *
-     * **The two empties survive.** `null` and an absent key mean *no art named*,
-     * so the caller's own default applies with nullish coalescing; `""` means
-     * *ship blank on purpose* and no default may replace it. That is
-     * {@link module:engine/helpers.resolveImg}'s rule, reached through it rather
-     * than restated, which is also what puts an art address and a body image
-     * through one ownership rule.
+     * `null` and an absent key mean *no art named*, so the caller's own default
+     * applies with nullish coalescing. The address goes through
+     * {@link module:engine/helpers.resolveImg}, which puts an art address and a
+     * body image through one ownership rule.
      *
      * An address nothing answers is reported against the note and treated as
      * unnamed, so the document takes its default rather than shipping a path
@@ -849,8 +844,8 @@ export class BasePackCompiler {
      * @param {string} key - The key it was authored at, for the message.
      * @param {string} type - The asset type a bare value takes.
      * @param {Iterable<string>} [accepts] - The types this slot accepts.
-     * @returns {string|null} The Foundry-relative path, `""` for a deliberate
-     *   blank, or `null` where the note names none.
+     * @returns {string|null} The Foundry-relative path, or `null` where the
+     *   note names none.
      */
     artPathOf(value, key, type, accepts) {
         const {
@@ -1216,7 +1211,7 @@ export class BasePackCompiler {
         // A note the index could not record is a note this pass declines —
         // counted and reported exactly as the loop below does for the same
         // refusal, because it *is* the same refusal: the index defers to
-        // `assertNoDeclaredPackage` for a retired `package:`, which is the
+        // `assertNoDeclaredPackage` for a declared `package:`, which is the
         // check this loop makes a few lines further down. All that changed is
         // which pass sees the note first.
         if (this.reportsCorpusProblems) {
@@ -1240,9 +1235,8 @@ export class BasePackCompiler {
         // this pass's own. Each note is then read for its **prose**: the
         // index carries what is *about* a note and deliberately not its text,
         // nor the `bodyLine`/`bodyColumn` a diagnostic needs, and this pass has
-        // to have the body anyway. So the read is one this pass was already
-        // making; what it no longer does is decide for itself which files to
-        // make it over.
+        // to have the body anyway. So the read is one this pass makes anyway;
+        // the index decides which files it covers.
         const dates = reckoningContext({ notes: this.corpus.records });
         for (const record of this.corpus.records) {
             if (!isNoteRecord(record)) continue;
@@ -1258,44 +1252,20 @@ export class BasePackCompiler {
                 continue;
             }
             applyComputedBeingAge(fm, this.worldPresent, dates);
-            // The retired frontmatter fields, refused before `selects` so a
-            // note is answered whichever pass would have claimed it — and
-            // whatever the declared value says.
-            //
-            // - `package:`: a note's package is the repository's
-            //   configured one, so declaring it restates a constant.
-            // - `draft:`: it excluded the note from the packs, the
-            //   manifest and the site, and no checker reported the links that
-            //   left dangling.
-            // - `aliases:`: it fed the alias index, which the bare
-            //   `[[Alias]]` form was looked up in; the form is retired, so the
-            //   list has no reader left. The nested `name.aliases` is a
-            //   different field: the infobox displays it, and the note name
-            //   check validates it.
-            //
-            // Both are reported and counted — never skipped, which is how a
-            // tree naming a package nothing answers to used to compile zero
-            // notes and exit 0. The file comes from the diagnostic locator, so
-            // neither message may repeat it.
+            // `package:` is refused before `selects`, so a note is answered
+            // whichever pass would have claimed it: a note's package is the
+            // repository's configured one, so declaring it restates a
+            // constant. It is reported and counted, never skipped. The file
+            // comes from the diagnostic locator, so the message must not
+            // repeat it.
             try {
                 assertNoDeclaredPackage(fm, { absPath });
-                assertNoDeclaredFolder(fm, { absPath });
-                assertNoDraftField(fm, { absPath });
-                assertNoAliasesField(fm, { absPath });
-                assertNoSectionField(fm, { absPath });
-                assertNoTraitsField(fm, { absPath });
             } catch (err) {
                 stats.declined++;
                 this.errorCount++;
                 this.noteError(err.message, err.position);
                 continue;
             }
-            // Checked before `selects`, and therefore for every note this
-            // package owns rather than only the ones some pass claims. A
-            // retired type is claimed by no pass, so the alternative is not a
-            // wrong document — it is no document, skipped as quietly as the
-            // thousands of notes that legitimately belong to another pass.
-            assertTypeNotRetired(fm.type, absPath);
             if (!this.selects(fm)) {
                 stats.skippedOther++;
                 continue;
@@ -1313,7 +1283,7 @@ export class BasePackCompiler {
                 // `shortcode`. It is not addressable, so there is nothing to
                 // derive from and nothing for a link to point at; the message
                 // names the reason rather than the missing field, because the
-                // field is no longer something an author writes.
+                // field is not something an author writes.
                 if (this.constructor.requiresId) {
                     throw new Error(
                         `${Label} note has no address, so it has no document id: ` +
@@ -1350,11 +1320,10 @@ export class BasePackCompiler {
                     stats.skippedOther++;
                     continue;
                 }
-                // The type-specific half of the retired-field family:
-                // what a note of *this* type may not write, which needs the
+                // What a note of *this* type may not write, which needs the
                 // type's own field declaration and so cannot be asked before
                 // `selects`. Counted as a declined note for the same reason
-                // they are — the alternative is a tree that compiles fewer
+                // a declared `package:` is — the alternative is a tree that compiles fewer
                 // documents than it has notes and exits 0.
                 this.assertAuthorable(fm);
             } catch (err) {

package/engine/being-age.mjs

@@ -59,11 +59,6 @@ import { worldInvariants } from "./calendar-notes.mjs";
 import { positionOfFrontmatterPath } from "./diagnostics.mjs";
 import { parseNoteDate } from "./note-dates.mjs";
 import { reckoningContext } from "./reckoning-markers.mjs";
-// `born`'s retired spelling is still read during its retirement window — see
-// `docs/content-format.md`'s "`birthday:` is the retired spelling of `born:`"
-// — so a being on the old spelling still computes an age rather than reading
-// as never born.
-import { readAliasedField } from "./retired-fields.mjs";
 
 /** The note type this module's derivation and check apply to. */
 const BEING_TYPE = "being";
@@ -170,7 +165,7 @@ export function applyComputedBeingAge(fm, presentRaw, dateContext = {}) {
         return;
     }
 
-    const born = readAliasedField(fm, "born", { inData: true });
+    const born = data?.born;
     const computed = computeAge(born, presentRaw, dateContext);
     if (computed !== null) data.age = computed;
 }
@@ -237,7 +232,7 @@ export function checkBeingAge(note, { index } = {}) {
     const data = dataOf(note?.fm);
     if (!data) return [];
 
-    const bornRaw = readAliasedField(note?.fm, "born", { inData: true });
+    const bornRaw = data.born;
     const ageRaw = data.age;
     if (ageRaw === undefined || ageRaw === null || ageRaw === "") return [];
     if (!index) return [];

package/engine/being-measurements.mjs

@@ -1,4 +1,15 @@
-/* SPDX-License-Identifier: GPL-3.0-or-later */
+/*
+ * 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 { positionOfFrontmatterPath } from "./diagnostics.mjs";
 

package/engine/calendar-artifacts.mjs

@@ -2,6 +2,12 @@
  * 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/engine/calendar-choice.mjs

@@ -1,4 +1,15 @@
-/* SPDX-License-Identifier: GPL-3.0-or-later */
+/*
+ * 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 { isAddressTuple, parseAddress, renderAddress } from "./address.mjs";
 import { positionOfFrontmatterPath } from "./diagnostics.mjs";

package/engine/calendar-format.mjs

@@ -1,4 +1,15 @@
-/* SPDX-License-Identifier: GPL-3.0-or-later */
+/*
+ * 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 { dayOfYear } from "./calendars.mjs";
 

package/engine/calendar-human.mjs

@@ -1,4 +1,15 @@
-/* SPDX-License-Identifier: GPL-3.0-or-later */
+/*
+ * 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 { calendarStructure } from "./calendars.mjs";
 import {

package/engine/compile-corpus.mjs

@@ -73,15 +73,17 @@ export async function buildCompileCorpus({
 }) {
     const resolved = config ?? loadPackConfig();
     const collected = problems ?? [];
+    const parsed = new Map();
     const records = indexRecordsFor({
         contentBase,
         assetsBase,
         config: resolved,
         skipDirectories,
         problems: collected,
+        parsed,
     });
     const corpusOptions = { config: resolved, records };
-    const sqlOptions = { ...corpusOptions, skipDirectories };
+    const sqlOptions = { ...corpusOptions, skipDirectories, parsed };
     return {
         records,
         problems: collected,

package/engine/content-address.mjs

@@ -148,12 +148,10 @@ export function addressSlug(fm) {
  * docs, neither of which contains a hyphen or names a type.
  *
  * **It is a pure function of the frontmatter**, and takes no options. Nothing
- * about the file the note was read from reaches it: the `README.md` convention
- * that made one note address a whole section is retired with the section itself,
- * so every note is addressed alike and there is one rule and no branch.
- * It takes no address scheme; the `landing` rule it once validated is
- * discarded; with that key retired, `prefix` was the only thing left in the
- * scheme and the paragraph above is the reason it never applied.
+ * about the file the note was read from reaches it, so every note is addressed
+ * alike and there is one rule and no branch. It takes no address scheme:
+ * `prefix` is the whole scheme and the paragraph above is the reason it never
+ * applies here.
  *
  * @param {object} fm - Parsed frontmatter.
  * @returns {string} The package-relative address, with a trailing slash and no
@@ -288,13 +286,12 @@ export const DOCUMENT_ID_NAMESPACE = "document";
  * The Foundry `_id` of the document a note compiles into, derived from its
  * canonical address.
  *
- * A note used to author this — an opaque 16-character string, one per note,
- * that said nothing its address did not and that no check guaranteed. The
- * address is the identity that *is* guaranteed: `content-lint` refuses a
+ * An authored id would be an opaque string that says nothing its address
+ * does not and that no check guarantees. The address is the identity that
+ * *is* guaranteed: `content-lint` refuses a
  * duplicate `(type, shortcode)` across every pack of a document type, which is
  * exactly the scope a primary document's id must be unique within. So the
- * derived id inherits a guard that already exists, where the authored one had
- * none.
+ * derived id inherits that guard.
  *
  * **The coupling this creates, stated plainly.** The address carries the
  * shortcode, so renaming a shortcode moves the document's id — where an
@@ -303,7 +300,7 @@ export const DOCUMENT_ID_NAMESPACE = "document";
  * note, so it is a breaking change either way; and a note that must keep its
  * identity across a rename pins an `id`, which is what the pin is for. One
  * thing genuinely degrades — `engine/address-diff.mjs` tells a rename from a
- * withdrawal by matching document ids, and can no longer do so for a note that
+ * withdrawal by matching document ids, and cannot do so for a note that
  * authors none. Its module note records that.
  *
  * @param {string} pkg - The owning **content** package.

package/engine/content-alerts.mjs

@@ -1,4 +1,16 @@
-/* SPDX-License-Identifier: GPL-3.0-or-later */
+/*
+ * 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 MarkdownIt from "markdown-it";
 import { parseExtensionAttributes, refusedAttributes } from "./extension-attributes.mjs";
 import { slugify } from "./content-slug.mjs";

package/engine/content-blocks.mjs

@@ -1,4 +1,15 @@
-/* SPDX-License-Identifier: GPL-3.0-or-later */
+/*
+ * 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
+ */
 
 /**
  * General divs and the `:::secret` disclosure fence.
@@ -30,8 +41,8 @@
  * whole page, or Hugo's of the whole site page — read the body as part of its
  * own document rather than as an isolated fragment. A fragment rendered on its
  * own has no access to the note's footnote definitions, which live at the top
- * level outside every block, so a reference inside one used to fall through as
- * literal `[^id]` text; reached by the one parse that also sees the
+ * level outside every block, so a reference inside one would fall through as
+ * literal `[^id]` text if rendered alone; reached by the one parse that also sees the
  * definitions, it resolves exactly as a reference inside a list item or a
  * block quote already does.
  *

package/engine/content-embeds.mjs

@@ -153,8 +153,8 @@ export function embedsIn(body, context) {
  * @param {object} index - From {@link module:engine/wikilinks.buildWikilinkIndex},
  *   or the equivalent the site and the book build.
  * @param {ParsedEmbed} embed - The embed.
- * @returns {{pathname: string}|{reason: string, target: string, type?: string}}
- *   The authored pathname the picture is at, or the finding.
+ * @returns {{pathname: string}|{reason: string, target: string, type?: string,
+ *   shortcode?: string, declared?: string}} The authored pathname the picture is at, or the finding.
  */
 export function resolveEmbed(index, embed) {
     // **Every link carries a label**, and an embed is a link. Without one there
@@ -170,7 +170,13 @@ export function resolveEmbed(index, embed) {
     if (read.record) {
         return { pathname: read.pathname };
     }
-    return { reason: read.reason, target: embed.written, type: read.type };
+    return {
+        reason: read.reason,
+        target: embed.written,
+        type: read.type,
+        ...(read.shortcode ? { shortcode: read.shortcode } : {}),
+        ...(read.declared ? { declared: read.declared } : {}),
+    };
 }
 
 /**
@@ -288,6 +294,8 @@ export function resolveEmbeds(body, { index }) {
                 offset: embed.index,
                 reason: resolved.reason,
                 ...(resolved.type ? { type: resolved.type } : {}),
+                ...(resolved.shortcode ? { shortcode: resolved.shortcode } : {}),
+                ...(resolved.declared ? { declared: resolved.declared } : {}),
             });
             continue;
         }

package/engine/content-figures.mjs

@@ -1,4 +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/engine/content-footnotes.mjs

@@ -1,4 +1,15 @@
-/* SPDX-License-Identifier: GPL-3.0-or-later */
+/*
+ * 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
+ */
 
 /**
  * Where a footnote's two halves belong, and the two findings that hold the