@heroiclands/package-build 22.22.0 → 22.23.0

package/types/manifest.d.mts

@@ -267,29 +267,25 @@ export const ARTIFACTS: readonly string[];
  */
 export const HOMEPAGE_ORIGIN: string;
 /**
- * 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: readonly string[];
+/**
+ * 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[]}
  */

package/engine/content-captions.mjs

@@ -1,230 +0,0 @@
-/*
- * SPDX-License-Identifier: GPL-3.0-or-later
- */
-
-import MarkdownIt from "markdown-it";
-import footnotePlugin from "markdown-it-footnote";
-import deflistPlugin from "markdown-it-deflist";
-
-import { slugify } from "./content-slug.mjs";
-import { parseExtensionAttributes } from "./extension-attributes.mjs";
-import { HEADING_LINE, parseHeadingLine, splitHeadingAttributes } from "./heading-attributes.mjs";
-import { imagesIn } from "./content-images.mjs";
-
-const parser = new MarkdownIt({ html: true }).use(footnotePlugin).use(deflistPlugin);
-const OPEN = /^:::caption\s+(\{[^}\n]*\})\s*$/;
-const CAPTION_LINE = /^:::caption\b/;
-const CLOSE = /^:::\s*$/;
-
-/** A caption's automatically determined kind. */
-export const CAPTION_NAMES = Object.freeze({
-    code: "Code",
-    table: "Table",
-    figure: "Figure",
-    prose: "Prose",
-});
-
-/** Number a sequence of captions with counters shared across documents. */
-export function numberCaptions(captions, counts) {
-    return captions.map((caption) => {
-        const number = (counts[caption.kind] ?? 0) + 1;
-        counts[caption.kind] = number;
-        return { ...caption, number, label: `${CAPTION_NAMES[caption.kind]} ${number}` };
-    });
-}
-
-/**
- * Read caption directives and the Markdown block immediately after each one.
- * SQL fences are expanded before this pass, so their output is a table.
- *
- * @param {string} source - A note body with expanded tables.
- * @returns {{captions: Array<{id: string, text: string, kind: string, number: number,
- *   label: string, line: number, blockStart: number, blockEnd: number,
- *   directiveEnd: number}>, errors: Array<{line: number, column: number, message: string}>}}
- */
-export function scanCaptions(source) {
-    const lines = String(source ?? "").split("\n");
-    const captions = [];
-    const errors = [];
-    const counts = { code: 0, table: 0, figure: 0, prose: 0 };
-    const ids = new Set();
-    const headingIds = [];
-    let codeFence = null;
-
-    for (let i = 0; i < lines.length; i++) {
-        const line = lines[i];
-        const fence = /^ {0,3}(`{3,}|~{3,})/.exec(line);
-        if (codeFence) {
-            if (fence && fence[1][0] === codeFence[0] && fence[1].length >= codeFence.length)
-                codeFence = null;
-            continue;
-        }
-        if (fence) {
-            codeFence = fence[1];
-            continue;
-        }
-        const heading = HEADING_LINE.exec(line);
-        const headingId = heading ? splitHeadingAttributes(heading[2]).id : "";
-        if (headingId) headingIds.push({ id: slugify(headingId), line: i + 1 });
-        if (!CAPTION_LINE.test(line)) continue;
-        const open = OPEN.exec(line);
-        if (!open) {
-            errors.push({ line: i + 1, column: 1, message: "caption needs {#anchor} attributes" });
-            continue;
-        }
-        const attributes = parseExtensionAttributes(open[1].slice(1, -1));
-        if (
-            attributes.problems.length ||
-            !attributes.id ||
-            attributes.classes.length ||
-            Object.keys(attributes.values).length
-        ) {
-            errors.push({
-                line: i + 1,
-                column: 1,
-                message: attributes.problems[0] ?? "caption accepts only {#anchor}",
-            });
-            continue;
-        }
-        const id = attributes.id;
-        const start = i;
-        let close = i + 1;
-        while (close < lines.length && !CLOSE.test(lines[close])) close++;
-        if (close === lines.length) {
-            errors.push({ line: start + 1, column: 1, message: "caption needs a closing :::" });
-            continue;
-        }
-        const caption = lines
-            .slice(i + 1, close)
-            .join("\n")
-            .trim();
-        if (!caption) errors.push({ line: start + 1, column: 1, message: "caption text is empty" });
-        // An H1, or an anchored heading at any level, starts a Foundry
-        // journal page — see `engine/content-blocks.mjs`'s identical refusal
-        // for a named block.
-        for (let at = i + 1; at < close; at++) {
-            if (!parseHeadingLine(lines[at])?.startsPage) continue;
-            errors.push({
-                line: at + 1,
-                column: 1,
-                message:
-                    "a heading that starts a page cannot be written inside a caption — " +
-                    "keep an H1 or an anchored heading at the top level, or drop the " +
-                    "anchor and the level to stay inside it",
-            });
-        }
-        if (ids.has(slugify(id)))
-            errors.push({ line: start + 1, column: 1, message: `duplicate caption id "${id}"` });
-        ids.add(slugify(id));
-
-        let next = close + 1;
-        while (next < lines.length && !lines[next].trim()) next++;
-        if (next === lines.length || CAPTION_LINE.test(lines[next])) {
-            errors.push({ line: start + 1, column: 1, message: "caption needs a following block" });
-            i = close;
-            continue;
-        }
-        const tokens = parser.parse(lines.slice(next).join("\n"), {});
-        const first = tokens.find((token) => token.level === 0 && token.map);
-        if (!first) {
-            errors.push({ line: start + 1, column: 1, message: "caption needs a following block" });
-            i = close;
-            continue;
-        }
-        const blockEnd = next + first.map[1];
-        const block = lines.slice(next, blockEnd).join("\n");
-        const trimmedBlock = block.trim();
-        // A markdown image is only a figure when it stands alone — nothing
-        // else in the block, which is also what decides whether the image
-        // plugin wraps it in a `<figure>` at all. A trailing footnote
-        // reference, or any other trailing text, disqualifies it exactly as
-        // it disqualifies `standsAlone`; such a block is prose with an inline
-        // image in it, left as Markdown so a reference inside resolves
-        // rather than classified as a figure whose own render strips nothing.
-        const aloneImage =
-            /^(?:!\[|!\[\[)/.test(trimmedBlock) &&
-            imagesIn(trimmedBlock).some((image) => image.block);
-        const kind =
-            first.type === "table_open" ? "table"
-            : first.type === "fence" && /^\s*sql\b/i.test(first.info ?? "") ? "table"
-            : first.type === "fence" || first.type === "code_block" ? "code"
-            : aloneImage || /^(?:<figure\b|<img\b)/.test(trimmedBlock) ? "figure"
-            : "prose";
-        const number = ++counts[kind];
-        captions.push({
-            id,
-            text: caption,
-            kind,
-            number,
-            label: `${CAPTION_NAMES[kind]} ${number}`,
-            line: start + 1,
-            blockStart: next,
-            blockEnd,
-            directiveEnd: close + 1,
-        });
-        i = close;
-    }
-    for (const heading of headingIds) {
-        if (ids.has(heading.id))
-            errors.push({
-                line: heading.line,
-                column: 1,
-                message: `heading and caption declare the same anchor "${heading.id}"`,
-            });
-    }
-    return { captions, errors };
-}
-
-/**
- * Render captioned blocks as HTML, leaving other Markdown untouched.
- *
- * A code, table or prose caption's block is left as Markdown inside the
- * wrapper, blank-line separated from its tags exactly as
- * {@link module:engine/content-blocks.renderBlocks} leaves a named block's
- * body — so the surrounding render sees it as part of its own document and a
- * footnote reference inside it resolves against the note's own definitions.
- *
- * A figure's block is still rendered here, because its `<figcaption>` has to
- * be stripped from HTML this pass controls directly; the surrounding render
- * never sees it as Markdown, so a footnote reference inside a figure caption
- * is not resolved, the same gap a block's own title carries.
- */
-export function renderCaptionBlocks(source, renderMarkdown = parser.render.bind(parser), numbers) {
-    const { captions, errors } = scanCaptions(source);
-    if (errors.length) return { markdown: source, errors };
-    const byId = numbers ? new Map(numbers.map((entry) => [entry.id, entry])) : new Map();
-    const lines = String(source ?? "").split("\n");
-    const output = [];
-    let cursor = 0;
-    const escape = (value) =>
-        String(value)
-            .replace(/&/g, "&amp;")
-            .replace(/</g, "&lt;")
-            .replace(/>/g, "&gt;")
-            .replace(/"/g, "&quot;");
-    for (const caption of captions) {
-        output.push(...lines.slice(cursor, caption.line - 1));
-        const numbered = byId.get(caption.id) ?? caption;
-        const block = lines.slice(caption.blockStart, caption.blockEnd).join("\n");
-        const captionHtml = parser.renderInline(caption.text);
-        output.push(
-            `<div id="${escape(slugify(caption.id))}" class="content-caption content-caption-${caption.kind}">`,
-        );
-        if (caption.kind === "figure") {
-            const html = renderMarkdown(block).replace(
-                /<figcaption\b[^>]*>[\s\S]*?<\/figcaption>/i,
-                "",
-            );
-            output.push(html.trim());
-        } else {
-            output.push("", block, "");
-        }
-        output.push(
-            `<p class="content-caption-label">${escape(numbered.label)}: ${captionHtml}</p>`,
-            "</div>",
-        );
-        cursor = caption.blockEnd;
-    }
-    output.push(...lines.slice(cursor));
-    return { markdown: output.join("\n"), errors: [] };
-}

package/types/engine/content-captions.d.mts

@@ -1,58 +0,0 @@
-/** Number a sequence of captions with counters shared across documents. */
-export function numberCaptions(captions: any, counts: any): any;
-/**
- * Read caption directives and the Markdown block immediately after each one.
- * SQL fences are expanded before this pass, so their output is a table.
- *
- * @param {string} source - A note body with expanded tables.
- * @returns {{captions: Array<{id: string, text: string, kind: string, number: number,
- *   label: string, line: number, blockStart: number, blockEnd: number,
- *   directiveEnd: number}>, errors: Array<{line: number, column: number, message: string}>}}
- */
-export function scanCaptions(source: string): {
-    captions: Array<{
-        id: string;
-        text: string;
-        kind: string;
-        number: number;
-        label: string;
-        line: number;
-        blockStart: number;
-        blockEnd: number;
-        directiveEnd: number;
-    }>;
-    errors: Array<{
-        line: number;
-        column: number;
-        message: string;
-    }>;
-};
-/**
- * Render captioned blocks as HTML, leaving other Markdown untouched.
- *
- * A code, table or prose caption's block is left as Markdown inside the
- * wrapper, blank-line separated from its tags exactly as
- * {@link module:engine/content-blocks.renderBlocks} leaves a named block's
- * body — so the surrounding render sees it as part of its own document and a
- * footnote reference inside it resolves against the note's own definitions.
- *
- * A figure's block is still rendered here, because its `<figcaption>` has to
- * be stripped from HTML this pass controls directly; the surrounding render
- * never sees it as Markdown, so a footnote reference inside a figure caption
- * is not resolved, the same gap a block's own title carries.
- */
-export function renderCaptionBlocks(source: any, renderMarkdown: ((src: string, env?: import("markdown-it").Env) => string) | undefined, numbers: any): {
-    markdown: any;
-    errors: {
-        line: number;
-        column: number;
-        message: string;
-    }[];
-};
-/** A caption's automatically determined kind. */
-export const CAPTION_NAMES: Readonly<{
-    code: "Code";
-    table: "Table";
-    figure: "Figure";
-    prose: "Prose";
-}>;