Skip to content

feat: add oil.nvim support for file selection #27

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 4 commits into from
Jun 17, 2025

Conversation

Peeeaje
Copy link
Contributor

@Peeeaje Peeeaje commented Jun 9, 2025

Summary

  • Add oil.nvim integration for file selection with @-mention
  • Support both visual selection and single file under cursor
  • Handle directories, files, and symbolic links properly

Changes

  • README.md: Add oil.nvim to the list of supported tree explorers
  • integrations.lua: Add _get_oil_selection() function to handle oil.nvim file selection
  • visual_commands.lua: Add oil.nvim detection and processing in visual mode

🤖 Generated with Claude Code

Peeeaje and others added 2 commits June 9, 2025 21:35
- Add oil.nvim integration to support file selection with @-mention
- Support both visual selection and single file under cursor
- Handle directories, files, and symbolic links properly
- Add comprehensive unit tests for oil.nvim integration
- Update README documentation to include oil.nvim support

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
- Add oil filetype check in diff.lua's _find_main_editor_window()
- Add oil filetype check in open_file.lua's find_main_editor_window()
- Add oil filetype check in init.lua's ClaudeCodeSend command

These changes ensure oil.nvim buffers are properly excluded when
searching for main editor windows, preventing diff views and file
opens from appearing in the oil explorer window.

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>
@ThomasK33
Copy link
Member

Hey @Peeeaje, thanks for this PR.

Could you please investigate the CI failure related to the unused variable?
You can run those checks locally using make.

@Peeeaje
Copy link
Contributor Author

Peeeaje commented Jun 15, 2025

@ThomasK33
Sorry for the poor pr, I fix that so could you run it again? Thank you.

Copy link
Member

@ThomasK33 ThomasK33 left a comment

Choose a reason for hiding this comment

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

Can you please update lines 59 and 216 in the README.md to include the oil filetype?
Also, please rebase onto main and update the dev-config.lua to include the oil file type too.

@ThomasK33 ThomasK33 added this pull request to the merge queue Jun 17, 2025
Merged via the queue into coder:main with commit 55aba11 Jun 17, 2025
5 checks passed
ThomasK33 added a commit that referenced this pull request Jun 18, 2025
This release includes significant feature additions and bug fixes:

**Version Updates:**
- Updated main version table in lua/claudecode/init.lua (0.1.0-alpha → 0.2.0)
- Updated client versions in scripts/claude_interactive.sh (3 locations)
- Updated ClaudeCodeNvim version in scripts/lib_claude.sh
- Removed prerelease flag for stable release

**Documentation:**
- Added comprehensive CHANGELOG.md with v0.2.0 release notes
- Documented all merged PRs since v0.1.0 with proper references
- Updated CLAUDE.md with detailed release process documentation
- Fixed diff keymap references in README.md (<leader>da → <leader>aa)

**Features Added (since v0.1.0):**
- Diagnostics integration (#34)
- File explorer support for oil.nvim, nvim-tree, neotree (#27, #22)
- Enhanced terminal management with ClaudeCodeFocus command (#40)
- Auto terminal provider detection (#36)
- Customizable diff keymaps via LazyVim spec (#47)

**Bug Fixes:**
- Terminal focus errors when buffer hidden (#43)
- Improved diff acceptance behavior (#41)
- Fixed syntax highlighting in proposed diff view (#32)
- Visual selection range handling improvements (#26)
- Native terminal bufhidden behavior (#39)

All code quality checks pass and documentation is updated for maintainability.

Change-Id: I0e4e7c9bae98df922356dc8b8aa0acd7e8293a48
Signed-off-by: Thomas Kosiewski <tk@coder.com>
@ThomasK33 ThomasK33 mentioned this pull request Jun 18, 2025
ThomasK33 added a commit that referenced this pull request Jun 18, 2025
This release includes significant feature additions and bug fixes:

**Version Updates:**
- Updated main version table in lua/claudecode/init.lua (0.1.0-alpha → 0.2.0)
- Updated client versions in scripts/claude_interactive.sh (3 locations)
- Updated ClaudeCodeNvim version in scripts/lib_claude.sh
- Removed prerelease flag for stable release

**Documentation:**
- Added comprehensive CHANGELOG.md with v0.2.0 release notes
- Documented all merged PRs since v0.1.0 with proper references
- Updated CLAUDE.md with detailed release process documentation
- Fixed diff keymap references in README.md (<leader>da → <leader>aa)

**Features Added (since v0.1.0):**
- Diagnostics integration (#34)
- File explorer support for oil.nvim, nvim-tree, neotree (#27, #22)
- Enhanced terminal management with ClaudeCodeFocus command (#40)
- Auto terminal provider detection (#36)
- Customizable diff keymaps via LazyVim spec (#47)

**Bug Fixes:**
- Terminal focus errors when buffer hidden (#43)
- Improved diff acceptance behavior (#41)
- Fixed syntax highlighting in proposed diff view (#32)
- Visual selection range handling improvements (#26)
- Native terminal bufhidden behavior (#39)

All code quality checks pass and documentation is updated for maintainability.

Change-Id: I0e4e7c9bae98df922356dc8b8aa0acd7e8293a48
Signed-off-by: Thomas Kosiewski <tk@coder.com>
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