Skip to content

feat(claude): manage the global Claude Code instructions - #86

Merged
thompsonson merged 3 commits into
mainfrom
chore/claude-md
Sep 20, 2026
Merged

thompsonson merged 3 commits into
mainfrom
chore/claude-md

Conversation

@thompsonson

Copy link
Copy Markdown
Owner

Brings ~/.claude/CLAUDE.md under chezmoi as dot_claude/CLAUDE.md, so every machine gets the same rules instead of each session being told them again.

What the file says

  • Communication style: short sentences, one per bullet, no filler, failures reported with their output.
  • Questions: read the docs, code, config and git history first; never re-ask what they answer.
  • Authority: how a session reports a block, what it may do unasked, and what needs Matt first.

Why

Three things were repeated in nearly every session: the style, asking what the documentation already answered, and stopping short of the work that was already authorised. They now live in one file that every session inherits.

Checks

  • chezmoi status clean; source and live copy match.
  • chezmoi apply --dry-run touches this file only.
  • ./tests/test.sh quick passes.

https://claude.ai/code/session_01VUd2Wv3nkSBpWgYBK6yhfh

~/.claude/CLAUDE.md was machine-local, so every machine drifted and each new
session had to be told the same rules again. Bring it under chezmoi: style
(short sentences, one per bullet), questions (read the docs first, never
re-ask what they answer), and authority (how a session reports a block, what
it may do unasked, what needs me first).

Claude-Session: https://claude.ai/code/session_01VUd2Wv3nkSBpWgYBK6yhfh
Sessions kept stopping at "you can push and open a PR", which is the part
they can do. Make it explicit: branches, commits, pushes and opening the
pull request are already authorised; only merging needs Matt.

Claude-Session: https://claude.ai/code/session_01VUd2Wv3nkSBpWgYBK6yhfh
Comment thread dot_claude/CLAUDE.md
Comment thread dot_claude/CLAUDE.md
Comment thread dot_claude/CLAUDE.md Outdated
Comment thread dot_claude/CLAUDE.md Outdated
Comment thread dot_claude/CLAUDE.md Outdated
Comment thread dot_claude/CLAUDE.md Outdated
…e to drive

From Matt's review of #86:

- Replace the "load bearing" ban with grounding terms in the repo's glossary,
  in style and before asking questions.
- Merging is his alone, stated as such.
- Test VMs exist to be driven: installing builds, changing nix.conf, systemd
  units and sudoers, and creating or destroying the VM are all authorised.
- Deleting is authorised for anything living only in a test environment.
- Ask-first now covers production and shared machines, not test ones.

Claude-Session: https://claude.ai/code/session_01VUd2Wv3nkSBpWgYBK6yhfh
@thompsonson

thompsonson commented Sep 20, 2026 •

Copy link
Copy Markdown
Owner Author

All six review points applied in e4d713b:

  • L11: the "load bearing" ban is gone; style now says ground every term in the local repository's glossary.
  • L20: Questions gains "use the repository's glossary terms, and check them before you ask".
  • L44: now reads "Merging a pull request is mine alone, and never yours".
  • L46: installing builds on a test VM is authorised, as is creating, rebuilding and destroying the VM itself.
  • L47: deleting anything that exists only inside a test environment is authorised; ask-first now covers data outside one.
  • L48: changing a test VM's system configuration (nix.conf, systemd units, sudoers) is authorised.

Ask-first now names production and shared machines, rather than any long-lived environment.

https://claude.ai/code/session_01VUd2Wv3nkSBpWgYBK6yhfh

@thompsonson
thompsonson merged commit 608aad8 into main Sep 20, 2026
4 checks passed
@thompsonson
thompsonson deleted the chore/claude-md branch September 20, 2026 09:15
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