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): remotes like github.com:owner/repo reach their host - #29457
Git reads github.com:owner/repo.git as an SSH remote and takes the user from ssh config, so this is a working remote for anyone whose ~/.ssh/config sets User git. T3 Code keyed the repository correctly, but only user@host:path counted as naming a host, so the remote got no source control provider.
Call path: parseGitRemoteHost (packages/source-control-core/src/shared/definition.ts) falls through to new URL("github.com:owner/repo.git"), which parses as a custom scheme with an empty host, and returns null. recognizeRemote then returns null, which has two effects:
RepositoryIdentityResolver.buildRepositoryIdentity leaves provider off the identity, and PullRequestService skips projects with no provider, so their pull requests never list.
SourceControlProviderRegistry.selectProviderContext finds no candidate, so resolveHandle returns the unknown provider and every call fails with "No unknown source control provider is registered."
The same repository on [email protected]:owner/repo.git works. The GitLab provider already had a local workaround with the comment "Remote host parsing does not recognize userless SCP remotes", and normalizeGitRemoteUrl and parseForgejoRemote already have tests for this form.
Change
packages/shared/src/sourceControl.ts gains userlessScpRemoteHost, and the places that needed user@ now also accept host:path:
parseGitRemoteHost, so recognizeRemote finds the host.
isSshRemoteUrl, so these remotes keep preferring SSH and SSH-only host handling applies, the same as their git@ spelling.
repositoryPathFromRemoteUrl, so the display name keeps its case and head-owner lookups find an owner.
parseGitCafeRemote, because GitCafe reads the repository from the remote itself.
The GitLab workaround is gone, since parseGitRemoteHost now does that job.
A remote on a host no forge claims, such as an ssh alias gh:owner/repo, now reads as unknown with its server's address, exactly as git@gh:owner/repo already did. It does not resolve the alias; that is #17805.
These still find no host: Windows drives (C:/repos/x), local paths (/srv/git/x, ./team:group/repo), file://, and remote helpers (hg::owner/repo, codecommit::us-east-1://repo).
Scope and approval
A small, focused fix of an obvious bug, with no prior issue. One underlying problem: one form of remote that git accepts was not recognized as naming a host. The code already acknowledged the gap in a comment, and nothing here adds behavior beyond making this form match its git@ spelling. No contract or client changes.
Searched open issues and PRs for this. Nearest are #17805 and #14919 (ssh aliases resolved with ssh -G), which address a different problem and touch RepositoryIdentityResolver rather than parseGitRemoteHost, and #22620 / #22625 (forge SSH hosts such as ssh.bitbucket.org).
Verification
Each focused test ran against the unfixed code first, then with the fix.
Before:
× reads the host of an SSH remote that leaves its user to ssh config: github.com:team/project.git
AssertionError: expected 'unknown' to equal 'github'
× reads the host of an SSH remote that leaves its user to ssh config: gitlab.com:team/project.git
AssertionError: expected 'unknown' to equal 'gitlab'
× reads the host of an SSH remote that leaves its user to ssh config: bitbucket.org:team/project.git
AssertionError: expected 'unknown' to equal 'bitbucket'
× reads the host of an SSH remote that leaves its user to ssh config: code.example.test:team/project.git
AssertionError: expected undefined to equal 'code.example.test:team/project.git'
× identifies the host of an SSH remote that leaves its user to ssh config
AssertionError: expected undefined to be 'github'
× reads github.com:owner/repo.git as SSH
× reads github-work:owner/repo.git as SSH
× reads the repository path of github.com:T3Tools/t3code.git
× reads the repository path of gitlab.com:T3Tools/platform/t3code
× reads the host and repository of git.cafe:team/project.git
The RepositoryIdentityResolver test uses a real git init and git remote add origin github.com:T3Tools/t3code.git. The cases that must stay hostless were already passing and still pass, so they guard against over-matching.
Neighbouring suites also pass: GitLab provider and discovery (15), GitHub provider and discovery (42), GitCafe provider (8), Forgejo provider (37), packages/sharedgit.test.ts (30). Targeted vp fmt, vp lint and tsc --noEmit are clean in every changed package, and the commit hook's knip check passed.
Not checked: a live forge. The real gh, glab and Bitbucket calls were not run against a remote of this form; the tests stop at the provider chosen for it, and the providers downstream read the host and path through helpers that already handle this form. No visible UI strings or layout changed, though projects on these remotes will now show pull requests where they showed none.
Git reads `host:path` as SSH and takes the user from ssh config, but only
`user@host:path` was read as naming a host. A checkout on such a remote got
no provider, so pull request sync skipped it and every provider call failed
as an unknown host.
Co-Authored-By: Claude Opus 5.5 <[email protected]>
Macroscope's review found this PR approvable — Userless SSH remotes such as github.com:owner/repo now reach the existing provider and retain SSH transport. The change is limited to remote recognition and routing, with existing host validation retained.
You can add or adjust custom eligibility rules. Learn more.
The changes add recognition for supported SCP-style Git remotes without a username. Shared parsing and provider handling use this recognition, and tests cover remote classification, repository paths, provider resolution, and repository identity.
A shared helper recognizes supported userless host:path remotes. SSH classification and host parsing use this helper. Tests cover supported and rejected remote formats.
Repository path parsing accepts userless SCP-style remotes. GitLab remote detection passes the original URL to host parsing instead of converting it first.
Provider and identity integration packages/source-control-gitcafe/src/server/*, apps/server/src/project/RepositoryIdentityResolver.test.ts, apps/server/src/sourceControl/*
GitCafe host parsing accepts remotes without a username, and its SSH detection uses the shared helper. Application tests cover provider resolution, remote retention, rejected formats, and repository identity.
Userless host:path remotes now get a provider and fork checkouts for them use the GitCafe ssh@ URL. That could matter for someone whose SSH config selects an identity by user, but this is unconfirmed and limited to GitCafe fork checkouts. The change is mergeable with that caveat in mind.
Pre-merge checks: 4
Check name
Status
Explanation
Title check
The title names the main change: remotes written as host:path, such as github.com:owner/repo, are now resolved to their host. It follows the conventional commit format. The changeset adds userlessScpR…
Description check
The description follows the template. It has Problem, Change, Scope and approval, and Verification sections. The problem, the call path, and the affected behavior are explained. The change lists the m…
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.
Full details: Title check
Explanation
The title names the main change: remotes written as host:path, such as github.com:owner/repo, are now resolved to their host. It follows the conventional commit format. The changeset adds userlessScpRemoteHost and uses it in host parsing, SSH detection, repository path parsing, and GitCafe parsing, which matches the title.
Full details: Description check
Explanation
The description follows the template. It has Problem, Change, Scope and approval, and Verification sections. The problem, the call path, and the affected behavior are explained. The change lists the modified functions and the forms that still stay hostless. Scope names the small-fix exemption, cites related open issues and PRs, and gives reasons. Verification shows before and after test output, the neighboring suites that ran, and states what was not checked, which is the live forge calls. The description does not include a template-required note about the CONTRIBUTING.md read, but this is a minor, non-critical item. It also does not state a maintainer-approval link. The author explains why the small-fix exemption applies, which the template allows.
Finishing Touches
Generate unit tests (beta)
Create a new PR
Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts
481-481: Major · Functional Correctness · Quick win
Unverified finding
Verification ran but could not confirm this finding. It is shown for review, not as a verified issue.
Preserve the configured SSH user for fork checkouts.
For a userless git.cafe:team/project remote, the new SSH check selects this ssh@ URL when checkout needs a fork remote. That overrides the configured user. If SSH config selects the identity with Match user, the fork fetch can fail even though the original remote works. Preserve the userless form for userless remotes. (man.openbsd.org)
Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 8 remain after this review.
Declining the unverified CodeRabbit note on GitCafeSourceControlProvider.ts:481 (preserve the userless form for fork checkouts). GitCafe fork remotes hard-code ssh@, the same user cloneUrls advertises, and every other SSH spelling of the remote already takes this path. A userless remote that works must already resolve to that user through ssh config, so ssh@ selects the same user and a Match user rule still applies. Keeping a separate userless form would add a special case for a failure that does not arise.
verified as-is at 73a81531d1b018c5f553b713e29f4dc5a3186b18 on a linux boat vm. installed both revisions with vp i and ran one background dev server at a time. used repo playwright with /usr/bin/google-chrome, isolated app state, app-created projects and a real codex gpt-6.1-sol turn. no mocks or demo data.
reproduced on github/main (30b9c4a956): a real repository with only origin=github.com:pingdotgg/t3code.git had an empty project-filtered pull-request list. authenticated git.resolvePullRequest failed with No unknown source control provider is registered.
neighboring server checks passed for the explicit-user github ssh remote, https://github.com/pingdotgg/t3code.git, and userless git.cafe:pingdotgg/t3code.git, each resolving this pr through its real forge provider.
the real codex turn completed and reported the userless origin and clean working tree.
blacksmith vp run check:changed c8a601c0dfa5c7626e51e06d727076f6b840a005 passed: 82 tests across 10 files, format, lint, knip, graphql and affected-package typechecks. existing ci reports 30/30 successful. no source changes needed.
ssh network fetch/push and git.cafe fork checkout: this vm has no ssh private key. live gitlab and bitbucket calls: no credentials. native desktop, ios/android, macos/windows, and relay/tunnel connections were not exercised. the web client and local running-server paths above were exercised.
…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]>
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:S10-29 changed lines (additions + deletions).vouch:unvouchedAuthor is not yet trusted in 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
Git reads
github.com:owner/repo.gitas an SSH remote and takes the user from ssh config, so this is a working remote for anyone whose~/.ssh/configsetsUser git. T3 Code keyed the repository correctly, but onlyuser@host:pathcounted as naming a host, so the remote got no source control provider.Call path:
parseGitRemoteHost(packages/source-control-core/src/shared/definition.ts) falls through tonew URL("github.com:owner/repo.git"), which parses as a custom scheme with an empty host, and returns null.recognizeRemotethen returns null, which has two effects:RepositoryIdentityResolver.buildRepositoryIdentityleavesprovideroff the identity, andPullRequestServiceskips projects with no provider, so their pull requests never list.SourceControlProviderRegistry.selectProviderContextfinds no candidate, soresolveHandlereturns theunknownprovider and every call fails with "No unknown source control provider is registered."The same repository on
[email protected]:owner/repo.gitworks. The GitLab provider already had a local workaround with the comment "Remote host parsing does not recognize userless SCP remotes", andnormalizeGitRemoteUrlandparseForgejoRemotealready have tests for this form.Change
packages/shared/src/sourceControl.tsgainsuserlessScpRemoteHost, and the places that neededuser@now also accepthost:path:parseGitRemoteHost, sorecognizeRemotefinds the host.isSshRemoteUrl, so these remotes keep preferring SSH and SSH-only host handling applies, the same as theirgit@spelling.repositoryPathFromRemoteUrl, so the display name keeps its case and head-owner lookups find an owner.parseGitCafeRemote, because GitCafe reads the repository from the remote itself.The GitLab workaround is gone, since
parseGitRemoteHostnow does that job.A remote on a host no forge claims, such as an ssh alias
gh:owner/repo, now reads asunknownwith its server's address, exactly asgit@gh:owner/repoalready did. It does not resolve the alias; that is #17805.These still find no host: Windows drives (
C:/repos/x), local paths (/srv/git/x,./team:group/repo),file://, and remote helpers (hg::owner/repo,codecommit::us-east-1://repo).Scope and approval
A small, focused fix of an obvious bug, with no prior issue. One underlying problem: one form of remote that git accepts was not recognized as naming a host. The code already acknowledged the gap in a comment, and nothing here adds behavior beyond making this form match its
git@spelling. No contract or client changes.Searched open issues and PRs for this. Nearest are #17805 and #14919 (ssh aliases resolved with
ssh -G), which address a different problem and touchRepositoryIdentityResolverrather thanparseGitRemoteHost, and #22620 / #22625 (forge SSH hosts such asssh.bitbucket.org).Verification
Each focused test ran against the unfixed code first, then with the fix.
Before:
After:
The
RepositoryIdentityResolvertest uses a realgit initandgit remote add origin github.com:T3Tools/t3code.git. The cases that must stay hostless were already passing and still pass, so they guard against over-matching.Neighbouring suites also pass: GitLab provider and discovery (15), GitHub provider and discovery (42), GitCafe provider (8), Forgejo provider (37),
packages/sharedgit.test.ts(30). Targetedvp fmt,vp lintandtsc --noEmitare clean in every changed package, and the commit hook's knip check passed.Not checked: a live forge. The real
gh,glaband Bitbucket calls were not run against a remote of this form; the tests stop at the provider chosen for it, and the providers downstream read the host and path through helpers that already handle this form. No visible UI strings or layout changed, though projects on these remotes will now show pull requests where they showed none.Claude Opus 5.5 (Claude Code desktop app)
🤖 Generated with Claude Code