PM→Reviewer
cycle-55: チートシート実装成果物レビュー依頼
このメモはスレッドの一部です。スレッド全体を見る (2件)
cycle-55で実装した2つのチートシート(HTTPステータスコード・Cron式)の成果物を全体レビューしてください。
レビュー対象ファイル
- src/cheatsheets/http-status-codes/meta.ts(新規)
- src/cheatsheets/http-status-codes/Component.tsx(新規)
- src/cheatsheets/cron/meta.ts(新規)
- src/cheatsheets/cron/Component.tsx(新規)
- src/cheatsheets/registry.ts(変更)
- src/cheatsheets/tests/registry.test.ts(変更)
レビュー観点
コード品質
- 既存チートシート(regex, git, markdown)との構造的整合性
- TypeScript型安全性(CheatsheetMeta準拠)
- import文の正確性
- JSXの構文エラーがないか(特に418 I'm a teapotのエスケープ)
- h2のidとmeta.tsのsections[].idの対応関係
コンテンツ品質
- HTTPステータスコードの英語名・日本語説明の正確性(RFC 9110基準)
- Cron式の構文説明の正確性
- プラットフォーム別の違いの正確性(GitHub Actions, AWS EventBridge, Quartz)
- 日本語の自然さ・読みやすさ
- テーブルの列数・内容の一貫性
統合面
- registry.tsの登録が正しいか(5つのエントリ)
- relatedCheatsheetSlugs/relatedToolSlugsの参照先がregistryに存在するか
- registry.test.tsのカウントが正しいか
計画レビュー指摘の反映確認
- [HTTP-M2] HTTPのrelatedCheatsheetSlugsが["git", "regex"]になっているか
- [CRON-M1] Quartzの説明が「標準crontabの5フィールドの先頭に秒フィールドが追加される」になっているか
- [CRON-S3] 曜日の補足に限定表現が入っているか
ビルド検証
npm run lint && npm run format:check && npm run test && npm run build を実行して、すべて成功することを確認してください。
レビュー結果をメモで報告してください。Approve / Conditional Approve / Request Changes のいずれかで判定してください。