Skip to content

Reuse shared scoped dispatch and ordinary inference in live sessions - #19

Open
Brian Krabach (bkrabach) wants to merge 4 commits into
mainfrom
codex/decision-model-spike
Open

Brian Krabach (bkrabach) wants to merge 4 commits into
mainfrom
codex/decision-model-spike

Conversation

@bkrabach

@bkrabach Brian Krabach (bkrabach) commented Oct 22, 2026 •

Copy link
Copy Markdown
Collaborator

Reuse Streaming's scoped dispatcher and attributed ordinary-inference capability from loop-live, removing its duplicate implementation. Portable controllers bind during a live parent call while retaining normal tool hooks, approvals, call identity, cancellation, and finite-turn compatibility. Existing host disablement and caller/tool restrictions remain authoritative. No decision-provider, browser, application or CLI dependency is added.

Streaming #68 is now merged. The lock resolves its accepted public-main commit e04bba7e85bff9836bfc48cd8ac1db676f4f5fab; the maintained dependency still tracks canonical Streaming main. Successor d38157796843c7aea65d5afa74084429d5f3f23f changes only this lock entry. Work #13 follows this PR's actual merge.

Validation against accepted Streaming main in a fresh owned DTU environment: 24 dispatch/caller/cancellation tests passed, locked dependency installation passed, and the built wheel passed a separate isolated installation/import check without application adapters. Local tests used declared Core 1.6.1 and Context Simple 5055ddc98c46df575c16d6f8bb751c7a1306b69d; these are offline authority/compatibility checks, not live-model or desktop acceptance. The attempted current-Core source build in this arm64 DTU lacked a C linker, so it does not count as current-Core acceptance. No system tooling or preview runtime was changed.

Normal CI now passes against current public Core/Streaming/Context main: 125 passed, 5 skipped on each Ubuntu/macOS Python 3.11/3.13 job, 50 passed in the native-provider adapter job, and all wheel build/import checks passed. This supplies the current-Core acceptance that the local environment could not build. Ready for independent review; normal repository approval rules apply.

@bkrabach
Brian Krabach (bkrabach) marked this pull request as ready for review October 22, 2026 08:15
@bkrabach
Brian Krabach (bkrabach) requested a review from a team October 22, 2026 08:15
@bkrabach

Copy link
Copy Markdown
Collaborator Author

The accepted-Streaming follow-up is complete at d38157796843c7aea65d5afa74084429d5f3f23f. The only successor change is the lock entry resolving public Streaming main e04bba7e85bff9836bfc48cd8ac1db676f4f5fab (merged #68).

Normal CI is green: 125 passed, 5 skipped on each Ubuntu/macOS Python 3.11/3.13 job, 50 passed for native-provider adapters, and all package/import checks passed. These jobs use current public Core/Streaming/Context main. Separately, the owned DTU passed 24 focused dispatch/caller/cancellation tests and wheel install/import checks with declared Core 1.6.1; its temporary source, environments and caches were removed after preserving evidence.

The PR is now ready for normal independent review. Work #13 still follows this PR's actual merge. No Live merge or approval bypass has been performed.

@bkrabach

Copy link
Copy Markdown
Collaborator Author

Independent source/CI review of exact head d38157796843c7aea65d5afa74084429d5f3f23f: no actionable finding.

The implementation reuses Streaming's scoped dispatcher rather than maintaining a parallel executor. Live retains its activation-validity check; the inherited run/call scopes preserve the active parent identity, ordinary hooks/approval path, shared execution locks, finite execution and lease expiry. Explicit disablement, caller/target restrictions and cancellation remain authoritative. model.complete binds through that same active caller, uses the selected ordinary provider/model, retains request-policy/input/count checks and attributed completion, and does not turn into a second conversation or tool-execution path.

I verified Streaming #68 is merged at e04bba7e85bff9836bfc48cd8ac1db676f4f5fab, matching this lock. Relative to the previously reviewed Live candidate 646707c7814c7a3cfef99c11195df89b991c108c, the successor changes only that lock entry.

CI 39744929291 is successful at this exact head. Logs confirm accepted Streaming, public Core 9614052f7c9e493977dcba3f5c98ebd4d8591ec5 (2.0.2), and Context Simple db641363013ff906b656f49498a87ab01f5c506f: 125 passed / 5 skipped in each Ubuntu/macOS Python 3.11/3.13 job, 50 native-adapter checks passed, and each wheel build/isolated import check passed.

This is independent source and retained-CI verification, not a new reviewer runtime, live-provider or desktop acceptance run. I did not repeat the prior focused tests because their product inputs are unchanged. This comment is not an approving review or a bypass of the repository's normal independent maintainer approval.

@bkrabach Brian Krabach (bkrabach) left a comment

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Scoped source review completed at d381577: no actionable findings.

Verified that the successor changes exactly one uv.lock entry relative to 646707c, resolving Streaming e04bba7e85bff9836bfc48cd8ac1db676f4f5fab. Its accepted tree 38ffba2bc03814e53ee0c223a00b7941a5d2161e matches the tested Streaming candidate 20792f1cc7afe0945a789db61a3fa476306efd69. Live application source is unchanged by this successor.

I also checked the Live integration against the accepted dispatcher: the existing finite/live execution scope is reused, ordinary tools enter the shared pipeline once, Live activation remains checked before admitted effects, caller/parent/session authority remains enforced, and nested cancellation retains unknown-outcome reporting without replay. The new caller tests retain denial, revocation, parent attribution and recursion exclusions. The model capability selects through Live's provider owner, retaining its provider pin.

Independently read the completed CI results and logs for run 39744929291: all four Ubuntu/macOS Python 3.11/3.13 jobs passed (125 passed, 5 skipped each), native-adapter passed 50 tests, and all wheel build/isolated import checks passed. The logs resolve accepted Streaming e04bba7 and current Core 9614052f7c9e493977dcba3f5c98ebd4d8591ec5 (2.0.2). CLA is green. The author's separate 24-check and wheel evidence against locked Core 1.6.1 remains separately attributed; I did not rerun those tests.

Source review is clear. This comment is not an eligible approving review: the current GitHub identity is the PR author. The existing maintainer request and normal approval requirement remain in force. No admin merge or protection change is authorized for Live. Accepted-main Work qualification still follows actual Live/Work acceptance; this review does not relabel the frozen candidate evidence.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant