refactor(ui): homogenize Account, Organization detail, and Admin pages (#4187)

* chore(git): ignore top-level .worktrees/ directory

* feat(core): add coreConfirmDialog shared destructive-confirm component

Options API dialog with simple yes/no, typed-confirmation gate, loading
mode, custom confirmColor/titleClass, and default slot override.

* feat(core): add coreAvatarUploader (v-badge camera overlay, no inline styles)

* test(core): use flushPromises in coreAvatarUploader test (cleaner than setTimeout)

* feat(admin): currentBreadcrumb state so sub-views push titles to the layout

* test(admin): split currentBreadcrumb test for clearer failure messages

* refactor(admin): banners on top, tabs in PageHeader, breadcrumb support for sub-views

* refactor(admin): resolveActiveTab uses allTabs (single source of truth)

* refactor(admin): drop v-container on list sub-views; users.view uses coreConfirmDialog

* refactor(admin): drop gratuitous div wrapper in admin.users.view (Vue 3 multi-root)

* refactor(admin): activity view — drop chrome + inline styles, use v-row/v-col + v-table hover

* refactor(admin): user detail — drop nested PageHeader, push breadcrumb, shared dialog

* refactor(users): profile component — coreAvatarUploader + native Vuetify labels, no inline styles

* refactor(users): drop dead isActiveOrg method from profile component

* refactor(admin): migrate layout to CoreSurfaceTabBar — chrome convergence with Account+Org

Drop inline v-tabs, activeTab data, $route watcher, isValidTab, tabTo, resolveActiveTab,
extraTabs computed. Replace with CoreSurfaceTabBar receiving allTabs (built-in + config extras)
and adminCan predicate. Matches user.view.vue + organization.detail.component.vue pattern.

* refactor(users): account dialogs — Delete Account + Leave Org use coreConfirmDialog

* refactor(organizations): detail — Delete Organization uses coreConfirmDialog with org-name gate

* fix(admin): remove unused beforeEach import in admin.user.view tests

* fix(admin): clear stale breadcrumb on blank user; fix org delete JSDoc

- publishBreadcrumb() now calls clearBreadcrumb() when the user record is
  null/blank, preventing stale title display during same-component navigation
  between /admin/users/:id routes (Copilot finding).
- Add test: "clears breadcrumb when user resets to blank record".
- Fix misleading JSDoc on deleteConfirmTarget() in organization.detail —
  the confirm button is guarded by v-if on the trigger, not by the empty
  confirmText fallback (Copilot finding).

* fix(review): address CodeRabbit + Copilot findings

- admin.users.view: replace inline :style binding on membership chips with
  :class="cursor-pointer" (no-inline-style convention, CR finding)
- admin.activity.view: add keyboard semantics to expandable rows (tabindex=0,
  role=button, aria-expanded, @keydown.enter+.space) + test (CR finding)
- organizations.detail layout test: fix tautological assertion for
  deleteConfirmTarget null fallback (CR finding)
- admin.store: add full JSDoc (@param/@returns) to setBreadcrumb + clearBreadcrumb
- admin.layout: add @returns {void} to clearError
- admin.user.view: add JSDoc to user watcher handler + beforeUnmount
- admin.user.view test: add JSDoc to mountView helper
- admin.layout test: add JSDoc to mountLayout helper
- core.avatarUploader: add JSDoc to triggerUpload + onFile
- core.confirmDialog: add JSDoc to data, canConfirm, modelValue watcher,
  onCancel, onConfirm; document the intentional non-self-close design
- user.profile.component: add @returns {void} to syncForm + syncOrganizations
- user.profile.component test: add JSDoc to mountProfile helper

Author: PierreBrisorgueil

.gitignore

@@ -81,3 +81,6 @@ mongod
 .github/*local.*
 .claude/worktrees/
 playwright.local.config.js
+
+# Top-level git worktrees (per superpowers convention)
+.worktrees/

src/modules/admin/stores/admin.store.js

@@ -55,6 +55,7 @@ export const useAdminStore = defineStore('admin', {
     users: [],
     organizations: [],
     error: null,
+    currentBreadcrumb: null,
     readiness: [],
     auditLogs: [],
     auditTotal: 0,
@@ -124,6 +125,25 @@ export const useAdminStore = defineStore('admin', {
       this.user = defaultUser();
     },
 
+    /**
+     * @desc Set (or clear) the current breadcrumb published by an admin sub-view.
+     *       A shallow copy is stored so callers cannot mutate the store state directly.
+     *       Pass `null` to clear (identical effect to `clearBreadcrumb()`).
+     * @param {{ title: string, titleClass?: string } | null} payload - Breadcrumb data, or null to clear.
+     * @returns {void}
+     */
+    setBreadcrumb(payload) {
+      this.currentBreadcrumb = payload ? { ...payload } : null;
+    },
+
+    /**
+     * @desc Clear the current breadcrumb (reset to null). Called by admin sub-views on unmount.
+     * @returns {void}
+     */
+    clearBreadcrumb() {
+      this.currentBreadcrumb = null;
+    },
+
     /**
      * @desc Fetch SaaS readiness checklist from the admin API.
      * @returns {Promise<void>}

src/modules/admin/tests/admin.activity.view.unit.tests.js

@@ -181,3 +181,44 @@ describe('admin.activity.view', () => {
     expect(getAuditLogs).not.toHaveBeenCalled();
   });
 });
+
+import { readFileSync } from 'fs';
+import { fileURLToPath } from 'url';
+import { dirname, resolve } from 'path';
+
+describe('admin.activity.view — template chrome', () => {
+  it('does NOT wrap its template in <v-container> (admin layout owns chrome)', () => {
+    const here = dirname(fileURLToPath(import.meta.url));
+    const sfc = readFileSync(resolve(here, '../views/admin.activity.view.vue'), 'utf8');
+    const tmpl = sfc.split('<script>')[0];
+    expect(tmpl).not.toMatch(/<v-container/);
+  });
+
+  it('has zero inline style="…" attributes in its template', () => {
+    const here = dirname(fileURLToPath(import.meta.url));
+    const sfc = readFileSync(resolve(here, '../views/admin.activity.view.vue'), 'utf8');
+    const tmpl = sfc.split('<script>')[0];
+    // Allow :style="…" (dynamic bindings, none expected here) and reject plain style="…".
+    // But neither should appear in this view after the refactor.
+    expect(tmpl).not.toMatch(/\bstyle\s*=\s*"/);
+    expect(tmpl).not.toMatch(/:style\s*=\s*"/);
+  });
+
+  it('uses v-table with the hover prop for row affordance', () => {
+    const here = dirname(fileURLToPath(import.meta.url));
+    const sfc = readFileSync(resolve(here, '../views/admin.activity.view.vue'), 'utf8');
+    const tmpl = sfc.split('<script>')[0];
+    expect(tmpl).toMatch(/<v-table[^>]*\bhover\b/);
+  });
+
+  it('expandable rows have keyboard semantics (tabindex, role, aria-expanded, keydown handlers)', () => {
+    const here = dirname(fileURLToPath(import.meta.url));
+    const sfc = readFileSync(resolve(here, '../views/admin.activity.view.vue'), 'utf8');
+    const tmpl = sfc.split('<script>')[0];
+    expect(tmpl).toMatch(/tabindex="0"/);
+    expect(tmpl).toMatch(/role="button"/);
+    expect(tmpl).toMatch(/aria-expanded/);
+    expect(tmpl).toMatch(/@keydown\.enter/);
+    expect(tmpl).toMatch(/@keydown\.space/);
+  });
+});