Repository navigation
feat(sdk): add owned host probe and catalog contracts - #844
Closed
bandzalkin wants to merge 56 commits into
Closed
bandzalkin wants to merge 56 commits into
bandzalkin wants to merge 56 commits into
Conversation
Give the repeated tool-completion and privacy checks one helper, and name the ACP call record so activity identity is not a bare tuple. Co-authored-by: Cursor <cursoragent@cursor.com>
This was referenced Oct 18, 2026
github-actions Bot
pushed a commit
to bandzalkin/band-sdk-python
that referenced
this pull request
Oct 19, 2026
Source-Base: a96fb54 Source-Head: ec23a22 Upstream-PR: band-ai#844
bandzalkin
force-pushed
the
feat/host-probe-refresh-base
branch
from
October 20, 2026 06:36
ec23a22 to
07e093a
Compare
bandzalkin
requested review from
AlexanderZ-Band and
amit-gazal-band
as code owners
October 20, 2026 06:36
github-actions Bot
pushed a commit
to bandzalkin/band-sdk-python
that referenced
this pull request
Oct 20, 2026
Source-Base: ecd2b1b Source-Head: 07e093a Upstream-PR: band-ai#844
…t-activity-detail-to-the-platform-INT-1744 Keep main's BackendTurns ownership of OpenCode cancellation and register the watcher as a turn worker so activity reporting outlives the handler. Reporter unit tests move to tests/runtime/workingreporter; main's workingstate package keeps the execution-level tests. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AgnNR4uiqD787Bg3yKdZpL
…detail-to-the-platform-INT-1744' into feat/python-sdk-report-activity-detail-to-the-platform-delivery-INT-1744 Port the startup-backlog claim-failure test edit to the split tests/runtime/execution package. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AgnNR4uiqD787Bg3yKdZpL
…detail-to-the-platform-delivery-INT-1744' into feat/python-sdk-report-activity-detail-to-the-platform-producers-INT-1744 Adopt main's agno and tool test layouts and carry the producer edits into them; keep the split CrewAI/Parlant coverage guards with main's new paths. Letta streams provider events through main's BackendTurns run, and the CrewAI flow tool activity wraps main's turn-owned tool execution. Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01AgnNR4uiqD787Bg3yKdZpL
…sion and ControlLink fixes)
…ontrolLink fix
…them A name longer than 1024 characters is a 422 on the activity endpoint, and the compatibility path reads a detail-bearing 422 as an old platform: one oversized tool name would latch activity detail off for the whole process. Clip at the serialization chokepoint instead, one constant, so a long name costs characters and nothing else. Guarded by tests/platform/activity/test_truncation.py (over-limit clipped, exact-limit untouched).
The lifecycle peer rejects narration posts with the stopped-execution answer, so the first reported tool call read as a platform stop and tore down the turn machinery around a parked approval. A running room echoes 201 like the posting peer before it; stopped peers keep rejecting.
bandzalkin
added a commit
to bandzalkin/band-sdk-python
that referenced
this pull request
Oct 20, 2026
bandzalkin
added a commit
to bandzalkin/band-sdk-python
that referenced
this pull request
Oct 20, 2026
bandzalkin
added a commit
to bandzalkin/band-sdk-python
that referenced
this pull request
Oct 20, 2026
bandzalkin
added a commit
to bandzalkin/band-sdk-python
that referenced
this pull request
Oct 20, 2026
bandzalkin
added a commit
to bandzalkin/band-sdk-python
that referenced
this pull request
Oct 20, 2026
bandzalkin
added a commit
to bandzalkin/band-sdk-python
that referenced
this pull request
Oct 20, 2026
bandzalkin
added a commit
to bandzalkin/band-sdk-python
that referenced
this pull request
Oct 20, 2026
bandzalkin
added a commit
to bandzalkin/band-sdk-python
that referenced
this pull request
Oct 20, 2026
bandzalkin
added a commit
to bandzalkin/band-sdk-python
that referenced
this pull request
Oct 20, 2026
bandzalkin
added a commit
to bandzalkin/band-sdk-python
that referenced
this pull request
Oct 20, 2026
bandzalkin
added a commit
to bandzalkin/band-sdk-python
that referenced
this pull request
Oct 20, 2026
bandzalkin
added a commit
to bandzalkin/band-sdk-python
that referenced
this pull request
Oct 20, 2026
github-actions Bot
pushed a commit
to bandzalkin/band-sdk-python
that referenced
this pull request
Oct 21, 2026
Source-Base: aff3692 Source-Head: b470206 Upstream-PR: band-ai#844
github-actions Bot
pushed a commit
to bandzalkin/band-sdk-python
that referenced
this pull request
Oct 21, 2026
Source-Base: aff3692 Source-Head: b470206 Upstream-PR: band-ai#844
The mask wrapper was attached only when the flag was already off, so an adapter that binds its observer before the emit sync (codex's item/started path) kept reporting named tool_calls after the choice was turned off. The observer now always wraps and reads the live flag on every report; the hook site gates on the OMP branch remain as the second guard. Red-first test: observer bound with names on, choice flipped off mid-turn, tool report must arrive nameless. Co-Authored-By: Claude <noreply@anthropic.com>
github-actions Bot
pushed a commit
to bandzalkin/band-sdk-python
that referenced
this pull request
Oct 21, 2026
Source-Base: aff3692 Source-Head: 5f3910a Upstream-PR: band-ai#844
Contributor
Author
|
Closing: band-agent does not consume this; may be re-proposed per feature. |
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.
Summary
This adds shared host-facing contracts for checking an adapter and discovering its models before opening a Band room. Backend implementations can use the same result, catalog, and cleanup semantics rather than inventing their own readiness checks.
PreflightResultwithok,reason, andremedy, plus explicitpreflight(timeout_s=...)andprobe_models(timeout_s=...)adapter entrypoints. Unsupported preflight returns a failed result; unsupported discovery raises rather than fabricating an empty catalog.ModelCatalogandModelChoice, adding provider and default metadata fields while keeping unknown reasoning-effort support distinct from an explicitly empty set.HarnessProbeowner that registers cleanup before asynchronous startup. Operation and cleanup reporting have separate budgets; a timeout or cancellation does not abandon late startup or release a resource that has not closed.cleanup_all()to retry, and omit raw exception details from background probe-cleanup diagnostics.A successful preflight means the harness answered its handshake, not that inference access is available. Standalone discovery is separate from the existing room-facing
list_models()hook. Backend-specific probes are outside this PR.Validation
Contract tests cover unsupported operations, late startup, cancellation, eager task execution, retained cleanup failures, and diagnostic privacy.