@lutrin/core 1.1.0 → 1.1.1

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@lutrin/core",
-  "version": "1.1.0",
+  "version": "1.1.1",
   "description": "Markdown → PowerPoint / HTML presentation compiler, themable: a generic default theme, organization brand kits distributed as .deckkit archives.",
   "type": "module",
   "scripts": {

package/src/deck/assets.mjs

@@ -642,11 +642,15 @@ export async function iconSvg(name, { color = 'primary' } = {}) {
   return svg.replace(/currentColor/g, `#${hex}`);
 }
 
+/** Icon raster density (px) — shared with the callers that separate the SVG
+ *  lookup from the rasterization to tell the two failures apart. */
+export const ICON_RASTER_PX = 384;
+
 /**
  * Renders a Lucide icon as a recolored PNG.
  * @returns {{png:Buffer,w:number,h:number}|null}
  */
-export async function renderIcon(name, { color = 'primary', rasterPx = 384 } = {}) {
+export async function renderIcon(name, { color = 'primary', rasterPx = ICON_RASTER_PX } = {}) {
   const svg = await iconSvg(name, { color });
   return svg ? svgToPng(svg, rasterPx) : null;
 }

package/src/html/render.mjs

@@ -910,7 +910,14 @@ body{margin:0;background:#${C.underground2};font-family:"${FONTS.body}",-apple-s
 .el{position:absolute;margin:0}
 a{color:#${C.primary};text-decoration:none}
 a:hover{text-decoration:underline}
-code{font-family:"${FONTS.mono}",monospace;color:#${C.primaryDarker}}
+/* inline code — every surface property EXPLICIT, not only the ones the theme
+   cares about. The fragment mode injects this CSS into hosts (VS Code
+   webview, Obsidian) whose own default stylesheets give "code" a padded,
+   theme-colored chip (VS Code: --vscode-textPreformat-background); any
+   property left undeclared here is repainted by the host — dark chips under
+   dark editor themes, unreadable on a light slide. (No backticks in these
+   comments: we are inside a JS template literal.) */
+code{font-family:"${FONTS.mono}",monospace;color:#${C.primaryDarker};background:transparent;padding:0;border-radius:0}
 
 /* chrome of content slides */
 .slide-title{position:absolute;left:${PAGE.margin}px;top:${SPACE.lg}px;width:${PAGE.width - 2 * PAGE.margin}px;height:${PAGE.titleHeight - SPACE.lg - 8}px;display:flex;align-items:center;font-size:${TYPE.slideTitle}pt;font-weight:700;line-height:1.15}
@@ -969,7 +976,10 @@ code{font-family:"${FONTS.mono}",monospace;color:#${C.primaryDarker}}
 .tl-axis-v.tl-no-arrow{background-size:2px 100%}
 .tl-arrow-v{position:absolute;bottom:0;left:50%;transform:translateX(-50%);width:0;height:0;border-top:14px solid #${C.neutralStroke};border-left:7px solid transparent;border-right:7px solid transparent}
 .tl-dot{display:flex;align-items:center;justify-content:center;background:#${C.primary};color:#${C.ground};border:2px solid #${C.ground};border-radius:50%;font-size:${TYPE.metricLabel}pt;font-weight:700;box-sizing:border-box}
-.quote blockquote{position:absolute;left:96px;right:32px;top:0;bottom:64px;display:flex;align-items:center;margin:0;font-size:${TYPE.quote}pt;font-style:italic;line-height:1.4}
+/* same host-default hazard as "code" above: the webview paints blockquote
+   with --vscode-textBlockQuote-background and a left border — neutralized
+   explicitly */
+.quote blockquote{position:absolute;left:96px;right:32px;top:0;bottom:64px;display:flex;align-items:center;margin:0;padding:0;background:transparent;border:0;font-size:${TYPE.quote}pt;font-style:italic;line-height:1.4}
 .quote-mark{position:absolute;left:0;top:-10px;font-size:72pt;font-weight:700;color:#${C.primary};line-height:1}
 .quote figcaption{position:absolute;right:32px;bottom:12px;font-size:${TYPE.body}pt;color:#${C.neutralSecondary}}
 .img-contain{object-fit:contain}

package/src/pptx/fonts.mjs

@@ -52,22 +52,29 @@ const FSTYPE_OFFSET = 8;
 const FSTYPE_RESTRICTED = 0x0002; // "Restricted License embedding": no
 
 /**
- * fsType of an sfnt font whose header (offset table) starts at `base`.
+ * Table record of an sfnt font whose header (offset table) starts at `base`.
  * The header: version on 4 bytes, numTables on 2, then three binary-search
  * fields we ignore; the table records follow at +12, 16 bytes each (tag,
- * checksum, offset, length).
- * @returns {number|null} null if the font has no OS/2 table
+ * checksum, offset, length). Offsets are counted from the START OF THE FILE —
+ * that is what lets two fonts of a .ttc collection share a table.
+ * @returns {{offset:number,length:number}|null}
  */
-function sfntFsType(buf, base) {
+function sfntTable(buf, base, tag) {
   const numTables = buf.readUInt16BE(base + 4);
   for (let k = 0; k < numTables; k++) {
     const rec = base + 12 + k * 16;
-    if (buf.toString('latin1', rec, rec + 4) !== 'OS/2') continue;
-    return buf.readUInt16BE(buf.readUInt32BE(rec + 8) + FSTYPE_OFFSET);
+    if (buf.toString('latin1', rec, rec + 4) === tag)
+      return { offset: buf.readUInt32BE(rec + 8), length: buf.readUInt32BE(rec + 12) };
   }
   return null;
 }
 
+/** fsType of an sfnt font. @returns {number|null} null without an OS/2 table */
+function sfntFsType(buf, base) {
+  const os2 = sfntTable(buf, base, 'OS/2');
+  return os2 ? buf.readUInt16BE(os2.offset + FSTYPE_OFFSET) : null;
+}
+
 /**
  * Reads the embedding permission declared by a font file.
  *
@@ -106,6 +113,99 @@ export function readFsType(file) {
   }
 }
 
+// ---------------------------------------------------------------------------
+// Embedded-font identity: the family name and style bits Windows matches by
+// ---------------------------------------------------------------------------
+
+/** fsSelection sits at +62 in the OS/2 table (after the vendor tag);
+ *  macStyle at +44 in head. Bit layouts differ: fsSelection carries italic in
+ *  bit 0 and bold in bit 5, macStyle bold in bit 0 and italic in bit 1. */
+const FSSELECTION_OFFSET = 62;
+const FSSELECTION_ITALIC = 0x01;
+const FSSELECTION_BOLD = 0x20;
+const MACSTYLE_OFFSET = 44;
+const MACSTYLE_BOLD = 0x01;
+const MACSTYLE_ITALIC = 0x02;
+
+/**
+ * The identity PowerPoint on Windows will match the embedded font by.
+ *
+ * GDI knows nothing of the `typeface` we write around the fntdata part: it
+ * registers the font under ITS OWN family name — name table, nameID 1, the
+ * Windows platform (3) records, UTF-16BE — and pairs the four styles of a
+ * family through the bold/italic bits (OS/2 fsSelection, head macStyle as a
+ * fallback). NameID 16, the "typographic family" that lets macOS group
+ * single-style webfont families under one name, is precisely what GDI does
+ * NOT read — which is how a deck can be flawless on the Mac that produced it
+ * and greet every Windows recipient with "unable to install embedded fonts".
+ *
+ * @param {string} file path to a .ttf, .otf or .ttc (first font read)
+ * @returns {{family:string|null,bold:boolean,italic:boolean}|null}
+ *          null if the file cannot be read at all; `family` null when the
+ *          font carries no Windows-platform family record (nothing to check).
+ */
+export function readFontIdentity(file) {
+  let buf;
+  try {
+    buf = fs.readFileSync(file);
+  } catch {
+    return null;
+  }
+  try {
+    const base = buf.toString('latin1', 0, 4) === 'ttcf' ? buf.readUInt32BE(12) : 0;
+
+    let family = null;
+    const name = sfntTable(buf, base, 'name');
+    if (name) {
+      const count = buf.readUInt16BE(name.offset + 2);
+      const storage = name.offset + buf.readUInt16BE(name.offset + 4);
+      let candidate = null; // any Windows record; an en-US one wins
+      for (let k = 0; k < count; k++) {
+        const rec = name.offset + 6 + k * 12;
+        const platform = buf.readUInt16BE(rec);
+        const language = buf.readUInt16BE(rec + 4);
+        const nameId = buf.readUInt16BE(rec + 6);
+        if (nameId !== 1 || (platform !== 3 && platform !== 0)) continue;
+        const at = storage + buf.readUInt16BE(rec + 10);
+        // copy before swap16(): it swaps IN PLACE and subarray() is a view
+        const value = Buffer.from(buf.subarray(at, at + buf.readUInt16BE(rec + 8)))
+          .swap16() // UTF-16BE in the file, utf16le for Buffer
+          .toString('utf16le');
+        if (platform === 3 && language === 0x409) {
+          candidate = value;
+          break;
+        }
+        candidate ??= value;
+      }
+      family = candidate;
+    }
+
+    let bold = false;
+    let italic = false;
+    const os2 = sfntTable(buf, base, 'OS/2');
+    if (os2 && os2.length >= FSSELECTION_OFFSET + 2) {
+      const fsSelection = buf.readUInt16BE(os2.offset + FSSELECTION_OFFSET);
+      bold = Boolean(fsSelection & FSSELECTION_BOLD);
+      italic = Boolean(fsSelection & FSSELECTION_ITALIC);
+    } else {
+      const head = sfntTable(buf, base, 'head');
+      if (head) {
+        const macStyle = buf.readUInt16BE(head.offset + MACSTYLE_OFFSET);
+        bold = Boolean(macStyle & MACSTYLE_BOLD);
+        italic = Boolean(macStyle & MACSTYLE_ITALIC);
+      }
+    }
+    return { family, bold, italic };
+  } catch {
+    // offset out of bounds, truncated table: malformed font file
+    return null;
+  }
+}
+
+/** Same string as far as Windows font matching cares: GDI compares family
+ *  names case-insensitively, and "é" may travel composed or decomposed. */
+const sameFamily = (a, b) => a.normalize('NFC').toLowerCase() === b.normalize('NFC').toLowerCase();
+
 /**
  * Embeds the brand font into a .pptx already written to disk.
  * Idempotent; touches nothing if the TTFs are absent.
@@ -123,7 +223,7 @@ export async function embedFonts(pptxPath) {
 
   // license filter: what is refused does not go into the deliverable, and the
   // author learns it here rather than by receiving a cease-and-desist
-  const variants = present.filter((v) => {
+  const licensed = present.filter((v) => {
     const fsType = readFsType(v.file);
     if (fsType == null) {
       warnings.push(
@@ -140,6 +240,44 @@ export async function embedFonts(pptxPath) {
     return true;
   });
 
+  // GDI coherence filter: Windows matches an embedded variant by the font's
+  // OWN identity (readFontIdentity), never by the typeface we declare around
+  // it. Webfonts commonly ship each weight as its own single-style family —
+  // the CSS @font-face re-groups them, macOS re-groups them through nameID 16,
+  // and GDI does neither: embedded as they are, every Windows recipient gets
+  // the "unable to install some embedded fonts / general failure" dialog at
+  // opening and reads the deck in the fallback font. Such a variant is NOT
+  // embedded — the deck keeps the documented installed-font fallback — and
+  // the author learns which table to rebuild, here rather than from a user's
+  // screenshot.
+  const STYLE_OF = {
+    regular: { bold: false, italic: false },
+    bold: { bold: true, italic: false },
+    italic: { bold: false, italic: true },
+  };
+  const variants = licensed.filter((v) => {
+    const id = readFontIdentity(v.file);
+    // unreadable or nameless: nothing to check against — the benefit of the
+    // doubt, same side as the unreadable fsType above
+    if (!id?.family) return true;
+    if (!sameFamily(id.family, FONTS.body)) {
+      warnings.push(
+        `Font "${path.basename(v.file)}" (${v.key}): its Windows family name is "${id.family}" while the theme declares "${FONTS.body}" — PowerPoint on Windows matches by that name and would fail to install the font at every recipient ("unable to install embedded fonts"). Not embedded; rebuild the font's name table (nameID 1) as "${FONTS.body}", or point fonts.files at a desktop cut of the family.`,
+      );
+      return false;
+    }
+    const want = STYLE_OF[v.key];
+    if (id.bold !== want.bold || id.italic !== want.italic) {
+      const said = (f) =>
+        [f.bold && 'bold', f.italic && 'italic'].filter(Boolean).join('+') || 'regular';
+      warnings.push(
+        `Font "${path.basename(v.file)}" (${v.key}): its style bits say "${said(id)}" where the ${v.key} slot requires "${said(want)}" — Windows pairs the styles of a family by those bits (OS/2 fsSelection, head macStyle) and would fail to install the font at every recipient. Not embedded; rebuild the font's style bits for the slot.`,
+      );
+      return false;
+    }
+    return true;
+  });
+
   if (!variants.length) return { count: 0, warnings };
 
   const zip = await JSZip.loadAsync(fs.readFileSync(pptxPath));

package/src/pptx/render.mjs

@@ -33,8 +33,9 @@ import {
 import { ALERT_BLOCK_TYPES, runsToText } from '../deck/parse.mjs';
 import {
   fetchRemoteImage,
+  iconSvg,
+  ICON_RASTER_PX,
   imageDims,
-  renderIcon,
   renderMath,
   renderMermaidCached,
   rasterAvailable,
@@ -947,6 +948,9 @@ function renderCover(pptx, scene) {
     bold: true,
     color: COLORS.neutralPrimary,
     fontFace: FONTS.body,
+    // explicit: a paragraph without `algn` inherits the master's titleStyle,
+    // which PptxGenJS hard-codes centered — the HTML output is left-aligned
+    align: 'left',
     valign: 'top',
   });
   if (LOGOS.cover && fs.existsSync(LOGOS.cover)) {
@@ -1000,6 +1004,7 @@ function renderSection(pptx, scene) {
     bold: true,
     color: COLORS.ground,
     fontFace: FONTS.body,
+    align: 'left', // same inheritance trap as the cover title
     valign: 'middle',
   });
   if (LOGOS.section && fs.existsSync(LOGOS.section)) {
@@ -1228,13 +1233,20 @@ async function renderDeckTo(scenes, meta, baseDir, outPath, tmp, opts = {}) {
   const iconWarnings = new Array(iconBlocks.length);
   await Promise.all(
     iconBlocks.map(async (b, k) => {
-      const out = await renderIcon(b.name, { color: b.color });
-      if (!out) {
+      // two failures, two diagnostics: an icon whose SVG cannot be FOUND is a
+      // deck problem (typo, package absent, offline), while an SVG found but
+      // not RASTERIZED is the missing resvg binary — already reported, with
+      // its remedy, by RASTER_UNAVAILABLE below. Conflating them sent Windows
+      // users hunting for a network problem when their VSIX simply shipped
+      // another platform's rasterizer.
+      const svg = await iconSvg(b.name, { color: b.color });
+      if (!svg) {
         iconWarnings[k] =
           `Icon "${iconLabel(b.name)}" not found — name unknown to Lucide, or the lucide-static package is absent and there is no network. The slide is rendered without it.`;
         return;
       }
-      icons.set(b, writeTmpPng(tmp(), `icon-${k}-${iconSlug(b.name)}`, out.png));
+      const out = await svgToPng(svg, ICON_RASTER_PX);
+      if (out) icons.set(b, writeTmpPng(tmp(), `icon-${k}-${iconSlug(b.name)}`, out.png));
     }),
   );
   const assetWarnings = iconWarnings.filter(Boolean);
@@ -1335,6 +1347,7 @@ async function renderDeckTo(scenes, meta, baseDir, outPath, tmp, opts = {}) {
           bold: true,
           color: COLORS.neutralPrimary,
           fontFace: FONTS.body,
+          align: 'left', // same inheritance trap as the cover title
           valign: 'middle',
         },
       );