Skip to content

feat: Update stack insights display - #3951

Merged
maciaszczykm merged 2 commits into
masterfrom
marcin/prod-5079-send-insights-to-workbenches
Aug 3, 2026
Merged

feat: Update stack insights display#3951
maciaszczykm merged 2 commits into
masterfrom
marcin/prod-5079-send-insights-to-workbenches

Conversation

@maciaszczykm

Copy link
Copy Markdown
Member

Continuation of #3938.

Test Plan

Zrzut ekranu 2026-08-3 o 10 43 45

Checklist

  • I have added a meaningful title and summary to convey the impact of this PR to a user.
  • If required, I have updated the Plural documentation accordingly.

Plural Flow: console

@linear

linear Bot commented Aug 3, 2026

Copy link
Copy Markdown

PROD-5079

@maciaszczykm

Copy link
Copy Markdown
Member Author

@greptileai

@soffi-ai

soffi-ai Bot commented Aug 3, 2026

Copy link
Copy Markdown

Soffi AI Summary

This PR refactors the insight display UI for stacks, stack runs, and flow alerts to consolidate header metadata (title, last-updated timestamp, and action buttons) into the shared InsightDisplay component itself, rather than duplicating that layout in each consumer.

The key motivation is DRY-ness and visual consistency: previously, StackInsights, StackRunInsights, and AlertInsight each maintained their own separate header row containing the "Insight" label, a last-updated timestamp, and action buttons (refresh, send to workbench, suggest fix). This PR introduces a headerActions prop on InsightDisplay that, when provided, switches the component into a "compact header" mode — embedding the title, timestamp, and any action nodes directly in the insight panel header. This eliminates the redundant outer header markup in each consumer and ensures all insight views share the same header treatment.

Secondary changes include making action buttons (AISuggestFix, SendInsightToWorkbenchButton, InsightRefresh, IconFrameRefreshButton) accept a small or size prop so they render correctly within the tighter integrated header, and minor CSS fixes to the ContentHeaderSC styled component (flexible height, minHeight guard, minWidth: 0 to prevent overflow).

Commits

Commit Summary
b3ca2a3 Fixes the insight header layout for flow alerts by passing action buttons through the new headerActions prop on InsightDisplay, removing a duplicated outer header section in AlertInsight.
3211416 Refactors StackInsights and StackRunInsights to use the new headerActions prop on InsightDisplay, consolidating the title, last-updated timestamp, and action buttons (refresh, send to workbench, suggest fix) into the shared component header and removing the now-redundant standalone header rows in each consumer. Also adds $compact mode to ContentHeaderSC for flexible height and adjusts action button components to accept a small prop.

Deploy in Soffi


Updated: 2026-08-03 08:45 UTC

@maciaszczykm
maciaszczykm marked this pull request as ready for review August 3, 2026 08:45
@maciaszczykm
maciaszczykm requested a review from a team August 3, 2026 08:45
@maciaszczykm maciaszczykm added the enhancement New feature or request label Aug 3, 2026
@greptile-apps

greptile-apps Bot commented Aug 3, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

The PR consolidates stack insight metadata and actions into the insight panel header and resolves flow names to canonical IDs before presenting flow-scoped workbench actions.

  • Adds compact insight headers with last-updated metadata and embedded actions.
  • Supports configurable refresh-icon sizing and button props for compact controls.
  • Resolves flow route parameters before passing a flow ID to the workbench action.

Confidence Score: 5/5

The PR appears safe to merge with no concrete blocking or independently actionable non-blocking defects identified.

The updated components preserve existing insight actions and refresh behavior while relocating their presentation, and flow workbench actions now receive the resolved canonical flow ID.

Important Files Changed

Filename Overview
assets/src/components/ai/insights/InsightDisplay.tsx Adds an optional compact header containing update metadata, evidence controls, and caller-provided actions; no concrete defect was established.
assets/src/components/stacks/insights/StackInsights.tsx Moves stack insight refresh, workbench, and suggested-fix controls into the shared insight header.
assets/src/components/stacks/run/insights/StackRunInsights.tsx Moves stack-run insight actions into the shared header while retaining run-level refetch behavior.
assets/src/components/utils/alerts/AlertInsight.tsx Resolves flow names or IDs to the canonical flow ID and waits for that resolution before exposing the flow-scoped workbench action.
assets/src/components/ai/chatbot/AISuggestFix.tsx Allows callers to pass design-system button props for compact presentation while preserving the panel-opening click handler.
assets/src/components/utils/RefreshIconFrame.tsx Adds a typed configurable icon size while preserving the existing large default.

Reviews (1): Last reviewed commit: "update stack insight display" | Re-trigger Greptile

@maciaszczykm
maciaszczykm merged commit 02051a3 into master Aug 3, 2026
15 of 17 checks passed
@maciaszczykm
maciaszczykm deleted the marcin/prod-5079-send-insights-to-workbenches branch August 3, 2026 15:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants