[fix] Open automation runs in filtered sessions [#5971] - #6013
[fix] Open automation runs in filtered sessions [#5971]#6013huige66631 wants to merge 2 commits into
Conversation
|
@huige66631 is attempting to deploy a commit to the agenta projects Team on Vercel. A member of the Team first needs to authorize it. |
|
✅ Thanks @huige66631! This PR now meets the contribution requirements and has been reopened. A maintainer will review it soon. |
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository YAML (base), Organization UI (inherited) Review profile: CHILL Plan: Pro Plus Run ID: ⛔ Files ignored due to path filters (2)
📒 Files selected for processing (9)
📝 WalkthroughSummary by CodeRabbit
WalkthroughThe change adds automation-mode navigation from the Overview page. The Sessions route maps this mode to trigger-origin scope and initializes the scope before rendering scoped sessions. Tests and design documents describe the route behavior and validation status. ChangesAutomation session navigation
Estimated code review effort: 3 (Moderate) | ~20 minutes Mergeability Score: 🔵 Low · up to The change opens automation runs in a filtered Sessions view and restores the trigger-origin scope, but a route transition could leave stale filtering applied and show users the wrong session set; confirm scope synchronization/reset behavior and complete the required frontend lint check before merging. Sequence Diagram(s)sequenceDiagram
participant AgentOverview
participant SessionsRoute
participant SessionScopeInitializer
participant SessionsPage
AgentOverview->>SessionsRoute: navigate with mode=automation
SessionsRoute->>SessionScopeInitializer: initialize trigger scope
SessionScopeInitializer->>SessionsPage: render after scope initialization
Possibly related PRs
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✨ Finishing Touches 💡 1🛠️ Fix failing CI checks 💡
🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary
Fixes #5971
The Automation runs "View all" link previously opened the default Sessions list.
This change adds
?mode=automationand restores the existing trigger-originsession scope before the Sessions page renders.
The regular Sessions link remains unchanged. Missing, unknown, and repeated
modevalues keep the existing behavior.Testing
Verified locally
git diff --check.200 {"status":"ok"}./sessions?mode=automation.Added or updated tests
QA follow-up
Demo
Checklist
Contributor Resources