Skip to content

Add standalone MCP shell server#7

Open
tg1482 wants to merge 2 commits intomainfrom
mcp-shell
Open

Add standalone MCP shell server#7
tg1482 wants to merge 2 commits intomainfrom
mcp-shell

Conversation

@tg1482
Copy link
Copy Markdown
Owner

@tg1482 tg1482 commented Feb 1, 2026

Summary

  • add a standalone mcp/ uv project for a shell MCP server
  • include policy loading, command allow/deny checks, and output truncation
  • provide a default policy template for local development

Testing

  • not run (not requested)

"git",
"ls",
"pwd",
"rg",
Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

we dont even have git and rg and uv commands wtf. what is this for?



@dataclass(frozen=True)
class ShellPolicy:
Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

our shellpolicy here is very limited to just the shell commands that we support baesd on README - make suer thats respected


from policy import ShellPolicy


Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

how will this change now that we have a file backed feature?

@tg1482
Copy link
Copy Markdown
Owner Author

tg1482 commented Feb 1, 2026

Simplified the policy: removed deny/env sections, switched to a single list aligned to the README shell commands, and dropped env overrides in the server (uses process env). Re: file-backed Loopy — not wired into this MCP server yet; that would be a separate MCP tool that calls against a path. Happy to add that in a follow-up if you want it here.

@tg1482
Copy link
Copy Markdown
Owner Author

tg1482 commented Feb 1, 2026

Simplified the policy: removed deny/env sections, switched to a single allowed_commands list aligned to the README shell commands, and dropped env overrides in the server (uses process env).

Re: file-backed Loopy — not wired into this MCP server yet; that would be a separate MCP tool that calls loopy.shell.run against a FileBackedLoopy path. Happy to add that in a follow-up if you want it here.

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.

1 participant