fix: clarify human participant invite contract#2011
Conversation
yuezengwu
left a comment
There was a problem hiding this comment.
Reviewed head d41a42a. This change formalizes the existing participant-invite contract without changing the membership model: CLI, reference docs, runtime guidance, SDK/shared comments, and compatibility tests now consistently cover eligible active same-organization humans and agents, silent membership changes, and addressed delivery. I verified that the centralized invite path still preserves organization, active/member-status, private-owner, caller-speaker, and managed-trial restrictions, while membership writes remain silent and wake-up stays in the addressed send path. No database or core data-structure changes. No blocking findings. Static diff review only; I did not rerun tests.
yuezengwu
left a comment
There was a problem hiding this comment.
Re-reviewed head 83379f4 after the scope narrowing. The new commit cleanly removes the service, shared-schema, SDK-contract, server-error, and broader test changes, leaving only the public CLI/reference/runtime wording plus focused active-human endpoint coverage required by the updated issue. Existing centralized invite behavior remains untouched, so organization, active/member-status, private-owner, caller-speaker, managed-trial, silent-membership, and addressed-send wake semantics stay unchanged. No database or core data-structure changes. No blocking findings; static diff review only, and I did not rerun tests.
Non-blocking: the PR body still says this PR adds user-side chat access, silent/addressed-delivery, and removed-human coverage, but those additions were removed by this commit. Please align the description with the narrowed diff.
|
Addressed. The PR body has been rewritten for the narrowed head:
The current diff is 7 files, +56/-13. |
Summary
chat invitecontract as accepting an eligible active same-organization human or agentchat sendrecovery hint, the runtime briefing, and the CLI referencechat sendfor attentionCloses #1965
Scope
This is an agent-facing contract/documentation correction, not a new feature. The existing membership path already supports human participants.
The PR does not change service behavior, authorization or visibility rules, shared schemas, legacy wire fields, server errors, SDK semantics, or the membership model.
Validation
pnpm check— passes with 16 pre-existing warnings and 1 pre-existing infopnpm typecheckpnpm testwas also run; the parallel monorepo run hit unrelated Web focus flakiness and skill-eval fixture timeouts under loadContext
The Context Tree defines chats as one shared group-conversation model, speakers as the addressable membership role, and installed CLI help as the public command truth. This change therefore corrects the public contract without adding a second participant model or changing delivery semantics.