Repository navigation
Replace fake registry SDK client in ChatGPT extension with projectless RegistryClient - #4354
Draft
ScriptedAlchemy with Copilot wants to merge 2 commits into
Draft
ScriptedAlchemy with Copilot wants to merge 2 commits into
ScriptedAlchemy with Copilot wants to merge 2 commits into
Conversation
|
Co-authored-by: ScriptedAlchemy <25274700+ScriptedAlchemy@users.noreply.github.com>
Copilot
AI
changed the title
[WIP] Simplify the daemon client boundary across HTTP and MCP operations
Replace fake registry SDK client in ChatGPT extension with projectless RegistryClient
Oct 19, 2026
This branch has not been deployed
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary
baseUrl,projectIdandtokenvalues, only to satisfy its config type. This PR replaces that with an explicit projectless registry capability.Motivation
The registry client was created with
http://127.0.0.1,"registry"and"unused-registry-reads-use-mcp", so a valid operation needed a fictitious endpoint and credential to be invoked. Registry scope should be expressed directly rather than through values that are never used.Changes
src/server/registry-client.ts(new):RegistryClientexposesprojectListandprojectContext. It takes no URL, project ID or token.OPERATIONScatalog in@tracedecay/sdk. Nothing is redefined by hand.failureFromSdktherefore needs no change.src/server/bridge.ts:#registryClient()now returns aRegistryClientwired to the existing projectlessServeSession.listProjectsandresolveProjectcall it directly. Project handles are unchanged.README.md: documents the registry path separately from the MCP and HTTP project operations.Test plan
bazel build //...passes when the change crosses package boundariesRun locally with
npxafter building the SDK withrslib, since this environment has no Bazel build:tsc --noEmitis clean.test/adapter.test.tspasses.test/extension.test.tsneeds the Bazel-builttracedecaybinary, which was not available. Its registry and project journeys have not been run against a real daemon, so please run them in CI.Checklist
CHANGELOG.mdupdated (under[Unreleased]if no version bump).envfiles included