feat(landing): GitHub, CI and code review page and homepage section by Makisuo · Pull Request #1534 · MapleTechLabs/maple

Makisuo · 2026-10-17T14:39:52Z

What

A new feature page, /features/github (en, ja, ko), and a homepage section for the GitHub integration: PR reviews, CI observability and the post-merge check.

Feature page

Hero, then a facts panel beside the /code-review analytics capture.
The review artifact: Maple's real comment on feat(settings): AI tab and email preferences with a local send schedule #1531, redrawn in the site's type and tokens instead of a GitHub screenshot (components/github/PrReviewComment.astro). Content is literal, like docs snippets.
CI section: the CI tab, a failed CI run's trace and a PR review sheet.
After-merge section: the real "after this shipped" comment from fix(settings): address audit findings across settings and billing #1524 (PostMergeComment.astro) with commit context, deploy markers and PR-closes-issue.
Capability ledger (8 rows), related links, CTA. error-tracking and ai-mcp-integration now link here.

Homepage

GitHubSection after the agent section: before merge, in CI, after it ships. Compact versions of both comment cards and a CSS crop of the CI tab capture (the full frame is unreadable at a third of the shell).

Screenshots, through the existing bun run screenshots workflow:

surface-code-review, gallery-code-review-pr, surface-ci, gallery-ci-trace added to shots.ts and checked in.
New seed scripts/seed-demo/code-review.ts (bun run seed:demo:code-review) fills the demo org with PR reviews and findings. It refuses any database but maple_screenshots. CI runs come from the existing scripts/seed-ci-runs.ts.
Both seeds are dated relative to now, not the demo anchor, so re-run them right before a capture.

Notes

No hero engraving yet: illustration is unset, so the hero uses the plain dot-grid treatment. A Roman engraving should follow like the other feature pages.
The fix(settings): address audit findings across settings and billing #1524 comment lists POST /mcp and GET /mcp twice each; the card keeps one row of each. Looks like a duplicate-row bug in the post-merge table, worth a separate look.
bun.lock picks up a few unrelated workspace version bumps because the lockfile on main was out of date.

Checks

bun run test in apps/landing: 85 passed.
astro build: 233 pages, no new warnings.
oxlint and oxfmt clean on the touched files.
Rendered /features/github, /ja/features/github and the homepage section in a browser at 1440 wide.

🤖 Generated with Claude Code

^{Need help on this PR? Tag @codesmith-bot with what you need.}

Seed the demo org with PR reviews (`bun run seed:demo:code-review`, refuses any database but maple_screenshots) and add four shots to the landing capture list: Code Review analytics, a PR review sheet, the CI tab and a failed CI run's trace. CI runs come from scripts/seed-ci-runs.ts against acme/web.

…ection /features/github in en, ja and ko: Code Review analytics with its facts panel, Maple's review comment from #1531 redrawn in the site's type, CI runs as traces, and the post-merge comment from #1524. The homepage gets a three-step section (before merge, in CI, after it ships) after the agent section.

coderabbitai · 2026-10-17T14:40:03Z

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 9 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used all 4 included reviews currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration

Configuration used: defaults
Review profile: CHILL
Plan: Advanced
Run ID: ea3423c9-f9d2-4199-b446-1050e91068e2

📥 Commits

Reviewing files that changed from the base of the PR and between 2247b59 and fb5e97d.

⛔ Files ignored due to path filters: 1

bun.lock is excluded by !**/*.lock

📒 Files selected for processing: 20

apps/landing/messages/en.json
apps/landing/messages/ja.json
apps/landing/messages/ko.json
apps/landing/public/screenshots/gallery-ci-trace.webp
apps/landing/public/screenshots/gallery-code-review-pr.webp
apps/landing/public/screenshots/surface-ci.webp
apps/landing/public/screenshots/surface-code-review.webp
apps/landing/scripts/screenshots/shots.ts
apps/landing/src/components/github/PostMergeComment.astro
apps/landing/src/components/github/PrReviewComment.astro
apps/landing/src/components/home/GitHubSection.astro
apps/landing/src/components/home/Home.astro
apps/landing/src/components/page/artifacts/PrReview.astro
apps/landing/src/components/page/sections/GitHubAfterMerge.astro
apps/landing/src/components/page/sections/GitHubCi.astro
apps/landing/src/components/page/slots.ts
apps/landing/src/lib/features.ts
apps/landing/src/lib/page-registry.ts
package.json
scripts/seed-demo/code-review.ts

Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands.}

maple-review-bot · 2026-10-17T14:40:07Z

Maple review

🟢 Confidence 9/10 · safe to merge
Static marketing page and homepage section; new registry ids, message keys and screenshot refs all resolve, and the CSS crop math checks out.
_{quality 100/100 · no findings · tests partial · risk low}

Adds the /features/github page in en/ja/ko, a homepage GitHub section and four checked-in screenshots, wired through the existing feature registry, artifact/section slots and Paraglide messages. Static content only; safe to merge.

github feature entry added with pr-review artifact and github-ci/github-after-merge sections
GitHubSection added to the homepage after the agent section
Both comment cards redrawn in site tokens instead of shipping GitHub screenshots
surface-ci, surface-code-review and both gallery captures added to shots.ts

Before merge

Manual · Re-run bun run seed:demo:code-review and scripts/seed-ci-runs.ts before re-capturing the screenshots

Production impact

Production traffic of the changed files (last 7 days)

File	Calls/day	Busiest operations
`apps/landing/src/components/github/PostMergeComment.astro`	23k	`POST /mcp` 11k/day, 0.0% err `POST /mcp` 11k/day, 0.0% err `GET /mcp` 722/day, 0.0% err

What was checked

MediaFrame takes label, so the CI shots keep real alt text (GitHubCi.astro:42)
CI crop offsets match the 2880×1620 capture at 1140×885 (GitHubSection.astro:85-96)
All 52 new keys exist in en/ja/ko; every new related slug resolves via featureBySlug

_{fb5e97d · Updated on every push. Reply "won't fix" to dismiss a finding, or mention @maple-review-bot to ask about one.}

maple-review-bot · 2026-10-17T14:50:23Z

Merged with 1 step from "Before merge" still open. If it is done, tick it on the review comment; if not, now is the time.

Manual · Re-run bun run seed:demo:code-review and scripts/seed-ci-runs.ts before re-capturing the screenshots

maple-review-bot · 2026-10-17T17:15:22Z

Maple: after this shipped

✅ Shipped clean. Nothing this change touches got worse in the hour after it deployed.

Deployed to maple-api (pr-1509) at 2026-10-17 14:55 UTC as c2bbc23, the first version after the merge (the merge commit itself never reported). Compared the 60 minutes before and after.

Operation	Calls/h	Errors	p95
`POST /mcp`	182 → 128	0.0% → 0.0%	1014 → 587 ms
`POST /mcp`	182 → 128	0.0% → 0.0%	1913 → 1977 ms
`GET /mcp`	0 → 7	0.0% → 0.0%	0 → 97 ms
`GET /mcp`	0 → 7	0.0% → 0.0%	0 → 1265 ms

_{Maple looks once, an hour after the first deploy that carries the merge.}

Makisuo added 2 commits October 17, 2026 16:39

Makisuo merged commit 1dcbcae into main Oct 17, 2026
34 checks passed

Makisuo deleted the feat/landing-github-page branch October 17, 2026 14:50

Navigation Menu

Repository navigation

feat(landing): GitHub, CI and code review page and homepage section - #1534

feat(landing): GitHub, CI and code review page and homepage section#1534
Makisuo merged 2 commits into
mainfrom
feat/landing-github-page

Makisuo commented Oct 17, 2026 •

edited by blacksmith-sh Bot

Loading

Uh oh!

coderabbitai Bot commented Oct 17, 2026

Review limit reached

Uh oh!

maple-review-bot Bot commented Oct 17, 2026 •

edited

Loading

Uh oh!

Uh oh!

maple-review-bot Bot commented Oct 17, 2026

Uh oh!

maple-review-bot Bot commented Oct 17, 2026

Uh oh!

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Conversation

Makisuo commented Oct 17, 2026 • edited by blacksmith-sh Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

What

Notes

Checks

Uh oh!

coderabbitai Bot commented Oct 17, 2026

Review limit reached

Uh oh!

maple-review-bot Bot commented Oct 17, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Maple review

Before merge

Production impact

Uh oh!

Uh oh!

maple-review-bot Bot commented Oct 17, 2026

Uh oh!

maple-review-bot Bot commented Oct 17, 2026

Maple: after this shipped

Uh oh!

Reviewers

Uh oh!

Assignees

Labels

Projects

Milestone

Development

Uh oh!

1 participant

Makisuo commented Oct 17, 2026 •

edited by blacksmith-sh Bot

Loading

maple-review-bot Bot commented Oct 17, 2026 •

edited

Loading