feat: migrate Channel manager UI to public contracts - #1
Merged
Merged
Conversation
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.
The Channel product renderer and Simulator adapter were maintained inside CordisX Host, which made plugin UI and service changes collide with Host work. This moves that ownership into the standalone Channel repository while keeping Manager chrome, credential capture, official credential-bearing adapters, persistence, transport, and the Channel runtime Host-owned.
The package contains the renderer, owned CSS, package/runtime manifest v8, and a Node service that registers only the exact manifest-authorized Simulator target through public
ctx.channel. It reads the Host-stamped service configuration revision and cannot self-report a lower or higher revision. It imports no Host-private renderer, launcher, service-config, credential, path, DOM, or transport implementation.Exact formal baselines:
3f0dbcd8b04ae83c920d2d913ac2c313af5f83f1dfb071e02eca0ef52f84d67b2393c25aced7d3f0Validation:
npm run checkThis PR establishes the independent plugin package. Deleting the old Host consumer and changing the Host default resolver to this exact package are a following Host owner change; until that lands, this merge alone is not the completed migration or user acceptance.
No live Codex Desktop launch or user acceptance was performed.