Skip to content

Merge design-audit remediation changes from 51278a70d#817

Closed
BigSimmo wants to merge 2 commits into
mainfrom
codex/design-audit-main-merge-safe-final-806-final2
Closed

Merge design-audit remediation changes from 51278a70d#817
BigSimmo wants to merge 2 commits into
mainfrom
codex/design-audit-main-merge-safe-final-806-final2

Conversation

@BigSimmo

@BigSimmo BigSimmo commented Jul 18, 2026

Copy link
Copy Markdown
Owner

Summary

  • Merged commit 51278a7 (design-audit remediation) onto current main lineage via explicit merge.
  • This is a single explicit merge commit to keep history aligned with the requested lineage.
  • Changes are scoped to RAG/scaling, site-map/codebase index updates, UI/service regressions, and migration metadata.

Checks run locally

  • npm run check:runtime
  • npm run test -- tests/site-map.test.ts tests/supabase-schema.test.ts
  • npm run test -- tests/audit-content-services-regressions.test.ts tests/audit-navigation-auth-regressions.test.ts
  • npm run check:github-actions
  • npm run check:ci-scope -- --dry-run
  • npm run check:ci-triage -- --dry-run
  • npm run check:pr-policy -- --dry-run
  • npm run typecheck
  • npm run lint

Notes

  • Merge completed with -X theirs strategy due historical conflict surface in this merge path.
  • docs/branch-review-ledger.md was left untouched (no conflict markers remaining).

Summary by CodeRabbit

  • New Features

    • Enhanced service comparison and selection controls with expandable checklists, source confidence details, and comparison information.
    • Added improved clinical search correction and faster text-based searching.
    • Added clearer form task guidance and session-expiration instructions.
  • Bug Fixes

    • Improved sitemap route coverage and redirect documentation.
    • Improved cleanup of removed registry content and document title indexing.
    • Refined mobile navigation behavior and accessibility labels.
  • Documentation

    • Expanded API and database schema route documentation.
    • Documented public utility routes and updated product route descriptions.

@supabase

supabase Bot commented Jul 18, 2026

Copy link
Copy Markdown

This pull request has been ignored for the connected project sjrfecxgysukkwxsowpy due to reaching the limit of concurrent preview branches.
Go to Project Integrations Settings ↗︎ if you wish to update this limit.


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

@coderabbitai

coderabbitai Bot commented Jul 18, 2026

Copy link
Copy Markdown
Contributor

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: c16cc7b9-d962-4af5-8e79-f51ff203b601

📥 Commits

Reviewing files that changed from the base of the PR and between 77ab4a6 and d508646.

📒 Files selected for processing (24)
  • docs/codebase-index.md
  • docs/site-map.md
  • playwright.config.ts
  • scripts/check-github-action-pins.mjs
  • scripts/generate-site-map.ts
  • src/app/layout.tsx
  • src/app/page.tsx
  • src/components/clinical-dashboard/master-search-header.tsx
  • src/components/forms/forms-home-page.tsx
  • src/components/services/services-navigator-page.tsx
  • src/lib/rag-candidate-sources.ts
  • src/lib/rag.ts
  • supabase/drift-manifest.json
  • supabase/migrations/20260714180000_patch_rag_and_corrector_scalability.sql
  • supabase/migrations/20260714190000_document_table_facts_trgm_idx.sql
  • supabase/migrations/20260717010000_harden_rag_scalability_patch.sql
  • supabase/schema.sql
  • tests/audit-content-services-regressions.test.ts
  • tests/audit-navigation-auth-regressions.test.ts
  • tests/site-map.test.ts
  • tests/supabase-schema.test.ts
  • tests/ui-accessibility.spec.ts
  • tests/ui-smoke.spec.ts
  • tests/ui-tools.spec.ts

📝 Walkthrough

Walkthrough

This PR updates route and schema documentation, sitemap generation, RAG retrieval and query-correction behavior, Supabase migrations, service/forms UI interactions, Playwright settings, and related regression tests.

Changes

Route documentation and tooling

Layer / File(s) Summary
Route discovery and sitemap generation
scripts/generate-site-map.ts
Separates API and public utility handlers, updates redirect extraction, and renders the new public utility route section.
Sitemap and codebase documentation
docs/site-map.md, docs/codebase-index.md
Documents public utility routes, updated redirects, expanded API mappings, and additional schema tables.
Workflow discovery resilience
scripts/check-github-action-pins.mjs
Handles missing workflow directories and normalizes discovered workflow paths.
Sitemap validation
tests/site-map.test.ts
Validates documentation coverage and placement across product, public handler, API, and redirect sections.

RAG retrieval and database scalability

Layer / File(s) Summary
Retrieval and abort-flow changes
src/lib/rag-candidate-sources.ts, src/lib/rag.ts
Removes AbortSignal query wrappers and related parameters while retaining retrieval fallbacks, telemetry, caching, enrichment, and classifier fallback behavior.
Query correction and registry schema
supabase/migrations/*, supabase/schema.sql
Adds registry cleanup triggers, title-word synchronization and indexing, trigram query correction, access controls, and index changes.
Schema manifest and replay validation
supabase/drift-manifest.json, tests/supabase-schema.test.ts
Updates schema metadata and verifies migration replay, cleanup matching, query-correction probes, permissions, and index parity.

Service, forms, and navigation UI

Layer / File(s) Summary
Expandable service navigator panels
src/components/services/services-navigator-page.tsx
Adds expandable checklist, confidence, and comparison panels with selection-aware state management and accessibility attributes.
Forms and dashboard navigation updates
src/components/forms/forms-home-page.tsx, src/app/page.tsx, src/app/layout.tsx, src/components/clinical-dashboard/master-search-header.tsx
Updates forms task content and session copy, narrows page parameter typing, removes the lifecycle import, and simplifies menu dismissal.
UI regression and accessibility coverage
tests/audit-content-services-regressions.test.ts, tests/audit-navigation-auth-regressions.test.ts, tests/ui-*.spec.ts, playwright.config.ts
Aligns UI assertions with updated headings and content, adds mobile dock checks, extends timeouts, configures reduced motion, and captures redirect errors.

Estimated code review effort: 4 (Complex) | ~60 minutes

Possibly related PRs

Suggested reviewers: copilot, cursoragent

✨ 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 codex/design-audit-main-merge-safe-final-806-final2

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

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.

1 participant