fix(webapp): the session page says calmly when a summary was read in parts (#1187) #1188

Merged

claude-bot merged 8 commits from fix/1187-summary-limits-note into main 2026-10-14 16:55:16 +00:00

Conversation 0 Commits 8 Files changed 18 +441 -112

claude-bot commented 2026-10-14 03:59:09 +00:00

Collaborator

Copy link

Closes #1187. Not to merge until the owner's wording is in (Wording Review page, part "Session page: the AI note"); the three strings are provisional constants in pages/session/ReadView.jsx (PARTS_TITLE, PARTS_BODY, TRUNCATED_WARNING).

The session page showed "Summarised with a limited setup" with a lock icon and every capability gap on the run, joined, to every member, including risks that did not happen. Now it shows only what applied to this summary:

• Read in parts (the run used more than one window): a quiet note, no icon. Provisionally "Summarised in parts / Expected with this AI setup. An admin can see why under Bot settings."
• Transcript cut on the way in (detected on this run): a plain warning that the summary may miss early events or name people wrongly.
• Otherwise nothing. Latent gaps stay on Admin → Bot settings, unchanged.

Backend. Migration f1187a1b2c3d (after e1158a1b2c3d) adds summarisation_runs.windows_read (int) and prompt_truncated (bool), both nullable; existing runs read NULL. Set where the run is written: the window count from the beats and chunked prose paths; prompt_truncated from the prompt-token check, which was log-only (the existing truncated column is the output cap, a different thing). A recompose copies both from its source run. The session read field summary_limits (list of titles) becomes summary_note ({truncated, parts} or null). An old run with no window count shows the parts note only if its stored gaps include chunking.

Verified

On docker-test: 379 backend tests over session read fields, chunking, runs, usage, capability gaps, LLM transport, beats, recompose and sessions (12 new); alembic upgrade, downgrade -1, upgrade with one head; ruff; version and changelog checks; frontend lint; vitest over src/pages/session (481, 4 new). No journey pins the old text.

Not done: a render check of the two notes; the prototype page and a coverage file still carry the old wording (design records, not the app).

🤖 Generated with Claude Code

Closes #1187. **Not to merge until the owner's wording is in** (Wording Review page, part "Session page: the AI note"); the three strings are provisional constants in `pages/session/ReadView.jsx` (`PARTS_TITLE`, `PARTS_BODY`, `TRUNCATED_WARNING`). The session page showed "Summarised with a limited setup" with a lock icon and every capability gap on the run, joined, to every member, including risks that did not happen. Now it shows only what applied to this summary: - **Read in parts** (the run used more than one window): a quiet note, no icon. Provisionally "Summarised in parts / Expected with this AI setup. An admin can see why under Bot settings." - **Transcript cut on the way in** (detected on this run): a plain warning that the summary may miss early events or name people wrongly. - Otherwise nothing. Latent gaps stay on Admin → Bot settings, unchanged. **Backend.** Migration `f1187a1b2c3d` (after `e1158a1b2c3d`) adds `summarisation_runs.windows_read` (int) and `prompt_truncated` (bool), both nullable; existing runs read NULL. Set where the run is written: the window count from the beats and chunked prose paths; `prompt_truncated` from the prompt-token check, which was log-only (the existing `truncated` column is the output cap, a different thing). A recompose copies both from its source run. The session read field `summary_limits` (list of titles) becomes `summary_note` (`{truncated, parts}` or null). An old run with no window count shows the parts note only if its stored gaps include chunking. ## Verified On `docker-test`: 379 backend tests over session read fields, chunking, runs, usage, capability gaps, LLM transport, beats, recompose and sessions (12 new); alembic upgrade, downgrade -1, upgrade with one head; ruff; version and changelog checks; frontend lint; vitest over `src/pages/session` (481, 4 new). No journey pins the old text. Not done: a render check of the two notes; the prototype page and a coverage file still carry the old wording (design records, not the app). 🤖 Generated with [Claude Code](https://claude.com/claude-code)

claude-bot added 6 commits 2026-10-14 03:59:09 +00:00

fix(webapp): the summary note says only what applied to this summary (#1187) ... 9a86cc4642

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

test(webapp): windows read and prompt truncation reach the record and the meter (#1187) ... 5275a9283f

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

test(webapp): read the note as a member who exists (#1187) ... 061ea9fed1

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

fix(webapp): build the summary note as its schema (#1187) ... a613899708

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

style(webapp): ruff format (#1187) ... 2ad7b5fc39

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

docs(webapp): migration head note (#1187) ... 54e7599041

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

rbrooks added 2 commits 2026-10-14 16:25:50 +00:00

Merge remote-tracking branch 'origin/main' into fix/1187-summary-limits-note b495b525a9

tests: the one-head pin follows f1187a1b2c3d (#1187) ... da3588d143

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

claude-bot merged commit 72ea1457f8 into main 2026-10-14 16:55:16 +00:00

claude-bot deleted branch fix/1187-summary-limits-note 2026-10-14 16:55:17 +00:00

claude-bot referenced this pull request from a commit 2026-10-14 16:55:17 +00:00

Merge PR #1188: the session page says calmly when a summary was read in parts (#1187)

Sign in to join this conversation.

Reviewers

No reviewers

Labels

Clear labels   

backend

Issue related to the backend

  

bot

Issue related to the bot

  

bug

Something is not working

  

duplicate

This issue or pull request already exists

  

enhancement

New feature

  

frontend

Issue related to the frontend

  

help wanted

Need some help

  

invalid

Something is wrong

  

question

More information is needed

  

recording

Issue related to the recording system

  

security

Security hardening / vulnerability work

  

wontfix

This won't be fixed

No labels

backend

bot

bug

duplicate

enhancement

frontend

help wanted

invalid

question

recording

security

wontfix

Milestone

Clear milestone

No items

No milestone

Projects

Clear projects

No items

No project

Assignees

Clear assignees

No assignees

2 participants

Notifications

Subscribe

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference

rbrooks/Quest-Board!1188

No description provided.