Skip to content

new settings.json file, changes to gitignore and CLAUDE files#91

Merged
sierragolflima merged 3 commits intomainfrom
claude-settings
Apr 19, 2026
Merged

new settings.json file, changes to gitignore and CLAUDE files#91
sierragolflima merged 3 commits intomainfrom
claude-settings

Conversation

@sierragolflima
Copy link
Copy Markdown
Contributor

Restructured Claude Code configuration for better security and team collaboration. Moved team-shared rules from .claude/settings.local.json (incorrectly checked into git) to .claude/settings.json (committed), while personal/machine-specific rules remain in the local file (now gitignored).

Tightened security by narrowing overly broad permissions. Destructive Azure commands were replaced with read-only patterns, and rm:, git commit:, and git push:* now require confirmation. Added a deny list to hard-block irreversible operations like force pushes, hard resets, sudo, and resource group deletions.

Moved the "never commit/push automatically" rule from CLAUDE.md to the permissions system. Runtime-enforced rules are deterministic, and this saves tokens on every session. Also cleaned up accidental one-off entries that had accumulated in the allow list.

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 16, 2026

Test Results

2 830 tests  ±0   2 817 ✅ ±0   5m 53s ⏱️ +4s
   34 suites ±0      13 💤 ±0 
   34 files   ±0       0 ❌ ±0 

Results for commit 177a5df. ± Comparison against base commit 453c31f.

♻️ This comment has been updated with latest results.

@sierragolflima
Copy link
Copy Markdown
Contributor Author

@copilot resolve the merge conflicts in this pull request

@sierragolflima sierragolflima marked this pull request as ready for review April 17, 2026 15:24
Copy link
Copy Markdown

@rosalie1984 rosalie1984 left a comment

Choose a reason for hiding this comment

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

discussed with Samuel and it makes sense to not have everything in CLAUDE.md and instead explicitly state hard permission and limits in the setting file - the 3 remaining conflicts can be resolved with git copilot - there are no test fails

Samuel Glauser added 2 commits April 19, 2026 23:02
# Conflicts:
#	.claude/settings.json
#	.gitignore
#	CLAUDE.md
@sierragolflima sierragolflima merged commit 65e9142 into main Apr 19, 2026
2 checks passed
@sierragolflima sierragolflima deleted the claude-settings branch April 19, 2026 21:26
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.

2 participants