Skip to content

Refuse tracked Claude project-local settings - #336

Merged
rmcdaniel merged 1 commit into
mainfrom
agent/general/claude-local-settings
Nov 13, 2026
Merged

rmcdaniel merged 1 commit into
mainfrom
agent/general/claude-local-settings

Conversation

@rmcdaniel

Copy link
Copy Markdown
Member

Part of durable-workflow/.github#355 (other repositories remain, so this does not close it).

Same change as waterline#290 and sdk-php#379:

  • .gitignore covers .claude/settings.local.json and **/.claude/settings.local.json.
  • scripts/check-public-boundary.sh refuses any tracked .claude/settings.local.json (root or nested) before the content scans. It runs git ls-files over the whole tracked tree, so it catches a copy that was already tracked before the PR range. It prints only the path and never reads the file. The existing content and metadata scans are unchanged.

Local checks: the boundary script passes on this branch. In a scratch repo with this script and .gitignore, tracked root and nested copies exit 1 without printing content, untracked copies are ignored and pass, and .claude/settings.json, .claude/settings.local.json.example and c/settings.local.json pass.

🤖 Generated with Claude Code

Add the durable-workflow/.github#355 tracked-path refusal to
scripts/check-public-boundary.sh, before the content scans. It scans the
complete tracked tree, prints only the path, and never reads the file.
.gitignore now covers root and nested .claude/settings.local.json.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@rmcdaniel rmcdaniel added the agent:general Pull request opened by General agent label Nov 13, 2026
@sonarqubecloud

Copy link
Copy Markdown

@rmcdaniel

Copy link
Copy Markdown
Member Author

Security: PASS. The .gitignore ignores .claude/settings.local.json and **/.claude/settings.local.json, and scripts/check-public-boundary.sh fails (exit 1) when git ls-files lists one. It prints only the path, never the file contents, matching .github#356 and sdk-php#379 for .github#355. (head ec710e4e)

@rmcdaniel
rmcdaniel merged commit 15c8a4c into main Nov 13, 2026
30 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

agent:general Pull request opened by General agent

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant