Skip to content

Agent-level RBAC with assumedRole#632

Open
fractlrao wants to merge 3 commits intomainfrom
rename-agent-role-to-agent-goal
Open

Agent-level RBAC with assumedRole#632
fractlrao wants to merge 3 commits intomainfrom
rename-agent-role-to-agent-goal

Conversation

@fractlrao
Copy link
Contributor

Summary

  • Rename role (agent purpose description) to goal in agent definitions
  • Add a new required role field to agents for RBAC role assignment
  • Rename escalatedRole to assumedRole with replace semantics — when set, the agent's role completely replaces the user's roles for permission checks
  • Record assumedRole in the Monitor for audit trail
  • Add agent-rbac tests validating role-based permission enforcement
  • Replace placeholder role "admin" with domain-specific roles (e.g. incident_ops, hr_ops, support_ops, network_ops, sales_ops) across all docs, examples, and tests

Test plan

  • All 404 existing tests pass
  • 3 new agent-rbac tests: assumedRole restricts permissions, assumedRole grants permissions, @withRole uses replace semantics
  • Syntax toString tests updated with matching input/output role values

`@withRole` now replaces the user's roles instead of adding to them,
preparing for agent-owned RBAC roles.
- Add required `role` field to agent definitions for RBAC role assignment
- Rename `escalatedRole` to `assumedRole` with replace semantics (not additive)
- Set `assumedRole` on the environment when an agent executes
- Add role tracking to Monitor for audit trail
- Add agent-rbac tests for role-based permission enforcement
- Replace placeholder `role "admin"` with domain-specific roles across
  docs, examples, and tests (e.g. incident_ops, hr_ops, support_ops)
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