Skip to content

Fix issue of zero-arg tools to be called#254

Merged
ahyatt merged 3 commits intomainfrom
zero-args
Mar 9, 2026
Merged

Fix issue of zero-arg tools to be called#254
ahyatt merged 3 commits intomainfrom
zero-args

Conversation

@ahyatt
Copy link
Copy Markdown
Owner

@ahyatt ahyatt commented Mar 9, 2026

We need to know when we execute tools at all. Previously we used the existance of the arg mapping to know when there's something to execute, but since tools may have no args, it's better to use the existence of tool uses itself.

This fixes issue #253

@ahyatt
Fix issue of zero-arg tools to be called
1350390 
We need to know when we execute tools at all.  Previously we used the existance
of the arg mapping to know when there's something to execute, but since tools
may have no args, it's better to use the existence of tool uses itself.

This fixes issue #253
@ahyatt ahyatt had a problem deploying to Continuous Integration March 9, 2026 15:17 — with GitHub Actions Error
@ahyatt ahyatt had a problem deploying to Continuous Integration March 9, 2026 15:17 — with GitHub Actions Error
@ahyatt ahyatt had a problem deploying to Continuous Integration March 9, 2026 15:17 — with GitHub Actions Error
@ahyatt ahyatt had a problem deploying to Continuous Integration March 9, 2026 15:17 — with GitHub Actions Failure
@ahyatt ahyatt had a problem deploying to Continuous Integration March 9, 2026 15:17 — with GitHub Actions Failure
@ahyatt ahyatt had a problem deploying to Continuous Integration March 9, 2026 15:17 — with GitHub Actions Error
@ahyatt ahyatt had a problem deploying to Continuous Integration March 9, 2026 15:17 — with GitHub Actions Error
@ahyatt ahyatt had a problem deploying to Continuous Integration March 9, 2026 16:31 — with GitHub Actions Error
@ahyatt ahyatt had a problem deploying to Continuous Integration March 9, 2026 16:31 — with GitHub Actions Error
@ahyatt ahyatt had a problem deploying to Continuous Integration March 9, 2026 16:31 — with GitHub Actions Error
@ahyatt ahyatt had a problem deploying to Continuous Integration March 9, 2026 16:31 — with GitHub Actions Error
@ahyatt ahyatt had a problem deploying to Continuous Integration March 9, 2026 16:31 — with GitHub Actions Failure
@ahyatt ahyatt had a problem deploying to Continuous Integration March 9, 2026 16:31 — with GitHub Actions Failure
@ahyatt ahyatt had a problem deploying to Continuous Integration March 9, 2026 16:31 — with GitHub Actions Error
@ahyatt ahyatt temporarily deployed to Continuous Integration March 9, 2026 16:47 — with GitHub Actions Inactive
@ahyatt ahyatt temporarily deployed to Continuous Integration March 9, 2026 16:47 — with GitHub Actions Inactive
@ahyatt ahyatt temporarily deployed to Continuous Integration March 9, 2026 16:47 — with GitHub Actions Inactive
@ahyatt ahyatt temporarily deployed to Continuous Integration March 9, 2026 16:47 — with GitHub Actions Inactive
@ahyatt ahyatt temporarily deployed to Continuous Integration March 9, 2026 16:47 — with GitHub Actions Inactive
@ahyatt ahyatt temporarily deployed to Continuous Integration March 9, 2026 16:47 — with GitHub Actions Inactive
@ahyatt ahyatt temporarily deployed to Continuous Integration March 9, 2026 16:47 — with GitHub Actions Inactive
@ahyatt ahyatt merged commit 951524c into main Mar 9, 2026
7 checks passed
@ahyatt ahyatt deleted the zero-args branch March 9, 2026 16:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@ahyatt