fix: the inbox opens a review at its diff, not at the empty working tree

packages/api/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@diffity/api",
-  "version": "0.10.10",
+  "version": "0.10.11",
   "private": true,
   "type": "module",
   "main": "./dist/index.js",

packages/cli/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@naturalcycles/diffity",
-  "version": "0.10.10",
+  "version": "0.10.11",
   "description": "Agent-agnostic, GitHub-style diff viewer and code review tool with a live agent loop",
   "type": "module",
   "bin": {

packages/cli/src/inbox/open-session.ts

@@ -12,12 +12,21 @@ import { checkInstanceHealth, findInstanceForRepo } from '../registry.js';
 export async function openPreparedSession(worktree: string, bundlePath: string, deps: OpenSessionDeps): Promise<OpenedSession> {
   const ref = deps.baseRefOf(bundlePath);
   const port = await deps.ensureServer(worktree, ref);
+  const url = sessionUrl(port, ref);
   try {
     deps.importBundle(worktree, bundlePath);
   } catch (err) {
-    return { url: `http://localhost:${port}/`, imported: false, importError: err instanceof Error ? err.message : String(err) };
+    return { url, imported: false, importError: err instanceof Error ? err.message : String(err) };
   }
-  return { url: `http://localhost:${port}/`, imported: true };
+  return { url, imported: true };
+}
+
+/**
+ * The diff page at the session's ref. The bare root redirects to the working-tree diff, which is
+ * empty for a clean worktree at the pull request's head.
+ */
+function sessionUrl(port: number, ref: string): string {
+  return `http://localhost:${port}/diff?ref=${encodeURIComponent(ref)}`;
 }
 
 /** The base commit a bundle was built against; the session diffs the worktree against it. */

packages/cli/tests/inbox-open.test.ts

@@ -88,7 +88,7 @@ describe('openPreparedSession', () => {
 
     const result = await openPreparedSession('/wt', '/b.json', deps);
 
-    expect(result).toEqual({ url: 'http://localhost:5599/', imported: true });
+    expect(result).toEqual({ url: 'http://localhost:5599/diff?ref=basesha', imported: true });
     expect(calls).toEqual(['ensure /wt basesha', 'import /wt /b.json']);
   });
 
@@ -99,7 +99,7 @@ describe('openPreparedSession', () => {
       importBundle: () => { throw new Error('head moved'); },
     };
     const result = await openPreparedSession('/wt', '/b.json', deps);
-    expect(result).toEqual({ url: 'http://localhost:5599/', imported: false, importError: 'head moved' });
+    expect(result).toEqual({ url: 'http://localhost:5599/diff?ref=basesha', imported: false, importError: 'head moved' });
   });
 });
 
@@ -179,7 +179,7 @@ describe('the inbox server routes', () => {
     try {
       const res = await fetch(`http://127.0.0.1:${port}/open/${encodeURIComponent('o/r#4')}`, { redirect: 'manual' });
       expect(res.status).toBe(302);
-      expect(res.headers.get('location')).toBe('http://localhost:7788/');
+      expect(res.headers.get('location')).toBe('http://localhost:7788/diff?ref=basesha');
 
       store.observe({ ...snapshot(), number: 8 }, true, 'now');
       const notReady = await fetch(`http://127.0.0.1:${port}/open/${encodeURIComponent('o/r#8')}`, { redirect: 'manual' });

packages/git/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@diffity/git",
-  "version": "0.10.10",
+  "version": "0.10.11",
   "private": true,
   "type": "module",
   "main": "./dist/index.js",

packages/github/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@diffity/github",
-  "version": "0.10.10",
+  "version": "0.10.11",
   "private": true,
   "type": "module",
   "main": "./dist/index.js",

packages/parser/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@diffity/parser",
-  "version": "0.10.10",
+  "version": "0.10.11",
   "private": true,
   "type": "module",
   "main": "./dist/index.js",

packages/ui/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@diffity/ui",
-  "version": "0.10.10",
+  "version": "0.10.11",
   "type": "module",
   "private": true,
   "scripts": {