UI: Surgically implement CSS and motion audit recommendations (Antigravity)

[BigSimmo]

This PR safely implements the outstanding recommendations from the animation and UI motion audit, surgically targeting only the necessary CSS files without touching API routes or test files.

• ISSUE-02: Halts the Therapy Compass rotational animation entirely when prefers-reduced-motion is active.
• ISSUE-05: Mirrors the html[data-motion="reduced"] attribute into the Therapy Compass component, honoring the in-app preference.
• IMP-01: Upgrades the clinician knob transition easing to the tactile spring cubic-bezier.
• IMP-02 & IMP-04: Introduces a GPU-composited sweep shimmer and a staggered card entrance utility to globals.css.

Zero regressions were verified via a full vitest test suite pass, typecheck, and lint pass.

Summary by CodeRabbit

• New Features
  • Refreshed loading visuals with smoother shimmer sweeps and staggered entrance animations.
  • Standardized skeleton segment timing for more consistent placeholder behavior.
• Accessibility
  • Improved reduced-motion handling: shimmer/stagger animations and spin effects are disabled in reduced-motion modes, with clear static fallback states.
• Bug Fixes
  • Reduced navigation/hydration race conditions by hardening UI readiness checks.
• Tests
  • Updated Playwright assertions to retry header presence and visibility together for more reliable runs.

[chatgpt-codex-connector]

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, add credits to your account and enable them for code reviews in your settings.

[supabase]

This pull request has been ignored for the connected project sjrfecxgysukkwxsowpy because there are no changes detected in supabase directory. You can change this behaviour in Project Integrations Settings ↗︎.

---

Preview Branches by Supabase.
Learn more about Supabase Branching ↗︎.

[coderabbitai]

Note

Reviews paused

It looks like this branch is under active development. To avoid overwhelming you with review comments due to an influx of new commits, CodeRabbit has automatically paused this review. You can configure this behavior by changing the reviews.auto_review.auto_pause_after_reviewed_commits setting.

Use the following commands to manage reviews:

• @coderabbitai resume to resume automatic reviews.
• @coderabbitai review to trigger a single review.

Use the checkboxes below for quick actions:

• ▶️ Resume reviews
• 🔍 Trigger review

📝 Walkthrough

Walkthrough

Skeleton loading animations, Therapy Compass reduced-motion handling, UI overlap retries, and the branch review ledger were updated with motion, CI, conflict-resolution, and review closeout records.

Changes

Motion behavior updates

Layer / File(s)	Summary
Skeleton shimmer and staggered loading src/app/globals.css, src/components/clinical-dashboard/answer-status.tsx	Skeleton shimmer now uses an animated overlay, while AnswerSkeleton segments receive staggered indices and reduced-motion guards.
Therapy Compass motion preferences src/components/therapy-compass/therapy-compass.css	The clinician knob uses the spring easing token, and reduced-motion rules disable spinning and minimize transitions for OS and application settings.

UI overlap test stabilization

Layer / File(s)	Summary
Hydration-safe search verification tests/ui-overlap.spec.ts	The home-flow check retries header#search count and visibility together during hydration and remounts.

Review ledger closeout

Layer / File(s)	Summary
Review and CI ledger closeout docs/branch-review-ledger.md	PR ledger entries record CI follow-ups, container-build guard details, motion-fix verification, conflict resolution, review completion, and duplicate cleanup.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Possibly related PRs

• BigSimmo/Database#944: Modifies the same gotoHome search-header overlap flow.
• BigSimmo/Database#1024: Modifies the Therapy Compass clinician knob motion behavior.
• BigSimmo/Database#1142: Updates global skeleton shimmer and reduced-motion animation behavior.

🚥 Pre-merge checks | ✅ 3 | ❌ 2

❌ Failed checks (2 warnings)

Check name	Status	Explanation	Resolution
Description check	⚠️ Warning	The description is on-topic, but it omits the required template sections and checklist format.	Reformat it to match the template: add Summary, Verification checkboxes, Risk and rollout, Notes, and any required Clinical Governance Preflight items.
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (3 passed)

Check name	Status	Explanation
Title check	✅ Passed	Title is concise and accurately summarizes the CSS and motion-audit UI changes.
Linked Issues check	✅ Passed	Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check	✅ Passed	Check skipped because no linked issues were found for this pull request.

---

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

[BigSimmo]

@cursoragent please resolve all CI failing tests after understanding the issue. Please also fix any comments and resolve them and also ensure that you fix with smallest required fix with best coding practice and u derstabding the fix rather than simple patch. Please fix any merge conflicts and ensure any issues are found. Please resolve any issues you find. Please also run bugbot. Please mark comments resolved also once you resolve them.

[cursor]

Taking a look!

[BigSimmo]

@codex review PR

[chatgpt-codex-connector]

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit:4d9cacd426

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[cursor]

CI / Bugbot / comments closeout

CI: Required checks on the prior tip were already green (PR required, Production UI, Static, Unit, Build). No failing product tests to repair.

Merge state: Branch was BEHIND main with a clean git merge-tree. Merged origin/main (no content conflicts). Tip is now caught up.

Bugbot (prior P2s fixed):

1. IMP-04 — shimmer-sweep is now wired on .animate-skeleton-shimmer::after (GPU translateX highlight actually runs).
2. IMP-02 — .stagger-item is applied on AnswerSkeleton bars with --stagger-index.
3. IMP-01 — clinician knob uses var(--ease-spring) instead of a hardcoded bezier.

ISSUE-02 / ISSUE-05 (Therapy Compass reduced-motion halt + html[data-motion="reduced"] mirror) retained as-is.

Review threads: None open (reviewThreads empty) — nothing to mark resolved. Bot issue comments (Codex quota / Supabase ignore / CodeRabbit rate-limit) are informational only.

Verification: focused Vitest 34/34; tsc --noEmit PASS; Bugbot re-review reports no remaining P0/P1/P2.

[coderabbitai]

Actionable comments posted: 1

🤖 Prompt for all review comments with AI agents

Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In `@src/app/globals.css`:
- Around line 2495-2507: Disable the shimmer-sweep animation for
.animate-skeleton-shimmer::after in both existing reduced-motion media-query
gates, while preserving the normal animation otherwise. Ensure AnswerSkeleton’s
pseudo-element has no running animation whenever either motion-reduction
preference is active.

🪄 Autofix (Beta)

Fix all unresolved CodeRabbit comments on this PR:

• Push a commit to this branch (recommended)
• Create a new PR with the fixes

---

ℹ️ Review info

⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: e6765eb5-ec51-4467-8ebd-1c9c443b0421

📥 Commits

Reviewing files that changed from the base of the PR and between fe09c33 and 9fcac0b.

📒 Files selected for processing (4)

• docs/branch-review-ledger.md
• src/app/globals.css
• src/components/clinical-dashboard/answer-status.tsx
• src/components/therapy-compass/therapy-compass.css

[github-actions]

CI triage

CI failed on this PR. Automated classification of the 2 failed job(s):

• Production UI — needs investigation: inspect the failing step and uploaded diagnostics; rerun only after classifying the cause.
• PR required — needs investigation: inspect the failing step and uploaded diagnostics; rerun only after classifying the cause.

Compared with main CI run #6205 (success).

Classification is evidence routing, not permission to ignore a failure. Exact quarantined Playwright identities remain governed by the flake ledger.

[cursor]

Closeout update (tip after conflict resolve)

Required CI: green on tip c5a145dd — Build, Static, Unit coverage, Production UI, Advisory UI, Safety/config, PR required all SUCCESS.

Merge conflicts: After main advanced, real conflicts appeared in Dockerfile + scripts/guard-next-build.mjs (main already landed a better RAM-gate fix). Resolved by adopting main’s evaluateNextBuildRamGuard + ALLOW_LOW_RAM_BUILD path. Branch is merge-tree clean vs origin/main.

Product fixes kept:

• ISSUE-02 / ISSUE-05 Therapy Compass reduced-motion + data-motion halt
• IMP-01 knob uses var(--ease-spring)
• IMP-04 shimmer-sweep wired on .animate-skeleton-shimmer::after
• IMP-02 .stagger-item applied in AnswerSkeleton

Bugbot: prior incomplete-wiring P2s fixed; re-review found no remaining P0/P1/P2.

Review threads: still none to resolve (bot quota/rate-limit issue comments only).

Note: a ledger closeout commit may follow this tip; required checks should re-run and stay green (docs-only).

[cursor]

Final closeout

Required CI: SUCCESS on tip 8ef0c1b2 (Build, Static, Unit, Advisory UI, Production UI, PR required).

Review threads: both resolved

1. Codex P2 shimmer wiring — already fixed (::after + shimmer-sweep)
2. CodeRabbit reduced-motion shimmer — fixed with explicit animation: none on ::after

Also fixed during babysit: main merge conflicts on RAM guard (adopted main’s ALLOW_LOW_RAM_BUILD), ledger duplicate after merge=union, and ui-overlapheader#search remount flake.

Bugbot: no remaining P0/P1/P2 on the motion product delta.

[BigSimmo]

@codex review PR

[chatgpt-codex-connector]

Codex Review: Didn't find any major issues. More of your lovely PRs please.

Reviewed commit:2becc74cd7

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

• Open a pull request for review
• Mark a draft as ready
• Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

[cursor]

Green tip confirmed

Tip f68ece86: required CI all SUCCESS (Build, Static, Unit, Advisory UI, Production UI, PR required).

Review threads: both remain resolved.

The earlier Production UI red on 2becc74c was a ui-therapy-nav-scroll timeout during scroll-reveal (flake/hang in scrollPrimarySurface); overlap suite stayed green after the gotoHome harden. Re-run on this tip passed cleanly.

[BigSimmo]

@cursoragent Inspect the PR and repository state, identify the root cause of every failing CI check, unresolved review comment, merge conflict, and related defect. Apply the smallest robust fix using existing project patterns and best practices, without unrelated changes. Run Bugbot and all relevant lint, type-check, test, build, and CI-equivalent commands, fix any issues found, then resolve only the review comments that are fully addressed. Summarise the root causes, changes made, validation results, and any remaining blockers.

[cursor]

PR #1297 is already merged into main as 3a8edb93 (2026-07-28 09:12 UTC). No open CI failures, conflicts, or unresolved review threads remain.

Root causes (historical, already addressed)

Item	Root cause	Disposition
Codex P2 shimmer unused	Keyframes added but not wired	Fixed: .animate-skeleton-shimmer::after runs shimmer-sweep
CodeRabbit reduced-motion shimmer	Infinite sweep relied only on global 0.01ms clamp	Fixed: explicit animation: none on both motion gates
Build / app-image red	Local <10 GiB RAM guard on ~7.8 GiB runners	Adopted main’s ALLOW_LOW_RAM_BUILD path
Static red	Ledger exact-duplicate after merge=union	Deduped
Production UI flake	header#search remount race; later therapy-nav scroll timeout	Hardened gotoHome; subsequent tip green

Current state

• CI on merge tip 9454c307: Build / Static / Unit / Advisory / Production UI / PR required / CircleCI verify all SUCCESS
• Review threads: both resolved
• Merge conflicts: none (merge-tree clean)
• Bugbot: CLEAN — no remaining P0/P1/P2
• Landed on main: shimmer/stagger, Therapy Compass reduced-motion, AnswerSkeleton stagger, spring knob token

Validation this pass

• typecheck PASS
• eslint PASS
• Vitest (guard/therapy/overlay) 39/39 PASS
• prettier PASS
• Bugbot CLEAN

Remaining blockers

None for #1297. The PR is merged and the product delta is on main.

[cursor]

Inspect / closeout re-pass

Failing CI: none on proven tip 9454c307 — Build, Static, Unit, Advisory UI, Production UI, PR required, CircleCI verify all SUCCESS.

Unresolved review threads: none (Codex shimmer-wiring + CodeRabbit reduced-motion already fixed and resolved).

Merge conflicts: none (git merge-tree clean). Synced one-commit main drift afterward.

Bugbot: CLEAN — no remaining P0/P1/P2 on the product delta.

Local validation: typecheck PASS; eslint PASS; Vitest (guard/therapy/overlay) 39/39; prettier PASS.

No new product code changes this pass — prior tip already contained the motion/a11y fixes. A ledger closeout commit may re-run checks; expected green.