fix(publisher): news mapper critical and test issues

CRITICAL: Removed ai_generated from news_articles select string — column doesn't
exist on news_articles, only on blog_articles. Kept mapper guard for future-proofing
when the column is eventually added. Added comment explaining the pattern.

IMPORTANT: Enhanced happy-path test with real Markdown assertion (HTML→MD transform)
and verified all t tags present (both 'news' and 'stadt'). Added dedicated test for
ai_generated guard with synthetic row to pin intent when column is later added.

Author: MaxBrych

packages/publisher/src/mappers.ts

@@ -322,6 +322,9 @@ export function newsToSpec(row: Row, htmlToMd: (html: string) => string): Publis
   if (publishedAt) tags.push(["published_at", String(Math.floor(Date.parse(publishedAt) / 1000))]);
   const category = str(row, "category");
   if (category && category !== "news") tags.push(["t", category]);
+  // ai_generated does not yet exist on news_articles (only on blog_articles). The guard
+  // is harmless when the field is absent and future-proofs the mapper if the column is
+  // later added, following the pattern of blog_articles migration.
   if (row["ai_generated"] === true) tags.push(["ai_generated", "true"]);
 
   return {

packages/publisher/src/sync.ts

@@ -168,7 +168,7 @@ export async function buildSpecs(
   if (deps.datasets.includes("news")) {
     const rows = await deps.fetchRows(
       "news_articles",
-      "select=id,slug,title,excerpt,content,cover_image_url,category,published_at,ai_generated,status,updated_at,created_at&status=eq.published",
+      "select=id,slug,title,excerpt,content,cover_image_url,category,published_at,status,updated_at,created_at&status=eq.published",
     );
     for (const row of rows) {
       const spec = newsToSpec(row, htmlToMarkdown);

packages/publisher/test/mappers.test.ts

@@ -206,7 +206,13 @@ describe("town news mapping (NIP-23)", () => {
     const tag = (n: string) => spec.tags.find((t) => t[0] === n)?.[1];
     assert.equal(tag("title"), "Stadtfest");
     assert.equal(tag("slug"), "stadtfest-2026");
-    assert.equal(tag("t"), "news");
+    // Verify HTML→Markdown: no HTML tags remain, real Markdown produced (bold as **)
+    assert.match(spec.content, /Hallo \*\*Röbel\*\*/);
+    assert.doesNotMatch(spec.content, /<[a-z]+>/i);
+    // Collect ALL t tags (not just the first) — should have both "news" and category "stadt"
+    const allTags = spec.tags.filter((t) => t[0] === "t").map((t) => t[1]);
+    assert.ok(allTags.includes("news"));
+    assert.ok(allTags.includes("stadt"));
     assert.equal(tag("ai_generated"), "true");
   });
 
@@ -232,6 +238,16 @@ describe("town news mapping (NIP-23)", () => {
     assert.ok(!json.includes("leak@example.com"));
     assert.ok(!json.includes("0xDEADBEEF"));
   });
+
+  it("newsToSpec: ai_generated flag is future-proofed (column does not exist on news_articles yet)", () => {
+    // ai_generated exists only on blog_articles; when the schema adds it to news_articles,
+    // the mapper already has the guard ready. Test with synthetic row to pin the intent.
+    const spec = newsToSpec(
+      { id: "n4", slug: "s", title: "T", content: "ok", status: "published", updated_at: "2026-07-02T10:00:00Z", ai_generated: true },
+      htmlToMarkdown,
+    )!;
+    assert.deepEqual(spec.tags.find((t) => t[0] === "ai_generated"), ["ai_generated", "true"]);
+  });
 });
 
 describe("marketplace mapping (NIP-15) — withdrawal FIRST", () => {