Skip to content

Container server fixes #128

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 7 commits into from
May 1, 2025
Merged

Container server fixes #128

merged 7 commits into from
May 1, 2025

Conversation

cmsparks
Copy link
Collaborator

@cmsparks cmsparks commented May 1, 2025

I already deployed this, but this includes:

  • Docker image size improvements
  • New container app, with increase max containers limit
  • UserContainer, which manages a container per user, as opposed to a container per MCP session
  • Tool description improvements
  • Track active # of containers
  • Add container blocklist

@cmsparks cmsparks requested review from Maximo-Guk and deloreyj May 1, 2025 14:41
this.server.tool(
'container_files_list',
'List working directory file tree. This just reads the contents of the current working directory',
{},
Copy link
Member

Choose a reason for hiding this comment

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

Cursor doesn't play well with empty object in tool calls

@cmsparks cmsparks force-pushed the csparks/docker-image-fixes branch from 5d8d3be to 6d29355 Compare May 1, 2025 16:03
@cmsparks cmsparks merged commit 3278c2f into main May 1, 2025
3 of 6 checks passed
@cmsparks
Copy link
Collaborator Author

cmsparks commented May 1, 2025

so the README linked from the blogpost is accurate i yolo merged

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.

2 participants