Skip to content

feat: Sync Claude settings from Elu-co-jp projects - #281

Merged
keito4 merged 1 commit into
mainfrom
feat/sync-claude-settings-20260102
Jan 2, 2026
Merged

feat: Sync Claude settings from Elu-co-jp projects#281
keito4 merged 1 commit into
mainfrom
feat/sync-claude-settings-20260102

Conversation

@keito4

@keito4 keito4 commented Jan 2, 2026

Copy link
Copy Markdown
Owner

概要

Elu-co-jp 配下の全プロジェクトから .claude/settings.local.json を収集し、
共通設定を抽出して DevContainer 設定に反映しました。

収集元

  • リポジトリ数: 12 件
  • 設定ファイル数: 19 件(サブディレクトリ含む)

変更内容

追加された許可設定 (4件)

カテゴリ 追加項目
npm Bash(npm test) - 引数なしバージョン
npm Bash(npm --version)
vercel Bash(vercel:*)
vercel Bash(vercel env:*)

その他の修正

  • devcontainer.json の構文エラーを修正(カンマ欠落)

収集元リポジトリ

  • realtime_interview
  • management_tools
  • cyber_ace_1on1
  • cloud_provisioning
  • job_description
  • recall_ai
  • package_manager
  • sandbox
  • pulse_survey
  • TOPPAN
  • workflow-use
  • platform-infra

影響範囲

  • DevContainer イメージをビルドする全プロジェクト
  • 次回の DevContainer イメージビルド時から有効化

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated development container configuration to enable npm testing and deployment-related commands during development workflows.

✏️ Tip: You can customize this high-level summary in your review settings.

Elu-co-jp 配下の全プロジェクト(12リポジトリ、19設定ファイル)から
settings.local.json を収集し、共通設定を抽出して DevContainer 設定に反映。

## 追加された許可設定 (4件)
- Bash(npm test) - 引数なしバージョン
- Bash(npm --version) - バージョン確認
- Bash(vercel:*) - Vercel CLI コマンド
- Bash(vercel env:*) - Vercel 環境変数管理

## 収集元リポジトリ
realtime_interview, management_tools, cyber_ace_1on1, cloud_provisioning,
job_description, recall_ai, package_manager, sandbox, pulse_survey,
TOPPAN, workflow-use, platform-infra

## その他の修正
- devcontainer.json の構文エラーを修正(カンマ欠落)

🤖 Generated with [Claude Code](https://claude.com/claude-code)

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

coderabbitai Bot commented Jan 2, 2026

Copy link
Copy Markdown

Caution

Review failed

The pull request is closed.

📝 Walkthrough

Walkthrough

The changes expand CLI permissions in the development container settings. Four new Bash command permissions are added to allow npm test/version and Vercel operations. A JSON syntax fix adds a trailing comma to the devcontainer feature configuration.

Changes

Cohort / File(s) Summary
Dev Container Configuration
​.devcontainer/claude-settings.json
Added four new Bash command permissions: Bash(npm test), Bash(npm --version), Bash(vercel:*), and Bash(vercel env:*) to the permissions.allow array.
Dev Container Settings
​.devcontainer/devcontainer.json
Added trailing comma after the docker-in-docker feature block for proper JSON formatting before subsequent feature entries.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related issues

Possibly related PRs

Suggested labels

released

Poem

🐰 Permissions bloom in config files,
npm tests now dance and vercel smiles,
A comma tucked in json's place,
Dev containers find their grace!


📜 Recent review details

Configuration used: defaults

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between d487eaa and 0afe756.

📒 Files selected for processing (2)
  • .devcontainer/claude-settings.json
  • .devcontainer/devcontainer.json

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 and usage tips.

@keito4
keito4 merged commit 1a5f7b7 into main Jan 2, 2026
11 of 13 checks passed
@keito4
keito4 deleted the feat/sync-claude-settings-20260102 branch January 29, 2026 00:51
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