-
-
Notifications
You must be signed in to change notification settings - Fork 39
feat: Comprehensive MCP implementation achieving full Claude Code IDE parity #30
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
thatguyinabeanie
wants to merge
58
commits into
greggh:main
Choose a base branch
from
thatguyinabeanie:main
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 27 commits
Commits
Show all changes
58 commits
Select commit
Hold shift + click to select a range
cffe2e0
save
thatguyinabeanie b876f8e
native lua neovim mcp server
thatguyinabeanie 47f9fe1
save
thatguyinabeanie d90918a
tests
thatguyinabeanie 9a6ebac
tests/
thatguyinabeanie 42583fb
.gitignore
thatguyinabeanie 487cb25
cleanup docs
thatguyinabeanie f0d3545
feat: add configurable Claude CLI path support with robust detection
thatguyinabeanie d762e65
feat: add project tree helper with TDD approach
thatguyinabeanie 2f5576a
feat: implement safe window toggle to prevent process interruption
thatguyinabeanie c72f8e8
Enhance documentation and implementation for Claude Code MCP integration
thatguyinabeanie 007ea84
feat: implement MCP integration with detailed documentation and serve…
thatguyinabeanie 725be5d
chore: update GitHub Actions checkout action to v4
thatguyinabeanie 42a3fff
feat: add file reference shortcut to README for Claude prompt input
thatguyinabeanie 6fa8d5c
feat: implement MCP server CLI entry with integration tests
thatguyinabeanie a9d95ed
feat: enhance MCP server CLI with remote and shell functionality
thatguyinabeanie 2be4f92
feat: add MCP server Ex commands and integration tests
thatguyinabeanie a767e27
feat: add plugin contract tests for claude-code.nvim
thatguyinabeanie 38232a3
save
thatguyinabeanie 14c6555
fixing tests little bylittle
thatguyinabeanie 879f877
fix: update Safe Window Toggle tests for improved accuracy
thatguyinabeanie fc8d525
fix specs
thatguyinabeanie 20a3c4a
fix: resolve all test failures and eliminate test pollution
thatguyinabeanie 306fae2
feat: implement comprehensive PR #30 review feedback with TDD approach
thatguyinabeanie e1cf8b8
feat: complete 100% of PR #30 review items with markdown formatting f…
thatguyinabeanie 211c55c
feat: add comprehensive tutorials documentation for claude-code.nvim
thatguyinabeanie b216286
docs: add comprehensive comments to complex code sections
thatguyinabeanie df118b9
fix: disable startup notifications by default
thatguyinabeanie 0e31904
feat: test against multiple Lua versions (5.1, 5.3, 5.4) in CI workflows
thatguyinabeanie 6b1c2a7
feat: add LuaJIT versions to CI test matrix
thatguyinabeanie ae3f2ec
fix: resolve MCP server CI test failures
thatguyinabeanie 9b8423d
Ci fixes (#1)
thatguyinabeanie ae5c1e0
feat: add MCP debug mode support (#2)
thatguyinabeanie 059dd36
fix: update tests for mcp-neovim-server integration
thatguyinabeanie b43778f
docs: clarify mapping between MCP resource URIs and config keys
thatguyinabeanie d78c3e0
fix: resolve CI test failures and improve test infrastructure
thatguyinabeanie 741c4e1
fix: resolve MCP server lag and add missing commands (#3)
thatguyinabeanie 733a628
feat: make CLI detection notifications configurable (#4)
thatguyinabeanie e19a455
docs: consolidate CONTRIBUTING.md and document MCP architecture history
thatguyinabeanie f376d22
refactor: flatten MCP module structure and remove experimental code
thatguyinabeanie af005aa
docs: consolidate all documentation into official help file
thatguyinabeanie c031dc9
docs: add floating window documentation and remove CI fixes summary
thatguyinabeanie 9911364
refactor: move test scripts to scripts/ directory
thatguyinabeanie 6ddd0f6
cleanup: remove duplicate test_mcp.sh from root directory
thatguyinabeanie 503d328
refactor: update tests for external mcp-neovim-server architecture
thatguyinabeanie 66f42f6
refactor: remove bin/ directory and simplify MCP integration
thatguyinabeanie 1cb5c94
fix: update test script and validation for new MCP architecture
thatguyinabeanie c5dd05a
fix: correct MCP module import in commands.lua
thatguyinabeanie 9f6f4ce
fix: update all MCP module imports from claude-code.mcp to claude-cod…
thatguyinabeanie e6fb0a7
Fix: Resolve CI failures and update tests
thatguyinabeanie 2dd6a39
Fix: Unclosed code block in project-tree-helper.md
thatguyinabeanie f8b8ca4
Fix: Add GEMINI.md to .gitignore and fix unclosed code block in proje…
thatguyinabeanie 97b1714
Fix: Sanitize buffer names to handle path separators
thatguyinabeanie eb57d80
Fix: Unfinished string in terminal.lua
thatguyinabeanie a5ce92b
Fix: Resolve syntax errors and duplicate content in terminal.lua
thatguyinabeanie 997a35a
claude.md
thatguyinabeanie 7983247
Merge upstream changes: floating window support and cleanup
thatguyinabeanie 8995a3f
Fix post-merge issues: restore missing functions and update tests
thatguyinabeanie File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,33 @@ | ||
{ | ||
"permissions": { | ||
"allow": [ | ||
"WebFetch(domain:docs.anthropic.com)", | ||
"Bash(claude mcp)", | ||
"Bash(claude mcp:*)", | ||
"Bash(npm install:*)", | ||
"Bash(npm run build:*)", | ||
"Bash(./test_mcp.sh)", | ||
"Bash(claude --mcp-debug \"test\")", | ||
"Bash(./bin/claude-code-mcp-server:*)", | ||
"Bash(claude --mcp-debug \"test connection\")", | ||
"Bash(lua tests:*)", | ||
"Bash(nvim:*)", | ||
"Bash(claude --version)", | ||
"Bash(timeout:*)", | ||
"Bash(./scripts/test_mcp.sh:*)", | ||
"Bash(make test:*)", | ||
"Bash(lua:*)", | ||
"Bash(gh pr view:*)", | ||
"Bash(gh api:*)", | ||
"Bash(git push:*)", | ||
"Bash(git commit -m \"$(cat <<'EOF'\nfeat: implement safe window toggle to prevent process interruption\n\n- Add safe window toggle functionality to hide/show Claude Code without stopping execution\n- Implement process state tracking for running, finished, and hidden states \n- Add comprehensive TDD tests covering hide/show behavior and edge cases\n- Create new commands: :ClaudeCodeSafeToggle, :ClaudeCodeHide, :ClaudeCodeShow\n- Add status monitoring with :ClaudeCodeStatus and :ClaudeCodeInstances\n- Support multi-instance environments with independent state tracking\n- Include user notifications for process state changes\n- Add comprehensive documentation in doc/safe-window-toggle.md\n- Update README with new window management features\n- Mark enhanced terminal integration as completed in roadmap\n\nThis addresses the UX issue where toggling Claude Code window would \naccidentally terminate long-running processes.\n\n🤖 Generated with [Claude Code](https://claude.ai/code)\n\nCo-Authored-By: Claude <[email protected]>\nEOF\n)\")", | ||
"Bash(/Users/beanie/source/claude-code.nvim/fix_mcp_tests.sh)", | ||
"Bash(gh pr list:*)", | ||
"Bash(./scripts/test.sh:*)", | ||
"Bash(gh pr comment:*)", | ||
"Bash(stylua:*)" | ||
], | ||
"deny": [] | ||
}, | ||
"enableAllProjectMcpServers": true | ||
} |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🛠️ Refactor suggestion
Extract complex git commit command to improve maintainability.
The git commit command is extremely long and contains complex embedded content that makes the JSON file difficult to read and maintain. Consider extracting this to a separate script file.
Create a separate commit script and reference it:
Then create
scripts/commit-safe-toggle.sh
with the commit message content.📝 Committable suggestion
🤖 Prompt for AI Agents