Skip to content

docs(issues): capture site-audit + measurement follow-ups (#012–#017)#1043

Merged
BigSimmo merged 2 commits into
mainfrom
claude/site-audit-quick-wins-21v9gb
Jul 21, 2026
Merged

docs(issues): capture site-audit + measurement follow-ups (#012–#017)#1043
BigSimmo merged 2 commits into
mainfrom
claude/site-audit-quick-wins-21v9gb

Conversation

@BigSimmo

@BigSimmo BigSimmo commented Jul 21, 2026

Copy link
Copy Markdown
Owner

Summary

Docs-only. Records the follow-ups surfaced by the 2026-07-21 site-audit + bundle/measurement pass into the durable docs/outstanding-issues.md ledger so they survive session context reset. No source, RAG, clinical, or config changes — a single markdown file.

Renumbered after merging main: main added its own #011 (auth DB-connection allocation) via a separate ledger update, so these follow-ups were shifted from #011#016 to #012#017 and the next-id marker to 018. main's #011 is kept intact; no other ledger rows changed.

Six new rows (#012#017), all P3:

RAG impact: no retrieval behaviour change — docs-only, no retrieval/ranking/selection/eval files touched.

Verification

  • npm run verify:pr-local
    • Verification not run: docs-only change (a single markdown ledger file). Formatted with Prettier so format:check passes; no code, tests, or config touched. The merge from main also carries its own already-merged changes (eval-canary workflow + branch-review-ledger), which are no-ops relative to main.

Risk and rollout

  • Risk: None — documentation only.
  • Rollback: revert the ledger commit.
  • Provider or production effects: None.

Notes

🤖 Generated with Claude Code

https://claude.ai/code/session_01UbhUVWVJRwDibC2YtJ6aRX

Summary by CodeRabbit

  • Documentation
    • Updated the outstanding-issues tracker with additional newly documented items (#011Bump @types/node from 25.9.1 to 25.9.2 #17).
    • Advanced the issue numbering to keep the tracker in sync with the latest additions.
    • Included follow-ups covering auth database connection handling, search-related behavior, image optimization, performance and bundle-size considerations, and real-world Web Vitals measurement via Lighthouse.

Durable ledger entries from the 2026-07-21 site-audit / measurement pass so they
survive session context reset:

- #11 slim the lazy cross-mode differentials chunk. Verified the 860 KB
  snapshot is correctly code-split OUT of the initial bundle via the dynamic
  import in cross-mode-links.tsx:150 — not a leak. The lazy chunk still loads the
  full snapshot to build a tiny catalog; a precomputed index would slim it.
- #12 route-chunk + mockup catalogue JSON weight (specifiers/forms/formulation;
  dev-only mockup chunks built though /mockups 404s in prod).
- #13 realize the next/image win on Supabase signed previews (#1024 images
  config is inert until previews move off raw <img>).
- #14 content-first fallback regression tests (CodeRabbit follow-up on #1024).
- #15 "big but not easy" structural + motion perf (PPR/static shells, sidebar
  grid-template-columns animation, Therapy Compass JSON defer, dialog dynamic
  imports).
- #16 field Web-Vitals baseline via live Lighthouse (in-sandbox prod vitals
  blocked on secrets; dev-mode CLS measured excellent).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01UbhUVWVJRwDibC2YtJ6aRX
@coderabbitai

coderabbitai Bot commented Jul 21, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

📝 Walkthrough

Walkthrough

Updates the outstanding-issues ledger by advancing the next issue ID to 018 and adding open items #011#017.

Changes

Outstanding issues ledger

Layer / File(s) Summary
Update issue ledger
docs/outstanding-issues.md
Advances the issues:next-id marker to 018 and adds seven open-item records covering authentication allocation, payload and chunk sizing, image optimization, fallback tests, deferred performance work, and Web Vitals measurement.

Estimated code review effort: 1 (Trivial) | ~2 minutes

Possibly related PRs

  • BigSimmo/Database#1046: Updates the same issue ledger counter and adds the authentication DB-connection allocation entry.
🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.
Title check ✅ Passed The title clearly summarizes the docs-only ledger update and the site-audit follow-ups it records.
Description check ✅ Passed The description follows the template with Summary, Verification, Risk and rollout, and Notes, and it includes the required verification note.
✨ Finishing Touches
📝 Generate docstrings
  • Create stacked PR
  • Commit on current branch
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch claude/site-audit-quick-wins-21v9gb

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

@supabase

supabase Bot commented Jul 21, 2026

Copy link
Copy Markdown

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 ↗︎.

@BigSimmo
BigSimmo marked this pull request as ready for review July 21, 2026 17:14
Resolves the docs/outstanding-issues.md conflict: main added #11 (auth
DB-connection allocation) via its own ledger update, so this renumbers the
site-audit follow-ups from #11-#16 to #12-#17 (next-id -> 018) and fixes the
#17 cross-reference. Keeps main's #11 intact; no other ledger rows changed.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01UbhUVWVJRwDibC2YtJ6aRX
@BigSimmo BigSimmo changed the title docs(issues): capture site-audit + measurement follow-ups (#011–#016) docs(issues): capture site-audit + measurement follow-ups (#012–#017) Jul 21, 2026
@BigSimmo
BigSimmo merged commit bdf5ef1 into main Jul 21, 2026
19 checks passed
@BigSimmo
BigSimmo deleted the claude/site-audit-quick-wins-21v9gb branch July 21, 2026 20:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants