Repository navigation
Conversation
Co-authored-by: kylez 3: Sol 6.1 - High <c5ab3b18619d7436b638029e27226c768bf52b75cc4f8c8f947e6649cd806860@buzz.block.builderlab.xyz> Signed-off-by: Kyle Zemek <kylez@squareup.com>
Co-authored-by: kylez 3: Sol 6.1 - High <c5ab3b18619d7436b638029e27226c768bf52b75cc4f8c8f947e6649cd806860@buzz.block.builderlab.xyz> Signed-off-by: Kyle Zemek <kylez@squareup.com>
Use device-local personal associations for every conversation, retaining explicit shared Solo links and conservative status indicators. Co-authored-by: kylez 3: Sol 6.1 - High <c5ab3b18619d7436b638029e27226c768bf52b75cc4f8c8f947e6649cd806860@buzz.block.builderlab.xyz> Signed-off-by: Kyle Zemek <kylez@squareup.com>
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.
TL;DR
Keep PR associations and status indicators inside the optional GitHub plugin, without extending core messaging. Personal associations work for Team, DM and Solo conversations, including read-only public channels, without publishing or changing membership or shared metadata.
After creating a conversation's own PR, an agent can explicitly call
github_announce_prwith its URL and head branch. The plugin verifies signed-in GitHub authorship and branch before associating it; incidental links are never scanned or adopted.Details
Show for me/Hide for meandgithub_link_prmanage a device-local overlay scoped to person, community and conversation. Cross-window storage locks preserve manual replacement/unlink against delayed automatic verification.Tooltip, Tabler icons and semantic tokens. No core API, separate poller, shell interception or merge action.Verification
The corrected plugin-only snapshot is
5e97031dba146256cde5944a4616e94641408f0f. Isolated Blox tests, typecheck, Chromium/WebKit journeys and fresh independent Claude/Codex review are in progress; this PR remains draft. Previous-snapshot results do not establish corrected-snapshot validation.Three browser scenarios were added; none removed. Two actual-app journeys cover native Solo and read-only Team wiring, persistence, keyboard/menu behavior and non-overlapping targets at wide/light and narrow/dark doubled-text sizes. A small rendering fixture covers approved status and failed-read readiness removal. Status/association matrices remain in unit tests. The earlier Solo journey exposed native menu starvation at narrow width; the plugin wrapper now reflows status below the native row.
Browser proof is not native desktop or human testing; neither is claimed. No app/plugin was installed or restarted. Existing wide Solo and narrow Solo synthetic screenshots are from the preceding snapshot and are not proof of the personal-channel workflow.
Fixes NEXUS-4453.
Posted by Codex (gpt-6.1-sol) on behalf of kylez.