@nathapp/nax 0.75.2 → 0.75.4

package/flows/nax-finish/steps/result.ts

@@ -1,3 +1,5 @@
+import { mkdir, writeFile } from "node:fs/promises";
+import { dirname } from "node:path";
 import type { FinishResult } from "../types";
 
 export function resultPath(repoRoot: string): string {
@@ -5,8 +7,14 @@ export function resultPath(repoRoot: string): string {
 }
 
 export const _resultDeps: { writeText: (p: string, s: string) => Promise<void> } = {
+  // node:fs, not Bun.write — this module runs inside acpx's Node process, where
+  // the `Bun` global does not exist (see the header of `../exec.ts`). The mkdir
+  // is not redundant: Bun.write creates missing parent directories implicitly,
+  // writeFile does not, and this is the one artifact the plugin needs on disk to
+  // report an outcome at all.
   writeText: async (p, s) => {
-    await Bun.write(p, s);
+    await mkdir(dirname(p), { recursive: true });
+    await writeFile(p, s, "utf8");
   },
 };
 

package/flows/nax-finish/types.ts

@@ -48,6 +48,20 @@ export interface FinishResult {
   status: "opened" | "promoted" | "already-ready" | "escalated" | "nothing-to-finish";
   url?: string;
   escalationReason?: string;
+  /**
+   * The findings behind an escalation. Persisted because the reason alone is a
+   * bare count ("3 finding(s) after 3 fix attempts"), and on the Telegram
+   * channel the composed PR comment — the only other thing carrying them — is
+   * never posted. Without this the findings survived only in acpx's run bundle.
+   */
+  findings?: Finding[];
+  /**
+   * Set when the escalation could not be delivered to its channel (forge
+   * comment failed, remote unrecognised). The result file is written before
+   * delivery is attempted, so an undelivered escalation is still reported
+   * rather than lost.
+   */
+  deliveryError?: string;
 }
 export interface RunResult {
   exitCode: number;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nathapp/nax",
-  "version": "0.75.2",
+  "version": "0.75.4",
   "description": "AI Coding Agent Orchestrator — loops until done",
   "type": "module",
   "bin": {
@@ -11,9 +11,10 @@
     "dev": "bun run bin/nax.ts",
     "build": "bun build bin/nax.ts --outdir dist --target bun --define \"GIT_COMMIT=\\\"$(git rev-parse --short HEAD)\\\"\"",
     "typecheck": "bun x tsc --noEmit && bun x tsc --noEmit -p tsconfig.contracts.json",
-    "lint": "bun x biome check src/ bin/ flows/ && bun run check:no-real-global-nax && bun run check:alias-internals && bun run check:deep-relatives && bun run check:nax-error && bun run check:logger-storyid && bun run check:log-format-layering && bun run check:file-sizes",
+    "lint": "bun x biome check src/ bin/ flows/ && bun run check:flows-no-bun && bun run check:no-real-global-nax && bun run check:alias-internals && bun run check:deep-relatives && bun run check:nax-error && bun run check:logger-storyid && bun run check:log-format-layering && bun run check:file-sizes",
     "lint:json": "bun x biome check src/ bin/ flows/ --reporter json && bun run check:nax-error 1>&2 && bun run check:logger-storyid 1>&2",
     "lint:fix": "bun x biome check --write src/ bin/ flows/",
+    "check:flows-no-bun": "bun run scripts/check-flows-no-bun.ts",
     "check:no-real-global-nax": "bun run scripts/check-no-real-global-nax.ts",
     "check:alias-internals": "bun run scripts/check-alias-internals.ts",
     "check:deep-relatives": "bun run scripts/check-deep-relatives.ts",