forked from mirrors/misskey
* chore(docs): .claude配下の再構成 * fix AGENTS.md * fix AGENTS.md * fix review * 行番号参照の除去 * docs: fix storybook note in vue reviewer agent * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * Potential fix for pull request finding Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com> * fix local review * fix --------- Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com> Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
4.6 KiB
4.6 KiB
| name | description |
|---|---|
| shipping-misskey-change | Use at every "finish" moment of a Misskey change — immediately before committing, opening a PR, merging, or handing the work back to the user even without a commit. Runs the final pre-ship checklist — `pnpm lint`, misskey-js regeneration (`pnpm build-misskey-js-with-types`) when backend API changed, `pnpm --filter backend check-migrations` when entities or migrations changed, SPDX header verification on new files, locale safety check (no edits to non-`ja-JP` locale yml files), and `CHANGELOG.md` Unreleased entry for user-visible changes. Must be consulted as the last step of every change — including uncommitted handoffs — to avoid CI failures and lost translations. This is NOT waived by having already invoked brainstorming, writing-plans, or any other upstream skill — invoke this regardless of what preceded it. |
shipping-misskey-change
Misskey の変更の finish 局面 (commit / PR / merge する直前、またはコミットせずユーザーに作業を返す直前) に必ず走らせる最終チェックリスト。
CI で落ちやすい / レビュアーから指摘されやすいポイントを 1 箇所に集めている。後で references を辿る余裕を作らないため、チェックリストは SKILL.md 本体に直書きする。
他スキル実行後も免除されない。 brainstorming / writing-plans / その他アップストリームスキルを先に呼んでいても、作業を返す直前・commit 直前のタイミングでこのスキルを呼ぶこと。
最終チェックリスト
このリストを TodoWrite に展開して 1 項目ずつ確認すること。該当しない項目は飛ばして良いが、判断は明示する。
- lint が通る — ECC 由来の /quality-gate コマンドで lint (typecheck + eslint) + 高速テストをまとめて回すのが基本。lint だけ単発で確認したいなら
pnpm lint直接でもよい - backend で
meta/paramDef/resを変更した →pnpm build-misskey-js-with-typesを実行してpackages/misskey-js/src/autogen/の差分も commit に含めた → 詳細手順は references/tasks/regenerate-misskey-js.md - エンティティ (
packages/backend/src/models/*.tsの@Column/@Entity/@Index) を変更した →pnpm --filter backend check-migrationsが pending DDL 0 件で通る - migration ファイルを追加した →
up()とdown()の両方を実装した / 既存のマージ済 migration は一切触っていない - 新規
.ts/.js/.cjs/.mjs/.vue/.scss/.htmlファイルを追加した → SPDX ヘッダーを付けた (.vue/.htmlは HTML コメント形式、その他は TS コメント形式) locales/を編集した →ja-JP.ymlだけ を変更しており、他言語 yml の diff は出ていない (git diff --name-only develop -- 'locales/*.yml' | grep -v '^locales/ja-JP\.yml$'が空)- ユーザーから見える変更 (機能追加 / 既存挙動変更) →
CHANGELOG.mdの## Unreleased直下の該当サブセクション (General / Client / Server) に 1 行追記した → 詳細書式は references/tasks/changelog-update.md - backend API endpoint を追加・変更した → misskey-api-reviewer agent を Task で起動して機械レビューする (endpoint-list 登録漏れ / misskey-js 再生成漏れ / meta・UUID / SPDX。lint や CI では拾いにくい 404・登録漏れの最終関門なので、該当する変更があれば飛ばさない)
- frontend の
.vueを追加・変更した → vue-component-reviewer agent を Task で起動して機械レビューする (SPDX 形式 / 命名 / i18n / SCSS 変数 / os.* / a11y / Storybook 併設) - (任意)
.claude/ハーネス自体の健全性を確認したい → ECC 由来の /harness-audit コマンドを実行
何のためのスキルか
これは「作業中に何を作るか」を決めるスキルではなく、「作り終わった後に CI を通す」スキル。working-on-backend / working-on-frontend から始まった作業の 出口 として機能する。
該当する変更がある場合は各 references/tasks/ を Read して詳細手順を踏むこと。pnpm lint だけは references を読まずに直接走らせて良い (/quality-gate でまとめて回せる)。