🔖 chore: release v1.0.4 + リリース手順スキル化#276
Conversation
- package.json を 1.0.4 に bump - CHANGELOG.md を v1.0.1 〜 v1.0.4 まで遡って一括更新 - v1.0.1(Security/Changed)/ v1.0.2(README から internal 言及削除) - v1.0.3(Input 再利用フックの named export + triggerProps による属性フォワード) - v1.0.4(Button forwardRef 対応 / #275 closes #270、他依存更新) - リリース手順を skill 化(.claude/skills/release-sparkle-design/) - 過去 v1.0.1〜v1.0.3 で CHANGELOG 更新漏れ + v1.0.3 では Release/Tag 未作成だった反省を反映 - bump → CHANGELOG → PR → tag → release → publish の一連の手順をチェックリスト化 - .gitignore の skill allowlist にも追加 Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Walkthroughリリース手順を AI スキルとして新規追加し、チェックリスト(事前確認・release PR 作成・PR マージ・タグ/Release・npm publish・完了報告)とトラブルシューティング、参考リンクを記載。スキルディレクトリを git 管理対象にし、CHANGELOG に Unreleased と既存リリース履歴を追記。 ChangesRelease Sparkle Design スキル
🎯 2 (Simple) | ⏱️ ~12 minutes
🚥 Pre-merge checks | ✅ 5✅ Passed checks (5 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 9f841ceddf
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| git tag vX.Y.Z | ||
| git push origin vX.Y.Z |
There was a problem hiding this comment.
Tag the release commit explicitly
The checklist uses git tag vX.Y.Z without a target SHA, which tags the current HEAD at execution time. If main advances between PR merge and this step, the tag (and subsequent gh workflow run ... --ref vX.Y.Z) can include unrelated commits and publish the wrong code. Since this section already asks the operator to identify the release commit SHA, the command should tag that exact SHA (for example git tag vX.Y.Z <release_sha>).
Useful? React with 👍 / 👎.
There was a problem hiding this comment.
Pull request overview
v1.0.4 のリリース準備として、パッケージバージョンの更新と CHANGELOG の追補、および今後の更新漏れ防止のためのリリース手順スキル(チェックリスト)を追加する PR です。
Changes:
package.jsonの version を1.0.4に更新CHANGELOG.mdに v1.0.1〜v1.0.4 のセクションを追補.claude/skills/release-sparkle-design/を追加し、.gitignoreの skills allowlist に反映
Reviewed changes
Copilot reviewed 3 out of 4 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| package.json | v1.0.4 リリースに向けた version bump |
| CHANGELOG.md | v1.0.1〜v1.0.4 の変更履歴を追補 |
| .gitignore | 新規 release スキルディレクトリを追跡対象に追加 |
| .claude/skills/release-sparkle-design/SKILL.md | リリース手順のチェックリストをスキルとして追加 |
| - **Input まわりの再利用フックを named export** (#263) | ||
| - `useInputContainerFocus` / `useMergeRefs` を public API として公開 | ||
| - **`Input.triggerProps` による ARIA / HTML 属性フォワード** (#268) |
| ### npm publish が失敗する場合 | ||
|
|
||
| - `pnpm-lock.yaml` の整合性が崩れていないか確認(`pnpm install --frozen-lockfile` を試す) | ||
| - `pnpm.overrides` は pnpm 10+ では `pnpm-workspace.yaml` に書く必要がある(package.json では読まれない) |
公開 OSS リポジトリのスキルなので、sparkle-design-internal(社内パッケージ) の存在を露出しないよう書き直す。 - description / trigger phrase から internal 言及を削除 - npm publish 節を sparkle-design(npmjs.org)一本に統一 - 公開確認・consumer 更新の節からも internal を削除 - リポジトリ選択 / sparkle ワークスペース構造の説明を簡素化 internal 向け release skill が必要になれば、CLAUDE.md のスキル同期ルール 「internal 専用スキルは sparkle-design-internal で直接編集して OK」に沿って 向こうで別途追加する想定。 Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
Actionable comments posted: 1
🧹 Nitpick comments (1)
.claude/skills/release-sparkle-design/SKILL.md (1)
80-80: ⚡ Quick winlockfile 整合性チェックには
--frozen-lockfileを使用してください。
pnpm installだけでは lockfile が更新される可能性があります。リリース PR で lockfile の整合性を確認する目的であれば、pnpm install --frozen-lockfileを使用することで、lockfile に変更が必要な場合にエラーで失敗し、意図しない更新を防げます。♻️ 推奨される修正
-- [ ] `pnpm install` で lockfile が壊れていないか確認 +- [ ] `pnpm install --frozen-lockfile` で lockfile の整合性を確認🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the rest with a brief reason, keep changes minimal, and validate. In @.claude/skills/release-sparkle-design/SKILL.md at line 80, Update the lockfile-check step that currently references running `pnpm install` so it uses `pnpm install --frozen-lockfile` instead; locate the checklist item line containing the text "`pnpm install` で lockfile が壊れていないか確認" and change the command to include `--frozen-lockfile` so the CI/PR check fails if the lockfile requires changes.
🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
Inline comments:
In @.claude/skills/release-sparkle-design/SKILL.md:
- Line 42: Add a short, explicit policy entry describing the
`.claude/worktrees/<name>` convention to the project documentation (e.g., in
CLAUDE.md or a CONTRIBUTING section) that explains the required directory
structure, when to create a worktree, naming rules for `<name>`,
permissions/ownership expectations, and how to clean up or remove worktrees;
reference the existing mention in SKILL.md and point readers to the canonical
doc for details so AI/maintainers follow the single source of truth about
`.claude/worktrees/<name>`.
---
Nitpick comments:
In @.claude/skills/release-sparkle-design/SKILL.md:
- Line 80: Update the lockfile-check step that currently references running
`pnpm install` so it uses `pnpm install --frozen-lockfile` instead; locate the
checklist item line containing the text "`pnpm install` で lockfile が壊れていないか確認"
and change the command to include `--frozen-lockfile` so the CI/PR check fails
if the lockfile requires changes.
🪄 Autofix (Beta)
Fix all unresolved CodeRabbit comments on this PR:
- Push a commit to this branch (recommended)
- Create a new PR with the fixes
ℹ️ Review info
⚙️ Run configuration
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
Run ID: f02579f1-19c9-4aa9-a2c0-432b7f2cc323
📒 Files selected for processing (1)
.claude/skills/release-sparkle-design/SKILL.md
| 2. **`sparkle-design` リポジトリのルートで作業する** | ||
|
|
||
| - 編集・テスト・git 操作はすべて `sparkle-design` のチェックアウトディレクトリで実行 | ||
| - worktree を使う場合は `.claude/worktrees/<name>` 配下で作業 |
There was a problem hiding this comment.
🧩 Analysis chain
🏁 Script executed:
#!/bin/bash
# Description: Check if worktree convention is documented
# Search for worktree path convention in project docs
rg -n -C2 '\.claude/worktrees' --type=md
rg -n -C2 'worktree' CLAUDE.mdRepository: goodpatch/sparkle-design
Length of output: 50
.claude/worktrees/<name> の規約をドキュメントに追加してください。
.claude/worktrees/<name> というworktree配置の規約がSKILL.mdで言及されていますが、CLAUDE.mdを含むプロジェクト内のドキュメントに記載されていません。AI やメンテナーが混乱する可能性があるため、この規約をプロジェクトドキュメントで明確に定義する必要があります。
🤖 Prompt for AI Agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.
In @.claude/skills/release-sparkle-design/SKILL.md at line 42, Add a short,
explicit policy entry describing the `.claude/worktrees/<name>` convention to
the project documentation (e.g., in CLAUDE.md or a CONTRIBUTING section) that
explains the required directory structure, when to create a worktree, naming
rules for `<name>`, permissions/ownership expectations, and how to clean up or
remove worktrees; reference the existing mention in SKILL.md and point readers
to the canonical doc for details so AI/maintainers follow the single source of
truth about `.claude/worktrees/<name>`.
Summary
.claude/skills/release-sparkle-design/) — 今後の更新漏れ対策背景: なぜ CHANGELOG 一括追補が必要か
過去のリリース状況を調べたところ:
CHANGELOG は v1.0.0 で止まっており、v1.0.1 以降の差分が完全に未反映でした。リリース手順が完全に手動で、ドキュメント化もされていなかったのが原因。
この PR で CHANGELOG.md を v1.0.1〜v1.0.4 まで遡って一括更新します。
リリース手順 skill 化
.claude/skills/release-sparkle-design/SKILL.mdを新規作成。sparkle-designとsparkle-design-internal両方をカバーskill 同期の仕組み(CLAUDE.md 記載)により、マージ後は
sparkle-design-internal/sparkle-design-docsにもpnpm sync:public-skills/pnpm sync:skillsで伝播します。v1.0.4 の主な変更内容
Added
React.forwardRef対応 (♻️ refactor: Button を React.forwardRef 対応にする (#270) #275, closes ♻️ refactor: Button を React.forwardRef 対応にしてほしい(PopoverTrigger 等で asChild 合成するため) #270)Trigger asChildから ref を直接渡せるようにChanged
user-invocable: true(🔧 chore: スキルに user-invocable: true を明示 #271)Dependencies
このマージ後にやること(リリーススキルのチェックリスト準拠)
git tag v1.0.3 16a42e5 && git push origin v1.0.3gh release create v1.0.3(notes は CHANGELOG.md の v1.0.3 セクション)git tag v1.0.4 && git push origin v1.0.4gh release create v1.0.4gh workflow run "Publish to npm"で npm publishTest plan
🤖 Generated with Claude Code