@drzl/validation-core 3.15.1 → 3.16.0

package/dist/index.cjs

@@ -513,32 +513,50 @@ function updateColumns(table) {
 function selectColumns(table) {
   return table.columns;
 }
+var reportedEngines = /* @__PURE__ */ new Set();
+var ENGINE_PACKAGE = { prettier: "prettier", biome: "@biomejs/biome" };
+function reportUnusableFormatter(engine, cause) {
+  if (reportedEngines.has(engine)) return;
+  reportedEngines.add(engine);
+  const pkg = ENGINE_PACKAGE[engine];
+  const remedy = engine === "prettier" ? 'Install prettier, which is an optional peer of @drzl/validation-core, or set format.engine to "auto" to accept whatever formatter is present.' : 'drzl formats with Biome by importing @biomejs/biome as a module; run the Biome CLI over the output instead, or set format.engine to "auto" or "prettier".';
+  const reason = cause instanceof Error ? cause.message : String(cause);
+  console.warn(
+    `[drzl] format.engine is "${engine}" but ${pkg} could not be used, so the generated files were left unformatted. ${remedy} Reason: ${reason}`
+  );
+}
 async function formatCode(code, filePath, fmt) {
   if (fmt && fmt.enabled === false) return code;
   const engine = fmt?.engine ?? "auto";
-  try {
-    if (engine === "prettier" || engine === "auto") {
+  if (engine === "prettier" || engine === "auto") {
+    try {
       const prettier = await import("prettier");
       const cfgRef = fmt?.configPath ?? filePath;
       const cfg = await prettier.resolveConfig(cfgRef).catch(() => null);
       return prettier.format(code, { ...cfg ?? {}, parser: "typescript", filepath: filePath });
+    } catch (err) {
+      if (engine === "prettier") reportUnusableFormatter("prettier", err);
     }
-  } catch {
   }
-  try {
-    if (engine === "biome" || engine === "auto") {
+  if (engine === "biome" || engine === "auto") {
+    try {
       const dynamicImport = Function("s", "return import(s)");
-      const biome = await dynamicImport("@biomejs/biome").catch(() => null);
+      const biome = await dynamicImport("@biomejs/biome");
       if (biome?.formatContent) {
         const res = await biome.formatContent(code, { filePath });
-        return (res && (res.content || res.formatted)) ?? code;
+        const out = res && (res.content || res.formatted);
+        if (typeof out === "string") return out;
+        throw new Error("@biomejs/biome formatContent returned no formatted content");
       }
       if (biome?.format) {
         const res = await biome.format(code, { filePath });
-        return res ?? code;
+        if (typeof res === "string") return res;
+        throw new Error("@biomejs/biome format returned no formatted content");
       }
+      throw new Error("@biomejs/biome exposes neither formatContent nor format");
+    } catch (err) {
+      if (engine === "biome") reportUnusableFormatter("biome", err);
     }
-  } catch {
   }
   return code;
 }

package/dist/index.d.cts

@@ -512,9 +512,21 @@ declare function selectColumns(table: Table): Column[];
  * anyone installing @drzl/cli. It is `--external` in every build script that can reach it, and
  * no-bundled-formatter.spec.ts builds those scripts and checks.
  *
- * Neither absence is an error. A consumer with no formatter gets the code as rendered, which is
- * valid TypeScript that merely looks worse, and losing generated files at the last step would be
- * a far worse trade than losing their whitespace.
+ * An absent formatter is never fatal. A consumer with no formatter gets the code as rendered,
+ * which is valid TypeScript that merely looks worse, and losing generated files at the last step
+ * would be a far worse trade than losing their whitespace.
+ *
+ * Whether it is worth saying so depends on what was asked for, which is the difference between the
+ * two branches below. `engine: 'auto'` asked for whatever happens to be installed, so finding
+ * nothing is an outcome and the code comes back unchanged in silence. Naming an engine is a
+ * request, and an unmet request that produces neither formatted output nor a message reads as
+ * "this is fine" when it is not: the consumer configured something, it did not happen, and nothing
+ * in the run says which. So a named engine that cannot be loaded warns on stderr and still returns
+ * the code, rather than throwing. Throwing would lose a whole generation over whitespace, and the
+ * consumer would not even be told what for: every generator branch in the CLI except the oRPC one
+ * wraps its `generate()` in a catch that prints "<name> generator missing. Install with: npm
+ * install @drzl/generator-<name>", so the headline would name a package they already have and the
+ * real reason would arrive as a trailing detail line. Measured, with a throw wired in on purpose.
  */
 declare function formatCode(code: string, filePath: string, fmt?: FormatOptions): Promise<any>;
 

package/dist/index.d.ts

@@ -512,9 +512,21 @@ declare function selectColumns(table: Table): Column[];
  * anyone installing @drzl/cli. It is `--external` in every build script that can reach it, and
  * no-bundled-formatter.spec.ts builds those scripts and checks.
  *
- * Neither absence is an error. A consumer with no formatter gets the code as rendered, which is
- * valid TypeScript that merely looks worse, and losing generated files at the last step would be
- * a far worse trade than losing their whitespace.
+ * An absent formatter is never fatal. A consumer with no formatter gets the code as rendered,
+ * which is valid TypeScript that merely looks worse, and losing generated files at the last step
+ * would be a far worse trade than losing their whitespace.
+ *
+ * Whether it is worth saying so depends on what was asked for, which is the difference between the
+ * two branches below. `engine: 'auto'` asked for whatever happens to be installed, so finding
+ * nothing is an outcome and the code comes back unchanged in silence. Naming an engine is a
+ * request, and an unmet request that produces neither formatted output nor a message reads as
+ * "this is fine" when it is not: the consumer configured something, it did not happen, and nothing
+ * in the run says which. So a named engine that cannot be loaded warns on stderr and still returns
+ * the code, rather than throwing. Throwing would lose a whole generation over whitespace, and the
+ * consumer would not even be told what for: every generator branch in the CLI except the oRPC one
+ * wraps its `generate()` in a catch that prints "<name> generator missing. Install with: npm
+ * install @drzl/generator-<name>", so the headline would name a package they already have and the
+ * real reason would arrive as a trailing detail line. Measured, with a throw wired in on purpose.
  */
 declare function formatCode(code: string, filePath: string, fmt?: FormatOptions): Promise<any>;
 

package/dist/index.js

@@ -450,32 +450,50 @@ function updateColumns(table) {
 function selectColumns(table) {
   return table.columns;
 }
+var reportedEngines = /* @__PURE__ */ new Set();
+var ENGINE_PACKAGE = { prettier: "prettier", biome: "@biomejs/biome" };
+function reportUnusableFormatter(engine, cause) {
+  if (reportedEngines.has(engine)) return;
+  reportedEngines.add(engine);
+  const pkg = ENGINE_PACKAGE[engine];
+  const remedy = engine === "prettier" ? 'Install prettier, which is an optional peer of @drzl/validation-core, or set format.engine to "auto" to accept whatever formatter is present.' : 'drzl formats with Biome by importing @biomejs/biome as a module; run the Biome CLI over the output instead, or set format.engine to "auto" or "prettier".';
+  const reason = cause instanceof Error ? cause.message : String(cause);
+  console.warn(
+    `[drzl] format.engine is "${engine}" but ${pkg} could not be used, so the generated files were left unformatted. ${remedy} Reason: ${reason}`
+  );
+}
 async function formatCode(code, filePath, fmt) {
   if (fmt && fmt.enabled === false) return code;
   const engine = fmt?.engine ?? "auto";
-  try {
-    if (engine === "prettier" || engine === "auto") {
+  if (engine === "prettier" || engine === "auto") {
+    try {
       const prettier = await import("prettier");
       const cfgRef = fmt?.configPath ?? filePath;
       const cfg = await prettier.resolveConfig(cfgRef).catch(() => null);
       return prettier.format(code, { ...cfg ?? {}, parser: "typescript", filepath: filePath });
+    } catch (err) {
+      if (engine === "prettier") reportUnusableFormatter("prettier", err);
     }
-  } catch {
   }
-  try {
-    if (engine === "biome" || engine === "auto") {
+  if (engine === "biome" || engine === "auto") {
+    try {
       const dynamicImport = Function("s", "return import(s)");
-      const biome = await dynamicImport("@biomejs/biome").catch(() => null);
+      const biome = await dynamicImport("@biomejs/biome");
       if (biome?.formatContent) {
         const res = await biome.formatContent(code, { filePath });
-        return (res && (res.content || res.formatted)) ?? code;
+        const out = res && (res.content || res.formatted);
+        if (typeof out === "string") return out;
+        throw new Error("@biomejs/biome formatContent returned no formatted content");
       }
       if (biome?.format) {
         const res = await biome.format(code, { filePath });
-        return res ?? code;
+        if (typeof res === "string") return res;
+        throw new Error("@biomejs/biome format returned no formatted content");
       }
+      throw new Error("@biomejs/biome exposes neither formatContent nor format");
+    } catch (err) {
+      if (engine === "biome") reportUnusableFormatter("biome", err);
     }
-  } catch {
   }
   return code;
 }

package/package.json

@@ -1,17 +1,30 @@
 {
   "name": "@drzl/validation-core",
-  "version": "3.15.1",
+  "version": "3.16.0",
   "private": false,
   "license": "Apache-2.0",
   "type": "module",
-  "main": "dist/index.js",
-  "types": "dist/index.d.ts",
+  "main": "./dist/index.cjs",
+  "module": "./dist/index.js",
+  "types": "./dist/index.d.ts",
+  "exports": {
+    ".": {
+      "import": {
+        "types": "./dist/index.d.ts",
+        "default": "./dist/index.js"
+      },
+      "require": {
+        "types": "./dist/index.d.cts",
+        "default": "./dist/index.cjs"
+      }
+    }
+  },
   "files": [
     "dist"
   ],
   "sideEffects": false,
   "dependencies": {
-    "@drzl/analyzer": "^1.15.0"
+    "@drzl/analyzer": "^1.18.0"
   },
   "peerDependencies": {
     "prettier": ">=3"