docs(skill): generate --scope による単一バンドルランタイム切替の案内を追加#288
Merged
Conversation
sparkle-design-internal#229 対応（sparkle-design-cli#68）で generate --scope オプションが追加されたことを受け、change-sparkle-config / setup-sparkle-design の複数テーマ案内を「ケース A: config分割」「ケース B: generate --scope」の 2択に更新。単一バンドル内でランタイムに切り替えたいケースを、これまでの 「config分割一択」の案内から適切な方に振り分けられるようにする。 NOTE: このPRは sparkle-design-cli#68 がマージ・npm公開（2.4.0）されるまで マージしないこと（npx --yes は常に最新を取得するため、未公開の挙動を 案内してしまう事故を防ぐ）。 Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_01Rt4q3UhBbZZ4XwAHsJKgyH
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: 📒 Files selected for processing (2)
Disabled knowledge base sources:
Walkthrough複数の固定配色バリアントを切り替える要望に対し、切替方式を確認してケースA/Bを案内する手順を追加しました。非推奨方式とスキル中断条件も明記しています。 Changes複数配色バリアント案内
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
Poem
✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Pull request overview
sparkle-design-cli に追加予定の generate --scope（単一バンドルでのランタイム切替）を、Claude skills の案内・参照ドキュメントに反映し、複数テーマ運用の導線を「ケース A/B」に整理するPRです。
Changes:
setup-sparkle-designの config リファレンスに、複数テーマ運用の「ケース A/B」（config 分割 vsgenerate --scope）の案内を追記change-sparkle-configで、複数バリアント要望時に 1 回だけ短くケース判定して適切な README 案内へ誘導する方針に更新
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| .claude/skills/setup-sparkle-design/references/config-reference.md | 複数テーマ運用の選択肢（ケース A/B）と generate --scope の位置づけを追記 |
| .claude/skills/change-sparkle-config/SKILL.md | 複数バリアント要望をケース分岐で案内し、本スキル範囲外として中断する指示を更新 |
| - **画面（ビルド）ごとにバリアントが固定される**（レイアウト単位で読み込む CSS を出し分けられる）→ sparkle-design-cli README の「複数のテーマ配色を 1 デプロイでサポートしたい場合」の **ケース A**（config を分割して複数回 `generate`。生成した複数 CSS を同時 import してはいけない）を案内する | ||
| - **単一バンドル内でランタイムに切り替えたい**（`data-*` 属性の付け外しだけで切り替え、ページ遷移や再ビルドを伴わない）→ 同 README の **ケース B**（`generate --scope` で、ベースの通常 `generate` 出力とセマンティックトークンだけをスコープしたリテラル値の差分 CSS を同時 import する。sparkle-design-cli `2.4.0` 以降が必要）を案内する | ||
| - 単一 CSS + 属性スコープでの Tailwind クラス上書きは（どちらのケースでも）非推奨 | ||
| いずれの場合も本スキルはここで中断する（実際の config 分割・`--scope` 実行はユーザー自身か別の作業に委ねる）。 |
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.
Summary
generate --scopeオプションが追加された（sparkle-design-cli#68, 現在ドラフト・未マージ・未リリース）change-sparkle-config/setup-sparkle-designの複数テーマ案内を、「ケース A: config分割（画面ごとに固定）」「ケース B:generate --scope（単一バンドルでランタイム切替）」の2択に更新。change-sparkle-configはユーザーの要望を1回だけ短く聞き分けて適切な方に誘導するこのPRは sparkle-design-cli#68 がマージ・npm公開（
2.4.0）されるまでマージしないこと。npx --yes sparkle-design-cliは常に最新版を取得するため、cli側が未公開のままこのPRをマージすると、AIエージェントが「まだ存在しない--scopeオプション」を案内してしまう事故に繋がる（workspace CLAUDE.md のリリース手順に記載のパターン）。Test plan
npx prettier --config ./.prettierrc --check対象2ファイル — clean2.4.0npm公開を確認してからマージするgh workflow run "Publish Skills"）の両方を実行する🤖 Generated with Claude Code
https://claude.ai/code/session_01Rt4q3UhBbZZ4XwAHsJKgyH