[obvious-fleet] Faster, more robust src/app/app: benchmark harness, measured optimizations, hardened inputs

[obvious-autobuild]

fleet-packet: genp-connvo-src-app-app

Status: IN PROGRESS — benchmark harness scaffolded, part branches landing

What changed and why

src/app/app/ (the authenticated app shell: layout/providers, dashboard, profile form, settings, meeting summary, match queue, professional queue pages) is being made measurably faster and more robust without changing normal-path behavior.

This first commit adds the measurement infrastructure:

• src/app/app/_lib/ (Next.js private folder, never routed): a before/after benchmark harness. npm run bench:app runs every suite, printing medians over interleaved repeated samples plus the machine load average; --json for machine-readable output.
• Protocol docs (src/app/app/_lib/README.md): every optimization is extracted into a pure helper that is equivalence-tested against a frozen verbatim copy of the original inline code, and benchmarked original-vs-optimized on the same machine.
• jsdom added as a devDependency so the vitest frontend project can run tests for src/ code (task-permitted toolchain install).

Part branches (obv/genp-connvo-src-app-app--<part>) add per-page optimizations and robustness fixes; the lead merges them into this branch and reruns the full verification.

How verified

• Baseline on main: npx tsc --noEmit = 29 pre-existing errors, 0 in src/app/app/ (errors live in src/hooks/, src/components/ui/chart.tsx, src/app/videocall/, test/convex/*); npx vitest run = 240/240 tests pass with 1 pre-existing teardown error in convex/prompts/prompts.test.ts after tests complete (vitest exit 1 on main for that reason alone); npm run lint is broken on main (next lint was removed in Next 16 and the legacy .eslintrc.json path fails under eslint-config-next 16) — pre-existing, untouched per scope.
• Harness smoke test: npm run bench:app prints load average and an empty table (all suites stubbed).
• Final numbers land here when part merges finish.

Not verified yet

• Part merges, final benchmark table, equivalence + fail-first test results.

Assumptions

• Adding jsdom (devDependency) and the bench:app npm script count as permitted toolchain/install, not dependency upgrades.
• The repo's OpenSpec process (openspec/AGENTS.md) asks for proposals on "optimize performance (changes behavior)" work; this run is explicitly behavior-preserving and autonomous, so no OpenSpec change proposal is filed.

[vercel]

The latest updates on your projects. Learn more about Vercel for GitHub.

Project	Deployment	Actions	Updated
connvo	Error		Oct 9, 2026 7:55pm UTC

[coderabbitai]

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

• Configuration used: Organization UI
• Review profile: CHILL
• Plan: Advanced
• Run ID: 631dd8d6-dc50-46a0-b622-a94389e151fe

---

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

• 🔍 Trigger review

• Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

---

_{Comment @coderabbitai help to get the list of available commands.}