julep icon indicating copy to clipboard operation
julep copied to clipboard

Update CLI tool request typing

Open creatorrr opened this issue 6 months ago • 2 comments
trafficstars

User description

Summary

  • use specific Tool definitions from agents-api autogen models for CLI

Testing

  • pyright cli/src/julep_cli/models.py (fails: No overloads for __init__ match the provided arguments)

PR Type

Enhancement


Description

  • Replace generic Any types with specific tool definitions in CreateToolRequest

  • Import and use precise models from agents_api.autogen

  • Improve type safety and code clarity for tool-related fields


Changes walkthrough 📝

Relevant files
Enhancement
models.py
Refactor CreateToolRequest to use typed tool definitions 

cli/src/julep_cli/models.py

  • Imported specific tool definition classes from agents_api.autogen
  • Updated CreateToolRequest fields to use precise types (e.g.,
    FunctionDef, SystemDef)
  • Removed Any types and replaced with relevant model classes
  • Enhanced type safety and maintainability for tool request modeling
  • +17/-9   

    Need help?
  • Type /help how to ... in the comments thread for any questions about Qodo Merge usage.
  • Check out the documentation for more information.

  • [!IMPORTANT] Update CreateToolRequest in models.py to use specific types from agents_api.autogen for improved type safety.

    • Typing Updates:
      • Update CreateToolRequest in models.py to use specific types from agents_api.autogen.
      • Replace Any with FunctionDef, IntegrationDef, SystemDef, ApiCallDef, Computer20241022Def, TextEditor20241022Def, and Bash20241022Def.
    • Testing:
      • pyright type checking fails due to mismatched __init__ arguments in models.py.

    This description was created by Ellipsis for 4124f881a63f946564312628c1a088d63e7a5050. You can customize this summary. It will automatically update as commits are pushed.

    creatorrr avatar May 17 '25 18:05 creatorrr