Review tab parallel review organization (#322)

* Review tab parallel review organization

* Fix ADE review findings

* Fix second ADE review findings

* Fix latest ADE review findings

* Fix latest ADE review findings

* Tighten review prompts and PR cache refresh

* Fix latest ADE review findings

* Fix latest ADE review findings

* Allow partial review completion

* Fix ADE review findings

* Fix follow-up ADE review findings

* Surface partial review coverage

* Fix latest ADE review findings

* Keep PR AI summaries local only

* Fix review follow-up edge cases

* Fix ADE review self-findings

* Fix final ADE review findings

* Add review finding copy actions

* ship: address review feedback and Windows package size

* ship: tighten Windows runtime package pruning

* ship: avoid readonly CRR metadata deletes

* ship: keep stash pop success after drop warning

* ship: bound review manifest prompts

Author: arul28

.ade/ade.yaml

@@ -1,11 +1,12 @@
 version: 1
 processes:
-  - id: xsi8oj88
-    name: dogfood onboardinign fixes
+  - id: gq3sy4rj
+    name: npm run dev:desktop
     command:
-      - ./scripts/dogfood.sh
-      - onboarding-fixes
-    cwd: apps/desktop
+      - npm
+      - run
+      - dev:desktop
+    cwd: .
     gracefulShutdownMs: 7000
     readiness:
       type: none

.claude/skills/plan/SKILL.md

@@ -293,7 +293,7 @@ function createRuntime() {
       listBranches: vi.fn(async () => [{ name: "main", current: true, ahead: 0, behind: 0, hasUpstream: true, upstream: "origin/main" }]),
       checkoutBranch: vi.fn(async () => ({ success: true })),
       stashPush: vi.fn(async () => ({ success: true })),
-      listStashes: vi.fn(async () => [{ ref: "stash@{0}", createdAt: "2026-04-06T00:00:00.000Z", subject: "test stash" }]),
+      listStashes: vi.fn(async () => [{ oid: "oid-0", ref: "stash@{0}", createdAt: "2026-04-06T00:00:00.000Z", subject: "test stash" }]),
       stashApply: vi.fn(async () => ({ success: true })),
       stashPop: vi.fn(async () => ({ success: true })),
       stashDrop: vi.fn(async () => ({ success: true })),
@@ -4515,6 +4515,51 @@ describe("adeRpcServer", () => {
 
   });
 
+  it("invokes review.startRun through ADE actions without dropping unlimited budgets", async () => {
+    const fixture = createRuntime();
+    const startArgs = {
+      target: { mode: "lane_diff", laneId: "lane-1" },
+      config: {
+        compareAgainst: { kind: "default_branch" },
+        selectionMode: "full_diff",
+        dirtyOnly: false,
+        modelId: "openai/gpt-5.4",
+        reasoningEffort: "medium",
+        budgets: {
+          unlimited: true,
+          maxFiles: Number.MAX_SAFE_INTEGER,
+          maxDiffChars: Number.MAX_SAFE_INTEGER,
+          maxPromptChars: Number.MAX_SAFE_INTEGER,
+          maxFindings: Number.MAX_SAFE_INTEGER,
+          maxFindingsPerPass: Number.MAX_SAFE_INTEGER,
+          maxPublishedFindings: Number.MAX_SAFE_INTEGER,
+        },
+        publishBehavior: "local_only",
+      },
+    };
+    const startRun = vi.fn(async (args: typeof startArgs) => ({
+      id: "review-run-1",
+      laneId: args.target.laneId,
+      config: args.config,
+      status: "queued",
+    }));
+    (fixture.runtime as any).reviewService = { startRun };
+    const handler = createAdeRpcRequestHandler({ runtime: fixture.runtime, serverVersion: "test" });
+    await initialize(handler, { callerId: "agent-1", role: "agent" });
+
+    const response = await callTool(handler, "run_ade_action", {
+      domain: "review",
+      action: "startRun",
+      args: startArgs,
+    });
+
+    expect(response?.isError).toBeUndefined();
+    expect(startRun).toHaveBeenCalledWith(startArgs);
+    expect(startRun.mock.calls[0][0].config.budgets).toEqual(startArgs.config.budgets);
+    expect(response.structuredContent.result.config.budgets).toEqual(startArgs.config.budgets);
+    expect(response.structuredContent.result.config.budgets.unlimited).toBe(true);
+  });
+
   it("binds service method context when invoking dynamic ADE actions", async () => {
     const fixture = createRuntime();
     const missionService = fixture.runtime.missionService as any;
@@ -4689,6 +4734,37 @@ describe("adeRpcServer", () => {
     expect(response.structuredContent.count).toBe(1);
   });
 
+  it("passes stash oid through destructive stash tools", async () => {
+    const fixture = createRuntime();
+    const handler = createAdeRpcRequestHandler({ runtime: fixture.runtime, serverVersion: "test" });
+
+    await initialize(handler, { callerId: "agent-1", role: "agent" });
+
+    const pop = await callTool(handler, "stash_pop", {
+      laneId: "lane-1",
+      stashRef: "stash@{0}",
+      stashOid: "oid-0",
+    });
+    const drop = await callTool(handler, "stash_drop", {
+      laneId: "lane-1",
+      stashRef: "stash@{0}",
+      stashOid: "oid-0",
+    });
+
+    expect(pop?.isError).toBeUndefined();
+    expect(drop?.isError).toBeUndefined();
+    expect(fixture.runtime.gitService.stashPop).toHaveBeenCalledWith({
+      laneId: "lane-1",
+      stashRef: "stash@{0}",
+      stashOid: "oid-0",
+    });
+    expect(fixture.runtime.gitService.stashDrop).toHaveBeenCalledWith({
+      laneId: "lane-1",
+      stashRef: "stash@{0}",
+      stashOid: "oid-0",
+    });
+  });
+
   it("returns resources for lane status/conflicts", async () => {
     const fixture = createRuntime();
     const handler = createAdeRpcRequestHandler({ runtime: fixture.runtime, serverVersion: "test" });

apps/ade-cli/src/adeRpcServer.test.ts

@@ -1046,7 +1046,8 @@ const TOOL_SPECS: ToolSpec[] = [
       additionalProperties: false,
       properties: {
         laneId: { type: "string", minLength: 1 },
-        stashRef: { type: "string", minLength: 1 }
+        stashRef: { type: "string", minLength: 1 },
+        stashOid: { type: "string", minLength: 1 }
       }
     }
   },
@@ -1055,11 +1056,12 @@ const TOOL_SPECS: ToolSpec[] = [
     description: "Pop a stash onto a lane and remove it from the stash list. Defaults to the current chat lane when laneId is omitted.",
     inputSchema: {
       type: "object",
-      required: ["stashRef"],
+      required: ["stashRef", "stashOid"],
       additionalProperties: false,
       properties: {
         laneId: { type: "string", minLength: 1 },
-        stashRef: { type: "string", minLength: 1 }
+        stashRef: { type: "string", minLength: 1 },
+        stashOid: { type: "string", minLength: 1 }
       }
     }
   },
@@ -1068,11 +1070,12 @@ const TOOL_SPECS: ToolSpec[] = [
     description: "Drop a stash from a lane. Defaults to the current chat lane when laneId is omitted.",
     inputSchema: {
       type: "object",
-      required: ["stashRef"],
+      required: ["stashRef", "stashOid"],
       additionalProperties: false,
       properties: {
         laneId: { type: "string", minLength: 1 },
-        stashRef: { type: "string", minLength: 1 }
+        stashRef: { type: "string", minLength: 1 },
+        stashOid: { type: "string", minLength: 1 }
       }
     }
   },
@@ -6526,21 +6529,24 @@ async function runTool(args: {
   if (name === "stash_apply") {
     const laneId = requireLaneIdForTool(runtime, session, toolArgs, "stash_apply");
     const stashRef = assertNonEmptyString(toolArgs.stashRef, "stashRef");
-    const action = await runtime.gitService.stashApply({ laneId, stashRef });
+    const stashOid = asOptionalTrimmedString(toolArgs.stashOid);
+    const action = await runtime.gitService.stashApply({ laneId, stashRef, ...(stashOid ? { stashOid } : {}) });
     return { action };
   }
 
   if (name === "stash_pop") {
     const laneId = requireLaneIdForTool(runtime, session, toolArgs, "stash_pop");
     const stashRef = assertNonEmptyString(toolArgs.stashRef, "stashRef");
-    const action = await runtime.gitService.stashPop({ laneId, stashRef });
+    const stashOid = assertNonEmptyString(toolArgs.stashOid, "stashOid");
+    const action = await runtime.gitService.stashPop({ laneId, stashRef, stashOid });
     return { action };
   }
 
   if (name === "stash_drop") {
     const laneId = requireLaneIdForTool(runtime, session, toolArgs, "stash_drop");
     const stashRef = assertNonEmptyString(toolArgs.stashRef, "stashRef");
-    const action = await runtime.gitService.stashDrop({ laneId, stashRef });
+    const stashOid = assertNonEmptyString(toolArgs.stashOid, "stashOid");
+    const action = await runtime.gitService.stashDrop({ laneId, stashRef, stashOid });
     return { action };
   }