Skip to content

Default the summary model to the chat model when unset in Settings #76

Description

@EurFelux

Motivation: The onboarding card backfills summaryModel with the chat model when the user enables auto-summary through the card, closing the "auto-summary on but no summary model → silent failure" gap. But the same gap still exists on the Settings path: if a user enables auto chapter summaries directly in Settings → Reading before ever configuring a summary model, resolveSummaryModel returns "not configured" and summaries silently fail.

Current behavior: summaryModel and the chat (assistant) model are two independent configs. resolveSummaryModel deliberately never falls back to the chat model. The SummaryModelPicker shows empty when unset.

Desired direction: In Settings, when summaryModel is unset, default the picker's displayed/effective value to the current chat (assistant) model — so a freshly-configured user who turns on auto-summary in Settings doesn't hit a silent failure. Keep resolveSummaryModel's no-implicit-fallback contract intact; this is a Settings-UI default (an explicit write when the user accepts it), not a runtime fallback.

Source: deferred §8 of docs/superpowers/specs/2026-06-09-onboarding-flow-design.md (onboarding card final review follow-up).

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    area:settingsSettings & preferencespolishUX refinement of an existing feature

    Projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions