docs(readme): adopt the shared Major Context header and footer

[andybons]

This is a cosmetic change, part of a family-wide README header/footer pass across the five Major Context repos (Moat, Keep, Gatekeeper, Bailey, Harness).

The badge row now follows the shared order: CI, Go Reference, License. There is no Release badge yet since this repo has no releases.

A shared footer was added linking the other four sibling repos, with Gatekeeper itself shown in bold instead of linked. The tiny standalone License section was removed since its content now lives in the footer's "MIT licensed. See LICENSE" line.

The Bailey link in the footer resolves once majorcontext/bailey goes public, which is why this PR is held until then — please don't merge it yet.

[claude]

📦 Previous Review (superseded by newer review)

Claude finished @andybons's task in 9s —— View job

---

PR review: docs(readme): adopt the shared Major Context header and footer

Docs-only change to README.md. It has no security or runtime impact, and I found nothing blocking.

Observations

• Duplicate intro. The new tagline paragraph (README.md:13) is followed directly by the old intro ("A credential-injecting TLS-intercepting proxy. Route HTTPS traffic through Gatekeeper…"). They say nearly the same thing. Consider dropping the old one, or merging the two.
• ASCII banner. The banner sits in a text fence, so it renders as monospace. It's about 82 columns wide, so it may wrap or scroll on narrow viewports such as mobile. It also has no text alternative for screen readers, though the tagline right below it partly covers that.
• Bailey link. The PR description says it will 404 until majorcontext/bailey goes public. Hold the merge until then, as noted.
• License link. The License section is removed and the footer keeps [LICENSE](LICENSE). The link only works if a LICENSE file exists at the repo root, which I didn't check.

Nothing else to flag. I did not run any tests, since this is a README-only change.

[Copilot]

Copilot review overview

🟡 Changes recommended

The update removes the semantic H1 and duplicates the introductory description.

Review effort: Balanced
Findings: 2

Open (2)

• Retain a semantic README H1 alongside the ASCII banner · New
• Remove duplicate README introduction summary · New

What changed in this PR

Adopts the shared Major Context README branding and repository footer.

Changes:

• Adds an ASCII banner and reordered badges.
• Adds sibling-repository and license links.
• Removes the standalone license section.

File	Description
README.md	Updates shared branding, badges, introduction, and footer.

---

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

[claude]

Claude finished @andybons's task in 13s —— View job

---

Review of PR #74 (README header/footer)

No blocking issues. This is a docs-only change to README.md, so none of the security, race or proxy-forwarding concerns apply.

Copilot's two findings are already addressed by the latest commit (f78485f):

• H1: # Gatekeeper is still on line 1, above the ASCII banner, so the document keeps a semantic title.
• Duplicate intro: The original intro paragraph is replaced by the new summary, not kept next to it. git diff origin/main...HEAD shows one intro paragraph, followed by the badges and "Full documentation".

Checks:

• The badge links point at ci.yml, pkg.go.dev and LICENSE. I didn't confirm that ci.yml and LICENSE exist in the repo.
• The footer is bold for Gatekeeper and links the four sibling repos. The Bailey link will 404 until that repo is public. As the PR description says, this is why it's held.
• Removing the ## License section leaves the license reference only in the footer. Check that nothing links to a #license anchor. I didn't search for one.

Minor: the banner's last line is all trailing spaces. It's harmless inside a code fence.

Please don't merge until Bailey is public.