Migrate harness CRUD to SDK commands and remove api-client.ts
Open
code-quality
- Dominant language
- TypeScript
- Stars
- 283
- Forks
- 95
- Avg merge
- 1d 2h
- Merged PRs (30d)
- 183
Description
From PR #1341 review (jariy17).
The SDK now has `CreateHarnessCommand`, `GetHarnessCommand`, `UpdateHarnessCommand`, `DeleteHarnessCommand`, `ListHarnessesCommand` in `@aws-sdk/client-bedrock-agentcore-control`, and `InvokeHarnessCommand` in `@aws-sdk/client-bedrock-agentcore`.
The SigV4 CRUD operations and ~150 lines of event stream parsing in `src/cli/aws/agentcore-harness.ts` can be replaced by SDK commands. The bearer token path for CUSTOM_JWT auth (line 426) still needs raw fetch.
Once migrated, `src/cli/aws/api-client.ts` becomes dead code and can be deleted together.
Contributor guide
Assessment
This issue has not been assessed yet.