@liiift-studio/sanity-font-manager 2.5.3 → 2.5.5

package/package.json

@@ -1,6 +1,6 @@
 {
 	"name": "@liiift-studio/sanity-font-manager",
-	"version": "2.5.3",
+	"version": "2.5.5",
 	"description": "Sanity Studio plugin — full font management suite with batch upload, format conversion, metadata extraction, CSS generation, collection/pair generation, and script variant support. Supports Sanity v3, v4, and v5.",
 	"license": "MIT",
 	"author": "Liiift Studio",
@@ -51,9 +51,7 @@
 	},
 	"dependencies": {
 		"base-64": "^1.0.0",
-		"lib-font": "^3.0.1",
 		"nanoid": "^5.0.0",
-		"pako": "^2.1.0",
 		"slugify": "^1.6.6"
 	},
 	"peerDependencies": {

package/src/index.js

@@ -1,5 +1,9 @@
 // Entry point for @liiift-studio/sanity-font-manager — exports all font manager components, hooks, and utilities
 
+// Bootstrap decompressors FIRST — sets globalThis.pako and globalThis.unbrotli
+// before lib-font can be evaluated by any bundler. This must be the first import.
+import './utils/setupDecompressors.js';
+
 // Components
 export { BatchUploadFonts } from './components/BatchUploadFonts.jsx';
 export { GenerateCollectionsPairsComponent } from './components/GenerateCollectionsPairsComponent.jsx';

package/src/utils/parseFont.js

@@ -1,26 +1,21 @@
-// Async font parser — wraps lib-font event model in a Promise with decompressor bootstrap
-
-// Lazy-loaded lib-font Font constructor — resolved on first parseFont() call.
-// All decompressor globals (pako for WOFF, unbrotli for WOFF2) are set dynamically
-// inside getFont() BEFORE lib-font is imported, guaranteeing correct evaluation order
-// regardless of how the bundler (tsup/esbuild/vite) reorders static imports.
-let _Font = null;
-
-/** Returns the lib-font Font constructor, bootstrapping decompressors on first call */
-async function getFont() {
-	if (!_Font) {
-		// Set pako (zlib) for WOFF decompression
-		const pako = await import('pako');
-		globalThis.pako = pako.default || pako;
-
-		// Set unbrotli for WOFF2 decompression — UMD side-effect sets globalThis.unbrotli
-		await import('../vendor/unbrotli.js');
-
-		// NOW safe to import lib-font — both globals are set
-		const mod = await import('lib-font');
-		_Font = mod.Font;
-	}
-	return _Font;
+// Async font parser — ensures decompressor globals are set, with graceful WOFF2 fallback
+
+import pako from 'pako';
+import '../vendor/unbrotli.js';
+import { Font } from 'lib-font';
+
+// Immediately set globals — this runs at module evaluation time.
+// lib-font's woff.js reads globalThis.pako and woff2.js reads globalThis.unbrotli
+// at their top level. If this module evaluates before lib-font (which tsup guarantees
+// since we import pako and unbrotli first), the globals will be set in time.
+//
+// When Vite re-bundles our package, it may reorder evaluation so lib-font's woff2.js
+// captures globalThis.unbrotli as undefined. In that case, WOFF2 parsing will fail
+// and buildUploadPlan falls back to TTF companion metadata.
+globalThis.pako = pako;
+// unbrotli.js UMD sets globalThis.unbrotli as a side effect — verify it worked
+if (!globalThis.unbrotli) {
+	console.warn('[parseFont] globalThis.unbrotli not set after vendor import — WOFF2 parsing may fail. TTF/OTF files will still work.');
 }
 
 /** Maximum font file size accepted for parsing (50 MB) */
@@ -30,6 +25,11 @@ const MAX_FONT_FILE_SIZE = 50 * 1024 * 1024;
  * Parse a font file from an ArrayBuffer.
  * Returns a lib-font Font object with all tables accessible via font.opentype.tables.*.
  *
+ * WOFF2 note: If the brotli decoder couldn't be initialized (common with Vite pre-bundling),
+ * WOFF2 files will fail to parse. The upload plan handles this gracefully — WOFF2 files
+ * that share a name with a TTF/OTF get metadata from the companion file. Standalone WOFF2
+ * uploads will show a clear error directing the user to also include TTF/OTF files.
+ *
  * @param {ArrayBuffer} buffer - Raw font file bytes
  * @param {string} filename - Original filename (used for format detection by lib-font)
  * @returns {Promise<import('lib-font').Font>} Parsed lib-font Font object
@@ -40,12 +40,18 @@ export async function parseFont(buffer, filename) {
 		throw new Error(`Font file exceeds ${MAX_FONT_FILE_SIZE / 1024 / 1024}MB limit: ${filename} (${(buffer.byteLength / 1024 / 1024).toFixed(1)}MB)`);
 	}
 
-	const Font = await getFont();
-
 	return new Promise((resolve, reject) => {
 		const font = new Font('font', { skipStyleSheet: true });
 		font.onload = (evt) => resolve(evt.detail.font);
-		font.onerror = (evt) => reject(new Error(evt.detail?.message || `Failed to parse ${filename}`));
-		font.fromDataBuffer(buffer, filename);
+		font.onerror = (evt) => {
+			const msg = evt.detail?.message || `Failed to parse ${filename}`;
+			reject(new Error(msg));
+		};
+		try {
+			font.fromDataBuffer(buffer, filename);
+		} catch (err) {
+			// lib-font may throw synchronously from WOFF2 constructor if brotli decoder is missing
+			reject(new Error(`${filename}: ${err.message}`));
+		}
 	});
 }

package/src/utils/setupDecompressors.js

@@ -0,0 +1,10 @@
+// Sets up globalThis.pako and globalThis.unbrotli for lib-font WOFF/WOFF2 decompression.
+// This module MUST be imported before lib-font is ever evaluated.
+// It runs synchronously at module evaluation time to set the globals.
+
+import pako from 'pako';
+import '../vendor/unbrotli.js';
+
+globalThis.pako = pako;
+// unbrotli.js is a UMD that sets globalThis.unbrotli as a side effect on evaluation.
+// The import above triggers that side effect.

package/dist/UploadModal-NQZGBJAH.js

@@ -1,7 +0,0 @@
-"use strict";Object.defineProperty(exports, "__esModule", {value: true});
-
-var _chunkPGSHWNGWjs = require('./chunk-PGSHWNGW.js');
-require('./chunk-MCKGQKYU.js');
-
-
-exports.default = _chunkPGSHWNGWjs.UploadModal;

package/dist/UploadModal-YJCVHM63.mjs

@@ -1,7 +0,0 @@
-import {
-  UploadModal
-} from "./chunk-WSQYNIIZ.mjs";
-import "./chunk-EBO3CZXG.mjs";
-export {
-  UploadModal as default
-};

package/dist/chunk-EBO3CZXG.mjs

@@ -1,15 +0,0 @@
-var __getOwnPropNames = Object.getOwnPropertyNames;
-var __require = /* @__PURE__ */ ((x) => typeof require !== "undefined" ? require : typeof Proxy !== "undefined" ? new Proxy(x, {
-  get: (a, b) => (typeof require !== "undefined" ? require : a)[b]
-}) : x)(function(x) {
-  if (typeof require !== "undefined") return require.apply(this, arguments);
-  throw Error('Dynamic require of "' + x + '" is not supported');
-});
-var __commonJS = (cb, mod) => function __require2() {
-  return mod || (0, cb[__getOwnPropNames(cb)[0]])((mod = { exports: {} }).exports, mod), mod.exports;
-};
-
-export {
-  __require,
-  __commonJS
-};

package/dist/chunk-MCKGQKYU.js

@@ -1,15 +0,0 @@
-"use strict";Object.defineProperty(exports, "__esModule", {value: true});var __getOwnPropNames = Object.getOwnPropertyNames;
-var __require = /* @__PURE__ */ ((x) => typeof require !== "undefined" ? require : typeof Proxy !== "undefined" ? new Proxy(x, {
-  get: (a, b) => (typeof require !== "undefined" ? require : a)[b]
-}) : x)(function(x) {
-  if (typeof require !== "undefined") return require.apply(this, arguments);
-  throw Error('Dynamic require of "' + x + '" is not supported');
-});
-var __commonJS = (cb, mod) => function __require2() {
-  return mod || (0, cb[__getOwnPropNames(cb)[0]])((mod = { exports: {} }).exports, mod), mod.exports;
-};
-
-
-
-
-exports.__require = __require; exports.__commonJS = __commonJS;