Skip to content

Commit ca44c7b

Browse files
committed
Explicitly state tools are not shells
Some LLMs don't understand that available tools are not shell commands, so they send shell commands with pipes. In this patch we try to guard against such explicitly errors by explicitly stating that the tools are not shell.
1 parent 028c702 commit ca44c7b

2 files changed

Lines changed: 3 additions & 0 deletions

File tree

src/rhos_ls_mcps/oc.py

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -75,6 +75,8 @@ async def openshift_cli_mcp_tool(command_str: str, ctx: Context) -> str:
7575
Runs an `oc` command as if it were run in a terminal.
7676
No need to provide credentials, they are already passed by the client.
7777
78+
THIS IS NOT A SHELL don't use pipes
79+
7880
Args:
7981
command_str: String with the command to run.
8082
Returns:

src/rhos_ls_mcps/osc.py

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -166,6 +166,7 @@ async def openstack_cli_mcp_tool(command_str: str, ctx: Context) -> str:
166166
- `nova list` is now `openstack server list`
167167
168168
DON'T EVER USE commands such as cinder, nova, glance. Use `openstack` instead.
169+
THIS IS NOT A SHELL don't use pipes
169170
170171
A complete list of commands is available using the help commands:
171172
- Global options and supported commands: `openstack --help` or `--help`

0 commit comments

Comments
 (0)