@formatjs/cli-lib 8.2.1 → 8.2.3

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@formatjs/cli-lib",
   "description": "Lib for CLI for formatjs.",
-  "version": "8.2.1",
+  "version": "8.2.3",
   "license": "MIT",
   "author": "Linjie Ding <linjie@airtable.com>",
   "type": "module",
@@ -20,9 +20,9 @@
     "loud-rejection": "^2",
     "tslib": "^2.8.1",
     "typescript": "^5.6.0",
-    "@formatjs/icu-messageformat-parser": "3.5.0",
-    "@formatjs/icu-skeleton-parser": "2.1.0",
-    "@formatjs/ts-transformer": "4.3.1"
+    "@formatjs/ts-transformer": "4.3.3",
+    "@formatjs/icu-messageformat-parser": "3.5.1",
+    "@formatjs/icu-skeleton-parser": "2.1.1"
   },
   "peerDependencies": {
     "@glimmer/syntax": "^0.84.3 || ^0.95.0",

package/src/cli.js

@@ -1,5 +1,6 @@
 import { program } from "commander";
-import { sync as globSync } from "fast-glob";
+import * as glob from "fast-glob";
+const globSync = glob.sync;
 import loudRejection from "loud-rejection";
 import compile from "./compile.js";
 import compileFolder from "./compile_folder.js";

package/src/extract.js

@@ -91,6 +91,14 @@ async function processFile(source, fn, { idInterpolationPattern, ...opts }) {
 */
 export async function extract(files, extractOpts) {
 	const { throws, readFromStdin, ...opts } = extractOpts;
+	// When throws is not explicitly true, we want to collect partial results
+	const shouldThrow = throws === true;
+	// Pass throws option to transformer for per-message error handling
+	const optsWithThrows = {
+		...opts,
+		throws: shouldThrow,
+		onMsgError: !shouldThrow ? (_, e) => warn(e.message) : undefined
+	};
 	let rawResults = [];
 	try {
 		if (readFromStdin) {
@@ -100,16 +108,33 @@ export async function extract(files, extractOpts) {
 				warn("Reading source file from TTY.");
 			}
 			const stdinSource = await getStdinAsString();
-			rawResults = [await processFile(stdinSource, "dummy", opts)];
+			rawResults = [await processFile(stdinSource, "dummy", optsWithThrows)];
 		} else {
-			rawResults = await Promise.all(files.map(async (fn) => {
-				debug("Extracting file:", fn);
-				const source = await readFile(fn, "utf8");
-				return processFile(source, fn, opts);
-			}));
+			// Use Promise.allSettled when throws is not explicitly true to collect partial results
+			if (!shouldThrow) {
+				const settledResults = await Promise.allSettled(files.map(async (fn) => {
+					debug("Extracting file:", fn);
+					const source = await readFile(fn, "utf8");
+					return processFile(source, fn, optsWithThrows);
+				}));
+				rawResults = settledResults.map((result) => {
+					if (result.status === "fulfilled") {
+						return result.value;
+					} else {
+						warn(String(result.reason));
+						return undefined;
+					}
+				});
+			} else {
+				rawResults = await Promise.all(files.map(async (fn) => {
+					debug("Extracting file:", fn);
+					const source = await readFile(fn, "utf8");
+					return processFile(source, fn, optsWithThrows);
+				}));
+			}
 		}
 	} catch (e) {
-		if (throws) {
+		if (shouldThrow) {
 			throw e;
 		} else {
 			warn(String(e));