refactor(server): integrate session MCP command stack#893
Merged
Conversation
This was referenced May 10, 2026
boudra
force-pushed
the
integration-session-mcp-command-stack
branch
from
760671c to
e75e898
Compare
boudra
force-pushed
the
integration-session-mcp-command-stack
branch
from
d8ca821 to
bfd6e3a
Compare
boudra
force-pushed
the
integration-session-mcp-command-stack
branch
2 times, most recently
from
44d16b4 to
76f6945
Compare
After the rebase, MCP `update_agent` was returning `success: false` for empty/no-op calls (no name, no labels, no settings). Origin/main returned `success: true` unconditionally. The audit flagged the change as out of scope for this stack — restore the old behavior at the MCP boundary. Session WS path keeps its accepted/rejected semantics (it surfaces an error when nothing was provided so the client can prompt the user).
boudra
force-pushed
the
integration-session-mcp-command-stack
branch
from
76f6945 to
b950153
Compare
boudra
added a commit
that referenced
this pull request
May 27, 2026
* Extract client SDK package * Polish SDK client identity defaults * Build client before dependent CI jobs * Restore daemon client server export * Extract protocol and client SDK packages * Fix provider override schema validation * Fix app test daemon client imports * Simplify workspace build targets * Fix CLI test server build bootstrap * Run SDK package tests in CI * Fix rebase package split drift * Restore lockfile registry metadata * Update SDK config test for prompt default * Move terminal stream router test to client package * Fix rebase drift for protocol imports * Fix SDK agent capability fixture * Restore legacy server client exports * Fix server export compatibility test * Advertise custom mode icon client capability * Remove server daemon-client exports * Format rebased mode control import * Fix rebase drift for protocol imports Files added by upstream PRs (#893, #1147, #1154) referenced the pre-split shared/ paths that this branch moves into @getpaseo/protocol. Redirect those imports to the protocol package so typecheck stays green after the rebase.
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
Integration PR for the Session/MCP command-boundary refactor stack. This is intentionally held as a draft until after the current stable release ships.
Included stack:
ea1390f9Extract permission response command30a5acc6Extract create agent commandfee2f4e8Extract agent lifecycle commands46794facrefactor(server): extract worktree archive commandc494969brefactor(server): share worktree create list commandsIntent
Session and the Agent MCP server are both entrypoints into the same daemon operations. This stack moves overlapping business orchestration into shared command modules so the entrypoints stay as transport adapters and stop drifting.
Release posture
Do not merge before the stable release. Keep this PR open as the single integration branch, then merge it once the release bug work is done and stable has shipped.
Verification
Each stacked PR/agent ran targeted tests plus format/typecheck/lint for its slice. CI should be the final gate for this integration branch.