fix(core): stop cleanup timer before re-setup to prevent timer leak

Call cleanup.stop() unconditionally before checking auto_close config
in events.setup(), so re-setup with auto_close disabled properly
clears the previous timer.

Author: mhiro2

lua/peekstack/core/events.lua

@@ -149,8 +149,9 @@ function M.setup()
     end,
   })
 
+  local cleanup = require("peekstack.core.cleanup")
+  cleanup.stop()
   if cfg.ui.popup.auto_close and cfg.ui.popup.auto_close.enabled then
-    local cleanup = require("peekstack.core.cleanup")
     cleanup.start()
   end
 end

tests/events_spec.lua

@@ -80,6 +80,35 @@ describe("peekstack.core.events", function()
     assert.equals(1, #after)
   end)
 
+  it("stops cleanup timer when re-setup disables auto_close", function()
+    local timer_store = require("peekstack.util.timer").get_store()
+
+    config.setup({
+      ui = {
+        quick_peek = { close_events = {} },
+        popup = {
+          auto_close = {
+            enabled = true,
+            idle_ms = 300000,
+            check_interval_ms = 60000,
+            ignore_pinned = true,
+          },
+        },
+      },
+    })
+    events.setup()
+    assert.is_not_nil(timer_store.cleanup)
+
+    config.setup({
+      ui = {
+        quick_peek = { close_events = {} },
+        popup = { auto_close = { enabled = false } },
+      },
+    })
+    events.setup()
+    assert.is_nil(timer_store.cleanup)
+  end)
+
   it("falls back to default close_events when quick_peek.close_events is invalid", function()
     config.setup({
       ui = {