feat(quoteforge[warnings]): warn when a template drops a block type it cannot render (#11)

Author: lordvins226

CHANGELOG.md

@@ -1,5 +1,15 @@
 # Changelog
 
+## Unreleased
+
+### Added
+- **`generate` and `slides` now warn when a template drops a block.** 15 of the 28 templates
+  render only a subset of block types and silently discard the rest — a `headline` on `memo`,
+  a `text` block on `ticket`, anything but `bullet-list`/`callout` on `grid`, `versus`, and
+  `prompt`. The card validated, rendered, exited 0, and was missing content the author had
+  written. The warning names the template and the dropped types. It reads each template's own
+  `block.type` guards at runtime, so it cannot drift from the markup.
+
 ## 1.1.0
 
 ### Changed

src/__tests__/template-warnings.test.ts

@@ -55,4 +55,64 @@ describe("templateWarnings", () => {
   test("stays silent when no eyebrow is set", () => {
     expect(templateWarnings({ template: "list", blocks: [] })).toEqual([]);
   });
+
+  test("warns when a template drops a block type it cannot render", () => {
+    const [warning] = templateWarnings({
+      template: "memo",
+      blocks: [
+        { type: "headline" },
+        { type: "callout", items: [{}] },
+      ],
+    });
+    expect(warning).toContain("memo");
+    expect(warning).toContain("headline");
+    expect(warning).toContain("will not appear");
+  });
+
+  test("names every dropped type once, without repeating duplicates", () => {
+    const [warning] = templateWarnings({
+      template: "prompt",
+      blocks: [{ type: "headline" }, { type: "headline" }, { type: "text" }],
+    });
+    expect(warning).toContain("headline");
+    expect(warning).toContain("text");
+    expect((warning ?? "").match(/headline/g)).toHaveLength(1);
+  });
+
+  test("stays silent when every block is one the template renders", () => {
+    expect(
+      templateWarnings({
+        template: "memo",
+        blocks: [{ type: "callout", items: [{}] }, { type: "text" }],
+      }),
+    ).toEqual([]);
+  });
+
+  test("stays silent for templates that render every block type", () => {
+    expect(
+      templateWarnings({
+        template: "manifesto",
+        blocks: [{ type: "headline" }, { type: "code" }, { type: "chart" }],
+      }),
+    ).toEqual([]);
+  });
+
+  test("says nothing about blocks when the template is unknown", () => {
+    expect(
+      templateWarnings({ template: "no-such-template", blocks: [{ type: "headline" }] }),
+    ).toEqual([]);
+  });
+
+  test("every shipped example renders all of its own blocks", async () => {
+    const { readdirSync, readFileSync } = await import("node:fs");
+    const dir = "content/examples";
+    for (const file of readdirSync(dir).filter((f) => f.endsWith(".json"))) {
+      const card = JSON.parse(readFileSync(`${dir}/${file}`, "utf-8")) as {
+        template: string;
+        eyebrow?: string;
+        blocks: { type: string }[];
+      };
+      expect([file, templateWarnings(card)]).toEqual([file, []]);
+    }
+  });
 });

src/renderer/template-warnings.ts

@@ -24,6 +24,12 @@ function templateSource(template: string): string {
   }
 }
 
+function handledBlockTypes(template: string): Set<string> | undefined {
+  const matches = templateSource(template).matchAll(/block\.type\s*==\s*"([a-z-]+)"/g);
+  const types = new Set([...matches].map((m) => m[1] as string));
+  return types.size > 0 ? types : undefined;
+}
+
 /**
  * Non-fatal layout advice. A card that trips one of these still renders — the
  * result just will not look like the template intends.
@@ -50,5 +56,21 @@ export function templateWarnings(card: WarnableCard): string[] {
     );
   }
 
+  const handled = handledBlockTypes(card.template);
+  if (handled) {
+    const dropped = [...new Set(card.blocks.map((b) => b.type))].filter(
+      (type) => !handled.has(type),
+    );
+
+    if (dropped.length > 0) {
+      const list = dropped.map((type) => `"${type}"`).join(", ");
+      warnings.push(
+        `Template "${card.template}" does not render ${list} — that content will not appear ` +
+          `in the image. Pick a template that handles it, or move the text into a block the ` +
+          `template renders.`,
+      );
+    }
+  }
+
   return warnings;
 }