Blades を分割版の CSS に移し、使う部品だけ読む

[5ym]

何を

• @anyblades/blades を 3.1.0-alpha.1 に上げました (版は固定のまま)。
  • alpha を使う理由: Pico を部品ごとに読めるようになった (@anyblades/blades/pico/<分類>/<部品> の exports) のがこの版からで、正式版 (latest = 3.0.0) にはまだ無いため。dist-tags は latest: 3.0.0 / canary: 3.1.0-alpha.1 / alpha: 0.28.0-alpha.8 (古い系列) で、最新の alpha は canary の 3.1.0-alpha.1 です。
  • 中身は 3.0.0 の各ファイルを分けただけで、部品ごとにつなげると 3.0.0 と同じでした (違いは utilities から IE 向けの -ms-touch-action と [aria-hidden=false][hidden] が消えたことだけ。どちらも denpa には効いていない)。
• src/app.css は Pico を丸ごとではなく、使う部品だけ同じ並びで読みます。
  • 読まない (当てる要素が無い): container・grid・overflow-auto・figure・色/日付/ファイルの入力欄・dropdown・tooltip
  • 読まない (読んでから打ち消していた): コード・検索欄の虫眼鏡・ナビ。打ち消しの塊 (anyblades/discussions#168 待ちだったもの) を消しました
  • 打ち消しのうち効いていたもの (<pre> の字の色と太さを周りに合わせる) と、ナビの部品が付けていた li の余白なしは残しました (+layout.svelte)
• 組んだ CSS を前後で比べ、消えた規則がどれも denpa の要素に当たらないか、打ち消しで効いていなかったことを確かめました (aside li などナビの部品の規則も、当たる側が自前で同じ値を持っている)。CSS は 99KB → 82KB。

確かめたこと

• wslc-compose で bun install (bun.lock 更新)・bun run lint・bun run check・bun run build
• e2e は CI
• 主要画面のスクリーンショット (Screenshots のワークフロー) を main とこの枝で撮って見比べ

🤖 Generated with Claude Code

https://claude.ai/code/session_017wBP4MK6j3oBwqbn95wZ8H

[claude]

Claude finished @5ym's task in 14s —— View job

---

レビュー結果

重大な問題は見つかりませんでした。

確認した内容

• package.json と bun.lock は 3.1.0-alpha.1 に固定されていて、^ の浮動はありません。
• src/app.css の @import は、全て layer(pico) 付きのまま分割されています。レイヤー順は変わりません。
• 打ち消しの塊を消した分は、pre の色・太さと、+layout.svelte の li の margin: 0 で補われています。
• +layout.svelte では、ul.links に align-items: center を足しています。ナビの nav ul が持っていた flex の揃えを、自前で持つ形に置き換えたものと読みました。

要確認 (差し戻しではなく、念のため)

1. 自分では bun install もビルドもしていないので、組んだ CSS の前後比較は PR 本文の記載を信じています。ul.burger-list の padding と margin は、Pico のナビの部品が nav ul に付けていたものがなくなります。文章の決まり (ul の padding-inline-start と margin-bottom) が当たるので、畳んだメニューのスクリーンショットで余白が崩れていないかだけ見てください。
2. alpha 版 (canary) への固定は、PR 本文の理由で納得できます。正式版が出たら戻す目印として、package.json の近くか Issue に残しておくと忘れにくいです。
3. テストは見た目だけの変更なので、e2e とスクリーンショットの比較で足ります。新規テストは不要と考えます。

セキュリティ上の懸念 (シークレットや入力値の扱い) はありません。

[claude]

Claude finished @5ym's task in 14s —— View job

---

レビュー結果

重大な問題は見つかりませんでした。

確認した内容

• @anyblades/blades は 3.1.0-alpha.1 に固定されていて、^ の浮動はありません。
• src/app.css の @import は全て layer(pico) 付きで、並びも Pico の _index.css と同じと書かれています。レイヤー順は変わりません。
• ul.burger-list は margin: 0; padding: 0 を自前で持っています。前回の「要確認」にした畳んだメニューの余白は心配ありません。
• ul.links の下の余白は、app.css の ul, … { margin-bottom: 0 } (233〜245 行付近) が打ち消します。li の margin: 0 も足されています。
• ul.links の align-items: center は、nav ul が持っていた flex の揃えを自前で持つ形で、筋が通っています。
• pre の色と太さの打ち消しは残っています。code・kbd などの打ち消しを消した分は、Pico のコードの部品を読まなくなったことで補われます。

要確認 (差し戻しではありません)

1. 自分では bun install もビルドも実行していません。CSS の前後比較は PR 本文の記載を信じています。
2. alpha (canary) への固定は理由に納得できます。正式版が出たら戻す目印を、Issue などに残しておくと忘れにくいです。
3. 見た目だけの変更なので、e2e とスクリーンショットの比較で足ります。新規テストは不要と考えます。

セキュリティ上の懸念 (シークレットや入力値の扱い) はありません。