Skip to content

feat: make git commits optional with no_commit config setting #304

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

peterlimg
Copy link

@peterlimg peterlimg commented May 24, 2025

Fix #303

This fix is generated via vibe coding and not fully tested to cover all edge cases. Anyway, it works good for me. Before the PR is merged, you can try this feature by change the mcp config to:

{
  "mcpServers": {
    "codemcp": {
      "command": "/usr/local/bin/uvx",
      "args": [
        "--from",
        "git+https://github.com/peterlimg/codemcp@main",
        "codemcp"
      ]
    }
  },
  "globalShortcut": "Shift+Space"
}

@ezyang
Copy link
Owner

ezyang commented May 24, 2025

Hi! Can you send the prompts you used? (Full transcripts even better.) Thanks!

@peterlimg
Copy link
Author

Hi @ezyang, it's some simple prompts I do on Windsurf with Claude 3.7.

Check the code and see if we can make it to support no git commit

That prompt basically did all the changes in commit 4a40a0f

Then I can see files will not be committed automatically, but one issue was that when files were untracked (not committed), updating them in new chat with codemcp will fail. From the error logs I can see he write_file, edit_file failed. So did another prompt to fix it:

write_file, edit_file requires git commit, fix it to do not rely on it

This prompt did changes on 66885cb

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.

Option to disable auto-commit in codemcp
3 participants