Repository navigation
Conversation
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.
Human comments
What was wrong
The Pi provider forwards extension dialogs, but has no generic path for a third-party Pi extension to publish the lifecycle of its own background work. Pi event-bus notifications remain inside the Pi process, and terminal UI status/widget requests are intentionally dropped. An extension can therefore have active subagents while BB has no native background-task cards or active-agent count for them.
What changed
bb:background-taskevents and abb:background-task:requestsnapshot hook to the injected Pi extension. Transport the selected, bounded fields over the existing private child/bridge pipe; validate at the receiving boundary and translate to existing nativebackgroundTaskdeltas.Changes stay in
plugins/provider-pi: no Pi Toolbox dependency, additional tools/workers, automatic model continuation, core/Plugin SDK exports, dependencies, SDK version bump, server/daemon wire change, or installed-provider changes. These are private messages between the bundled extension and its bridge;HOST_DAEMON_PROTOCOL_VERSIONis unchanged.This is separate from #5271 (Windows launching), and complementary to #5255 (built-in background shell tools). This PR observes work owned by third-party extensions; it does not execute/cancel that work, collect its output, or deliver results to the model. A Pi Toolbox adapter is deliberately outside this PR.
How you verified
pnpm exec turbo run lint typecheck test prepare:bundled --filter=bb-plugin-provider-pi --concurrency=2 --force: all 11 tasks passed, 242 tests passed, one existing Bun-only test skipped.oxfmt --checkpassed for all nine changed files;git diff --checkpassed.session_startexited before producing an interaction request, on both the patched and original provider checkouts. This pre-RPC initialization behavior is not claimed as fixed here; scripted protocol tests verify that an emitted candidate startup request is scoped, forwarded, and cleaned up correctly.No installed provider or user Pi configuration was changed. No browser/UI visual verification or native Windows/Bun run was performed for this PR; the existing Bun-only test is skipped on this host.