Skip to content

docs: add IAM permissions guide and policy files#689

Merged
tejaskash merged 3 commits intomainfrom
docs/iam-permissions
Mar 27, 2026
Merged

docs: add IAM permissions guide and policy files#689
tejaskash merged 3 commits intomainfrom
docs/iam-permissions

Conversation

@tejaskash
Copy link
Copy Markdown
Contributor

@tejaskash tejaskash commented Mar 26, 2026

Summary

  • Adds docs/PERMISSIONS.md covering enterprise IAM setup for the AgentCore CLI. The first half is a setup guide for platform admins (role creation, CDK bootstrap, permission boundaries, feature-based scoping, troubleshooting). The second half is a complete action-by-action reference of every IAM permission the CLI requires.
  • Adds ready-to-use policy JSON files under docs/policies/: one for the developer role (direct SDK calls + CDK role assumption) and one for the CloudFormation execution role (infrastructure provisioning).

Test plan

  • Verify all markdown links resolve correctly
  • Verify JSON policy files are valid IAM policy documents
  • Spot-check IAM actions against CLI source code

Add PERMISSIONS.md covering enterprise IAM setup for the AgentCore CLI,
including admin setup instructions, CDK bootstrap configuration,
permission boundaries, feature-based scoping, and a full action-by-action
reference. Includes ready-to-use JSON policy documents for both the
developer role and the CloudFormation execution role.
@tejaskash tejaskash requested a review from a team March 26, 2026 21:40
@github-actions github-actions bot added the size/l PR size: L label Mar 26, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 26, 2026

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 46.06% 6568 / 14259
🔵 Statements 45.62% 6980 / 15297
🔵 Functions 44.78% 1182 / 2639
🔵 Branches 46.16% 4343 / 9407
Generated in workflow #1456 for commit 00df337 by the Vitest Coverage Report Action

Copy link
Copy Markdown
Contributor

@Hweinstock Hweinstock left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

After thinking about a bit, we really do require some crazy permissions atm.

- Add iam-policy-boundary.json for execution role permission boundaries
- Add deny statements to CFN execution role policy (ForceExecutionRoleBoundary,
  PreventBoundaryRemoval, PreventBoundaryPolicyTampering)
- Scope CDK bootstrap role ARNs to single account (ACCOUNT_ID placeholder)
- Expand permission boundaries section with step-by-step enterprise hardening guide
@github-actions github-actions bot added size/l PR size: L and removed size/l PR size: L labels Mar 27, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Package Tarball

aws-agentcore-0.3.0-preview.8.0.tgz

How to install

npm install https://github.com/aws/agentcore-cli/releases/download/pr-689-tarball/aws-agentcore-0.3.0-preview.8.0.tgz

…ct changes

The CDK constructs don't currently attach permissionsBoundary to IAM roles
they create, so the ForceExecutionRoleBoundary deny statement would block
deployments. Mark Step 2 as a recommended future configuration until the
constructs are updated.
@github-actions github-actions bot added size/l PR size: L and removed size/l PR size: L labels Mar 27, 2026
@tejaskash tejaskash merged commit 7a70cf4 into main Mar 27, 2026
21 of 23 checks passed
@tejaskash tejaskash deleted the docs/iam-permissions branch March 27, 2026 18:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/l PR size: L

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants