feat: structure chat summaries as a headline plus bullets - #29203
Merged
Conversation
The summary panel prompt now asks for a short declarative headline followed by 2-4 markdown bullets. Summary normalization keeps line breaks, validation caps lines instead of sentences and requires a headline first line, and the panel renders the summary as markdown so bullets are not flattened.
Adopts the structured-object approach from #28013: the model returns headline and bullets fields that the server serializes to markdown, so validation can check each field instead of parsing the rendered string. Keeps the capability-first headline wording, enforces the 20-word headline limit, and renders through InlineMarkdown so backticked identifiers become inline code.
david-fraley
marked this pull request as ready for review
September 11, 2026 13:20
…t and cover rendering Says 20 words or fewer so the prompt matches summaryHeadlineMaxWords, adds "Changes..." to the headline openers and present tense to the bullet instruction to match what the model actually produces, and adds play assertions covering bullet list items, inline code, headline-only summaries and plain-text links.
Drops the play-function assertions per FE1, so the stories only cover visual states for Pixel, removes the comments flagged as unnecessary, and uses the wrap-anywhere utility instead of an arbitrary overflow-wrap property.
Collaborator
Author
|
Thanks, all applied in 0d2efa5:
Created by Coder Agents on behalf of @david-fraley. |
DanielleMaywood
approved these changes
Sep 11, 2026
david-fraley
enabled auto-merge (squash)
September 11, 2026 20:07
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
2 participants
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.
Rewrites the chat summary prompt (
chatSummaryGenerationPrompt) so the summary panel shows a capability-first headline plus optional bullets instead of 1-3 sentences of prose. The headline names what the chat defines, controls, or configures in present tense rather than recapping what happened.Implementation follows #28013: the model returns structured
headlineandbulletsfields, the server serializes them to markdown, and validation checks each field. Identifiers are backticked and render as inline code. None of that PR's panel UX changes are included here.ELI5: The "what is this chat about" box used to write a blob of sentences. Now it writes one short title line and a few bullets, so it is easier to skim.
Example output
Live generation against this PR's own chat (claude-sonnet-4-5):
Proof
Storybook story
pages/AgentsPage/ChatSummary:Created by Coder Agents on behalf of @david-fraley.