@yagni-app/code-staging 1.2.7-staging.2059.1 → 1.2.7-staging.2062.1

package/dist/extension/pipeline/headlessGo.js

@@ -557,6 +557,15 @@ export async function runHeadlessGo(argv, deps = {}) {
     const baseSha = startTree?.head ?? "";
     const preRunDirt = startTree ? parseChangedPaths(startTree.status) : [];
     const onBase = baseSha ? { base: baseSha } : {};
+    // Its reviewer reads the change from that commit too (YAG-1041). A
+    // continuation starts past the attempt's earlier commits, so its start is
+    // not where the change begins: its reviewer keeps the bare `git diff`.
+    const reviewLane = lane && baseSha && !continuation
+        ? {
+            ...lane,
+            stages: lane.stages.map((s) => (s.id === "review" ? { ...s, diffBase: baseSha } : s)),
+        }
+        : lane;
     let gate;
     let lastVerify;
     // A cloud session's gate also runs the host's secret and binary scans on
@@ -606,7 +615,7 @@ export async function runHeadlessGo(argv, deps = {}) {
         ...(budget ? { budget } : {}),
         ...(deps.stages ? { stages: deps.stages } : {}),
         grounded: deps.grounded ?? grounding,
-        ...(lane ? { lane } : {}),
+        ...(reviewLane ? { lane: reviewLane } : {}),
         ...(mission ? { mission } : {}),
         ...(runVerify ? { runVerify } : {}),
         ...(baseline ? { runWorkstreamCheck: makeWorkstreamCheck({ baseline }) } : {}),

package/dist/extension/pipeline/personas.d.ts

@@ -50,10 +50,11 @@ export declare function builderPersonaClause(workstream?: PartitionWorkstream, s
  * Resolve the system-prompt body for a stage. For non-review stages this is the
  * role body; for the review stage it appends the lens-specific clause (or, with
  * no lens, the one-reviewer clause covering all three) and the required
- * findings output contract so the reviewer's output parses. A stage carrying a
- * `personaClause` (the fan-out builders) gets it appended last, and one
- * carrying a `persona` (the cloud plan stage's host persona) runs with that in
- * place of its agent's body.
+ * findings output contract so the reviewer's output parses. A review carrying
+ * a `diffBase` (a cloud session's) diffs from that commit in its first step. A
+ * stage carrying a `personaClause` (the fan-out builders) gets it appended
+ * last, and one carrying a `persona` (the cloud plan stage's host persona)
+ * runs with that in place of its agent's body.
  *
  * `grounded` defaults true (every real /go stage). The M6 eval passes
  * `grounded: false` to select the grounding-free bodies + lens clauses for the

package/dist/extension/pipeline/personas.js

@@ -86,12 +86,22 @@ What was done.
 Any record_decision calls you made, and why.
 ## Notes
 Anything the reviewer should know (exact file paths, key functions touched).`;
+/** The reviewer's first step, which a stage's `diffBase` rewrites (see `personaBody`). */
+const REVIEWER_DIFF_STEP = "1. `git diff` to see the change.";
+/**
+ * The reviewer's first step on a change read from `base`, the commit a cloud
+ * session started on: its model may have committed its work, which a bare
+ * `git diff` no longer shows (YAG-1041).
+ */
+function reviewerDiffStepFrom(base) {
+    return `1. \`git diff ${base}\` to see the change: everything this session changed since its base commit ${base}, committed or not (a bare \`git diff\` shows only what is not committed yet). \`git log ${base}..HEAD\` lists its commits, and \`git status\` lists new files git does not track yet.`;
+}
 const REVIEWER_BODY = `You are a senior, adversarial code reviewer. Analyze the change for real problems, not style nits. Bash is read-only here (\`git diff\`, \`git log\`, \`git show\`); do NOT modify files or run builds.
 
 You are grounded in how THIS company works — that context is your edge over a repo-only reviewer.
 
 Strategy:
-1. \`git diff\` to see the change.
+${REVIEWER_DIFF_STEP}
 2. Read the modified files and their surroundings.
 3. Judge the change through your assigned lens (below).`;
 const ADVISOR_BODY = `You are a senior advisor, brought in on ONE hard call by an agent that is already working the problem. You are the strongest model in this system and you are expensive, so you are consulted, not driven — the caller does the work, you supply the judgment.
@@ -300,7 +310,7 @@ Anything the reviewer should know (exact file paths, key functions touched).`;
 const REVIEWER_BLIND = `You are a senior, adversarial code reviewer. Analyze the change for real problems, not style nits. Bash is read-only here (\`git diff\`, \`git log\`, \`git show\`); do NOT modify files or run builds.
 
 Strategy:
-1. \`git diff\` to see the change.
+${REVIEWER_DIFF_STEP}
 2. Read the modified files and their surroundings.
 3. Judge the change through your assigned lens (below).`;
 const ADVISOR_BLIND = `You are a senior advisor, brought in on ONE hard call by an agent that is already working the problem. You are the strongest model in this system and you are expensive, so you are consulted, not driven — the caller does the work, you supply the judgment.
@@ -428,10 +438,11 @@ export function builderPersonaClause(workstream, siblings = []) {
  * Resolve the system-prompt body for a stage. For non-review stages this is the
  * role body; for the review stage it appends the lens-specific clause (or, with
  * no lens, the one-reviewer clause covering all three) and the required
- * findings output contract so the reviewer's output parses. A stage carrying a
- * `personaClause` (the fan-out builders) gets it appended last, and one
- * carrying a `persona` (the cloud plan stage's host persona) runs with that in
- * place of its agent's body.
+ * findings output contract so the reviewer's output parses. A review carrying
+ * a `diffBase` (a cloud session's) diffs from that commit in its first step. A
+ * stage carrying a `personaClause` (the fan-out builders) gets it appended
+ * last, and one carrying a `persona` (the cloud plan stage's host persona)
+ * runs with that in place of its agent's body.
  *
  * `grounded` defaults true (every real /go stage). The M6 eval passes
  * `grounded: false` to select the grounding-free bodies + lens clauses for the
@@ -443,7 +454,10 @@ export function personaBody(stage, opts = {}) {
     const base = stage.persona?.trim() || bodies[stage.agent];
     if (!base)
         throw new Error(`No persona body for agent "${stage.agent}"`);
-    const parts = [base];
+    const diffBase = stage.diffBase;
+    const parts = [
+        diffBase ? base.replace(REVIEWER_DIFF_STEP, () => reviewerDiffStepFrom(diffBase)) : base,
+    ];
     if (stage.id === "review") {
         const clauses = grounded ? LENS_CLAUSES : BLIND_LENS_CLAUSES;
         if (stage.lens)

package/dist/extension/pipeline/types.d.ts

@@ -72,6 +72,12 @@ export interface PipelineStage {
      * plan stage takes the host's `plan-persona.md` (Reeve's persona) here.
      */
     persona?: string;
+    /**
+     * The commit a review reads the change from (a cloud session's start,
+     * YAG-1041): its first step is `git diff <diffBase>`, since a model that
+     * commits its work leaves a bare `git diff` empty. Absent: `git diff`.
+     */
+    diffBase?: string;
     /**
      * The stage's own clock, in milliseconds (`stageClock.ts`). Past it the
      * child refuses every tool call, so the model writes its answer from what it

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yagni-app/code-staging",
-  "version": "1.2.7-staging.2059.1",
+  "version": "1.2.7-staging.2062.1",
   "description": "YAGNI Code: a terminal coding agent that already knows your company. One YAGNI login routes the model and grounds the agent in your team's context.",
   "license": "SEE LICENSE IN LICENSE.md",
   "author": "YAGNI, Inc. <jack@yagni.app> (https://yagni.app)",
@@ -58,5 +58,5 @@
     "turndown": "^7.2.4",
     "typebox": "^1.3.15"
   },
-  "yagniSourceSha": "6e9cbb06e7ae4d5b15b14fecc749fb5fbab2e93b"
+  "yagniSourceSha": "a52f06325dd8542ceb7055b6fc9a7c965c8c77bd"
 }