refactor(provider-acp-registry): move the ACP Registry into its own package by juliusmarminge · Pull Request #17405 · pingdotgg/t3code

juliusmarminge · 2026-10-09T05:17:37Z

The ACP Registry was the last ACP flavor I could move without settling the model-manifest question (Antigravity is waiting on that). This moves it into @t3tools/provider-acp-registry, built on @t3tools/provider-acp.

Package

./settings: AcpRegistrySettings and AcpRegistryDistributionPreference, moved out of contracts.
./client: label, settings form, hasDefaultInstance: false, and the registry glyph clients show while an agent's own icon loads. Web and mobile both register it.
./server: AcpRegistryDriver, AcpRegistryAdapterV2Driver, and acpRegistryManagedBinaryDirectories, which the terminal manager uses to keep managed agents off PATH.
./server/AcpRegistrySupport and ./server/AcpRegistryRuntimeCoordinator: the catalog and coordinator services. Each has its own subpath and is imported as a namespace. AcpRegistrySupport.layerFromHost builds both from ProviderHost paths, which replaces the server's AcpRegistryCatalog.ts. Layers are top-level exports reached through the namespace, not static methods on the service class; a second commit makes the same change for AcpSessionRuntime and OpenCodeRuntimeError.is.
./testing: the adapter, probe, and Devin helpers the server's ACP tests use.

Boundaries

The package reaches the server only through ProviderHost. Host settings gain withSnapshot, which the catalog uses to check whether an installed agent is still referenced before it deletes the agent. It's backed by ServerSettingsService.withSettingsSnapshot, so the check can't race a settings write.
layerTestProviderHost now also provides TestProviderHostSettings, so tests can change settings mid-test. The catalog's uninstall tests use it in place of the real settings service.
Two generic pieces moved to shared homes: PtyAdapter to @t3tools/shared, and ProviderAuthFlow to provider-core.
The probe test stays in the server because it drives the server's ACP mock agent script.
The package's tsconfig uses the server's lib (no DOM), which matches its Node runtime.
@noble/hashes moves from the server to this package, its only user.

Verification

Typecheck: provider-acp-registry, provider-core, provider-testing, shared, contracts, server, web, mobile.
Tests: the package, plus the server's Registry adapter and probe, ACP adapter, terminal, provider registry, Codex ChatGPT auth, adapter registry, replay fixtures, and web settings and provider-icon tests. 1293 passed before the rebase. After rebasing onto current main I reran the Registry, terminal, and provider-registry tests, and those 204 passed.
vp lint and knip pass on the changed workspaces.

Done by Claude Opus 5.5 in Claude Code.

🤖 Generated with Claude Code

…ackage The ACP Registry moves into @t3tools/provider-acp-registry, built on @t3tools/provider-acp: - ./settings: AcpRegistrySettings and the distribution preference, moved out of contracts. - ./client: label, settings form, no default instance, and the registry glyph clients fall back to while an agent's own icon loads. - ./server: the driver, the adapter driver, and the managed-binary directories the terminal manager keeps off agent PATHs. - ./server/AcpRegistrySupport and ./server/AcpRegistryRuntimeCoordinator: the catalog and coordinator services, imported as namespaces. AcpRegistryCatalog.layerFromHost builds both from ProviderHost paths. - ./testing: adapter, probe, and Devin helpers the server's ACP tests use. The package reaches the server only through ProviderHost. Its settings gain withSnapshot, which the catalog uses to check whether an installed agent is still referenced before removing it. The test host's settings can now change during a test (TestProviderHostSettings). Two generic pieces move to shared packages: PtyAdapter to @t3tools/shared and ProviderAuthFlow to provider-core. Co-Authored-By: Claude Opus 5.5 <[email protected]>

macroscopeapp · 2026-10-09T05:22:59Z

Approvability

Verdict: Not approved

Macroscope's review found this PR not approvable — This is a broad ACP Registry package extraction that also changes shared ProviderHost/service wiring, client registration, and authentication/session integration. The implementation is largely moved rather than rewritten, but the cross-package runtime and sensitive auth surface warrant focused human review.

^{You can add or adjust custom eligibility rules. Learn more.}

github-actions · 2026-10-09T05:23:56Z

Thread transfer impact

✅ Thread transfer remains within every enforced ceiling.

Provider	Metric	Main baseline	This PR	Impact	PR ceiling
Codex	Total thread wire	4.9 KiB	4.9 KiB	0 B (0.0%)	6.8 KiB	✅
Codex	Thread snapshot wire	3.8 KiB	3.8 KiB	0 B (0.0%)	4.9 KiB	✅
Codex	Live turn WebSocket wire	1.2 KiB	1.2 KiB	0 B (0.0%)	2.0 KiB	✅
Codex	Live turn WebSocket decoded	20.8 KiB	20.8 KiB	0 B (0.0%)	29.3 KiB	✅
Codex	Live turn messages	1	1	0 (0.0%)	8	✅
Claude	Total thread wire	5.0 KiB	5.0 KiB	+24 B (+0.5%)	6.8 KiB	✅
Claude	Thread snapshot wire	3.8 KiB	3.8 KiB	0 B (0.0%)	4.9 KiB	✅
Claude	Live turn WebSocket wire	1.2 KiB	1.2 KiB	+24 B (+2.0%)	2.0 KiB	✅
Claude	Live turn WebSocket decoded	21.2 KiB	21.2 KiB	+41 B (+0.2%)	29.3 KiB	✅
Claude	Live turn messages	1	2	+1 (+100.0%)	8	✅

Baseline: 563645c · PR result: 3770e65 · Source CI: success

Scenario and decoded snapshot size

10 historical turns, 5 command tools per turn, 878.9 KiB retained MCP result per historical turn, and a 1.05 MiB retained result in the measured turn.

Codex decoded thread snapshot: 108.5 KiB
Claude decoded thread snapshot: 108.8 KiB

Updated in place by a trusted workflow. PR artifacts are strictly validated and never executed.

coderabbitai · 2026-10-09T05:28:33Z

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.config.ts
Review profile: CHILL
Plan: Team
Run ID: 1d7c0eb8-34c0-470b-aa3d-5031f8f6395b

📥 Commits

Reviewing files that changed from the base of the PR and between 8f8d18f and 3770e65.

📒 Files selected for processing (8)

apps/server/src/provider/ProviderInstanceRegistryHydration.ts
apps/server/src/server.ts
packages/provider-acp-registry/src/server/AcpRegistryRuntimeCoordinator.test.ts
packages/provider-acp-registry/src/server/AcpRegistryRuntimeCoordinator.ts
packages/provider-acp-registry/src/server/AcpRegistrySupport.ts
packages/provider-acp/src/server/AcpSessionRuntime.ts
packages/provider-opencode/src/server/OpenCodeRuntime.ts
packages/provider-opencode/src/server/v2/OpenCode2Server.ts

Included review availability: This review used your included allowance. Your plan provides up to 10 included reviews per hour; 8 remain after this review.

📝 Walkthrough

Walkthrough

The ACP Registry provider code and settings schema now live in a dedicated workspace package with client, server, and testing entry points. Mobile, web, and server import the package. Server catalog construction uses ProviderHost settings snapshots, and shared ProviderAuthFlow and PtyAdapter APIs have package exports.

Changes

ACP Registry package extraction

Layer / File(s)	Summary
Package settings and public APIs `packages/provider-acp-registry/*`, `packages/provider-core/package.json`, `packages/provider-core/src/server/ProviderAuthFlow.ts`, `packages/shared/package.json`, `packages/shared/src/PtyAdapter.ts`, `packages/contracts/src/settings.ts`	The new package defines the ACP Registry settings schema and client, server, and testing entry points. The schema is removed from contracts. Provider core and shared packages expose ProviderAuthFlow and PtyAdapter APIs.
ProviderHost catalog integration `packages/provider-core/src/server/ProviderHost.ts`, `packages/provider-core/src/server/managedProvider.test.ts`, `packages/provider-testing/src/host.ts`, `packages/provider-acp-registry/src/server/AcpRegistrySupport.ts`, `packages/provider-acp-registry/src/server/AcpRegistrySupport.test.ts`, `apps/server/src/provider/*`, `apps/server/src/server.ts`	ProviderHost adds stable settings snapshots, and the test host adds mutable settings support. The ACP Registry catalog reads settings and host paths through ProviderHost. Server startup and registry hydration use the package catalog layer.
ACP Registry implementation and application wiring `packages/provider-acp-registry/src/server/`, `apps/mobile/`, `apps/web/`, `apps/server/src/orchestration-v2/`, `apps/server/src/provider/`, `apps/server/src/terminal/`, `apps/server/src/ws.ts`, `packages/shared/src/PtyAdapter.ts`, `package.json`, `scripts/release-smoke.ts`	ACP Registry server modules and tests use package-local settings and imports. Mobile and web register the package client. Server code imports package drivers, support, adapter helpers, and shared PtyAdapter. Workspace checks and the release smoke fixture include the package.

Provider runtime API updates

Layer / File(s)	Summary
OpenCode runtime error guard `packages/provider-opencode/src/server/OpenCodeRuntime.ts`, `packages/provider-opencode/src/server/v2/OpenCode2Server.ts`	OpenCode replaces the static runtime-error predicate with an exported type guard. Runtime error handling and server verification use the new guard.
ACP session runtime layer API `packages/provider-acp/src/server/AcpSessionRuntime.ts`	The ACP session runtime class no longer declares its static layer factory.

Priority: ➖ Normal

Estimated code review effort: 3 (Moderate) | ~25 minutes

Change: Refactor

Merge Risk: ⚪ Minimal · up to 3770e

No actionable merge-blocking issue is identified; the inspected catalog wiring supports merging after normal checks.

🚥 Pre-merge checks | ✅ 3 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Description check		The description explains the package structure, architectural boundaries, and verification results. It does not provide the required Scope and approval information, and it does not explicitly separate…	Add a Scope and approval section with a linked issue or discussion and explicit maintainer approval, or explain why this focused refactor qualifies for an exemption. Add explicit Problem and Change sections that state the underlying problem…

✅ Passed checks (3 passed)

Check name	Status	Explanation
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.
Title check		The title clearly and concisely describes the primary change: moving the ACP Registry into its own package.

Full details: Description check

Explanation

The description explains the package structure, architectural boundaries, and verification results. It does not provide the required Scope and approval information, and it does not explicitly separate the Problem and Change sections.

Resolution

Add a Scope and approval section with a linked issue or discussion and explicit maintainer approval, or explain why this focused refactor qualifies for an exemption. Add explicit Problem and Change sections that state the underlying problem, expected behavior, and how the refactor addresses it. Keep the existing verification details and state any checks that remain incomplete after rebasing if applicable.

Fix all pre-merge checks with AI

✨ Finishing Touches

📝 Generate docstrings

Commit to this branch
Create a new PR

🧪 Generate unit tests (beta)

Commit to this branch
Create a new PR

Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

_{Comment @coderabbitai help to get the list of available commands.}

Service modules expose their layers as top-level exports reached through the namespace import (`AcpRegistrySupport.layerFromHost`, `AcpRegistryRuntimeCoordinator.layer`), not as static members on the service class. AcpSessionRuntime already exported `layer`, so its duplicate static goes; OpenCodeRuntimeError.is becomes isOpenCodeRuntimeError. Co-Authored-By: Claude Opus 5.5 <[email protected]>

Brings in upstream through pingdotgg#17405 (43f8a8d). - Adapted: Windows work-account sign-in in the server browser now runs after upstream's presentAsChrome (pingdotgg#16963) on each headless tab. ServerBrowser.test.ts checks that persistent tabs pause Entra sign-in and incognito tabs do not, and is now in personal-fixes. - Adapted: since pingdotgg#17316 the desktop runs tabs Ved opens for a remote environment locally, so the desktop's sign-in path carries them again. BrowserSession.test.ts checks that persistent sessions get the sign-in hook. browser-import.md says which account each tab uses. - BrowserSession.ts: kept the sign-in hook beside upstream's external-protocol prompt (pingdotgg#16961). - Remote open: kept Cody's Tailscale SSH login (user@host) for VS Code and Zed beside upstream's JetBrains Toolbox links (pingdotgg#17271). JetBrains links carry no login, as upstream designed. - DesktopAppIdentity: took upstream's RFC 9110 runtime name (pingdotgg#17264), so Cody's runtime name is "Cody <stage>". The About panel still says "Cody". - OpenCode package split (pingdotgg#17345): the Copilot usage limits, MCP re-add, and final-answer instruction patches moved with their files. The driver calls readOpenCodeUsageLimits from the new package. - personal-fixes.json: points the OpenCode tests at their new package paths and adds runtimeInstructions.test.ts. Drops opencodeUsageReader.test.ts, a path that never existed. - Superseded: none this round.

Upstream pingdotgg#17405 turned the factory into an Effect.

* fix(web): link pull requests to threads in folders that aren't Git repos (pingdotgg#15946) Co-authored-by: PR Batch Tester <[email protected]> Co-authored-by: Claude Opus 5.5 (1M context) <[email protected]> * feat(web): find messages and plans in the current thread (pingdotgg#10439) Co-authored-by: Yash Singh <[email protected]> Co-authored-by: Julius Marminge <[email protected]> Co-authored-by: Claude Opus 5.5 (1M context) <[email protected]> * feat(web): improve terminal scrollback navigation and snapshots (pingdotgg#17091) * docs(internals): add a checklist for adding a provider (pingdotgg#17229) Co-authored-by: Claude Opus 5.5 (1M context) <[email protected]> * fix(mobile): keep native screens ordered during stack pops (pingdotgg#17231) * fix(server): pairing tokens work on Node versions that cannot bind booleans (pingdotgg#16730) * fix(mobile): HTML pages in a thread no longer trap scrolling on Android (pingdotgg#17211) Co-authored-by: Claude Opus 5.5 <[email protected]> * fix(web): centered scrollers no longer shift when the scrollbar appears (pingdotgg#17077) Co-authored-by: maria-rcks <[email protected]> * fix(web): distinguish thread search matches from code tints (pingdotgg#17263) * fix(server): Pi extension wakes get an owned continuation turn (pingdotgg#17214) Co-authored-by: Claude Opus 5.5 <[email protected]> * fix(server): Pi discovers optional T3 tools on demand (pingdotgg#17220) * fix(web): stack merge dialog closes as soon as you confirm (pingdotgg#17116) * fix(server): Pi editor dialogs prefill the answer composer (pingdotgg#17206) * fix(desktop): generate valid User-Agent that follows RFC 9110 guidelines (pingdotgg#17264) * fix(server): Pi discovers workspace skills and commands (pingdotgg#17190) * fix(mobile): preserve navigation after native swipe back (pingdotgg#17268) * fix(server): keep newly discovered models out of legacy groups (pingdotgg#14314) Co-authored-by: PR Batch Tester <[email protected]> Co-authored-by: Claude Opus 5.5 (1M context) <[email protected]> * feat(editors): open remote projects in JetBrains IDEs over SSH (pingdotgg#17271) Co-authored-by: Claude Opus 5.5 <[email protected]> * test(desktop): expect JetBrains IDEs among remote editors (pingdotgg#17291) Co-authored-by: Claude Opus 5.5 <[email protected]> * fix(server): recognize authenticated GitHub Enterprise hosts (pingdotgg#11059) Co-authored-by: Claude Code <[email protected]> Co-authored-by: Julius Marminge <[email protected]> * fix(connect): relay client updates itself and skips incompatible cloudflared (pingdotgg#17275) Co-authored-by: Claude Opus 5.5 <[email protected]> * fix(shared): relay client install waits out a brief Windows file lock (pingdotgg#16998) Co-authored-by: Claude Opus 5.5 (1M context) <[email protected]> * fix(shared): release relay install locks on cancellation (pingdotgg#10585) Co-authored-by: Claude Opus 5.5 <[email protected]> * chore(shared): bump managed cloudflared to 2026.10.0 (pingdotgg#11184) Co-authored-by: Claude Opus 5.5 <[email protected]> * fix(shared): bound cloudflared download with 10-minute timeout (pingdotgg#14139) Co-authored-by: Kevin Rajan <[email protected]> * refactor(provider-core): add provider-core and provider-testing packages (pingdotgg#17299) Co-authored-by: Claude Opus 5.5 <[email protected]> * refactor(settings): drop the legacy per-driver providers map (pingdotgg#17300) Co-authored-by: Claude Opus 5.5 <[email protected]> * refactor(provider-pi): move Pi into its own provider package (pingdotgg#17302) Co-authored-by: Claude Opus 5.5 <[email protected]> * feat(models): tell users when a CLI update unlocks a new model (pingdotgg#17307) Co-authored-by: Claude Opus 5.5 <[email protected]> * fix(web): collapsed composer reserves room for wide send actions (pingdotgg#17016) * fix(muse): workflow subagents no longer stall on hidden approvals (pingdotgg#17329) Co-authored-by: Claude Opus 5.5 <[email protected]> * refactor(provider-core): share attachment prompts, notifications, and event loggers (pingdotgg#17330) Co-authored-by: Claude Opus 5.5 <[email protected]> * fix(web): file previews handle downloads, in-page links, and repo paths, and favicons stop leaking internal hosts (pingdotgg#16950) Co-authored-by: Claude Opus 5.5 (1M context) <[email protected]> * fix(server): environment-hosted browser tabs behave like a normal browser (pingdotgg#16963) Co-authored-by: Claude Opus 5.5 (1M context) <[email protected]> * fix(desktop): browser tab fixes for fullscreen, shortcuts, links, reload and hidden tabs (pingdotgg#16961) Co-authored-by: Claude Opus 5.5 (1M context) <[email protected]> * fix(web): desktop opens remote environments' browser tabs locally (pingdotgg#17316) Co-authored-by: Claude Opus 5.5 <[email protected]> * fix(desktop): the t3 command warns instead of installing behind another t3 (pingdotgg#17351) Co-authored-by: Claude Opus 5.5 <[email protected]> * fix(web): images, video, HTML and PDF preview in a thread before its first message (pingdotgg#17352) Co-authored-by: Claude Opus 5.5 <[email protected]> * refactor(provider-muse): move Muse Code into its own provider package (pingdotgg#17331) Co-authored-by: Claude Opus 5.5 <[email protected]> * fix(web): semantic branch naming hint lines up with its setting (pingdotgg#16972) * fix(mobile): restore chat image previews in the v5 stack (pingdotgg#17361) * feat(mobile): fade working threads and match web's status labels (pingdotgg#17368) Co-authored-by: Claude Opus 5.5 <[email protected]> * fix(server): agent browser tools stop bloating history, fall back sensibly, and respect ownership (pingdotgg#16956) Co-authored-by: Claude Opus 5.5 (1M context) <[email protected]> * fix(web): add room for thread timeline markers (pingdotgg#17372) * fix(web): drop sidebar context before cancelling pointer drag (pingdotgg#17373) * refactor(providers): namespace-import service modules in core, Muse, Pi, and testing (pingdotgg#17375) Co-authored-by: Claude Opus 5.5 <[email protected]> * fix(auth): show connection permissions and enforce session lifetime (pingdotgg#17370) Co-authored-by: Bear Huddleston <[email protected]> * refactor(provider-opencode): move OpenCode into its own provider package (pingdotgg#17345) Co-authored-by: Claude Opus 5.5 <[email protected]> * refactor(provider-cursor): move Cursor into its own provider package (pingdotgg#17349) Co-authored-by: Claude Opus 5.5 <[email protected]> * refactor(provider-acp): move the shared ACP adapter into its own package (pingdotgg#17354) Co-authored-by: Claude Opus 5.5 <[email protected]> * refactor(provider-grok): move Grok into its own provider package (pingdotgg#17357) Co-authored-by: Claude Opus 5.5 <[email protected]> * fix(server): speed up long thread message sync (pingdotgg#17387) * fix(desktop): cancel backend pipe reads to avoid slow shutdown (pingdotgg#17386) * refactor(providers): adapter factories yield their services (pingdotgg#17381) * fix(web): show a row spinner instead of a banner when expanding a folder (pingdotgg#17378) * fix(server): a timed-out browser drag no longer exits the server (pingdotgg#17360) Co-authored-by: Claude Opus 5.5 <[email protected]> * fix(server): a logged-out Claude CLI no longer reports as authenticated (pingdotgg#15459) * fix(server): Pi loads every selected skill without losing prompt text (pingdotgg#17194) * fix(server): keep the Claude MCP token out of process arguments (pingdotgg#17408) * fix(server): reconcile Pi native session rewinds (pingdotgg#13839) Co-authored-by: Claude Opus 5.5 (1M context) <[email protected]> * test(provider-pi): cover continuation offers through the driver (pingdotgg#17407) * refactor(provider-acp-registry): move the ACP Registry into its own package (pingdotgg#17405) * fix(server): relay client updates no longer drop the host off T3 Connect (pingdotgg#17366) Co-authored-by: Claude Opus 5.5 <[email protected]> --------- Co-authored-by: shivam <[email protected]> Co-authored-by: PR Batch Tester <[email protected]> Co-authored-by: Claude Opus 5.5 (1M context) <[email protected]> Co-authored-by: Jono Kemball <[email protected]> Co-authored-by: Yash Singh <[email protected]> Co-authored-by: Julius Marminge <[email protected]> Co-authored-by: Wout Stiens <[email protected]> Co-authored-by: Julius Marminge <[email protected]> Co-authored-by: chise <[email protected]> Co-authored-by: Dara Adedeji <[email protected]> Co-authored-by: maria <[email protected]> Co-authored-by: maria-rcks <[email protected]> Co-authored-by: oliver <[email protected]> Co-authored-by: jztmanyl <[email protected]> Co-authored-by: Bilal Bakr <[email protected]> Co-authored-by: Grzegorz Mandziak <[email protected]> Co-authored-by: Scott Norteman <[email protected]> Co-authored-by: Aditya Garud <[email protected]> Co-authored-by: Aaron Queen <[email protected]> Co-authored-by: Kevin Rajan <[email protected]> Co-authored-by: Kevin Rajan <[email protected]> Co-authored-by: Theo Browne <[email protected]> Co-authored-by: Daniel Alvim <[email protected]> Co-authored-by: Bear Huddleston <[email protected]> Co-authored-by: Yordis Prieto <[email protected]>

@juliusmarminge

## What's Changed * fix(server): keep the Claude MCP token out of process arguments by @juliusmarminge in pingdotgg/t3code#17408 * fix(server): reconcile Pi native session rewinds by @StiensWout in pingdotgg/t3code#13839 * test(provider-pi): cover continuation offers through the driver by @juliusmarminge in pingdotgg/t3code#17407 * refactor(provider-acp-registry): move the ACP Registry into its own package by @juliusmarminge in pingdotgg/t3code#17405 * fix(server): relay client updates no longer drop the host off T3 Connect by @t3dotgg in pingdotgg/t3code#17366 * fix(connect): Cloudflare and other VPN addresses no longer show as Tailscale by @shivamhwp in pingdotgg/t3code#17158 **Full Changelog**: pingdotgg/t3code@v0.0.46-nightly.20261009.2861...v0.0.46-nightly.20261009.2873 Upstream release: https://github.com/pingdotgg/t3code/releases/tag/v0.0.46-nightly.20261009.2873

@juliusmarminge

## What's Changed * fix(server): keep the Claude MCP token out of process arguments by @juliusmarminge in pingdotgg/t3code#17408 * fix(server): reconcile Pi native session rewinds by @StiensWout in pingdotgg/t3code#13839 * test(provider-pi): cover continuation offers through the driver by @juliusmarminge in pingdotgg/t3code#17407 * refactor(provider-acp-registry): move the ACP Registry into its own package by @juliusmarminge in pingdotgg/t3code#17405 * fix(server): relay client updates no longer drop the host off T3 Connect by @t3dotgg in pingdotgg/t3code#17366 * fix(connect): Cloudflare and other VPN addresses no longer show as Tailscale by @shivamhwp in pingdotgg/t3code#17158 **Full Changelog**: pingdotgg/t3code@v0.0.46-nightly.20261009.2861...v0.0.46-nightly.20261009.2873 Upstream release: https://github.com/pingdotgg/t3code/releases/tag/v0.0.46-nightly.20261009.2873

…ackage (pingdotgg#17405) Fork adaptation: Service keys moved into provider-acp-registry and shared take those packages' names. (cherry picked from commit 43f8a8d) Upstream-PR: 17405

github-actions Bot added vouch:trusted Author is trusted by repo permissions or the VOUCHED list. size:XL 500-999 changed lines (additions + deletions). labels Oct 9, 2026

juliusmarminge force-pushed the t3/provider-acp-registry branch from ce42275 to 8f8d18f Compare October 9, 2026 05:20

juliusmarminge changed the title ~~t3/provider acp registry~~ refactor(provider-acp-registry): move the ACP Registry into its own package Oct 9, 2026

juliusmarminge marked this pull request as ready for review October 9, 2026 05:20

juliusmarminge added the macroscope-review Opt PRs made by unvouched contributors in for Macroscope review. Vouched contributors auto-reviews label Oct 9, 2026

juliusmarminge merged commit 43f8a8d into main Oct 9, 2026
30 of 31 checks passed

juliusmarminge deleted the t3/provider-acp-registry branch October 9, 2026 06:08

kalvenschraut mentioned this pull request Oct 9, 2026

chore(rtvision): sync upstream and prepare 0.0.76 RTVision/t3code#85

Merged

Mnigos added a commit to Mnigos/t3code that referenced this pull request Oct 9, 2026

test(acp): registry compaction tests use the Effect adapter factory

3fed233

Upstream pingdotgg#17405 turned the factory into an Effect.

olafura mentioned this pull request Nov 7, 2026

test(features): ledger upstream T3 Code through 99544c9c19 hal-c2/hal-c2#1085

Merged

Navigation Menu

Repository navigation

refactor(provider-acp-registry): move the ACP Registry into its own package - #17405

refactor(provider-acp-registry): move the ACP Registry into its own package#17405
juliusmarminge merged 2 commits into
mainfrom
t3/provider-acp-registry

juliusmarminge commented Oct 9, 2026 •

edited

Loading

Uh oh!

macroscopeapp Bot commented Oct 9, 2026 •

edited

Loading

Uh oh!

github-actions Bot commented Oct 9, 2026 •

edited

Loading

Uh oh!

coderabbitai Bot commented Oct 9, 2026 •

edited

Loading

Walkthrough

Changes

❌ Failed checks (1 warning)

Uh oh!

Uh oh!

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Conversation

juliusmarminge commented Oct 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

macroscopeapp Bot commented Oct 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Approvability

Uh oh!

github-actions Bot commented Oct 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Thread transfer impact

Uh oh!

coderabbitai Bot commented Oct 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

❌ Failed checks (1 warning)

Uh oh!

Uh oh!

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

juliusmarminge commented Oct 9, 2026 •

edited

Loading

macroscopeapp Bot commented Oct 9, 2026 •

edited

Loading

github-actions Bot commented Oct 9, 2026 •

edited

Loading

coderabbitai Bot commented Oct 9, 2026 •

edited

Loading