diff --git a/CLAVITOR-PRINCIPLES.md b/CLAVITOR-PRINCIPLES.md index c8bba4a..44b5b1a 100644 --- a/CLAVITOR-PRINCIPLES.md +++ b/CLAVITOR-PRINCIPLES.md @@ -1298,7 +1298,7 @@ Implements Threat A defense per CLAVITOR-PRINCIPLES.md Part 2. - Two-strike lockdown within 2h window - Locked state persisted in encrypted agent record -Author: Claude (agent session 20250408-001)" +Author: vault-agent <20250408-001>" 4. git push -u origin agent/rate-limit-defense-20250408 5. gh pr create --title "Agent: Threat A rate limiting" \ --body "$(cat <<'EOF' @@ -1337,8 +1337,10 @@ EOF Create `.github/PULL_REQUEST_TEMPLATE/agent_pr_template.md`: ```markdown -## Agent Session -- Session ID: +## Agent Identity +- Agent name: +- Session ID: +- Model: - Principles reviewed: CLAVITOR-PRINCIPLES.md v - Daily checks: @@ -1366,6 +1368,28 @@ Create `.github/PULL_REQUEST_TEMPLATE/agent_pr_template.md`: ``` +### Agent naming convention + +Use **functional names** based on domain, not model names: + +| Subproject | Agent name | Git signature | +|------------|-----------|---------------| +| `clavis-vault` | `vault-agent` | `vault-agent ` | +| `clavis-cli` | `cli-agent` | `cli-agent ` | +| `clavis-chrome/firefox/safari` | `extension-agent` | `extension-agent ` | +| `clavis-crypto` | `crypto-agent` | `crypto-agent ` | +| `clavitor.ai/admin` | `central-agent` | `central-agent ` | +| Cross-cutting / architecture | `architect-agent` | `architect-agent ` | +| Security reviews | `security-agent` | `security-agent ` | + +**Why:** Release notes show "vault-agent added rate limiting" not "Claude added rate limiting." The domain matters, the model doesn't. + +**Commit signature format:** +``` +Author: vault-agent <20250408-001> +Author: cli-agent <20250408-003> +``` + **Human review points:** Even agent-authored PRs require human approval before merge: