feat(www): add /github landing page for the GitHub App#299
feat(www): add /github landing page for the GitHub App#299peaktwilight wants to merge 2 commits into
Conversation
Astro page at foxguard.dev/github with the install CTA, what-it-flags grid, sandboxing/safety story, and a self-host fallback pointing at ghcr.io/pwnkit-labs/foxguard-github-app. Held as draft until the GitHub App is registered under PwnKit-Labs: - APP_INSTALL_URL is # for now - APP_REGISTERED is false - A "Coming soon — track #246" banner sits above the hero - Install button is visually disabled When the App is registered, three constants at the top of github.astro flip to the live values and the banner disappears. Easy single-PR follow-up. Tracking: #246. Build verified via 'cd www && npm run build' — /github/index.html lands in dist/.
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: defaults Review profile: CHILL Plan: Pro Plus Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Org renamed PwnKit-Labs -> 0sec-labs on 2026-04-08. Update the four remaining stale refs on the GitHub App landing page: - TODO comment 'registered under PwnKit-Labs' -> '0sec-labs' - SELF_HOST_IMAGE constant ghcr.io/pwnkit-labs/* -> ghcr.io/0sec-labs/* - Three URLs to github.com/PwnKit-Labs/foxguard issues/blob -> 0sec-labs/foxguard - Display text 'PwnKit-Labs/foxguard' -> '0sec-labs/foxguard' No layout or copy change beyond the rebrand. Astro build verified clean.
|
Status / hold-state note for this draft so future-me (or anyone else) doesn't lose the thread: Blocking on: GitHub App registration under `0sec-labs`. Specifically need:
That's the entire follow-up. Three-line change, no other rework needed. Won't be blocking on:
What's already on this branch:
Won't merge this until App is registered. Shipping a /github page where the install button points to `#` and the banner says "coming soon" is honest but bad UX as a permanent state. If anyone needs to ship something at `/github` before then, the easier move is to flip the banner to a marketing-page tone and link to issue #246 explicitly. Happy to do that pivot if there's a reason. |
|
Dashboard/blocker note (2026-05-19): this PR should be tracked as a blocked follow-up to #246.\n\nWhy blocked:\n- The page is implemented, but the install CTA still needs the real GitHub App install URL.\n- Merging it while the CTA is disabled / points at |
DRAFT — held until the GitHub App is registered under PwnKit-Labs and we have the real Marketplace install URL. Tracked in #246.
Summary
New Astro page at `foxguard.dev/github`. Hero, three-step "how it works," what-it-flags grid (8 categories), trust/safety story, self-host fallback, soft CTA back to the local CLI.
What flips when the App is registered
Three constants at the top of `www/src/pages/github.astro`:
```ts
const APP_INSTALL_URL = '#'; // → real Marketplace URL
const APP_HOMEPAGE = APP_INSTALL_URL; // → same
const SELF_HOST_IMAGE = 'ghcr.io/pwnkit-labs/foxguard-github-app';
const APP_REGISTERED = false; // → true
```
Flipping `APP_REGISTERED` to `true`:
That's the entire follow-up PR after registration.
Why ship the page in draft now rather than after registration
Build check
`cd www && npm run build` — `/github/index.html` is generated, no errors. Confirmed locally.
Test plan