refactor(web): redesign Select component and migrate WorkplaceSelector

[lyzno1]

Summary

Redesign the base/ui/select component to align with the project's established DropdownMenu pattern and shadcn/ui best practices, while migrating WorkplaceSelector from Headless UI to fix focus and semantics issues.

SelectTrigger

• Remove clearable, onClear, loading, and variant (destructive) boolean props — these are business behaviors that don't belong in a base component (architecture-avoid-boolean-props)
• Remove class-variance-authority dependency; use a simple Record for size variants
• Keep size (small / regular / large) as a pure layout dimension
• Consolidate trigger + content padding into trigger-level size classes
• Built-in chevron icon via BaseSelect.Icon

SelectItem → pass-through (composition pattern)

• Change from compound (ItemText + ItemIndicator baked in) to children pass-through, consistent with DropdownMenuItem
• Export SelectItemText and SelectItemIndicator as composable styled wrappers, following DropdownMenuRadioItemIndicator pattern
• Consumers compose: <SelectItem><SelectItemText>…</SelectItemText><SelectItemIndicator /></SelectItem>

SelectGroupLabel / SelectSeparator

• Convert from bare re-exports to styled wrappers reusing menuGroupLabelClassName / menuSeparatorClassName from menu-shared.ts

WorkplaceSelector migration

• Migrate from Headless UI Menu to base/ui/select, fixing the focus ring bug (outline-hidden + :focus-visible) and correcting menu → listbox semantics
• Override SelectTrigger defaults via className for the custom trigger appearance
• Use SelectItem + SelectItemText composition for custom item layout (avatar + name + PlanBadge)

Callers

• Update all 5 production callers to use the new SelectItemText + SelectItemIndicator composition API

Tests

• Remove 8 clearable/loading/destructive tests
• Add custom composition test (SelectItem without indicator)
• All 18 tests pass

From Cursor

Checklist

• This change requires a documentation update, included: Dify Document
• I understand that this PR may be closed in case there was no previous discussion or issues. (This doesn't apply to typos!)
• I've added a test for each change that was introduced, and I tried as much as possible to make a single atomic change.
• I've updated the documentation accordingly.
• I ran make lint && make type-check (backend) and cd web && pnpm exec vp staged (frontend) to appease the lint gods

Made with Cursor

[codecov]

Codecov Report

❌ Patch coverage is 87.09677% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 85.10%. Comparing base (8baa864) to head (f7d8e3d).
⚠️ Report is 10 commits behind head on main.

Files with missing lines	Patch %	Lines
web/app/components/base/ui/select/index.tsx	55.55%	4 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##             main   #35293   +/-   ##
=======================================
  Coverage   85.10%   85.10%           
=======================================
  Files        4399     4399           
  Lines      201355   201344   -11     
  Branches    38299    38292    -7     
=======================================
- Hits       171365   171356    -9     
+ Misses      26886    26884    -2     
  Partials     3104     3104           

Flag	Coverage Δ
web	86.23% <87.09%> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 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.