Skip to content

feat(config): Peer model profiles via createApp - #1657

Merged
dcramer merged 18 commits into
mainfrom
feat/app-model-profiles
Aug 23, 2026
Merged

feat(config): Peer model profiles via createApp#1657
dcramer merged 18 commits into
mainfrom
feat/app-model-profiles

Conversation

@sentry-junior

@sentry-junior sentry-junior Bot commented Aug 22, 2026

Copy link
Copy Markdown
Contributor

Add createApp({ defaultProfile, profiles }) so hosts define the full profile catalog in app code.

Profile names are host-owned labels. Handoff can switch to any configured profile except the active one. The router and conversation projection always use the configured defaultProfile; they do not treat standard or handoff as magic profile names.

For compatibility, env-only setup still creates standard and handoff profiles from AI_MODEL, AI_HANDOFF_MODEL, and AI_MODEL_PROFILES. These profile env vars now emit deprecation warnings.

Requested by David Cramer.

--

View Junior Session [Sentry]

Co-Authored-By: David Cramer <david@sentry.io>
@vercel

vercel Bot commented Aug 22, 2026

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
junior-docs Ready Ready Preview Aug 22, 2026 11:05pm

Request Review

Use createApp({ profiles }) instead of a nested models object.

Co-Authored-By: David Cramer <david@sentry.io>
@sentry-junior sentry-junior Bot changed the title feat(config): Configure model profiles in createApp feat(config): Configure profiles in createApp Aug 22, 2026
Handoff can switch to any configured profile except the active one,
including standard. Profile maps may set standard and handoff model ids
instead of treating those names as reserved.

Co-Authored-By: David Cramer <david@sentry.io>
@sentry-junior sentry-junior Bot changed the title feat(config): Configure profiles in createApp feat(config): Peer model profiles via createApp Aug 22, 2026
Co-Authored-By: David Cramer <david@sentry.io>
Comment thread packages/junior/src/app.ts
Comment thread packages/junior/src/chat/services/turn-router.ts Outdated
Comment thread packages/docs/src/content/docs/reference/config-and-env.md
Comment thread packages/junior/src/chat/services/turn-router.ts
Comment thread packages/junior/src/chat/conversations/history.ts
Co-Authored-By: David Cramer <david@sentry.io>
Comment thread packages/junior/src/chat/agent/index.ts Outdated
@dcramer
dcramer marked this pull request as ready for review August 22, 2026 17:07
@github-actions github-actions Bot added the risk: high PR risk score: high label Aug 22, 2026
Comment thread packages/junior/src/chat/services/turn-router.ts
Comment thread packages/junior/src/chat/agent/index.ts Outdated
Co-Authored-By: David Cramer <david@sentry.io>
Co-Authored-By: David Cramer <david@sentry.io>
Comment thread packages/junior/src/chat/agent/index.ts Outdated
# Conflicts:
#	packages/junior/src/chat/services/turn-router.ts
#	packages/junior/tests/component/runtime/agent-run-model-handoff.test.ts

Co-Authored-By: David Cramer <david@sentry.io>
Co-Authored-By: David Cramer <david@sentry.io>
@dcramer

dcramer commented Aug 22, 2026

Copy link
Copy Markdown
Member

update this from main

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit a792822. Configure here.

Comment thread packages/junior/src/chat/agent/index.ts
@dcramer
dcramer merged commit dddc951 into main Aug 23, 2026
43 of 44 checks passed
@dcramer
dcramer deleted the feat/app-model-profiles branch August 23, 2026 00:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

risk: high PR risk score: high

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant