-
Notifications
You must be signed in to change notification settings - Fork 4
add bigfile.nvim #82
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
add bigfile.nvim #82
Conversation
WalkthroughThe bigfile.nvim plugin is integrated into the Nobbz Neovim configuration. It adds lazy-loading on DeferredUIEnter with a 512 MiB threshold, disabling features like LSP, treesitter, and syntax highlighting for large files. Changes span the Nix package declaration, a new Lua configuration module, and registration in the main init. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~8 minutes
Possibly related PRs
Poem
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches
🧪 Generate unit tests (beta)
Tip 📝 Customizable high-level summaries are now available!You can now customize how CodeRabbit generates the high-level summary in your pull requests — including its content, structure, tone, and formatting.
Example:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
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.
Pull Request Overview
This PR adds the bigfile.nvim plugin to improve editor performance when working with large files by automatically disabling resource-intensive features like LSP, treesitter, and syntax highlighting for files larger than 512 MiB.
Key changes:
- Integrates bigfile.nvim plugin with configuration to disable features for large files
- Adds the plugin to the Nix package dependencies
- Loads the bigfile module early in the initialization sequence
Reviewed Changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.
| File | Description |
|---|---|
| plugins/nobbz/lua/nobbz/init.lua | Adds early loading of bigfile module before other editor features |
| plugins/nobbz/lua/nobbz/bigfile.lua | Configures bigfile.nvim with 512 MiB threshold and disables multiple features for performance |
| nvim.nix | Adds bigfile-nvim package dependency to the Nix build configuration |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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.
Actionable comments posted: 1
🧹 Nitpick comments (1)
plugins/nobbz/lua/nobbz/bigfile.lua (1)
7-7: Improve comment clarity.The comment referencing
<### Overriding the detection of big files>appears to be copied from documentation and is not helpful in this context.Apply this diff to improve the comment:
- pattern = { "*", }, -- autocmd pattern or function see <### Overriding the detection of big files> + pattern = { "*" }, -- apply to all files
📜 Review details
Configuration used: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (3)
nvim.nix(1 hunks)plugins/nobbz/lua/nobbz/bigfile.lua(1 hunks)plugins/nobbz/lua/nobbz/init.lua(1 hunks)
🧰 Additional context used
📓 Path-based instructions (1)
plugins/**/*.lua
⚙️ CodeRabbit configuration file
plugins/**/*.lua: Please ensure all of the following:
- Plugins are lazyloaded if possible through
nobbz.lazylocalhas to be used- coding follows the luarocks styleguide, but with 2 spaces indentation
Files:
plugins/nobbz/lua/nobbz/bigfile.luaplugins/nobbz/lua/nobbz/init.lua
🧠 Learnings (2)
📓 Common learnings
Learnt from: NobbZ
Repo: NobbZ/nobbz-vim PR: 0
File: :0-0
Timestamp: 2025-04-12T09:24:45.451Z
Learning: NobbZ uses a homebrewn shallow abstraction over `lz.n` rather than using lazy.nvim directly. References to Lazy.nvim's specific features should be avoided in reviews of NobbZ's repository.
📚 Learning: 2025-04-12T09:24:45.451Z
Learnt from: NobbZ
Repo: NobbZ/nobbz-vim PR: 0
File: :0-0
Timestamp: 2025-04-12T09:24:45.451Z
Learning: NobbZ uses a homebrewn shallow abstraction over `lz.n` rather than using lazy.nvim directly. References to Lazy.nvim's specific features should be avoided in reviews of NobbZ's repository.
Applied to files:
plugins/nobbz/lua/nobbz/bigfile.lua
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)
- GitHub Check: Agent
🔇 Additional comments (2)
plugins/nobbz/lua/nobbz/init.lua (1)
34-34: LGTM!The submodule load follows the existing pattern correctly and is positioned appropriately among the other submodule loads.
plugins/nobbz/lua/nobbz/bigfile.lua (1)
5-18: I need to identify which bigfile.nvim is used in the codebase. Let me check the plugin configuration and dependencies.Based on my research, the documented default features for bigfile.nvim are: "indent_blankline", "illuminate", "lsp", "treesitter", "syntax", "matchparen", "vimopts", and "filetype". The configuration in the review exactly matches these documented feature names from LunarVim/bigfile.nvim.
All feature names in the code are valid and correctly correspond to the plugin's API. No invalid feature names were found.
Summary by CodeRabbit