Rename BaseArchiver to GenericArchiver + some other tidyups

src/auto_archiver/archivers/__init__.py

@@ -5,5 +5,5 @@
 from .instagram_tbot_archiver import InstagramTbotArchiver
 from .telegram_archiver import TelegramArchiver
 from .vk_archiver import VkArchiver
-from .base_archiver.base_archiver import BaseArchiver as YoutubeDLArchiver
+from .generic_archiver.generic_archiver import GenericArchiver as YoutubeDLArchiver
 from .instagram_api_archiver import InstagramAPIArchiver

src/auto_archiver/archivers/generic_archiver/__init__.py

@@ -0,0 +1 @@
+from .generic_archiver import GenericArchiver

src/auto_archiver/archivers/base_archiver/base_archiver.py → src/auto_archiver/archivers/generic_archiver/generic_archiver.py

@@ -9,7 +9,7 @@
 from ...core import Metadata, Media, ArchivingContext
 
 
-class BaseArchiver(Archiver):
+class GenericArchiver(Archiver):
     name = "youtubedl_archiver" #left as is for backwards compat
 
     def __init__(self, config: dict) -> None:
@@ -76,7 +76,8 @@ def keys_to_clean(self, extractor_key: str, video_data: dict) -> dict:
             return base_keys + [] 
         elif extractor_key == "Bluesky":
             # bluesky API response for non video URLs is already clean, nothing to add
-            return base_keys + [] 
+            return base_keys + []
+
 
         return base_keys
 

src/auto_archiver/archivers/youtubedl_archiver.py

@@ -1,2 +1,2 @@
 # temporary hack, as we implement module
-from .youtubedl_archiver import *
+from .generic_archiver.generic_archiver import GenericArchiver as YoutubeDLArchiver