article-backup 0.3.10__tar.gz → 0.3.12__tar.gz

{article_backup-0.3.10 → article_backup-0.3.12}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: article-backup
-Version: 0.3.10
+Version: 0.3.12
 Summary: Локальный бэкап статей с Sponsr.ru и Boosty.to в Markdown с Hugo-интеграцией
 Author-email: Eugene Chaykin <eugene@chayk.in>
 License: Apache-2.0
@@ -91,6 +91,11 @@ hugo:
   title: "Бэкап статей"
   language_code: "ru"
 
+sync:
+  # stop: остановиться, если авторизация любого источника не прошла
+  # continue: пропустить проблемные источники и собрать сайт из доступных данных
+  on_error: stop
+
 auth:
   sponsr_cookie_file: ./sponsr_cookie.txt
   boosty_cookie_file: ./boosty_cookie.txt
@@ -142,6 +147,8 @@ article-backup
 python backup.py
 ```
 
+Перед скачиванием выполняется проверка авторизации для всех источников. По умолчанию `sync.on_error: stop`: если один токен протух, скачивание не начинается и команда завершается с ошибкой. Если указать `sync.on_error: continue`, источники с ошибками авторизации будут пропущены, остальные источники синхронизируются, а Docker-запуск продолжит сборку Hugo-сайта.
+
 ### Скачать один пост по URL
 
 ```bash

{article_backup-0.3.10 → article_backup-0.3.12}/README.md

@@ -60,6 +60,11 @@ hugo:
   title: "Бэкап статей"
   language_code: "ru"
 
+sync:
+  # stop: остановиться, если авторизация любого источника не прошла
+  # continue: пропустить проблемные источники и собрать сайт из доступных данных
+  on_error: stop
+
 auth:
   sponsr_cookie_file: ./sponsr_cookie.txt
   boosty_cookie_file: ./boosty_cookie.txt
@@ -111,6 +116,8 @@ article-backup
 python backup.py
 ```
 
+Перед скачиванием выполняется проверка авторизации для всех источников. По умолчанию `sync.on_error: stop`: если один токен протух, скачивание не начинается и команда завершается с ошибкой. Если указать `sync.on_error: continue`, источники с ошибками авторизации будут пропущены, остальные источники синхронизируются, а Docker-запуск продолжит сборку Hugo-сайта.
+
 ### Скачать один пост по URL
 
 ```bash

{article_backup-0.3.10 → article_backup-0.3.12}/article_backup.egg-info/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.4
 Name: article-backup
-Version: 0.3.10
+Version: 0.3.12
 Summary: Локальный бэкап статей с Sponsr.ru и Boosty.to в Markdown с Hugo-интеграцией
 Author-email: Eugene Chaykin <eugene@chayk.in>
 License: Apache-2.0
@@ -91,6 +91,11 @@ hugo:
   title: "Бэкап статей"
   language_code: "ru"
 
+sync:
+  # stop: остановиться, если авторизация любого источника не прошла
+  # continue: пропустить проблемные источники и собрать сайт из доступных данных
+  on_error: stop
+
 auth:
   sponsr_cookie_file: ./sponsr_cookie.txt
   boosty_cookie_file: ./boosty_cookie.txt
@@ -142,6 +147,8 @@ article-backup
 python backup.py
 ```
 
+Перед скачиванием выполняется проверка авторизации для всех источников. По умолчанию `sync.on_error: stop`: если один токен протух, скачивание не начинается и команда завершается с ошибкой. Если указать `sync.on_error: continue`, источники с ошибками авторизации будут пропущены, остальные источники синхронизируются, а Docker-запуск продолжит сборку Hugo-сайта.
+
 ### Скачать один пост по URL
 
 ```bash

{article_backup-0.3.10 → article_backup-0.3.12}/article_backup.egg-info/SOURCES.txt

@@ -24,4 +24,5 @@ tests/test_slug_safety.py
 tests/test_sponsr_formatting_fix.py
 tests/test_sponsr_normalize.py
 tests/test_sponsr_tags.py
+tests/test_sync_policy.py
 tests/test_video_embed.py

{article_backup-0.3.10 → article_backup-0.3.12}/backup.py

@@ -26,7 +26,7 @@ def generate_hugo_config(config: Config):
         return json.dumps(value, ensure_ascii=False)
 
     content = f'''baseURL = {toml_str(config.hugo.base_url)}
-languageCode = {toml_str(config.hugo.language_code)}
+locale = {toml_str(config.hugo.language_code)}
 title = {toml_str(config.hugo.title)}
 relativeURLs = true
 
@@ -91,16 +91,35 @@ def get_downloader(platform: str, config: Config, source: Source, db: Database):
         raise ValueError(f"Неизвестная платформа: {platform}")
 
 
-def sync_all(config: Config, db: Database):
-    """Синхронизирует всех авторов из конфига."""
+def preflight_sources(config: Config, db: Database):
+    """Проверяет доступность источников до начала синхронизации."""
+    ready_sources: list[Source] = []
     errors: list[tuple[Source, Exception]] = []
+
     for source in config.sources:
+        try:
+            downloader = get_downloader(source.platform, config, source, db)
+            downloader.check_auth()
+            ready_sources.append(source)
+        except Exception as e:
+            print(f"[{source.platform}] Ошибка проверки авторизации {source.author}: {e}")
+            errors.append((source, e))
+
+    return ready_sources, errors
+
+
+def sync_all(config: Config, db: Database, sources: list[Source] | None = None):
+    """Синхронизирует всех авторов из конфига."""
+    errors: list[tuple[Source, Exception]] = []
+    for source in sources if sources is not None else config.sources:
         try:
             downloader = get_downloader(source.platform, config, source, db)
             downloader.sync()
         except Exception as e:
             print(f"[{source.platform}] Ошибка при синхронизации {source.author}: {e}")
             errors.append((source, e))
+            if config.sync.on_error == 'stop':
+                break
     return errors
 
 
@@ -181,7 +200,15 @@ def main():
             if not config.sources:
                 print("Нет источников в конфиге. Добавьте секцию 'sources'.")
                 sys.exit(1)
-            sync_errors = sync_all(config, db)
+            ready_sources, preflight_errors = preflight_sources(config, db)
+            if preflight_errors:
+                sync_errors.extend(preflight_errors)
+                if config.sync.on_error == 'stop':
+                    print("\nОстановлено из-за ошибок проверки авторизации.")
+                else:
+                    print("\nИсточники с ошибками проверки авторизации будут пропущены.")
+            if not preflight_errors or config.sync.on_error == 'continue':
+                sync_errors.extend(sync_all(config, db, ready_sources))
 
     ensure_site_content_link(config)
     generate_hugo_config(config)
@@ -190,7 +217,8 @@ def main():
         print(f"\nЗавершено с ошибками: {len(sync_errors)}")
         for source, error in sync_errors:
             print(f"  - [{source.platform}] {source.author}: {error}")
-        sys.exit(1)
+        if config.sync.on_error == 'stop':
+            sys.exit(1)
 
     print("\nГотово!")
 

{article_backup-0.3.10 → article_backup-0.3.12}/pyproject.toml

@@ -1,6 +1,6 @@
 [project]
 name = "article-backup"
-version = "0.3.10"
+version = "0.3.12"
 description = "Локальный бэкап статей с Sponsr.ru и Boosty.to в Markdown с Hugo-интеграцией"
 readme = "README.md"
 license = {text = "Apache-2.0"}

{article_backup-0.3.10 → article_backup-0.3.12}/src/boosty.py

@@ -43,6 +43,17 @@ class BoostyDownloader(BaseDownloader):
             'User-Agent': 'Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36',
         })
 
+    def check_auth(self):
+        """Проверяет доступ к ленте автора минимальным API-запросом."""
+        url = f"{self.API_BASE}/blog/{self.source.author}/post/?limit=1"
+
+        def do_request():
+            resp = self.session.get(url, timeout=self.TIMEOUT)
+            resp.raise_for_status()
+            return resp
+
+        retry_request(do_request, max_retries=3)
+
     def fetch_posts_list(
         self,
         existing_ids: set[str] | None = None,
@@ -183,19 +194,21 @@ class BoostyDownloader(BaseDownloader):
                     })
 
             elif block_type == "ok_video":
+                # Превью скачивается всегда (force=True обходит фильтр asset_types)
+                preview = block.get("previewUrl") or block.get("preview") or ""
+                if preview:
+                    assets.append({
+                        "url": preview,
+                        "alt": f"video-preview-{block.get('id', '')}",
+                        "force": True,
+                    })
+                # Видео скачивается через обычный механизм (фильтруется по asset_types)
                 video_url = self._extract_ok_video_player_url(block)
                 if video_url:
                     assets.append({
                         "url": video_url,
                         "alt": block.get("title") or f"video-{block.get('id', '')}",
                     })
-                else:
-                    preview = block.get("previewUrl") or block.get("preview") or ""
-                    if preview:
-                        assets.append({
-                            "url": preview,
-                            "alt": f"video-preview-{block.get('id', '')}",
-                        })
 
         return assets
 
@@ -281,16 +294,27 @@ class BoostyDownloader(BaseDownloader):
                 return f"\n🎵 **{title}**: [слушать]({url})\n"
 
         elif block_type == "ok_video":
+            # Определяем ссылку на видео (приоритет: локальный файл > ok.ru/video > videoembed)
             video_url = self._extract_ok_video_player_url(block)
+            video_link = ""
             if video_url:
-                local = asset_map.get(video_url)
-                if local:
-                    return f"\n[\U0001f4f9 Видео](assets/{local})\n"
-                return f"\n[\U0001f4f9 Видео]({video_url})\n"
-
-            fallback_url = self._extract_ok_video_fallback_url(block)
-            if fallback_url:
-                return f"\n[\U0001f4f9 Видео]({fallback_url})\n"
+                local_video = asset_map.get(video_url)
+                if local_video:
+                    video_link = f"assets/{local_video}"
+                else:
+                    video_link = video_url
+            if not video_link:
+                video_link = self._extract_ok_video_fallback_url(block)
+            if not video_link:
+                return ""
+
+            # Определяем превью-картинку
+            preview_url = block.get("previewUrl") or block.get("preview") or ""
+            local_preview = asset_map.get(preview_url) if preview_url else None
+
+            if local_preview:
+                return f"\n[![\U0001f4f9 Видео](assets/{local_preview})]({video_link})\n"
+            return f"\n[\U0001f4f9 Видео]({video_link})\n"
 
         elif block_type and block_type not in self._warned_unknown_block_types:
             print(f"  [boosty] Пропущен неподдерживаемый тип блока: {block_type}")

{article_backup-0.3.10 → article_backup-0.3.12}/src/config.py

@@ -34,12 +34,18 @@ class HugoConfig:
     default_theme: str = "light"
 
 
+@dataclass
+class SyncConfig:
+    on_error: Literal['stop', 'continue'] = "stop"
+
+
 @dataclass
 class Config:
     output_dir: Path
     auth: Auth
     sources: list[Source] = field(default_factory=list)
     hugo: HugoConfig = field(default_factory=HugoConfig)
+    sync: SyncConfig = field(default_factory=SyncConfig)
 
 
 def load_config(config_path: Path) -> Config:
@@ -105,7 +111,18 @@ def load_config(config_path: Path) -> Config:
         default_theme=hugo_data.get('default_theme', HugoConfig.default_theme),
     )
 
-    return Config(output_dir=output_dir, auth=auth, sources=sources, hugo=hugo)
+    # sync
+    sync_data = data.get('sync', {})
+    if sync_data is None:
+        sync_data = {}
+    if not isinstance(sync_data, dict):
+        raise ValueError("Секция 'sync' должна быть объектом")
+    sync_on_error = sync_data.get('on_error', SyncConfig.on_error)
+    if sync_on_error not in ('stop', 'continue'):
+        raise ValueError("sync.on_error должен быть 'stop' или 'continue'")
+    sync = SyncConfig(on_error=sync_on_error)
+
+    return Config(output_dir=output_dir, auth=auth, sources=sources, hugo=hugo, sync=sync)
 
 
 def _to_path(value: str | None) -> Path | None:

{article_backup-0.3.10 → article_backup-0.3.12}/src/downloader.py

@@ -98,6 +98,10 @@ class BaseDownloader(ABC):
         """Настройка сессии (cookies, headers)."""
         pass
 
+    def check_auth(self):
+        """Проверяет, что авторизация позволяет читать источник."""
+        raise NotImplementedError(f"{self.PLATFORM} не реализует проверку авторизации")
+
     @abstractmethod
     def fetch_posts_list(
         self,
@@ -314,10 +318,11 @@ class BaseDownloader(ABC):
 
         def download_one(asset: dict) -> tuple[str, str | None]:
             url = asset["url"]
+            force = asset.get("force", False)
             try:
                 # Предварительная проверка (если расширение есть)
                 ext = Path(urlparse(url).path).suffix.lower()
-                if ext and not should_download_asset(url, None, self.source.asset_types):
+                if ext and not force and not should_download_asset(url, None, self.source.asset_types):
                     return url, None
 
                 def do_request():
@@ -330,7 +335,7 @@ class BaseDownloader(ABC):
                     content_type = response.headers.get('Content-Type', '')
 
                     # Полная проверка после получения Content-Type
-                    if not should_download_asset(url, content_type, self.source.asset_types):
+                    if not force and not should_download_asset(url, content_type, self.source.asset_types):
                         return url, None
 
                     filename_base = self._make_asset_filename(url, content_type, asset.get('alt'))

{article_backup-0.3.10 → article_backup-0.3.12}/src/sponsr.py

@@ -44,6 +44,18 @@ class SponsorDownloader(BaseDownloader):
             'X-Requested-With': 'XMLHttpRequest',
         })
 
+    def check_auth(self):
+        """Проверяет доступ к проекту минимальным API-запросом."""
+        project_id = self._get_project_id()
+        api_url = f"https://sponsr.ru/project/{project_id}/more-posts/?offset=0"
+
+        def do_request():
+            resp = self.session.get(api_url, timeout=self.TIMEOUT)
+            resp.raise_for_status()
+            return resp
+
+        retry_request(do_request, max_retries=3)
+
     def _get_project_id(self) -> str:
         """Получает project_id со страницы проекта."""
         if self._project_id:
@@ -640,6 +652,8 @@ class SponsorDownloader(BaseDownloader):
         markdown = markdown.replace('@@@LBR@@@', r'\[')
         markdown = markdown.replace('@@@RBR@@@', r'\]')
         # Заменяем маркеры пробелов, вставленные в DOM
+        markdown = re.sub(r'[ \t]*@@@SP@@@[ \t]*', '@@@SP@@@', markdown)
+        markdown = re.sub(r'(?:@@@SP@@@)+', '@@@SP@@@', markdown)
         markdown = markdown.replace('@@@SP@@@', ' ')
 
         # Удаляем bidi-маркеры, которые ломают пробелы рядом с текстом

{article_backup-0.3.10 → article_backup-0.3.12}/tests/test_config_hardening.py

@@ -17,6 +17,24 @@ class ConfigHardeningTests(unittest.TestCase):
 
             self.assertEqual(cfg.output_dir, Path("./backup"))
             self.assertEqual(cfg.sources, [])
+            self.assertEqual(cfg.sync.on_error, "stop")
+
+    def test_load_config_accepts_sync_continue_policy(self):
+        with tempfile.TemporaryDirectory() as tmp:
+            cfg_path = Path(tmp) / "config.yaml"
+            cfg_path.write_text("sync:\n  on_error: continue\n", encoding="utf-8")
+
+            cfg = load_config(cfg_path)
+
+            self.assertEqual(cfg.sync.on_error, "continue")
+
+    def test_load_config_rejects_unknown_sync_policy(self):
+        with tempfile.TemporaryDirectory() as tmp:
+            cfg_path = Path(tmp) / "config.yaml"
+            cfg_path.write_text("sync:\n  on_error: ignore\n", encoding="utf-8")
+
+            with self.assertRaisesRegex(ValueError, "sync.on_error"):
+                load_config(cfg_path)
 
     def test_generate_hugo_config_escapes_quotes(self):
         with tempfile.TemporaryDirectory() as tmp:
@@ -42,6 +60,8 @@ class ConfigHardeningTests(unittest.TestCase):
 
                 self.assertIn('title = "Bob\'s \\"backup\\""', toml)
                 self.assertIn('baseURL = "https://example.com/a\\"b"', toml)
+                self.assertIn('locale = "ru"', toml)
+                self.assertNotIn('languageCode', toml)
                 self.assertIn('default_theme = "light\\"mode"', toml)
             finally:
                 os.chdir(old_cwd)

{article_backup-0.3.10 → article_backup-0.3.12}/tests/test_sponsr_normalize.py

@@ -119,6 +119,7 @@ class SponsorNormalizeTests(unittest.TestCase):
         # Ожидаем пробелы вокруг **жирное**
         self.assertIn('слово **жирное** слово', result)
         self.assertNotIn('слово**жирное**слово', result)
+        self.assertNotIn('**жирное**  слово', result)
 
     def test_real_world_case_from_issue(self):
         """Тест реального случая из issue."""

article_backup-0.3.12/tests/test_sync_policy.py

@@ -0,0 +1,129 @@
+import unittest
+import sys
+import tempfile
+from pathlib import Path
+
+import backup
+from src.config import Auth, Config, Source, SyncConfig
+
+
+class DummyDownloader:
+    checks: list[str] = []
+    synced: list[str] = []
+    check_failures: dict[str, Exception] = {}
+    sync_failures: dict[str, Exception] = {}
+
+    def __init__(self, config, source, db):
+        self.source = source
+
+    def check_auth(self):
+        DummyDownloader.checks.append(self.source.author)
+        error = DummyDownloader.check_failures.get(self.source.author)
+        if error:
+            raise error
+
+    def sync(self):
+        DummyDownloader.synced.append(self.source.author)
+        error = DummyDownloader.sync_failures.get(self.source.author)
+        if error:
+            raise error
+
+
+class SyncPolicyTests(unittest.TestCase):
+    def setUp(self):
+        self.old_get_downloader = backup.get_downloader
+        backup.get_downloader = lambda platform, config, source, db: DummyDownloader(config, source, db)
+        DummyDownloader.checks = []
+        DummyDownloader.synced = []
+        DummyDownloader.check_failures = {}
+        DummyDownloader.sync_failures = {}
+
+    def tearDown(self):
+        backup.get_downloader = self.old_get_downloader
+
+    def make_config(self, on_error):
+        return Config(
+            output_dir=Path("/tmp/test"),
+            auth=Auth(),
+            sources=[
+                Source(platform="sponsr", author="good"),
+                Source(platform="boosty", author="bad"),
+            ],
+            sync=SyncConfig(on_error=on_error),
+        )
+
+    def test_preflight_continue_filters_failed_sources(self):
+        config = self.make_config("continue")
+        DummyDownloader.check_failures = {"bad": RuntimeError("401 Unauthorized")}
+
+        ready_sources, errors = backup.preflight_sources(config, object())
+
+        self.assertEqual([source.author for source in ready_sources], ["good"])
+        self.assertEqual([source.author for source, _ in errors], ["bad"])
+        self.assertEqual(DummyDownloader.checks, ["good", "bad"])
+
+    def test_sync_all_continue_keeps_syncing_after_source_error(self):
+        config = self.make_config("continue")
+        DummyDownloader.sync_failures = {"good": RuntimeError("boom")}
+
+        errors = backup.sync_all(config, object())
+
+        self.assertEqual([source.author for source, _ in errors], ["good"])
+        self.assertEqual(DummyDownloader.synced, ["good", "bad"])
+
+    def test_sync_all_stop_stops_after_first_source_error(self):
+        config = self.make_config("stop")
+        DummyDownloader.sync_failures = {"good": RuntimeError("boom")}
+
+        errors = backup.sync_all(config, object())
+
+        self.assertEqual([source.author for source, _ in errors], ["good"])
+        self.assertEqual(DummyDownloader.synced, ["good"])
+
+    def test_main_continue_preflight_errors_do_not_exit_with_failure(self):
+        config = self.make_config("continue")
+        DummyDownloader.check_failures = {"bad": RuntimeError("401 Unauthorized")}
+
+        class DummyDatabase:
+            def __init__(self, path):
+                self.path = path
+
+            def __enter__(self):
+                return self
+
+            def __exit__(self, exc_type, exc, tb):
+                return False
+
+        old_argv = sys.argv
+        old_load_config = backup.load_config
+        old_database = backup.Database
+        old_ensure_link = backup.ensure_site_content_link
+        old_generate_hugo_config = backup.generate_hugo_config
+
+        with tempfile.TemporaryDirectory() as tmp:
+            cfg_path = Path(tmp) / "config.yaml"
+            cfg_path.write_text("", encoding="utf-8")
+            config.output_dir = Path(tmp) / "backup"
+
+            try:
+                sys.argv = ["backup.py", "--config", str(cfg_path)]
+                backup.load_config = lambda path: config
+                backup.Database = DummyDatabase
+                backup.ensure_site_content_link = lambda cfg: None
+                backup.generate_hugo_config = lambda cfg: None
+
+                backup.main()
+            except SystemExit as e:
+                self.fail(f"main() exited with {e.code} for continue policy")
+            finally:
+                sys.argv = old_argv
+                backup.load_config = old_load_config
+                backup.Database = old_database
+                backup.ensure_site_content_link = old_ensure_link
+                backup.generate_hugo_config = old_generate_hugo_config
+
+        self.assertEqual(DummyDownloader.synced, ["good"])
+
+
+if __name__ == "__main__":
+    unittest.main()

{article_backup-0.3.10 → article_backup-0.3.12}/tests/test_video_embed.py

@@ -139,8 +139,8 @@ class BoostyVideoEmbedTests(unittest.TestCase):
              patch('src.boosty.load_auth_header', return_value='Bearer fake_token'):
             self.downloader = BoostyDownloader(self.config, self.source, self.db)
 
-    def test_ok_video_uses_player_url(self):
-        """ok_video блок → markdown-ссылка на лучший playerUrl."""
+    def test_ok_video_uses_player_url_no_preview(self):
+        """ok_video без превью → простая текстовая ссылка на playerUrl."""
         blocks = [
             {
                 "type": "ok_video",
@@ -164,13 +164,15 @@ class BoostyVideoEmbedTests(unittest.TestCase):
         # Не должно быть старого формата
         self.assertNotIn('📹 Видео:', result)
 
-    def test_ok_video_uses_local_file_if_downloaded(self):
-        """ok_video с playerUrl должен ссылаться на локальный asset, если скачан."""
+    def test_ok_video_clickable_preview_with_local_video(self):
+        """ok_video: превью скачано + видео скачано → кликабельная картинка на локальный файл."""
+        preview_url = "https://iv.okcdn.ru/videoPreview?id=1"
         video_url = "https://vd.example/high?id=1"
         blocks = [
             {
                 "type": "ok_video",
                 "id": "abc",
+                "preview": preview_url,
                 "playerUrls": [{"type": "high", "url": video_url}],
             }
         ]
@@ -181,14 +183,43 @@ class BoostyVideoEmbedTests(unittest.TestCase):
             tags=[], assets=[]
         )
 
-        result = self.downloader._to_markdown(post, {video_url: "video-1.mp4"})
+        asset_map = {preview_url: "video-preview-abc.jpg", video_url: "video-1.mp4"}
+        result = self.downloader._to_markdown(post, asset_map)
 
-        self.assertIn('[📹 Видео](assets/video-1.mp4)', result)
+        self.assertIn('[![📹 Видео](assets/video-preview-abc.jpg)](assets/video-1.mp4)', result)
 
-    def test_ok_video_falls_back_to_vid_url(self):
-        """При отсутствии playerUrls используем ok.ru/video/{vid}."""
+    def test_ok_video_clickable_preview_with_fallback_url(self):
+        """ok_video: превью скачано, видео нет → кликабельная картинка на ok.ru/video."""
+        preview_url = "https://iv.okcdn.ru/videoPreview?id=1"
         blocks = [
-            {"type": "ok_video", "id": "uuid-1", "vid": "11386338749172"},
+            {
+                "type": "ok_video",
+                "id": "uuid-1",
+                "vid": "11386338749172",
+                "preview": preview_url,
+            }
+        ]
+        post = Post(
+            post_id='1', title='Test',
+            content_html=json.dumps(blocks),
+            post_date='2025-01-01', source_url='https://test.com',
+            tags=[], assets=[]
+        )
+
+        asset_map = {preview_url: "video-preview-uuid-1.jpg"}
+        result = self.downloader._to_markdown(post, asset_map)
+
+        self.assertIn('[![📹 Видео](assets/video-preview-uuid-1.jpg)](https://ok.ru/video/11386338749172)', result)
+
+    def test_ok_video_preview_not_downloaded_falls_back_to_text_link(self):
+        """ok_video: превью есть в блоке но не скачано → обычная текстовая ссылка."""
+        blocks = [
+            {
+                "type": "ok_video",
+                "id": "uuid-1",
+                "vid": "11386338749172",
+                "preview": "https://iv.okcdn.ru/videoPreview?id=1",
+            }
         ]
         post = Post(
             post_id='1', title='Test',
@@ -200,6 +231,7 @@ class BoostyVideoEmbedTests(unittest.TestCase):
         result = self.downloader._to_markdown(post, {})
 
         self.assertIn('[📹 Видео](https://ok.ru/video/11386338749172)', result)
+        self.assertNotIn('![', result)
 
     def test_ok_video_falls_back_to_embed_id(self):
         """Legacy fallback: если есть только id, оставляем videoembed/{id}."""
@@ -219,13 +251,16 @@ class BoostyVideoEmbedTests(unittest.TestCase):
 
     def test_ok_video_with_surrounding_text(self):
         """ok_video между текстовыми блоками."""
+        preview_url = "https://iv.okcdn.ru/preview?id=2"
+        video_url = "https://vd.example/medium?id=2"
         blocks = [
             {"type": "text", "content": json.dumps(["Посмотрите видео:"])},
             {"type": "text", "modificator": "BLOCK_END"},
             {
                 "type": "ok_video",
                 "id": "999888777",
-                "playerUrls": [{"type": "medium", "url": "https://vd.example/medium?id=2"}],
+                "preview": preview_url,
+                "playerUrls": [{"type": "medium", "url": video_url}],
             },
             {"type": "text", "content": json.dumps(["Вот такие дела."])},
             {"type": "text", "modificator": "BLOCK_END"},
@@ -237,14 +272,15 @@ class BoostyVideoEmbedTests(unittest.TestCase):
             tags=[], assets=[]
         )
 
-        result = self.downloader._to_markdown(post, {})
+        asset_map = {preview_url: "preview.jpg", video_url: "video.mp4"}
+        result = self.downloader._to_markdown(post, asset_map)
 
         self.assertIn('Посмотрите видео:', result)
-        self.assertIn('[📹 Видео](https://vd.example/medium?id=2)', result)
+        self.assertIn('[![📹 Видео](assets/preview.jpg)](assets/video.mp4)', result)
         self.assertIn('Вот такие дела.', result)
 
-    def test_extract_assets_prefers_ok_video_player_url(self):
-        """_extract_assets для ok_video должен добавлять видео URL, а не только preview."""
+    def test_extract_assets_ok_video_with_player_urls_extracts_both(self):
+        """_extract_assets для ok_video с playerUrls: и превью, и видео."""
         blocks = [
             {
                 "type": "ok_video",
@@ -260,11 +296,16 @@ class BoostyVideoEmbedTests(unittest.TestCase):
 
         assets = self.downloader._extract_assets(blocks)
 
-        self.assertEqual(len(assets), 1)
-        self.assertEqual(assets[0]["url"], "https://vd.example/high?id=1")
+        self.assertEqual(len(assets), 2)
+        # Первый — превью (с force=True)
+        self.assertEqual(assets[0]["url"], "https://iv.okcdn.ru/videoPreview?id=1")
+        self.assertIn("video-preview-", assets[0]["alt"])
+        self.assertTrue(assets[0].get("force"))
+        # Второй — видео
+        self.assertEqual(assets[1]["url"], "https://vd.example/high?id=1")
 
-    def test_extract_assets_ok_video_falls_back_to_preview(self):
-        """_extract_assets: если playerUrls пусты, берём preview."""
+    def test_extract_assets_ok_video_without_player_urls_extracts_preview(self):
+        """_extract_assets: если playerUrls пусты, берём только preview (с force)."""
         blocks = [
             {
                 "type": "ok_video",
@@ -278,6 +319,25 @@ class BoostyVideoEmbedTests(unittest.TestCase):
         self.assertEqual(len(assets), 1)
         self.assertEqual(assets[0]["url"], "https://iv.okcdn.ru/videoPreview?id=1")
         self.assertIn("video-preview-", assets[0]["alt"])
+        self.assertTrue(assets[0].get("force"))
+
+    def test_extract_assets_ok_video_no_preview_only_video(self):
+        """_extract_assets: если нет preview, только видео."""
+        blocks = [
+            {
+                "type": "ok_video",
+                "id": "video-id",
+                "playerUrls": [
+                    {"type": "high", "url": "https://vd.example/high?id=1"},
+                ],
+            }
+        ]
+
+        assets = self.downloader._extract_assets(blocks)
+
+        self.assertEqual(len(assets), 1)
+        self.assertEqual(assets[0]["url"], "https://vd.example/high?id=1")
+        self.assertFalse(assets[0].get("force", False))
 
     def test_ok_video_player_url_all_empty(self):
         """playerUrls с пустыми url → fallback на vid/id."""
@@ -367,5 +427,64 @@ class BoostyVideoEmbedTests(unittest.TestCase):
         self.assertNotIn('📹', result)
 
 
+class DownloadAssetsForceTests(unittest.TestCase):
+    """Тесты force-флага при скачивании assets."""
+
+    def setUp(self):
+        self.config = Config(output_dir=Path('/tmp/test'), auth=Auth())
+        # asset_types без image — обычные картинки фильтруются
+        self.source = Source(platform='boosty', author='test_author',
+                             asset_types=['video'])
+        self.db = MagicMock(spec=Database)
+
+        with patch('src.boosty.load_cookie', return_value='fake_cookie'), \
+             patch('src.boosty.load_auth_header', return_value='Bearer fake_token'):
+            self.downloader = BoostyDownloader(self.config, self.source, self.db)
+
+    @patch('src.downloader.retry_request')
+    def test_force_asset_bypasses_type_filter(self, mock_retry):
+        """Asset с force=True скачивается даже если тип не в asset_types."""
+        import tempfile
+        import os
+
+        with tempfile.TemporaryDirectory() as tmpdir:
+            assets_dir = Path(tmpdir)
+
+            # Мокаем ответ для картинки-превью
+            mock_response = MagicMock()
+            mock_response.headers = {'Content-Type': 'image/jpeg'}
+            mock_response.iter_content.return_value = [b'fake image data']
+            mock_response.close = MagicMock()
+            mock_retry.return_value = mock_response
+
+            assets = [
+                {"url": "https://iv.okcdn.ru/preview.jpg", "alt": "video-preview-1", "force": True},
+            ]
+
+            result = self.downloader._download_assets(assets, assets_dir)
+
+            # Должна быть скачана, несмотря на то что image не в asset_types
+            self.assertEqual(len(result), 1)
+            self.assertIn("https://iv.okcdn.ru/preview.jpg", result)
+
+    @patch('src.downloader.retry_request')
+    def test_non_force_asset_filtered_by_type(self, mock_retry):
+        """Обычный asset фильтруется по asset_types."""
+        import tempfile
+
+        with tempfile.TemporaryDirectory() as tmpdir:
+            assets_dir = Path(tmpdir)
+
+            assets = [
+                {"url": "https://example.com/photo.jpg", "alt": "photo"},
+            ]
+
+            result = self.downloader._download_assets(assets, assets_dir)
+
+            # Не должна быть скачана — image не в asset_types
+            self.assertEqual(len(result), 0)
+            mock_retry.assert_not_called()
+
+
 if __name__ == '__main__':
     unittest.main()