@mestreyoda/fabrica 0.2.30 → 0.2.33

package/defaults/fabrica/prompts/developer.md

@@ -44,6 +44,8 @@ The `.worktrees/` directory sits NEXT TO the repo folder (not inside it). This k
 
 Never create or implement the project under `~/.openclaw/workspace/<slug>` unless the task message explicitly says that directory is the canonical repo path. If the repo already contains scaffolded files, do not re-initialize the project with `npm init`, `uv init`, `cargo init`, or a second skeleton generator — keep the existing stack and modify the scaffold inside the assigned worktree. Once you are in the assigned worktree, stay there for the rest of the task and do not switch back to the main checkout.
 
+IMPORTANT: the shell `exec` tool is stateless and does NOT remember a previous `cd`. That means every shell command that depends on the repo location must explicitly prefix the worktree path, e.g. `cd "$WORKTREE" && <command>` or use absolute file paths. If `pwd` or tool output ever shows `~/.openclaw/workspace`, `~/`, or any directory outside the assigned worktree, stop, re-enter the worktree, and only continue with commands that include the explicit `cd "$WORKTREE" && ...` prefix.
+
 ### 2. Implement the changes
 
 - Read the issue description and comments thoroughly

package/defaults/fabrica/prompts/tester.md

@@ -56,6 +56,8 @@ fi
 
 **IMPORTANT:** Always verify you are on the correct branch before running tests. If you test on `main` and the feature code is not there, your results will be WRONG.
 
+IMPORTANT: the shell `exec` tool is stateless and does NOT remember a previous `cd`. That means every repo-scoped command must explicitly include the worktree path, e.g. `cd "$WORKTREE" && <command>` or use absolute file paths inside the assigned worktree. If `pwd` or any command output shows `~/.openclaw/workspace`, `~/`, or any directory outside the assigned worktree, stop, re-enter the worktree, and rerun the command with an explicit `cd "$WORKTREE" && ...` prefix before trusting the result.
+
 ### 2. Run QA contract (MANDATORY)
 
 ```bash

package/dist/index.js

@@ -113905,8 +113905,8 @@ import fsSync from "node:fs";
 import path5 from "node:path";
 import { fileURLToPath as fileURLToPath3 } from "node:url";
 function getCurrentVersion() {
-  if ("0.2.30") {
-    return "0.2.30";
+  if ("0.2.33") {
+    return "0.2.33";
   }
   try {
     const pkgPath = path5.join(THIS_DIR, "..", "..", "package.json");
@@ -131466,6 +131466,13 @@ function detectExecutionContractViolation(messages) {
       evidence: nestedCommand
     };
   }
+  const worktreeDrift = evidenceEntries.map((entry) => matchWorktreeDriftEvidence(entry.text)).find((match) => Boolean(match));
+  if (worktreeDrift) {
+    return {
+      reason: "worktree_drift",
+      evidence: worktreeDrift
+    };
+  }
   for (const entry of assistantEntries) {
     const metaSkillUsage = matchStrongMetaSkillUsage(entry.text);
     if (metaSkillUsage) {
@@ -131491,6 +131498,19 @@ function detectExecutionContractViolation(messages) {
   }
   return null;
 }
+function matchWorktreeDriftEvidence(text) {
+  if (!text) return null;
+  const patterns = [
+    /ENOENT: .*\/home\/ubuntu\/\.openclaw\/workspace(?:\/[^\s'"`]+)?/,
+    /^\/home\/ubuntu\/\.openclaw\/workspace$/m,
+    /\/home\/ubuntu\/\.openclaw\/workspace(?:\/[^\s'"`]+)?/
+  ];
+  for (const pattern of patterns) {
+    const match = text.match(pattern);
+    if (match?.[0]) return match[0];
+  }
+  return null;
+}
 function collectWorkerTranscriptEvidence(messages) {
   const evidence = [];
   for (const message of messages) {
@@ -131914,48 +131934,49 @@ async function handleWorkerAgentEnd(opts) {
     }).catch(() => {
     });
   }
-  if (!observation.result) {
-    if (observation.executionContractViolation) {
-      if (context2) {
-        const violationPayload = {
-          sessionKey: opts.sessionKey,
-          projectSlug: context2.projectSlug,
-          issueId: context2.issueId,
-          role,
-          reason: "invalid_execution_path",
-          violationReason: observation.executionContractViolation.reason,
-          evidence: observation.executionContractViolation.evidence
-        };
-        await updateIssueRuntime(opts.workspaceDir, context2.projectSlug, context2.issueId, {
-          inconclusiveCompletionAt: (/* @__PURE__ */ new Date()).toISOString(),
-          inconclusiveCompletionReason: "invalid_execution_path"
-        }).catch(() => {
-        });
-        await log(opts.workspaceDir, "worker_execution_contract_violation", violationPayload).catch(() => {
-        });
-        await log(opts.workspaceDir, "worker_execution_recovery_started", violationPayload).catch(() => {
-        });
-        await log(opts.workspaceDir, "worker_completion_inconclusive", {
-          ...violationPayload
-        }).catch(() => {
-        });
-      } else {
-        const violationPayload = {
-          sessionKey: opts.sessionKey,
-          role,
-          reason: "invalid_execution_path",
-          violationReason: observation.executionContractViolation.reason,
-          evidence: observation.executionContractViolation.evidence
-        };
-        await log(opts.workspaceDir, "worker_execution_contract_violation", violationPayload).catch(() => {
-        });
-        await log(opts.workspaceDir, "worker_result_skipped", {
-          ...violationPayload
-        }).catch(() => {
-        });
-      }
-      return { applied: false, reason: "invalid_execution_path" };
+  if (observation.executionContractViolation && (!observation.result || observation.executionContractViolation.reason === "worktree_drift")) {
+    if (context2) {
+      const violationPayload = {
+        sessionKey: opts.sessionKey,
+        projectSlug: context2.projectSlug,
+        issueId: context2.issueId,
+        role,
+        reason: "invalid_execution_path",
+        violationReason: observation.executionContractViolation.reason,
+        evidence: observation.executionContractViolation.evidence
+      };
+      await updateIssueRuntime(opts.workspaceDir, context2.projectSlug, context2.issueId, {
+        inconclusiveCompletionAt: (/* @__PURE__ */ new Date()).toISOString(),
+        inconclusiveCompletionReason: "invalid_execution_path"
+      }).catch(() => {
+      });
+      await log(opts.workspaceDir, "worker_execution_contract_violation", violationPayload).catch(() => {
+      });
+      await log(opts.workspaceDir, "worker_execution_recovery_started", violationPayload).catch(() => {
+      });
+      await log(opts.workspaceDir, "worker_completion_inconclusive", {
+        ...violationPayload,
+        hadResultLine: Boolean(observation.result)
+      }).catch(() => {
+      });
+    } else {
+      const violationPayload = {
+        sessionKey: opts.sessionKey,
+        role,
+        reason: "invalid_execution_path",
+        violationReason: observation.executionContractViolation.reason,
+        evidence: observation.executionContractViolation.evidence
+      };
+      await log(opts.workspaceDir, "worker_execution_contract_violation", violationPayload).catch(() => {
+      });
+      await log(opts.workspaceDir, "worker_result_skipped", {
+        ...violationPayload
+      }).catch(() => {
+      });
     }
+    return { applied: false, reason: "invalid_execution_path" };
+  }
+  if (!observation.result) {
     if (context2 && observation.activityObserved) {
       await updateIssueRuntime(opts.workspaceDir, context2.projectSlug, context2.issueId, {
         inconclusiveCompletionAt: (/* @__PURE__ */ new Date()).toISOString(),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mestreyoda/fabrica",
-  "version": "0.2.30",
+  "version": "0.2.33",
   "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",