fix: replace_at causes duplicates

lua/harpoon/config.lua

@@ -149,6 +149,7 @@ function M.get_default_config()
                 elseif list_item_a == nil or list_item_b == nil then
                     return false
                 end
+
                 return list_item_a.value == list_item_b.value
             end,
 

lua/harpoon/list.lua

@@ -29,13 +29,14 @@ end
 ---@param items any[]
 ---@param element any
 ---@param config HarpoonPartialConfigItem?
-local function index_of(items, element, config)
+local function index_of(items, length, element, config)
     local equals = config and config.equals
         or function(a, b)
             return a == b
         end
     local index = -1
-    for i, item in ipairs(items) do
+    for i = 1, length do
+        local item = items[i]
         if equals(element, item) then
             index = i
             break
@@ -81,8 +82,8 @@ function HarpoonList:new(config, name, items)
     return setmetatable({
         items = items,
         config = config,
-        _length = guess_length(items),
         name = name,
+        _length = guess_length(items),
         _index = 1,
     }, self)
 end
@@ -108,21 +109,31 @@ end
 ---@param item? HarpoonListItem
 function HarpoonList:replace_at(idx, item)
     item = item or self.config.create_list_item(self.config)
-    Extensions.extensions:emit(
-        Extensions.event_names.REPLACE,
-        { list = self, item = item, idx = idx }
-    )
+    local current_idx = index_of(self.items, self._length, item, self.config)
+
     self.items[idx] = item
+
+    if current_idx ~= idx then
+        self.items[current_idx] = nil
+    end
+
     if idx > self._length then
         self._length = idx
+    else
+        self._length = determine_length(self.items, self._length)
     end
+
+    Extensions.extensions:emit(
+        Extensions.event_names.REPLACE,
+        { list = self, item = item, idx = idx }
+    )
 end
 
 ---@param item? HarpoonListItem
 function HarpoonList:add(item)
     item = item or self.config.create_list_item(self.config)
 
-    local index = index_of(self.items, item, self.config)
+    local index = index_of(self.items, self._length, item, self.config)
     Logger:log("HarpoonList:add", { item = item, index = index })
 
     if index == -1 then
@@ -151,7 +162,7 @@ end
 ---@return HarpoonList
 function HarpoonList:prepend(item)
     item = item or self.config.create_list_item(self.config)
-    local index = index_of(self.items, item, self.config)
+    local index = index_of(self.items, self._length, item, self.config)
     Logger:log("HarpoonList:prepend", { item = item, index = index })
     if index == -1 then
         local stop_idx = prepend_to_array(self.items, item)
@@ -214,7 +225,7 @@ end
 
 function HarpoonList:get_by_display(name)
     local displayed = self:display()
-    local index = index_of(displayed, name, self.config)
+    local index = index_of(displayed, #displayed, name, self.config)
     if index == -1 then
         return nil
     end
@@ -232,23 +243,23 @@ function HarpoonList:resolve_displayed(displayed, length)
     local change = 0
     for i = 1, self._length do
         local v = self.items[i]
-        local index = index_of(displayed, v)
+        local index = index_of(displayed, self._length, v)
         if index == -1 then
             change = change + 1
         end
     end
 
     for i = 1, length do
         local v = displayed[i]
-        local index = index_of(list_displayed, v)
+        local index = index_of(list_displayed, self._length, v)
         if v == "" then
             new_list[i] = nil
         elseif index == -1 then
             new_list[i] = self.config.create_list_item(self.config, v)
             change = change + 1
         else
             local index_in_new_list =
-                index_of(new_list, self.items[index], self.config)
+                index_of(new_list, length, self.items[index], self.config)
 
             if index_in_new_list == -1 then
                 new_list[i] = self.items[index]

lua/harpoon/test/list_spec.lua

@@ -251,6 +251,26 @@ describe("list", function()
             { value = "threethree" },
             { value = "four" },
         }, list.items)
+
+        list:replace_at(2, { value = "one" })
+        eq(4, list:length())
+        eq({
+            nil,
+            { value = "one" },
+            { value = "threethree" },
+            { value = "four" },
+        }, list.items)
+
+        list:replace_at(5, { value = "one" })
+        eq(5, list:length())
+        eq({
+            nil,
+            nil,
+            { value = "threethree" },
+            { value = "four" },
+            { value = "one" },
+        }, list.items)
+
     end)
 
     it("resolve_displayed", function()