@jwilger/pi-development-system 0.8.10 → 0.9.0

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@jwilger/pi-development-system",
-  "version": "0.8.10",
+  "version": "0.9.0",
   "description": "A pi extension package representing a seasoned approach to software development using a full AI SDLC.",
   "keywords": [
     "pi-package"

package/skills/delivery-discipline/SKILL.md

@@ -52,7 +52,7 @@ hides every later failure. A push to a red trunk is a hard stop unless the
 commit is `fix(ci): …` (or another `fix`), touches only what the failing log names, and
 stays small (an unpushed diff over 8000 characters is never exempt); Jev must read it as
 only repairing the failing run, which needs the failing log and Jev online. Otherwise the
-push is a hard stop. If `gh`
+push is a hard stop. A call that commits and pushes together on a red trunk is refused outright (commit in one call, push in the next). If `gh`
 is missing or unauthenticated, CI reads as unknown and the push is allowed:
 check CI by other means before relying on it.
 

package/src/state/ci-command.ts

@@ -53,10 +53,14 @@ export async function runCiWatch(
       deps.pi.appendEntry(CI_ENTRY_TYPE, observed);
     },
   });
-  ctx.ui.notify(
-    `CI on ${config.value.delivery.trunk}: ${final.status}${final.sha === undefined ? "" : ` (${final.sha.slice(0, 7)})`}`,
-    final.status === "red" ? "error" : "info",
-  );
+  try {
+    ctx.ui.notify(
+      `CI on ${config.value.delivery.trunk}: ${final.status}${final.sha === undefined ? "" : ` (${final.sha.slice(0, 7)})`}`,
+      final.status === "red" ? "error" : "info",
+    );
+  } catch {
+    // stale context after a reload: the status line and entries were already updated
+  }
   return final;
 }
 
@@ -67,10 +71,15 @@ export function registerCiCommand(deps: CiCommandDeps): void {
     handler: async (_args, ctx) => {
       ctx.ui.notify("watching CI on the trunk…", "info");
       runCiWatch(deps, ctx).catch((cause: unknown) => {
-        ctx.ui.notify(
-          `CI watch failed: ${cause instanceof Error ? cause.message : String(cause)}`,
-          "error",
-        );
+        // ctx goes stale if the session is replaced or reloaded mid-watch; there is nobody left to tell.
+        try {
+          ctx.ui.notify(
+            `CI watch failed: ${cause instanceof Error ? cause.message : String(cause)}`,
+            "error",
+          );
+        } catch {
+          // stale context
+        }
       });
     },
   });