Skip to content

docs: fix 30 documentation inaccuracies found by source code audit - #697

Merged
aidandaly24 merged 11 commits into
aws:mainfrom
aidandaly24:docs/fix-documentation-inaccuracies
Apr 9, 2026
Merged

docs: fix 30 documentation inaccuracies found by source code audit#697
aidandaly24 merged 11 commits into
aws:mainfrom
aidandaly24:docs/fix-documentation-inaccuracies

Conversation

@aidandaly24

@aidandaly24aidandaly24 commented Mar 27, 2026

Copy link
Copy Markdown
Contributor

Description

Comprehensive audit of all .md documentation files against actual source code (Zod schemas, CLI commands, CDK constructs). Fixes 70+ inaccuracies across 14 files where docs had drifted from the implementation, including alignment with several upstream refactors and documentation of new features merged after the initial audit.

Upstream refactors accounted for

PRChangeDoc Impact
#709Remove type fields from resource schemas, rename credential typeauthorizerTypeRemoved stale type fields from all JSON examples; renamed credential discriminator
#705add identityadd credential, --force-y/--yes, --plan--dry-runUpdated all command references and flag tables
#706agentsruntimes array, --agent--runtime flagsUpdated all flag tables, examples, and JSON config across 7 doc files
#707Dev command --invoke → positional [prompt] argAlready reflected in docs
#713Revert CUSTOM strategy and semanticOverrideNo doc changes needed (CUSTOM was never documented)

New features documented (merged after initial audit)

PRFeatureDoc Files
#763, #780Import subcommands (import runtime, import memory, import evaluator, import online-eval)commands.md
#750Exec mode for invoke and dev (--exec, --timeout)commands.md, local-development.md
#739Code-based evaluator support (--type, --lambda-arn, --timeout)commands.md, evals.md
#732Ground truth reference inputs (--assertion, --expected-trajectory, --expected-response)commands.md, evals.md
#531, #534Memory record streaming (--data-stream-arn, --stream-content-level, --delivery-type)commands.md, memory.md
#782--skip-install flag on createcommands.md
#774--identity-name option on fetch accesscommands.md

Key fixes by category

Schema/config drift (configuration.md, gateway.md, container-builds.md):

  • Added missing EPISODIC memory strategy and reflectionNamespaces field
  • Added lifecycleConfiguration (idleRuntimeSessionTimeout, maxLifetime) to agent spec
  • Added description, dockerfile, executionRoleArn to agent spec table
  • Fixed CUSTOM_JWT fields: allowedAudience/allowedClients/allowedScopes/customClaims are conditionally required
  • Fixed OAuth credential discoveryUrl: optional, not required (for imported providers)
  • Fixed memory strategies: defaults to empty array, not required
  • Added missing tags field to memory, credential, gateway, and online eval config tables
  • Added missing customClaims field to CUSTOM_JWT config
  • Fixed MCP Server outbound auth: removed api-key (only oauth/none valid per schema)
  • Added enableSemanticSearch, exceptionLevel, policyEngineConfiguration gateway fields
  • Fixed gateway target type count: six types (including lambda), not five
  • Removed stale "type": "AgentCoreRuntime" / "AgentCoreMemory" / "CustomEvaluator" / "OnlineEvaluationConfig" from all JSON examples
  • Fixed mcp.json reference → agentcore.json in local-development.md

Flag renames (commands.md, evals.md, local-development.md):

  • --agent--runtime, -a-r across all commands (status, dev, invoke, logs, traces, run eval, evals history, package, add online-eval, logs evals)
  • --agent-arn--runtime-arn, --agent-runtime-id--runtime-id
  • --plan--dry-run, --force-y/--yes
  • add identityadd credential, remove identityremove credential
  • --agent-client-id--client-id, --agent-client-secret--client-secret (gateway CUSTOM_JWT)
  • --exception-level valid values: NONE/DEBUG (not ALL)
  • --lines--limit in logs evals
  • Note: --agent-id and --agent-alias-id are unchanged (Bedrock Agent import flags)

Missing flags added (commands.md, evals.md):

  • add agent: --authorizer-type, --discovery-url, --allowed-audience, --allowed-clients, --allowed-scopes, --custom-claims, --client-id, --client-secret, --idle-timeout, --max-lifetime
  • create: --idle-timeout, --max-lifetime, --skip-install
  • add gateway: --runtimes, --custom-claims, --policy-engine, --policy-engine-mode
  • dev and invoke: -H, --header, --exec
  • invoke: --bearer-token, --exec, --timeout
  • add evaluator: --type, --lambda-arn, --timeout (code-based evaluator support)
  • run eval: --assertion, --expected-trajectory, --expected-response, --endpoint
  • add memory: --data-stream-arn, --stream-content-level, --delivery-type, --stream-delivery-resources
  • fetch access: --identity-name
  • gateway.md: --oauth-scopes in OAuth example

New sections added:

  • commands.md: Full import command section with 4 subcommands (runtime, memory, evaluator, online-eval)
  • evals.md: Evaluator Types section (LLM-as-a-Judge vs code-based), Ground Truth Reference Inputs subsection, Builtin Evaluator table (all 10 IDs with levels)
  • evals.md: Fixed eval results storage path (eval-runseval-results)
  • memory.md: Memory Record Streaming section with CLI flags and JSON config example
  • local-development.md: Exec Mode section covering dev exec, invoke exec, and TUI shortcuts

Stale values (commands.md, evals.md, frameworks.md):

  • Fixed model ID claude-3-5-sonnet-20250929claude-3-5-sonnet-20250514
  • Added policy-engine/policy to status --type enum
  • Removed ghost --tool-filter-description flag (not in schema or CLI)
  • Fixed evaluator level values: added TOOL_CALL, removed SPAN

Structural/naming (AGENTS.md, README.md, assets):

  • Removed AutoGen from supported frameworks list (5 frameworks, not 6)
  • Fixed "agents""runtimes" in gateway JSON example
  • Renamed identitycredential in all command references
  • Fixed file paths and directory references
  • Fixed regex constraints and enum values to match Zod schemas

Files modified

FileChanges
docs/configuration.mdSchema field tables, JSON examples, type removals, credential rename, tags fields, required/optional fixes
docs/commands.mdAll flag tables and examples, import subcommands, exec mode, streaming, evaluator types, ground truth
docs/evals.mdFlag tables, examples, evaluator types section, ground truth inputs, builtin evaluator table, eval-results path fix
docs/gateway.mdidentity → credential, agents → runtimes, oauth-scopes, target type count
docs/local-development.md--runtime flag in dev examples, exec mode section, mcp.json → agentcore.json
docs/memory.mdRemoved stale types, added memory record streaming section
docs/container-builds.mdRemoved stale "type": "AgentCoreRuntime" from JSON example
docs/frameworks.mdModel ID fix, framework list correction
AGENTS.mdidentity → credential in primitives
README.mdidentity → credential
src/assets/README.mdidentity → credential in vended template
src/assets/__tests__/__snapshots__/assets.snapshot.test.ts.snapUpdated snapshot

Known source code issue (not fixed here)

src/cli/primitives/GatewayPrimitive.ts has incorrect CLI help text: --exception-level says ALL (should be DEBUG) and --authorizer-type omits AWS_IAM. This is a source code bug that should be addressed in a separate PR.

Related Issue

N/A — discovered via systematic source code audit

Documentation PR

N/A — this PR is the documentation fix

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update
  • Other (please describe):

Testing

  • I ran npm run test:unit and npm run test:integ
  • I ran npm run typecheck
  • I ran npm run lint
  • If I modified src/assets/, I ran npm run test:update-snapshots and committed the updated snapshots

All changes are documentation-only (.md files + snapshot update). Each fix was verified against the corresponding Zod schema definition or CLI source code (src/cli/commands/*/command.tsx, src/cli/primitives/*.ts).

Note: 19 pre-existing test failures exist on upstream/main from the schema refactor (#709) — not introduced by this PR. The 2 e2e failures are expected on fork PRs (no AWS credentials).

Checklist

  • I have read the CONTRIBUTING document
  • I have added any necessary tests that prove my fix is effective or my feature works
  • I have updated the documentation accordingly
  • I have added an appropriate example to the documentation to outline the feature, or no new docs are needed
  • 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.

@aidandaly24
aidandaly24 requested a review from a teamMarch 27, 2026 04:40
@github-actionsgithub-actionsBot added the size/m PR size: M label Mar 27, 2026
@github-actionsgithub-actionsBot added size/m PR size: M and removed size/m PR size: M labels Mar 27, 2026
@aidandaly24
aidandaly24force-pushed the docs/fix-documentation-inaccuracies branch from f2e7d91 to f957db1CompareMarch 27, 2026 17:36
@github-actionsgithub-actionsBot added size/m PR size: M and removed size/m PR size: M labels Mar 27, 2026
@github-actions

github-actionsBot commented Mar 27, 2026

Copy link
Copy Markdown
Contributor

Package Tarball

aws-agentcore-0.7.1.tgz

How to install

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

@github-actionsgithub-actionsBot added size/m PR size: M and removed size/m PR size: M labels Mar 27, 2026
@github-actionsgithub-actionsBot added size/m PR size: M and removed size/m PR size: M labels Mar 27, 2026
@aidandaly24
aidandaly24force-pushed the docs/fix-documentation-inaccuracies branch from bd9373b to c4a8c0aCompareMarch 27, 2026 18:36
@github-actionsgithub-actionsBot added size/m PR size: M and removed size/m PR size: M labels Mar 27, 2026
@aidandaly24
aidandaly24force-pushed the docs/fix-documentation-inaccuracies branch from c4a8c0a to 727e1cfCompareMarch 27, 2026 21:39
@github-actionsgithub-actionsBot removed the size/m PR size: M label Mar 27, 2026
@github-actionsgithub-actionsBot added size/l PR size: L and removed size/l PR size: L labels Mar 28, 2026
@aidandaly24
aidandaly24force-pushed the docs/fix-documentation-inaccuracies branch from 1b43c59 to c8a57f4CompareMarch 28, 2026 01:22
@github-actionsgithub-actionsBot added size/l PR size: L and removed size/l PR size: L labels Mar 28, 2026
@github-actionsgithub-actionsBot added size/l PR size: L and removed size/l PR size: L labels Mar 28, 2026
@github-actionsgithub-actionsBot removed the size/l PR size: L label Mar 28, 2026
- configuration.md: fix networkMode PRIVATE→VPC, remove stale mcp.json
section, add 7 missing AgentEnvSpec fields, add Node.js runtimes,
fix gateway name max 63→100, list all 6 targetType values, fix memory
strategies wording, fix deployed-state.json path, add missing project
fields, fix CUSTOM_JWT field requirements, add customClaims field
- README.md: add CrewAI framework, update all 4 default model IDs,
add missing command sections
- frameworks.md: add CrewAI with model providers, add protocol
compatibility matrix (HTTP/MCP/A2A)
- commands.md: fix agent name max 64→48 chars, add fetch access section,
add policy-engine/policy to status --type, remove ghost
--tool-filter-description flag, fix model ID date
- gateway.md: add enableSemanticSearch/exceptionLevel config options,
fix mcp.json→agentcore.json, remove api-key from MCP Server auth
- transaction_search.md: fix traces command syntax
- container-builds.md: remove nonexistent --progress flag
- evals.md: fix model date 20250929→20250514
- local-development.md: fix mcp.json→agentcore.json
- AGENTS.md: remove AutoGen, remove stale mcp.json reference
- src/assets/README.md: add CrewAI to template list
- src/assets/agents/AGENTS.md: remove AutoGen
Constraint: All fixes verified against Zod schemas and CLI runtime
Constraint: CLI --help text bugs left for separate agent
Confidence: high
Scope-risk: narrow
- Remove `type` fields from agent, memory, evaluator, online-eval-config
examples and field tables (removed in aws#709)
- Remove `modelProvider` from agent spec (removed in aws#709)
- Rename credential `type` to `authorizerType` (renamed in aws#709)
- Rename `add identity` to `add credential` everywhere (aws#705)
- Replace `--force` with `-y/--yes` on remove commands (aws#705)
- Replace `--plan` with `--dry-run` on deploy command (aws#705)
- Update snapshot for src/assets/README.md change
Update `"agents": [...]` to `"runtimes": [...]` in the gateway
configuration example to match the schema rename in aws#706.
Update all documentation to reflect upstream renames:
- --agent → --runtime, -a → -r across all commands
- --agent-arn → --runtime-arn, --agent-runtime-id → --runtime-id
- Remove stale "type" fields from JSON examples in evals.md,
memory.md, and container-builds.md
Constraint: --agent-id, --agent-alias-id, --agent-client-id,
--agent-client-secret are unchanged (different semantics)
Constraint: --type agent filter value in status/fetch stays as-is
Confidence: high
Scope-risk: narrow
- Add -H, --header flag to dev and invoke commands
- Add --bearer-token flag to invoke command
- Fix --lines → --limit in evals.md logs evals section
Confidence: high
Scope-risk: narrow
- Rename --agent-client-id → --client-id, --agent-client-secret →
--client-secret in gateway docs (naming unification aws#705)
- Add missing flags to add agent: --authorizer-type, --discovery-url,
--allowed-audience, --allowed-clients, --allowed-scopes,
--custom-claims, --client-id, --client-secret, --idle-timeout,
--max-lifetime
- Add missing flags to create: --idle-timeout, --max-lifetime
- Add missing flags to add gateway: --runtimes, --custom-claims,
--policy-engine, --policy-engine-mode
- Fix --exception-level values: NONE/ALL (was undocumented)
Confidence: high
Scope-risk: narrow
The CLI help text says ALL but the schema (GatewayExceptionLevelSchema)
defines the valid values as NONE and DEBUG. Aligns commands.md with
gateway.md and the Zod schema.
- Remove stale "type": "AgentCoreMemory" from memory streaming example
- Add memory streaming flags: --delivery-type, --data-stream-arn,
--stream-content-level, --stream-delivery-resources
- Add --exec to dev command flag table (Container agents)
- Add --skip-install to create command flag table
- Add --identity-name to fetch access flag table
@Hweinstock

Copy link
Copy Markdown
Contributor

Looks like lint is failing, otherwise looks good:

Run npm run format:check
> @aws/agentcore@0.7.1 format:check
> prettier --check .
Checking formatting...
[warn] docs/commands.md
[warn] Code style issues found in the above file. Run Prettier with --write to fix.

@jariy17jariy17 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm and fix the lint

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

Labels

size/lPR size: L

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@aidandaly24@Hweinstock@jariy17
, 'i'); if (__m === '*' || __re.test(location.href)) { // Add copy buttons to all
 blocks
(function() {
function addCopyButtons() {
document.querySelectorAll('pre code').forEach(function(codeBlock) {
if (codeBlock.parentElement.hasAttribute('data-copy-added')) return;
codeBlock.parentElement.setAttribute('data-copy-added', 'true');
var btn = document.createElement('button');
btn.textContent = 'Copy';
btn.style.cssText = 'position:absolute;top:4px;right:4px;padding:2px 8px;font-size:11px;background:#4ecdc4;border:none;border-radius:4px;color:#1a1a2e;cursor:pointer;opacity:0.7;transition:opacity 0.2s;';
btn.onmouseover = function() { this.style.opacity = '1'; };
btn.onmouseout = function() { this.style.opacity = '0.7'; };
btn.onclick = function() {
navigator.clipboard.writeText(codeBlock.textContent).then(function() {
btn.textContent = 'Copied!';
setTimeout(function() { btn.textContent = 'Copy'; }, 1500);
});
};
codeBlock.parentElement.style.position = 'relative';
codeBlock.parentElement.appendChild(btn);
});
}
addCopyButtons();
// Re-run on dynamic content
var observer = new MutationObserver(addCopyButtons);
observer.observe(document.body, { childList: true, subtree: true });
})();
}
} catch(__e) { console.warn('[Userscript:Add Copy Buttons to Code Blocks]', __e); }
})();
(function(){
try {
var __m = "github.com";
var __re = new RegExp('^' + "github\\.com" + '
docs: fix 30 documentation inaccuracies found by source code audit by aidandaly24 · Pull Request #697 · aws/agentcore-cli · GitHub
Skip to content

docs: fix 30 documentation inaccuracies found by source code audit - #697

Merged
aidandaly24 merged 11 commits into
aws:mainfrom
aidandaly24:docs/fix-documentation-inaccuracies
Apr 9, 2026
Merged

docs: fix 30 documentation inaccuracies found by source code audit#697
aidandaly24 merged 11 commits into
aws:mainfrom
aidandaly24:docs/fix-documentation-inaccuracies

Conversation

@aidandaly24

@aidandaly24aidandaly24 commented Mar 27, 2026

Copy link
Copy Markdown
Contributor

Description

Comprehensive audit of all .md documentation files against actual source code (Zod schemas, CLI commands, CDK constructs). Fixes 70+ inaccuracies across 14 files where docs had drifted from the implementation, including alignment with several upstream refactors and documentation of new features merged after the initial audit.

Upstream refactors accounted for

PRChangeDoc Impact
#709Remove type fields from resource schemas, rename credential typeauthorizerTypeRemoved stale type fields from all JSON examples; renamed credential discriminator
#705add identityadd credential, --force-y/--yes, --plan--dry-runUpdated all command references and flag tables
#706agentsruntimes array, --agent--runtime flagsUpdated all flag tables, examples, and JSON config across 7 doc files
#707Dev command --invoke → positional [prompt] argAlready reflected in docs
#713Revert CUSTOM strategy and semanticOverrideNo doc changes needed (CUSTOM was never documented)

New features documented (merged after initial audit)

PRFeatureDoc Files
#763, #780Import subcommands (import runtime, import memory, import evaluator, import online-eval)commands.md
#750Exec mode for invoke and dev (--exec, --timeout)commands.md, local-development.md
#739Code-based evaluator support (--type, --lambda-arn, --timeout)commands.md, evals.md
#732Ground truth reference inputs (--assertion, --expected-trajectory, --expected-response)commands.md, evals.md
#531, #534Memory record streaming (--data-stream-arn, --stream-content-level, --delivery-type)commands.md, memory.md
#782--skip-install flag on createcommands.md
#774--identity-name option on fetch accesscommands.md

Key fixes by category

Schema/config drift (configuration.md, gateway.md, container-builds.md):

  • Added missing EPISODIC memory strategy and reflectionNamespaces field
  • Added lifecycleConfiguration (idleRuntimeSessionTimeout, maxLifetime) to agent spec
  • Added description, dockerfile, executionRoleArn to agent spec table
  • Fixed CUSTOM_JWT fields: allowedAudience/allowedClients/allowedScopes/customClaims are conditionally required
  • Fixed OAuth credential discoveryUrl: optional, not required (for imported providers)
  • Fixed memory strategies: defaults to empty array, not required
  • Added missing tags field to memory, credential, gateway, and online eval config tables
  • Added missing customClaims field to CUSTOM_JWT config
  • Fixed MCP Server outbound auth: removed api-key (only oauth/none valid per schema)
  • Added enableSemanticSearch, exceptionLevel, policyEngineConfiguration gateway fields
  • Fixed gateway target type count: six types (including lambda), not five
  • Removed stale "type": "AgentCoreRuntime" / "AgentCoreMemory" / "CustomEvaluator" / "OnlineEvaluationConfig" from all JSON examples
  • Fixed mcp.json reference → agentcore.json in local-development.md

Flag renames (commands.md, evals.md, local-development.md):

  • --agent--runtime, -a-r across all commands (status, dev, invoke, logs, traces, run eval, evals history, package, add online-eval, logs evals)
  • --agent-arn--runtime-arn, --agent-runtime-id--runtime-id
  • --plan--dry-run, --force-y/--yes
  • add identityadd credential, remove identityremove credential
  • --agent-client-id--client-id, --agent-client-secret--client-secret (gateway CUSTOM_JWT)
  • --exception-level valid values: NONE/DEBUG (not ALL)
  • --lines--limit in logs evals
  • Note: --agent-id and --agent-alias-id are unchanged (Bedrock Agent import flags)

Missing flags added (commands.md, evals.md):

  • add agent: --authorizer-type, --discovery-url, --allowed-audience, --allowed-clients, --allowed-scopes, --custom-claims, --client-id, --client-secret, --idle-timeout, --max-lifetime
  • create: --idle-timeout, --max-lifetime, --skip-install
  • add gateway: --runtimes, --custom-claims, --policy-engine, --policy-engine-mode
  • dev and invoke: -H, --header, --exec
  • invoke: --bearer-token, --exec, --timeout
  • add evaluator: --type, --lambda-arn, --timeout (code-based evaluator support)
  • run eval: --assertion, --expected-trajectory, --expected-response, --endpoint
  • add memory: --data-stream-arn, --stream-content-level, --delivery-type, --stream-delivery-resources
  • fetch access: --identity-name
  • gateway.md: --oauth-scopes in OAuth example

New sections added:

  • commands.md: Full import command section with 4 subcommands (runtime, memory, evaluator, online-eval)
  • evals.md: Evaluator Types section (LLM-as-a-Judge vs code-based), Ground Truth Reference Inputs subsection, Builtin Evaluator table (all 10 IDs with levels)
  • evals.md: Fixed eval results storage path (eval-runseval-results)
  • memory.md: Memory Record Streaming section with CLI flags and JSON config example
  • local-development.md: Exec Mode section covering dev exec, invoke exec, and TUI shortcuts

Stale values (commands.md, evals.md, frameworks.md):

  • Fixed model ID claude-3-5-sonnet-20250929claude-3-5-sonnet-20250514
  • Added policy-engine/policy to status --type enum
  • Removed ghost --tool-filter-description flag (not in schema or CLI)
  • Fixed evaluator level values: added TOOL_CALL, removed SPAN

Structural/naming (AGENTS.md, README.md, assets):

  • Removed AutoGen from supported frameworks list (5 frameworks, not 6)
  • Fixed "agents""runtimes" in gateway JSON example
  • Renamed identitycredential in all command references
  • Fixed file paths and directory references
  • Fixed regex constraints and enum values to match Zod schemas

Files modified

FileChanges
docs/configuration.mdSchema field tables, JSON examples, type removals, credential rename, tags fields, required/optional fixes
docs/commands.mdAll flag tables and examples, import subcommands, exec mode, streaming, evaluator types, ground truth
docs/evals.mdFlag tables, examples, evaluator types section, ground truth inputs, builtin evaluator table, eval-results path fix
docs/gateway.mdidentity → credential, agents → runtimes, oauth-scopes, target type count
docs/local-development.md--runtime flag in dev examples, exec mode section, mcp.json → agentcore.json
docs/memory.mdRemoved stale types, added memory record streaming section
docs/container-builds.mdRemoved stale "type": "AgentCoreRuntime" from JSON example
docs/frameworks.mdModel ID fix, framework list correction
AGENTS.mdidentity → credential in primitives
README.mdidentity → credential
src/assets/README.mdidentity → credential in vended template
src/assets/__tests__/__snapshots__/assets.snapshot.test.ts.snapUpdated snapshot

Known source code issue (not fixed here)

src/cli/primitives/GatewayPrimitive.ts has incorrect CLI help text: --exception-level says ALL (should be DEBUG) and --authorizer-type omits AWS_IAM. This is a source code bug that should be addressed in a separate PR.

Related Issue

N/A — discovered via systematic source code audit

Documentation PR

N/A — this PR is the documentation fix

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update
  • Other (please describe):

Testing

  • I ran npm run test:unit and npm run test:integ
  • I ran npm run typecheck
  • I ran npm run lint
  • If I modified src/assets/, I ran npm run test:update-snapshots and committed the updated snapshots

All changes are documentation-only (.md files + snapshot update). Each fix was verified against the corresponding Zod schema definition or CLI source code (src/cli/commands/*/command.tsx, src/cli/primitives/*.ts).

Note: 19 pre-existing test failures exist on upstream/main from the schema refactor (#709) — not introduced by this PR. The 2 e2e failures are expected on fork PRs (no AWS credentials).

Checklist

  • I have read the CONTRIBUTING document
  • I have added any necessary tests that prove my fix is effective or my feature works
  • I have updated the documentation accordingly
  • I have added an appropriate example to the documentation to outline the feature, or no new docs are needed
  • 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.

@aidandaly24
aidandaly24 requested a review from a teamMarch 27, 2026 04:40
@github-actionsgithub-actionsBot added the size/m PR size: M label Mar 27, 2026
@github-actionsgithub-actionsBot added size/m PR size: M and removed size/m PR size: M labels Mar 27, 2026
@aidandaly24
aidandaly24force-pushed the docs/fix-documentation-inaccuracies branch from f2e7d91 to f957db1CompareMarch 27, 2026 17:36
@github-actionsgithub-actionsBot added size/m PR size: M and removed size/m PR size: M labels Mar 27, 2026
@github-actions

github-actionsBot commented Mar 27, 2026

Copy link
Copy Markdown
Contributor

Package Tarball

aws-agentcore-0.7.1.tgz

How to install

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

@github-actionsgithub-actionsBot added size/m PR size: M and removed size/m PR size: M labels Mar 27, 2026
@github-actionsgithub-actionsBot added size/m PR size: M and removed size/m PR size: M labels Mar 27, 2026
@aidandaly24
aidandaly24force-pushed the docs/fix-documentation-inaccuracies branch from bd9373b to c4a8c0aCompareMarch 27, 2026 18:36
@github-actionsgithub-actionsBot added size/m PR size: M and removed size/m PR size: M labels Mar 27, 2026
@aidandaly24
aidandaly24force-pushed the docs/fix-documentation-inaccuracies branch from c4a8c0a to 727e1cfCompareMarch 27, 2026 21:39
@github-actionsgithub-actionsBot removed the size/m PR size: M label Mar 27, 2026
@github-actionsgithub-actionsBot added size/l PR size: L and removed size/l PR size: L labels Mar 28, 2026
@aidandaly24
aidandaly24force-pushed the docs/fix-documentation-inaccuracies branch from 1b43c59 to c8a57f4CompareMarch 28, 2026 01:22
@github-actionsgithub-actionsBot added size/l PR size: L and removed size/l PR size: L labels Mar 28, 2026
@github-actionsgithub-actionsBot added size/l PR size: L and removed size/l PR size: L labels Mar 28, 2026
@github-actionsgithub-actionsBot removed the size/l PR size: L label Mar 28, 2026
- configuration.md: fix networkMode PRIVATE→VPC, remove stale mcp.json
section, add 7 missing AgentEnvSpec fields, add Node.js runtimes,
fix gateway name max 63→100, list all 6 targetType values, fix memory
strategies wording, fix deployed-state.json path, add missing project
fields, fix CUSTOM_JWT field requirements, add customClaims field
- README.md: add CrewAI framework, update all 4 default model IDs,
add missing command sections
- frameworks.md: add CrewAI with model providers, add protocol
compatibility matrix (HTTP/MCP/A2A)
- commands.md: fix agent name max 64→48 chars, add fetch access section,
add policy-engine/policy to status --type, remove ghost
--tool-filter-description flag, fix model ID date
- gateway.md: add enableSemanticSearch/exceptionLevel config options,
fix mcp.json→agentcore.json, remove api-key from MCP Server auth
- transaction_search.md: fix traces command syntax
- container-builds.md: remove nonexistent --progress flag
- evals.md: fix model date 20250929→20250514
- local-development.md: fix mcp.json→agentcore.json
- AGENTS.md: remove AutoGen, remove stale mcp.json reference
- src/assets/README.md: add CrewAI to template list
- src/assets/agents/AGENTS.md: remove AutoGen
Constraint: All fixes verified against Zod schemas and CLI runtime
Constraint: CLI --help text bugs left for separate agent
Confidence: high
Scope-risk: narrow
- Remove `type` fields from agent, memory, evaluator, online-eval-config
examples and field tables (removed in aws#709)
- Remove `modelProvider` from agent spec (removed in aws#709)
- Rename credential `type` to `authorizerType` (renamed in aws#709)
- Rename `add identity` to `add credential` everywhere (aws#705)
- Replace `--force` with `-y/--yes` on remove commands (aws#705)
- Replace `--plan` with `--dry-run` on deploy command (aws#705)
- Update snapshot for src/assets/README.md change
Update `"agents": [...]` to `"runtimes": [...]` in the gateway
configuration example to match the schema rename in aws#706.
Update all documentation to reflect upstream renames:
- --agent → --runtime, -a → -r across all commands
- --agent-arn → --runtime-arn, --agent-runtime-id → --runtime-id
- Remove stale "type" fields from JSON examples in evals.md,
memory.md, and container-builds.md
Constraint: --agent-id, --agent-alias-id, --agent-client-id,
--agent-client-secret are unchanged (different semantics)
Constraint: --type agent filter value in status/fetch stays as-is
Confidence: high
Scope-risk: narrow
- Add -H, --header flag to dev and invoke commands
- Add --bearer-token flag to invoke command
- Fix --lines → --limit in evals.md logs evals section
Confidence: high
Scope-risk: narrow
- Rename --agent-client-id → --client-id, --agent-client-secret →
--client-secret in gateway docs (naming unification aws#705)
- Add missing flags to add agent: --authorizer-type, --discovery-url,
--allowed-audience, --allowed-clients, --allowed-scopes,
--custom-claims, --client-id, --client-secret, --idle-timeout,
--max-lifetime
- Add missing flags to create: --idle-timeout, --max-lifetime
- Add missing flags to add gateway: --runtimes, --custom-claims,
--policy-engine, --policy-engine-mode
- Fix --exception-level values: NONE/ALL (was undocumented)
Confidence: high
Scope-risk: narrow
The CLI help text says ALL but the schema (GatewayExceptionLevelSchema)
defines the valid values as NONE and DEBUG. Aligns commands.md with
gateway.md and the Zod schema.
- Remove stale "type": "AgentCoreMemory" from memory streaming example
- Add memory streaming flags: --delivery-type, --data-stream-arn,
--stream-content-level, --stream-delivery-resources
- Add --exec to dev command flag table (Container agents)
- Add --skip-install to create command flag table
- Add --identity-name to fetch access flag table
@Hweinstock

Copy link
Copy Markdown
Contributor

Looks like lint is failing, otherwise looks good:

Run npm run format:check
> @aws/agentcore@0.7.1 format:check
> prettier --check .
Checking formatting...
[warn] docs/commands.md
[warn] Code style issues found in the above file. Run Prettier with --write to fix.

@jariy17jariy17 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm and fix the lint

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

Labels

size/lPR size: L

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@aidandaly24@Hweinstock@jariy17
, 'i'); if (__m === '*' || __re.test(location.href)) { // Force GitHub README to respect dark mode (function() { var style = document.createElement('style'); style.textContent = ' .markdown-body { color-scheme: dark light; } .markdown-body pre { background: #161b22 !important; } .markdown-body code { background: rgba(110, 118, 129, 0.4) !important; } .markdown-body table th, .markdown-body table td { border-color: #30363d !important; } .markdown-body img { background: #0d1117; } .markdown-body blockquote { border-left-color: #8b949e; } .markdown-body hr { border-color: #30363d; } '; document.head.appendChild(style); })(); } } catch(__e) { console.warn('[Userscript:GitHub Dark Mode README Fix]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + ' docs: fix 30 documentation inaccuracies found by source code audit by aidandaly24 · Pull Request #697 · aws/agentcore-cli · GitHub
Skip to content

docs: fix 30 documentation inaccuracies found by source code audit - #697

Merged
aidandaly24 merged 11 commits into
aws:mainfrom
aidandaly24:docs/fix-documentation-inaccuracies
Apr 9, 2026
Merged

docs: fix 30 documentation inaccuracies found by source code audit#697
aidandaly24 merged 11 commits into
aws:mainfrom
aidandaly24:docs/fix-documentation-inaccuracies

Conversation

@aidandaly24

@aidandaly24aidandaly24 commented Mar 27, 2026

Copy link
Copy Markdown
Contributor

Description

Comprehensive audit of all .md documentation files against actual source code (Zod schemas, CLI commands, CDK constructs). Fixes 70+ inaccuracies across 14 files where docs had drifted from the implementation, including alignment with several upstream refactors and documentation of new features merged after the initial audit.

Upstream refactors accounted for

PRChangeDoc Impact
#709Remove type fields from resource schemas, rename credential typeauthorizerTypeRemoved stale type fields from all JSON examples; renamed credential discriminator
#705add identityadd credential, --force-y/--yes, --plan--dry-runUpdated all command references and flag tables
#706agentsruntimes array, --agent--runtime flagsUpdated all flag tables, examples, and JSON config across 7 doc files
#707Dev command --invoke → positional [prompt] argAlready reflected in docs
#713Revert CUSTOM strategy and semanticOverrideNo doc changes needed (CUSTOM was never documented)

New features documented (merged after initial audit)

PRFeatureDoc Files
#763, #780Import subcommands (import runtime, import memory, import evaluator, import online-eval)commands.md
#750Exec mode for invoke and dev (--exec, --timeout)commands.md, local-development.md
#739Code-based evaluator support (--type, --lambda-arn, --timeout)commands.md, evals.md
#732Ground truth reference inputs (--assertion, --expected-trajectory, --expected-response)commands.md, evals.md
#531, #534Memory record streaming (--data-stream-arn, --stream-content-level, --delivery-type)commands.md, memory.md
#782--skip-install flag on createcommands.md
#774--identity-name option on fetch accesscommands.md

Key fixes by category

Schema/config drift (configuration.md, gateway.md, container-builds.md):

  • Added missing EPISODIC memory strategy and reflectionNamespaces field
  • Added lifecycleConfiguration (idleRuntimeSessionTimeout, maxLifetime) to agent spec
  • Added description, dockerfile, executionRoleArn to agent spec table
  • Fixed CUSTOM_JWT fields: allowedAudience/allowedClients/allowedScopes/customClaims are conditionally required
  • Fixed OAuth credential discoveryUrl: optional, not required (for imported providers)
  • Fixed memory strategies: defaults to empty array, not required
  • Added missing tags field to memory, credential, gateway, and online eval config tables
  • Added missing customClaims field to CUSTOM_JWT config
  • Fixed MCP Server outbound auth: removed api-key (only oauth/none valid per schema)
  • Added enableSemanticSearch, exceptionLevel, policyEngineConfiguration gateway fields
  • Fixed gateway target type count: six types (including lambda), not five
  • Removed stale "type": "AgentCoreRuntime" / "AgentCoreMemory" / "CustomEvaluator" / "OnlineEvaluationConfig" from all JSON examples
  • Fixed mcp.json reference → agentcore.json in local-development.md

Flag renames (commands.md, evals.md, local-development.md):

  • --agent--runtime, -a-r across all commands (status, dev, invoke, logs, traces, run eval, evals history, package, add online-eval, logs evals)
  • --agent-arn--runtime-arn, --agent-runtime-id--runtime-id
  • --plan--dry-run, --force-y/--yes
  • add identityadd credential, remove identityremove credential
  • --agent-client-id--client-id, --agent-client-secret--client-secret (gateway CUSTOM_JWT)
  • --exception-level valid values: NONE/DEBUG (not ALL)
  • --lines--limit in logs evals
  • Note: --agent-id and --agent-alias-id are unchanged (Bedrock Agent import flags)

Missing flags added (commands.md, evals.md):

  • add agent: --authorizer-type, --discovery-url, --allowed-audience, --allowed-clients, --allowed-scopes, --custom-claims, --client-id, --client-secret, --idle-timeout, --max-lifetime
  • create: --idle-timeout, --max-lifetime, --skip-install
  • add gateway: --runtimes, --custom-claims, --policy-engine, --policy-engine-mode
  • dev and invoke: -H, --header, --exec
  • invoke: --bearer-token, --exec, --timeout
  • add evaluator: --type, --lambda-arn, --timeout (code-based evaluator support)
  • run eval: --assertion, --expected-trajectory, --expected-response, --endpoint
  • add memory: --data-stream-arn, --stream-content-level, --delivery-type, --stream-delivery-resources
  • fetch access: --identity-name
  • gateway.md: --oauth-scopes in OAuth example

New sections added:

  • commands.md: Full import command section with 4 subcommands (runtime, memory, evaluator, online-eval)
  • evals.md: Evaluator Types section (LLM-as-a-Judge vs code-based), Ground Truth Reference Inputs subsection, Builtin Evaluator table (all 10 IDs with levels)
  • evals.md: Fixed eval results storage path (eval-runseval-results)
  • memory.md: Memory Record Streaming section with CLI flags and JSON config example
  • local-development.md: Exec Mode section covering dev exec, invoke exec, and TUI shortcuts

Stale values (commands.md, evals.md, frameworks.md):

  • Fixed model ID claude-3-5-sonnet-20250929claude-3-5-sonnet-20250514
  • Added policy-engine/policy to status --type enum
  • Removed ghost --tool-filter-description flag (not in schema or CLI)
  • Fixed evaluator level values: added TOOL_CALL, removed SPAN

Structural/naming (AGENTS.md, README.md, assets):

  • Removed AutoGen from supported frameworks list (5 frameworks, not 6)
  • Fixed "agents""runtimes" in gateway JSON example
  • Renamed identitycredential in all command references
  • Fixed file paths and directory references
  • Fixed regex constraints and enum values to match Zod schemas

Files modified

FileChanges
docs/configuration.mdSchema field tables, JSON examples, type removals, credential rename, tags fields, required/optional fixes
docs/commands.mdAll flag tables and examples, import subcommands, exec mode, streaming, evaluator types, ground truth
docs/evals.mdFlag tables, examples, evaluator types section, ground truth inputs, builtin evaluator table, eval-results path fix
docs/gateway.mdidentity → credential, agents → runtimes, oauth-scopes, target type count
docs/local-development.md--runtime flag in dev examples, exec mode section, mcp.json → agentcore.json
docs/memory.mdRemoved stale types, added memory record streaming section
docs/container-builds.mdRemoved stale "type": "AgentCoreRuntime" from JSON example
docs/frameworks.mdModel ID fix, framework list correction
AGENTS.mdidentity → credential in primitives
README.mdidentity → credential
src/assets/README.mdidentity → credential in vended template
src/assets/__tests__/__snapshots__/assets.snapshot.test.ts.snapUpdated snapshot

Known source code issue (not fixed here)

src/cli/primitives/GatewayPrimitive.ts has incorrect CLI help text: --exception-level says ALL (should be DEBUG) and --authorizer-type omits AWS_IAM. This is a source code bug that should be addressed in a separate PR.

Related Issue

N/A — discovered via systematic source code audit

Documentation PR

N/A — this PR is the documentation fix

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update
  • Other (please describe):

Testing

  • I ran npm run test:unit and npm run test:integ
  • I ran npm run typecheck
  • I ran npm run lint
  • If I modified src/assets/, I ran npm run test:update-snapshots and committed the updated snapshots

All changes are documentation-only (.md files + snapshot update). Each fix was verified against the corresponding Zod schema definition or CLI source code (src/cli/commands/*/command.tsx, src/cli/primitives/*.ts).

Note: 19 pre-existing test failures exist on upstream/main from the schema refactor (#709) — not introduced by this PR. The 2 e2e failures are expected on fork PRs (no AWS credentials).

Checklist

  • I have read the CONTRIBUTING document
  • I have added any necessary tests that prove my fix is effective or my feature works
  • I have updated the documentation accordingly
  • I have added an appropriate example to the documentation to outline the feature, or no new docs are needed
  • 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.

@aidandaly24
aidandaly24 requested a review from a teamMarch 27, 2026 04:40
@github-actionsgithub-actionsBot added the size/m PR size: M label Mar 27, 2026
@github-actionsgithub-actionsBot added size/m PR size: M and removed size/m PR size: M labels Mar 27, 2026
@aidandaly24
aidandaly24force-pushed the docs/fix-documentation-inaccuracies branch from f2e7d91 to f957db1CompareMarch 27, 2026 17:36
@github-actionsgithub-actionsBot added size/m PR size: M and removed size/m PR size: M labels Mar 27, 2026
@github-actions

github-actionsBot commented Mar 27, 2026

Copy link
Copy Markdown
Contributor

Package Tarball

aws-agentcore-0.7.1.tgz

How to install

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

@github-actionsgithub-actionsBot added size/m PR size: M and removed size/m PR size: M labels Mar 27, 2026
@github-actionsgithub-actionsBot added size/m PR size: M and removed size/m PR size: M labels Mar 27, 2026
@aidandaly24
aidandaly24force-pushed the docs/fix-documentation-inaccuracies branch from bd9373b to c4a8c0aCompareMarch 27, 2026 18:36
@github-actionsgithub-actionsBot added size/m PR size: M and removed size/m PR size: M labels Mar 27, 2026
@aidandaly24
aidandaly24force-pushed the docs/fix-documentation-inaccuracies branch from c4a8c0a to 727e1cfCompareMarch 27, 2026 21:39
@github-actionsgithub-actionsBot removed the size/m PR size: M label Mar 27, 2026
@github-actionsgithub-actionsBot added size/l PR size: L and removed size/l PR size: L labels Mar 28, 2026
@aidandaly24
aidandaly24force-pushed the docs/fix-documentation-inaccuracies branch from 1b43c59 to c8a57f4CompareMarch 28, 2026 01:22
@github-actionsgithub-actionsBot added size/l PR size: L and removed size/l PR size: L labels Mar 28, 2026
@github-actionsgithub-actionsBot added size/l PR size: L and removed size/l PR size: L labels Mar 28, 2026
@github-actionsgithub-actionsBot removed the size/l PR size: L label Mar 28, 2026
- configuration.md: fix networkMode PRIVATE→VPC, remove stale mcp.json
section, add 7 missing AgentEnvSpec fields, add Node.js runtimes,
fix gateway name max 63→100, list all 6 targetType values, fix memory
strategies wording, fix deployed-state.json path, add missing project
fields, fix CUSTOM_JWT field requirements, add customClaims field
- README.md: add CrewAI framework, update all 4 default model IDs,
add missing command sections
- frameworks.md: add CrewAI with model providers, add protocol
compatibility matrix (HTTP/MCP/A2A)
- commands.md: fix agent name max 64→48 chars, add fetch access section,
add policy-engine/policy to status --type, remove ghost
--tool-filter-description flag, fix model ID date
- gateway.md: add enableSemanticSearch/exceptionLevel config options,
fix mcp.json→agentcore.json, remove api-key from MCP Server auth
- transaction_search.md: fix traces command syntax
- container-builds.md: remove nonexistent --progress flag
- evals.md: fix model date 20250929→20250514
- local-development.md: fix mcp.json→agentcore.json
- AGENTS.md: remove AutoGen, remove stale mcp.json reference
- src/assets/README.md: add CrewAI to template list
- src/assets/agents/AGENTS.md: remove AutoGen
Constraint: All fixes verified against Zod schemas and CLI runtime
Constraint: CLI --help text bugs left for separate agent
Confidence: high
Scope-risk: narrow
- Remove `type` fields from agent, memory, evaluator, online-eval-config
examples and field tables (removed in aws#709)
- Remove `modelProvider` from agent spec (removed in aws#709)
- Rename credential `type` to `authorizerType` (renamed in aws#709)
- Rename `add identity` to `add credential` everywhere (aws#705)
- Replace `--force` with `-y/--yes` on remove commands (aws#705)
- Replace `--plan` with `--dry-run` on deploy command (aws#705)
- Update snapshot for src/assets/README.md change
Update `"agents": [...]` to `"runtimes": [...]` in the gateway
configuration example to match the schema rename in aws#706.
Update all documentation to reflect upstream renames:
- --agent → --runtime, -a → -r across all commands
- --agent-arn → --runtime-arn, --agent-runtime-id → --runtime-id
- Remove stale "type" fields from JSON examples in evals.md,
memory.md, and container-builds.md
Constraint: --agent-id, --agent-alias-id, --agent-client-id,
--agent-client-secret are unchanged (different semantics)
Constraint: --type agent filter value in status/fetch stays as-is
Confidence: high
Scope-risk: narrow
- Add -H, --header flag to dev and invoke commands
- Add --bearer-token flag to invoke command
- Fix --lines → --limit in evals.md logs evals section
Confidence: high
Scope-risk: narrow
- Rename --agent-client-id → --client-id, --agent-client-secret →
--client-secret in gateway docs (naming unification aws#705)
- Add missing flags to add agent: --authorizer-type, --discovery-url,
--allowed-audience, --allowed-clients, --allowed-scopes,
--custom-claims, --client-id, --client-secret, --idle-timeout,
--max-lifetime
- Add missing flags to create: --idle-timeout, --max-lifetime
- Add missing flags to add gateway: --runtimes, --custom-claims,
--policy-engine, --policy-engine-mode
- Fix --exception-level values: NONE/ALL (was undocumented)
Confidence: high
Scope-risk: narrow
The CLI help text says ALL but the schema (GatewayExceptionLevelSchema)
defines the valid values as NONE and DEBUG. Aligns commands.md with
gateway.md and the Zod schema.
- Remove stale "type": "AgentCoreMemory" from memory streaming example
- Add memory streaming flags: --delivery-type, --data-stream-arn,
--stream-content-level, --stream-delivery-resources
- Add --exec to dev command flag table (Container agents)
- Add --skip-install to create command flag table
- Add --identity-name to fetch access flag table
@Hweinstock

Copy link
Copy Markdown
Contributor

Looks like lint is failing, otherwise looks good:

Run npm run format:check
> @aws/agentcore@0.7.1 format:check
> prettier --check .
Checking formatting...
[warn] docs/commands.md
[warn] Code style issues found in the above file. Run Prettier with --write to fix.

@jariy17jariy17 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm and fix the lint

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

Labels

size/lPR size: L

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@aidandaly24@Hweinstock@jariy17
, 'i'); if (__m === '*' || __re.test(location.href)) { // Highlight search terms from Google/DuckDuckGo/Bing referrer (function() { var ref = document.referrer; var terms = []; if (ref.includes('google.com') || ref.includes('duckduckgo.com') || ref.includes('bing.com')) { var url = new URL(ref); var q = url.searchParams.get('q') || url.searchParams.get('p'); if (q) { terms = q.split(/\s+/).filter(function(t) { return t.length > 2; }); } } if (terms.length === 0) return; var style = document.createElement('style'); style.textContent = '.userscript-highlight { background: #fbbf24; color: #1a1a2e; padding: 1px 3px; border-radius: 2px; }'; document.head.appendChild(style); function highlight(node) { if (node.nodeType === 3) { // text node var text = node.textContent; var found = false; terms.forEach(function(term) { var regex = new RegExp('(' + term.replace(/[.*+?^${}()|[\]\\]/g, '\\') + ')', 'gi'); if (regex.test(text)) { found = true; var frag = document.createDocumentFragment(); var parts = text.split(regex); parts.forEach(function(part, i) { if (i % 2 === 0) { frag.appendChild(document.createTextNode(part)); } else { var span = document.createElement('span'); span.className = 'userscript-highlight'; span.textContent = part; frag.appendChild(span); } }); node.parentNode.replaceChild(frag, node); } }); } else if (node.nodeType === 1 && node.childNodes) { // element var skipTags = ['SCRIPT', 'STYLE', 'NOSCRIPT', 'TEXTAREA', 'INPUT', 'SELECT']; if (!skipTags.includes(node.tagName)) { Array.from(node.childNodes).forEach(highlight); } } } highlight(document.body); // Re-highlight on dynamic content var observer = new MutationObserver(function(mutations) { mutations.forEach(function(m) { m.addedNodes.forEach(function(node) { if (node.nodeType === 1 || node.nodeType === 3) highlight(node); }); }); }); observer.observe(document.body, { childList: true, subtree: true }); })(); } } catch(__e) { console.warn('[Userscript:Highlight Search Terms]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + ' docs: fix 30 documentation inaccuracies found by source code audit by aidandaly24 · Pull Request #697 · aws/agentcore-cli · GitHub
Skip to content

docs: fix 30 documentation inaccuracies found by source code audit - #697

Merged
aidandaly24 merged 11 commits into
aws:mainfrom
aidandaly24:docs/fix-documentation-inaccuracies
Apr 9, 2026
Merged

docs: fix 30 documentation inaccuracies found by source code audit#697
aidandaly24 merged 11 commits into
aws:mainfrom
aidandaly24:docs/fix-documentation-inaccuracies

Conversation

@aidandaly24

@aidandaly24aidandaly24 commented Mar 27, 2026

Copy link
Copy Markdown
Contributor

Description

Comprehensive audit of all .md documentation files against actual source code (Zod schemas, CLI commands, CDK constructs). Fixes 70+ inaccuracies across 14 files where docs had drifted from the implementation, including alignment with several upstream refactors and documentation of new features merged after the initial audit.

Upstream refactors accounted for

PRChangeDoc Impact
#709Remove type fields from resource schemas, rename credential typeauthorizerTypeRemoved stale type fields from all JSON examples; renamed credential discriminator
#705add identityadd credential, --force-y/--yes, --plan--dry-runUpdated all command references and flag tables
#706agentsruntimes array, --agent--runtime flagsUpdated all flag tables, examples, and JSON config across 7 doc files
#707Dev command --invoke → positional [prompt] argAlready reflected in docs
#713Revert CUSTOM strategy and semanticOverrideNo doc changes needed (CUSTOM was never documented)

New features documented (merged after initial audit)

PRFeatureDoc Files
#763, #780Import subcommands (import runtime, import memory, import evaluator, import online-eval)commands.md
#750Exec mode for invoke and dev (--exec, --timeout)commands.md, local-development.md
#739Code-based evaluator support (--type, --lambda-arn, --timeout)commands.md, evals.md
#732Ground truth reference inputs (--assertion, --expected-trajectory, --expected-response)commands.md, evals.md
#531, #534Memory record streaming (--data-stream-arn, --stream-content-level, --delivery-type)commands.md, memory.md
#782--skip-install flag on createcommands.md
#774--identity-name option on fetch accesscommands.md

Key fixes by category

Schema/config drift (configuration.md, gateway.md, container-builds.md):

  • Added missing EPISODIC memory strategy and reflectionNamespaces field
  • Added lifecycleConfiguration (idleRuntimeSessionTimeout, maxLifetime) to agent spec
  • Added description, dockerfile, executionRoleArn to agent spec table
  • Fixed CUSTOM_JWT fields: allowedAudience/allowedClients/allowedScopes/customClaims are conditionally required
  • Fixed OAuth credential discoveryUrl: optional, not required (for imported providers)
  • Fixed memory strategies: defaults to empty array, not required
  • Added missing tags field to memory, credential, gateway, and online eval config tables
  • Added missing customClaims field to CUSTOM_JWT config
  • Fixed MCP Server outbound auth: removed api-key (only oauth/none valid per schema)
  • Added enableSemanticSearch, exceptionLevel, policyEngineConfiguration gateway fields
  • Fixed gateway target type count: six types (including lambda), not five
  • Removed stale "type": "AgentCoreRuntime" / "AgentCoreMemory" / "CustomEvaluator" / "OnlineEvaluationConfig" from all JSON examples
  • Fixed mcp.json reference → agentcore.json in local-development.md

Flag renames (commands.md, evals.md, local-development.md):

  • --agent--runtime, -a-r across all commands (status, dev, invoke, logs, traces, run eval, evals history, package, add online-eval, logs evals)
  • --agent-arn--runtime-arn, --agent-runtime-id--runtime-id
  • --plan--dry-run, --force-y/--yes
  • add identityadd credential, remove identityremove credential
  • --agent-client-id--client-id, --agent-client-secret--client-secret (gateway CUSTOM_JWT)
  • --exception-level valid values: NONE/DEBUG (not ALL)
  • --lines--limit in logs evals
  • Note: --agent-id and --agent-alias-id are unchanged (Bedrock Agent import flags)

Missing flags added (commands.md, evals.md):

  • add agent: --authorizer-type, --discovery-url, --allowed-audience, --allowed-clients, --allowed-scopes, --custom-claims, --client-id, --client-secret, --idle-timeout, --max-lifetime
  • create: --idle-timeout, --max-lifetime, --skip-install
  • add gateway: --runtimes, --custom-claims, --policy-engine, --policy-engine-mode
  • dev and invoke: -H, --header, --exec
  • invoke: --bearer-token, --exec, --timeout
  • add evaluator: --type, --lambda-arn, --timeout (code-based evaluator support)
  • run eval: --assertion, --expected-trajectory, --expected-response, --endpoint
  • add memory: --data-stream-arn, --stream-content-level, --delivery-type, --stream-delivery-resources
  • fetch access: --identity-name
  • gateway.md: --oauth-scopes in OAuth example

New sections added:

  • commands.md: Full import command section with 4 subcommands (runtime, memory, evaluator, online-eval)
  • evals.md: Evaluator Types section (LLM-as-a-Judge vs code-based), Ground Truth Reference Inputs subsection, Builtin Evaluator table (all 10 IDs with levels)
  • evals.md: Fixed eval results storage path (eval-runseval-results)
  • memory.md: Memory Record Streaming section with CLI flags and JSON config example
  • local-development.md: Exec Mode section covering dev exec, invoke exec, and TUI shortcuts

Stale values (commands.md, evals.md, frameworks.md):

  • Fixed model ID claude-3-5-sonnet-20250929claude-3-5-sonnet-20250514
  • Added policy-engine/policy to status --type enum
  • Removed ghost --tool-filter-description flag (not in schema or CLI)
  • Fixed evaluator level values: added TOOL_CALL, removed SPAN

Structural/naming (AGENTS.md, README.md, assets):

  • Removed AutoGen from supported frameworks list (5 frameworks, not 6)
  • Fixed "agents""runtimes" in gateway JSON example
  • Renamed identitycredential in all command references
  • Fixed file paths and directory references
  • Fixed regex constraints and enum values to match Zod schemas

Files modified

FileChanges
docs/configuration.mdSchema field tables, JSON examples, type removals, credential rename, tags fields, required/optional fixes
docs/commands.mdAll flag tables and examples, import subcommands, exec mode, streaming, evaluator types, ground truth
docs/evals.mdFlag tables, examples, evaluator types section, ground truth inputs, builtin evaluator table, eval-results path fix
docs/gateway.mdidentity → credential, agents → runtimes, oauth-scopes, target type count
docs/local-development.md--runtime flag in dev examples, exec mode section, mcp.json → agentcore.json
docs/memory.mdRemoved stale types, added memory record streaming section
docs/container-builds.mdRemoved stale "type": "AgentCoreRuntime" from JSON example
docs/frameworks.mdModel ID fix, framework list correction
AGENTS.mdidentity → credential in primitives
README.mdidentity → credential
src/assets/README.mdidentity → credential in vended template
src/assets/__tests__/__snapshots__/assets.snapshot.test.ts.snapUpdated snapshot

Known source code issue (not fixed here)

src/cli/primitives/GatewayPrimitive.ts has incorrect CLI help text: --exception-level says ALL (should be DEBUG) and --authorizer-type omits AWS_IAM. This is a source code bug that should be addressed in a separate PR.

Related Issue

N/A — discovered via systematic source code audit

Documentation PR

N/A — this PR is the documentation fix

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update
  • Other (please describe):

Testing

  • I ran npm run test:unit and npm run test:integ
  • I ran npm run typecheck
  • I ran npm run lint
  • If I modified src/assets/, I ran npm run test:update-snapshots and committed the updated snapshots

All changes are documentation-only (.md files + snapshot update). Each fix was verified against the corresponding Zod schema definition or CLI source code (src/cli/commands/*/command.tsx, src/cli/primitives/*.ts).

Note: 19 pre-existing test failures exist on upstream/main from the schema refactor (#709) — not introduced by this PR. The 2 e2e failures are expected on fork PRs (no AWS credentials).

Checklist

  • I have read the CONTRIBUTING document
  • I have added any necessary tests that prove my fix is effective or my feature works
  • I have updated the documentation accordingly
  • I have added an appropriate example to the documentation to outline the feature, or no new docs are needed
  • 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.

@aidandaly24
aidandaly24 requested a review from a teamMarch 27, 2026 04:40
@github-actionsgithub-actionsBot added the size/m PR size: M label Mar 27, 2026
@github-actionsgithub-actionsBot added size/m PR size: M and removed size/m PR size: M labels Mar 27, 2026
@aidandaly24
aidandaly24force-pushed the docs/fix-documentation-inaccuracies branch from f2e7d91 to f957db1CompareMarch 27, 2026 17:36
@github-actionsgithub-actionsBot added size/m PR size: M and removed size/m PR size: M labels Mar 27, 2026
@github-actions

github-actionsBot commented Mar 27, 2026

Copy link
Copy Markdown
Contributor

Package Tarball

aws-agentcore-0.7.1.tgz

How to install

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

@github-actionsgithub-actionsBot added size/m PR size: M and removed size/m PR size: M labels Mar 27, 2026
@github-actionsgithub-actionsBot added size/m PR size: M and removed size/m PR size: M labels Mar 27, 2026
@aidandaly24
aidandaly24force-pushed the docs/fix-documentation-inaccuracies branch from bd9373b to c4a8c0aCompareMarch 27, 2026 18:36
@github-actionsgithub-actionsBot added size/m PR size: M and removed size/m PR size: M labels Mar 27, 2026
@aidandaly24
aidandaly24force-pushed the docs/fix-documentation-inaccuracies branch from c4a8c0a to 727e1cfCompareMarch 27, 2026 21:39
@github-actionsgithub-actionsBot removed the size/m PR size: M label Mar 27, 2026
@github-actionsgithub-actionsBot added size/l PR size: L and removed size/l PR size: L labels Mar 28, 2026
@aidandaly24
aidandaly24force-pushed the docs/fix-documentation-inaccuracies branch from 1b43c59 to c8a57f4CompareMarch 28, 2026 01:22
@github-actionsgithub-actionsBot added size/l PR size: L and removed size/l PR size: L labels Mar 28, 2026
@github-actionsgithub-actionsBot added size/l PR size: L and removed size/l PR size: L labels Mar 28, 2026
@github-actionsgithub-actionsBot removed the size/l PR size: L label Mar 28, 2026
- configuration.md: fix networkMode PRIVATE→VPC, remove stale mcp.json
section, add 7 missing AgentEnvSpec fields, add Node.js runtimes,
fix gateway name max 63→100, list all 6 targetType values, fix memory
strategies wording, fix deployed-state.json path, add missing project
fields, fix CUSTOM_JWT field requirements, add customClaims field
- README.md: add CrewAI framework, update all 4 default model IDs,
add missing command sections
- frameworks.md: add CrewAI with model providers, add protocol
compatibility matrix (HTTP/MCP/A2A)
- commands.md: fix agent name max 64→48 chars, add fetch access section,
add policy-engine/policy to status --type, remove ghost
--tool-filter-description flag, fix model ID date
- gateway.md: add enableSemanticSearch/exceptionLevel config options,
fix mcp.json→agentcore.json, remove api-key from MCP Server auth
- transaction_search.md: fix traces command syntax
- container-builds.md: remove nonexistent --progress flag
- evals.md: fix model date 20250929→20250514
- local-development.md: fix mcp.json→agentcore.json
- AGENTS.md: remove AutoGen, remove stale mcp.json reference
- src/assets/README.md: add CrewAI to template list
- src/assets/agents/AGENTS.md: remove AutoGen
Constraint: All fixes verified against Zod schemas and CLI runtime
Constraint: CLI --help text bugs left for separate agent
Confidence: high
Scope-risk: narrow
- Remove `type` fields from agent, memory, evaluator, online-eval-config
examples and field tables (removed in aws#709)
- Remove `modelProvider` from agent spec (removed in aws#709)
- Rename credential `type` to `authorizerType` (renamed in aws#709)
- Rename `add identity` to `add credential` everywhere (aws#705)
- Replace `--force` with `-y/--yes` on remove commands (aws#705)
- Replace `--plan` with `--dry-run` on deploy command (aws#705)
- Update snapshot for src/assets/README.md change
Update `"agents": [...]` to `"runtimes": [...]` in the gateway
configuration example to match the schema rename in aws#706.
Update all documentation to reflect upstream renames:
- --agent → --runtime, -a → -r across all commands
- --agent-arn → --runtime-arn, --agent-runtime-id → --runtime-id
- Remove stale "type" fields from JSON examples in evals.md,
memory.md, and container-builds.md
Constraint: --agent-id, --agent-alias-id, --agent-client-id,
--agent-client-secret are unchanged (different semantics)
Constraint: --type agent filter value in status/fetch stays as-is
Confidence: high
Scope-risk: narrow
- Add -H, --header flag to dev and invoke commands
- Add --bearer-token flag to invoke command
- Fix --lines → --limit in evals.md logs evals section
Confidence: high
Scope-risk: narrow
- Rename --agent-client-id → --client-id, --agent-client-secret →
--client-secret in gateway docs (naming unification aws#705)
- Add missing flags to add agent: --authorizer-type, --discovery-url,
--allowed-audience, --allowed-clients, --allowed-scopes,
--custom-claims, --client-id, --client-secret, --idle-timeout,
--max-lifetime
- Add missing flags to create: --idle-timeout, --max-lifetime
- Add missing flags to add gateway: --runtimes, --custom-claims,
--policy-engine, --policy-engine-mode
- Fix --exception-level values: NONE/ALL (was undocumented)
Confidence: high
Scope-risk: narrow
The CLI help text says ALL but the schema (GatewayExceptionLevelSchema)
defines the valid values as NONE and DEBUG. Aligns commands.md with
gateway.md and the Zod schema.
- Remove stale "type": "AgentCoreMemory" from memory streaming example
- Add memory streaming flags: --delivery-type, --data-stream-arn,
--stream-content-level, --stream-delivery-resources
- Add --exec to dev command flag table (Container agents)
- Add --skip-install to create command flag table
- Add --identity-name to fetch access flag table
@Hweinstock

Copy link
Copy Markdown
Contributor

Looks like lint is failing, otherwise looks good:

Run npm run format:check
> @aws/agentcore@0.7.1 format:check
> prettier --check .
Checking formatting...
[warn] docs/commands.md
[warn] Code style issues found in the above file. Run Prettier with --write to fix.

@jariy17jariy17 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm and fix the lint

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

Labels

size/lPR size: L

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@aidandaly24@Hweinstock@jariy17
, 'i'); if (__m === '*' || __re.test(location.href)) { // Strip utm_, fbclid, gclid, etc. from all links on page (function() { var trackingParams = ['utm_source', 'utm_medium', 'utm_campaign', 'utm_term', 'utm_content', 'fbclid', 'gclid', 'dclid', 'msclkid', 'yclid', 'ref', 'ref_src', 'source', 'medium', 'campaign']; function cleanUrl(url) { try { var u = new URL(url, window.location.origin); var changed = false; trackingParams.forEach(function(p) { if (u.searchParams.has(p)) { u.searchParams.delete(p); changed = true; } }); return changed ? u.toString() : url; } catch (e) { return url; } } function cleanLinks() { document.querySelectorAll('a[href]').forEach(function(a) { var clean = cleanUrl(a.href); if (clean !== a.href) a.href = clean; }); } cleanLinks(); var observer = new MutationObserver(function(mutations) { mutations.forEach(function(m) { m.addedNodes.forEach(function(node) { if (node.nodeType === 1) { if (node.tagName === 'A') cleanLinks(); node.querySelectorAll('a[href]').forEach(function(a) { var clean = cleanUrl(a.href); if (clean !== a.href) a.href = clean; }); } }); }); }); observer.observe(document.body, { childList: true, subtree: true }); })(); } } catch(__e) { console.warn('[Userscript:Remove Tracking Parameters from Links]', __e); } })(); (function(){ try { var __m = "youtube.com"; var __re = new RegExp('^' + "youtube\\.com" + ' docs: fix 30 documentation inaccuracies found by source code audit by aidandaly24 · Pull Request #697 · aws/agentcore-cli · GitHub
Skip to content

docs: fix 30 documentation inaccuracies found by source code audit - #697

Merged
aidandaly24 merged 11 commits into
aws:mainfrom
aidandaly24:docs/fix-documentation-inaccuracies
Apr 9, 2026
Merged

docs: fix 30 documentation inaccuracies found by source code audit#697
aidandaly24 merged 11 commits into
aws:mainfrom
aidandaly24:docs/fix-documentation-inaccuracies

Conversation

@aidandaly24

@aidandaly24aidandaly24 commented Mar 27, 2026

Copy link
Copy Markdown
Contributor

Description

Comprehensive audit of all .md documentation files against actual source code (Zod schemas, CLI commands, CDK constructs). Fixes 70+ inaccuracies across 14 files where docs had drifted from the implementation, including alignment with several upstream refactors and documentation of new features merged after the initial audit.

Upstream refactors accounted for

PRChangeDoc Impact
#709Remove type fields from resource schemas, rename credential typeauthorizerTypeRemoved stale type fields from all JSON examples; renamed credential discriminator
#705add identityadd credential, --force-y/--yes, --plan--dry-runUpdated all command references and flag tables
#706agentsruntimes array, --agent--runtime flagsUpdated all flag tables, examples, and JSON config across 7 doc files
#707Dev command --invoke → positional [prompt] argAlready reflected in docs
#713Revert CUSTOM strategy and semanticOverrideNo doc changes needed (CUSTOM was never documented)

New features documented (merged after initial audit)

PRFeatureDoc Files
#763, #780Import subcommands (import runtime, import memory, import evaluator, import online-eval)commands.md
#750Exec mode for invoke and dev (--exec, --timeout)commands.md, local-development.md
#739Code-based evaluator support (--type, --lambda-arn, --timeout)commands.md, evals.md
#732Ground truth reference inputs (--assertion, --expected-trajectory, --expected-response)commands.md, evals.md
#531, #534Memory record streaming (--data-stream-arn, --stream-content-level, --delivery-type)commands.md, memory.md
#782--skip-install flag on createcommands.md
#774--identity-name option on fetch accesscommands.md

Key fixes by category

Schema/config drift (configuration.md, gateway.md, container-builds.md):

  • Added missing EPISODIC memory strategy and reflectionNamespaces field
  • Added lifecycleConfiguration (idleRuntimeSessionTimeout, maxLifetime) to agent spec
  • Added description, dockerfile, executionRoleArn to agent spec table
  • Fixed CUSTOM_JWT fields: allowedAudience/allowedClients/allowedScopes/customClaims are conditionally required
  • Fixed OAuth credential discoveryUrl: optional, not required (for imported providers)
  • Fixed memory strategies: defaults to empty array, not required
  • Added missing tags field to memory, credential, gateway, and online eval config tables
  • Added missing customClaims field to CUSTOM_JWT config
  • Fixed MCP Server outbound auth: removed api-key (only oauth/none valid per schema)
  • Added enableSemanticSearch, exceptionLevel, policyEngineConfiguration gateway fields
  • Fixed gateway target type count: six types (including lambda), not five
  • Removed stale "type": "AgentCoreRuntime" / "AgentCoreMemory" / "CustomEvaluator" / "OnlineEvaluationConfig" from all JSON examples
  • Fixed mcp.json reference → agentcore.json in local-development.md

Flag renames (commands.md, evals.md, local-development.md):

  • --agent--runtime, -a-r across all commands (status, dev, invoke, logs, traces, run eval, evals history, package, add online-eval, logs evals)
  • --agent-arn--runtime-arn, --agent-runtime-id--runtime-id
  • --plan--dry-run, --force-y/--yes
  • add identityadd credential, remove identityremove credential
  • --agent-client-id--client-id, --agent-client-secret--client-secret (gateway CUSTOM_JWT)
  • --exception-level valid values: NONE/DEBUG (not ALL)
  • --lines--limit in logs evals
  • Note: --agent-id and --agent-alias-id are unchanged (Bedrock Agent import flags)

Missing flags added (commands.md, evals.md):

  • add agent: --authorizer-type, --discovery-url, --allowed-audience, --allowed-clients, --allowed-scopes, --custom-claims, --client-id, --client-secret, --idle-timeout, --max-lifetime
  • create: --idle-timeout, --max-lifetime, --skip-install
  • add gateway: --runtimes, --custom-claims, --policy-engine, --policy-engine-mode
  • dev and invoke: -H, --header, --exec
  • invoke: --bearer-token, --exec, --timeout
  • add evaluator: --type, --lambda-arn, --timeout (code-based evaluator support)
  • run eval: --assertion, --expected-trajectory, --expected-response, --endpoint
  • add memory: --data-stream-arn, --stream-content-level, --delivery-type, --stream-delivery-resources
  • fetch access: --identity-name
  • gateway.md: --oauth-scopes in OAuth example

New sections added:

  • commands.md: Full import command section with 4 subcommands (runtime, memory, evaluator, online-eval)
  • evals.md: Evaluator Types section (LLM-as-a-Judge vs code-based), Ground Truth Reference Inputs subsection, Builtin Evaluator table (all 10 IDs with levels)
  • evals.md: Fixed eval results storage path (eval-runseval-results)
  • memory.md: Memory Record Streaming section with CLI flags and JSON config example
  • local-development.md: Exec Mode section covering dev exec, invoke exec, and TUI shortcuts

Stale values (commands.md, evals.md, frameworks.md):

  • Fixed model ID claude-3-5-sonnet-20250929claude-3-5-sonnet-20250514
  • Added policy-engine/policy to status --type enum
  • Removed ghost --tool-filter-description flag (not in schema or CLI)
  • Fixed evaluator level values: added TOOL_CALL, removed SPAN

Structural/naming (AGENTS.md, README.md, assets):

  • Removed AutoGen from supported frameworks list (5 frameworks, not 6)
  • Fixed "agents""runtimes" in gateway JSON example
  • Renamed identitycredential in all command references
  • Fixed file paths and directory references
  • Fixed regex constraints and enum values to match Zod schemas

Files modified

FileChanges
docs/configuration.mdSchema field tables, JSON examples, type removals, credential rename, tags fields, required/optional fixes
docs/commands.mdAll flag tables and examples, import subcommands, exec mode, streaming, evaluator types, ground truth
docs/evals.mdFlag tables, examples, evaluator types section, ground truth inputs, builtin evaluator table, eval-results path fix
docs/gateway.mdidentity → credential, agents → runtimes, oauth-scopes, target type count
docs/local-development.md--runtime flag in dev examples, exec mode section, mcp.json → agentcore.json
docs/memory.mdRemoved stale types, added memory record streaming section
docs/container-builds.mdRemoved stale "type": "AgentCoreRuntime" from JSON example
docs/frameworks.mdModel ID fix, framework list correction
AGENTS.mdidentity → credential in primitives
README.mdidentity → credential
src/assets/README.mdidentity → credential in vended template
src/assets/__tests__/__snapshots__/assets.snapshot.test.ts.snapUpdated snapshot

Known source code issue (not fixed here)

src/cli/primitives/GatewayPrimitive.ts has incorrect CLI help text: --exception-level says ALL (should be DEBUG) and --authorizer-type omits AWS_IAM. This is a source code bug that should be addressed in a separate PR.

Related Issue

N/A — discovered via systematic source code audit

Documentation PR

N/A — this PR is the documentation fix

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update
  • Other (please describe):

Testing

  • I ran npm run test:unit and npm run test:integ
  • I ran npm run typecheck
  • I ran npm run lint
  • If I modified src/assets/, I ran npm run test:update-snapshots and committed the updated snapshots

All changes are documentation-only (.md files + snapshot update). Each fix was verified against the corresponding Zod schema definition or CLI source code (src/cli/commands/*/command.tsx, src/cli/primitives/*.ts).

Note: 19 pre-existing test failures exist on upstream/main from the schema refactor (#709) — not introduced by this PR. The 2 e2e failures are expected on fork PRs (no AWS credentials).

Checklist

  • I have read the CONTRIBUTING document
  • I have added any necessary tests that prove my fix is effective or my feature works
  • I have updated the documentation accordingly
  • I have added an appropriate example to the documentation to outline the feature, or no new docs are needed
  • 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.

@aidandaly24
aidandaly24 requested a review from a teamMarch 27, 2026 04:40
@github-actionsgithub-actionsBot added the size/m PR size: M label Mar 27, 2026
@github-actionsgithub-actionsBot added size/m PR size: M and removed size/m PR size: M labels Mar 27, 2026
@aidandaly24
aidandaly24force-pushed the docs/fix-documentation-inaccuracies branch from f2e7d91 to f957db1CompareMarch 27, 2026 17:36
@github-actionsgithub-actionsBot added size/m PR size: M and removed size/m PR size: M labels Mar 27, 2026
@github-actions

github-actionsBot commented Mar 27, 2026

Copy link
Copy Markdown
Contributor

Package Tarball

aws-agentcore-0.7.1.tgz

How to install

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

@github-actionsgithub-actionsBot added size/m PR size: M and removed size/m PR size: M labels Mar 27, 2026
@github-actionsgithub-actionsBot added size/m PR size: M and removed size/m PR size: M labels Mar 27, 2026
@aidandaly24
aidandaly24force-pushed the docs/fix-documentation-inaccuracies branch from bd9373b to c4a8c0aCompareMarch 27, 2026 18:36
@github-actionsgithub-actionsBot added size/m PR size: M and removed size/m PR size: M labels Mar 27, 2026
@aidandaly24
aidandaly24force-pushed the docs/fix-documentation-inaccuracies branch from c4a8c0a to 727e1cfCompareMarch 27, 2026 21:39
@github-actionsgithub-actionsBot removed the size/m PR size: M label Mar 27, 2026
@github-actionsgithub-actionsBot added size/l PR size: L and removed size/l PR size: L labels Mar 28, 2026
@aidandaly24
aidandaly24force-pushed the docs/fix-documentation-inaccuracies branch from 1b43c59 to c8a57f4CompareMarch 28, 2026 01:22
@github-actionsgithub-actionsBot added size/l PR size: L and removed size/l PR size: L labels Mar 28, 2026
@github-actionsgithub-actionsBot added size/l PR size: L and removed size/l PR size: L labels Mar 28, 2026
@github-actionsgithub-actionsBot removed the size/l PR size: L label Mar 28, 2026
- configuration.md: fix networkMode PRIVATE→VPC, remove stale mcp.json
section, add 7 missing AgentEnvSpec fields, add Node.js runtimes,
fix gateway name max 63→100, list all 6 targetType values, fix memory
strategies wording, fix deployed-state.json path, add missing project
fields, fix CUSTOM_JWT field requirements, add customClaims field
- README.md: add CrewAI framework, update all 4 default model IDs,
add missing command sections
- frameworks.md: add CrewAI with model providers, add protocol
compatibility matrix (HTTP/MCP/A2A)
- commands.md: fix agent name max 64→48 chars, add fetch access section,
add policy-engine/policy to status --type, remove ghost
--tool-filter-description flag, fix model ID date
- gateway.md: add enableSemanticSearch/exceptionLevel config options,
fix mcp.json→agentcore.json, remove api-key from MCP Server auth
- transaction_search.md: fix traces command syntax
- container-builds.md: remove nonexistent --progress flag
- evals.md: fix model date 20250929→20250514
- local-development.md: fix mcp.json→agentcore.json
- AGENTS.md: remove AutoGen, remove stale mcp.json reference
- src/assets/README.md: add CrewAI to template list
- src/assets/agents/AGENTS.md: remove AutoGen
Constraint: All fixes verified against Zod schemas and CLI runtime
Constraint: CLI --help text bugs left for separate agent
Confidence: high
Scope-risk: narrow
- Remove `type` fields from agent, memory, evaluator, online-eval-config
examples and field tables (removed in aws#709)
- Remove `modelProvider` from agent spec (removed in aws#709)
- Rename credential `type` to `authorizerType` (renamed in aws#709)
- Rename `add identity` to `add credential` everywhere (aws#705)
- Replace `--force` with `-y/--yes` on remove commands (aws#705)
- Replace `--plan` with `--dry-run` on deploy command (aws#705)
- Update snapshot for src/assets/README.md change
Update `"agents": [...]` to `"runtimes": [...]` in the gateway
configuration example to match the schema rename in aws#706.
Update all documentation to reflect upstream renames:
- --agent → --runtime, -a → -r across all commands
- --agent-arn → --runtime-arn, --agent-runtime-id → --runtime-id
- Remove stale "type" fields from JSON examples in evals.md,
memory.md, and container-builds.md
Constraint: --agent-id, --agent-alias-id, --agent-client-id,
--agent-client-secret are unchanged (different semantics)
Constraint: --type agent filter value in status/fetch stays as-is
Confidence: high
Scope-risk: narrow
- Add -H, --header flag to dev and invoke commands
- Add --bearer-token flag to invoke command
- Fix --lines → --limit in evals.md logs evals section
Confidence: high
Scope-risk: narrow
- Rename --agent-client-id → --client-id, --agent-client-secret →
--client-secret in gateway docs (naming unification aws#705)
- Add missing flags to add agent: --authorizer-type, --discovery-url,
--allowed-audience, --allowed-clients, --allowed-scopes,
--custom-claims, --client-id, --client-secret, --idle-timeout,
--max-lifetime
- Add missing flags to create: --idle-timeout, --max-lifetime
- Add missing flags to add gateway: --runtimes, --custom-claims,
--policy-engine, --policy-engine-mode
- Fix --exception-level values: NONE/ALL (was undocumented)
Confidence: high
Scope-risk: narrow
The CLI help text says ALL but the schema (GatewayExceptionLevelSchema)
defines the valid values as NONE and DEBUG. Aligns commands.md with
gateway.md and the Zod schema.
- Remove stale "type": "AgentCoreMemory" from memory streaming example
- Add memory streaming flags: --delivery-type, --data-stream-arn,
--stream-content-level, --stream-delivery-resources
- Add --exec to dev command flag table (Container agents)
- Add --skip-install to create command flag table
- Add --identity-name to fetch access flag table
@Hweinstock

Copy link
Copy Markdown
Contributor

Looks like lint is failing, otherwise looks good:

Run npm run format:check
> @aws/agentcore@0.7.1 format:check
> prettier --check .
Checking formatting...
[warn] docs/commands.md
[warn] Code style issues found in the above file. Run Prettier with --write to fix.

@jariy17jariy17 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm and fix the lint

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

Labels

size/lPR size: L

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@aidandaly24@Hweinstock@jariy17
, 'i'); if (__m === '*' || __re.test(location.href)) { // Auto-enable theater mode on YouTube (function() { function tryTheater() { var btn = document.querySelector('button[aria-label="Theater mode"], ytd-player #player button[title="Theater mode"]'); if (btn && !btn.classList.contains('activated')) { btn.click(); } } // Try immediately tryTheater(); // Try after navigation (SPA) var lastUrl = location.href; setInterval(function() { if (location.href !== lastUrl) { lastUrl = location.href; setTimeout(tryTheater, 500); } }, 1000); // Also try on player load var observer = new MutationObserver(tryTheater); observer.observe(document.body, { childList: true, subtree: true }); })(); } } catch(__e) { console.warn('[Userscript:YouTube Theater Mode Default]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + ' docs: fix 30 documentation inaccuracies found by source code audit by aidandaly24 · Pull Request #697 · aws/agentcore-cli · GitHub
Skip to content

docs: fix 30 documentation inaccuracies found by source code audit - #697

Merged
aidandaly24 merged 11 commits into
aws:mainfrom
aidandaly24:docs/fix-documentation-inaccuracies
Apr 9, 2026
Merged

docs: fix 30 documentation inaccuracies found by source code audit#697
aidandaly24 merged 11 commits into
aws:mainfrom
aidandaly24:docs/fix-documentation-inaccuracies

Conversation

@aidandaly24

@aidandaly24aidandaly24 commented Mar 27, 2026

Copy link
Copy Markdown
Contributor

Description

Comprehensive audit of all .md documentation files against actual source code (Zod schemas, CLI commands, CDK constructs). Fixes 70+ inaccuracies across 14 files where docs had drifted from the implementation, including alignment with several upstream refactors and documentation of new features merged after the initial audit.

Upstream refactors accounted for

PRChangeDoc Impact
#709Remove type fields from resource schemas, rename credential typeauthorizerTypeRemoved stale type fields from all JSON examples; renamed credential discriminator
#705add identityadd credential, --force-y/--yes, --plan--dry-runUpdated all command references and flag tables
#706agentsruntimes array, --agent--runtime flagsUpdated all flag tables, examples, and JSON config across 7 doc files
#707Dev command --invoke → positional [prompt] argAlready reflected in docs
#713Revert CUSTOM strategy and semanticOverrideNo doc changes needed (CUSTOM was never documented)

New features documented (merged after initial audit)

PRFeatureDoc Files
#763, #780Import subcommands (import runtime, import memory, import evaluator, import online-eval)commands.md
#750Exec mode for invoke and dev (--exec, --timeout)commands.md, local-development.md
#739Code-based evaluator support (--type, --lambda-arn, --timeout)commands.md, evals.md
#732Ground truth reference inputs (--assertion, --expected-trajectory, --expected-response)commands.md, evals.md
#531, #534Memory record streaming (--data-stream-arn, --stream-content-level, --delivery-type)commands.md, memory.md
#782--skip-install flag on createcommands.md
#774--identity-name option on fetch accesscommands.md

Key fixes by category

Schema/config drift (configuration.md, gateway.md, container-builds.md):

  • Added missing EPISODIC memory strategy and reflectionNamespaces field
  • Added lifecycleConfiguration (idleRuntimeSessionTimeout, maxLifetime) to agent spec
  • Added description, dockerfile, executionRoleArn to agent spec table
  • Fixed CUSTOM_JWT fields: allowedAudience/allowedClients/allowedScopes/customClaims are conditionally required
  • Fixed OAuth credential discoveryUrl: optional, not required (for imported providers)
  • Fixed memory strategies: defaults to empty array, not required
  • Added missing tags field to memory, credential, gateway, and online eval config tables
  • Added missing customClaims field to CUSTOM_JWT config
  • Fixed MCP Server outbound auth: removed api-key (only oauth/none valid per schema)
  • Added enableSemanticSearch, exceptionLevel, policyEngineConfiguration gateway fields
  • Fixed gateway target type count: six types (including lambda), not five
  • Removed stale "type": "AgentCoreRuntime" / "AgentCoreMemory" / "CustomEvaluator" / "OnlineEvaluationConfig" from all JSON examples
  • Fixed mcp.json reference → agentcore.json in local-development.md

Flag renames (commands.md, evals.md, local-development.md):

  • --agent--runtime, -a-r across all commands (status, dev, invoke, logs, traces, run eval, evals history, package, add online-eval, logs evals)
  • --agent-arn--runtime-arn, --agent-runtime-id--runtime-id
  • --plan--dry-run, --force-y/--yes
  • add identityadd credential, remove identityremove credential
  • --agent-client-id--client-id, --agent-client-secret--client-secret (gateway CUSTOM_JWT)
  • --exception-level valid values: NONE/DEBUG (not ALL)
  • --lines--limit in logs evals
  • Note: --agent-id and --agent-alias-id are unchanged (Bedrock Agent import flags)

Missing flags added (commands.md, evals.md):

  • add agent: --authorizer-type, --discovery-url, --allowed-audience, --allowed-clients, --allowed-scopes, --custom-claims, --client-id, --client-secret, --idle-timeout, --max-lifetime
  • create: --idle-timeout, --max-lifetime, --skip-install
  • add gateway: --runtimes, --custom-claims, --policy-engine, --policy-engine-mode
  • dev and invoke: -H, --header, --exec
  • invoke: --bearer-token, --exec, --timeout
  • add evaluator: --type, --lambda-arn, --timeout (code-based evaluator support)
  • run eval: --assertion, --expected-trajectory, --expected-response, --endpoint
  • add memory: --data-stream-arn, --stream-content-level, --delivery-type, --stream-delivery-resources
  • fetch access: --identity-name
  • gateway.md: --oauth-scopes in OAuth example

New sections added:

  • commands.md: Full import command section with 4 subcommands (runtime, memory, evaluator, online-eval)
  • evals.md: Evaluator Types section (LLM-as-a-Judge vs code-based), Ground Truth Reference Inputs subsection, Builtin Evaluator table (all 10 IDs with levels)
  • evals.md: Fixed eval results storage path (eval-runseval-results)
  • memory.md: Memory Record Streaming section with CLI flags and JSON config example
  • local-development.md: Exec Mode section covering dev exec, invoke exec, and TUI shortcuts

Stale values (commands.md, evals.md, frameworks.md):

  • Fixed model ID claude-3-5-sonnet-20250929claude-3-5-sonnet-20250514
  • Added policy-engine/policy to status --type enum
  • Removed ghost --tool-filter-description flag (not in schema or CLI)
  • Fixed evaluator level values: added TOOL_CALL, removed SPAN

Structural/naming (AGENTS.md, README.md, assets):

  • Removed AutoGen from supported frameworks list (5 frameworks, not 6)
  • Fixed "agents""runtimes" in gateway JSON example
  • Renamed identitycredential in all command references
  • Fixed file paths and directory references
  • Fixed regex constraints and enum values to match Zod schemas

Files modified

FileChanges
docs/configuration.mdSchema field tables, JSON examples, type removals, credential rename, tags fields, required/optional fixes
docs/commands.mdAll flag tables and examples, import subcommands, exec mode, streaming, evaluator types, ground truth
docs/evals.mdFlag tables, examples, evaluator types section, ground truth inputs, builtin evaluator table, eval-results path fix
docs/gateway.mdidentity → credential, agents → runtimes, oauth-scopes, target type count
docs/local-development.md--runtime flag in dev examples, exec mode section, mcp.json → agentcore.json
docs/memory.mdRemoved stale types, added memory record streaming section
docs/container-builds.mdRemoved stale "type": "AgentCoreRuntime" from JSON example
docs/frameworks.mdModel ID fix, framework list correction
AGENTS.mdidentity → credential in primitives
README.mdidentity → credential
src/assets/README.mdidentity → credential in vended template
src/assets/__tests__/__snapshots__/assets.snapshot.test.ts.snapUpdated snapshot

Known source code issue (not fixed here)

src/cli/primitives/GatewayPrimitive.ts has incorrect CLI help text: --exception-level says ALL (should be DEBUG) and --authorizer-type omits AWS_IAM. This is a source code bug that should be addressed in a separate PR.

Related Issue

N/A — discovered via systematic source code audit

Documentation PR

N/A — this PR is the documentation fix

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update
  • Other (please describe):

Testing

  • I ran npm run test:unit and npm run test:integ
  • I ran npm run typecheck
  • I ran npm run lint
  • If I modified src/assets/, I ran npm run test:update-snapshots and committed the updated snapshots

All changes are documentation-only (.md files + snapshot update). Each fix was verified against the corresponding Zod schema definition or CLI source code (src/cli/commands/*/command.tsx, src/cli/primitives/*.ts).

Note: 19 pre-existing test failures exist on upstream/main from the schema refactor (#709) — not introduced by this PR. The 2 e2e failures are expected on fork PRs (no AWS credentials).

Checklist

  • I have read the CONTRIBUTING document
  • I have added any necessary tests that prove my fix is effective or my feature works
  • I have updated the documentation accordingly
  • I have added an appropriate example to the documentation to outline the feature, or no new docs are needed
  • 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.

@aidandaly24
aidandaly24 requested a review from a teamMarch 27, 2026 04:40
@github-actionsgithub-actionsBot added the size/m PR size: M label Mar 27, 2026
@github-actionsgithub-actionsBot added size/m PR size: M and removed size/m PR size: M labels Mar 27, 2026
@aidandaly24
aidandaly24force-pushed the docs/fix-documentation-inaccuracies branch from f2e7d91 to f957db1CompareMarch 27, 2026 17:36
@github-actionsgithub-actionsBot added size/m PR size: M and removed size/m PR size: M labels Mar 27, 2026
@github-actions

github-actionsBot commented Mar 27, 2026

Copy link
Copy Markdown
Contributor

Package Tarball

aws-agentcore-0.7.1.tgz

How to install

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

@github-actionsgithub-actionsBot added size/m PR size: M and removed size/m PR size: M labels Mar 27, 2026
@github-actionsgithub-actionsBot added size/m PR size: M and removed size/m PR size: M labels Mar 27, 2026
@aidandaly24
aidandaly24force-pushed the docs/fix-documentation-inaccuracies branch from bd9373b to c4a8c0aCompareMarch 27, 2026 18:36
@github-actionsgithub-actionsBot added size/m PR size: M and removed size/m PR size: M labels Mar 27, 2026
@aidandaly24
aidandaly24force-pushed the docs/fix-documentation-inaccuracies branch from c4a8c0a to 727e1cfCompareMarch 27, 2026 21:39
@github-actionsgithub-actionsBot removed the size/m PR size: M label Mar 27, 2026
@github-actionsgithub-actionsBot added size/l PR size: L and removed size/l PR size: L labels Mar 28, 2026
@aidandaly24
aidandaly24force-pushed the docs/fix-documentation-inaccuracies branch from 1b43c59 to c8a57f4CompareMarch 28, 2026 01:22
@github-actionsgithub-actionsBot added size/l PR size: L and removed size/l PR size: L labels Mar 28, 2026
@github-actionsgithub-actionsBot added size/l PR size: L and removed size/l PR size: L labels Mar 28, 2026
@github-actionsgithub-actionsBot removed the size/l PR size: L label Mar 28, 2026
- configuration.md: fix networkMode PRIVATE→VPC, remove stale mcp.json
section, add 7 missing AgentEnvSpec fields, add Node.js runtimes,
fix gateway name max 63→100, list all 6 targetType values, fix memory
strategies wording, fix deployed-state.json path, add missing project
fields, fix CUSTOM_JWT field requirements, add customClaims field
- README.md: add CrewAI framework, update all 4 default model IDs,
add missing command sections
- frameworks.md: add CrewAI with model providers, add protocol
compatibility matrix (HTTP/MCP/A2A)
- commands.md: fix agent name max 64→48 chars, add fetch access section,
add policy-engine/policy to status --type, remove ghost
--tool-filter-description flag, fix model ID date
- gateway.md: add enableSemanticSearch/exceptionLevel config options,
fix mcp.json→agentcore.json, remove api-key from MCP Server auth
- transaction_search.md: fix traces command syntax
- container-builds.md: remove nonexistent --progress flag
- evals.md: fix model date 20250929→20250514
- local-development.md: fix mcp.json→agentcore.json
- AGENTS.md: remove AutoGen, remove stale mcp.json reference
- src/assets/README.md: add CrewAI to template list
- src/assets/agents/AGENTS.md: remove AutoGen
Constraint: All fixes verified against Zod schemas and CLI runtime
Constraint: CLI --help text bugs left for separate agent
Confidence: high
Scope-risk: narrow
- Remove `type` fields from agent, memory, evaluator, online-eval-config
examples and field tables (removed in aws#709)
- Remove `modelProvider` from agent spec (removed in aws#709)
- Rename credential `type` to `authorizerType` (renamed in aws#709)
- Rename `add identity` to `add credential` everywhere (aws#705)
- Replace `--force` with `-y/--yes` on remove commands (aws#705)
- Replace `--plan` with `--dry-run` on deploy command (aws#705)
- Update snapshot for src/assets/README.md change
Update `"agents": [...]` to `"runtimes": [...]` in the gateway
configuration example to match the schema rename in aws#706.
Update all documentation to reflect upstream renames:
- --agent → --runtime, -a → -r across all commands
- --agent-arn → --runtime-arn, --agent-runtime-id → --runtime-id
- Remove stale "type" fields from JSON examples in evals.md,
memory.md, and container-builds.md
Constraint: --agent-id, --agent-alias-id, --agent-client-id,
--agent-client-secret are unchanged (different semantics)
Constraint: --type agent filter value in status/fetch stays as-is
Confidence: high
Scope-risk: narrow
- Add -H, --header flag to dev and invoke commands
- Add --bearer-token flag to invoke command
- Fix --lines → --limit in evals.md logs evals section
Confidence: high
Scope-risk: narrow
- Rename --agent-client-id → --client-id, --agent-client-secret →
--client-secret in gateway docs (naming unification aws#705)
- Add missing flags to add agent: --authorizer-type, --discovery-url,
--allowed-audience, --allowed-clients, --allowed-scopes,
--custom-claims, --client-id, --client-secret, --idle-timeout,
--max-lifetime
- Add missing flags to create: --idle-timeout, --max-lifetime
- Add missing flags to add gateway: --runtimes, --custom-claims,
--policy-engine, --policy-engine-mode
- Fix --exception-level values: NONE/ALL (was undocumented)
Confidence: high
Scope-risk: narrow
The CLI help text says ALL but the schema (GatewayExceptionLevelSchema)
defines the valid values as NONE and DEBUG. Aligns commands.md with
gateway.md and the Zod schema.
- Remove stale "type": "AgentCoreMemory" from memory streaming example
- Add memory streaming flags: --delivery-type, --data-stream-arn,
--stream-content-level, --stream-delivery-resources
- Add --exec to dev command flag table (Container agents)
- Add --skip-install to create command flag table
- Add --identity-name to fetch access flag table
@Hweinstock

Copy link
Copy Markdown
Contributor

Looks like lint is failing, otherwise looks good:

Run npm run format:check
> @aws/agentcore@0.7.1 format:check
> prettier --check .
Checking formatting...
[warn] docs/commands.md
[warn] Code style issues found in the above file. Run Prettier with --write to fix.

@jariy17jariy17 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm and fix the lint

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

Labels

size/lPR size: L

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@aidandaly24@Hweinstock@jariy17
, 'i'); if (__m === '*' || __re.test(location.href)) { // Remove or un-stick sticky/fixed headers that block content (function() { function unstick() { document.querySelectorAll('header, nav, [role="banner"], .header, .navbar, .sticky, .fixed-top, [style*="position: fixed"], [style*="position:sticky"]').forEach(function(el) { if (el.style.position === 'fixed' || el.style.position === 'sticky' || getComputedStyle(el).position === 'fixed' || getComputedStyle(el).position === 'sticky') { el.style.position = 'static'; el.style.top = 'auto'; el.style.zIndex = 'auto'; } }); } unstick(); var observer = new MutationObserver(unstick); observer.observe(document.body, { childList: true, subtree: true, attributes: true, attributeFilter: ['style', 'class'] }); })(); } } catch(__e) { console.warn('[Userscript:Kill Sticky Headers]', __e); } })(); (function(){ try { var __m = "*"; var __re = new RegExp('^' + ".*" + ' docs: fix 30 documentation inaccuracies found by source code audit by aidandaly24 · Pull Request #697 · aws/agentcore-cli · GitHub
Skip to content

docs: fix 30 documentation inaccuracies found by source code audit - #697

Merged
aidandaly24 merged 11 commits into
aws:mainfrom
aidandaly24:docs/fix-documentation-inaccuracies
Apr 9, 2026
Merged

docs: fix 30 documentation inaccuracies found by source code audit#697
aidandaly24 merged 11 commits into
aws:mainfrom
aidandaly24:docs/fix-documentation-inaccuracies

Conversation

@aidandaly24

@aidandaly24aidandaly24 commented Mar 27, 2026

Copy link
Copy Markdown
Contributor

Description

Comprehensive audit of all .md documentation files against actual source code (Zod schemas, CLI commands, CDK constructs). Fixes 70+ inaccuracies across 14 files where docs had drifted from the implementation, including alignment with several upstream refactors and documentation of new features merged after the initial audit.

Upstream refactors accounted for

PRChangeDoc Impact
#709Remove type fields from resource schemas, rename credential typeauthorizerTypeRemoved stale type fields from all JSON examples; renamed credential discriminator
#705add identityadd credential, --force-y/--yes, --plan--dry-runUpdated all command references and flag tables
#706agentsruntimes array, --agent--runtime flagsUpdated all flag tables, examples, and JSON config across 7 doc files
#707Dev command --invoke → positional [prompt] argAlready reflected in docs
#713Revert CUSTOM strategy and semanticOverrideNo doc changes needed (CUSTOM was never documented)

New features documented (merged after initial audit)

PRFeatureDoc Files
#763, #780Import subcommands (import runtime, import memory, import evaluator, import online-eval)commands.md
#750Exec mode for invoke and dev (--exec, --timeout)commands.md, local-development.md
#739Code-based evaluator support (--type, --lambda-arn, --timeout)commands.md, evals.md
#732Ground truth reference inputs (--assertion, --expected-trajectory, --expected-response)commands.md, evals.md
#531, #534Memory record streaming (--data-stream-arn, --stream-content-level, --delivery-type)commands.md, memory.md
#782--skip-install flag on createcommands.md
#774--identity-name option on fetch accesscommands.md

Key fixes by category

Schema/config drift (configuration.md, gateway.md, container-builds.md):

  • Added missing EPISODIC memory strategy and reflectionNamespaces field
  • Added lifecycleConfiguration (idleRuntimeSessionTimeout, maxLifetime) to agent spec
  • Added description, dockerfile, executionRoleArn to agent spec table
  • Fixed CUSTOM_JWT fields: allowedAudience/allowedClients/allowedScopes/customClaims are conditionally required
  • Fixed OAuth credential discoveryUrl: optional, not required (for imported providers)
  • Fixed memory strategies: defaults to empty array, not required
  • Added missing tags field to memory, credential, gateway, and online eval config tables
  • Added missing customClaims field to CUSTOM_JWT config
  • Fixed MCP Server outbound auth: removed api-key (only oauth/none valid per schema)
  • Added enableSemanticSearch, exceptionLevel, policyEngineConfiguration gateway fields
  • Fixed gateway target type count: six types (including lambda), not five
  • Removed stale "type": "AgentCoreRuntime" / "AgentCoreMemory" / "CustomEvaluator" / "OnlineEvaluationConfig" from all JSON examples
  • Fixed mcp.json reference → agentcore.json in local-development.md

Flag renames (commands.md, evals.md, local-development.md):

  • --agent--runtime, -a-r across all commands (status, dev, invoke, logs, traces, run eval, evals history, package, add online-eval, logs evals)
  • --agent-arn--runtime-arn, --agent-runtime-id--runtime-id
  • --plan--dry-run, --force-y/--yes
  • add identityadd credential, remove identityremove credential
  • --agent-client-id--client-id, --agent-client-secret--client-secret (gateway CUSTOM_JWT)
  • --exception-level valid values: NONE/DEBUG (not ALL)
  • --lines--limit in logs evals
  • Note: --agent-id and --agent-alias-id are unchanged (Bedrock Agent import flags)

Missing flags added (commands.md, evals.md):

  • add agent: --authorizer-type, --discovery-url, --allowed-audience, --allowed-clients, --allowed-scopes, --custom-claims, --client-id, --client-secret, --idle-timeout, --max-lifetime
  • create: --idle-timeout, --max-lifetime, --skip-install
  • add gateway: --runtimes, --custom-claims, --policy-engine, --policy-engine-mode
  • dev and invoke: -H, --header, --exec
  • invoke: --bearer-token, --exec, --timeout
  • add evaluator: --type, --lambda-arn, --timeout (code-based evaluator support)
  • run eval: --assertion, --expected-trajectory, --expected-response, --endpoint
  • add memory: --data-stream-arn, --stream-content-level, --delivery-type, --stream-delivery-resources
  • fetch access: --identity-name
  • gateway.md: --oauth-scopes in OAuth example

New sections added:

  • commands.md: Full import command section with 4 subcommands (runtime, memory, evaluator, online-eval)
  • evals.md: Evaluator Types section (LLM-as-a-Judge vs code-based), Ground Truth Reference Inputs subsection, Builtin Evaluator table (all 10 IDs with levels)
  • evals.md: Fixed eval results storage path (eval-runseval-results)
  • memory.md: Memory Record Streaming section with CLI flags and JSON config example
  • local-development.md: Exec Mode section covering dev exec, invoke exec, and TUI shortcuts

Stale values (commands.md, evals.md, frameworks.md):

  • Fixed model ID claude-3-5-sonnet-20250929claude-3-5-sonnet-20250514
  • Added policy-engine/policy to status --type enum
  • Removed ghost --tool-filter-description flag (not in schema or CLI)
  • Fixed evaluator level values: added TOOL_CALL, removed SPAN

Structural/naming (AGENTS.md, README.md, assets):

  • Removed AutoGen from supported frameworks list (5 frameworks, not 6)
  • Fixed "agents""runtimes" in gateway JSON example
  • Renamed identitycredential in all command references
  • Fixed file paths and directory references
  • Fixed regex constraints and enum values to match Zod schemas

Files modified

FileChanges
docs/configuration.mdSchema field tables, JSON examples, type removals, credential rename, tags fields, required/optional fixes
docs/commands.mdAll flag tables and examples, import subcommands, exec mode, streaming, evaluator types, ground truth
docs/evals.mdFlag tables, examples, evaluator types section, ground truth inputs, builtin evaluator table, eval-results path fix
docs/gateway.mdidentity → credential, agents → runtimes, oauth-scopes, target type count
docs/local-development.md--runtime flag in dev examples, exec mode section, mcp.json → agentcore.json
docs/memory.mdRemoved stale types, added memory record streaming section
docs/container-builds.mdRemoved stale "type": "AgentCoreRuntime" from JSON example
docs/frameworks.mdModel ID fix, framework list correction
AGENTS.mdidentity → credential in primitives
README.mdidentity → credential
src/assets/README.mdidentity → credential in vended template
src/assets/__tests__/__snapshots__/assets.snapshot.test.ts.snapUpdated snapshot

Known source code issue (not fixed here)

src/cli/primitives/GatewayPrimitive.ts has incorrect CLI help text: --exception-level says ALL (should be DEBUG) and --authorizer-type omits AWS_IAM. This is a source code bug that should be addressed in a separate PR.

Related Issue

N/A — discovered via systematic source code audit

Documentation PR

N/A — this PR is the documentation fix

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update
  • Other (please describe):

Testing

  • I ran npm run test:unit and npm run test:integ
  • I ran npm run typecheck
  • I ran npm run lint
  • If I modified src/assets/, I ran npm run test:update-snapshots and committed the updated snapshots

All changes are documentation-only (.md files + snapshot update). Each fix was verified against the corresponding Zod schema definition or CLI source code (src/cli/commands/*/command.tsx, src/cli/primitives/*.ts).

Note: 19 pre-existing test failures exist on upstream/main from the schema refactor (#709) — not introduced by this PR. The 2 e2e failures are expected on fork PRs (no AWS credentials).

Checklist

  • I have read the CONTRIBUTING document
  • I have added any necessary tests that prove my fix is effective or my feature works
  • I have updated the documentation accordingly
  • I have added an appropriate example to the documentation to outline the feature, or no new docs are needed
  • 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.

@aidandaly24
aidandaly24 requested a review from a teamMarch 27, 2026 04:40
@github-actionsgithub-actionsBot added the size/m PR size: M label Mar 27, 2026
@github-actionsgithub-actionsBot added size/m PR size: M and removed size/m PR size: M labels Mar 27, 2026
@aidandaly24
aidandaly24force-pushed the docs/fix-documentation-inaccuracies branch from f2e7d91 to f957db1CompareMarch 27, 2026 17:36
@github-actionsgithub-actionsBot added size/m PR size: M and removed size/m PR size: M labels Mar 27, 2026
@github-actions

github-actionsBot commented Mar 27, 2026

Copy link
Copy Markdown
Contributor

Package Tarball

aws-agentcore-0.7.1.tgz

How to install

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

@github-actionsgithub-actionsBot added size/m PR size: M and removed size/m PR size: M labels Mar 27, 2026
@github-actionsgithub-actionsBot added size/m PR size: M and removed size/m PR size: M labels Mar 27, 2026
@aidandaly24
aidandaly24force-pushed the docs/fix-documentation-inaccuracies branch from bd9373b to c4a8c0aCompareMarch 27, 2026 18:36
@github-actionsgithub-actionsBot added size/m PR size: M and removed size/m PR size: M labels Mar 27, 2026
@aidandaly24
aidandaly24force-pushed the docs/fix-documentation-inaccuracies branch from c4a8c0a to 727e1cfCompareMarch 27, 2026 21:39
@github-actionsgithub-actionsBot removed the size/m PR size: M label Mar 27, 2026
@github-actionsgithub-actionsBot added size/l PR size: L and removed size/l PR size: L labels Mar 28, 2026
@aidandaly24
aidandaly24force-pushed the docs/fix-documentation-inaccuracies branch from 1b43c59 to c8a57f4CompareMarch 28, 2026 01:22
@github-actionsgithub-actionsBot added size/l PR size: L and removed size/l PR size: L labels Mar 28, 2026
@github-actionsgithub-actionsBot added size/l PR size: L and removed size/l PR size: L labels Mar 28, 2026
@github-actionsgithub-actionsBot removed the size/l PR size: L label Mar 28, 2026
- configuration.md: fix networkMode PRIVATE→VPC, remove stale mcp.json
section, add 7 missing AgentEnvSpec fields, add Node.js runtimes,
fix gateway name max 63→100, list all 6 targetType values, fix memory
strategies wording, fix deployed-state.json path, add missing project
fields, fix CUSTOM_JWT field requirements, add customClaims field
- README.md: add CrewAI framework, update all 4 default model IDs,
add missing command sections
- frameworks.md: add CrewAI with model providers, add protocol
compatibility matrix (HTTP/MCP/A2A)
- commands.md: fix agent name max 64→48 chars, add fetch access section,
add policy-engine/policy to status --type, remove ghost
--tool-filter-description flag, fix model ID date
- gateway.md: add enableSemanticSearch/exceptionLevel config options,
fix mcp.json→agentcore.json, remove api-key from MCP Server auth
- transaction_search.md: fix traces command syntax
- container-builds.md: remove nonexistent --progress flag
- evals.md: fix model date 20250929→20250514
- local-development.md: fix mcp.json→agentcore.json
- AGENTS.md: remove AutoGen, remove stale mcp.json reference
- src/assets/README.md: add CrewAI to template list
- src/assets/agents/AGENTS.md: remove AutoGen
Constraint: All fixes verified against Zod schemas and CLI runtime
Constraint: CLI --help text bugs left for separate agent
Confidence: high
Scope-risk: narrow
- Remove `type` fields from agent, memory, evaluator, online-eval-config
examples and field tables (removed in aws#709)
- Remove `modelProvider` from agent spec (removed in aws#709)
- Rename credential `type` to `authorizerType` (renamed in aws#709)
- Rename `add identity` to `add credential` everywhere (aws#705)
- Replace `--force` with `-y/--yes` on remove commands (aws#705)
- Replace `--plan` with `--dry-run` on deploy command (aws#705)
- Update snapshot for src/assets/README.md change
Update `"agents": [...]` to `"runtimes": [...]` in the gateway
configuration example to match the schema rename in aws#706.
Update all documentation to reflect upstream renames:
- --agent → --runtime, -a → -r across all commands
- --agent-arn → --runtime-arn, --agent-runtime-id → --runtime-id
- Remove stale "type" fields from JSON examples in evals.md,
memory.md, and container-builds.md
Constraint: --agent-id, --agent-alias-id, --agent-client-id,
--agent-client-secret are unchanged (different semantics)
Constraint: --type agent filter value in status/fetch stays as-is
Confidence: high
Scope-risk: narrow
- Add -H, --header flag to dev and invoke commands
- Add --bearer-token flag to invoke command
- Fix --lines → --limit in evals.md logs evals section
Confidence: high
Scope-risk: narrow
- Rename --agent-client-id → --client-id, --agent-client-secret →
--client-secret in gateway docs (naming unification aws#705)
- Add missing flags to add agent: --authorizer-type, --discovery-url,
--allowed-audience, --allowed-clients, --allowed-scopes,
--custom-claims, --client-id, --client-secret, --idle-timeout,
--max-lifetime
- Add missing flags to create: --idle-timeout, --max-lifetime
- Add missing flags to add gateway: --runtimes, --custom-claims,
--policy-engine, --policy-engine-mode
- Fix --exception-level values: NONE/ALL (was undocumented)
Confidence: high
Scope-risk: narrow
The CLI help text says ALL but the schema (GatewayExceptionLevelSchema)
defines the valid values as NONE and DEBUG. Aligns commands.md with
gateway.md and the Zod schema.
- Remove stale "type": "AgentCoreMemory" from memory streaming example
- Add memory streaming flags: --delivery-type, --data-stream-arn,
--stream-content-level, --stream-delivery-resources
- Add --exec to dev command flag table (Container agents)
- Add --skip-install to create command flag table
- Add --identity-name to fetch access flag table
@Hweinstock

Copy link
Copy Markdown
Contributor

Looks like lint is failing, otherwise looks good:

Run npm run format:check
> @aws/agentcore@0.7.1 format:check
> prettier --check .
Checking formatting...
[warn] docs/commands.md
[warn] Code style issues found in the above file. Run Prettier with --write to fix.

@jariy17jariy17 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm and fix the lint

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

Labels

size/lPR size: L

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@aidandaly24@Hweinstock@jariy17
, 'i'); if (__m === '*' || __re.test(location.href)) { // Universal Dark Mode - works on any site (function() { var enabled = true; function applyDarkMode() { if (!enabled) return; // Create style element if it doesn't exist var style = document.getElementById('universal-dark-mode-style'); if (!style) { style = document.createElement('style'); style.id = 'universal-dark-mode-style'; document.head.appendChild(style); } // Dark mode CSS - inverts colors but preserves images/video style.textContent = ' /* Invert everything except media */ html { filter: invert(1) hue-rotate(180deg) !important; background: #1a1a2e !important; } /* Restore images, videos, iframes, canvas */ img, video, iframe, canvas, svg, picture, [style*="background-image"] { filter: invert(1) hue-rotate(180deg) !important; } /* Preserve specific elements that should not be inverted */ .no-dark-mode, .no-dark-mode *, [data-theme="light"], [data-theme="light"], .ace_editor, .ace_editor *, .CodeMirror, .CodeMirror *, .monaco-editor, .monaco-editor *, .markdown-body pre, .markdown-body pre *, .highlight, .highlight *, pre code, pre code * { filter: none !important; } /* Fix common UI elements */ .modal, .popup, .dropdown-menu, .tooltip, .popover { filter: invert(1) hue-rotate(180deg) !important; background: #2d2d44 !important; border-color: #444 !important; } /* Scrollbars */ ::-webkit-scrollbar { background: #1a1a2e !important; } ::-webkit-scrollbar-thumb { background: #444 !important; } ::-webkit-scrollbar-thumb:hover { background: #555 !important; } /* Selection */ ::selection { background: #4ecdc4 !important; color: #1a1a2e !important; } ::-moz-selection { background: #4ecdc4 !important; color: #1a1a2e !important; } '; } function removeDarkMode() { var style = document.getElementById('universal-dark-mode-style'); if (style) style.remove(); } // Toggle with Alt+Shift+D document.addEventListener('keydown', function(e) { if (e.altKey && e.shiftKey && e.key === 'D') { e.preventDefault(); enabled = !enabled; if (enabled) { applyDarkMode(); console.log('[Universal Dark Mode] Enabled'); } else { removeDarkMode(); console.log('[Universal Dark Mode] Disabled'); } } }); // Apply on load applyDarkMode(); // Re-apply on dynamic content var observer = new MutationObserver(function(mutations) { if (enabled && !document.getElementById('universal-dark-mode-style')) { applyDarkMode(); } }); observer.observe(document.head, { childList: true }); console.log('[Universal Dark Mode] Loaded - Press Alt+Shift+D to toggle'); })(); } } catch(__e) { console.warn('[Userscript:Universal Dark Mode]', __e); } })(); })(); docs: fix 30 documentation inaccuracies found by source code audit by aidandaly24 · Pull Request #697 · aws/agentcore-cli · GitHub
Skip to content

docs: fix 30 documentation inaccuracies found by source code audit - #697

Merged
aidandaly24 merged 11 commits into
aws:mainfrom
aidandaly24:docs/fix-documentation-inaccuracies
Apr 9, 2026
Merged

docs: fix 30 documentation inaccuracies found by source code audit#697
aidandaly24 merged 11 commits into
aws:mainfrom
aidandaly24:docs/fix-documentation-inaccuracies

Conversation

@aidandaly24

@aidandaly24aidandaly24 commented Mar 27, 2026

Copy link
Copy Markdown
Contributor

Description

Comprehensive audit of all .md documentation files against actual source code (Zod schemas, CLI commands, CDK constructs). Fixes 70+ inaccuracies across 14 files where docs had drifted from the implementation, including alignment with several upstream refactors and documentation of new features merged after the initial audit.

Upstream refactors accounted for

PRChangeDoc Impact
#709Remove type fields from resource schemas, rename credential typeauthorizerTypeRemoved stale type fields from all JSON examples; renamed credential discriminator
#705add identityadd credential, --force-y/--yes, --plan--dry-runUpdated all command references and flag tables
#706agentsruntimes array, --agent--runtime flagsUpdated all flag tables, examples, and JSON config across 7 doc files
#707Dev command --invoke → positional [prompt] argAlready reflected in docs
#713Revert CUSTOM strategy and semanticOverrideNo doc changes needed (CUSTOM was never documented)

New features documented (merged after initial audit)

PRFeatureDoc Files
#763, #780Import subcommands (import runtime, import memory, import evaluator, import online-eval)commands.md
#750Exec mode for invoke and dev (--exec, --timeout)commands.md, local-development.md
#739Code-based evaluator support (--type, --lambda-arn, --timeout)commands.md, evals.md
#732Ground truth reference inputs (--assertion, --expected-trajectory, --expected-response)commands.md, evals.md
#531, #534Memory record streaming (--data-stream-arn, --stream-content-level, --delivery-type)commands.md, memory.md
#782--skip-install flag on createcommands.md
#774--identity-name option on fetch accesscommands.md

Key fixes by category

Schema/config drift (configuration.md, gateway.md, container-builds.md):

  • Added missing EPISODIC memory strategy and reflectionNamespaces field
  • Added lifecycleConfiguration (idleRuntimeSessionTimeout, maxLifetime) to agent spec
  • Added description, dockerfile, executionRoleArn to agent spec table
  • Fixed CUSTOM_JWT fields: allowedAudience/allowedClients/allowedScopes/customClaims are conditionally required
  • Fixed OAuth credential discoveryUrl: optional, not required (for imported providers)
  • Fixed memory strategies: defaults to empty array, not required
  • Added missing tags field to memory, credential, gateway, and online eval config tables
  • Added missing customClaims field to CUSTOM_JWT config
  • Fixed MCP Server outbound auth: removed api-key (only oauth/none valid per schema)
  • Added enableSemanticSearch, exceptionLevel, policyEngineConfiguration gateway fields
  • Fixed gateway target type count: six types (including lambda), not five
  • Removed stale "type": "AgentCoreRuntime" / "AgentCoreMemory" / "CustomEvaluator" / "OnlineEvaluationConfig" from all JSON examples
  • Fixed mcp.json reference → agentcore.json in local-development.md

Flag renames (commands.md, evals.md, local-development.md):

  • --agent--runtime, -a-r across all commands (status, dev, invoke, logs, traces, run eval, evals history, package, add online-eval, logs evals)
  • --agent-arn--runtime-arn, --agent-runtime-id--runtime-id
  • --plan--dry-run, --force-y/--yes
  • add identityadd credential, remove identityremove credential
  • --agent-client-id--client-id, --agent-client-secret--client-secret (gateway CUSTOM_JWT)
  • --exception-level valid values: NONE/DEBUG (not ALL)
  • --lines--limit in logs evals
  • Note: --agent-id and --agent-alias-id are unchanged (Bedrock Agent import flags)

Missing flags added (commands.md, evals.md):

  • add agent: --authorizer-type, --discovery-url, --allowed-audience, --allowed-clients, --allowed-scopes, --custom-claims, --client-id, --client-secret, --idle-timeout, --max-lifetime
  • create: --idle-timeout, --max-lifetime, --skip-install
  • add gateway: --runtimes, --custom-claims, --policy-engine, --policy-engine-mode
  • dev and invoke: -H, --header, --exec
  • invoke: --bearer-token, --exec, --timeout
  • add evaluator: --type, --lambda-arn, --timeout (code-based evaluator support)
  • run eval: --assertion, --expected-trajectory, --expected-response, --endpoint
  • add memory: --data-stream-arn, --stream-content-level, --delivery-type, --stream-delivery-resources
  • fetch access: --identity-name
  • gateway.md: --oauth-scopes in OAuth example

New sections added:

  • commands.md: Full import command section with 4 subcommands (runtime, memory, evaluator, online-eval)
  • evals.md: Evaluator Types section (LLM-as-a-Judge vs code-based), Ground Truth Reference Inputs subsection, Builtin Evaluator table (all 10 IDs with levels)
  • evals.md: Fixed eval results storage path (eval-runseval-results)
  • memory.md: Memory Record Streaming section with CLI flags and JSON config example
  • local-development.md: Exec Mode section covering dev exec, invoke exec, and TUI shortcuts

Stale values (commands.md, evals.md, frameworks.md):

  • Fixed model ID claude-3-5-sonnet-20250929claude-3-5-sonnet-20250514
  • Added policy-engine/policy to status --type enum
  • Removed ghost --tool-filter-description flag (not in schema or CLI)
  • Fixed evaluator level values: added TOOL_CALL, removed SPAN

Structural/naming (AGENTS.md, README.md, assets):

  • Removed AutoGen from supported frameworks list (5 frameworks, not 6)
  • Fixed "agents""runtimes" in gateway JSON example
  • Renamed identitycredential in all command references
  • Fixed file paths and directory references
  • Fixed regex constraints and enum values to match Zod schemas

Files modified

FileChanges
docs/configuration.mdSchema field tables, JSON examples, type removals, credential rename, tags fields, required/optional fixes
docs/commands.mdAll flag tables and examples, import subcommands, exec mode, streaming, evaluator types, ground truth
docs/evals.mdFlag tables, examples, evaluator types section, ground truth inputs, builtin evaluator table, eval-results path fix
docs/gateway.mdidentity → credential, agents → runtimes, oauth-scopes, target type count
docs/local-development.md--runtime flag in dev examples, exec mode section, mcp.json → agentcore.json
docs/memory.mdRemoved stale types, added memory record streaming section
docs/container-builds.mdRemoved stale "type": "AgentCoreRuntime" from JSON example
docs/frameworks.mdModel ID fix, framework list correction
AGENTS.mdidentity → credential in primitives
README.mdidentity → credential
src/assets/README.mdidentity → credential in vended template
src/assets/__tests__/__snapshots__/assets.snapshot.test.ts.snapUpdated snapshot

Known source code issue (not fixed here)

src/cli/primitives/GatewayPrimitive.ts has incorrect CLI help text: --exception-level says ALL (should be DEBUG) and --authorizer-type omits AWS_IAM. This is a source code bug that should be addressed in a separate PR.

Related Issue

N/A — discovered via systematic source code audit

Documentation PR

N/A — this PR is the documentation fix

Type of Change

  • Bug fix
  • New feature
  • Breaking change
  • Documentation update
  • Other (please describe):

Testing

  • I ran npm run test:unit and npm run test:integ
  • I ran npm run typecheck
  • I ran npm run lint
  • If I modified src/assets/, I ran npm run test:update-snapshots and committed the updated snapshots

All changes are documentation-only (.md files + snapshot update). Each fix was verified against the corresponding Zod schema definition or CLI source code (src/cli/commands/*/command.tsx, src/cli/primitives/*.ts).

Note: 19 pre-existing test failures exist on upstream/main from the schema refactor (#709) — not introduced by this PR. The 2 e2e failures are expected on fork PRs (no AWS credentials).

Checklist

  • I have read the CONTRIBUTING document
  • I have added any necessary tests that prove my fix is effective or my feature works
  • I have updated the documentation accordingly
  • I have added an appropriate example to the documentation to outline the feature, or no new docs are needed
  • 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.

@aidandaly24
aidandaly24 requested a review from a teamMarch 27, 2026 04:40
@github-actionsgithub-actionsBot added the size/m PR size: M label Mar 27, 2026
@github-actionsgithub-actionsBot added size/m PR size: M and removed size/m PR size: M labels Mar 27, 2026
@aidandaly24
aidandaly24force-pushed the docs/fix-documentation-inaccuracies branch from f2e7d91 to f957db1CompareMarch 27, 2026 17:36
@github-actionsgithub-actionsBot added size/m PR size: M and removed size/m PR size: M labels Mar 27, 2026
@github-actions

github-actionsBot commented Mar 27, 2026

Copy link
Copy Markdown
Contributor

Package Tarball

aws-agentcore-0.7.1.tgz

How to install

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

@github-actionsgithub-actionsBot added size/m PR size: M and removed size/m PR size: M labels Mar 27, 2026
@github-actionsgithub-actionsBot added size/m PR size: M and removed size/m PR size: M labels Mar 27, 2026
@aidandaly24
aidandaly24force-pushed the docs/fix-documentation-inaccuracies branch from bd9373b to c4a8c0aCompareMarch 27, 2026 18:36
@github-actionsgithub-actionsBot added size/m PR size: M and removed size/m PR size: M labels Mar 27, 2026
@aidandaly24
aidandaly24force-pushed the docs/fix-documentation-inaccuracies branch from c4a8c0a to 727e1cfCompareMarch 27, 2026 21:39
@github-actionsgithub-actionsBot removed the size/m PR size: M label Mar 27, 2026
@github-actionsgithub-actionsBot added size/l PR size: L and removed size/l PR size: L labels Mar 28, 2026
@aidandaly24
aidandaly24force-pushed the docs/fix-documentation-inaccuracies branch from 1b43c59 to c8a57f4CompareMarch 28, 2026 01:22
@github-actionsgithub-actionsBot added size/l PR size: L and removed size/l PR size: L labels Mar 28, 2026
@github-actionsgithub-actionsBot added size/l PR size: L and removed size/l PR size: L labels Mar 28, 2026
@github-actionsgithub-actionsBot removed the size/l PR size: L label Mar 28, 2026
- configuration.md: fix networkMode PRIVATE→VPC, remove stale mcp.json
section, add 7 missing AgentEnvSpec fields, add Node.js runtimes,
fix gateway name max 63→100, list all 6 targetType values, fix memory
strategies wording, fix deployed-state.json path, add missing project
fields, fix CUSTOM_JWT field requirements, add customClaims field
- README.md: add CrewAI framework, update all 4 default model IDs,
add missing command sections
- frameworks.md: add CrewAI with model providers, add protocol
compatibility matrix (HTTP/MCP/A2A)
- commands.md: fix agent name max 64→48 chars, add fetch access section,
add policy-engine/policy to status --type, remove ghost
--tool-filter-description flag, fix model ID date
- gateway.md: add enableSemanticSearch/exceptionLevel config options,
fix mcp.json→agentcore.json, remove api-key from MCP Server auth
- transaction_search.md: fix traces command syntax
- container-builds.md: remove nonexistent --progress flag
- evals.md: fix model date 20250929→20250514
- local-development.md: fix mcp.json→agentcore.json
- AGENTS.md: remove AutoGen, remove stale mcp.json reference
- src/assets/README.md: add CrewAI to template list
- src/assets/agents/AGENTS.md: remove AutoGen
Constraint: All fixes verified against Zod schemas and CLI runtime
Constraint: CLI --help text bugs left for separate agent
Confidence: high
Scope-risk: narrow
- Remove `type` fields from agent, memory, evaluator, online-eval-config
examples and field tables (removed in aws#709)
- Remove `modelProvider` from agent spec (removed in aws#709)
- Rename credential `type` to `authorizerType` (renamed in aws#709)
- Rename `add identity` to `add credential` everywhere (aws#705)
- Replace `--force` with `-y/--yes` on remove commands (aws#705)
- Replace `--plan` with `--dry-run` on deploy command (aws#705)
- Update snapshot for src/assets/README.md change
Update `"agents": [...]` to `"runtimes": [...]` in the gateway
configuration example to match the schema rename in aws#706.
Update all documentation to reflect upstream renames:
- --agent → --runtime, -a → -r across all commands
- --agent-arn → --runtime-arn, --agent-runtime-id → --runtime-id
- Remove stale "type" fields from JSON examples in evals.md,
memory.md, and container-builds.md
Constraint: --agent-id, --agent-alias-id, --agent-client-id,
--agent-client-secret are unchanged (different semantics)
Constraint: --type agent filter value in status/fetch stays as-is
Confidence: high
Scope-risk: narrow
- Add -H, --header flag to dev and invoke commands
- Add --bearer-token flag to invoke command
- Fix --lines → --limit in evals.md logs evals section
Confidence: high
Scope-risk: narrow
- Rename --agent-client-id → --client-id, --agent-client-secret →
--client-secret in gateway docs (naming unification aws#705)
- Add missing flags to add agent: --authorizer-type, --discovery-url,
--allowed-audience, --allowed-clients, --allowed-scopes,
--custom-claims, --client-id, --client-secret, --idle-timeout,
--max-lifetime
- Add missing flags to create: --idle-timeout, --max-lifetime
- Add missing flags to add gateway: --runtimes, --custom-claims,
--policy-engine, --policy-engine-mode
- Fix --exception-level values: NONE/ALL (was undocumented)
Confidence: high
Scope-risk: narrow
The CLI help text says ALL but the schema (GatewayExceptionLevelSchema)
defines the valid values as NONE and DEBUG. Aligns commands.md with
gateway.md and the Zod schema.
- Remove stale "type": "AgentCoreMemory" from memory streaming example
- Add memory streaming flags: --delivery-type, --data-stream-arn,
--stream-content-level, --stream-delivery-resources
- Add --exec to dev command flag table (Container agents)
- Add --skip-install to create command flag table
- Add --identity-name to fetch access flag table
@Hweinstock

Copy link
Copy Markdown
Contributor

Looks like lint is failing, otherwise looks good:

Run npm run format:check
> @aws/agentcore@0.7.1 format:check
> prettier --check .
Checking formatting...
[warn] docs/commands.md
[warn] Code style issues found in the above file. Run Prettier with --write to fix.

@jariy17jariy17 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

lgtm and fix the lint

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

Labels

size/lPR size: L

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@aidandaly24@Hweinstock@jariy17