Repository navigation
feat(mcp): resolve room-scoped servers at native OMP connection startup - #798
Open
bandzalkin wants to merge 2 commits into
Open
bandzalkin wants to merge 2 commits into
bandzalkin wants to merge 2 commits into
Conversation
bandzalkin
force-pushed
the
feat/omp-owned-mcp-resolver
branch
from
October 14, 2026 00:59
dd7181b to
a625466
Compare
bandzalkin
added a commit
to bandzalkin/band-sdk-python
that referenced
this pull request
Oct 14, 2026
Source-Base: 4184abb Source-Head: a625466 Applied-Over: 8fd9ca6 Conflict-Notes: None; exact feature delta, excluding CHANGELOG.md and FORK-CHANGES.md Upstream-PR: band-ai#798
github-actions Bot
pushed a commit
to bandzalkin/band-sdk-python
that referenced
this pull request
Oct 14, 2026
Source-Base: 4184abb Source-Head: a625466 Applied-Over: 8fd9ca6 Conflict-Notes: None; exact feature delta, excluding CHANGELOG.md and FORK-CHANGES.md Upstream-PR: band-ai#798
bandzalkin
added a commit
to bandzalkin/band-sdk-python
that referenced
this pull request
Oct 15, 2026
Source-Base: 4184abb Source-Head: a625466 Applied-Over: fda806c Conflict-Notes: None; exact feature delta over the ported native OMP lifecycle; retained upstream band-ai#709 and band-ai#804 adaptations from the prerequisite. CHANGELOG.md and FORK-CHANGES.md excluded. Upstream-PR: band-ai#798
bandzalkin
added a commit
to bandzalkin/band-sdk-python
that referenced
this pull request
Oct 15, 2026
Source-Base: 4184abb Source-Head: a625466 Applied-Over: 00f3dfb Conflict-Notes: None; exact feature delta over the ported native OMP lifecycle; retained upstream band-ai#709 and band-ai#804 adaptations from the prerequisite. CHANGELOG.md and FORK-CHANGES.md excluded. Upstream-PR: band-ai#798
github-actions Bot
pushed a commit
to bandzalkin/band-sdk-python
that referenced
this pull request
Oct 15, 2026
Source-Base: 4184abb Source-Head: a625466 Applied-Over: 00f3dfb Conflict-Notes: None; exact feature delta over the ported native OMP lifecycle; retained upstream band-ai#709 and band-ai#804 adaptations from the prerequisite. CHANGELOG.md and FORK-CHANGES.md excluded. Upstream-PR: band-ai#798
bandzalkin
force-pushed
the
feat/omp-owned-mcp-resolver
branch
from
October 15, 2026 03:43
a625466 to
4c393d0
Compare
Contributor
Author
|
Rebased onto current main ( |
bandzalkin
added a commit
to bandzalkin/band-sdk-python
that referenced
this pull request
Oct 15, 2026
Source-Base: 242a3c9 Source-Head: 4c393d0 Applied-Over: 4aa5d48 Upstream-PR: band-ai#798
github-actions Bot
pushed a commit
to bandzalkin/band-sdk-python
that referenced
this pull request
Oct 15, 2026
Source-Base: 242a3c9 Source-Head: 4c393d0 Applied-Over: 4aa5d48 Upstream-PR: band-ai#798
github-actions Bot
pushed a commit
to bandzalkin/band-sdk-python
that referenced
this pull request
Oct 15, 2026
Source-Base: 242a3c9 Source-Head: 4c393d0 Upstream-PR: band-ai#798
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.
Rebased onto main
d7ab3429cbdb(2026-10-15)Rebased onto current main (after #709, #804, #789/#775, #796 and #802) as its prerequisite's commit plus one commit; head
4c393d0625a0d2d50b205b9d76d2452b3b8b36f6. Applied cleanly on rebased #763.CHANGELOG.mdedits are dropped: Release Please generates it.Verified at the new head (macOS, Python 3.13):
uv sync --all-packages --locked --extra dev, all pre-commit hooks, and the fullpytest -q: 8686 passed, 739 skipped. Verification notes further down refer to the previous head.Summary
Resolve room-scoped external MCP servers at native OMP connection startup, under the same owner that creates and retires the connection. Related: #709.
Changes
Stack and merge order
Prerequisite: #763 at
242a3c96268ee7f322213656a658ac5bb1c75133.Exact feature-only range:
242a3c96268ee7f322213656a658ac5bb1c75133..4c393d0625a0d2d50b205b9d76d2452b3b8b36f6.Merge #763, then #798. This main-targeted branch includes its prerequisite until that merge; the range above is resolver-only. Copilot and Codex/Claude lanes are independent.
All eight feature patches rebased unchanged. The inherited portability corrections calibrate natural decoder depth and prove positive peer/root/descendant settlement instead of requiring exit zero. The Windows launcher-hosted forced-close limit is disclosed in #763 and docs/acp.md; this PR does not make close graceful.
Testing
Prepared head differs from approved/tested
ab77e212only in CHANGELOG.md. At that tested head, permission/RPC/OMP-adapter scope: 142 passed on managed Python 3.13. The prerequisite correction scope passed managed 3.11/3.12/3.13 and fresh non-privileged Linux, with an explicit positively proved no-natural-window skip on 3.11. All five static hooks passed freshly at the prepared head.Historical pre-correction all-adapter/OMP/registry scope at
dd7181bf: 2827 passed, 3 skipped, 56 warnings; not a new-head full-suite claim. Source/index-delta stable patch ID remains50b5b6044975fc3940661f78580c05438135e3d3, excluding only changelog/fork metadata. No new numeric fork graph is claimed.send_eventremains best-effort, not durable delivery. Local evidence is not native-provider/live-room acceptance or a remote matrix receipt; every new-head CI lane is awaited separately.