revert(org): hold the org view and People redesigns for #9687

Merged
WA
2d ago
+6880-32229

Summary

Takes the redesigned organization view (#9708), the People page redesign (#9720) and its landing test fix (#9715) off staging, so main can keep releasing while the new UI ships all at once with #9687, which already carries all three.

After this, staging's /o UI matches main except 6 integration files that carry functional work later PRs added to the legacy view:

  • Multiple accounts per integration (#9716).
  • Jama and Flow saved credentials (#9723). Connect and Reconnect on the legacy detail page now open the saved-credential modal, and Disconnect goes through Search, since #9723 wired those only into the new view.

Only views are reverted. Hooks, contracts, key factories and lib helpers stay at staging's versions, and the legacy view adapts to them. Exports that only the redesigned views used are unexported where their own file still uses them, or deleted, so the unused-exports baseline only shrinks (2 entries).

Three API routes go: Home calendar, Home email and search-history sources. Only the new view called them, and none is on main. No reverted commit has a migration or schema change.

Merging this into #9687

#9687 already contains these commits, so a plain staging merge there would delete the org view (257 conflicting files in a dry run). After this merges, #9687 takes staging in three steps:

  1. Merge staging up to this commit's parent.
  2. git merge -s ours <this squash commit>.
  3. Merge the rest of staging.

A simulation changed 0 org-view files on #9687 and brought in every other staging commit.

Type of Change

  • Other: revert, held for #9687

Testing

  • type-check passes.
  • check:test-types, explicit-any, skills, biome, test-patterns, boundaries, react-query, source-conventions, file-names, utils, client-boundary, comment-hygiene, api-validation:strict, guidance-refs and unused-exports all pass. The unused-exports baseline shrank by one entry.
  • Focused vitest: 351 files, 2,830 tests pass, covering app/o, all of hooks/, credential groups, org lib, search, People, _shell and workspace Home.
  • test-assistant-actions-e2e.ts is back at main's legacy-view selectors.
  • Needs a staging check of /o Home, Integrations (including Jama or Flow connect) and People after merge.