Skip to content

Audit redundant UI and add site guide and append-only builder journal - #71

Merged
erinepshovel-code merged 2 commits into
mainfrom
fix/site-audit-builder
Sep 17, 2026
Merged

erinepshovel-code merged 2 commits into
mainfrom
fix/site-audit-builder

Conversation

@erinepshovel-code

Copy link
Copy Markdown
Contributor

The site repeats orientation and entry prompts, loads map/status assets on unrelated pages, and attaches overlapping export toolbars to Way units. At 320px the Way overflowed; its 20 units had 180 export buttons.

This changes the owning website surfaces: one guide at /about-the-site/, a shared compact footer, a compatibility redirect from /start/, route-specific assets, and exports on reading/source fields rather than their parent containers. The Way fits at 320px with 120 remaining export buttons. Ordinary reading pages omit six asset references (40,792 uncompressed source bytes, not a latency claim).

/by-the-builder/ publishes one dated, attributed AI-builder entry. The history guard rejects changes, removals, reordering, bad metadata, or invented correction targets; corrections append new records. PR/push checks bind prior Git history and fail when it is missing. This is a release policy, not administrator-proof storage.

File plan:

  • src/_includes, src/home, src/start, CSS/JS: consolidate navigation, guide ownership and resources; preserve all source bodies. Risk: route/UI regression; checked with generated links and browser suite.
  • src/about-the-site, src/by-the-builder, src/_data/builder.json, src/accessibility: public guidance, attributed history and honest scope. Risk: overclaiming; linked primary guidance and explicit limits.
  • scripts/check-builder-history.mjs, journal tests, package/workflows: append-only gate with full checkout history. Risk: missing baseline; fail closed and test actual temporary Git history.
  • docs/site-audit-2026-09-17.md and README: findings, authority record, measurements and usage.

Verification: 13 source/journal tests, 14 generated checks, 2,884 internal references across 126 HTML pages, static budgets, and 43 browser/accessibility checks passed locally. Local browser was available Chromium 139 because downloading the locked build failed; CI must independently run its declared browser. Full release checks required before merge. Manual assistive-technology/user testing and real-user Web Vitals remain hmmm.

Guidance: current skill-lib and METAPAT, W3C WAI, GOV.UK progressive enhancement, and web.dev; source bodies, licenses and claim authority remain unchanged.

@chatgpt-codex-connector

chatgpt-codex-connector Bot commented Sep 17, 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-09-17T10:05:28.416556Z a53669d New commits
ℹ️ 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.

@erinepshovel-code
erinepshovel-code merged commit 11aaddb into main Sep 17, 2026
5 checks passed
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