Skip to content

Revert "feat: add support for LTM metadata (#1281)"#1338

Open
Hweinstock wants to merge 1 commit into
aws:mainfrom
Hweinstock:revert/ltm-metadata
Open

Revert "feat: add support for LTM metadata (#1281)"#1338
Hweinstock wants to merge 1 commit into
aws:mainfrom
Hweinstock:revert/ltm-metadata

Conversation

@Hweinstock
Copy link
Copy Markdown
Contributor

Reverting LTM metadata changes until after the bug bash.

@github-actions github-actions Bot added the size/xl PR size: XL label May 20, 2026
@Hweinstock Hweinstock deployed to e2e-testing May 20, 2026 23:25 — with GitHub Actions Active
@github-actions github-actions Bot added the agentcore-harness-reviewing AgentCore Harness review in progress label May 20, 2026
@agentcore-devx-automation agentcore-devx-automation Bot added the claude-security-reviewing Claude Code /security-review in progress label May 20, 2026
@github-actions
Copy link
Copy Markdown
Contributor

Package Tarball

aws-agentcore-0.14.1.tgz

How to install

gh release download pr-1338-tarball --repo aws/agentcore-cli --pattern "*.tgz" --dir /tmp/pr-tarball
npm install -g /tmp/pr-tarball/aws-agentcore-0.14.1.tgz

@agentcore-devx-automation
Copy link
Copy Markdown
Contributor

Claude Security Review: no high-confidence findings. (run)

@agentcore-devx-automation agentcore-devx-automation Bot removed the claude-security-reviewing Claude Code /security-review in progress label May 20, 2026
Copy link
Copy Markdown

@agentcore-cli-automation agentcore-cli-automation left a comment

Choose a reason for hiding this comment

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

Clean revert of #1281. I checked:

  • The diff is the exact inverse of commit 44b5c2c (880 insertions/266 deletions in the original; 266 insertions/880 deletions here).
  • No orphaned references to IndexedKey, indexedKey, MAX_INDEXED_KEYS, INDEXED_KEY_NAME_PATTERN, etc. remain anywhere in src/, schemas/, or docs/.
  • The @aws-sdk/client-bedrock-agentcore-control downgrade (^3.1048.0^3.1039.0) and the corresponding WaiterState import path change (@smithy/core/client@smithy/util-waiter) are consistent with the SDK version that's pinned, and @smithy/util-waiter is still in the shrinkwrap.
  • Telemetry attributes indexed_key_count / has_indexed_keys are removed from both the schema and the add.memory callsites (CLI and TUI hook), keeping them in sync.

Looks good to merge.

@github-actions github-actions Bot removed the agentcore-harness-reviewing AgentCore Harness review in progress label May 20, 2026
@Hweinstock Hweinstock marked this pull request as ready for review May 20, 2026 23:29
@Hweinstock Hweinstock requested a review from a team May 20, 2026 23:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size/xl PR size: XL

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants