Merge 78a52d75ca into e77632e4eb

flake.lock

@@ -1152,15 +1152,15 @@
     "plugin-nvim-metals": {
       "flake": false,
       "locked": {
-        "lastModified": 1728295172,
-        "narHash": "sha256-ja/+MNxZ3H9io9jDwm5rhE6iKNi86a22eCOY75g19O8=",
-        "owner": "scalameta",
+        "lastModified": 1728033462,
+        "narHash": "sha256-XqOkZ/5eex1mrNUf6vrJjDjibBbb0bSMYSciKAoVD0A=",
+        "owner": "ksonj",
         "repo": "nvim-metals",
-        "rev": "f861db9fda55939797ac1b05238c49b0dcdc3bdb",
+        "rev": "4d86c32f6c028de197e28dab0f6a0250c0d1c16a",
         "type": "github"
       },
       "original": {
-        "owner": "scalameta",
+        "owner": "ksonj",
         "repo": "nvim-metals",
         "type": "github"
       }

flake.nix

@@ -207,7 +207,7 @@
     };
 
     plugin-nvim-metals = {
-      url = "github:scalameta/nvim-metals";
+      url = "github:ksonj/nvim-metals";
       flake = false;
     };
 

modules/plugins/languages/scala.nix

@@ -4,17 +4,23 @@
   lib,
   ...
 }: let
-  inherit (lib) types optionalString mkPackageOption;
+  inherit (lib) types optionalString;
   inherit (lib.modules) mkIf mkMerge;
   inherit (lib.nvim.types) mkGrammarOption;
   inherit (lib.nvim.binds) mkMappingOption;
   inherit (lib.options) mkOption mkEnableOption;
   inherit (lib.nvim.dag) entryAfter;
 
-  listCommandsAction =
-    if config.vim.telescope.enable
-    then ''require("telescope").extensions.metals.commands()''
-    else ''require("metals").commands()'';
+  mkBinding = binding: "vim.api.nvim_buf_set_keymap(bufnr, 'n', '${binding.value}', '<cmd>lua ${binding.action}<CR>', {noremap=true, silent=true, desc='${binding.description}'})";
+
+  listCommandsBinding = {
+    value = cfg.lsp.extraMappings.listCommands;
+    description = "List Metals commands";
+    action =
+      if config.vim.telescope.enable
+      then ''require("telescope").extensions.metals.commands()''
+      else ''require("metals").commands()'';
+  };
 
   cfg = config.vim.languages.scala;
 
@@ -23,22 +29,23 @@
 in {
   options.vim.languages.scala = {
     enable = mkEnableOption "Scala language support";
-
     treesitter = {
       enable = mkEnableOption "Scala treesitter" // {default = config.vim.languages.enableTreesitter;};
       package = mkGrammarOption pkgs "scala";
     };
-
     lsp = {
       enable = mkEnableOption "Scala LSP support (metals)" // {default = config.vim.languages.enableLSP;};
-      package = mkPackageOption pkgs "metals" {
-        default = ["metals"];
+      package = mkOption {
+        type = types.package;
+        default = pkgs.metals;
+        description = ''
+          metals package to use
+        '';
       };
       extraMappings = {
         listCommands = mkMappingOption "List Metals commands" "<leader>lc";
       };
     };
-
     dap = {
       enable = mkOption {
         description = "Scala Debug Adapter support";
@@ -72,7 +79,6 @@ in {
       };
     };
   };
-
   config = mkIf cfg.enable (
     mkMerge [
       (mkIf cfg.treesitter.enable {
@@ -82,12 +88,13 @@ in {
       (mkIf (cfg.lsp.enable || cfg.dap.enable) {
         vim = {
           startPlugins = ["nvim-metals"];
-          pluginRC.nvim-metals = entryAfter ["lsp-setup"] ''
+          luaConfigRC.nvim-metals = entryAfter ["lsp-setup"] ''
+            -- Scala nvim-metals config
             local metals_caps = capabilities  -- from lsp-setup
 
             local attach_metals_keymaps = function(client, bufnr)
               attach_keymaps(client, bufnr)  -- from lsp-setup
-              vim.api.nvim_buf_set_keymap(bufnr, 'n', '${cfg.lsp.extraMappings.listCommands}', '<cmd>lua ${listCommandsAction}<CR>', {noremap=true, silent=true, desc='Show all Metals commands'})
+              ${mkBinding listCommandsBinding}
             end
 
             metals_config = require('metals').bare_config()
@@ -118,18 +125,14 @@ in {
               }
             )
 
+
             -- without doing this, autocommands that deal with filetypes prohibit messages from being shown
             vim.opt_global.shortmess:remove("F")
 
-            local lsp_group = vim.api.nvim_create_augroup('lsp', { clear = true })
-
-            vim.api.nvim_create_autocmd('FileType', {
-                group = lsp_group,
-                pattern = {'java', 'scala', 'sbt'},
-                callback = function()
-                    require('metals').initialize_or_attach(metals_config)
-                end,
-            })
+            vim.cmd([[augroup lsp]])
+            vim.cmd([[autocmd!]])
+            vim.cmd([[autocmd FileType java,scala,sbt lua require('metals').initialize_or_attach(metals_config)]])
+            vim.cmd([[augroup end]])
           '';
         };
       })