Skip to content

docs: move maintainer-only agent ops out of the public CLAUDE.md - #1348

Merged
Dumbris merged 1 commit into
mainfrom
claude/zcode-subagent-setup-893dbf
Sep 22, 2026
Merged

Dumbris merged 1 commit into
mainfrom
claude/zcode-subagent-setup-893dbf

Conversation

@Dumbris

@Dumbris Dumbris commented Sep 22, 2026

Copy link
Copy Markdown
Member

Pull Request

Description

CLAUDE.md is loaded by every agent session in this repo, and this repo is public — outside contributors read it too. Two of its sections were maintainer-private operating policy that means nothing to them, and in one case can actively mislead:

  • Autonomous Operation Constraints — a "zero interruption" policy, a "do NOT ask for plan approval" rule, and escalation triggers naming personal reviewer CLIs, their model tiers and their current quota state. A contributor following "never ask a human, just assume" is not the behaviour we want from a drive-by agent session on a fork.
  • Model Routing — per-task model ladders, session-model switching guidance and quota discipline for accounts and CLI subscriptions a contributor does not have.

Both now live in the maintainer's own global agent config, outside version control. The header also dropped an aside about a private fleet scheduler ("every Paperclip heartbeat") that no reader of this repo can act on.

Net: -40 lines, one line changed, nothing added.

Nothing contributor-facing was touched. The file keeps its orientation role in full: project overview, editions (personal/server build tags), the architecture directory table, build/test/lint commands — including the two golangci-lint invocations CI runs and the -skip regex for internal/server — configuration, MCP protocol, security model, key implementation details, debugging, and development guidelines.

AGENTS.md was reviewed and already contains only contributor-facing guidance; it is unchanged.

Testing

  • I have tested these changes locally
  • I have added/updated tests that prove my fix is effective or my feature works
  • All existing tests pass

Documentation-only change — no code paths touched, so no tests were added. Verified by inspection that the remaining file contains no references to personal tooling, model quotas, or maintainer directives, and pre-commit hooks (whitespace, EOF, merge-conflict, large-file and executable checks) pass.

CLAUDE.md is loaded by every agent session in a public repo, so outside
contributors read it too. Two sections were maintainer-private operating
policy that means nothing to them and can actively mislead:

- Autonomous Operation Constraints — a zero-interruption policy, a
  "do not ask for plan approval" rule, and escalation triggers naming
  personal reviewer CLIs, their model tiers and their quota state.
- Model Routing — per-task model ladders, session-model switching and
  quota discipline for accounts a contributor does not have.

Both now live in the maintainer's own global agent config. Nothing that
helps someone contribute was removed: architecture, build/test/lint
commands, editions, configuration, MCP protocol, security model,
debugging and development guidelines are untouched.

Also drops the "every Paperclip heartbeat" aside from the header — that
is private fleet tooling, not something a reader of this repo can act on.
@Dumbris
Dumbris merged commit 0600d4d into main Sep 22, 2026
18 checks passed
@Dumbris
Dumbris deleted the claude/zcode-subagent-setup-893dbf branch September 22, 2026 15:54
@codecov-commenter

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

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