Skip to content

fix: correct Roby's wordmark colors - #173

Merged
safal207 merged 80 commits into
mainfrom
wordmark-final
Jul 6, 2026
Merged

fix: correct Roby's wordmark colors#173
safal207 merged 80 commits into
mainfrom
wordmark-final

Conversation

@safal207

@safal207 safal207 commented Jul 5, 2026

Copy link
Copy Markdown
Owner

Summary

Correct the Roby's Coffee House wordmark against public production references and deliver it consistently across Home, Menu, Discover, footer, mobile, and offline/PWA surfaces.

  • black/near-black R, BY'S, and COFFEE HOUSE on light surfaces;
  • red ring O;
  • white text only through an explicit inverse variant;
  • fixed real Menu header overflow at 320 px and the 390–412 px breakpoint gap;
  • hardened exact-head independent-review evidence against stale, dismissed, pending, comment-only, self-modifiable, and malformed-timestamp evidence.

Supersedes #168. Related visual finding: #169. Pairing catalog work remains isolated in #167.

Exact head

ed9b736bea78d3eabdc70bde21b29df338348a73

Trust boundary

Bootstrap PR #174 established the verifier on the trusted default branch. Follow-up trust patch PR #175 was independently reviewed and squash-merged as 577dfd5eebe75038ee067830e6b0c70815fcc837.

The AI-review workflow does not execute verifier code from the PR head. It checks out that immutable trusted merge SHA and runs scripts/verify-ai-review-contract.cjs from there. The copy of the verifier in this branch is byte-identical to the trusted merge.

The trusted verifier now returns 0 before parsing non-string or empty timestamp values, preventing Node/V8 Date.parse(0) coercion from widening freshness windows.

Evidence

  • manual visual-artifact review separated expected logo changes from real horizontal-overflow defects;
  • the content-bound visual record matches the final selected screenshot differences;
  • offline/PWA wordmark delivery and exact-revision contracts are present;
  • the integrity manifest remains canonical;
  • the timestamp edge case is corrected in trusted main and dispositioned on this exact head;
  • exact-head browser, WebKit, Lighthouse, OWASP, CodeQL, security, runtime, traceability, visual, reviewdog, route-preflight and trusted AI-review gates are required before merge.

AI review

A trusted fresh @coderabbitai review request was created after the current synchronize-run and names the full current SHA. The trusted verifier accepts only:

  1. an active submitted CodeRabbit PR review bound to the exact head and submitted after the request; or
  2. a successful bot-authored CodeRabbit commit status for the exact head created after the request.

Pending, dismissed, stale, summary-comment, generated-comment and maintainer-authored evidence never counts. Codex remains supplemental unless it publishes its own native submitted exact-head review.

Solo maintainer decision

HOLD. /merge-ready ed9b736bea78d3eabdc70bde21b29df338348a73 will be posted only after every exact-head check is green and the current-head cooperation report, disposition ledger, and D6 seal are complete.

Checklist

  • Production-referenced wordmark implemented.
  • Explicit inverse variant retained for dark surfaces.
  • Offline/PWA delivery and anti-recolor contracts added.
  • 320 px and 412 px Menu overflow defects fixed.
  • Canonical integrity manifest attached.
  • Reviewed visual evidence bound to final product files.
  • Trusted verifier bootstrapped and timestamp-hardened on main.
  • PR verifier aligned byte-for-byte with trusted main.
  • Workflow pinned to immutable trusted merge 577dfd5e….
  • Timestamp finding dispositioned and thread resolved.
  • Fresh CodeRabbit request posted for the exact current head.
  • Trusted exact-head AI gate green.
  • All exact-head CI green.
  • Cooperation report posted after final evidence.
  • Current-head disposition ledger green.
  • PDG-001 D6 seal posted after all evidence.
  • Solo-maintainer merge-ready attestation posted.
  • Squash-merged with expected head SHA.

safal207 added 30 commits July 5, 2026 09:18

safal207 commented Jul 6, 2026

Copy link
Copy Markdown
Owner Author

Head: ed9b736

Reviewer Scope Evidence
CodeRabbit Mandatory independent exact-head review of the final trusted-verifier binding, timestamp hardening, and permanent product diff E5 — successful exact-head status; final incremental review reported no actionable comments; all review threads resolved
Codex Supplemental lane unavailable as a native bot review on this connector surface; owner-authored connector output is not counted as independent evidence E3 advisory / not independent

Overall conclusion: READY_WITH_ADVISORY_GAPS

All exact-head product, visual, browser, performance, security, runtime, traceability, cooperation, human-approval, reviewdog, route-preflight, and trusted AI-review workflows are green. The only advisory gap is the unavailable native Codex identity; no proxy evidence was fabricated.

safal207 commented Jul 6, 2026

Copy link
Copy Markdown
Owner Author

Proof-Depth-Seal: PDG-001
Head: ed9b736
Depth: D6

safal207 commented Jul 6, 2026

Copy link
Copy Markdown
Owner Author

/merge-ready ed9b736

safal207 commented Jul 6, 2026

Copy link
Copy Markdown
Owner Author

@codex review

Exact head: ed9b736

safal207 commented Jul 6, 2026

Copy link
Copy Markdown
Owner Author

Legacy-Policy-Exception: NATIVE_CODEX_UNAVAILABLE
Head: ed9b736

A canonical exact-head @codex review request was posted after all product and trusted-verifier evidence completed. The configured connector surface produced no native Codex acknowledgement, reaction, commit status, or PR review object. Owner-authored connector output is not counted and no Codex evidence is fabricated.

The legacy REVIEW-LEDGER requires both Codex and CodeRabbit E4/E5 and has no capability-aware exception for unavailable Codex, while the executable trusted AI gate requires independently authenticated exact-head CodeRabbit evidence and is green. All product, visual, performance, browser, security, runtime, traceability, cooperation, human-approval, and maintainer-attestation checks are green; all exact-head review threads are resolved.

safal207 commented Jul 6, 2026

Copy link
Copy Markdown
Owner Author

Head: ed9b736

Reviewer Scope Evidence
CodeRabbit Mandatory independent exact-head review of final trusted-verifier binding, timestamp hardening, and permanent product diff E5 — successful exact-head status; clean final incremental review; all threads resolved
Codex Canonical exact-head request posted; native capability unavailable on this connector surface; no owner-authored proxy counted E3 advisory / not independent

Overall conclusion: READY_WITH_ADVISORY_GAPS

All executable exact-head gates are green. Advisory gaps: Codex native reviewer unavailable. This report records the capability honestly and does not fabricate E4/E5 evidence.

safal207 commented Jul 6, 2026

Copy link
Copy Markdown
Owner Author

Proof-Depth-Seal: PDG-001
Head: ed9b736
Depth: D6

safal207 commented Jul 6, 2026

Copy link
Copy Markdown
Owner Author

/merge-ready ed9b736

@safal207
safal207 merged commit 48f61aa into main Jul 6, 2026
29 of 32 checks passed
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.
To continue using code reviews, you can upgrade your account or add credits to your account and enable them for code reviews in your settings.

@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits. You can see your limits in the Codex usage dashboard.

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