Skip to content

Fix Studio Team member route context#601

Merged
potter-sun merged 2 commits into
devfrom
feat/2026-05-09_studio-team-member-links
May 9, 2026
Merged

Fix Studio Team member route context#601
potter-sun merged 2 commits into
devfrom
feat/2026-05-09_studio-team-member-links

Conversation

@potter-sun
Copy link
Copy Markdown
Collaborator

What changed

  • Preserve Team context when Console Web opens Studio from Team list/detail pages.
  • Keep backend Team member identity as the primary Studio route identity when it exists.
  • Use workflow/script identities as focus so asset deep links do not replace the Team member context.
  • Preserve teamId through Studio bind/invoke/member switching and return links.
  • Fix a mobile Team page layout issue found during design review: document-mode pages no longer lose 80px of horizontal content width on small screens, and Team roster view toggles now meet the 44px touch target.

Why

Issue #585 needs Studio links to stay scoped to the selected Team and member. Without this, users can enter Studio from a Team surface and lose the Team/member context, which makes the route look correct at the Scope level but wrong at the product level.

Impact

  • Console Web only.
  • No backend changes.
  • No CLI frontend changes.
  • Team/Studio route behavior becomes consistent across Team home, Team detail, asset links, Studio lifecycle transitions, and return navigation.

Validation

  • npm --prefix apps/aevatar-console-web run tsc
  • npm --prefix apps/aevatar-console-web run jest -- --selectProjects jsdom --runTestsByPath src/pages/teams/home.test.tsx src/pages/teams/detail.test.tsx src/pages/studio/index.test.tsx src/shared/studio/navigation.test.ts --runInBand
  • npm --prefix apps/aevatar-console-web run jest -- --runTestsByPath src/shared/studio/navigation.test.ts --runInBand
  • npm --prefix apps/aevatar-console-web run jest -- --selectProjects jsdom --runTestsByPath src/pages/teams/home.test.tsx src/pages/teams/detail.test.tsx --runInBand
  • git diff --check
  • /design-review browser screenshot verification for Team home/detail mobile layout

Closes #585

@codecov
Copy link
Copy Markdown

codecov Bot commented May 9, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 72.01%. Comparing base (65e4859) to head (ee088ab).

@@            Coverage Diff             @@
##              dev     #601      +/-   ##
==========================================
- Coverage   72.02%   72.01%   -0.01%     
==========================================
  Files        1255     1255              
  Lines       90723    90723              
  Branches    11877    11877              
==========================================
- Hits        65343    65337       -6     
- Misses      20699    20703       +4     
- Partials     4681     4683       +2     
Flag Coverage Δ
ci 72.01% <ø> (-0.01%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 3 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@potter-sun potter-sun marked this pull request as ready for review May 9, 2026 07:16
@potter-sun potter-sun requested a review from AbigailDeng as a code owner May 9, 2026 07:16
@potter-sun potter-sun merged commit 36f96f7 into dev May 9, 2026
13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Console web: keep Studio links scoped by teamId and memberId

1 participant