QA Session Recordings

ComfyUI Frontend · Automated QA · PR #10833 · main @ 17d2870 · #10833 @ 311c492 · QA @ 3bc88f5 · CI Job · 2026-04-03T10:03:43+09:00 → 2026-04-03 01:17 UTC
QA Badge
PR #10833 aims to
fix: show file extensions in workflow sidebar
## Summary - Workflow sidebar now displays file extensions (.json, .app.json) instead of stripping them - Uses `node.key` to derive leaf labels, bypassing PrimeVue Tree's label mutation - Removes unused `getFilenameDetails` import
Test focus: Verify that file names in the Workflows sidebar now include their full extensions (e.g., '.json').
Prerequisites: Ensure at least one saved workflow exists in the workspace (e.g., 'my-workflow.json')
Steps: Click the Workflows tab to open the sidebar → Wait for the workflow tree to render → Take a screenshot of the Workflows sidebar
🐧LinuxReport
Before main
After PR
AI Comparative Review
# linux QA Video Report - Generated at: 2026-04-03T01:11:20.479Z - Model: `gemini-3-flash-preview` - Target: https://github.com/Comfy-Org/ComfyUI_frontend/pull/10833 - Before video: `./qa-artifacts/qa-report-Linux-23929150579/qa-before-session.mp4` (3.3 MB) - After video: `./qa-artifacts/qa-report-Linux-23929150579/qa-session.mp4` (2.5 MB) - Mode: **Comparative (before/after)** ## AI Review ## Summary The PR aims to ensure that file extensions (like `.json` or `.app.json`) are displayed in the workflow sidebar instead of being stripped. This is achieved by using the file's key (derived from its path) to determine the label for leaf nodes in the sidebar tree. **Evaluation:** - **BEFORE video**: Does **not** demonstrate the old behavior or the bug. The user opens the "Templates" modal and saves a workflow, but never opens the "Workflows" sidebar tab where the file list (and thus the bug) would be visible. - **AFTER video**: Does **not** prove the fix. Similar to the BEFORE video, the user interacts with the "Templates" modal and the main graph but fails to open the "Workflows" sidebar to verify the file labels. - **Result**: The videos are **inconclusive** as they do not show the UI component affected by the PR (`BaseWorkflowsSidebarTab.vue`). ## Behavior Changes | Behavior | Before (main) | After (PR) | Verdict | |----------|---------------|------------|---------| | Workflow Sidebar Labels | Not demonstrated; sidebar was not opened. | Not demonstrated; sidebar was not opened. | No Change (Inconclusive) | | File Saving | User saves a workflow as "my-workflow". | Not performed in this video. | No Change | | Templates Modal | Modal opens and displays template categories/cards. | Modal opens and displays template categories/cards. | No Change | ## Timeline Comparison | Time | Type | Severity | Before (main) | After (PR) | |------|------|----------|---------------|------------| | 0:05-0:09 | Behavior | None | User opens the "Templates" modal (4th icon in sidebar) and closes it. | User opens the "Templates" modal and closes it. | | 0:11-0:14 | Behavior | None | User opens the menu and performs "Save As" for a workflow. | User hovers over the "Unsaved Workflow" tab but does not perform any action. | | 0:14-0:19 | Behavior | None | Tab name updates to "my-workflow". | Video ends. | ## Confirmed Issues None observed, as the relevant UI component was not shown. --- ## Possible Issues (Needs Human Verification) ### Missing Visual Verification of Fix `MINOR` `0:00-0:14` `Confidence: HIGH` The PR specifically modifies the `BaseWorkflowsSidebarTab.vue` component. To verify this change, the "Workflows" sidebar (the top icon in the left sidebar) needs to be opened to see the list of saved files. Neither video performs this step, focusing instead on the "Templates" modal (a different UI element). The "Visual verification" step requested in the PR test plan was not captured. ## Overall Risk The PR risk is **low** because it primarily involves a string manipulation change for UI labels and is backed by a unit test (`treeUtil.test.ts`) that correctly verifies the new logic. However, the provided videos do not confirm the visual fix in the actual application environment. ## Verdict The videos provided do not demonstrate the behavior described in the PR. The "Workflows" sidebar remains closed in both recordings. {"verdict": "INCONCLUSIVE", "risk": "low", "confidence": "high"}