Repository navigation
refactor: 古い参照・重複・使われていないコードを整理する - #28
Merged
Merged
Conversation
原則「削る」に、部品を組み合わせたページで適用するルールを追加した。 - 必須: 見出し・配置・まとまりで伝わることを、小見出し・補助ラベル・説明文で繰り返さない - 必須: 項目どうしの関係（グループ・主従）はレイアウトで示し、説明文で補わない - 必須: 削るのは重複した表示だけ。入力欄の見えるラベル（WCAG 3.3.2）と構造（1.3.1）、アイコンだけの操作の名前は残す - 推奨: 補足文が要りそうなら、先に配置・見出し・ラベルの語を直す - 出典カタログ（ゲシュタルトの近接・共通地域）、ルールブック 01・05、DESIGN.md の Don't に反映 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Yb44feYKDNj2ukQeT2j8Ks
…8341/design-md を追加 - レジストリ item `design-md`: リポジトリ直下に DESIGN.md、docs/design-principles/ に原則の全文を置く - プロダクト用 DESIGN.md（templates/PRODUCT-DESIGN.md）を、コードから読み取れない考え方だけの構成に書き直した（プロダクトの前提・画面領域ごとの方向性・大事にすること・外した必須）。トークンや部品の一覧は載せない - 原則の要約（判断の順序・ルールの強さ・方向性と各原則の必須）は scripts/sync-product-design.ts が docs/design-principles から生成し、make check で鮮度を検査 - build-registry-css.ts を build-registry-assets.ts に改名し、DESIGN.md と原則（ファクトリーの部分を除く）も registry-assets/ に作る - README のクイックスタート、ds-build-ui、DESIGN.md、ルールブック 01、design-principles/README を更新。ADR-0019、changeset、改善ログ Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Yb44feYKDNj2ukQeT2j8Ks
- 古い参照を修正: 存在しない make ターゲット（check:contrast）、ds-improve のトークンと contrast.rules.json のパス、factory.md の README の節名、ワークフローのコメント（pnpm → make、手順の置き場所）、publish-preview.ts の使い方 - トークンを変えた後の手順を make generate に統一（CONTRIBUTING・03・06・operations・ds-improve・ds-factory） - Makefile と重複していた pnpm スクリプト（storybook:preview・factory:create・release:*・preview:publish）と、create.mjs のその削除処理を外した - design-review から、コードレビュー・Next.js 専用の項目を外した（画面デザインのレビューに限る） - DESIGN.md の「作業前に必ず読むもの」（全文を読む指示・古いパス）と、古い名前の 'use client' の一覧を直した - 重複を正本へのリンクに: DESIGN.md の共通規約とルールブック 04、ファイル構成のツリー（COMPONENT_STRUCTURE.md）、06 章と ds-improve、factory.md と ds-factory、make check の説明 - トークンの CSS 化の規則を packages/tokens/scripts/lib/css.ts に集め、build.ts とスライドで共有。build.ts の cssVar も cssVarName を使う - 使われていない関数（dimensionToString・relFromRepo・srcDir・oklchToHex・Layout）と、内部でしか使わない export を削除 - create.mjs: lstat の二重呼び出しをやめ、--force で既にあるシンボリックリンクがあっても落ちないようにした - make clean でスライドの出力（dist/）も消す Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Yb44feYKDNj2ukQeT2j8Ks
Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Yb44feYKDNj2ukQeT2j8Ks
# Conflicts: # docs/improvement/log.md # package.json
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
変更内容
全体を見直し、古い参照・矛盾・重複・使われていないコードを整理しました。振る舞いと見た目は変えていません。
古い参照・矛盾
check:contrast）、ds-improve のトークンとcontrast.rules.jsonのパス、factory.md の README の節名、ワークフローのコメント（pnpm storybook:preview・pnpm release:*→ make、手順の置き場所）、publish-preview.ts の使い方'use client'の対象が古いケバブケースの名前で手書きされていた → ルールブック 04 へmake generateに統一削除
storybook:preview・factory:create・release:version・release:publish・preview:publish）と、create.mjs でそれを消すだけの処理design-reviewに混ざっていたコードレビューの項目と Next.js 専用の項目（「next/link を使っているか」など）。画面デザインのレビューに限りました。「提案だけ行い修正はしない」は残していますdimensionToString・relFromRepo・srcDir・oklchToHex・Layout）と、内部でしか使わない export重複を正本へのリンクに
make checkの説明packages/tokens/scripts/lib/css.tsに集め、build.ts とスライドで共有。スライドのトークン CSS は、影の色の書き方が tokens.css と同じrgb(… / …)になっただけで、ほかは変更前と一致することを確認小さな修正
--forceで既にあるシンボリックリンクがあっても落ちないように（生成と--forceの上書きを実行して確認）make cleanでスライドの出力（dist/）も消すStorybook プレビュー
なし（コンポーネントとトークンの値は変更なし）
完了の定義
make checkとmake testが通る（CI で確認。build.ts は手元で Style Dictionary を入れられず未実行）docs/decisions/に ADR を足した（方針の変更なし）改善ループ
docs/improvement/log.mdに 1 行追記したGenerated by Claude Code