feat(dashboard): require workflow selection before loading runs - #42363
Merged
Conversation
Add a workflow selector to the Runs tab so logs are only fetched for the chosen workflow instead of all workflows at once. - Add selectedWorkflowFilter state (empty = none selected) - fetchRuns() is a no-op when no workflow is selected (clears runs) - selectWorkflowFilter() sets the filter and triggers fetchRuns() - init() no longer fetches runs on startup; user picks a workflow first - buildLogsCommand() includes the workflow name when one is selected - /api/runs now accepts workflow_name query param and forwards it to getRuns() - Runs tab shows a prompt to select a workflow when none is chosen - workflow_name column removed from run rows (redundant with the filter) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Contributor
Comment MemoryNote This comment is managed by comment memory.It stores persistent context for this thread in the code block at the top of this comment.
|
Contributor
There was a problem hiding this comment.
Pull request overview
This PR improves the Agentic Workflows Dashboard “Runs” tab performance by requiring the user to select a single workflow before runs are fetched, avoiding expensive “download logs for every workflow” behavior on initial load and tab switches.
Changes:
- Added a workflow
<select>control to the Runs tab and gated run rendering/fetching onselectedWorkflowFilter. - Updated the frontend run-fetch request to include
workflow_name, and updated the server endpoint to forward that filter togetRuns(). - Simplified the runs table row display by removing the per-row workflow name column (redundant when filtered).
Show a summary per file
| File | Description |
|---|---|
| .github/extensions/agentic-workflows-dashboard/web/index.html | Adds workflow selector UI and gates run list rendering until a workflow is chosen. |
| .github/extensions/agentic-workflows-dashboard/web/app.js | Implements selected-workflow gating in the compiled web bundle and passes workflow_name when fetching runs. |
| .github/extensions/agentic-workflows-dashboard/src/app.ts | Implements selected-workflow gating in the TypeScript source and updates the suggested gh aw logs command. |
| .github/extensions/agentic-workflows-dashboard/extension.mjs | Extends /api/runs to accept workflow_name and forward it to getRuns(). |
Review details
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 4/4 changed files
- Comments generated: 2
- Review effort level: Low
Comment on lines
+318
to
+324
| if (!this.selectedWorkflowFilter) { | ||
| this.runs = []; | ||
| this.runsMeta = null; | ||
| this.selectedRun = null; | ||
| this.loadRunPage(1); | ||
| return; | ||
| } |
Comment on lines
+3490
to
+3496
| if (!this.selectedWorkflowFilter) { | ||
| this.runs = []; | ||
| this.runsMeta = null; | ||
| this.selectedRun = null; | ||
| this.loadRunPage(1); | ||
| return; | ||
| } |
…x compat The Copilot extension sandbox does not support the /promises sub-path export. Replace the import with promisify() over the callback execFile from node:child_process, which the sandbox allows. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Without detached:true, Node.js spawns gh-aw.exe in the same process group as the extension sandbox process. On Windows, this causes gh-aw to hang indefinitely regardless of stdio settings. Setting detached:true creates a new process group (CREATE_NEW_PROCESS_GROUP) which lets gh-aw initialize properly and exit. Also ensures app.js is rebuilt when dashboard-cli.ts changes. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
This was referenced Jun 30, 2026
Contributor
|
🎉 This pull request is included in a new release. Release: |
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 join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
Summary\n\nThis PR makes workflow selection mandatory before run data is fetched in the Agentic Workflows Dashboard. It also ships two platform-compatibility fixes for Windows and restricted sandb