Push non-primary emails to Lobbyside (other_emails column) — take 2

[TropicolX]

Summary

Re-introduces the other_emails Lobbyside custom column (reverted in #3870), without the change that caused the incident.

Root cause of the revert: #3869 added email-addresses to the include on GET /users/current. That endpoint has no such serializer relationship (EmailAddress is an ActiveModel wrapper, not an AR model), so the request errored and the awaited current-user/auth bootstrap failed site-wide.

This PR instead:

• Loads email addresses via the dedicated GET /api/v1/email-addresses endpoint — the same call the institution flow already uses in prod.
• Runs it fire-and-forget + .catch() in syncCurrentUser, off the awaited path, so a failure there can never break auth again. Worst case is an empty column.
• Re-adds the pure other_emails derivation (excludes primary, dedupes, comma-joins) + setVisitor wiring + {{did-update}} dep.

Verification (the part skipped last time)

• Regression test: forces /email-addresses to 500 and asserts the current user still loads.
• Test that currentUser.emailAddresses populates from the dedicated endpoint.
• 4 unit tests for the derivation.
• Live-checked the deployed endpoint on prod with a real session — returns the account's full email list for a normal user, no institution gating.
• Lint + template-lint + Glint + all 15 tests green.

Test plan

• filtered ember tests green
• Confirm OTHER_EMAILS column populates on the Vercel preview before merge

Made with Cursor

[codecov]

Bundle Report

Changes will decrease total bundle size by 1.33kB (-0.0%) ⬇️. This is within the configured threshold ✅

Detailed changes

Bundle name	Size	Change
client-array-push	39.16MB	-1.33kB (-0.0%) ⬇️

Affected Assets, Files, and Routes:

view changes for bundle: client-array-push

Assets Changed:

Asset Name	Size Change	Total Size	Change (%)
assets/chunk.*.js	-82 bytes	41.53kB	-0.2%
assets/chunk.*.js	-1.24kB	3.43MB	-0.04%

Files in assets/chunk.*.js:

• ./services/authenticator.ts → Total Size: 7.08kB

[github-actions]

Test Results

  1 files  ±0    1 suites  ±0   6m 15s ⏱️ - 3m 42s
722 tests +6  659 ✅ +6  63 💤 ±0  0 ❌ ±0 
737 runs  +6  674 ✅ +6  63 💤 ±0  0 ❌ ±0 

Results for commit fd841fe. ± Comparison against base commit 0f74fce.

♻️ This comment has been updated with latest results.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ All tests successful. No failed tests found.

📢 Thoughts on this report? Let us know!

[TropicolX]

bugbot run

[cursor]

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

^{Reviewed by Cursor Bugbot for commit fd841fe. Configure here.}