Skip to content

Fix post-overhaul audit findings - #86

Merged
MrAlders0n merged 1 commit into
mainfrom
codex/fix-post-overhaul-audit
Sep 2, 2026
Merged

Fix post-overhaul audit findings#86
MrAlders0n merged 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.

@MrAlders0n
MrAlders0n merged commit dc9b8e4 into main Sep 2, 2026
3 checks passed
@MrAlders0n
MrAlders0n deleted the codex/fix-post-overhaul-audit branch September 2, 2026 15:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

2 participants