You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
fix(source-control): renamed files in GitHub, GitLab and Azure DevOps diffs show as renames - #29774
In a GitHub, GitLab or Azure DevOps pull request, a renamed file shows up as an ordinary change under its new name. The diff gives no sign that it moved, and a rename with no edits appears as an empty change.
The server builds each forge's diff as a git-style patch with rename from and rename to lines, but leaves out similarity index. @pierre/diffs marks a git-style section as a rename only when it sees that line. Without it, the parser sets the type to change and drops prevName. Hunk expansion is affected too: the file contents loader has no old path, so it sends the new path for the old side. For GitHub this reads the new path at the merge base, where the file does not exist yet, and expanding context on a renamed and edited file fails.
Change
Each builder now writes similarity index before the rename lines, like the GitCafe builder already does (added in #24276, with the comment that Pierre needs this header). None of these APIs gives a score, so the value is 100% for a rename with no hunks and no withheld changes, and 0% otherwise. Pierre reads 100% as rename-pure and anything else as rename-changed.
GitHub toFilesPatch: changed if it has hunks or a non-zero additions/deletions count.
GitLab mergeRequestPatch: changed if it has hunks or is too_large/collapsed.
Azure DevOps patchHeader: uses the rename-pure/rename-changed kind Azure already reports.
Scope and approval
Very small, focused fix for an obvious bug, so no prior issue: three forge builders leave out a header that their GitCafe sibling already writes for the same reason, and real git always writes it. This is one root cause, so all three builders are in one PR. No contract, client or default changes.
Overlap search: I searched the local PR index and GitHub for "similarity index" and for rename + diff/pull request. The only hit is #10822 (closed, unrelated). #27331 changes the Azure diff slice budget but not the rename header lines.
Verification
Red on main: the 5 extended rename assertions fail in gitHubPullRequestJson.test.ts (2), gitLabMergeRequestJson.test.ts (1) and azureDevOpsDiff.test.ts (2), for example AssertionError: expected 'diff --git a/src/old.ts b/src/new.ts\…' to be 'diff --git a/src/old.ts b/src/new.ts\…', because the similarity index line is missing.
Green: vp test run passes 56/56 (GitHub), 44/44 (GitLab) and 14/14 (Azure DevOps), rerun after rebasing onto the latest main.
vp run check:changed passes: format, lint, knip, graphql, typecheck (13 packages), tests (4 files, 115 passed).
End to end, I parsed each builder's real output with the web app's @pierre/diffs:
Before, the GitHub/GitLab text for a renamed and edited file gave {"name":"src/new.ts","type":"change"} with no prevName.
After, GitHub, GitLab and Azure DevOps give {"name":"src/new.ts","prevName":"src/old.ts","type":"rename-changed"}, and a rename with no hunks gives rename-pure.
Not checked: I did not open a real GitHub, GitLab or Azure DevOps pull request in a client, so the rendered rename label and the expansion request are inferred from the parser output and from createDiffFileContentsLoader, not captured.
Reviewed with sol-loop: 1 round with GPT-6.1 Sol on high. Sol verdict: approved.
Created with Claude Opus 5.5 in Claude Code (T3 Code).
Macroscope's review found this PR approvable — Adds rename metadata to GitHub, GitLab and Azure DevOps diff output so renamed files remain recognizable as renames. The change is limited to diff representation and does not modify repository contents or product defaults.
No code changes detected at eb53486. Prior analysis still applies.
You can add or adjust custom eligibility rules. Learn more.
Reviewing files that changed from the base of the PR and between b84f59e and eb53486.
You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.
Use the checkbox below for a quick retry:
Trigger review
No actionable comments were generated in the recent review.
Walkthrough
Azure DevOps and GitLab now include similarity-index headers in patches for renamed files. GitHub retains its existing similarity calculation, with a comment documenting the rename-header requirement. Tests assert the expected headers across the three providers.
Azure DevOps adds 100% for pure renames and 0% for other renames. GitLab adds 0% when hunks exist or the diff is too large or collapsed, and 100% otherwise. GitHub documents its existing calculation. Tests assert the expected headers.
A narrow GitHub binary-rename case may be shown as unchanged; address or explicitly accept this before merging.
Pre-merge checks: 4
Check name
Status
Explanation
Title check
The title clearly and concisely describes the main fix across GitHub, GitLab, and Azure DevOps diffs.
Description check
The description includes complete Problem, Change, Scope and approval, and Verification sections. It explains the root cause, implementation, test results, limitations, and approval rationale.
Linked Issues check
Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check
Check skipped because no linked issues were found for this pull request.
Finishing Touches
Generate unit tests (beta)
Create a new PR
Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
Tip
🎯 Customize your risk assessments
Guide how CodeRabbit assesses merge risk and security architecture risk. Highlight sensitive changes and explain your security boundaries using Merge Risk Instructions and Security Risk Instructions in your review settings. Security architecture risk assessments require the Advanced plan or above.
✅ Automatic approvals, with your rules
Let CodeRabbit approve pull requests once required threads are resolved, pre-merge checks pass, and your risk limits are met. Enable Automatic approvals in review settings, with overrides for teams or file paths.
CodeRabbit Verify
Run CodeRabbit Verify to test the changes in this pull request.
merged current main into the existing head without rebasing. this brings in the upstream fix for the file-save browser test's discovery-poll hang; no additional source changes were needed. the diff remains six files with 11 added lines.
verified on b84f59eff76a53b3471e7cf9f867194110f5ee12: blacksmith vp run check:changed github/main passed format, lint, knip, graphql, 13 package typechecks, and 115 tests. the exact previously failing fileSaveSessions.browser.test.tsx also passed on blacksmith. runtime checks passed each adapter's pure and edited rename output through the client's installed pierre parser and confirmed the old path and rename type. two independent source reviews approved this head.
the github binary-rename ambiguity mentioned in the bot summary predates this change: a hunkless response with zero line counts exposes neither binary content changes nor a similarity score. this patch preserves the rename paths without expanding scope to binary-content detection. live forge requests and client rendering remain unverified.
verified unchanged head eb53486db5405d39367ceb6151971e356b1a92f2 against github/main at 61c3794c32 on a linux boat vm. installed both checkouts with vp i and ran one dev server at a time with isolated state. drove the web app using repository playwright and /usr/bin/google-chrome.
created a project and thread through the app and completed real codex gpt-6.1-sol turns on both versions. opened the live github diff for pingdotgg/t3code pr 19463, selected commit 23d9dbd, and exercised its renamed files without changing that pr.
main: boundedZip.ts and zipDirectory.ts appeared only under their new paths. expanding unchanged context on the edited rename failed with PullRequestOperationError: This file is not part of the change request.
this head: both show the old path, arrow, and new path. expanding the edited rename loaded the previously omitted lines, including const entries = Object.values(zip.files) and the archive-entry guard.
neighboring behavior: the pure rename preserves both paths; the ordinary officeArchive.ts edit renders its changed import; switching back to the whole-pr diff preserves both renames.
blacksmith: the three existing adapter suites passed, 114 tests total (github 56, gitlab 44, azure devops 14). no source changes or pushes were needed.
main, same live commit:
pr head, same live commit:
Not verified
live gitlab and azure devops requests: those integrations are not configured in this environment; their coverage here is the existing adapter tests. native desktop, ios/android, remote/tunnel connections, binary renames, and the oversized github whole-pr fallback were not exercised. the github per-commit view exercised the changed files-api builder directly. the uploaded recording was inspected and its browser playback and seeking passed.
…19)
* perf(client-runtime): expanding deferred work skips an unused timeline rebuild (pingdotgg#29114)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): honor relocated systemd user configuration (pingdotgg#29244)
* fix(web): review comments keep a trailing blank line in their diff (pingdotgg#29575)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): side chat and merge-back context no longer cut an emoji in half (pingdotgg#29572)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* perf(server): loading a file's diff skips unused repository lookups (pingdotgg#29115)
Co-authored-by: Claude Opus 5.5 <[email protected]>
Co-authored-by: maria-rcks <[email protected]>
* fix(cli): make relative install homes produce working commands (pingdotgg#29241)
* fix(forgejo): dismissed reviews no longer read as requested changes or approvals (pingdotgg#29567)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(shared): Linux file links preserve literal backslashes (pingdotgg#29245)
* fix(gitcafe): PR link previews show a GitHub-merged pull request as merged (pingdotgg#29595)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(codex): restoring a checkpoint works after the Codex session was archived (pingdotgg#29582)
Co-authored-by: Claude Opus 5.5 <[email protected]>
Co-authored-by: maria-rcks <[email protected]>
* perf(acp): ACP agents skip decoding output twice for unused diagnostics (pingdotgg#29100)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): diffs keep blank context lines under diff.suppressBlankEmpty (pingdotgg#29579)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(scripts): check:changed passes when only lint-ignored code changed (pingdotgg#29596)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* perf(client): streamed row updates no longer rescan question history (pingdotgg#29105)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* perf(desktop): keep Linux MIME helpers off the startup path (pingdotgg#29247)
* perf(server): shell updates read a burst of threads once (pingdotgg#29101)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(opencode): forks from compacted threads no longer fail (pingdotgg#29221)
* fix(scripts): check:changed lints Astro components and runs knip for them (pingdotgg#29573)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(desktop): Linux .deb no longer installs files any local user can overwrite (pingdotgg#28351)
Co-authored-by: PR Batch Tester <[email protected]>
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(desktop): native builds prepare server bytecode again after the Effect 4.1.2 upgrade (pingdotgg#29780)
Co-authored-by: maria-rcks <[email protected]>
* fix(codex): a background command's wake no longer hangs when its output tail splits an emoji (pingdotgg#29168)
Co-authored-by: Claude Opus 5.5 <[email protected]>
Co-authored-by: maria-rcks <[email protected]>
* test(scripts): expect the unmatched-pattern flag in the Astro check:changed test (pingdotgg#29839)
Co-authored-by: maria-rcks <[email protected]>
* fix(mobile): drafts on threads with colons in their ids upload attachments early (pingdotgg#29717)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): provider refresh no longer reverts an enriched sign-in status (pingdotgg#29627)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): forked Claude threads keep their compaction chain (pingdotgg#27577)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): Claude subagents left by a stopped process no longer stay Running (pingdotgg#28356)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): usage-limit auto-resume no longer arms children of settled parents (pingdotgg#27951)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(cursor): a write-after-end in the Cursor SDK no longer stops the server (pingdotgg#27703)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): avoid reparsing streamed replies for timestamps (pingdotgg#25252)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(web): project settings can set a checkout's grouping rule again (pingdotgg#23641)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): reject oversized goal objectives before dispatch (pingdotgg#23033)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): checkpoints no longer rehash every file when one entry is skip-worktree or assume-unchanged (pingdotgg#23624)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(web,mobile): chat images reload when the agent rewrites the file (pingdotgg#29741)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(web): show the project pin before the remote badge (pingdotgg#29704)
* fix(server): Codex usage keeps requests from interrupted turns (pingdotgg#29415)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(client-runtime): connect to the primary environment while the browser reports offline (pingdotgg#29347)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): a failed worktree add no longer leaves its new branch behind (pingdotgg#28373)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(provider-acp): Kiro's html_render pages show in the thread (pingdotgg#28253)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): streamed Browser clicks no longer wait behind queued wheel bursts (pingdotgg#28236)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): Codex turns that fail with an unlisted error code no longer stay running (pingdotgg#28040)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(opencode): sessions open through OpenCode 2 after an in-place upgrade from 1.x (pingdotgg#27779)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): T3 Connect not-linked fix no longer tells Desktop users to run t3 service restart (pingdotgg#27660)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): anchor Claude wake cooldowns to result arrival (pingdotgg#25061)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(web): match new-thread hero to selected checkout (pingdotgg#25064)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): stop PR lookups in emptied worktree folders (pingdotgg#24803)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): agent session scans no longer hang the server on an unreadable .git/config (pingdotgg#23690)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* docs(connect): held webhooks are kept per managed link (pingdotgg#23612)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): delegated children keep replies after their parent settles (pingdotgg#23456)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(preview): explain invalid URLs and locators (pingdotgg#23396)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(web): terminal controls no longer cover the first line (pingdotgg#23373)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): stop maintaining the unused event correlation index (pingdotgg#23211)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* docs(mcp): document existing recurring-task tools (pingdotgg#23199)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): release the previous session when sending to another instance (pingdotgg#23198)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(claude): a background-task wake run no longer hangs after a user turn absorbed its wake (pingdotgg#28010)
Co-authored-by: maria-rcks <[email protected]>
* fix(server): OpenCode no longer drops a $skill read before its directory scan ends (pingdotgg#29535)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): coalesced webhook runs use the prompt saved before dispatch (pingdotgg#29476)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): slow forge CLI calls no longer delay checkpoints and turn starts (pingdotgg#28431)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(relay): T3 Connect repairs tunnel ingress that drifted from a matching origin (pingdotgg#27961)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(provider-core): npm-installed Claude in ~/.local gets npm updates (pingdotgg#27957)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): git hooks keep the user's locale (pingdotgg#27778)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(pi): hold messages until compaction ends (pingdotgg#24388)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): show why Claude refuses full access as root (pingdotgg#23804)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): preview click timeouts identify the target (pingdotgg#23489)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(cursor): supporting files no longer hide sibling skills (pingdotgg#23326)
Co-authored-by: Claude Fable 5.1 <[email protected]>
* fix(web): usage-limit retry notice shows the timezone (pingdotgg#23029)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): preserve nested Claude subagent completion across turns (pingdotgg#23561)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(opencode): publish ordered pending-work snapshots (pingdotgg#21460)
Co-authored-by: Exotic209093 <[email protected]>
Co-authored-by: Issue triage verification <triage@localhost>
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(opencode): decode compaction history from OpenCode 2.1 (pingdotgg#22661)
Co-authored-by: Claude Fable 5.1 <[email protected]>
* fix(server): import valid conversations within each transcript record limit (pingdotgg#23192)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): imported Claude sessions resume on their first turn (pingdotgg#23219)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* feat(web): ask for a GitHub star when idle, and star in one click (pingdotgg#29922)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): subagent questions reach a busy parent agent (pingdotgg#29909)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* feat(web): celebrate a GitHub star with confetti (pingdotgg#29987)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(pi): short-lived Pi processes no longer lose rotated OAuth tokens (pingdotgg#30187)
* feat(web): add refresh to provider status banners (pingdotgg#24861)
* fix(server): side chats no longer open with a model switch from the parent's turn (pingdotgg#29752)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(web): copied code blocks keep their blank lines in lists, quotes and longer fences (pingdotgg#29755)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(web): compact model picker row indicators (pingdotgg#26415)
* fix(web): open project picker from boring sidebar's header (pingdotgg#26495)
Co-authored-by: maria-rcks <[email protected]>
* fix(client): project removal counts only listed threads, not side chats or subagents (pingdotgg#29917)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(opencode): text generation errors explain known provider failures without raw provider text (pingdotgg#29877)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(mobile): multi-file patches show every changed file (pingdotgg#29878)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): handoff context names every file of a multi-file patch (pingdotgg#29905)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): commits and PR text see changes outside a subfolder project under diff.relative (pingdotgg#29879)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(web): opening a project from the command palette no longer lands in a subagent or side chat (pingdotgg#29891)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(web): device host checks show the error without a stack trace (pingdotgg#29912)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* feat(web): open full access menu from the new thread context strip (pingdotgg#30275)
* fix(web): long server lists in thread details now page 4 at a time (pingdotgg#28684)
Co-authored-by: Claude Opus 5.5 <[email protected]>
Co-authored-by: Julius Marminge <[email protected]>
* fix(web): keep the project filter menu aligned with its trigger
* fix(github): Rebase stack no longer conflicts after a lower layer is squash-merged (pingdotgg#29919)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): changing a task's wake policy no longer reverts a checkpointed parent run (pingdotgg#25662)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* perf(server): settled PR lookups stop re-spawning git every sweep (pingdotgg#29721)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): generated commit and PR text ignores git signature output (pingdotgg#29580)
Co-authored-by: Claude Opus 5.5 <[email protected]>
Co-authored-by: maria-rcks <[email protected]>
* fix(web): picking a branch in a subfolder project keeps the thread in the project folder (pingdotgg#29908)
Co-authored-by: Claude Opus 5.5 <[email protected]>
Co-authored-by: Maria <[email protected]>
* fix(mobile): close and reopen pull requests from the actions menu (pingdotgg#29962)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(web): mouse wheel over the thread minimap glides instead of jumping (pingdotgg#29038)
* fix(server): managed policy covers Kiro (pingdotgg#29915)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): t3 update installs from the policy's update mirror (pingdotgg#29911)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(desktop): Ctrl+R on macOS reaches the preview page instead of reloading it (pingdotgg#29893)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): native forks after an undo no longer copy later turns (pingdotgg#29902)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): idle compaction still runs after a queued message is removed or steered (pingdotgg#29904)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): guests of a forked thread can open files on the messages it inherited (pingdotgg#29794)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(evals): resume browser verification after unavailable status (pingdotgg#30298)
* fix(shared): saving one source control token no longer drops the other hosts' tokens (pingdotgg#29554)
Signed-off-by: Yordis Prieto <[email protected]>
* fix(desktop): a policy that stops startup no longer points at another release (pingdotgg#30319)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* docs: pin an absolute binary path under a permission cap (pingdotgg#29916)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(mobile): Android follow-up menu shows in full with the keyboard open (pingdotgg#29263)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(web): deleting the open thread no longer opens a snoozed or settled thread (pingdotgg#29643)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(web): closing Settings or Usage no longer jumps back to a quote's source (pingdotgg#29013)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): squash-merged branches no longer offer push & create PR (pingdotgg#28494)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(mobile): settings providers show the ACP registry icon (pingdotgg#28135)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): Grok usage limits tell Legacy-mode xAI accounts to switch to Privacy Mode (pingdotgg#28097)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(usage): label a proxied Claude account by its e-mail, and drop .json from hub rows (pingdotgg#28065)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(gitlab): a signed-out default glab host no longer hides other signed-in hosts (pingdotgg#27840)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* perf(server): prune events the projections already cover (pingdotgg#29806)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): GPT-6.1 Sol no longer compacts after every tool call on managed Codex (pingdotgg#30004)
Co-authored-by: maria-rcks <[email protected]>
Co-authored-by: Claude Opus 5.5 <[email protected]>
* feat(policy): admins set managed policy from a Group Policy template (pingdotgg#29862)
Co-authored-by: Matthew Feroz <[email protected]>
Co-authored-by: Claude Opus 5.5 <[email protected]>
* perf(server): bound streaming recovery reads by active runs (pingdotgg#30357)
* perf(server): read pull request metadata from a lightweight snapshot
PR discovery used the full shell snapshot, which derived activity and counted historical items across active threads. Read only PR metadata and its event sequence in one transaction, then decode after releasing the reader. Preserve settled backfill, targeted refreshes, legacy PR normalization, and guarded updates.
Validation: check:changed passed with 80 tests after merging main; CI passed on 988187f. Native Electron measurements and a concurrent-write control confirmed metadata parity and coherent sequences. Synthetic measurements cover PR reads and service initialization, not full desktop startup.
Model: GPT 6.1 Sol. Harness: Codex in T3 Code.
* fix(server): cancelling worktree setup no longer leaves its new branch behind (pingdotgg#29781)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* feat(mobile): add on-device Android dictation (pingdotgg#26734)
Co-authored-by: Julius Marminge <[email protected]>
Co-authored-by: Claude Opus 5.5 <[email protected]>
Co-authored-by: Alex Jankowiak <[email protected]>
* feat(server): read usage limit source keys from a secret provider (pingdotgg#27760)
Signed-off-by: Yordis Prieto <[email protected]>
Co-authored-by: maria-rcks <[email protected]>
Co-authored-by: Julius Marminge <[email protected]>
Co-authored-by: Claude Opus 5.5 <[email protected]>
* feat(device): read the EAS access token from a secret provider (pingdotgg#27763)
Signed-off-by: Yordis Prieto <[email protected]>
Co-authored-by: maria-rcks <[email protected]>
Co-authored-by: Julius Marminge <[email protected]>
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): route missing read-only queries to readers
## Problem
Several read-only queries in projection reads, legacy shell reconciliation, and peer link lookups ran on the SQLite writer. This added avoidable work to the writer connection and could delay writes.
## Change
Marked `getThreadProjection`, the independent legacy importer selectors, and peer link reads with the SQLite `readOnly` context marker. The full projection read uses one reader transaction so concurrent commits cannot mix its local and inherited history. Reads inside existing write transactions retain the writer and see uncommitted changes. Added file-backed routing coverage for reader selection and transaction visibility.
## Scope and approval
This is a focused fix for missing read-only routing boundaries identified in the parent audit. It changes no SQL selection logic or pool configuration. Full projection reads now hold one consistent snapshot.
## Verification
The file-backed tests recorded projection queries, importer selectors, and peer link lookups on reader connections. They also confirmed that reads inside write transactions see uncommitted projection, repair, shell, and link updates. A deterministic concurrent-commit regression failed before the projection transaction fix and passes afterward. Focused suites passed, with 90 tests. `vp run check:changed` passed, including 72 tests in its final run, and `git diff --check` passed.
No startup time saving was measured. The existing shell snapshot query cost is unchanged.
GPT 6.1 Sol, Codex
* perf(server): index delegated recovery candidates (pingdotgg#30359)
Add guarded partial indexes for delegated recovery candidate selectors and retain existing recovery eligibility. Include migration and query-plan regression coverage, plus preview-upgrade expectations.
GPT 6.1 Sol, Codex.
* feat(server,web,mobile,desktop): Copy thread copies a link that opens the thread (pingdotgg#26290)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* feat(mobile,web): copied thread links open the mobile app (pingdotgg#29815)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(antigravity): rate-limited turns keep their reset time (pingdotgg#29594)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): startup no longer hangs on a shell stuck in the kernel (pingdotgg#27938)
Co-authored-by: Claude Opus 5.5 <[email protected]>
Co-authored-by: maria-rcks <[email protected]>
* fix(server): reverting files before the first commit no longer leaves them staged (pingdotgg#28647)
* fix(server): truncated Claude task output no longer starts with a replacement character (pingdotgg#28643)
* fix(shared): relative file links with routing punctuation open as files (pingdotgg#28637)
* fix(server): messages cut by a restart no longer stay marked as streaming (pingdotgg#28646)
* fix(server): Antigravity Auto asks instead of approving every request (pingdotgg#23698)
Co-authored-by: Claude <[email protected]>
Co-authored-by: maria-rcks <[email protected]>
* fix(server): Codex threads open in another client are no longer silently replaced (pingdotgg#21045)
Co-authored-by: maria-rcks <[email protected]>
* fix(server): WSL servers avoid ports already used by Windows (pingdotgg#22962)
Co-authored-by: maria-rcks <[email protected]>
* perf(web): reuse repeated inline skill lookups (pingdotgg#29008)
Co-authored-by: maria-rcks <[email protected]>
* fix(opencode): expand home-relative executable paths (pingdotgg#29326)
Co-authored-by: maria-rcks <[email protected]>
* fix(server): title regeneration works after a thread's worktree is cleaned up (pingdotgg#29606)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): dev servers a Codex agent starts show under its thread (pingdotgg#29687)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): status no longer fetches a remote named after a local tracked branch (pingdotgg#29670)
Co-authored-by: Claude Opus 5.5 <[email protected]>
Co-authored-by: maria-rcks <[email protected]>
* fix(server): preview snapshots no longer cut an emoji in half (pingdotgg#29600)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): restart notes no longer split an emoji in a long background task label (pingdotgg#29647)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(acp): Always allow no longer cancels the turn when an agent only offers allow-once (pingdotgg#29463)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): forking a Codex thread archived outside T3 Code no longer fails (pingdotgg#29618)
Co-authored-by: Claude Opus 5.5 <[email protected]>
Co-authored-by: maria-rcks <[email protected]>
* perf(shared): stop single-token message search at the best hit (pingdotgg#28930)
* fix(preview): a dropped stream withdraws its stale file picker (pingdotgg#29655)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(provider-acp): ACP threads keep receiving events after a rollback (pingdotgg#28987)
Co-authored-by: Trevor Edwards <[email protected]>
Co-authored-by: Claude Opus 5.5 <[email protected]>
* perf(opencode): provider event logs skip per-token reply deltas (pingdotgg#29673)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(web): open files from Claude and ACP tool-call diffs (pingdotgg#29309)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): files tree labels ignored folders inside git submodules (pingdotgg#29650)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): server updates reclaim partial database backups left by killed updates (pingdotgg#29464)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): PR watch wakes no longer split an emoji in a long comment (pingdotgg#29633)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): pinning a settled thread stamps its return to the active list (pingdotgg#29602)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): Claude's commands no longer run Electron apps as Node (pingdotgg#29413)
* docs: keybindings guide lists the rightPanelOpen when key (pingdotgg#29677)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* perf(shared): avoid redundant visibility work on status updates (pingdotgg#28965)
* fix(desktop): failed settings saves no longer leave temp files behind (pingdotgg#29635)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(acp): a final agent message without a trailing newline is no longer dropped (pingdotgg#29605)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): terminal history keeps output after an unterminated OSC (pingdotgg#29604)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(cli): t3 up's pairing hint keeps --base-dir (pingdotgg#29684)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(shared): preserve omitted components in zero-major caret ranges (pingdotgg#29335)
Co-authored-by: maria-rcks <[email protected]>
* fix(server): thread titles use linked PR context when the URL is followed by a colon or bold markers (pingdotgg#29656)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(source-control): remotes like github.com:owner/repo reach their host (pingdotgg#29457)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): terminal history keeps text with C1 characters (pingdotgg#29649)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): worktree handoff follows the project's start-from-origin setting (pingdotgg#29810)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): review keeps tracked changes when untracked files overflow the listing (pingdotgg#29615)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): browser setup advice names the server's T3 home (pingdotgg#29646)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix: honor generic MIME aliases in attachment previews (pingdotgg#29360)
* fix(client-runtime): terminals fit their pane again after a reconnect (pingdotgg#29696)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): Run now on a scheduled task says why it was refused (pingdotgg#29620)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): a preview_open whose first load fails no longer leaves a hidden tab behind (pingdotgg#29865)
Co-authored-by: Claude Opus 5.5 <[email protected]>
Co-authored-by: maria-rcks <[email protected]>
* fix(cursor): Cursor text generation names usage-limit failures (pingdotgg#29748)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): review diffs keep working with untracked files outside the sparse cone (pingdotgg#29880)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(opencode): nested custom commands like /git/commit run as commands (pingdotgg#29765)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(source-control): renamed files in GitHub, GitLab and Azure DevOps diffs show as renames (pingdotgg#29774)
Co-authored-by: Claude Opus 5.5 <[email protected]>
Co-authored-by: maria-rcks <[email protected]>
* fix(opencode): a command that exits non-zero is stored as failed, not completed (pingdotgg#29824)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): linked environments pick up the other machine's new name (pingdotgg#29779)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): document text skips tracked deletions and field codes (pingdotgg#29796)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): OTLP export settings survive a saved token streaming mode (pingdotgg#29786)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): Word text boxes are no longer extracted twice for agents (pingdotgg#29827)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): new projects are created when core.safecrlf refuses LF files (pingdotgg#29843)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(web): drafts saved by older versions load instead of being wiped (pingdotgg#29828)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): Antigravity text generation shows sign-in and setup errors (pingdotgg#29823)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(web): picking a custom model named like a Codex alias keeps that model (pingdotgg#29789)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): turn checkpoints keep LF files when core.safecrlf is true (pingdotgg#29826)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(shared): find in thread matches a word typed with a curly apostrophe (pingdotgg#29759)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): review rows for nested repositories load in large diffs (pingdotgg#29693)
Co-authored-by: Claude Opus 5.5 <[email protected]>
Co-authored-by: maria-rcks <[email protected]>
* fix(server): t3_worktree_list lists refs after storage cleanup removes a thread's worktree (pingdotgg#29645)
Co-authored-by: Claude Opus 5.5 <[email protected]>
Co-authored-by: maria-rcks <[email protected]>
* fix(provider-core): text generation context cuts no longer split an emoji (pingdotgg#29653)
Co-authored-by: Claude Opus 5.5 <[email protected]>
Co-authored-by: maria-rcks <[email protected]>
* fix(server): html_preview no longer splits an emoji in a long console line (pingdotgg#29844)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): diffs keep submodule changes under diff.submodule=log (pingdotgg#29852)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(opencode): OpenCode 2 titles for image messages skip images the server cannot open (pingdotgg#29819)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): orchestrator_capabilities no longer offers providers the project turned off (pingdotgg#29842)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(web): big terminal selections no longer break sending (pingdotgg#29804)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* test(web): file editor dismissal test waits for the editor to render (pingdotgg#30350)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* test(web): worked-for heading test clicks once the list is revealed (pingdotgg#30404)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(azure-devops): legacy DefaultCollection pull request URLs can be pasted to check out (pingdotgg#29763)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(shared): preview_open reports about: and mailto: as unsupported instead of misreading them (pingdotgg#29614)
Co-authored-by: Claude Opus 5.5 <[email protected]>
Co-authored-by: maria-rcks <[email protected]>
* fix(server): delegated threads no longer share one provider event log (pingdotgg#29770)
Co-authored-by: Taylor Dolezal <[email protected]>
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): a queued provider switch no longer ends background work (pingdotgg#30401)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(kiro): slow kiro-cli starts no longer show Kiro as unavailable (pingdotgg#29841)
Co-authored-by: Claude Opus 5.5 <[email protected]>
Co-authored-by: maria-rcks <[email protected]>
* fix(client): mark unread sticks on a thread reopened within ten minutes (pingdotgg#29814)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(pi): extension errors name folder extensions instead of "index" (pingdotgg#29768)
Co-authored-by: Claude Opus 5.5 <[email protected]>
Co-authored-by: maria-rcks <[email protected]>
* fix(github): Enterprise Server pull request URLs can be pasted to check out (pingdotgg#29892)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): a peer refusing one call no longer marks its link as failing (pingdotgg#29914)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* perf(server): Claude skill discovery resolves each repository root once (pingdotgg#29757)
Co-authored-by: Taylor Dolezal <[email protected]>
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): t3 service uninstall leaves another T3 home's service alone (pingdotgg#29872)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): PR watches wake again when a push leaves the branch conflicting (pingdotgg#29767)
Co-authored-by: Claude Opus 5.5 <[email protected]>
Co-authored-by: maria-rcks <[email protected]>
* fix(muse): the thread error banner no longer shows unredacted failure text (pingdotgg#29910)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): provider settings changed mid-refresh no longer revert to the old status (pingdotgg#29894)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(shared): accept whitespace after engine range operators (pingdotgg#29730)
* perf(server): stop tracing per-event live coalescer work (pingdotgg#29758)
Co-authored-by: Taylor Dolezal <[email protected]>
Co-authored-by: Claude Opus 5.5 <[email protected]>
* test(desktop): the Windows shell kill test waits for each spawn before advancing the clock (pingdotgg#30463)
Co-authored-by: maria-rcks <[email protected]>
* fix(shared): ignore unsupported command output control parameters (pingdotgg#29736)
* fix(cli): t3 update no longer says a service awaiting restart runs the new version (pingdotgg#29901)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(mcp): accept CR and mixed line endings in response streams (pingdotgg#29727)
Co-authored-by: maria-rcks <[email protected]>
* fix(server): preview tabs that fail during setup no longer leave their page open (pingdotgg#29885)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(provider-core): long subagent titles no longer end in a broken character (pingdotgg#29906)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(shared): ignore unsupported command output erase modes (pingdotgg#29737)
* fix(server): a failed scheduled run shows its error without a stack trace (pingdotgg#29889)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* perf(web): long threads stream with 70% less work-row derivation (pingdotgg#29971)
* fix(pi): send SIGTERM before closing Pi's stdin on teardown (pingdotgg#30025)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): clean up child worktrees after their parent settles (pingdotgg#30026)
* fix(server): Claude context meter shows the window for models outside the catalog (pingdotgg#30051)
Co-authored-by: maria-rcks <[email protected]>
* fix(server): quitting no longer waits on an unreachable telemetry endpoint (pingdotgg#30033)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(client): keep snoozed failures hidden without completion times (pingdotgg#30063)
* perf(server): PR sync stops reading pull requests closed over a day ago (pingdotgg#30071)
Co-authored-by: Claude Opus 5.5 <[email protected]>
Co-authored-by: maria-rcks <[email protected]>
* fix(acp): file changes show the patch of every diff in a tool call (pingdotgg#28631)
* fix(server): Antigravity sign-in no longer blames the network while the runtime starts (pingdotgg#30079)
Co-authored-by: wenshen <[email protected]>
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(relay): read migrations in checkout paths with spaces (pingdotgg#30080)
Co-authored-by: Dominik <[email protected]>
* fix(cli): validate npm runtime before caching it (pingdotgg#30099)
* fix(server): SSH device host no longer reconnects on a single slow health probe (pingdotgg#30159)
* fix(server): read continued core.bare values during onboarding (pingdotgg#30174)
Co-authored-by: maria-rcks <[email protected]>
* fix(server): published repos fall back to HTTPS when SSH is refused (pingdotgg#30142)
Co-authored-by: SAPHID <[email protected]>
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(opencode): fail stalled event subscriptions promptly (pingdotgg#30201)
* fix(server): a link card the user moved past no longer blocks auto-settle (pingdotgg#29847)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): deleting a thread also removes its agent preview recordings (pingdotgg#29853)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(server): review guides no longer fail on the repository's checkout hooks (pingdotgg#29907)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(desktop): bump Electron to 44.10.1 for October Chromium security backports (pingdotgg#30161)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* chore(repos): sync reference repos from shallow clones (pingdotgg#30472)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* chore(repos): sync Effect 4.1.2 and Alchemy beta.85, rename effect-smol (pingdotgg#30481)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* Revert "fix(web): terminal controls no longer cover the first line" (pingdotgg#30493)
* feat(web): show a newly created pairing link without expanding the client list (pingdotgg#30498)
Co-authored-by: Claude Opus 5.5 <[email protected]>
* fix(safe): remediate inherited v0.1.7 advisories
Apply age-qualified dependency fixes and refresh frozen stage projections. Keep the two remaining tooling advisories outside shipped production graphs and record the reviewed v0.1.7 anchor.
Co-Authored-By: GPT-6.1-Sol <[email protected]>
* fix(safe): hide denied triage instead of suggesting upstream command
Honor the baked remoteContent policy in the server capability and boot recovery. Remove the upstream CLI fallback from triage failure toasts, cover denied recovery, and correct the security inventory.
Co-Authored-By: OpenAI Codex <[email protected]>
* ci(safe): run macOS jobs on macos-15 because macos-14 runners are no longer provisioned
Hosted macos-14 jobs are cancelled before start with 'configuration macos-14-arm64-us-default is not supported'.
Co-Authored-By: Claude Sonnet 5.5 <[email protected]>
* chore(safe): refresh network baseline for macos-15 workflow labels
Co-Authored-By: Claude Sonnet 5.5 <[email protected]>
* chore(safe): approve reviewed v0.1.7 artifact host contexts
Reproduce the packaged CI host inventory from final-head bundles and installed production dependency text. Review the changed client contexts and mirror them into CLI scan paths; record conditional provider connector endpoints honestly.
Co-Authored-By: Codex <[email protected]>
---------
Signed-off-by: Yordis Prieto <[email protected]>
Co-authored-by: Andrei <[email protected]>
Co-authored-by: Claude Opus 5.5 <[email protected]>
Co-authored-by: Lakshmi Tanmay <[email protected]>
Co-authored-by: Alex Southwell <[email protected]>
Co-authored-by: maria-rcks <[email protected]>
Co-authored-by: shivam <[email protected]>
Co-authored-by: PR Batch Tester <[email protected]>
Co-authored-by: maria <[email protected]>
Co-authored-by: Julius Marminge <[email protected]>
Co-authored-by: Exotic209093 <[email protected]>
Co-authored-by: Issue triage verification <triage@localhost>
Co-authored-by: Theo Browne <[email protected]>
Co-authored-by: astarktc <[email protected]>
Co-authored-by: Utkarsh Patil <[email protected]>
Co-authored-by: Yash Singh <[email protected]>
Co-authored-by: Bilal Bakr <[email protected]>
Co-authored-by: Maria <[email protected]>
Co-authored-by: oliver <[email protected]>
Co-authored-by: Yordis Prieto <[email protected]>
Co-authored-by: Matthew Feroz <[email protected]>
Co-authored-by: Alex <[email protected]>
Co-authored-by: Alex Jankowiak <[email protected]>
Co-authored-by: Julius Marminge <[email protected]>
Co-authored-by: AdEx-Partners-DE <[email protected]>
Co-authored-by: Abhishek B R <[email protected]>
Co-authored-by: parmartejas <[email protected]>
Co-authored-by: Derek Trimm <[email protected]>
Co-authored-by: Tyler Jennings <[email protected]>
Co-authored-by: luke <[email protected]>
Co-authored-by: MarcoDotIO <[email protected]>
Co-authored-by: Hieu Le <[email protected]>
Co-authored-by: Trevor <[email protected]>
Co-authored-by: Trevor Edwards <[email protected]>
Co-authored-by: Johan Vaz <[email protected]>
Co-authored-by: u9g <[email protected]>
Co-authored-by: Taylor Dolezal <[email protected]>
Co-authored-by: Taylor Dolezal <[email protected]>
Co-authored-by: Romil <[email protected]>
Co-authored-by: Entrpi <[email protected]>
Co-authored-by: yaacovcorcos <[email protected]>
Co-authored-by: Chunhao Zhang <[email protected]>
Co-authored-by: levelorbit <[email protected]>
Co-authored-by: Advait Paliwal <[email protected]>
Co-authored-by: wenshen18643 <[email protected]>
Co-authored-by: wenshen <[email protected]>
Co-authored-by: domvox <[email protected]>
Co-authored-by: Dominik <[email protected]>
Co-authored-by: gyurix <[email protected]>
Co-authored-by: Harjot Singh <[email protected]>
Co-authored-by: SAPHID <[email protected]>
Co-authored-by: Valtrizt Khalifah Warri <[email protected]>
Co-authored-by: GPT-6.1-Sol <[email protected]>
Co-authored-by: OpenAI Codex <[email protected]>
wellguimaraes
pushed a commit
to wellguimaraes/floripa-code
that referenced
this pull request
Nov 10, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
size:XS0-9 changed lines (additions + deletions).vouch:trustedAuthor is trusted by repo permissions or the VOUCHED list.
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
In a GitHub, GitLab or Azure DevOps pull request, a renamed file shows up as an ordinary change under its new name. The diff gives no sign that it moved, and a rename with no edits appears as an empty change.
The server builds each forge's diff as a git-style patch with
rename fromandrename tolines, but leaves outsimilarity index.@pierre/diffsmarks a git-style section as a rename only when it sees that line. Without it, the parser sets the type tochangeand dropsprevName. Hunk expansion is affected too: the file contents loader has no old path, so it sends the new path for the old side. For GitHub this reads the new path at the merge base, where the file does not exist yet, and expanding context on a renamed and edited file fails.Change
Each builder now writes
similarity indexbefore the rename lines, like the GitCafe builder already does (added in #24276, with the comment that Pierre needs this header). None of these APIs gives a score, so the value is100%for a rename with no hunks and no withheld changes, and0%otherwise. Pierre reads100%asrename-pureand anything else asrename-changed.toFilesPatch: changed if it has hunks or a non-zero additions/deletions count.mergeRequestPatch: changed if it has hunks or istoo_large/collapsed.patchHeader: uses therename-pure/rename-changedkind Azure already reports.Scope and approval
Very small, focused fix for an obvious bug, so no prior issue: three forge builders leave out a header that their GitCafe sibling already writes for the same reason, and real git always writes it. This is one root cause, so all three builders are in one PR. No contract, client or default changes.
Overlap search: I searched the local PR index and GitHub for "similarity index" and for rename + diff/pull request. The only hit is #10822 (closed, unrelated). #27331 changes the Azure diff slice budget but not the rename header lines.
Verification
gitHubPullRequestJson.test.ts(2),gitLabMergeRequestJson.test.ts(1) andazureDevOpsDiff.test.ts(2), for exampleAssertionError: expected 'diff --git a/src/old.ts b/src/new.ts\…' to be 'diff --git a/src/old.ts b/src/new.ts\…', because thesimilarity indexline is missing.vp test runpasses 56/56 (GitHub), 44/44 (GitLab) and 14/14 (Azure DevOps), rerun after rebasing onto the latest main.vp run check:changedpasses: format, lint, knip, graphql, typecheck (13 packages), tests (4 files, 115 passed).@pierre/diffs:{"name":"src/new.ts","type":"change"}with noprevName.{"name":"src/new.ts","prevName":"src/old.ts","type":"rename-changed"}, and a rename with no hunks givesrename-pure.createDiffFileContentsLoader, not captured.Reviewed with sol-loop: 1 round with GPT-6.1 Sol on high. Sol verdict: approved.
Created with Claude Opus 5.5 in Claude Code (T3 Code).
🤖 Generated with Claude Code