Skip to content

Resolve default models from provider catalogs - #1002

Merged
ymichael merged 1 commit into
mainfrom
bb/default-model-selection-thr_yb8et7e9v6
Aug 4, 2026
Merged

Resolve default models from provider catalogs#1002
ymichael merged 1 commit into
mainfrom
bb/default-model-selection-thr_yb8et7e9v6

Conversation

@ymichael

@ymichael ymichael commented Aug 4, 2026

Copy link
Copy Markdown
Collaborator

Summary

  • remove the hard-coded Codex model and stop seeding fresh personal projects with a concrete model
  • resolve an omitted model from the selected provider catalog on the target host, preferring the provider-marked default and then the first model
  • preserve explicit and remembered selections, fail before thread creation when discovery is unavailable, and update CLI/agent guidance

Testing

  • pnpm exec turbo run test --filter=@bb/server --force (1,317 tests)
  • pnpm exec turbo run test --filter=@bb/cli --filter=@bb/templates --force (382 tests)
  • pnpm exec turbo run typecheck --filter=@bb/templates --filter=@bb/cli --filter=@bb/server
  • Prettier, generated-template, and diff checks

No host-daemon protocol change is required because this uses the existing provider.list_models RPC.

@ymichael
ymichael merged commit 43b23f7 into main Aug 4, 2026
10 checks passed
@ymichael
ymichael deleted the bb/default-model-selection-thr_yb8et7e9v6 branch August 4, 2026 21:12
@SawyerHood SawyerHood mentioned this pull request Aug 8, 2026
SawyerHood added a commit that referenced this pull request Aug 8, 2026
## Summary

Prepares the 0.36.0 release: version lockstep bump and release notes.

- bump `bb-app` and `@bb/desktop` to `0.36.0` via
`scripts/bump-version.mjs`
- add the `0.36.0` section to `CHANGELOG.md`
- add the `0.36.0` entry to `RELEASE_META` in
`apps/web/src/landing/changelog.ts`

## Release notes

Headline is "Fixes and improvements". The notes lead with the loopback
bind default (#1125), because it is the one change that needs user
action before the upgrade. That section links to
`docs/multiple-devices.md` for the bb connect and Tailscale Serve setup
steps rather than repeating them.

Two risks are called out explicitly in the notes:

- The host daemon protocol moved 69 → 82 in this range. A remote daemon
that reaches the server at a direct `host:38886` address loses its route
and cannot self-update, so the route must move first.
- A thread that names no model now resolves one from the provider
catalog on the target host (#1002), and fails to start when that host
cannot list models.

A Thanks section credits the external contributors in this range:
@ben-vargas, @Diffuzmetall, @kschrader, and @toasterman234.

Commands stay as inline code spans throughout. The `/changelog` parser
handles only paragraphs and bullets, and its inline renderer supports
only `code` and `strong`, so a fenced block or a Markdown link would
render as literal text on the marketing site.

## Validation

- `node .github/workflows/check-version-lockstep.mjs` — bb-app=0.36.0
@bb/desktop=0.36.0
- `pnpm exec turbo run typecheck test --filter=@bb/config
--filter=@bb/server --filter=bb-app --force` — 1367 tests passed
- `pnpm exec turbo run smoke:tarball --filter=bb-app --force` — passed
- `git diff --check` — clean

Unrelated pre-existing issue found while validating:
`apps/server/test/internal/internal-skill-trees.test.ts` asserts a file
mode of `0o644`, which depends on the process umask. It fails under
`umask 002` and passes under `umask 022`. CI is unaffected.

## Follow-ups

- Redeploy `@bb/web` after this lands so `/changelog` shows 0.36.0.
- `publish-bb-app.yml` and `build-desktop.yml` still need to run from
`main` at this commit.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-authored-by: Claude Opus 5 (1M context) <noreply@anthropic.com>
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