@effijs/forge 1.0.0-alpha.1 → 1.0.0-alpha.3

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@effijs/forge",
-  "version": "1.0.0-alpha.1",
+  "version": "1.0.0-alpha.3",
   "type": "module",
   "description": "Modern scaffolding engine for EffiJS",
   "author": "Yevhenii Kyrychok",

package/src/Forge.js

@@ -1,4 +1,5 @@
 import path from "path";
+import "./shims/repairInquirer.js";
 import inquirer from "inquirer";
 import chalk from "chalk";
 import { compileHandlebars, registerHandlebarsHelpers } from "./engines/handlebars.js";
@@ -13,6 +14,15 @@ import { addFile } from "./fs/addFile.js";
 import { modifyFile, appendToFile } from "./fs/modifyFile.js";
 import { processTemplate, processDirectory } from "./fs/processTemplate.js";
 
+function collectActionErrors(result, errors = []) {
+  if (!result || typeof result !== "object") return errors;
+  if (result.success === false && result.error) errors.push(String(result.error));
+  if (Array.isArray(result.results)) {
+    for (const child of result.results) collectActionErrors(child, errors);
+  }
+  return errors;
+}
+
 async function applyPromptDefaults(questions, options) {
   const answers = {};
   const list = Array.isArray(questions) ? questions : [];
@@ -145,9 +155,15 @@ export class Forge {
           typeof generator.prompts === "function"
             ? await generator.prompts(this)
             : generator.prompts;
-        answers = yes
+        const useDefaults = yes || dryRun;
+        answers = useDefaults
           ? await applyPromptDefaults(questions, options)
           : await this.prompt(questions);
+        if (dryRun && !yes && Array.isArray(questions) && questions.length) {
+          console.log(
+            chalk.dim("Dry run is using prompt defaults. Pass flags to override.\n"),
+          );
+        }
       }
 
       const data = { ...answers, ...options };
@@ -166,9 +182,16 @@ export class Forge {
       const { results, hasFailure } = await runActions(generator, ctx);
 
       if (hasFailure) {
+        const errors = [
+          ...new Set(results.flatMap((result) => collectActionErrors(result))),
+        ];
         console.error(
           chalk.red("\n❌ Generation failed — one or more actions failed\n"),
         );
+        for (const error of errors) {
+          console.error(chalk.red(`• ${error}`));
+        }
+        if (errors.length) console.error("");
         return { success: false, results };
       }
 

package/src/cli/ForgeCLI.js

@@ -1,4 +1,5 @@
 import { Command } from "commander";
+import "../shims/repairInquirer.js";
 import inquirer from "inquirer";
 import fs from "fs/promises";
 import path from "path";
@@ -44,7 +45,8 @@ export class ForgeCLI {
 
         Object.assign(runOptions, parseUnknownFlags(this.argv ?? process.argv));
 
-        await this.forge.run(generator, runOptions);
+        const result = await this.forge.run(generator, runOptions);
+        if (result?.success === false) process.exitCode = 1;
       });
 
     this.program

package/src/shims/repairInquirer.js

@@ -0,0 +1,18 @@
+import Module from "node:module";
+import path from "node:path";
+import { fileURLToPath } from "node:url";
+
+const shim = fileURLToPath(new URL("./strip-ansi.cjs", import.meta.url));
+const inquirerDir = `${path.sep}inquirer${path.sep}`;
+const original = Module._resolveFilename;
+
+if (!original.__forgeStripAnsi) {
+  function resolveFilename(request, parent, ...rest) {
+    if (request === "strip-ansi" && parent?.filename?.includes(inquirerDir)) {
+      return shim;
+    }
+    return original.call(this, request, parent, ...rest);
+  }
+  resolveFilename.__forgeStripAnsi = true;
+  Module._resolveFilename = resolveFilename;
+}

package/src/shims/strip-ansi.cjs

@@ -0,0 +1,16 @@
+"use strict";
+
+// inquirer 9's nested ora@5 does require("strip-ansi") and expects the v6 function.
+// A hoisted strip-ansi v7 is ESM, so Node's require() returns a module namespace.
+function ansiRegex() {
+  const pattern = [
+    "[\\u001B\\u009B][[\\]()#;?]*(?:(?:(?:(?:;[-a-zA-Z\\d\\/#&.:=?%@~_]+)*|[a-zA-Z\\d]+(?:;[-a-zA-Z\\d\\/#&.:=?%@~_]*)*)?\\u0007)",
+    "(?:(?:\\d{1,4}(?:;\\d{0,4})*)?[\\dA-PR-TZcf-ntqry=><~]))",
+  ].join("|");
+  return new RegExp(pattern, "g");
+}
+
+module.exports = function stripAnsi(string) {
+  if (typeof string !== "string") return string;
+  return string.replace(ansiRegex(), "");
+};