Add conditional for Python agent tools - #25677
Conversation
Sorry, something went wrong.
Don Jayamanne (DonJayamanne)
left a comment
There was a problem hiding this comment.
This solution will break simple prompts such as create a python file to do xyz and verify it works correctly
The problem here is that the model uses this tool to install npm packages.
Thats wrong.
I would try updating the prompts to include text such as Do NOT use this tool to install npm or other non-python packages
Sorry, something went wrong.
| "required": [] | ||
| } | ||
| }, | ||
| "when": "editorLangId == python || resourceLangId == python || notebookType == jupyter-notebook" |
There was a problem hiding this comment.
This would break simple cases like a pomrpt such as create a python/notebook file that does ....
Sorry, something went wrong.
There was a problem hiding this comment.
gotcha- I was going off of Connor's comment about using a "when" statement but this makes sense
Sorry, something went wrong.
Don Jayamanne (DonJayamanne)
left a comment
There was a problem hiding this comment.
I'd suggest tweaking to be more descriptive when not to use this
IMPORTANT: This tool should only be used for Python environments and Python Packages - do not use for non-Python packages.
Sorry, something went wrong.
fixes #25644