feat(agent): group scheduled task results by task name with collapse/expand#961
Conversation
…expand - Add ScheduledTaskResultGroup component for accordion-style grouping - Group results by job name with expand/collapse behavior - Only one group expanded at a time (accordion pattern) - Update ScheduleResults (newtab) for consistency - Each group shows: task name, latest timestamp, result count Fixes browseros-ai#950
|
PR author is not in the allowed authors list. |
|
All contributors have signed the CLA. Thank you! |
|
I have read the CLA Document and I hereby sign the CLA |
There was a problem hiding this comment.
Pull request overview
This PR improves scheduled task result browsing by grouping task runs by task name and adding accordion-style collapse/expand behavior, addressing the “flat chronological list” usability problem described in #950.
Changes:
- Added an accordion group component to display a task header (name/status/latest time/count) with an expandable run history.
- Updated the scheduled task results page to group runs by job ID and render via the new group component (single-group expansion).
- Updated the newtab “ScheduleResults” preview to use the same grouped UI and cap the display to the top 3 groups.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 6 comments.
| File | Description |
|---|---|
| packages/browseros-agent/apps/agent/entrypoints/newtab/index/ScheduleResults.tsx | Switches newtab results from a flat list to grouped, accordion-style task result groups. |
| packages/browseros-agent/apps/agent/entrypoints/app/scheduled-tasks/ScheduledTaskResults.tsx | Switches the main app results list to grouped rendering and wires in single-group expansion state. |
| packages/browseros-agent/apps/agent/entrypoints/app/scheduled-tasks/ScheduledTaskResultGroup.tsx | New reusable group component implementing the grouped header + expandable run list UI. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Ensure running task groups are always visible in newtab preview - Add fallback job name when jobs not yet loaded (prevents empty state flash) - Extract groupRunsByJob helper to reduce code duplication - Remove unused imports Addresses review comments on PR browseros-ai#961
|
All review comments addressed. Here's a summary of the fixes:
Tested: Screenshot shows grouped results with accordion behavior working correctly. |
|
Consolidated into #1038 which includes updated security hardening and unified context features. |
Summary
Changes
New Component:
ScheduledTaskResultGroup.tsxUpdated:
ScheduledTaskResults.tsxScheduledTaskResultGroupfor each groupUpdated:
ScheduleResults.tsx(newtab)Motivation
Scheduled task results were displayed in a flat chronological list, making it hard to find results for a specific task when multiple tasks exist. This change groups results by task name with collapse/expand, solving the "train layout" problem described in #950.
Fixes #950