Skip to content

ASND-133: programmatic agent creation#45

Open
matiasedgeandnode wants to merge 2 commits intomainfrom
ma/ASND-133-api-access
Open

ASND-133: programmatic agent creation#45
matiasedgeandnode wants to merge 2 commits intomainfrom
ma/ASND-133-api-access

Conversation

@matiasedgeandnode
Copy link
Collaborator

@matiasedgeandnode matiasedgeandnode commented Feb 2, 2026 •
edited
Loading

Summary

  • Add AmpersendManagementClient for agent CRUD operations via API key auth (Python + TypeScript)
  • Require explicit agentAddress in ApiClient config instead of deriving from server response
  • Client now sends agentAddress on login and verifies server response matches

Test plan

  • Run uv run pytest python/ampersend-sdk/tests/unit/ampersend/test_management.py
  • Run pnpm --filter ampersend-sdk test management.test.ts
  • Verify existing treasurer tests still pass

@matiasedgeandnode matiasedgeandnode marked this pull request as ready for review February 2, 2026 19:29
@matiasedgeandnode matiasedgeandnode changed the title ASND-133: add AmpersendManagementClient to Python SDK ASND-133: programmatic agent creation Feb 3, 2026
@matiasedgeandnode matiasedgeandnode force-pushed the ma/ASND-133-api-access branch 2 times, most recently from aae1545 to 70b0a34 Compare February 5, 2026 19:37
@matiasedgeandnode matiasedgeandnode force-pushed the ma/ASND-133-api-access branch 2 times, most recently from c412e20 to 13f74be Compare February 11, 2026 18:10
@matiasedgeandnode matiasedgeandnode force-pushed the ma/ASND-133-api-access branch 3 times, most recently from 4bf75f2 to 86d949a Compare March 4, 2026 14:49
@matiasedgeandnode
Add AmpersendManagementClient, require explicit agentAddress on login
31cffbe 
- Add AmpersendManagementClient for agent CRUD via API key auth (Python + TS)
- ApiClient now requires agentAddress in config instead of deriving from server
- Login request sends agentAddress, client verifies server response matches
- Server handles agent deployment via ephemeral key pattern (no client signing)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@pcarranzav pcarranzav pcarranzav approved these changes

@tmigone tmigone Awaiting requested review from tmigone

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@matiasedgeandnode @pcarranzav