Skip to content

Ian/threads no order#14

Merged
ianmacartney merged 11 commits intomainfrom
ian/threads-no-order
Apr 11, 2025
Merged

Ian/threads no order#14
ianmacartney merged 11 commits intomainfrom
ian/threads-no-order

Conversation

@ianmacartney
Copy link
Copy Markdown
Member

@convex-dev/agent@0.0.5-alpha.1 has made each agent-as-tool into its own thread, setting the parent thread id for later
It now also doesn't have a specific order for threads, just uses creation time.
Using order and subOrder on messages might be a similar footgun, we'll see if those go away too

Fixes #13


By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@ianmacartney
Copy link
Copy Markdown
Member Author

Also this fixes the issue where when passed recentMessages=0 it was still fetching one document (filed get-convex/convex-helpers#534)

@ianmacartney ianmacartney merged commit 656c285 into main Apr 11, 2025
1 check passed
@ianmacartney ianmacartney deleted the ian/threads-no-order branch April 11, 2025 08:52
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.

Error executing tool. Concurrency error when writing to "messages" table.

1 participant