Skip to content

[runtime] Avoid create new runner context each time execute action. #392

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
xintongsong merged 1 commit into from
Dec 25, 2025
Merged

[runtime] Avoid create new runner context each time execute action. #392

xintongsong merged 1 commit into from
Dec 25, 2025

Conversation

@wenjin272
Copy link
Collaborator

@wenjin272 wenjin272 commented Dec 24, 2025 •
edited
Loading

Linked issue: #344

Purpose of change

Currently, we create a new RunnerContext each time execute a new action, which cause the repeated creation for AgentPlan.
In this pr, we support create a RunnerContext once, and reuse it across actions, only switch the action related context in RunnerContext before execute a new action.

Tests

Existed tests can verify this change.

API

No

Documentation

  • doc-needed
  • doc-not-needed

@github-actions github-actions bot added priority/major Default priority of the PR or issue. fixVersion/0.2.0 The feature or bug should be implemented/fixed in the 0.2.0 version. doc-not-needed Your PR changes do not impact docs labels Dec 24, 2025
xintongsong
xintongsong approved these changes Dec 25, 2025
View reviewed changes
Copy link
Contributor

@xintongsong xintongsong left a comment

Choose a reason for hiding this comment

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

LGTM, merging.

@xintongsong xintongsong merged commit d1b2601 into apache:main Dec 25, 2025
14 of 15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@xintongsong xintongsong xintongsong approved these changes

Assignees

No one assigned

Labels

doc-not-needed Your PR changes do not impact docs fixVersion/0.2.0 The feature or bug should be implemented/fixed in the 0.2.0 version. priority/major Default priority of the PR or issue.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@wenjin272 @xintongsong