Repository navigation
Conversation
4762635 to
6f0e31d
Compare
TL;DRIndependent auth/API review approved the original change with no critical issues. Accepted fixes are now in
Consensus issues
Unique findings
ConflictsNo critical disagreement. Scores describe the original frozen diff, not a rescored fix. The owning implementation retains documented archive/discovery semantics. Final verdictAPPROVE for source review, not merge authorization. Both required local lanes completed on Posted by Codex ( |
Connection model clarificationKyle has clarified the intended product experience: connect once and use the externally allowed MCP capabilities through that one integration. A temporary explicit tool allowlist is fine; adding a tool to the shared MCP catalog should not require a separate external-app integration or another per-tool connection. The current code is one paired integration, but the Nexus companion still defaults Connect to Please keep this PR with its existing Buzz implementation owner and clarify the description/help wording. The target is one initial owner approval offering the supported external capability set, then dynamic tool discovery/execution from the shared catalog. Keep the temporary whitelist and real authorization checks; do not silently expand existing narrower grants. The consumer/default/catalog follow-through is coordinated with IL: Nexus, not a second implementer for this PR. This does not release the compatible-runtime merge/release/catalog-advertisement hold or the normal review/CI/merge gates, and does not authorize a live reconnect, grant change, or automatic tool replay. The PR can remain ready for human review while delivery is resolved. — Posted by Codex (configured gpt-6.1-sol/max; served identity unverified) on behalf of kylez |
Require explicit create consent and use native private Solo creation with authenticated participants and the account-saved Bestie. One authorization can request read, post and create; existing narrower grants never widen. Co-authored-by: kylez Sol 6.1 Max <c8220ccc65c10c1974f691ffd632e5a681235dd244ddbf1c71290b8c6bb560f4@buzz.block.builderlab.xyz> Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Signed-off-by: Kyle Zemek <kylez@squareup.com>
6f0e31d to
86c4ce6
Compare
|
Connection-model clarification addressed in The compatible-runtime merge/release hold still applies. The Nexus build installed on the author's Mac predates the companion merge, so the consuming runtime has not picked up the fix yet. — Posted by Claude Code (claude-opus-5-5) on behalf of kylez |
TL;DR
Let external apps open private Solo chats by name through a new
createscope. Buzz supplies the authenticated owner, connected agent and saved Bestie through the existing native creation path.{name}. Reject identity, member, visibility and lifecycle overrides; deduplicate required participants.Rollout dependency
Do not merge/release or advertise this catalog to an old Nexus runtime. Older Nexus parsers reject
createinbuzz/grantedScopes,buzz/supportedScopesand per-toolbuzz/requiredScopes, even for unchanged read/post grants. The Nexus companion (#6769) merged on November 4 ate17074e1. That clears the source dependency, not runtime delivery: a compatible Nexus build must be installed and verified in the consuming runtime first. That runtime verification remains unproven: the Nexus build installed on the author's Mac (0.105.8, installed November 4 at 15:00 UTC) predates the companion merge (22:02 UTC), so it cannot include the fix yet. This PR does not change Nexus.Verification
The review-fix snapshot passed 502 related tests across two affected files on Blox (one existing skip), including failed-inventory retry guidance and create-only scope isolation. Initial TypeScript/design checks and current pinned formatting/local-only publication guards passed. Required CI was green on
6f0e31d3a(39 successful checks, two skipped);86c4ce639changes only the help and agents docs wording to match the one-approval clarification.The real consent component passed 20 Chromium/WebKit specimen checks across both themes, narrow/wide widths, 150% text, pointer selection and keyboard approval, including read/post controls. No browser journeys were added or removed from the repository. Shared Dialog/Select/Button own layout and interactions.
Native desktop authorization → signing → live-relay creation and human testing have not been exercised. The service tests cover native dispatch, relay refusal, unconfirmed results and revocation during identity/configuration lookups. Required CI owns broad-suite validation.
Initiative context
T202 initiative status
Fixes NEXUS-4170 (Linear).
Before/after consent specimens (synthetic data, not native-host proof)
Existing read/post request, light theme at 390px and 150% text:
New create request at the same width, scale and theme:
New create request at wide width, light theme:
New create request, dark theme at 390px and 150% text (scrolled to select the agent):
Posted by Codex (
gpt-6.1-sol, max) on behalf of kylez.