Add CLAUDE.md and consolidate AI agent conventions#168
Merged
Conversation
Consolidate all project conventions, coding standards, build commands, workflow rules, and CI requirements into CLAUDE.md. Rewrite AGENTS.md as a thin redirect with a fallback summary so both Claude Code and Codex operate under identical rules from one source of truth.
Track .claude/commands/ in git so the team shares slash commands. Settings files remain gitignored (local-only). Commands: /check-ci, /test-all, /test-crate, /review-changes, /verify
Contributor
ChristianPavilonis
left a comment
ChristianPavilonis
left a comment
There was a problem hiding this comment.
Do we need the TODO.md in the flow?
I use plan mode for this, just feels like an extra file always changing in source control.
Agents now present plans inline in the conversation and get approval there. No more writing to or appending review sections to TODO.md.
…ator, code-architect Specialized agents in .claude/agents/ for distributed work: - code-simplifier: post-implementation cleanup - verify-app: end-to-end verification pipeline - build-validator: cross-target build matrix validation - code-architect: architectural design review
- Add .github/pull_request_template.md with summary, changes table, closes section, test plan checkboxes, and checklist - Add .claude/agents/pr-creator.md subagent for creating/updating PRs using the project template with CI gate verification - Update CLAUDE.md subagent table to include pr-creator
- Add .github/ISSUE_TEMPLATE/ with three form-based templates: Bug Report, Story, and Task — each with relevant fields and scope dropdowns matching the project's crate structure - Add config.yml to enable blank issues alongside templates - Update PR template to prompt for linking an existing ticket with pre-filled "Closes #" syntax
Auto-format hook was reformatting YAML files (quote style, indentation). Restore originals from main.
- build-validator: add --manifest-path for demo app builds (excluded from workspace) - verify-app: same manifest-path fix, plus proper $pid-based process cleanup instead of job control
- Add .claude/agents/repo-explorer.md for read-only codebase mapping - Add .claude/settings.json with organized permission allowlist (tracked) - Remove PostToolUse hook — formatting enforced by CI and /check-ci - Add subagent workflow, selection matrix, and repo-explorer to CLAUDE.md - Disable blank issues in issue template config - Normalize YAML quote style and indentation in .github/ configs - Update .gitignore to track .claude/settings.json
- Add .claude/agents/issue-creator.md with GitHub issue types via GraphQL (Task, Bug, Story, Epic, Initiative) - Update CLAUDE.md subagent table to include issue-creator - Fix PR template: edgezero dev → edgezero-cli dev
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
3 participants
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
CLAUDE.mdas the single source of truth so Claude Code and Codex share identical project conventionsAGENTS.mdas a thin redirect to eliminate duplication and driftChanges
CLAUDE.mdAGENTS.mdCLAUDE.mdwith 8-rule fallback summary for Codex.claude/settings.json.claude/commands//check-ci,/test-all,/test-crate,/review-changes,/verify.claude/agents/code-simplifier,verify-app,build-validator,code-architect,pr-creator,issue-creator,repo-explorer.github/ISSUE_TEMPLATE/.github/pull_request_template.md.github/*.yml.gitignore.claude/settings.json,.claude/commands/,.claude/agents/examples/app-demo/Cargo.lockCloses
Closes #167
Test plan
cargo test --workspace --all-targetscargo clippy --workspace --all-targets --all-features -- -D warningscargo check --workspace --all-targets --features "fastly cloudflare"wasm32-wasip1(Fastly) /wasm32-unknown-unknown(Cloudflare)edgezero-cli devChecklist
{id}syntax (not:id)edgezero_core(nothttpcrate)