@turntrout/subfont 1.3.3 → 1.4.0

package/README.md

@@ -71,6 +71,7 @@ subfont path/to/index.html -i --cache
 |  `--concurrency N` |         | Max worker threads (capped by available memory, ~50 MB each) |
 |   `--chrome-flags` |         | Custom Chrome flags for `--dynamic`                          |
 |    `--source-maps` |   off   | Preserve CSS source maps (slower)                            |
+|         `--strict` |   off   | Exit non-zero if any warnings are emitted                    |
 |     `-s, --silent` |   off   | Suppress all console output                                  |
 |      `-d, --debug` |   off   | Verbose timing and font glyph detection info                 |
 |  `--relative-urls` |   off   | Emit relative URLs instead of root-relative                  |

package/lib/parseCommandLineOptions.js

@@ -132,6 +132,12 @@ module.exports = function parseCommandLineOptions(argv) {
       type: 'boolean',
       default: false,
     })
+    .options('strict', {
+      describe:
+        'Exit with a non-zero status code if any warnings are emitted during the run',
+      type: 'boolean',
+      default: false,
+    })
     .wrap(require('yargs').terminalWidth());
 
   const { _: inputFiles, ...rest } = yargs.argv;

package/lib/subfont.js

@@ -36,6 +36,7 @@ module.exports = async function subfont(
     concurrency,
     chromeFlags = [],
     cache = false,
+    strict = false,
   },
   console
 ) {
@@ -174,17 +175,19 @@ module.exports = async function subfont(
     );
   }
 
+  let sawWarning = false;
+  const origEmit = assetGraph.emit;
+  assetGraph.emit = function (event, err, ...rest) {
+    if (event === 'warn') {
+      if (isExtensionlessEnoent(err)) return false;
+      sawWarning = true;
+    }
+    return origEmit.call(this, event, err, ...rest);
+  };
   if (silent) {
     assetGraph.on('warn', () => {});
   } else {
-    const origEmit = assetGraph.emit;
-    assetGraph.emit = function (event, err, ...rest) {
-      if (event === 'warn' && isExtensionlessEnoent(err)) {
-        return false;
-      }
-      return origEmit.call(this, event, err, ...rest);
-    };
-    await assetGraph.logEvents({ console });
+    await assetGraph.logEvents({ console, stopOnWarning: strict });
   }
 
   const outerTimings = {};
@@ -346,6 +349,15 @@ module.exports = async function subfont(
       `[subfont timing] post-subsetFonts processing: ${outerTimings['post-subsetFonts processing']}ms`
     );
 
+  if (strict && sawWarning) {
+    // In non-silent mode, assetgraph's logEvents normally exits earlier via
+    // stopOnWarning. This guard covers silent mode and warnings that slipped
+    // past a transform boundary.
+    throw new Error(
+      'subfont: --strict was set and one or more warnings were emitted; refusing to write output.'
+    );
+  }
+
   phaseStart = Date.now();
   if (!dryRun) {
     await assetGraph.writeAssetsToDisc(

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@turntrout/subfont",
-  "version": "1.3.3",
+  "version": "1.4.0",
   "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"