Nvim-related news — Activity since 2023-08-30

• nvim-treesitter/nvim-treesitter repo issues, pull requests, and releases

  • Avoid non-standard parser installation #2300 [clason] bug enhancement

    As pointed out in #2295 (comment), nvim-treesitter can install parsers in a directory that is not inside its own...

    • clason: Already implemented on main (nvim-treesitter 1.0).

  • Show registered modules in health check #2883 [clason] enhancement

    For debugging issues without provided config, it would be very useful if :checkhealth could list the enabled (and/or...

    • clason: Obsolete with the removal of modules for nvim-treesitter 1.0.

  • Add a way to reload a parser without restarting Neovim #5328 [darthdeus] enhancement

    Is your feature request related to a problem? Please describe. When I work on a treesitter grammar a new language I...

    • clason: Agree; this is not safe nor reasonable to support officially. In any case, parser management is...

    • theHamsta: When you do a TSInstall it will automatically unload the old grammar and load the new one. This...

  • feat: add NQC #5334 [amaanq]

    Not Quite C 😄

    • lucario387: I don't understand what's going on here but something's going to master branch

    • ObserverOfTime: HolyC when?

    • amaanq: HolyC when? One day... 😵

  • feat(doxygen): update queries from upstream #5337 [amaanq]

    • clason

    • ObserverOfTime

  • Error in decoration provider treesitter/highlighter.win #5332 [qgates] bug

    Describe the bug Using telescope or opening file directly, in nvim running on 64-bit windows 10, produces the...

    • qgates: @clason apologies but this may be unrelated to #3092. I have tried everything suggested in that...

    • qgates: @clason Ok, some progress... The problem appears to be that nvim-treesitter, or one of its...

    • clason: The bash detection is intentional via a codeblock injection: npx degit antfu/vitesse...

  • nvim_treesitter#statusline() not work as expected for java #5338 [yimingwangdell] bug

    Describe the bug class A extends B implements C -> @ Override it only show "@ Override" as method name for...

    • clason: use tree-sitter-context; that function is deprecated and will be removed in 1.0 (you might also...

    • yimingwangdell: @clason any document about how to config lualine with this feature of treesitter?

    • clason: No, you'd have to roll your own.

  • fix(ecma_injections): limit function names to alphanumeric characters only #5339 [Hippo0o]

    fixes #5316

    • lucario387

  • ecma(javascript) injection has massive performance hits when $(`...`) is present #5316 [Hippo0o] bug

    Describe the bug The parser experiences significant slowdown when a JavaScript file contains a function call with a...

    • amaanq: @amaanq it would need something like this: (#lua-match? @injection.language...

  • feat: add Kconfig #5336 [amaanq]

    Linux Kernel Configuration

    • amaanq: Something other than type would be nice for entry titles, wasn't sure what'd be good

    • ObserverOfTime

      • ObserverOfTime: Maybe @text.title?

        • amaanq: prompt is a string-like title, more like a header for the symbol ones that is good though

        • ObserverOfTime: Are you sure it shouldn't be the other way around? "General setup" seems like @text.title to...

        • amaanq: Makes a lot more sense, thought about it w/ a fresh mind, I picked @symbol

        • amaanq: nvm, constant is nicer for consistency w/ other languages' screaming snake case identifiers

      • ObserverOfTime: Suggested change (symbol) @variable ...

        • amaanq: symbol is kconfig's "notation" for anything used as an identifier pretty much, titles,...

        • amaanq: This might help: string after menu is a prompt, identifiers after config are symbols

    • ObserverOfTime: Move this down with the rest?

  • feat: add snakemake #4302 [osthomas]

    Hi! Snakemake is a workflow management system based on Python: https://snakemake.readthedocs.io/en/stable/ The...

    • amaanq: @clason needs your approval (It's a bit annoying I can't merge w/ mine if someone else requested changes)

  • LaTeX: match only text #1558 [stsewd]

    Looks like what you actually want to match is the text inside, not the whole thing that includes {}, for...

    • stsewd: Looks like this is fixed in master.

  • E5113: Error while calling lua chunk: vim/shared.lua:0: opt[proto: 0x7f1b9fb867c0]: expected table, got nil #5115 [cronyakatsuki] bug

    Describe the bug I'm using lazy.nvim and sometimes when opening a file I get this error with treesitter Failed to...

    • Blebesquefee: Did you tried this command ? :TSUpdate It works for me

  • Tracking issue: language support requests #2282 [clason] enhancement good first issue help wanted tracking

    This is a tracking issue collecting requests for new languages to add to nvim-treesitter. If you would like to add a...

    • cpkio: HURL Run and test HTTP requests with plain...

    • snowmang1: Cabal A language used (at least to my knowledge) in config description, specifically for the...

    • xixiaofinland: Salesforce languages Apex Apex is the backend programming language in Salesforce platform, its...

  • Feat: Add unison #5340 [tapegram]

    Attempting to add unison Appreciate this tool thanks for the help!

    • lucario387: Please follow the captures listed in contributing.md

    • lucario387: Can't the match, with, cases string themselves be used instead of nodes?

      • tapegram: I tried with these as strings and it breaks, though I'm having trouble seeing the details of the...

    • lucario387: I will take this as an example of following CONTRIBUTING.md Only @variable, not the suffix .other.member

    • clason: Suggested change branch =...

    • amaanq

      • amaanq: If you have parser.c requires_generate is not needed Also, do not include maybe.c, if scanner.c...

        • tapegram: Removed maybe.c! Earlier the tests were failing in this PR until I added...

        • lucario387: It no longer fails. 🎉

      • amaanq: forall and do sound like @repeat equals sounds like @keyword.operator

        • tapegram: forall - is a type level thing in unison...

        • amaanq: Thanks, not familiar w/ the language so I just go off of preconceived notions about certain...

      • amaanq: does let declare only functions or a variable/other stuff?

        • tapegram: let opens a "block" wherever an expression is expected...

        • tapegram: Going to move it into the keyword block instead of keyword.function if that makes sense

        • amaanq: yeah that's good

      • amaanq: unnecessary brackets

      • amaanq: Suggested change (doc_block)...

      • amaanq: or might be @keyword.operator

      • amaanq: not sure what the wrapped parenthesis indicates, there's no parent node to these two

        • lucario387: Can these kind of patterns deduct consecutive nodes, or will they always be resolved to 2...

        • tapegram: Removed the outer parens from this. Also tried the outer parens on the next line (record_field...

        • amaanq: Ah I see, my bad. Were you intending to match two items at the root level? Specifying a parent...

        • amaanq: Can these kind of patterns deduct consecutive nodes, or will they always be resolved to 2...

    • ObserverOfTime

      • ObserverOfTime: @type.qualifier if they don't affect memory.

      • ObserverOfTime: Suggested change (comment) @comment ...

        • ObserverOfTime: Still missing the injection.

        • tapegram: my b, just added

      • ObserverOfTime: Suggested change (">") ...

  • refactor: remove `include-children` from queries #5226 [lucario387] NEXT

    As the injection range issue has been resolved, there isn't any reason to keep the workarounds anymore I will do...

  • fix: auto install #4911 [lewis6991] NEXT

  • injections(nix): dynamic language injection via comments #4658 [figsoda] NEXT

    continuation of #3902 since neovim/neovim#21548 is in stable requires neovim/neovim#23015 to work correctly since...

  • docs: fix typo in README.md #5341 [socherbyc]

    • ObserverOfTime

  • Invalid node type at position for language bash #5345 [Maritsu] bug

    Describe the bug Upon trying to do anything in a bash script, NVIM errors out, saying "invalid node type at position...

    • clason: Duplicate of #3092

  • feat(c_sharp): add indent query #5344 [vlada-dudr]

    I have been using it for some months now and seems to mostly work ok.

    • lucario387: Is there any reason why only switch sections with 2 cases have branches, and only the second one...

      • vlada-dudr: Because: switch (var) { case "a": foo(); break; } is one switch_section starting on case:...

      • lucario387: Couldn't it be something like ; following current indent query for C (switch_body)...

      • vlada-dudr: wouldn't it make it like this? switch (var) { case "b": case "a": foo(); break; } Thus without...

      • lucario387: it's our current version for C. If that style is not a standard for C# then you can ignore this.

      • lucario387: Your queries are fine :D. I don't think this case_switch is an issue. I have done the same for...

    • lucario387: I can see a lot of (parent body: (block ... @capture)) here. Can it be replaced with only (block ...)

      • lucario387: Personal guess: You need this because you capture the parent node with @indent.begin, which is...

      • vlada-dudr: I wish I remember the reason. But I clearly do remember that I couldn't make it work properly...

      • lucario387: At its current state, I fear this is too messy for me, you, future you and others to maintain...

      • vlada-dudr: hm, I will try what happens

      • vlada-dudr: I see why I might do it this way. You may have most of these conditionals and loops not being...

    • lucario387: Imo, Some of these, from looking at the AST, can be replaced with a single (block) capture,

      • vlada-dudr: I did the whole thing incrementally over time, so i don't remember exactly all the reasons for...

      • vlada-dudr: It is because of indenting inside of those statements, like: if ( with() && many() ...

      • lucario387: Ok, so another language like dart, swift and friends where the conditions of the if statement...

    • lucario387: Why these and not (block) @indent.begin? Is there a case where the body field is not a block and...

      • lucario387: Okay I can see better why this is declared as such. I'd still prefer to capture the node...

      • vlada-dudr: yeah, classes, structs and similar have declaration_list in their bodies, but e.g. methods and...

    • lucario387: The if_statement should be similar enough to C that I suggest you can use the C's indent queries...

      • vlada-dudr: I remember I was looking onto C's version, but I understood only like half of it, so i did this,...

    • lucario387: Using your example codeblock, I got this result with the if statement if ( with() &&...

      • vlada-dudr: hm, weird I am getting this: if ( with() && many() && conditions() ) { foo(); }

    • lucario387: Edit: Consider this obsolete.

    • vlada-dudr: Can you point, how to run the tests? I could probably add some of them to make it easier to...

    • lucario387: Can you point, how to run the tests For this PR, you'll need to add example c_sharp files to...

    • vlada-dudr: That's the part which I understood, but I have no clue how to run the thing

    • lucario387: oh, just be inside the git repo, and do a ./scripts/run_tests.sh indent/c_sharp_spec.lua and see...

    • clason: No, you also need to have plenary installed and in your standard runtimepath. (I've said it...

    • vlada-dudr: Thanks @clason, that's the missing part - I expected that test machinery would grab it itself,...

    • theHamsta: Can you add some tests? At least there should be some test files on how this is expected to work.

      • vlada-dudr: Yeah, I started writing something. But I probably don't get how it is supposed to work. I...

• nvim-tree/nvim-tree.lua repo issues, pull requests, and releases

  • Marks / bookmarks symbol is misaligned if renderer.group_empty = true #2392 [tjex] bug regression PR please reproduced

    Description When marking a file, and renderer.group_empty = true the mark symbol gets set one line above where the...

    • alex-courtis: regression since d8b154c

    • alex-courtis: I would be most grateful for a PR to resolve this: wiki: Development That looks about the right...

  • feat: add api.marks.bulk.trash #2391 [pr4th4m]

    API to trash bulk files using external trash cli or similar Tested with OS: MacOS...

    • alex-courtis: Many thanks for your contribution. We do need some work: remove unnecessary refresh handle...

      • alex-courtis: No changes here, just a refactor.

      • alex-courtis: We don't need this, as trash.remove already does this.

    • alex-courtis: Many thanks for this one; trash is a useful feature.

  • git submodule/worktree watcher incorrect #2382 [alex-courtis] bug

    Description git watcher for submodules watches the .git file inside the submodule rather than the appropriate...

  • fix(#2382): git watcher handles worktrees and submodules, via --absolute-git-dir when it is available #2389 [alex-courtis]

    fixes #2382 This is clunky: using yet another map for project roots is fragile and not easily maintainable. Possible...

  • Add option to always open files from the picker in a fixed split (e.g. left) #2388 [darthdeus] feature request

    Can this functionality be implemented utilising API? Looking at the docs I don't think so, seems this needs to hook...

    • alex-courtis: It looks like you have all the tools available to achieve your goals. Closing this one; please...

  • Nvim-Tree Keeps Creating "Zombie" Buffers? #2383 [eyalz800] bug reproduced

    Description Recently I found that in my workflow there are many unnamed buffers created, I tried to isolate the...

    • alex-courtis: We could do something like vim.api.nvim_buf_get_name(bufnr) == "" and not...

    • alex-courtis: I've had a go at this and the complexity and risk is too great: detecting the no name new is...

  • User should be able to change default icon even if nvim-web-devicons is available #2361 [2ax3ax5] feature request awaiting feedback

    I'd like to set the nvim-tree.renderer.icons.glyphs.default to a desired value even if nvim-web-devicons are...

    • alex-courtis: The above change should enable your desired behaviour. Please comment if you experience issues.

  • quit_on_open is false, but still close file explorer when open file #2379 [zshnb] bug awaiting feedback

    Description i set quit_on_open = false, but when i open a file, the panel still closed Neovim version nvim...

  • Allow specific statuscolumn in view struct. #2394 [gukz] feature request upstream: neovim

    Neovim has support the new feature: statuscolumn neovim/neovim#20621 please allow user config this property and set...

    • alex-courtis: related: #1510 missing diagnostic placement

    • alex-courtis: This is fantastic news! It allows more flexibility for sign placement. I don't think this is...

  • `OptionSet` is triggered when `tree` open in some cases. #2334 [peter-lyr] OS Windows WSL awaiting feedback OS Windows PowerShell

    Description I like the tree to stay open, but one day I run nvim-qt in a somewhat underperforming Windows 10...

    • alex-courtis: This isn't affecting functionality and I'm not sure that it's something we can resolve in...

  • indent marker at level 0 when using `inline_arrows` #2315 [anna328p] bug reproduced

    Is this a question? No Can this functionality be implemented utilising API? Don't think so Is your feature request...

  • Option To Prevent Buffer Ejection #2218 [alex-courtis] feature request PR please QOL

    nvim-tree always ejects a buffer that opens in the nvim-tree window. This could be disabled via an option like...

    • alex-courtis: Fixed #2344

  • Set a ratio for width and height of floating window #2195 [Johnson0907] feature request PR please

    I want the configuration to allows user to set ratio for width and height of floating window. Refer to the...

    • alex-courtis: :help VimResized looks like the event to use - that's how telescope is doing it. I reckon you...

  • api.fs.rename_full #2148 [alex-courtis] PR please API

    Discussed in #2145 Originally posted by mosheavni April 18, 2023 I migrated to the on_attach function, but I cannot...

  • A git-dirty folder is (inconsistently) highlighted by `NvimTreeGitStaged` instead of `NvimTreeGitDirty` #2048 [dasupradyumna] bug awaiting feedback

    Description Hello, I use nvim-tree with git highlight files enabled and git status icons disabled. Recently, while...

    • alex-courtis: Closing due to inactivity and lack of reproducer. Please comment if you have an issues.

  • `full_rename` to create dirs as required #1982 [mangelozzi] feature request PR please QOL

    NerdTree move command is very user friendly for moving files around. The closest I have found in NerdTree is...

    • alex-courtis: full_rename is gone, however we can cover this use case in the above.

  • feat(#1917): add diagnostic highlighting and icon placement #2396 [alex-courtis]

    fixes #1917 #1510

  • feat: add renderer.icons.diagnostic_placement #2152 [baahrens]

    Attempts to close #1510 This is my first proper PR in this project, so please let me know if this is the wrong...

    • alex-courtis: superseded by #2396

  • fix(#2127) WIP: Open files with relative paths #2329 [xiantang]

    close #2127 Open files with relative paths add experimental.open.relative_path flag

    • alex-courtis: Are relative paths working when you are testing? Are you intending to complete this task?

  • NvimTree commands are not working anymore. #2384 [HicaroD] bug

    Description Commands are not working anymore. I just updated my NvimTree an hour ago, and suddenly it stopped...

    • thatanjan: Sorry for the late reply @alex-courtis. I did a clean room setup. Looks like it is a problem...

    • thatanjan: I have fixed it finally. For some reason, removing the following code solves the problem. local...

    • alex-courtis: You are calling setup twice, the first time with an empty config. Just call it once with your...

    • alex-courtis: Looks like we are all good.

  • When bulk moving, set destination based on the directory under cursor #2395 [raine] feature request

    Describe the solution you'd like When using bulk move (bmv) to move files, it would be great if the directory on the...

    • alex-courtis: I noticed this the other day. This prompt is inconsistent with other operations e.g. paste and...

    • gegoune: I don't mind breaking changes. But if we are breaking do we want to do anything else as well? Do...

  • NvimTreeOpenedFileIcon Highlight Group #2398 [LinuxIsCool] feature request

    Why is there no NvimTreeOpenedFileIcon Highlight Group? I see NvimTreeOpenedFolderIcon highlight group. But files...

created at 2023-09-02 22:04 by dinghy 1.3.0.