Improve Workflow Studio hierarchy and publish status recovery#2882
Open
AbigailDeng wants to merge 2 commits into
Open
Improve Workflow Studio hierarchy and publish status recovery#2882AbigailDeng wants to merge 2 commits into
AbigailDeng wants to merge 2 commits into
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. @@ Coverage Diff @@
## dev #2882 +/- ##
=======================================
Coverage 83.44% 83.45%
=======================================
Files 1376 1376
Lines 112700 112700
Branches 14365 14365
=======================================
+ Hits 94048 94052 +4
+ Misses 12099 12096 -3
+ Partials 6553 6552 -1
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
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.
Problem and solution
The Workflow Studio packed identity, status, and low-frequency commands into one crowded header; the graph controls competed with the minimap; the node inspector obscured the usable canvas; and several labels could not recover their full text. The page also retained a transient publish
Errorafter an authoritative refresh reported no active or completed binding.This change:
dataas a fresh response;lgbreakpoint for bottom-sheet layout and canvas inset behavior, and exposes async publish status changes through a polite live region.Impacted paths
apps/aevatar-console-web/src/pages/team-member-workflow-studio/**apps/aevatar-console-web/src/shared/graphs/**apps/aevatar-console-web/src/shared/navigation/**apps/aevatar-console-web/src/app.navigation.test.tsapps/aevatar-console-web/src/global.lessapps/aevatar-console-web/src/locales/en-US.tsNo backend, root configuration, CI, or documentation files changed.
Validation
4b23102d2: changes, FKST host policy, fast gates, console-web, and coverage quality passed; backend-heavy jobs skipped as expected for a frontend-only diff.git diff --check: passed.The clean
origin/devbaseline currently reports 82 Biome errors, 83 warnings, and 11 infos. Per explicit user approval, changed-file Biome is the lint evidence for local validation and pre-commit; TypeScript, Jest, build, hooks, browser QA, and PR checks remain mandatory. No hook was bypassed and--no-verifywas not used.Browser QA
Authenticated in-app browser against the hosted backend through
http://127.0.0.1:5173:My Teams -> xiaomi home -> Workflowand confirmed the six-node workflow reportsPublished, with no staleError.1280x720: inspector, controls, and minimap do not overlap; no horizontal page overflow.390x844: six nodes load and Fit View places all six inside the canvas; identity/actions and controls/minimap do not overlap; no horizontal page overflow.820x844and390x844: the bottom inspector remains within the editor, its header and Close action remain visible, its body retains internal scrolling, and the page has no horizontal overflow.Screenshots:
/Users/xiezixin/.fkst/artifacts/aevatar-frontend/20260720T072338Z-be8fedbe/screenshots/workflow-after-origin-dev-sync-desktop.png/Users/xiezixin/.fkst/artifacts/aevatar-frontend/20260720T072338Z-be8fedbe/screenshots/workflow-after-origin-dev-sync-mobile.pngLocal browser QA is evidence only. The repository does not currently provide a GitHub-hosted frontend browser E2E check.
Recurrence prevention
Error.refetch()could resolve withisError: trueplus old cacheddata.dataas fresh without first validating the refetch result..refetch()call sites; one separate pre-existing directservicesResult.dataconsumer remains insrc/pages/studio/index.tsxand is intentionally outside this PR.FKST provenance
/Users/xiezixin/.fkst/runtime/aevatar-frontend/manual/fix-2026-07-20_workflow-studio-visual-hierarchyfix/2026-07-20_workflow-studio-visual-hierarchyorigin/dev@248c4e2ecb4387a4e8fc94de1414a3fc32f50e0420260720T072338Z-be8fedbe