Fixed PR comments

Formatting and changelog

docs/release-notes/rl-0.8.md

@@ -192,3 +192,7 @@
 - Add formatters for go: [gofmt](https://go.dev/blog/gofmt),
   [golines](https://github.com/segmentio/golines) and
   [gofumpt](https://github.com/mvdan/gofumpt)
+
+[UltraGhostie](https://github.com/UltraGhostie)
+
+- Add [harpoon](https://github.com/ThePrimeagen/harpoon) plugin for navigation

modules/plugins/utility/harpoon/config.nix

@@ -5,8 +5,6 @@
   ...
 }: let
   inherit (lib.modules) mkIf;
-  inherit (lib.lists) optionals;
-  inherit (lib.strings) concatStrings;
   inherit (lib.nvim.binds) pushDownDefault mkKeymap;
 
   cfg = config.vim.navigation.harpoon;
@@ -23,14 +21,6 @@ in {
         setupModule = "harpoon";
         inherit (cfg) setupOpts;
 
-        before = ''
-        '';
-
-        after = ''
-          local harpoon = require("harpoon")
-          harpoon.setup({})
-        '';
-
         cmd = ["Harpoon"];
 
         keys = [

modules/plugins/utility/harpoon/default.nix

@@ -1,4 +1,4 @@
-_: {
+{
   imports = [
     ./harpoon.nix
     ./config.nix

modules/plugins/utility/harpoon/harpoon.nix

@@ -4,29 +4,6 @@
   inherit (lib.nvim.binds) mkMappingOption;
   inherit (lib.nvim.types) mkPluginSetupOption luaInline;
   inherit (lib.generators) mkLuaInline;
-  setupOptions = {
-    defaults = {
-      save_on_toggle = mkOption {
-        description = "any time the ui menu is closed then we will save the state back to the backing list, not to the fs";
-        type = bool;
-        default = false;
-      };
-      sync_on_ui_close = mkOption {
-        description = "any time the ui menu is closed then the state of the list will be sync'd back to the fs";
-        type = bool;
-        default = false;
-      };
-      key = mkOption {
-        description = "how the out list key is looked up. This can be useful when using worktrees and using git remote instead of file path";
-        type = luaInline;
-        default = mkLuaInline ''
-          function()
-            return vim.loop.cwd()
-          end
-        '';
-      };
-    };
-  };
 in {
   options.vim.navigation.harpoon = {
     mappings = {
@@ -38,8 +15,33 @@ in {
       file4 = mkMappingOption "Go to marked file 4 [Harpoon]" "<C-;>";
     };
 
-    enable = mkEnableOption "Harpoon: quick bookmarks on keybinds";
+    enable = mkEnableOption "Quick bookmarks on keybinds [Harpoon]";
 
-    setupOpts = mkPluginSetupOption "Harpoon" setupOptions;
+    setupOpts = mkPluginSetupOption "Harpoon" {
+      defaults = {
+        save_on_toggle = mkOption {
+          type = bool;
+          default = false;
+          description = ''            any time the ui menu is closed then we will save the
+                    state back to the backing list, not to the fs'';
+        };
+        sync_on_ui_close = mkOption {
+          type = bool;
+          default = false;
+          description = ''            any time the ui menu is closed then the state of the
+                    list will be sync'd back to the fs'';
+        };
+        key = mkOption {
+          type = luaInline;
+          default = mkLuaInline ''
+            function()
+              return vim.loop.cwd()
+            end
+          '';
+          description = ''            how the out list key is looked up. This can be useful
+                    when using worktrees and using git remote instead of file path'';
+        };
+      };
+    };
   };
 }