Add Claude setup#168
Conversation
60853ad to
b9a1de1
Compare
There was a problem hiding this comment.
Code Review
This pull request updates the client infrastructure setup runbook and access permissions documentation to include guidelines for setting up Claude (platform.claude.com) organizations and billing. A review comment points out that a local settings file .claude/settings.local.json was accidentally committed and should be removed, with .claude/ added to .gitignore.
I am having trouble creating individual review comments. Click here to see my feedback.
.claude/settings.local.json (1-7)
It appears that .claude/settings.local.json was accidentally committed.
This file contains local permissions for a client-specific MCP server (brewale-abtion-mcp), which is unrelated to this company handbook repository. Local settings files generated by Claude Code (like *.local.json) are developer-specific and should not be committed to version control.
Recommendation
- Remove this file from the repository.
- Add
.claude/(or at least*.local.json) to the repository's.gitignorefile to prevent accidental commits in the future.
It needs an organization, so instructions are needed.
b9a1de1 to
14356e5
Compare
runephilosof-abtion
left a comment
There was a problem hiding this comment.
This is only file name that starts with upper case.
That should be fixed, when the file is being renamed.
|
|
||
| This applies only to `platform.claude.com` (the Anthropic Console used for API keys and agents). The Claude Code / claude.ai user subscription for day-to-day coding is a separate matter, included in our options and description and billed to the client at our normal pricing. | ||
|
|
||
| - Every client that needs API keys or agents has their own Claude organization, named after the team's google group. E.g. `danish-shipping`. |
There was a problem hiding this comment.
I prefer to avoid mentioning concrete clients in this public repo. However, I can see that this has been done before in this file 🤷
It needs an organization, so instructions are needed.