Skip to content

chore: update CP APIs to support namespace re-design#450

Open
nborges-aws wants to merge 2 commits intomainfrom
chore/namespaceRedesignControlPlane
Open

chore: update CP APIs to support namespace re-design#450
nborges-aws wants to merge 2 commits intomainfrom
chore/namespaceRedesignControlPlane

Conversation

@nborges-aws
Copy link
Copy Markdown
Contributor

Summary:

Migrates the memory control-plane to the new namespaceTemplates field, deprecating the legacy namespaces field while maintaining full backward compatibility.

Description of changes:

  • MemoryClient (memory/client.py) — 13 methods updated with parallel namespace_templates kwargs
  • MemoryControlPlaneClient (memory/controlplane.py) — parallel namepsace_templates kwargs
  • Episodic methods gained reflection_namespace_templates, deprecating reflection_namespaces

Behavior changes for users

  • No breaking changes. All existing code using namespaces= / reflection_namespaces= continues to work, with a DeprecationWarning surfaced to encourage migration.
  • Calling a method with both namespaces= and namespace_templates= raises ValueError with a message pointing users to the new name.

Testing

  • New TestResolveNamespaceTemplates unit tests cover the happy paths, mutual exclusivity, and deprecation warning emission
  • Existing method tests updated to assert on the new namespaceTemplates key rather than namespaces. Existing legacy-kwarg callers in tests are preserved as DeprecationWarning-triggering coverage for the backward-compatible path.
  • Full suite passing : 1706 passed, 0 failures. 16 DeprecationWarnings are expected — messaging for legacy namespace field

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

@nborges-aws nborges-aws requested a review from a team May 1, 2026 18:19
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 1, 2026

✅ No Breaking Changes Detected

No public API breaking changes found in this PR.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant