Skip to content

feat: add DatasetClient for Dataset Management API#451

Closed
jariy17 wants to merge 2 commits intomainfrom
feature/dataset-management
Closed

feat: add DatasetClient for Dataset Management API#451
jariy17 wants to merge 2 commits intomainfrom
feature/dataset-management

Conversation

@jariy17
Copy link
Copy Markdown
Contributor

@jariy17 jariy17 commented May 2, 2026

Summary

  • New DatasetClient in src/bedrock_agentcore/dataset/client.py following the GatewayClient pattern
  • Single bedrock-agentcore-control boto3 client with _ALLOWED_CP_METHODS allowlist and __getattr__ dispatch
  • High-level helpers: create_dataset_and_wait, update_dataset_and_wait, delete_dataset_and_wait
  • clientToken idempotency, get_paginator/get_waiter support, response unwrapping fixed
  • 24 tests, 100% coverage (threshold: 90%)

Test plan

  • Unit tests covering initialization, dispatch, CRUD, wait helpers, error cases
  • 100% coverage on bedrock_agentcore/dataset/
  • All hooks pass (tsc, eslint, etc.)

🤖 Generated with Claude Code

jariy17 and others added 2 commits May 1, 2026 22:08
Implements DatasetClient mirroring GatewayClient pattern with single
bedrock-agentcore-control boto3 client, __getattr__ dispatch for
create/get/list/update/delete_dataset, and wait helpers.
- Export DatasetClient from top-level bedrock_agentcore package
- Unwrap response["dataset"] in polling lambdas for create/update_and_wait
- Add clientToken (uuid4) idempotency to create_dataset_and_wait
- Guard delete_dataset_and_wait with response.get("datasetId") + ValueError
- Add get_paginator and get_waiter to _ALLOWED_CP_METHODS
- Update tests to 100% coverage (24 tests) covering all new behaviours

Co-Authored-By: Claude Sonnet 4.6 (1M context) <noreply@anthropic.com>
@jariy17 jariy17 requested a review from a team May 2, 2026 14:17
@jariy17 jariy17 temporarily deployed to auto-approve May 2, 2026 14:17 — with GitHub Actions Inactive
@jariy17 jariy17 temporarily deployed to auto-approve May 2, 2026 14:17 — with GitHub Actions Inactive
@jariy17 jariy17 temporarily deployed to auto-approve May 2, 2026 14:17 — with GitHub Actions Inactive
@jariy17 jariy17 temporarily deployed to auto-approve May 2, 2026 14:17 — with GitHub Actions Inactive
@jariy17 jariy17 temporarily deployed to auto-approve May 2, 2026 14:17 — with GitHub Actions Inactive
@jariy17 jariy17 temporarily deployed to auto-approve May 2, 2026 14:17 — with GitHub Actions Inactive
@jariy17 jariy17 deployed to auto-approve May 2, 2026 14:17 — with GitHub Actions Active
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented May 2, 2026

✅ No Breaking Changes Detected

No public API breaking changes found in this PR.

@jariy17 jariy17 closed this May 2, 2026
@jariy17 jariy17 deleted the feature/dataset-management branch May 2, 2026 14:18
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