Skip to content

feat: /land-and-deploy, /canary, /benchmark + perf review (v0.7.0)#183

Open
garrytan wants to merge 5 commits intomainfrom
garrytan/land-and-deploy
Open

feat: /land-and-deploy, /canary, /benchmark + perf review (v0.7.0)#183
garrytan wants to merge 5 commits intomainfrom
garrytan/land-and-deploy

Conversation

@garrytan
Copy link
Owner

Summary

  • /land-and-deploy — merges PR, waits for CI and deploy workflows, runs canary health check on production. Auto-detects merge queues, deploy platforms, and production URLs. One-click revert if something breaks. Timing data logged for retrospectives.
  • /canary — standalone post-deploy monitoring with browse daemon. 10-minute watch loop with screenshots, console error detection, performance regression alerts. Baseline comparison mode.
  • /benchmark — performance regression detection. Web Vitals, bundle sizes, request counts via performance.getEntries(). Baseline comparison, trend analysis, performance budgets.
  • Performance & Bundle Impact review checklist category — catches heavy deps (moment.js → date-fns), missing lazy loading, synchronous scripts, fetch waterfalls. Added to review/checklist.md, picked up by both /review and /ship.
  • {{DEPLOY_BOOTSTRAP}} resolver — auto-detects deploy platform (Vercel, Netlify, Fly.io, GH Actions), production URL, merge method. Persists to CLAUDE.md like test bootstrap.
  • "Deployed" row in Review Readiness Dashboard from /land-and-deploy JSONL entries.
  • 3 TODOs superseded (/merge, deploy-verify, post-deploy verification).

Incorporates canary/benchmark patterns from community PR #151 (HMAKT99).

Test Coverage

271 tests pass across skill-validation and gen-skill-docs. All 3 new skills registered and validated (frontmatter, preamble, contributor mode, update check).

Pre-Landing Review

No issues found. All changes are additive skill templates and mechanical registrations.

Design Review

No frontend files changed — design review skipped.

TODOS

  • /merge skill → Completed v0.7.0 (superseded by /land-and-deploy)
  • Deploy-verify skill → Completed v0.7.0 (superseded by /land-and-deploy)
  • Post-deploy verification → Completed v0.7.0 (superseded by /land-and-deploy)

Test plan

  • bun run gen:skill-docs — 18 skills generated, no stale placeholders
  • bun run skill:check — all skills green (canary: 16 cmds, benchmark: 7, land-and-deploy: 5)
  • bun test — 271 pass, 0 fail, 1098 assertions

🤖 Generated with Claude Code

garrytan and others added 5 commits March 17, 2026 23:01
Three new skills that close the deploy loop:
- /canary: standalone post-deploy monitoring with browse daemon
- /benchmark: performance regression detection with Web Vitals
- /land-and-deploy: merge PR, wait for deploy, canary verify production

Incorporates patterns from community PR #151.

Co-Authored-By: HMAKT99 <HMAKT99@users.noreply.github.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
New Pass 2 (INFORMATIONAL) category catching heavy dependencies
(moment.js, lodash full), missing lazy loading, synchronous scripts,
CSS @import blocking, fetch waterfalls, and tree-shaking breaks.

Both /review and /ship automatically pick this up via checklist.md.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
- New generateDeployBootstrap() resolver auto-detects deploy platform
  (Vercel, Netlify, Fly.io, GH Actions, etc.), production URL, and
  merge method. Persists to CLAUDE.md like test bootstrap.
- Review Readiness Dashboard now shows a "Deployed" row from
  /land-and-deploy JSONL entries (informational, never gates shipping).

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Superseded by /land-and-deploy:
- /merge skill — review-gated PR merge
- Deploy-verify skill
- Post-deploy verification (ship + browse)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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