Skip to content

Refactored Runs -> Threads#202

Merged
drbrady8800 merged 6 commits intomainfrom
add_thread
Jun 5, 2025
Merged

Refactored Runs -> Threads#202
drbrady8800 merged 6 commits intomainfrom
add_thread

Conversation

@ritzz26
Copy link
Contributor

@ritzz26 ritzz26 commented Jun 5, 2025

Overview

According to #60, renamed Runs to Threads a part of the first step of the big refactor.

@ritzz26 ritzz26 requested a review from Copilot June 5, 2025 02:07
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR renames the context identifier from RunContext to ThreadContext (and run_id to thread_id) as part of a larger refactor.

  • Updated function signatures and inline documentation examples in various tool-related documentation files.
  • Modified API example endpoints and Swagger documentation to reflect the new thread_id naming convention.
  • Ensured consistency across code snippets and explanations within the docs.

Reviewed Changes

Copilot reviewed 75 out of 75 changed files in this pull request and generated no comments.

Show a summary per file
File Description
docs/tools/tool-library/rag-tool.md Updated function parameter and description from RunContext to ThreadContext.
docs/tools/tool-library/playwright-tool.md Replaced RunContext with ThreadContext in several tool functions.
docs/tools/tool-library/meeting-baas-tool.md Transitioned RunContext to ThreadContext for meeting tool functions.
docs/tools/tool-library/imap-tool.md Renamed RunContext to ThreadContext for email-related functions.
docs/tools/tool-library/image-generator-tool.md Updated async function signature to use ThreadContext.
docs/tools/tool-library/human-interrupt-tool.md Changed parameter name from RunContext to ThreadContext in function definition.
docs/tools/tool-library/github-tool.md Replaced RunContext with ThreadContext across multiple GitHub tool functions.
docs/tools/tool-library/database-tool.md Updated database tool functions to use ThreadContext.
docs/tools/tool-library/browser-use-tool.md Changed async function signature from RunContext to ThreadContext.
docs/tools/tool-library/automatic-tools.md Updated tool enabling function to use ThreadContext.
docs/tools/tool-library/authorized-rest-api-tool.md Renamed parameters from RunContext to ThreadContext in API tool functions.
docs/tools/tool-library/airbnb-calendar-tool.md Replaced RunContext with ThreadContext in Airbnb calendar tool functions.
docs/tools/index.md Updated all references and code examples to replace RunContext with ThreadContext.
docs/tools/building-tools.md Revised example code and descriptions in tool-building docs to use ThreadContext.
docs/interacting-with-agents/[DEPRECIATED]-rest-api-no-swagger.md Changed run_id to thread_id in API examples and endpoint documentation.
docs/glossary.md Updated glossary entry from RunContext to ThreadContext.
docs/core-concepts/event-types.md Modified event examples to use thread_context instead of run_context.
docs/building-agents/index.md Changed explanations and code examples to refer to ThreadContext.
docs/building-agents/advanced-custom-agents.md Adjusted subagent propagation documentation to use thread_id.
docs/assets/swagger.json Updated Swagger endpoints and parameter names from run_id to thread_id.

@ritzz26 ritzz26 requested a review from drbrady8800 June 5, 2025 02:30
Copy link
Contributor

@drbrady8800 drbrady8800 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good and works, found some instances of run and pushed up updates for them. Also added an auto migrations for agent_runs -> agent_threads. Appreciate the effort, stuff like this can be very tedious.

@drbrady8800 drbrady8800 merged commit 98cb99d into main Jun 5, 2025
2 checks passed
@drbrady8800 drbrady8800 deleted the add_thread branch June 5, 2025 16:41
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.

3 participants