docs: agent operating rules + mobile-Dispatch working method#28
Merged
Conversation
docs(agents): add CLAUDE.md auto-loaded agent operating rules Encode the maintainer's working rules so every Claude Code session in this repo picks them up automatically (CLAUDE.md auto-loads per-repo). - CLAUDE.md (root): concise rule set, points to the detailed doc. - docs/AGENT_WORKFLOW.md: long-form rules, halt list, branch/commit conventions, reporting format, Windows/PowerShell notes, rationale. - docs/README.md: link the new workflow doc from the docs hub. Rules optimize for remote phone-driven sessions via Dispatch: no interactive pickers (plain-text questions only), autonomous reversible defaults, halt only for destructive/irreversible actions, feature branch only, plan-but-do-not-block, test before handing back. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> @
docs(agents): forbid skipping git hooks / signing as a top-level rule Add an explicit short-form rule (new rule 5) in CLAUDE.md: never use --no-verify, never disable/skip/bypass pre-commit or commit-msg hooks or commit signing unless the maintainer explicitly asks; fix the underlying issue instead. Mirrors and strengthens AGENT_WORKFLOW.md section 4. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> @
docs(agents): post-reorg path fixes + PR-consolidation rule Rebased onto current main (post #26/#27 docs reorg). Adjust for the new docs/ tree and capture the maintainer's PR-cadence preference. - CLAUDE.md: repoint build-guide reference from the old repo-root BUILDGUIDE_*.md files to docs/build-guides/ (moved by #27). - docs/README.md: re-add the AGENT_WORKFLOW.md pointer into the reorganized "Internal development records" section (the old pre-reorg edit was dropped during rebase). - CLAUDE.md + AGENT_WORKFLOW.md: add a new rule (short form rule 5; detail in AGENT_WORKFLOW.md section 4) — consolidate work, at most one PR per work-session/batch, branches may accumulate commits and sit for days, never open a PR proactively, default posture is keep-on-branch. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> @
docs(agents): record mobile-Dispatch working method + 2026-06-09 session log Note in CLAUDE.md (not the README) that this repo is primarily driven via mobile Dispatch (Adam running Claude Code sessions from his phone while the desktop runs), and log today's session: README trim (#26 merged), docs/ reorg (#27 merged), auto-capture wiring (phase-13.x.9, parked), these agent rules, and the Step 5c cognition training/eval harness (parked; lightgbm 4.6.0 kept for local runs). Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> @
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
Reviewer's GuideAdds repo-wide agent operating rules optimized for mobile Dispatch workflows by introducing an auto-loaded CLAUDE.md at the repo root and a detailed AGENT_WORKFLOW.md in docs/, and wires them into the docs index. Flow diagram for agent decision workflow and halt rulesflowchart TD
A[Start task in Claude Code session<br/>CLAUDE_md auto-loaded] --> B[Need user input?]
B -->|No| C[Proceed autonomously using repo rules]
B -->|Yes| D[Ask question in plain-text reply<br/>Provide recommended default]
D --> C
C --> E[At a decision or fork]
E --> F{Is choice conservative and easily reversible?}
F -->|Yes| G[Pick conservative option<br/>Record one-line note]
G --> H[Continue work on feature branch]
F -->|No| I{Does action touch halt list?<br/>push / PR / main / history / data / access}
I -->|No| H
I -->|Yes| J[Stop and request explicit approval<br/>Show exact commands]
H --> K[Run relevant tests]
K --> L[Prepare final report<br/>Changes, decisions, tests, approvals needed, questions]
L --> M[End session]
File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Hey - I've found 1 issue, and left some high level feedback:
- Rule 3’s halt list in
docs/AGENT_WORKFLOW.mdincludes external publishing (PyPI, registries, releases), but the short-form halt list inCLAUDE.mddoes not; consider adding that class of actions toCLAUDE.mdto avoid the short guide giving a looser impression than the detailed one. - There is a fair amount of duplicated wording between
CLAUDE.mdanddocs/AGENT_WORKFLOW.mdfor the top-level rules; you might want to centralize the canonical descriptions in one place and keep the other strictly as a short pointer/summary to reduce drift over time.
Prompt for AI Agents
Please address the comments from this code review:
## Overall Comments
- Rule 3’s halt list in `docs/AGENT_WORKFLOW.md` includes external publishing (PyPI, registries, releases), but the short-form halt list in `CLAUDE.md` does not; consider adding that class of actions to `CLAUDE.md` to avoid the short guide giving a looser impression than the detailed one.
- There is a fair amount of duplicated wording between `CLAUDE.md` and `docs/AGENT_WORKFLOW.md` for the top-level rules; you might want to centralize the canonical descriptions in one place and keep the other strictly as a short pointer/summary to reduce drift over time.
## Individual Comments
### Comment 1
<location path="docs/AGENT_WORKFLOW.md" line_range="5-14" />
<code_context>
+rationale, the exact halt list, branch/commit conventions, and the reporting
</code_context>
<issue_to_address>
**nitpick (typo):** Use a consistent spelling for the term "halt list" throughout the document.
You use both "halt list" and "halt-list" for the same concept. Please pick one form and use it consistently across the file for clarity.
</issue_to_address>Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
Comment on lines
+5
to
+14
| rationale, the exact halt list, branch/commit conventions, and the reporting | ||
| format. | ||
|
|
||
| **Audience:** any AI agent (Claude Code or otherwise) working in this repo. | ||
| **Context:** the maintainer (Adam, [@nah414](https://github.com/nah414)) often | ||
| drives sessions remotely from a phone via Dispatch. Every rule below is shaped | ||
| by that: minimize round-trips, never block on something that can't be answered | ||
| with a thumb, and make autonomous progress safe by keeping it reversible. | ||
|
|
||
| **Precedence:** these rules override default agent behavior. A direct |
There was a problem hiding this comment.
nitpick (typo): Use a consistent spelling for the term "halt list" throughout the document.
You use both "halt list" and "halt-list" for the same concept. Please pick one form and use it consistently across the file for clarity.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Establishes a repo-root
CLAUDE.md(auto-loaded into every Claude Code session) plus a detailed companiondocs/AGENT_WORKFLOW.md, so agent operating rules are picked up automatically without re-pasting prompts each session. Optimized for Adam driving sessions remotely from a phone via mobile Dispatch.Agent operating rules (short form; full detail in
docs/AGENT_WORKFLOW.md)main, force-push, history rewrite, deleting data, access/permission changes.main— no push/PR without explicit approval.--no-verify; fix the underlying issue.Mobile-dispatch working method
CLAUDE.mdrecords that this repo is primarily driven via mobile Dispatch (phone-driven Claude Code sessions while the desktop runs), which is the rationale for the no-picker / plain-text / autonomous-reversible posture.Dated session log
CLAUDE.mdincludes a tight session log. 2026-06-09: README trim (#26, merged); root build-guide/planning docs reorganized intodocs/(#27, merged); auto-capture baseline wired intoDriftDetector.run_cycle(branchphase-13.x.9-auto-capture-wiring, parked); these agent rules established; Step 5c cognition-classifier training + evaluation harness built (branchphase-13-step5c-cognition-training-harness, parked;lightgbm 4.6.0kept installed for local runs).Notes
main(includes merged docs: clean up and trim main README into a concise overview #26/docs: consolidate root build-guide and planning docs into docs/ tree #27); build-guide references and thedocs/README.mdpointer were updated to the post-reorgdocs/tree.🤖 Generated with Claude Code
Summary by Sourcery
Document agent operating rules and workflow for AI assistants working in the repo, optimized for mobile Dispatch sessions, and surface them via an auto-loaded root CLAUDE.md file.
Documentation: