Render uncommon diff entries correctly

Author: itsjling

app/page.tsx

@@ -30,6 +30,7 @@ type DiffFile = {
   patch: string;
   snippet: string;
   sourceUrl?: string;
+  comparisonUrl?: string;
   summary: FileSummary;
   noteReady?: boolean;
 };
@@ -577,6 +578,16 @@ export default function Home() {
                     Open file ↗
                   </a>
                 ) : null}
+                {currentFile.comparisonUrl ? (
+                  <a
+                    className="text-button"
+                    href={currentFile.comparisonUrl}
+                    target="_blank"
+                    rel="noreferrer"
+                  >
+                    Open comparison ↗
+                  </a>
+                ) : null}
               </div>
             </div>
 

scripts/build-diff-data.mjs

@@ -646,6 +646,9 @@ function resolveCheckoutTarget() {
   const repository = githubRepository(remoteUrl);
   const headLabel = branch || currentHead;
   const hasCommittedChanges = mergeBaseOid !== currentHead;
+  const hasUncommittedChanges = Boolean(
+    tryRepo(['status', '--porcelain=v1', '-z']),
+  );
   const isDefaultBranchCheckout = branch === defaultBranch.name;
 
   return {
@@ -659,6 +662,7 @@ function resolveCheckoutTarget() {
     remote,
     sourceRepositoryUrl: repository?.webUrl,
     baseRepositoryUrl: repository?.webUrl,
+    hasUncommittedChanges,
     target: {
       kind: 'checkout',
       ...(remote ? { remote: remote.name } : {}),
@@ -812,6 +816,13 @@ function githubFileUrl(repositoryUrl, ref, path) {
   return `${repositoryUrl}/blob/${encodeURIComponent(ref)}/${filePath}`;
 }
 
+function githubComparisonUrl(repositoryUrl, base, head) {
+  if (!repositoryUrl || !base || !head || head === 'WORKTREE') {
+    return undefined;
+  }
+  return `${repositoryUrl}/compare/${encodeURIComponent(base)}...${encodeURIComponent(head)}`;
+}
+
 function build() {
   const localWorkspace =
     tryRepo(['rev-parse', '--is-inside-work-tree']) === 'true';
@@ -887,6 +898,11 @@ function build() {
       file.status === 'deleted' ? target.base : target.head,
       file.path,
     );
+    const comparisonUrl = githubComparisonUrl(
+      target.hasUncommittedChanges ? undefined : target.sourceRepositoryUrl,
+      target.base,
+      target.head,
+    );
     return {
       path: file.path,
       ...(file.oldPath ? { oldPath: file.oldPath } : {}),
@@ -899,6 +915,7 @@ function build() {
       patch: textPatch,
       snippet: binary ? '' : compactSnippet(textPatch),
       ...(sourceUrl ? { sourceUrl } : {}),
+      ...(comparisonUrl ? { comparisonUrl } : {}),
     };
   });
 

tests/remote-targets.test.mjs

@@ -141,6 +141,13 @@ test("names worktree-only checkout changes without comparing a branch to itself"
 
   try {
     await writeFile(join(fixture.repo, "working.txt"), "working tree work\n");
+    git(
+      fixture.repo,
+      "remote",
+      "set-url",
+      "origin",
+      "https://github.com/example/diffsplain.git",
+    );
 
     run(fixture.repo, ["--checkout", "--output", output]);
     const payload = JSON.parse(await readFile(output, "utf8"));
@@ -152,6 +159,10 @@ test("names worktree-only checkout changes without comparing a branch to itself"
       payload.change.summary,
       "Shows staged, unstaged, and untracked changes in the current checkout.",
     );
+    assert.ok(
+      payload.files.every((file) => file.comparisonUrl === undefined),
+      "uncommitted work must not link to a commit-only comparison",
+    );
 
     git(fixture.repo, "add", "working.txt");
     git(fixture.repo, "commit", "-qm", "local main work");
@@ -161,6 +172,10 @@ test("names worktree-only checkout changes without comparing a branch to itself"
 
     assert.notEqual(committed.repo.base, committed.repo.head);
     assert.equal(committed.change.title, "Local changes on main");
+    assert.match(
+      committed.files[0].comparisonUrl,
+      /^https:\/\/github\.com\/example\/diffsplain\/compare\//,
+    );
   } finally {
     await rm(fixture.root, { recursive: true, force: true });
   }
@@ -199,6 +214,104 @@ test("builds a remote repo target without a local checkout", async () => {
   }
 });
 
+test("renders uncommon range entries with the right content and GitHub links", async () => {
+  const fixture = await makeRemoteRepo();
+  const output = join(fixture.root, "uncommon-range.json");
+
+  try {
+    await writeFile(join(fixture.repo, "deleted.txt"), "remove me\n");
+    await writeFile(join(fixture.repo, "moved-from.txt"), "move me\n");
+    await writeFile(join(fixture.repo, "changed.bin"), Buffer.from([0, 1]));
+    await writeFile(
+      join(fixture.repo, "long.txt"),
+      Array.from({ length: 240 }, (_, index) => `before ${index}\n`).join(""),
+    );
+    git(fixture.repo, "add", ".");
+    git(fixture.repo, "commit", "-qm", "uncommon base");
+    const base = git(fixture.repo, "rev-parse", "HEAD");
+
+    await writeFile(join(fixture.repo, "changed.bin"), Buffer.from([0, 2]));
+    await writeFile(join(fixture.repo, "added.bin"), Buffer.from([0, 4]));
+    await rm(join(fixture.repo, "deleted.txt"));
+    git(fixture.repo, "mv", "moved-from.txt", "moved-to.txt");
+    await writeFile(
+      join(fixture.repo, "long.txt"),
+      Array.from({ length: 240 }, (_, index) => `after ${index}\n`).join(""),
+    );
+    git(fixture.repo, "add", ".");
+    git(fixture.repo, "commit", "-qm", "uncommon changes");
+    const head = git(fixture.repo, "rev-parse", "HEAD");
+    git(fixture.repo, "remote", "set-url", "origin", "https://github.com/example/diffsplain.git");
+    const before = checkoutState(fixture.repo);
+
+    run(fixture.repo, ["--base", base, "--head", head, "--output", output]);
+    const payload = JSON.parse(await readFile(output, "utf8"));
+    const files = Object.fromEntries(payload.files.map((file) => [file.path, file]));
+    const source = (ref, path) =>
+      `https://github.com/example/diffsplain/blob/${ref}/${path}`;
+    const comparison = `https://github.com/example/diffsplain/compare/${base}...${head}`;
+
+    assert.deepEqual(
+      payload.files.map((file) => file.path),
+      ["added.bin", "changed.bin", "deleted.txt", "long.txt", "moved-to.txt"],
+    );
+    assert.equal(files["added.bin"].status, "binary");
+    assert.equal(files["added.bin"].isBinary, true);
+    assert.equal(files["added.bin"].patch, "");
+    assert.equal(files["added.bin"].sourceUrl, source(head, "added.bin"));
+    assert.equal(files["added.bin"].comparisonUrl, comparison);
+    assert.equal(files["changed.bin"].status, "binary");
+    assert.equal(files["changed.bin"].isBinary, true);
+    assert.equal(files["changed.bin"].patch, "");
+    assert.equal(files["changed.bin"].sourceUrl, source(head, "changed.bin"));
+    assert.equal(files["changed.bin"].comparisonUrl, comparison);
+    assert.equal(files["deleted.txt"].status, "deleted");
+    assert.equal(files["deleted.txt"].isBinary, false);
+    assert.match(files["deleted.txt"].patch, /-remove me/);
+    assert.equal(files["deleted.txt"].sourceUrl, source(base, "deleted.txt"));
+    assert.equal(files["deleted.txt"].comparisonUrl, comparison);
+    assert.equal(files["moved-to.txt"].status, "renamed");
+    assert.equal(files["moved-to.txt"].oldPath, "moved-from.txt");
+    assert.match(files["moved-to.txt"].patch, /similarity index 100%/);
+    assert.equal(files["moved-to.txt"].sourceUrl, source(head, "moved-to.txt"));
+    assert.equal(files["moved-to.txt"].comparisonUrl, comparison);
+    assert.equal(files["long.txt"].status, "modified");
+    assert.equal(files["long.txt"].isBinary, false);
+    assert.equal(files["long.txt"].isTruncated, true);
+    assert.ok(files["long.txt"].snippet.split("\n").length <= 180);
+    assert.match(files["long.txt"].snippet, /^@@ /m);
+    assert.equal(files["long.txt"].sourceUrl, source(head, "long.txt"));
+    assert.equal(files["long.txt"].comparisonUrl, comparison);
+    assert.deepEqual(checkoutState(fixture.repo), before);
+  } finally {
+    await rm(fixture.root, { recursive: true, force: true });
+  }
+});
+
+test("keeps links out of worktree entries and leaves the checkout untouched", async () => {
+  const fixture = await makeRemoteRepo();
+  const output = join(fixture.root, "uncommon-worktree.json");
+
+  try {
+    await writeFile(join(fixture.repo, "worktree.bin"), Buffer.from([0, 1]));
+    const before = checkoutState(fixture.repo);
+
+    run(fixture.repo, ["--worktree", "--output", output]);
+    const payload = JSON.parse(await readFile(output, "utf8"));
+    const [file] = payload.files;
+
+    assert.equal(file.path, "worktree.bin");
+    assert.equal(file.status, "binary");
+    assert.equal(file.isBinary, true);
+    assert.equal(file.patch, "");
+    assert.equal(file.sourceUrl, undefined);
+    assert.equal(file.comparisonUrl, undefined);
+    assert.deepEqual(checkoutState(fixture.repo), before);
+  } finally {
+    await rm(fixture.root, { recursive: true, force: true });
+  }
+});
+
 test("builds a pull request range through gh without changing the checkout", async () => {
   const fixture = await makeRemoteRepo();
   const bin = join(fixture.root, "bin");