Repository navigation
docs(plan): propose SDK high-level parity coverage (#1471) - #1916
Merged
Merged
Conversation
samuv
added this pull request to stack #1918
September 25, 2026 08:46
samuv
changed the title
10 tasks done
samuv
marked this pull request as ready for review
September 25, 2026 08:48
samuv
changed the title
samuv
force-pushed
the
plan/sdk-high-level-parity-coverage
branch
from
September 25, 2026 08:56
64ecd2d to
b9b2eac
Compare
samuv
force-pushed
the
plan/sdk-high-level-parity-coverage
branch
3 times, most recently
from
September 25, 2026 12:24
2d1c6bc to
5191964
Compare
Signed-off-by: Samuele Verzi <samu@stacklok.com> Co-Authored-By: Codex <codex@openai.com>
Classify the upstream guardrail RPCs as session operations and the new TUI builtin as SDK-backed. Update the public API and transport contract for the stacked implementation. Signed-off-by: Samuele Verzi <samu@stacklok.com>
samuv
force-pushed
the
plan/sdk-high-level-parity-coverage
branch
from
September 25, 2026 12:47
5191964 to
d7cef19
Compare
Merged
2 tasks
jbeda
added a commit
that referenced
this pull request
Sep 25, 2026
…mit-coauthor-guidance * origin/main: (32 commits) docs: plan optional native Kubernetes execution provider (#1579) feat(mecatui): restore session status metrics (#1891) docs(plan): bound agent inventory viewport (#1946) docs: include user-docs in matlatl and use local documentation links (#1934) feat(mecatui): bound sessions inventory (#1944) feat(studio): show session delegation activity (#1907) docs(plan): define Studio session activity (#1885) Release TypeScript SDK v0.4.0 (#1940) Release v0.0.41 (#1941) docs(mecatui): plan bounded sessions inventory (#1906) feat(studio): add exact plan control and authorization review prerequisite (#1909) test(app): eliminate real ToolHive probe deadline wait (#1937) docs(plan): define Studio chat approvals and exact plan control (#1888) test(sdk): enforce SDK/TUI high-level parity (#1471) (#1917) docs(plan): propose SDK high-level parity coverage (#1471) (#1916) docs: improve gRPC schema table readability (#1842) test(app): eliminate real waits in durable learning tests (#1933) feat(studio): align chat transcript and composer (#1886) docs(plan): define Studio chat transcript and composer (#1880) feat: correlate provider requests with the root session (#1883) ...
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
1 participant
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
Define the acceptance contract for #1471. A public RPC receives high-level SDK coverage only when a public SDK operation invokes that exact descriptor. The SDK and TUI remain independent clients.
The plan classifies all 86 public RPC descriptors: 84 covered and exactly two reasoned raw-only exceptions,
StreamSessionEventsandStreamSessionLive. It separately classifies all 31 TUI builtins: 24 SDK-backed, three application/presentation-only, three operator/configuration-only, and one debug-only. Eight declarations absent from the TUI's separate known-name registry are audited without changing dispatch. Four public-export workflow examples are required.Two upstream guardrail RPCs added after the first draft receive public
Session.guardrailCoverage()andSession.guardrailReviewDetail()methods. They are gRPC-only until the server provides HTTP routes.Development stage
Contract linkage
docs/acceptance/sdk-high-level-parity-coverage.mdin this PR.Interface conformance
The plan adds two session-scoped TypeScript methods over existing gRPC descriptors. It changes no protobuf contract, exported Go API, CLI, configuration, event, persistence, or TUI dispatch behavior. API Extractor reports and the generated public reference are required for the additive SDK API. The existing SDK release workflow owns the changelog entry.
Issue relationship
Relates to #1471.
Type of change
Test plan
task docsandtask site:buildon the stacked candidate.ac-trace: 11 acceptance criteria, zero missing proofs (report-only while proposed).Runtime gates and example execution are reported in implementation PR #1917.
Changes
docs/acceptance/sdk-high-level-parity-coverage.mddocs/acceptance/README.mdUser-facing change
None in this plan PR.
Special notes for reviewers
Implementation PR #1917 is stacked on this plan for joint review. Plan approval and merge establish the baseline before implementation merge.