UI: Fix custom tabs not showing in the manager

[ndelangen]

Closes #25322
Closes: #25758

What I did

• change Tabs type addon to always render via a query parameter &tab={tabId}
• Fix memoization of multiple layers

Checklist for Contributors

Testing

The changes in this PR are covered in the following automated tests:

• stories
• unit tests
• integration tests
• end-to-end tests

Manual testing

• Open storybook
• expect to see no tabs
• Add a custom tab type addon in .storybook/manager.ts
• Restart + open storybook
• expect the "canvas"-tab + any tabs manually configured to show up
• Clicking between both tabs show should you the content of canvas / tabs
• The URL should not contain a tab=... when the "Canvas" tab is active
• The active tan should be highlighted with a bar
• Tools should only be shown on the "Canvas" tab

Documentation

• Add or update documentation reflecting your changes
• If you are deprecating/removing a feature, make sure to update
  MIGRATION.MD

Checklist for Maintainers

• When this PR is ready for testing, make sure to add ci:normal, ci:merged or ci:daily GH label to it to run a specific set of sandboxes. The particular set of sandboxes can be found in code/lib/cli/src/sandbox-templates.ts

• Make sure this PR contains one of the labels below:

  Available labels

  • bug: Internal changes that fixes incorrect behavior.
  • maintenance: User-facing maintenance tasks.
  • dependencies: Upgrading (sometimes downgrading) dependencies.
  • build: Internal-facing build tooling & test updates. Will not show up in release changelog.
  • cleanup: Minor cleanup style change. Will not show up in release changelog.
  • documentation: Documentation only changes. Will not show up in release changelog.
  • feature request: Introducing a new feature.
  • BREAKING CHANGE: Changes that break compatibility in some way with current major version.
  • other: Changes that don't fit in the above categories.

🦋 Canary release

This PR does not have a canary release associated. You can request a canary release of this pull request by mentioning the @storybookjs/core team here.

core team members can create a canary release here or locally with gh workflow run --repo storybookjs/storybook canary-release-pr.yml --field pr=<PR_NUMBER>

[JReinhold]

Looks good to me, only a few minor comments. Also needs a migration note mentioning the URL for tabs have now changed, and that tabId is now passed into match and how to use it.

Will you do a Documentation Request too?

[ndelangen]

@JReinhold I added the migration notes, and submitted a documentation update request 👍 Thank you for the reminder!

[socket-security]

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package	New capabilities	Transitives	Size	Publisher
npm/@playwright/test@1.41.1	None	0	25.3 kB	yurys

🚮 Removed packages: npm/@playwright/test@1.36.0, npm/playwright-core@1.36.0, npm/playwright@1.36.0

View full report↗︎

[shilman]

This migration note should describe the URL structure before the change and after. The current content should be moved down to "Addon author changes" since it's really not end user facing.

[shilman]

Also we can merge this PR as is and fix it in a follow-on PR, since time is tight

[shilman]

LGTM if you split up that migration note after merge :D