Skip to content

api: add actor_uid to Assignment alongside actor - #736

Merged
Haven Xia (HavenXia) merged 3 commits into
agent-substrate:mainfrom
troychiu:fix-issue-653-actor-uid-assignment
Aug 6, 2026
Merged

api: add actor_uid to Assignment alongside actor#736
Haven Xia (HavenXia) merged 3 commits into
agent-substrate:mainfrom
troychiu:fix-issue-653-actor-uid-assignment

Conversation

@troychiu

Copy link
Copy Markdown
Contributor

Fixes #653 by adding actor_uid to Assignment alongside actor

  • Tests pass
  • Appropriate changes to documentation are included in the PR

@troychiu Troy Chiu (troychiu) changed the title Fix issue 653 actor uid assignment api: add actor_uid to Assignment alongside actor Aug 4, 2026
@troychiu

Copy link
Copy Markdown
Contributor Author

cc Zoe Zhao (@zoez7)

@troychiu
Troy Chiu (troychiu) force-pushed the fix-issue-653-actor-uid-assignment branch from 0637e50 to e09dd6f Compare August 5, 2026 17:45
@juli4n

Copy link
Copy Markdown
Collaborator

Change looks good. Some tests seem to be failing though.

@troychiu

Copy link
Copy Markdown
Contributor Author

Change looks good. Some tests seem to be failing though.

I took a look, and it doesn't appear to be related to this PR. #218 seems relevant.

@troychiu

Copy link
Copy Markdown
Contributor Author

Just did a rebase onto main.

@troychiu

Copy link
Copy Markdown
Contributor Author

Haven Xia (@HavenXia) thanks for the review. The failure went away.

@troychiu

Troy Chiu (troychiu) commented Aug 6, 2026

Copy link
Copy Markdown
Contributor Author

Hi Haven Xia (@HavenXia) , do you mind merging this one? Thank you.

@HavenXia

Copy link
Copy Markdown
Collaborator

Hi Haven Xia (Haven Xia (@HavenXia)) , do you mind merging this one? Thank you.

Sure! Sorry for the delay

@HavenXia
Haven Xia (HavenXia) merged commit 9748993 into agent-substrate:main Aug 6, 2026
11 checks passed
@troychiu

Copy link
Copy Markdown
Contributor Author

No worries and thank you!

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.

Work Assignment should use Actor UID instead of (atespace, name)

3 participants