docs: add Copilot + @claude agent configuration#127
Conversation
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
There was a problem hiding this comment.
Pull request overview
Adds repository configuration/docs to support Claude Code automation and GitHub Copilot agent guidance as part of the “Chitin assembly line v2” rollout.
Changes:
- Add GitHub Actions workflows to run Claude Code on
@claudementions and to run an automated PR code review job. - Add
CLAUDE.mdproject context/instructions for Claude sessions. - Add
.github/copilot-instructions.mddescribing Copilot’s “Tier C” operating constraints and local build commands.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 5 comments.
| File | Description |
|---|---|
CLAUDE.md |
Adds agent identity bootstrap + project/build context documentation |
.github/workflows/claude.yml |
Adds an @claude-triggered Claude Code workflow |
.github/workflows/claude-code-review.yml |
Adds a PR-triggered Claude Code Review workflow |
.github/copilot-instructions.md |
Adds repo-specific Copilot guidance/governance + build steps |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| 1. **Role**: developer / reviewer / ops / security / planner | ||
| 2. **Driver**: human / claude-code / copilot / ci | ||
|
|
||
| Then run: `scripts/write-persona.sh <driver> <role>` |
There was a problem hiding this comment.
scripts/write-persona.sh is referenced here, but there is no such file under scripts/ in this repo. This instruction can’t be followed as-is; either add the script or update the doc to point at the correct existing script/command.
| Then run: `scripts/write-persona.sh <driver> <role>` | |
| Then record the selected `driver` and `role` as the session identity and continue with those values for the rest of the session. |
| ShellForge is the execution harness for the Chitin platform — agent loop, LLM providers, tool-use, drift detection. | ||
|
|
||
| **Module**: `github.com/chitinhq/shellforge` | ||
| **Language**: Go 1.22+ | ||
|
|
There was a problem hiding this comment.
The module path and Go version documented here don’t match the repository configuration: go.mod declares module github.com/AgentGuardHQ/shellforge and go 1.18, and CI currently sets up Go 1.21. Please align these values (or update go.mod/CI if the repo has actually moved to github.com/chitinhq/shellforge + Go 1.22+).
| ```bash | ||
| go build ./... | ||
| go test ./... | ||
| golangci-lint run |
There was a problem hiding this comment.
The build instructions include golangci-lint run, but this repo doesn’t appear to have a .golangci.yml (or any other golangci-lint configuration/CI step). If linting is required, add the config + CI integration; otherwise remove it from the documented build steps to avoid misleading setup guidance.
| golangci-lint run |
| - **Language**: Go 1.22+ | ||
| - **Module**: `github.com/chitinhq/shellforge` |
There was a problem hiding this comment.
The Go version and module path documented here don’t match the repo’s go.mod (module github.com/AgentGuardHQ/shellforge, go 1.18) or CI (Go 1.21). Please align this doc with the actual module path/version, or update the codebase config if the intent is to move to Go 1.22+ and the github.com/chitinhq/shellforge module.
| - **Language**: Go 1.22+ | |
| - **Module**: `github.com/chitinhq/shellforge` | |
| - **Language**: Go 1.18+ | |
| - **Module**: `github.com/AgentGuardHQ/shellforge` |
| ```bash | ||
| go build ./... | ||
| go test ./... | ||
| golangci-lint run |
There was a problem hiding this comment.
The instructions tell contributors/agents to run golangci-lint run, but the repository doesn’t include a golangci-lint configuration file (e.g., .golangci.yml) and it’s not invoked in CI. Either add/configure golangci-lint or remove it from the documented workflow to prevent broken setup steps.
| golangci-lint run |
Summary
Part of Chitin assembly line v2 rollout.
🤖 Generated with Claude Code