Skip to content

refactor: type AgentInstance identifiers as UUIDs - #2562

Draft
EItanya wants to merge 2 commits into
mainfrom
refactor-typed-agent-instance-ids
Draft

refactor: type AgentInstance identifiers as UUIDs#2562
EItanya wants to merge 2 commits into
mainfrom
refactor-typed-agent-instance-ids

Conversation

@EItanya

@EItanya EItanya commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Summary

  • represent AgentInstance, share, checkpoint, and A2A context identifiers as uuid.UUID in the database layer
  • migrate the corresponding PostgreSQL columns from TEXT to native UUID columns
  • generate UUID-typed SQLC models and query parameters, converting only at protobuf/service boundaries
  • preserve empty pagination cursors while comparing non-empty cursors as UUIDs
  • update database tests to use valid UUID fixtures

Testing

  • go test ./...
  • make -C go lint

Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io>
@EItanya
EItanya requested a review from a team as a code owner August 25, 2026 19:32
Signed-off-by: Eitan Yarmush <eitan.yarmush@solo.io>
@EItanya
EItanya marked this pull request as draft August 25, 2026 20:38
@EItanya

EItanya commented Aug 25, 2026

Copy link
Copy Markdown
Contributor Author

Drafted this as I'm not convinced it's worth it

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