@mestreyoda/fabrica 0.2.12 → 0.2.13

package/CHANGELOG.md

@@ -1,5 +1,11 @@
 # Changelog
 
+## 0.2.13 - 2026-03-31
+
+- Disabled automatic pretty logging on TTY so the plugin no longer depends on `pino-pretty` during load.
+- Added a safe one-shot fallback to structured logs when pretty transport resolution or initialization fails.
+- Added logger transport regression coverage and promoted it into the hot-path test lane.
+
 ## 0.2.12 - 2026-03-31
 
 - Made the published plugin self-contained by replacing remaining runtime helper imports from `openclaw/plugin-sdk`.

package/dist/index.js

@@ -94160,7 +94160,7 @@ var require_thread_stream = __commonJS({
 var require_transport2 = __commonJS({
   "node_modules/pino/lib/transport.js"(exports2, module2) {
     "use strict";
-    var { createRequire: createRequire4 } = __require("module");
+    var { createRequire: createRequire5 } = __require("module");
     var { existsSync: existsSync2 } = __require("node:fs");
     var getCallers = require_caller();
     var { join: join7, isAbsolute, sep } = __require("node:path");
@@ -94363,7 +94363,7 @@ var require_transport2 = __commonJS({
         for (const filePath of callers) {
           try {
             const context2 = filePath === "node:repl" ? process.cwd() + sep : filePath;
-            fixTarget2 = createRequire4(context2).resolve(origin);
+            fixTarget2 = createRequire5(context2).resolve(origin);
             break;
           } catch (err) {
             continue;
@@ -96136,17 +96136,35 @@ var require_pino = __commonJS({
 });
 
 // lib/observability/logger.ts
+import { createRequire } from "node:module";
+function warnPrettyFallback(error48) {
+  if (prettyFallbackWarned) return;
+  prettyFallbackWarned = true;
+  const reason = error48 instanceof Error ? error48.message : String(error48);
+  console.warn(`[fabrica] pretty logging unavailable, falling back to structured logs: ${reason}`);
+}
 function createTransport() {
-  const pretty = process.env.LOG_PRETTY === "1" || process.stdout.isTTY && process.env.NODE_ENV !== "production";
-  if (!pretty) return void 0;
-  return import_pino.default.transport({
-    target: "pino-pretty",
-    options: {
-      colorize: true,
-      translateTime: "SYS:standard",
-      ignore: "pid,hostname"
-    }
-  });
+  if (process.env.LOG_PRETTY !== "1") return void 0;
+  let target;
+  try {
+    target = require2.resolve("pino-pretty");
+  } catch (error48) {
+    warnPrettyFallback(error48);
+    return void 0;
+  }
+  try {
+    return import_pino.default.transport({
+      target,
+      options: {
+        colorize: true,
+        translateTime: "SYS:standard",
+        ignore: "pid,hostname"
+      }
+    });
+  } catch (error48) {
+    warnPrettyFallback(error48);
+    return void 0;
+  }
 }
 function createFabricaLogger(destination) {
   return (0, import_pino.default)({
@@ -96205,13 +96223,15 @@ function getRootLogger() {
 function getLogger(bindings) {
   return bindings ? rootLogger.child(bindings) : rootLogger;
 }
-var import_pino, rootLogger;
+var import_pino, require2, prettyFallbackWarned, rootLogger;
 var init_logger = __esm({
   "lib/observability/logger.ts"() {
     "use strict";
     import_pino = __toESM(require_pino(), 1);
     init_esm();
     init_context3();
+    require2 = createRequire(import.meta.url);
+    prettyFallbackWarned = false;
     rootLogger = createFabricaLogger();
   }
 });
@@ -111327,8 +111347,8 @@ import fsSync from "node:fs";
 import path5 from "node:path";
 import { fileURLToPath as fileURLToPath3 } from "node:url";
 function getCurrentVersion() {
-  if ("0.2.12") {
-    return "0.2.12";
+  if ("0.2.13") {
+    return "0.2.13";
   }
   try {
     const pkgPath = path5.join(THIS_DIR, "..", "..", "package.json");
@@ -115962,7 +115982,7 @@ var classify_exports = {};
 __export(classify_exports, {
   classifyStep: () => classifyStep
 });
-import { createRequire } from "node:module";
+import { createRequire as createRequire2 } from "node:module";
 function loadRules() {
   if (cachedRules) return cachedRules;
   cachedRules = _require("../configs/classification-rules.json");
@@ -115977,7 +115997,7 @@ var init_classify = __esm({
     init_runtime_paths();
     init_llm_retry();
     init_zod();
-    _require = createRequire(import.meta.url);
+    _require = createRequire2(import.meta.url);
     LlmResponseSchema = external_exports.object({
       payloads: external_exports.array(external_exports.object({ text: external_exports.string() })).min(1)
     }).passthrough();
@@ -116063,7 +116083,7 @@ var interview_exports = {};
 __export(interview_exports, {
   interviewStep: () => interviewStep
 });
-import { createRequire as createRequire2 } from "node:module";
+import { createRequire as createRequire3 } from "node:module";
 function loadTemplates() {
   if (cachedTemplates) return cachedTemplates;
   cachedTemplates = _require2("../configs/interview-templates.json");
@@ -116073,7 +116093,7 @@ var _require2, cachedTemplates, interviewStep;
 var init_interview = __esm({
   "lib/intake/steps/interview.ts"() {
     "use strict";
-    _require2 = createRequire2(import.meta.url);
+    _require2 = createRequire3(import.meta.url);
     cachedTemplates = null;
     interviewStep = {
       name: "interview",
@@ -140221,8 +140241,8 @@ function validateSpecQuality(input) {
 }
 
 // lib/intake/steps/triage.ts
-import { createRequire as createRequire3 } from "node:module";
-var _require3 = createRequire3(import.meta.url);
+import { createRequire as createRequire4 } from "node:module";
+var _require3 = createRequire4(import.meta.url);
 var cachedMatrix = null;
 function loadMatrix() {
   if (cachedMatrix) return cachedMatrix;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mestreyoda/fabrica",
-  "version": "0.2.12",
+  "version": "0.2.13",
   "description": "Autonomous software engineering pipeline for OpenClaw. Turns ideas into deployed code via intake, dispatch, review, test, and merge.",
   "type": "module",
   "main": "./dist/index.js",
@@ -43,7 +43,7 @@
     "test": "npm run test:hot-path",
     "test:unit": "vitest run",
     "test:e2e": "VITEST_E2E=1 vitest run lib/services/bootstrap.e2e.test.ts lib/services/pipeline.e2e.test.ts tests/e2e/qa-bootstrap.e2e.test.ts tests/e2e/orchestration-smoke.e2e.test.ts",
-    "test:hot-path": "VITEST_E2E=1 vitest run --maxWorkers=1 tests/unit/message-builder.test.ts tests/unit/worker-context-hook.test.ts tests/unit/reviewer-session.test.ts tests/unit/reviewer-completion.test.ts tests/unit/reviewer-completion-side-effects.test.ts tests/unit/reactive-dispatch-hook.test.ts tests/unit/subagent-lifecycle-hook.test.ts tests/unit/subagent-lifecycle-slot-cleanup.test.ts tests/unit/dispatch-identity.test.ts tests/unit/work-finish-slot.test.ts tests/unit/heartbeat-health-session.test.ts tests/unit/reviewer-poll-pass.test.ts tests/unit/heartbeat-canonical-pr.test.ts tests/e2e/orchestration-smoke.e2e.test.ts",
+    "test:hot-path": "VITEST_E2E=1 vitest run --maxWorkers=1 tests/unit/message-builder.test.ts tests/unit/worker-context-hook.test.ts tests/unit/reviewer-session.test.ts tests/unit/reviewer-completion.test.ts tests/unit/reviewer-completion-side-effects.test.ts tests/unit/reactive-dispatch-hook.test.ts tests/unit/subagent-lifecycle-hook.test.ts tests/unit/subagent-lifecycle-slot-cleanup.test.ts tests/unit/dispatch-identity.test.ts tests/unit/work-finish-slot.test.ts tests/unit/heartbeat-health-session.test.ts tests/unit/reviewer-poll-pass.test.ts tests/unit/heartbeat-canonical-pr.test.ts tests/unit/logger-transport.test.ts tests/e2e/orchestration-smoke.e2e.test.ts",
     "test:all": "npm run test:unit && npm run test:e2e",
     "test:watch": "vitest",
     "prepublishOnly": "npm run build && npm run verify:runtime-boundary && npm run test:all && npm run verify:installability"