Skip to content

Show item previews in collapsed QA report severity groups - #1214

Open
pfbyjy wants to merge 1 commit into
stagingfrom
claude/message-preview-summary-yhsabp
Open

Show item previews in collapsed QA report severity groups#1214
pfbyjy wants to merge 1 commit into
stagingfrom
claude/message-preview-summary-yhsabp

Conversation

@pfbyjy

@pfbyjy pfbyjy commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Summary

Improves the QA report UI by displaying previews of action items in the collapsed state of severity group headers, while showing the tier effect when expanded.

Changes

  • Added findingPreview() function to extract and format a single-line preview from a finding's title, detail, or recommendation (flattened whitespace, capped at 120 characters)
  • Extended the SeverityGroups component to compute previews for each tier's items and include them in the group data structure
  • Updated the severity group header to conditionally display:
    • Collapsed: Item previews joined with " · " separator (with full text in tooltip)
    • Expanded: Tier effect description (using group-open: CSS selector)

Implementation details

  • Previews are computed once during group construction, not on every render
  • Empty previews are filtered out, so groups with no previewable items fall back to showing the tier effect in both states
  • The preview text is truncated with ellipsis and uses text-pretty for better line breaking when space allows
  • CSS classes use Tailwind's group-open: modifier to toggle visibility based on <details> open state

https://claude.ai/code/session_01AWVwKFa9EGQMtwWrZbmDyU


Note

Low Risk
Presentation-only change in action-items.tsx with no API or data handling impact.

Overview
Collapsed QA severity groups now show a one-line glimpse of what’s inside instead of only the tier effect text.

SeverityGroups builds per-item previews via new findingPreview() (title, then detail/recommendation; whitespace flattened, max 120 chars). When a tier has previews, the <summary> shows them joined with · while collapsed (group-open:hidden, truncated, full list in title). When expanded, that line hides and the existing tier effect line returns (hidden group-open:block). Groups with no previewable text keep the tier effect in both states.

Reviewed by Cursor Bugbot for commit ed87844. Bugbot is set up for automated code reviews on this repo. Configure here.

A collapsed group spent its whole header line on a static per-tier
sentence — "The defect can decide trials", "Does not change the
verdict" — which the MUST FIX / SHOULD FIX / OPTIONAL badge next to it
already says. Nothing on the row described the defects, so the only way
to learn what a group held was to open it.

Collapsed, the line now carries the items' own titles, joined and
truncated to one line, with the full list in the tooltip. Open, the
items are on screen, so the tier effect takes the line back.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01AWVwKFa9EGQMtwWrZbmDyU
@vercel

vercel Bot commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

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

Project Deployment Actions Updated (UTC)
oddish-app Ready Ready Preview Aug 12, 2026 5:52pm

@github-actions

Copy link
Copy Markdown
Contributor

Oddish preview

Commit: ed8784422bc6d3c1d40c971c19791eb8bf989bf3

Surface Link Target
Frontend https://pr-1214.oddish.app Vercel preview for ed87844
Backend staging staging
Database staging branch staging branch

Vercel deployment URL: https://oddish-qznizs03r.oddish.app

Plan:

  • Frontend deploy: true
  • Backend deploy: false
  • Migrations: false

This comment is updated by the PR Preview workflow.

@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.

Risk: low. Approved — Cursor Bugbot passed with no findings requiring human review; Cursor Security Agent was not present. No reviewers assigned.

Open in Web View Automation 

Sent by Cursor Approval Agent: Pull Request Router and Approver

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants