Skip to content

fix(schema): add executionRoleArn to JSON schema#733

Closed
jesseturner21 wants to merge 1 commit intomainfrom
fix/add-executionRoleArn-to-json-schema
Closed

fix(schema): add executionRoleArn to JSON schema#733
jesseturner21 wants to merge 1 commit intomainfrom
fix/add-executionRoleArn-to-json-schema

Conversation

@jesseturner21
Copy link
Copy Markdown
Contributor

Description

The executionRoleArn field was added to the Zod schema (AgentEnvSpecSchema) in #729 but the JSON schema (agentcore.schema.v1.json) was not updated to match. This adds the missing executionRoleArn string property to the JSON schema, placed between requestHeaderAllowlist and authorizerType to match the Zod schema order.

Related Issue

Closes #729 (follow-up)

Type of Change

  • Bug fix

Testing

  • I ran npm run test:unit and npm run test:integ
  • I ran npm run typecheck
  • I ran npm run lint

Checklist

  • I have read the CONTRIBUTING document
  • My changes generate no new warnings
  • Any dependent changes have been merged and published

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the
terms of your choice.

The executionRoleArn field was added to the Zod schema in #729 but the
JSON schema was not updated to match.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@jesseturner21 jesseturner21 requested a review from a team March 30, 2026 20:24
@github-actions github-actions bot added the size/xs PR size: XS label Mar 30, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Package Tarball

aws-agentcore-0.4.0.tgz

How to install

npm install https://github.com/aws/agentcore-cli/releases/download/pr-733-tarball/aws-agentcore-0.4.0.tgz

@github-actions
Copy link
Copy Markdown
Contributor

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 46.01% 6555 / 14246
🔵 Statements 45.58% 6965 / 15280
🔵 Functions 44.71% 1176 / 2630
🔵 Branches 46.11% 4334 / 9399
Generated in workflow #1531 for commit f59fdde by the Vitest Coverage Report Action

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/xs PR size: XS

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant