fix(agents-api): Remove unnecessary 'type' property from tool type definition

[creatorrr]

Signed-off-by: Diwank Singh Tomer [email protected]

---

Important

Removed 'type' property from tool classes and added computed 'type' property using a function in Tools.py, openapi_model.py, and tasks.py.

• Behavior:
  • Removed type property from CreateToolRequest, NamedToolChoice, PatchToolRequest, Tool, and UpdateToolRequest in Tools.py.
  • Added computed type property to TaskTool, Tool, UpdateToolRequest, and PatchToolRequest in openapi_model.py.
  • Updated task_to_spec() in tasks.py to use computed type property.
• Misc:
  • Removed type property from Tool and NamedToolChoice in models.tsp.

^{This description was created by for 289e87e. It will automatically update as commits are pushed.}

[ellipsis-dev]

👍 Looks good to me! Reviewed everything up to 289e87e in 11 seconds

More details

• Looked at 273 lines of code in 4 files
• Skipped 0 files when reviewing.
• Skipped posting 1 drafted comments based on config settings.

1. agents-api/agents_api/common/protocol/tasks.py:227

• Draft comment:
  Ensure that the 'type' property is correctly set for each tool before accessing it here. The removal of the 'type' property from the Tool model means it must be dynamically determined or set elsewhere.
• Reason this comment was not posted:
  Comment did not seem useful.

Workflow ID: wflow_aLixZCUrGZZ83gGG

---

You can customize Ellipsis with 👍 / 👎 feedback, review rules, user-specific overrides, quiet mode, and more.

[creatorrr]

@Vedantsahai18 this fixes the problem that you had while creating the tool for agent. So now this snippet works correctly:

[ins] In [1]: client.agents.tools.create(agent_id=agent.id,
         ...:     name="web_search",
         ...:     integration={
         ...:         "provider": "brave",
         ...:         "method": "search",
         ...:         "setup": {},
         ...:     },
         ...: )