feat: display model name on observation/summary cards#1349
Open
ousamabenyounes wants to merge 2 commits intothedotmack:mainfrom
Open
feat: display model name on observation/summary cards#1349ousamabenyounes wants to merge 2 commits intothedotmack:mainfrom
ousamabenyounes wants to merge 2 commits intothedotmack:mainfrom
Conversation
Records which AI model (haiku, sonnet, opus, etc.) produced each observation and summary, and displays it as a badge in the web UI. Changes: - Migration 24: ADD COLUMN model TEXT to observations and session_summaries - Storage: storeObservation(), storeSummary(), storeObservations() accept model param - ResponseProcessor: reads CLAUDE_MEM_MODEL from settings and passes to storage - Types: model field added to ObservationRecord, SessionSummaryRecord, Observation, Summary - UI: model badge displayed next to project badge on ObservationCard and SummaryCard - CSS: .card-model style added to viewer-template.html - Tests: 4 new tests for model field storage (observations + summaries) Fixes thedotmack#1265 Vibe-coded by Ousama Ben Younes Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
…hedotmack#1265) The model field was stored in the DB but missing from SELECT queries in PaginationHelper, SessionStore, and observations/recent.ts, so the web UI badge never received the data. Added model to all relevant queries, updated types, and added tests for getAllRecentObservations. Co-Authored-By: Claude Opus 4.6 <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
Fixes #1265 — Adds model attribution to observations and summaries, displayed as a badge in the web UI.
When testing different models (haiku, sonnet, opus) for observation/summary generation, there was no way to see which model produced each card. This PR adds end-to-end support: DB schema, storage, API, and UI.
Changes
Database
ALTER TABLE observations ADD COLUMN model TEXT+ same forsession_summariesStorage layer
storeObservation(),storeSummary(),storeObservations(),storeObservationsAndMarkComplete()— all accept optionalmodelparamResponseProcessorreadsCLAUDE_MEM_MODELfrom settings and passes it to storageTypes
modelfield added toObservationRecord,SessionSummaryRecord(backend) andObservation,Summary(frontend)UI
ObservationCardandSummaryCard.card-modelCSS styling inviewer-template.htmlTest plan
bun test tests/sqlite/observations.test.ts tests/sqlite/summaries.test.ts)Vibe-coded by Ousama Ben Younes 🎧
🤖 Generated with Claude Code (Claude Opus 4.6)