Skip to content

feat(skills): アカウント別の配色スキル account-accent を追加 - #86

Merged
hitalin merged 1 commit into
mainfrom
feat/new-for-notedeck-1.85
Oct 13, 2026
Merged

hitalin merged 1 commit into
mainfrom
feat/new-for-notedeck-1.85

Conversation

@hitalin

@hitalin hitalin commented Oct 13, 2026

Copy link
Copy Markdown
Collaborator

なぜ

NoteDeck v1.85.0 で themes.apply が accountId を受けるようになり、テーマをそのアカウントのカラムだけに当てられるようになった (notedeck-dev/notedeck#1247)。複数アカウントを使う人が、どのアカウントのカラムかを色でひと目で見分けられるようにする (誤爆防止)。

内容

スキル account-accent (アカウント別の配色、0.1.0)。

  • アカウント一覧を取り、元にするテーマはユーザーに選んでもらう (今選ばれているテーマを読む手段が無いので推測しない)
  • アカウントごとに色相を離した accent を決め、元テーマの props を写して accent 周りだけ変えた派生テーマを themes.create で作る
  • 同意を得てから themes.apply({ id, accountId }) で当てる。themes.apply は確認ダイアログが出ずにすぐ反映されるため、先に一覧を見せる。accountId を付け忘れるとデッキ全体が変わる旨も明記
  • 戻し方 (そのアカウントのテーマ管理カラムから外す) と、色の直し方 (themes.update) を案内する

検討して出さなかったもの

期限つきミュートのプラグインも作ったが出さない。本家 Misskey Web はミュート時に期間を選べる標準機能なので、NoteDeck 本体の機能として足す (notedeck-dev/notedeck#1260)。

確認

  • registry:build / レジストリ完全性チェック OK
  • pnpm contract:verify: 149 件中、合格 149 / 不合格 0
  • 実機では未確認。v1.85.0 の公開後にマージする

🤖 Generated with Claude Code

複数アカウントの誤爆を防ぐため、アカウントごとに accent だけを変えた派生
テーマを作り、そのアカウントのカラムだけに当てる。NoteDeck 1.85.0 の
themes.apply の accountId を使う。

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Oct 13, 2026

Copy link
Copy Markdown

Review in Change Stack →

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration
  • Configuration used: defaults
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 9a95a89b-a4b6-4797-8f48-1ed539250024
📥 Commits

Reviewing files that changed from the base of the PR and between b8b7c94 and 0fefdd0.

⛔ Files ignored due to path filters (1)
  • public/registry/skills/account-accent/icon.svg is excluded by !**/*.svg
📒 Files selected for processing (9)
  • public/registry/index.json
  • public/registry/plugins.json
  • public/registry/queries.json
  • public/registry/skills.json
  • public/registry/skills/account-accent/api.json
  • public/registry/skills/account-accent/locales/en.json
  • public/registry/skills/account-accent/skill.md
  • public/registry/themes.json
  • public/registry/widgets.json
 ___________________________________________________________________
< That's not technical debt - that's technical *predatory lending*. >
 -------------------------------------------------------------------
  \
   \   \
        \ /\
        ( )
      .( o ).
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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 Oct 13, 2026
@hitalin
hitalin merged commit 67f0b7f into main Oct 13, 2026
4 of 5 checks passed
@hitalin

hitalin commented Oct 15, 2026

Copy link
Copy Markdown
Collaborator Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Oct 15, 2026 •

Copy link
Copy Markdown
⚠️ Action not completed

Pull request is closed.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

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