Skip to content

chore: Add codex+claude as provisioned agents - #455

Merged
dpup merged 1 commit into
mainfrom
dpup/moat-yaml
Aug 14, 2026
Merged

chore: Add codex+claude as provisioned agents#455
dpup merged 1 commit into
mainfrom
dpup/moat-yaml

Conversation

@dpup

@dpup dpup commented Aug 14, 2026

Copy link
Copy Markdown
Collaborator

No description provided.

@dpup
dpup merged commit 220a517 into main Aug 14, 2026
4 checks passed
@dpup
dpup deleted the dpup/moat-yaml branch August 14, 2026 18:15
@claude

claude Bot commented Aug 14, 2026

Copy link
Copy Markdown
Contributor

Claude finished @dpup's task in 3m 33s —— View job


Review of PR #455

  • Gather context (diff, changed files)
  • Run /code-review skill
  • Post review summary

Change: Adds agents: [claude, codex] to the repo-root moat.yaml (a personal/dev container config), a one-line addition.

Findings: None. This is a correct, minimal application of an existing, documented config mechanism:

  • Both claude and codex are valid registered agent names (internal/providers/claude/provider.go, internal/providers/codex/provider.go).
  • Claude's default dependencies (node, git, claude-code) are already satisfied by the file's dependencies: list — no duplication or conflict.
  • Codex's default dependency codex-cli is correctly omitted and will be auto-provisioned.
  • Codex requires the openai grant, which isn't explicitly listed, but ExpandAgents/AppendDerivedGrants auto-derives and appends it — matching the documented behavior and example in docs/content/reference/02-moat-yaml.md (agents: [claude, codex] is the doc's own example).
  • No CI or test parses this repo-root file directly, so there's no schema-validation risk.

No bugs, security issues, or CLAUDE.md convention violations found. LGTM.

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