Skip to content

docs(agents): .claude/settings.json に pnpm lint/test/build のパーミッションを追加 #17616

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
fruitriin wants to merge 5 commits into
base: develop
Choose a base branch
from
+17 −0
Open

docs(agents): .claude/settings.json に pnpm lint/test/build のパーミッションを追加 #17616

Changes from 4 commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
98e8870
chore: .claude/settings.json に pnpm lint/test/build のパーミッションを追加
fruitriin Jun 24, 2026
6d18d47
開発プロセスで頻出のパーミッションの追加
fruitriin Jun 24, 2026
a780301
fix
fruitriin Jun 24, 2026
d56a204
レビュー指摘内容反映
fruitriin Jun 26, 2026
dc1916d
fix indent
fruitriin Jun 28, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
17 changes: 17 additions & 0 deletions .claude/settings.json
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,21 @@
{
"permissions": {
"allow": [
"Bash(pnpm lint)",
"Bash(pnpm --filter * lint)",
"Bash(pnpm --filter backend check-migrations)",
"Bash(pnpm --filter * typecheck)",
"Bash(pnpm build)",
Comment thread
kakkokari-gtyih marked this conversation as resolved.
Outdated
"Bash(pnpm build-misskey-js-with-types)",
"Bash(cp .github/misskey/test.yml .config/test.yml)",
"Bash(ncp .github/misskey/test.yml .config/test.yml)",
"Bash(pnpm backend-unit-test)",

@samunohito samunohito Jun 25, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Suggested change
"Bash(pnpm backend-unit-test)",

"Bash(pnpm --filter backend test)"があるので不要かと思います（やってることは同じ / LLMが混乱しそうなので統一したほうが良さそう）

@fruitriin fruitriin Jun 26, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

settings.json は システムプロンプトに注入されないので、LLMが選択に悩む原因にはならないと思います
LLMがどちらの選択もとりえるならどちらもないと設定がない方が選ばれたときに止まってしまうほうが不都合かなと。

"Bash(pnpm --filter backend test)",
Comment thread
kakkokari-gtyih marked this conversation as resolved.
Outdated
"Bash(pnpm --filter backend test:e2e)",
"Bash(pnpm --filter backend test:fed)",
"Bash(pnpm --filter frontend test)"
]
},
"enabledPlugins": {
"frontend-design@claude-plugins-official": true,
"superpowers@claude-plugins-official": true,
Expand Down