Repository navigation
Move new chat and manager actions into sidebar - #5
Merged
SawyerHood merged 5 commits intoMay 12, 2026
Merged
Conversation
SawyerHood
marked this pull request as ready for review
May 12, 2026 22:23
SawyerHood
marked this pull request as draft
May 12, 2026 22:27
SawyerHood
marked this pull request as ready for review
May 12, 2026 22:35
brsbl
referenced
this pull request
in brsbl/bb
Jun 6, 2026
The Loading apps... / Couldn't load apps. notice rendered after the Create App... row, so Create App was not last while apps were loading or failed to load. Fold the app-load status above Create App inside the trailing row group so Create App is always the final row in every app state. Normal installed-app and no-app ordering and the #5/get-bb#6 ARIA/toolbar fixes are unchanged. Add tests for the loading and error states asserting the notice precedes Create App and Create App stays last. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
15 tasks
ymichael
added a commit
that referenced
this pull request
Aug 17, 2026
The only provider without an existing bridge, built canonical-only (no legacy dialect; the legacy in-process adapter stays untouched and live). Per decision #5 the bridge owns its app-server children: one per bb thread with #1402 supervision (finalize on close with bounded grace, pipe destruction, per-callback currency checks, allowlisted child env, SIGTERM->SIGKILL escalation, kill-all on stdin close). Unexpected child death settles open turns as failed and the next turn restores from the rollout with a session/replaced notification. Translation shared, not duplicated: the adapter's ~1,200 lines of stateful closures extracted byte-identically into codex/translator.ts + session-params.ts (adapter 2,255 -> 348 lines); event-translation, schemas, interactive-requests, permission-mapping imported by both dialects. Bridge-minted structural ids (bt<entropy>-<serial>-<codexId>) keep parentToolCallId consistency with no maps and reverse-map for steer/interrupt/fork including legacy-persisted raw codex ids. Archived-session resumes reply SESSION_NOT_RESTORABLE with the original text preserved for the unarchive-retry path. Handshake: sessionRestore, archiveSync, nameSync, goalState, manualCompaction all true; fork checkpoint; runtime approval policy. Conformance 11/11 against real supervised fake-app-server children. Suites 973 default + 253 isolated, zero legacy assertion changes; the daemon build produces bb-codex-bridge.mjs, wired into the bundle manifest, launcher required-artifacts, and the tarball smoke. Co-Authored-By: Claude Fable 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.
Summary
useIsMutatingso pending hires keep the action disabled after the dialog closes.Validation
pnpm exec turbo run typecheck --filter=@bb/appNODE_OPTIONS=--no-experimental-webstorage pnpm exec turbo run test --filter=@bb/app -- src/components/dialogs/HireManagerDialog.test.tsx src/components/sidebar/ProjectList.test.tsx src/components/sidebar/AppSidebar.test.tsx src/hooks/mutations/project-mutations.test.tsxpnpm exec turbo run build --filter=@bb/appgit diff --check