Repository navigation
chore: 未使用のテンプレ種別(static-html・base-overlay)とhtml stackを削除する - #656
Conversation
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01DWnryT8C2ehisFTnFkKjfr
|
Warning Review limit reachedNext included review available in 15 minutes. View limit detailsLimit details: You’ve used all 2 included reviews currently available. You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository. Review configuration: ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Advanced Run ID: 📒 Files selected for processing (8)
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 |
Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01DWnryT8C2ehisFTnFkKjfr
ddryo
left a comment
There was a problem hiding this comment.
Codex Review (by GPT-6)
Review baseline: 295cb87
Summary
未使用のstatic-html・base-overlay型とhtml stackの削除について、全8ファイルの差分と関連処理を確認しました。既存テンプレの登録、CLIの取得・言語overlay・variant抽出、サイトのサムネイル解決、プレビューのビルド単位との整合性を確認し、Blocking・Advisoryともに指摘はありません。
Validation
packages/lism-cliでnr test src/commands/create.test.tsを実行し、26テスト成功。packages/lism-cliでnr typecheckを実行し、成功。- 削除した型・メッセージの残存参照を検索し、実装側に削除漏れがないことを確認。
- サイト・プレビューの実ビルドは未実行。
Positives
未使用種別の削除に範囲を絞り、既存の言語overlayとvariant抽出を維持しています。型定義・実装・専用テスト・文言・運用文書が併せて整理されています。
概要
使われていないテンプレ種別を2つ削除します。
static-html型とstackのhtml: LPテンプレのHTML版をCLIで配る計画(feat(lism-cli): LP テンプレの HTML ビルド版を CLI 配布できるようにする #375)を見送るため。base-overlay型: #366の設計で入ったが、#372でLPにはsingle-project-variant型が採用され、使うテンプレが1つも無いため。HTML版を見送る理由
<script type="module">のため、file://で開くとブラウザがブロックして動かない。HTMLが欲しい場合は、Astro版を
npm run buildしてdist/を使えば足ります。base-overlayを消す理由
#372では、variant間で構造の違い(独自のkeyframesやclass prefix)が大きく、共通baseにする利点が薄いとしてLPへの採用を見送りました。残る想定用途は未作成の
webカテゴリだけですが、業種ごとに構造が違う点は同じです。今後もsingle-project-variant型に倣います。変更内容
templates/manifest.ts:StaticHtmlTemplateDef・BaseOverlayTemplateDefとStackの'html'を削除packages/lism-cli: 両型の分岐(取得・マージ・package名の書き換え・HTML向け案内)、stackの表示名html、専用メッセージ2件、テスト3件を削除apps/site/src/config/templates.ts: stackのhtmlと、sourcePathを持たない種別向けの例外処理を削除scripts/build-previews.mjs: stackのhtmlと、非対応kindのスキップ処理を削除documents/cli-guide.md: 両型の規約・予定の記述を削除docs/decisions.md: 2つの決定を追記確認
pnpm --filter lism-cli test: 110件成功nr typecheck: エラー0件nr lint: エラー0件(既存の警告のみ)Refs #375
🤖 Generated with Claude Code
https://claude.ai/code/session_01DWnryT8C2ehisFTnFkKjfr