From fdb6c2ecfe17654d90d891e4153f8e825fead9bc Mon Sep 17 00:00:00 2001
From: funman300 <funman300@gmail.com>
Date: Fri, 1 May 2026 21:41:35 +0000
Subject: [PATCH] fix(engine): bundle FiraMono into SVG fontdb as last-resort
 fallback
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

The hayeah card SVGs reference Bitstream Vera Sans and Arial by name.
The lenient FontResolver from efa063f appends Family::SansSerif and
Family::Serif so unmatched named families fall through to whatever
the system serves under those CSS generics — which works on machines
with a normal fontconfig setup, and silently fails on minimal Linux
installs, fresh Wayland sessions, or chroots where the generic
aliases don't resolve to anything either. The visible symptom on the
player's second machine was "card font didn't carry over": rank and
suit glyphs vanished from the cards because every lookup path hit a
None.

shared_fontdb now also include_bytes!()s the bundled
assets/fonts/main.ttf into the fontdb after load_system_fonts, and
pins each CSS generic (sans-serif, serif, monospace, cursive,
fantasy) to "Fira Mono". Named-family lookups still prefer the
system db first when those families exist, so machines with a normal
font setup behave identically; only when SansSerif/Serif fall through
does the resolver land on FiraMono — guaranteed present because it's
embedded in the binary.

The bundled font is ~170 KB; the binary already include_bytes!()s the
six audio WAVs and the embedded card-theme SVGs, so this fits the
existing self-contained-binary policy.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---
 solitaire_engine/src/assets/svg_loader.rs | 43 +++++++++++++++++++----
 1 file changed, 36 insertions(+), 7 deletions(-)

diff --git a/solitaire_engine/src/assets/svg_loader.rs b/solitaire_engine/src/assets/svg_loader.rs
index fa1ed7e..97c7db0 100644
--- a/solitaire_engine/src/assets/svg_loader.rs
+++ b/solitaire_engine/src/assets/svg_loader.rs
@@ -153,13 +153,28 @@ pub fn rasterize_svg(svg_bytes: &[u8], target: UVec2) -> Result<Image, SvgLoader
 }
 
 /// Returns a process-wide font database populated with the OS-installed
-/// fonts the user has available. Initialised lazily on first SVG that
-/// references text, then shared (via `Arc`) across every subsequent
-/// rasterisation. `usvg::Options::default()` ships an empty `fontdb`,
-/// so without this call any text glyph in an SVG renders with no font
-/// match — the visible symptom on the bundled hayeah artwork is the
-/// "No match for Arial font-family" warn spam plus glyphs that fall
-/// through to whatever shape-only path usvg uses for missing fonts.
+/// fonts plus the bundled FiraMono-Medium face. Initialised lazily on
+/// first SVG that references text, then shared (via `Arc`) across every
+/// subsequent rasterisation.
+///
+/// `usvg::Options::default()` ships an empty `fontdb`, so without this
+/// call any text glyph in an SVG renders with no font match — the
+/// visible symptom on the bundled hayeah artwork is the "No match for
+/// Arial font-family" warn spam plus glyphs that fall through to
+/// whatever shape-only path usvg uses for missing fonts.
+///
+/// **Bundled font as last-resort fallback.** Loading only system fonts
+/// breaks on minimal Linux installs, fresh Wayland sessions, and
+/// chroots where fontconfig has nothing usable to serve as
+/// `sans-serif`. The cards on the bundled hayeah theme reference
+/// `Bitstream Vera Sans` and `Arial` by name — if neither is installed
+/// AND the resolver's CSS-generic fallbacks (`SansSerif`/`Serif`) also
+/// don't resolve, the rank/suit text vanishes entirely. Loading the
+/// project's bundled FiraMono via `include_bytes!()` and pinning it as
+/// the generic-family target guarantees a working last-resort glyph
+/// source on every machine. This was the cause of "card font didn't
+/// carry over" on a fresh second-machine pull.
+///
 /// `load_system_fonts` is comparatively expensive (~50–200 ms on a
 /// typical desktop) so we only pay it once for the lifetime of the
 /// process, gated by `OnceLock`.
@@ -168,6 +183,20 @@ fn shared_fontdb() -> Arc<fontdb::Database> {
     DB.get_or_init(|| {
         let mut db = fontdb::Database::new();
         db.load_system_fonts();
+        // The bundled FiraMono lives at the workspace root, so the
+        // include_bytes! path goes up three levels from this source
+        // file (assets → src → solitaire_engine → workspace root).
+        db.load_font_data(include_bytes!("../../../assets/fonts/main.ttf").to_vec());
+        // Pin the CSS generics to the bundled face as the resolution
+        // target. Named-family lookups (Bitstream Vera Sans, Arial)
+        // still try the system db first; only when those miss does
+        // the resolver fall through to SansSerif / Serif, and now
+        // those are guaranteed to land on FiraMono.
+        db.set_sans_serif_family("Fira Mono");
+        db.set_serif_family("Fira Mono");
+        db.set_monospace_family("Fira Mono");
+        db.set_cursive_family("Fira Mono");
+        db.set_fantasy_family("Fira Mono");
         Arc::new(db)
     })
     .clone()