@jwilger/pi-development-system 0.25.0 → 0.26.0

package/package.json

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

package/src/context/status.ts

@@ -14,7 +14,9 @@ const ciLabel = (state: DevsysState): string | undefined => {
 };
 
 const activeReview = (state: DevsysState): string | undefined => {
-  const review = state.activeSlice === undefined ? undefined : reviewOf(state, state.activeSlice);
+  // Without an active slice (before /devsys-start exists) show the most recent review instead of nothing.
+  const review =
+    state.activeSlice === undefined ? state.reviews?.at(-1) : reviewOf(state, state.activeSlice);
   return review === undefined ? undefined : reviewLabel(review);
 };
 

package/src/review/digest.ts

@@ -42,6 +42,10 @@ export async function snapshotDiff(
       // diff.mnemonicPrefix / diff.noprefix change the `a/` `b/` header prefixes the file keys are cut from.
       "--src-prefix=a/",
       "--dst-prefix=b/",
+      // Every changed path gets a `diff --git` section: no submodule log summaries, no textconv that
+      // can hide an edit whose converted text is unchanged.
+      "--submodule=short",
+      "--no-textconv",
     ];
     const [diff, stat, others] = await Promise.all([
       exec("git", [...plain, "--full-index", "--no-renames", range], { cwd, timeout: 15_000 }),

package/src/review/review-tools.ts

@@ -127,6 +127,14 @@ export function createReviewStartTool(
         );
       }
 
+      if (action === "fix-findings") {
+        return reply(
+          `${reviewLabel(review)}. Round ${review.rounds.length} of ${slice} has blocking or should-fix findings and the diff has not changed since: fix them first, then start again. ` +
+            "Re-running the review on unchanged code does not clear a finding. If a finding is wrong, record a `review.unsatisfied` departure with the evidence (devsys_record_departure).",
+          true,
+        );
+      }
+
       const round = review.rounds.length + 1;
       const resolved = resolveSlot(
         config.value.models,