@mjasnikovs/pi-task 0.38.0 → 0.38.1

package/dist/task/accept-debt.d.ts

@@ -102,10 +102,17 @@ export declare function readAcceptDebts(cwd: string): Promise<AcceptDebt[]>;
  * as the 'accepted' class — that one asserts a human weighed the failing artifact.
  *
  * Best-effort by construction (appendDebt swallows its own faults): the ledger is an
- * auditing aid and must never break the gate sequence that calls it. `origin`
- * defaults to 'accepted', which is the legacy 2-field on-disk shape.
+ * auditing aid and must never break the gate sequence that calls it.
+ *
+ * `origin` is REQUIRED and deliberately has no default. It used to default to
+ * 'accepted' (the legacy 2-field on-disk shape), and that default silently absorbed
+ * a dropped argument: the eight-recorder collapse migrated one call in
+ * scripts/ignored-writes-ab.ts without its origin, so a run-level 'final-gate'
+ * demotion was stamped as a human 'accepted'. The wrappers each carried their class
+ * in the NAME, so no migration of them could lose it; a defaulted parameter can.
+ * Making it explicit turns that whole class of slip into a compile error.
  */
-export declare function recordDebt(cwd: string, taskId: string, reason: string, origin?: DebtOrigin): Promise<void>;
+export declare function recordDebt(cwd: string, taskId: string, reason: string, origin: DebtOrigin): Promise<void>;
 /**
  * The reason text a CROSS-TASK DELETION debt stores (mx5 run 12 PROMPT 2): the task's
  * work deleted a file a DIFFERENT task's commit introduced, verify FAILed, and the

package/dist/task/accept-debt.js

@@ -179,10 +179,17 @@ async function appendDebt(cwd, entry) {
  * as the 'accepted' class — that one asserts a human weighed the failing artifact.
  *
  * Best-effort by construction (appendDebt swallows its own faults): the ledger is an
- * auditing aid and must never break the gate sequence that calls it. `origin`
- * defaults to 'accepted', which is the legacy 2-field on-disk shape.
+ * auditing aid and must never break the gate sequence that calls it.
+ *
+ * `origin` is REQUIRED and deliberately has no default. It used to default to
+ * 'accepted' (the legacy 2-field on-disk shape), and that default silently absorbed
+ * a dropped argument: the eight-recorder collapse migrated one call in
+ * scripts/ignored-writes-ab.ts without its origin, so a run-level 'final-gate'
+ * demotion was stamped as a human 'accepted'. The wrappers each carried their class
+ * in the NAME, so no migration of them could lose it; a defaulted parameter can.
+ * Making it explicit turns that whole class of slip into a compile error.
  */
-export async function recordDebt(cwd, taskId, reason, origin = 'accepted') {
+export async function recordDebt(cwd, taskId, reason, origin) {
     await appendDebt(cwd, { taskId: taskId.trim(), reason: normaliseReason(reason), origin });
 }
 /**

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@mjasnikovs/pi-task",
-    "version": "0.38.0",
+    "version": "0.38.1",
     "description": "Deterministic task planning and spec-orchestration for local models — crash-safe /task pipelines with verify/enforce gates, a real-time remote web view, and web/docs/fetch/worker subagent tools.",
     "type": "module",
     "main": "./dist/index.js",