Skip to content

docs(site): point every site link at https://jubershaikh.com/forgekit/ - #168

Merged
CodeWithJuber merged 1 commit into
masterfrom
claude/forgekit-deep-review-issues-ww6y0h
Sep 26, 2026
Merged

CodeWithJuber merged 1 commit into
masterfrom
claude/forgekit-deep-review-issues-ww6y0h

Conversation

@CodeWithJuber

Copy link
Copy Markdown
Owner

What & why

GitHub Pages serves this project's site from the custom domain jubershaikh.com; https://codewithjuber.github.io/forgekit/ only redirects there. Until today that redirect went to plain http://. Cloudflare's "Always Use HTTPS" is now on for jubershaikh.com. That's a zone setting, not part of this diff.

This PR points every link we control at https://jubershaikh.com/forgekit/ directly, so visitors and link-preview scrapers skip the redirect:

  • brand.json: site.url. The status page, the /diagrams/ gallery and their canonical and social tags all derive from it.
  • Landing and 404 pages: the canonical, og:url, og:image, twitter:image, favicon, touch icon and JSON-LD URLs.
  • Docs: every "Open the interactive diagram" link in README, ARCHITECTURE, ONBOARDING, GUIDE, the substrate, routing and plan docs, the eight docs-site pages and docs/diagrams/README.md.
  • CHANGELOG: an [Unreleased] entry. Historical entries keep their original text.

Each new URL was checked live: /, og.jpg, favicon.svg, apple-touch-icon.png, /status/, /diagrams/, a diagram page and an SVG all return 200 with zero redirects.

Checklist

  • npm test passes: 1,701 tests, 1,697 pass, 0 fail, 4 platform-gated skips (Node 22)
  • npm run check passes (Biome lint + format)
  • New public functions have a test (n/a: URL change only)
  • Conventional commit message (feat:/fix:/docs: …)
  • CHANGELOG.md updated under ## [Unreleased]
  • No new runtime dependency (dev deps ok)
  • Substrate/docs updated if this changes forge substrate, forge impact, router/gate, or MCP substrate tools (n/a)

Risk & rollback

  • Risk level: very low. Only URLs change; the site itself is unchanged. The docs check, all 13 diagram receipts and all 19 MDX pages still pass.
  • Rollback plan: revert the merge commit. The old URLs keep working through the redirect.

Extra checks (tick if applicable)

  • npm run typecheck passes
  • Input validated at boundaries; errors handled (no swallowing): n/a
  • Authorization/ownership checked (if it touches access): n/a
  • Logs contain no secrets/PII
  • If AI-assisted: I understand it, verified the package APIs, and it has tests

🤖 Generated with Claude Code

https://claude.ai/code/session_01GVVG2VDETWsDxMu6MBWPz2


Generated by Claude Code

GitHub Pages serves the project site from the custom domain, and
codewithjuber.github.io/forgekit/ only redirects there (until the domain
enforced HTTPS, it redirected to plain http://). brand.json's site URL (which
the status page, the diagram gallery and their social tags derive from), the
landing and 404 pages' canonical, Open Graph and JSON-LD URLs, and every
"Open the interactive diagram" link now use the custom domain directly, so
links and link previews resolve with no redirect. Each new URL was checked:
200, zero redirects.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01GVVG2VDETWsDxMu6MBWPz2
@CodeWithJuber
CodeWithJuber marked this pull request as ready for review September 26, 2026 23:38
@CodeWithJuber
CodeWithJuber merged commit b156c71 into master Sep 26, 2026
14 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.

2 participants