Skip to content

Drop already existing "computer" if the selected computer tool is renamed #208

Merged
pimpale merged 3 commits intomainfrom
g/fix-computer-overlap
Nov 24, 2025
Merged

Drop already existing "computer" if the selected computer tool is renamed #208
pimpale merged 3 commits intomainfrom
g/fix-computer-overlap

Conversation

@pimpale
Copy link
Copy Markdown
Collaborator

@pimpale pimpale commented Nov 23, 2025

Proof: https://hud.ai/trace/2eb24328-f518-47b7-a4c5-88884677ea3c


Note

Adjust Claude tool conversion to prefer a selected computer tool and drop the original computer, add setup error logging, and bump anthropic to >=0.70.

  • Agents (Claude):
    • Update tool conversion in hud/agents/claude.py:
      • Prefer selected computer tool (e.g., anthropic_computer), expose as computer.
      • If a different computer already exists, warn and drop it.
      • Allow to_api_tool to return None and skip such tools; maintain tool_mapping and has_computer_tool accordingly.
  • Agents (Base):
    • In hud/agents/base.py, log individual setup tool errors before returning failure.
  • Dependencies:
    • Pin anthropic to >=0.70 in pyproject.toml.

Written by Cursor Bugbot for commit 37a9905. This will update automatically on new commits. Configure here.

Copy link
Copy Markdown
Contributor

@Parth220 Parth220 left a comment

Choose a reason for hiding this comment

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

LGTM, are there any updates we should make to the docs for the new claude agent changes?

Copy link
Copy Markdown
Contributor

@Parth220 Parth220 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, just tested it with a full run of SheetBench and SpreadsheetBench.

Nice call on pinning the anthropic api as well.

@pimpale pimpale merged commit 6c19533 into main Nov 24, 2025
10 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