@turntrout/subfont 1.3.1 → 1.3.3

package/lib/collectTextsByPage.js

@@ -231,7 +231,7 @@ function findFontFamiliesWithFeatureSettings(
 
       if (recorded === true) {
         result = true;
-      } else {
+      } else if (result !== true) {
         if (!result) result = new Set();
         for (const family of recorded) {
           result.add(family.toLowerCase());
@@ -1214,3 +1214,5 @@ module.exports = collectTextsByPage;
 // Exported for testing only
 module.exports._extractFeatureTagsFromDecl = extractFeatureTagsFromDecl;
 module.exports._resolveFeatureSettings = resolveFeatureSettings;
+module.exports._findFontFamiliesWithFeatureSettings =
+  findFontFamiliesWithFeatureSettings;

package/lib/subfont.js

@@ -52,6 +52,16 @@ module.exports = async function subfont(
     );
   }
 
+  // Prevent postcss plugins (colormin, convert-values, etc.) invoked by
+  // cssnano from walking the filesystem for a "browserslist" config.
+  // Under pnpm, `node_modules/.bin/browserslist` is a shell shim that
+  // browserslist mis-parses as browser queries, throwing
+  // BrowserslistError and silently aborting CSS minification.
+  // Setting BROWSERSLIST short-circuits the walk entirely.
+  if (!process.env.BROWSERSLIST && !process.env.BROWSERSLIST_CONFIG) {
+    process.env.BROWSERSLIST = 'defaults';
+  }
+
   const formats = ['woff2'];
 
   function logToConsole(severity, ...args) {

package/lib/warnAboutMissingGlyphs.js

@@ -60,43 +60,43 @@ async function warnAboutMissingGlyphs(
         const isMissing = !characterSetLookup.has(codePoint);
 
         if (isMissing) {
-          // Find all occurrences of the missing character in the source,
-          // not just the first, so that every location is reported.
-          const locations = [];
-          const sourceText = htmlOrSvgAsset.text;
+          // Report only the first location plus a count of remaining
+          // occurrences. A character like U+200B can appear thousands of
+          // times on a page and per-occurrence lines drown the log.
+          let firstLocation;
+          let occurrences = 0;
           if (char.length > 0) {
+            const sourceText = htmlOrSvgAsset.text;
             let searchIdx = 0;
             while (true) {
               const charIdx = sourceText.indexOf(char, searchIdx);
               if (charIdx === -1) break;
-              if (!linesAndColumns) {
-                linesAndColumns = new LinesAndColumns(sourceText);
+              occurrences++;
+              if (occurrences === 1) {
+                if (!linesAndColumns) {
+                  linesAndColumns = new LinesAndColumns(sourceText);
+                }
+                const position = linesAndColumns.locationForIndex(charIdx);
+                firstLocation = `${htmlOrSvgAsset.urlOrDescription}:${
+                  position.line + 1
+                }:${position.column + 1}`;
               }
-              const position = linesAndColumns.locationForIndex(charIdx);
-              locations.push(
-                `${htmlOrSvgAsset.urlOrDescription}:${position.line + 1}:${
-                  position.column + 1
-                }`
-              );
               searchIdx = charIdx + char.length;
             }
           }
 
-          if (locations.length === 0) {
-            locations.push(
-              `${htmlOrSvgAsset.urlOrDescription} (generated content)`
-            );
+          if (!firstLocation) {
+            firstLocation = `${htmlOrSvgAsset.urlOrDescription} (generated content)`;
           }
 
-          for (const location of locations) {
-            missingGlyphsErrors.push({
-              codePoint,
-              char,
-              htmlOrSvgAsset,
-              fontUsage,
-              location,
-            });
-          }
+          missingGlyphsErrors.push({
+            codePoint,
+            char,
+            htmlOrSvgAsset,
+            fontUsage,
+            location: firstLocation,
+            occurrences,
+          });
           missedAny = true;
         }
       }
@@ -123,12 +123,14 @@ async function warnAboutMissingGlyphs(
 
   if (missingGlyphsErrors.length) {
     const errorLog = missingGlyphsErrors.map(
-      ({ char, fontUsage, location }) =>
-        `- \\u{${char.codePointAt(0).toString(16)}} (${char}) in font-family '${
+      ({ char, fontUsage, location, occurrences }) => {
+        const extra = occurrences > 1 ? ` (+${occurrences - 1} more)` : '';
+        return `- \\u{${char.codePointAt(0).toString(16)}} (${char}) in font-family '${
           fontUsage.props['font-family']
         }' (${fontUsage.props['font-weight']}/${
           fontUsage.props['font-style']
-        }) at ${location}`
+        }) at ${location}${extra}`;
+      }
     );
 
     const message = `Missing glyph fallback detected.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@turntrout/subfont",
-  "version": "1.3.1",
+  "version": "1.3.3",
   "description": "Automatically subset web fonts to only the characters used on your pages. Fork of Munter/subfont with modern defaults.",
   "engines": {
     "node": ">=18.0.0"