Merge develop: 교체 이미지 재다운로드 반영 [skip-notion]

Author: sungbin1015

scripts/notion_to_md.py

@@ -379,14 +379,19 @@ def download_image(url: str, slug: str, index: int) -> str:
         ext = ".jpg"
     filename = f"img-{index:02d}{ext}"
     filepath = f"{save_dir}/{filename}"
-    if not os.path.exists(filepath):
-        r = requests.get(url, timeout=30)
-        r.raise_for_status()
-        with open(filepath, "wb") as f:
-            f.write(r.content)
-        log(f"이미지 저장: {filepath}")
-    else:
-        log(f"이미지 캐시: {filepath}")
+    # Notion 이미지 URL은 매번 바뀌므로(서명 URL) 파일명 존재만으로 캐시하면
+    # 교체된 이미지가 반영되지 않는다. 항상 내려받아 내용이 다를 때만 덮어쓴다.
+    r = requests.get(url, timeout=30)
+    r.raise_for_status()
+    new_content = r.content
+    if os.path.exists(filepath):
+        with open(filepath, "rb") as f:
+            if f.read() == new_content:
+                log(f"이미지 변경 없음: {filepath}")
+                return f"/img/{_last_seg}/{slug}/{filename}"
+    with open(filepath, "wb") as f:
+        f.write(new_content)
+    log(f"이미지 저장: {filepath}")
     return f"/img/{_last_seg}/{slug}/{filename}"
 
 

scripts/tests/test_notion_to_md.py

@@ -5,6 +5,7 @@
 import hashlib
 import json
 import os
+import shutil
 import sys
 import tempfile
 from pathlib import Path
@@ -634,6 +635,55 @@ def test_multiple_shift_enters(self):
         assert "A  \nB  \nC" in out
 
 
+# ── 이미지 재다운로드: Notion 이미지 교체 시 같은 index 파일 덮어쓰기 ──────────
+
+class TestDownloadImageRedownload:
+    """Notion에서 이미지를 교체하면 URL은 바뀌지만 img-NN 파일명은 동일하다.
+    파일이 존재해도 새 내용이면 덮어써야 stale 이미지가 남지 않는다."""
+
+    def setup_method(self):
+        self.tmpdir = tempfile.mkdtemp()
+        self._orig = n.STATIC_IMG_DIR
+        n.STATIC_IMG_DIR = self.tmpdir
+
+    def teardown_method(self):
+        n.STATIC_IMG_DIR = self._orig
+        shutil.rmtree(self.tmpdir, ignore_errors=True)
+
+    def _mock_response(self, content):
+        m = MagicMock()
+        m.content = content
+        m.raise_for_status = lambda: None
+        return m
+
+    def _prewrite(self, content):
+        save_dir = os.path.join(self.tmpdir, "blog", "x")
+        os.makedirs(save_dir, exist_ok=True)
+        path = os.path.join(save_dir, "img-00.png")
+        with open(path, "wb") as f:
+            f.write(content)
+        return path
+
+    @patch.object(n, "requests")
+    def test_overwrites_when_content_changed(self, mock_requests):
+        path = self._prewrite(b"OLD-IMAGE")
+        mock_requests.get.return_value = self._mock_response(b"NEW-IMAGE")
+        n.download_image("https://notion/new.png", "blog/x", 0)
+        with open(path, "rb") as f:
+            assert f.read() == b"NEW-IMAGE"
+
+    @patch.object(n, "requests")
+    def test_no_rewrite_when_identical(self, mock_requests):
+        # 내용 동일 시 재기록하지 않아 git churn 방지 (mtime 유지)
+        path = self._prewrite(b"SAME-IMAGE")
+        os.utime(path, (1_000_000, 1_000_000))
+        mock_requests.get.return_value = self._mock_response(b"SAME-IMAGE")
+        n.download_image("https://notion/x.png", "blog/x", 0)
+        assert os.path.getmtime(path) == 1_000_000
+        with open(path, "rb") as f:
+            assert f.read() == b"SAME-IMAGE"
+
+
 # ── 리스트 항목의 이미지 자식은 최상위(전체 폭)로 hoist ──────────────────────
 
 class TestListItemImageHoisting: