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

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

Conversation

Peeeaje
Copy link

@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 <[email protected]>
- 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 <[email protected]>
@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.

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