Skip to content

Fix post-overhaul audit findings - #86

Open
n30nex wants to merge 1 commit into
mainfrom
codex/fix-post-overhaul-audit
Open

Fix post-overhaul audit findings#86
n30nex wants to merge 1 commit into
mainfrom
codex/fix-post-overhaul-audit

Conversation

@n30nex

@n30nex n30nex commented Aug 30, 2026

Copy link
Copy Markdown
Member

Summary

  • replace or remove the confirmed dead Calgary, StoonMesh, Mesh Quebec, and Saguenay contacts, record a fresh review date for every kept contact and listing, and make expired contacts or overdue reviews fail validation
  • keep the region editor lightweight until someone chooses a proposal type, defer Turnstile until the review step, remove the unused national partition download, and expose a deterministic startup state
  • replace the 274 KB shared PNG with separate 1 KB SVG header and favicon assets
  • require validate, browser, and lighthouse plus resolved review conversations on main while preserving the existing code-owner review and administrator break-glass rules (repository setting, applied and verified separately)

Measured result

  • editor first mobile load: about 6.19 MB before, 471 KB now; no membership CSV, national partition, or Turnstile request before a proposal starts
  • header logo + favicon: about 275 KB before, 2.3 KB transferred now
  • community directory: 24 listings and 57 kept contacts; 52 verified, 5 recently checked but still marked for manual review

Validation

  • 73 content tests and 66 editor tests
  • 14 Python content tests
  • strict bilingual build: 57 English and 57 French pages
  • built-link audit: 128 pages and 12,671 local references
  • Playwright matrix: 389 passed, 34 intentional skips; three end-of-run mobile WebKit resource/time failures all passed in isolated one-worker reruns (18/18 accessibility checks and 1/1 responsive table case)
  • Lighthouse: 0.98–1.00 performance, 1.00 accessibility, 0.96 best practices, 0.92–1.00 SEO across all audited English and French routes
  • final English homepage, French homepage, and French editor inspected in the rendered build with no horizontal overflow

Closes #81
Closes #82
Closes #83
Closes #84
Closes #85

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Aug 30, 2026

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
📝 Code Review Completed 2026-08-30T04:08:59.949823Z 307f100 PR opened
ℹ️ 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" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant