Reapply "fix(web): demote Plugins and Integrations to nav rail footer (#1806)" (#2360)

[lefarcen]

This reverts #2360 (commit 1ab87580), which had itself reverted #1806. The end result is to re-apply #1806's nav-rail / hero changes on top of current main.

Why

#2360 rolled back #1806 to put Plugins / Integrations back in the primary nav-rail group, restore the avatar gear-button dropdown, and bring back the pre-#1806 HomeHero output-type chip layout. After living with that rollback for a day the previous decision still stands: Plugins and Integrations belong in the nav-rail footer, the gear button should open the Settings dialog directly without an intermediate dropdown, and HomeHero reads better with the demoted chip row. Rather than cherry-pick the pieces back, this PR reverts #2360 wholesale and reapplies the original #1806 design.

What users will see

• Plugins and Integrations move back to the nav-rail footer (below the divider), no longer in the primary nav-rail group.
• The gear button reopens the Settings dialog directly. The intermediate dropdown menu (Language, Appearance, Follow on X, Join Discord, Use everywhere, Settings) is removed again; Language and Appearance live in the Settings dialog, and the Discord / Use everywhere / model-switcher chips remain in the topbar.
• HomeHero returns to the demoted output-type chip layout from fix(web): demote Plugins and Integrations to nav rail footer #1806.
• Unit-test and e2e expectations that Revert "fix(web): demote Plugins and Integrations to nav rail footer (#1806)" #2360 had rolled back are re-applied to match the restored UI.

Surface area

• UI — new page / dialog / panel / menu item / setting / empty state in apps/web or apps/desktop (including Electron menu bar)
• Keyboard shortcut — new or changed
• CLI / env var — new od subcommand or flag, new tools-dev / tools-pack / tools-pr flag, or new OD_* env var
• API / contract — new /api/* endpoint, new SSE event, or changed shape in packages/contracts
• Extension point — new entry under `skills/`, `design-systems/`, `design-templates/`, or `craft/`, or change to the skills protocol
• i18n keys — added new translation keys
• New top-level dependency — adding any new entry to the root `package.json`
• Default behavior change — changes what existing users experience without opting in (default model, default setting, file/SQLite schema, auto-network on startup, auto-install)
• None — internal refactor, docs, tests, or translation update only

Screenshots

Verified locally under namespace `revert-2360` (http://127.0.0.1:17927) and compared side-by-side against a fresh checkout of `main`. Reviewers can reproduce with:

```
pnpm tools-dev run web --namespace revert-2360 --daemon-port 17926 --web-port 17927
```

Bug fix verification

N/A — this is a PR-level revert (of the #2360 rollback), not a targeted bug fix backed by a red spec.

Validation

• `pnpm guard` — pass
• `pnpm typecheck` — pass (whole repo)
• `pnpm --filter @open-design/web test` — pass (193 test files / 1783 tests)
• Local `pnpm tools-dev run web --namespace revert-2360 --daemon-port 17926 --web-port 17927` started, hand-compared against a `main` checkout.

Conflict resolution notes

Reverting #2360 hit one conflict file: `apps/web/src/components/EntryShell.tsx`, three blocks. Four PRs (#2285, #2330, #2390, #2386) had landed on top of #2360 and added analytics imports, an `onDeleteTemplate` prop, onboarding telemetry, and a Discord topbar badge. Resolution:

• Imports — keep the post-Revert "fix(web): demote Plugins and Integrations to nav rail footer (#1806)" #2360 `useAnalytics`, `trackHomeNavClick` / `trackHomeToolbarClick` / `trackPageView`, `clearOnboardingSessionId` / `getOrCreateOnboardingSessionId`, and `Tracking*` type imports added by feat(analytics): ship PostHog v2 event schema #2285 / feat(analytics): unify page_name + onboarding/design-system page_views #2390, and collapse the wide `{ LOCALE_LABEL, LOCALES, useI18n, useT, type Locale }` import back to `{ useT }` (the rest was only used by the now-removed dropdown).
• Hook block — keep `const analytics = useAnalytics();` (still consumed by `trackHomeNavClick` in `changeView` and `trackHomeToolbarClick` on the Use everywhere chip), drop `avatarMenuRef` / `useGithubStars` / `modelSummary`.
• `avatarMenu` JSX — take the simple `settings-icon-btn` from the pre-Revert "fix(web): demote Plugins and Integrations to nav rail footer (#1806)" #2360 side. This intentionally drops the in-dropdown `trackHomeToolbarClick(..., element: 'settings')` call that feat(analytics): ship PostHog v2 event schema #2285 had hung off the dropdown's Settings row; there's no equivalent slot on the icon-only button. The `use_everywhere` toolbar telemetry, the nav-rail `home_nav_click` telemetry, and the OnboardingView `page_view` telemetry are all preserved.

Outside the conflict blocks, all four later PRs' additions (#2330 `onDeleteTemplate` prop, #2390 OnboardingView telemetry, #2386 topbar Discord badge, #2285 `navElementForView` helper) auto-merge cleanly and are kept.

Diffstat matches #1806 in shape (16 files, +546 / -644 vs. #1806's +633 / -545), confirming this is a faithful re-apply of #1806 on top of the four follow-up PRs.

[github-actions]

Visual regression review

Head: 134fb62 · Base: a5e43ae

Some cases used the nearest available ancestor baseline instead of the exact base SHA.

1 changed · 0 unchanged · 0 missing baseline · 0 failed

Changed cases

Case	Main	PR	Diff
visual-home baseline 5 commit(s) behind

Visual diff is advisory only and does not block merging.