[saphid]

Diff viewing depended on a built-in panel. This adds an installable Diff extension using typed diff and host code-view APIs.

It supports changed-file navigation, split and unified views, expandable file content, comments and composer context. Parsed paths preserve real a/ and b/ directories and decoded quoted names through full-content loading, fold/reveal, Open and Copy path actions.

Review

Depends on #13822. Review this layer’s last commit; the comparison against main includes its dependencies.

Verification

Check	First run	Final
CI=true vp i (reuse when lockfile unchanged)	pass	pass
SDK and runtime builds	pass	pass
vp lint	pass	pass
vp fmt --check	pass	pass
git merge-tree against fetched main	pass	pass
vp run knip:check	pass	pass
server.test.ts and RpcAuthorization.test.ts	pass	pass
extension-runtime tests	pass	pass
web/server/desktop and touched workspace typechecks	pass	pass
Effect import subpath case check	pass	pass
release smoke	pass	pass
apps/server suite	local HTTP port collision (rc 1)	local HTTP port collision (rc 1)
sdk build	pass	pass
build	pass	pass
check	pass	pass
test	pass	pass
typecheck	pass	pass
audit	pass	pass
web test	pass	pass
bundle freshness (integrated version; generated bundles excluded from these PRs)	N/A	N/A; integrated version checked separately

The only failing results are server HTTP tests on the author's machine: an unrelated local service occupies the same port number on IPv4, causing bootstrap requests to receive that service's 403 or 404 responses. The same failures occur on unmodified main, and the same suite passes in full on layers containing all server changes, including Terminal. CI on Linux is the authority for the server suite.

The Default host Diff renderer uses native parsed-path resolvers, preserving real a/ and b/ directories and decoded rename paths. The regression fails before the fix and passes afterward. The host regression belongs to Web; the pack-dependent path-identity tests and fixture belong to Diff.

The final integrated extension rebuild and bundle-freshness check passed. Generated bundles are excluded from the review layers and built from source there.

Earlier integration testing used real web and Electron desktop clients and real GitHub workflows. Desktop testing verified Browser menus, picture-in-picture and pressed-state styling; web testing verified Diff gap expansion, folding/reveal and Open/Copy actions for all four path cases, the Version Control workflow and Files remote editor routing. Agents testing verified launch, script viewing and opening a seeded session. This evidence covers the integrated implementation; these individual PR branches have not had a fresh real-client pass.

Testing against local Forgejo verified both native and extension reviewer pickers in the web client: candidates appeared once without repeated listing, user and team assignment/removal succeeded through the picker, and disabled-state reasons appeared on hover and keyboard focus. GitLab workflows remain unverified in a real client, and mobile extension support remains follow-up work.

Standalone path handling was verified in real desktop and web clients.

Current parity assessment: 182 PARITY / 0 SHIPPED-UNVERIFIED / 0 GAP / 2 DEFERRED of 184 scored rows, plus one N/A.

GPT-6.1 Sol via T3 Code

[juliusmarminge]

Thanks for working on this. We merged the orchestrator V2 rewrite in #2829, and we are closing this PR as part of that transition.

This extension layer includes the pre-V2 extension host dependency. EnvironmentExtensions and agentSessionsApi depend on deleted ProjectionSnapshotQuery, OrchestrationEngineService and ProviderSessionDirectory; provider credential wiring modifies deleted ProviderService. Rebuild the host on V2 and recreate this dependent layer.

Sorry for the extra work this creates. If the change is still needed on V2, please rebuild it on current main, verify it there, and open a new PR linking back here. We're closing the current implementation without assuming the underlying request is resolved.