From 25e0d3064e03d9bcb7b502059bf79528e55fc261 Mon Sep 17 00:00:00 2001
From: J0J0 Todos <jojo@peek-a-boo.at>
Date: Sat, 17 Jan 2026 08:51:03 +0100
Subject: [PATCH 1/2] importsource: Only register listener if configured

and remove the now redundant config sanity check in suggest_removal().
---
 beetsplug/importsource.py | 29 ++++++++++++++---------------
 1 file changed, 14 insertions(+), 15 deletions(-)

diff --git a/beetsplug/importsource.py b/beetsplug/importsource.py
index e42be3f1f..67330ef1a 100644
--- a/beetsplug/importsource.py
+++ b/beetsplug/importsource.py
@@ -26,17 +26,19 @@ class ImportSourcePlugin(BeetsPlugin):
             }
         )
         self.import_stages = [self.import_stage]
-        self.register_listener("item_removed", self.suggest_removal)
-        # In order to stop future removal suggestions for an album we keep
-        # track of `mb_albumid`s in this set.
-        self.stop_suggestions_for_albums = set()
-        # During reimports (import --library) both the import_task_choice and
-        # the item_removed event are triggered. The item_removed event is
-        # triggered first. For the import_task_choice event we prevent removal
-        # suggestions using the existing stop_suggestions_for_album mechanism.
-        self.register_listener(
-            "import_task_choice", self.prevent_suggest_removal
-        )
+        # Only register removal suggestion listeners if the feature is enabled
+        if self.config["suggest_removal"]:
+            # In order to stop future removal suggestions for an album we keep
+            # track of `mb_albumid`s in this set.
+            self.stop_suggestions_for_albums = set()
+            self.register_listener("item_removed", self.suggest_removal)
+            # During reimports (import --library) both the import_task_choice and
+            # the item_removed event are triggered. The item_removed event is
+            # triggered first. For the import_task_choice event we prevent removal
+            # suggestions using the existing stop_suggestions_for_album mechanism.
+            self.register_listener(
+                "import_task_choice", self.prevent_suggest_removal
+            )
 
     def prevent_suggest_removal(self, session, task):
         if task.skip:
@@ -60,10 +62,7 @@ class ImportSourcePlugin(BeetsPlugin):
 
     def suggest_removal(self, item):
         """Prompts the user to delete the original path the item was imported from."""
-        if (
-            not self.config["suggest_removal"]
-            or item.mb_albumid in self.stop_suggestions_for_albums
-        ):
+        if item.mb_albumid in self.stop_suggestions_for_albums:
             return
 
         if "source_path" not in item:

From 82bd4887137157f5c3e44e656083fc975bdfa630 Mon Sep 17 00:00:00 2001
From: J0J0 Todos <jojo@peek-a-boo.at>
Date: Sat, 17 Jan 2026 09:10:21 +0100
Subject: [PATCH 2/2] importsource: Test listeners based on config

---
 test/plugins/test_importsource.py | 32 +++++++++++++++++++++++++++++++
 1 file changed, 32 insertions(+)

diff --git a/test/plugins/test_importsource.py b/test/plugins/test_importsource.py
index a4f498181..20de3b6dd 100644
--- a/test/plugins/test_importsource.py
+++ b/test/plugins/test_importsource.py
@@ -144,3 +144,35 @@ class ImportSourceTest(PluginMixin, AutotagImportTestCase):
         plugin = plugins._instances[0]
         mock_task = MockTask()
         plugin.prevent_suggest_removal(None, mock_task)
+
+
+class ImportSourceTestListenerRegistration(PluginMixin, AutotagImportTestCase):
+    """Test listener registration based on config."""
+
+    plugin = "importsource"
+    preload_plugin = False
+
+    def setUp(self):
+        preserve_plugin_listeners()
+        super().setUp()
+
+    def test_listeners_not_registered_when_disabled(self):
+        """Test that listeners are not registered when suggest_removal is False."""
+        self.config[self.plugin]["suggest_removal"] = False
+        self.load_plugins()
+
+        plugin = plugins._instances[0]
+        assert not hasattr(plugin, "stop_suggestions_for_albums")
+        assert "item_removed" not in plugin._raw_listeners
+        assert "import_task_choice" not in plugin._raw_listeners
+
+    def test_listeners_registered_when_enabled(self):
+        """Test that listeners are registered when suggest_removal is True."""
+        self.config[self.plugin]["suggest_removal"] = True
+        self.load_plugins()
+
+        plugin = plugins._instances[0]
+        assert hasattr(plugin, "stop_suggestions_for_albums")
+        assert isinstance(plugin.stop_suggestions_for_albums, set)
+        assert "item_removed" in plugin._raw_listeners
+        assert "import_task_choice" in plugin._raw_listeners