Skip to content

docs(skills): 編集系スキルに変更理由を残させる (notedeck#1052) - #42

Merged
hitalin merged 2 commits into
mainfrom
feat/edit-reason-for-write-skills
Aug 12, 2026
Merged

docs(skills): 編集系スキルに変更理由を残させる (notedeck#1052)#42
hitalin merged 2 commits into
mainfrom
feat/edit-reason-for-write-skills

Conversation

@hitalin

@hitalin hitalin commented Aug 12, 2026

Copy link
Copy Markdown
Collaborator

NoteDeck の編集履歴が「誰が・なぜ」を記録できるようになった (notedeck-dev/notedeck#1052)。履歴を持つ write capability を呼ぶ配布スキルに、reason へ変更理由を渡す指示を追加する。

渡した理由は承認ダイアログに出て、承認後そのまま編集履歴に残る。「AI が直したはずが壊れた」を後から辿れるようになる。

対象 (9 件)

スキル version 呼ぶ capability
aiscript-surgeon 0.1.0 → 0.2.0 plugins.update / plugins.revert / widgets.update / widgets.revert
skill-gardener 0.1.0 → 0.2.0 skills.replaceSection / skills.revert
skill-author 1.0.0 → 1.1.0 skills.append / skills.replaceSection
plugin-author 1.2.1 → 1.3.0 plugins.update / plugins.revert / widgets.update
widget-author 1.1.1 → 1.2.0 widgets.update / widgets.revert
theme-author 1.0.0 → 1.1.0 theme.create / theme.update
theme-reference 1.0.0 → 1.1.0 theme.create
seasonal-theme 0.1.0 → 0.2.0 theme.create
writing-coach 0.1.0 → 0.2.0 skills.replaceSection

プラグイン・ウィジェットの配布アイテムに該当はなかった (AiScript から編集系 capability を呼ぶものが無い)。

書き方の方針

新しい作法を足したのではなく、既にあった指示を記録に繋いだ。 対象の多くは元から「変更理由を 1 行添えて同意を得る」「なぜ直るかを説明してから適用する」と書いてあったので、その口頭で述べていた理由をそのまま reason に渡す形に揃えている。

他人のアイテムを書き換えるスキル (aiscript-surgeon / skill-gardener) ほど効く。巻き戻し (*.revert) にも理由を残させているので、何を試して駄目だったかが次の診断の材料になる。

無関係な差分について

再ビルドで reaction-piggybackcreatedAt / updatedAt がビルド実行時刻から git コミット日時に直る差分が出た。前回のビルドが未コミット状態で走った名残で、本 PR の趣旨とは無関係なため chore(registry) として別コミットに分けてある。配布物の中身は変わっていない。

確認

  • pnpm run registry:build 成功
  • node scripts/check-registry-integrity.mjs — 完全性チェック OK (警告 0 件)

🤖 Generated with Claude Code

Summary by CodeRabbit

  • New Features
    • Updated several authoring skills to require explanations for edits, updates, and rollbacks.
    • Change reasons now appear in approval dialogs and are preserved in edit history.
    • Theme creation and editing can record the rationale behind seasonal or corrective changes.
    • Updated skill versions and refreshed registry metadata.
  • Documentation
    • Added examples and guidance for providing change and rollback reasons.

hitalin and others added 2 commits August 13, 2026 02:40
reaction-piggyback の createdAt / updatedAt がビルド実行時刻のまま残って
いたので、git コミット日時に揃える。以降のビルドで無関係な差分が出ない
ようにするための追従で、配布物の中身は変わっていない。

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
NoteDeck の編集履歴が「誰が・なぜ」を記録できるようになった。履歴を持つ
write capability (skills / plugins / widgets / theme / カスタム CSS) を呼ぶ
スキルに、`reason` へ変更理由を渡す指示を追加する。

承認ダイアログに出て、そのまま編集履歴に残る。他人のアイテムを書き換える
スキル (aiscript-surgeon / skill-gardener) ほど効く — 「AI が直したはずが
壊れた」を後から辿るのがこの手のスキルの用途そのもののため。

対象 9 件。いずれも既に「変更理由を 1 行添えて同意を得る」「なぜ直るかを
説明する」という指示を持っていたので、その理由をそのまま reason に渡す形に
揃えた。新しい作法を足したのではなく、口頭で述べていた理由を記録に残す。

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying with  Cloudflare Workers  Cloudflare Workers

The latest updates on your project. Learn more about integrating Git with Workers.

Status Name Latest Commit Updated (UTC)
✅ Deployment successful!
View logs
misstore 6bed728 Aug 12 2026, 05:41 PM

@coderabbitai

coderabbitai Bot commented Aug 12, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 209edd34-3546-41c7-b975-811916538df4

📥 Commits

Reviewing files that changed from the base of the PR and between 63fe03a and 6bed728.

📒 Files selected for processing (24)
  • public/registry/index.json
  • public/registry/plugins.json
  • public/registry/queries.json
  • public/registry/skills.json
  • public/registry/skills/aiscript-surgeon/api.json
  • public/registry/skills/aiscript-surgeon/skill.md
  • public/registry/skills/plugin-author/api.json
  • public/registry/skills/plugin-author/skill.md
  • public/registry/skills/seasonal-theme/api.json
  • public/registry/skills/seasonal-theme/skill.md
  • public/registry/skills/skill-author/api.json
  • public/registry/skills/skill-author/skill.md
  • public/registry/skills/skill-gardener/api.json
  • public/registry/skills/skill-gardener/skill.md
  • public/registry/skills/theme-author/api.json
  • public/registry/skills/theme-author/skill.md
  • public/registry/skills/theme-reference/api.json
  • public/registry/skills/theme-reference/skill.md
  • public/registry/skills/widget-author/api.json
  • public/registry/skills/widget-author/skill.md
  • public/registry/skills/writing-coach/api.json
  • public/registry/skills/writing-coach/skill.md
  • public/registry/themes.json
  • public/registry/widgets.json

📝 Walkthrough

Walkthrough

The registry timestamps and plugin timestamps are refreshed. Nine skills receive version and hash updates. Their instructions now document reason values for edits, creation, replacements, and rollbacks.

Changes

Skill registry update

Layer / File(s) Summary
Skill reason workflows
public/registry/skills/*/api.json, public/registry/skills/*/skill.md
Nine skills now require or document reason values for update, replace, create, and revert operations.
Skill catalog synchronization
public/registry/skills.json
The registry updates versions and SHA-512 hashes for the nine changed skills.
Registry metadata refresh
public/registry/index.json, public/registry/plugins.json, public/registry/queries.json, public/registry/themes.json, public/registry/widgets.json
Registry timestamps are refreshed. The reaction-piggyback plugin timestamps are also updated. Counts and structure remain unchanged.

Estimated code review effort: 2 (Simple) | ~15 minutes

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch feat/edit-reason-for-write-skills

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@hitalin hitalin self-assigned this Aug 12, 2026
@hitalin
hitalin merged commit c5d6c32 into main Aug 12, 2026
4 of 5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant