Repository navigation
Update to BB 0.47.1 and SDK 0.6.79, release 0.1.5 - #14
Merged
Merged
Conversation
BB removed the threadPanelAction slot, so the Advisor panel now registers through experimental_tab as a thread action tab. The id stays "reviews", so tabs users already opened still restore and the existing openThreadPanel calls still resolve. Provider catalogs now report reasoning efforts as open strings rather than an enum. Advisor accepts any non-empty level, so a level a provider adds round-trips through settings and spawn instead of failing the RPC contract. Pin zod to 4.6.5, the version BB and the SDK use, so the agent tool's schema type matches the SDK's. Raise engines to BB 0.47.1 and SDK 0.6.79: older hosts lack experimental_tab and keep 0.1.4. Ship the dist/package.json the new builder writes. Cover the reviewer's reasoning source on every spawn path: a saved default resolved from the catalog, a changed level, and a configured fallback level. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Reasoning levels became open strings, so a provider level literally named "default" passed validation and collided with the saved model-default sentinel: the settings dropdown showed two options with the same value, and choosing the level spawned the model's default instead. Such a level is no longer offered or accepted as an explicit choice; the model default it would duplicate stays available. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
The previous bundle was built in a working tree whose untracked files fed the CSS candidate scan, so app.css carried an unused utility, and its source map embedded that checkout's path. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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.
Brings Advisor up to the current stable bb (0.47.1, SDK 0.6.79) and includes #13 on top of it. Merging this also merges #13, since its commit is part of this branch.
What changed
experimental_tab. bb removedapp.slots.threadPanelAction. The panel now registers as a thread action tab (context: { kind: "thread" },availability: "action"), following the migration in bb's plugin-authoring reference. The id staysreviews, so saved tabs restore and the existingopenThreadPanel({ actionId: "reviews" })openers still resolve.reasoningEffortasstringrather than an enum. Advisor accepts any non-empty level, so a level a provider adds round-trips through settings and the spawn instead of failing the RPC contract. The exception is a level literally nameddefault: it would collide with Advisor's saved "Model default" choice, so it isn't offered. Saved selections are still checked against the live catalog.parameters.bb >=0.47.1andbbPluginSdk >=0.6.79. SDK 0.4.87 has noexperimental_tab, so older hosts stay on 0.1.4. bb coerces the running version, so nightlies (e.g. 0.47.2-nightly) satisfy the range.dist/package.jsonand minifies the server bundle. It is committed and listed infiles.distis built from a clean checkout. The builder embeds the build directory in one source-map entry, so a local rebuild changes only that line indist/server.js.map.defaultresolved from the catalog, a changed level, and a configured fallback level.Validation
npm ci && npm run verify && npm pack --dry-runon Node 22.19.0 (CI's version) and Node 26: typecheck, 132 tests, build (Verified BB 0.47.1 / SDK 0.6.79). CI passes.defaultcollision and a dist built from a dirty tree. Both are fixed, and it approvede2a6591. It traced the tab migration through the bb 0.47.1 host source:openThreadPanelresolves the action id, a launcher withoutrunopens the panel,flushlayout is honored, and saved tabs restore by id.🤖 Generated with Claude Code