Skip to content

chore: PR-9 - loud bookingUrl warning + portal audit fully green - #9

Merged
Eth-Interchained merged 1 commit into
mainfrom
hyperagent/2026-07-07-pr9-warning-audit
Jul 7, 2026
Merged

chore: PR-9 - loud bookingUrl warning + portal audit fully green#9
Eth-Interchained merged 1 commit into
mainfrom
hyperagent/2026-07-07-pr9-warning-audit

Conversation

@Eth-Interchained

Copy link
Copy Markdown
Owner

Summary

PR-9 pays two debts from the live day: the booking-button incident gets its loud render-time warning, and the two standing portal audit warnings die — the audit is fully green for the first time.

What's inside

  • Loud absence: one-time console.warn when the anchor salon lacks a valid https bookingUrl — the button never hides silently again (journalctl-visible, incident-derived)
  • Audit zero: <Head> description on the shell page + the primary keyword woven honestly into copy and intent. portal audit → "All checks passed — no issues found." The CI gate now protects a zero-warning baseline.

Tests run (locally, real engine)

tsc clean · unit 30/30 · live 4/4 vs nedbd 2.6.1 · portal audit CLEAN · portal build clean

Deploy note

Code-only (no seed changes): git pull && pnpm run build && restart. No re-seed needed.

Two debts from the live day, both paid:

- pages.ts: warn ONCE per process when the anchor salon has no valid
  https bookingUrl — the booking button hiding silently is how the
  live-site incident stayed invisible for minutes. Conversion-critical
  absence is now loud in journalctl.
- routes/index.page.tsx: <Head> description + "aveda salon" keyword
  woven honestly into the shell copy and intent — portal audit now
  reports "All checks passed — no issues found." First fully-clean
  audit in the repo's history; the CI gate now guards a zero-warning
  baseline.

Verified: tsc clean · unit 30/30 · live 4/4 vs nedbd 2.6.1 ·
portal audit CLEAN · build clean.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
@Eth-Interchained
Eth-Interchained merged commit 4d64dba into main Jul 7, 2026
1 check 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