Skip to content

Feature/logic optimization 20260309 02#155

Merged
tptpp merged 6 commits intomainfrom
feature/logic-optimization-20260309-02
Mar 10, 2026
Merged

Feature/logic optimization 20260309 02#155
tptpp merged 6 commits intomainfrom
feature/logic-optimization-20260309-02

Conversation

@yhjun1026
Copy link
Collaborator

Description

Please include a summary of the change and which issue is fixed. Please also include relevant motivation and context. List any dependencies that are required for this change.

How Has This Been Tested?

Please describe the tests that you ran to verify your changes. Provide instructions so we can reproduce. Please also list any relevant details for your test configuration

Snapshots:

Include snapshots for easier review.

Checklist:

  • My code follows the style guidelines of this project
  • I have already rebased the commits and make the commit message conform to the project standard.
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • Any dependent changes have been merged and published in downstream modules

yhjun1026 and others added 6 commits March 9, 2026 18:12
- Only pass 'context' kwarg to tools that explicitly accept it
- Prevents 'unexpected keyword argument' errors for tools like read_file
- Move tool_context building before argument filtering for cleaner logic

Ultraworked with [Sisyphus](https://github.com/code-yeongyu/oh-my-opencode)
Co-authored-by: Sisyphus <clio-agent@sisyphuslabs.ai>
@tptpp tptpp merged commit f00bc93 into main Mar 10, 2026
2 of 3 checks passed
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.

2 participants