From 7aa7f5a3d69b2bed2bda9f2011d1c7503a09524d Mon Sep 17 00:00:00 2001
From: Bryan Van Deusen <bryan.vandeusen@fabledsword.com>
Date: Tue, 26 May 2026 10:35:06 -0400
Subject: [PATCH 01/20] =?UTF-8?q?feat(thumb-backfill):=20=5Fthumb=5Fis=5Fv?=
 =?UTF-8?q?alid=20helper=20=E2=80=94=20JPEG/PNG=20magic-byte=20check=20on?=
 =?UTF-8?q?=20the=20on-disk=20thumbnail=20file?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---
 backend/app/tasks/thumbnail.py    | 24 +++++++++++++++++++++
 tests/test_backfill_thumbnails.py | 35 +++++++++++++++++++++++++++++++
 2 files changed, 59 insertions(+)
 create mode 100644 tests/test_backfill_thumbnails.py

diff --git a/backend/app/tasks/thumbnail.py b/backend/app/tasks/thumbnail.py
index dec644f..ab465ce 100644
--- a/backend/app/tasks/thumbnail.py
+++ b/backend/app/tasks/thumbnail.py
@@ -17,6 +17,30 @@ from ._sync_engine import sync_session_factory as _sync_session_factory
 
 IMAGES_ROOT = Path("/images")
 
+THUMB_MAGIC_JPEG = b"\xff\xd8\xff"
+THUMB_MAGIC_PNG = b"\x89PNG\r\n\x1a\n"
+
+
+def _thumb_is_valid(path: Path) -> bool:
+    """Return True iff `path` exists and starts with a JPEG or PNG magic header.
+
+    The on-disk thumbnail format is set by services/thumbnailer.py — JPEG for
+    opaque sources, PNG for alpha sources. Anything else (missing file, OSError,
+    truncated, wrong magic) is invalid.
+    """
+    try:
+        with path.open("rb") as f:
+            head = f.read(12)
+    except OSError:
+        return False
+    if len(head) < 8:
+        return False
+    if head[:3] == THUMB_MAGIC_JPEG:
+        return True
+    if head[:8] == THUMB_MAGIC_PNG:
+        return True
+    return False
+
 
 @celery.task(
     name="backend.app.tasks.thumbnail.generate_thumbnail",
diff --git a/tests/test_backfill_thumbnails.py b/tests/test_backfill_thumbnails.py
new file mode 100644
index 0000000..5537577
--- /dev/null
+++ b/tests/test_backfill_thumbnails.py
@@ -0,0 +1,35 @@
+"""Thumbnail backfill: _thumb_is_valid helper + backfill_thumbnails planner."""
+
+import pytest
+
+from backend.app.tasks.thumbnail import _thumb_is_valid
+
+pytestmark = pytest.mark.integration
+
+
+def test_thumb_is_valid_jpeg(tmp_path):
+    p = tmp_path / "good.jpg"
+    p.write_bytes(b"\xff\xd8\xff\xe0" + b"\x00" * 100)
+    assert _thumb_is_valid(p) is True
+
+
+def test_thumb_is_valid_png(tmp_path):
+    p = tmp_path / "good.png"
+    p.write_bytes(b"\x89PNG\r\n\x1a\n" + b"\x00" * 100)
+    assert _thumb_is_valid(p) is True
+
+
+def test_thumb_is_valid_too_short(tmp_path):
+    p = tmp_path / "tiny"
+    p.write_bytes(b"\xff\xd8")
+    assert _thumb_is_valid(p) is False
+
+
+def test_thumb_is_valid_wrong_magic(tmp_path):
+    p = tmp_path / "garbage"
+    p.write_bytes(b"\x00" * 12)
+    assert _thumb_is_valid(p) is False
+
+
+def test_thumb_is_valid_missing_file(tmp_path):
+    assert _thumb_is_valid(tmp_path / "nope") is False

From a41eddae3f042bfea1ffcfe8c36183a8ca03403f Mon Sep 17 00:00:00 2001
From: Bryan Van Deusen <bryan.vandeusen@fabledsword.com>
Date: Tue, 26 May 2026 10:36:21 -0400
Subject: [PATCH 02/20] =?UTF-8?q?feat(thumb-backfill):=20backfill=5Fthumbn?=
 =?UTF-8?q?ails=20planner=20task=20=E2=80=94=20keyset-paginates=20ImageRec?=
 =?UTF-8?q?ord,=20NULLs=20bad=20thumb=20paths,=20enqueues=20generate=5Fthu?=
 =?UTF-8?q?mbnail=20for=20NULL/missing/corrupt?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---
 backend/app/tasks/thumbnail.py    |  55 ++++++++
 tests/test_backfill_thumbnails.py | 207 ++++++++++++++++++++++++++++++
 tests/test_tasks_register.py      |   4 +
 3 files changed, 266 insertions(+)

diff --git a/backend/app/tasks/thumbnail.py b/backend/app/tasks/thumbnail.py
index ab465ce..0061bb1 100644
--- a/backend/app/tasks/thumbnail.py
+++ b/backend/app/tasks/thumbnail.py
@@ -74,3 +74,58 @@ def generate_thumbnail(self, image_id: int) -> dict:
         session.add(record)
         session.commit()
         return {"status": "ok", "image_id": image_id, "path": str(result.path)}
+
+
+@celery.task(
+    name="backend.app.tasks.thumbnail.backfill_thumbnails",
+    bind=True,
+)
+def backfill_thumbnails(self) -> dict:
+    """Scan ImageRecord and enqueue generate_thumbnail for rows whose
+    thumbnail is missing, gone from disk, or has wrong magic bytes.
+
+    Keyset paginates by id ASC, page size 500. NULLs out thumbnail_path for
+    rows that point at a missing or corrupt file before enqueueing — keeps
+    the DB self-consistent on partial runs and makes re-runs safe.
+
+    Returns {"enqueued": N, "ok": M, "regenerated": K} where:
+      - enqueued = total generate_thumbnail.delay() calls
+      - ok       = rows whose existing thumbnail file is valid (skipped)
+      - regenerated = subset of enqueued that had a non-NULL thumbnail_path
+                      cleared (i.e. missing + corrupt)
+    """
+    from sqlalchemy import select, update
+
+    SessionLocal = _sync_session_factory()
+    enqueued = 0
+    ok = 0
+    regenerated = 0
+    last_id = 0
+    with SessionLocal() as session:
+        while True:
+            rows = session.execute(
+                select(ImageRecord.id, ImageRecord.thumbnail_path)
+                .where(ImageRecord.id > last_id)
+                .order_by(ImageRecord.id.asc())
+                .limit(500)
+            ).all()
+            if not rows:
+                break
+            for image_id, thumb_path in rows:
+                if thumb_path is None:
+                    generate_thumbnail.delay(image_id)
+                    enqueued += 1
+                elif _thumb_is_valid(Path(thumb_path)):
+                    ok += 1
+                else:
+                    session.execute(
+                        update(ImageRecord)
+                        .where(ImageRecord.id == image_id)
+                        .values(thumbnail_path=None)
+                    )
+                    generate_thumbnail.delay(image_id)
+                    enqueued += 1
+                    regenerated += 1
+            session.commit()
+            last_id = rows[-1][0]
+    return {"enqueued": enqueued, "ok": ok, "regenerated": regenerated}
diff --git a/tests/test_backfill_thumbnails.py b/tests/test_backfill_thumbnails.py
index 5537577..faf2a77 100644
--- a/tests/test_backfill_thumbnails.py
+++ b/tests/test_backfill_thumbnails.py
@@ -1,7 +1,10 @@
 """Thumbnail backfill: _thumb_is_valid helper + backfill_thumbnails planner."""
 
+from pathlib import Path
+
 import pytest
 
+from backend.app.models import ImageRecord
 from backend.app.tasks.thumbnail import _thumb_is_valid
 
 pytestmark = pytest.mark.integration
@@ -33,3 +36,207 @@ def test_thumb_is_valid_wrong_magic(tmp_path):
 
 def test_thumb_is_valid_missing_file(tmp_path):
     assert _thumb_is_valid(tmp_path / "nope") is False
+
+
+# --- backfill_thumbnails planner tests ------------------------------------
+
+
+class _Ctx:
+    def __init__(self, s):
+        self.s = s
+
+    def __enter__(self):
+        return self.s
+
+    def __exit__(self, *a):
+        return False
+
+
+def _sf(db_sync):
+    """sessionmaker-like returning the test's bound session, matching the
+    pattern in tests/test_backfill_phash.py."""
+    class _SM:
+        def __call__(self):
+            return _Ctx(db_sync)
+
+    return _SM()
+
+
+def _sha(prefix: str) -> str:
+    return f"{prefix}".ljust(64, "0")[:64]
+
+
+def _rec(db_sync, path, *, sha, thumb_path=None, mime="image/jpeg"):
+    rec = ImageRecord(
+        path=str(path), sha256=sha, size_bytes=1, mime=mime,
+        width=64, height=64, origin="imported_filesystem",
+        integrity_status="unknown",
+        thumbnail_path=str(thumb_path) if thumb_path is not None else None,
+    )
+    db_sync.add(rec)
+    db_sync.flush()
+    return rec
+
+
+def _write_jpeg(p: Path) -> Path:
+    p.parent.mkdir(parents=True, exist_ok=True)
+    p.write_bytes(b"\xff\xd8\xff\xe0" + b"\x00" * 100)
+    return p
+
+
+def _write_png(p: Path) -> Path:
+    p.parent.mkdir(parents=True, exist_ok=True)
+    p.write_bytes(b"\x89PNG\r\n\x1a\n" + b"\x00" * 100)
+    return p
+
+
+def _write_garbage(p: Path) -> Path:
+    p.parent.mkdir(parents=True, exist_ok=True)
+    p.write_bytes(b"\x00" * 12)
+    return p
+
+
+def test_backfill_null_path_enqueued(db_sync, tmp_path, monkeypatch):
+    from backend.app.tasks import thumbnail as m
+
+    src = tmp_path / "a.bin"
+    src.write_bytes(b"x")
+    rec = _rec(db_sync, src, sha=_sha("a"), thumb_path=None)
+    db_sync.commit()
+
+    monkeypatch.setattr(m, "_sync_session_factory", lambda: _sf(db_sync))
+    delayed: list[int] = []
+    monkeypatch.setattr(
+        m.generate_thumbnail, "delay", lambda image_id: delayed.append(image_id)
+    )
+
+    result = m.backfill_thumbnails()
+    assert result == {"enqueued": 1, "ok": 0, "regenerated": 0}
+    assert delayed == [rec.id]
+
+
+def test_backfill_missing_file_clears_and_enqueues(db_sync, tmp_path, monkeypatch):
+    from backend.app.tasks import thumbnail as m
+
+    src = tmp_path / "b.bin"
+    src.write_bytes(b"x")
+    rec = _rec(
+        db_sync, src, sha=_sha("b"),
+        thumb_path=tmp_path / "thumbs" / "missing.jpg",
+    )
+    db_sync.commit()
+
+    monkeypatch.setattr(m, "_sync_session_factory", lambda: _sf(db_sync))
+    delayed: list[int] = []
+    monkeypatch.setattr(
+        m.generate_thumbnail, "delay", lambda image_id: delayed.append(image_id)
+    )
+
+    result = m.backfill_thumbnails()
+    db_sync.expire_all()
+    assert result == {"enqueued": 1, "ok": 0, "regenerated": 1}
+    assert delayed == [rec.id]
+    assert db_sync.get(ImageRecord, rec.id).thumbnail_path is None
+
+
+def test_backfill_valid_jpeg_skipped(db_sync, tmp_path, monkeypatch):
+    from backend.app.tasks import thumbnail as m
+
+    src = tmp_path / "c.bin"
+    src.write_bytes(b"x")
+    thumb = _write_jpeg(tmp_path / "thumbs" / "c.jpg")
+    rec = _rec(db_sync, src, sha=_sha("c"), thumb_path=thumb)
+    db_sync.commit()
+
+    monkeypatch.setattr(m, "_sync_session_factory", lambda: _sf(db_sync))
+    delayed: list[int] = []
+    monkeypatch.setattr(
+        m.generate_thumbnail, "delay", lambda image_id: delayed.append(image_id)
+    )
+
+    result = m.backfill_thumbnails()
+    db_sync.expire_all()
+    assert result == {"enqueued": 0, "ok": 1, "regenerated": 0}
+    assert delayed == []
+    assert db_sync.get(ImageRecord, rec.id).thumbnail_path == str(thumb)
+
+
+def test_backfill_valid_png_skipped(db_sync, tmp_path, monkeypatch):
+    from backend.app.tasks import thumbnail as m
+
+    src = tmp_path / "d.bin"
+    src.write_bytes(b"x")
+    thumb = _write_png(tmp_path / "thumbs" / "d.png")
+    _rec(db_sync, src, sha=_sha("d"), thumb_path=thumb)
+    db_sync.commit()
+
+    monkeypatch.setattr(m, "_sync_session_factory", lambda: _sf(db_sync))
+    delayed: list[int] = []
+    monkeypatch.setattr(
+        m.generate_thumbnail, "delay", lambda image_id: delayed.append(image_id)
+    )
+
+    result = m.backfill_thumbnails()
+    assert result == {"enqueued": 0, "ok": 1, "regenerated": 0}
+    assert delayed == []
+
+
+def test_backfill_corrupt_magic_clears_and_enqueues(db_sync, tmp_path, monkeypatch):
+    from backend.app.tasks import thumbnail as m
+
+    src = tmp_path / "e.bin"
+    src.write_bytes(b"x")
+    thumb = _write_garbage(tmp_path / "thumbs" / "e.jpg")
+    rec = _rec(db_sync, src, sha=_sha("e"), thumb_path=thumb)
+    db_sync.commit()
+
+    monkeypatch.setattr(m, "_sync_session_factory", lambda: _sf(db_sync))
+    delayed: list[int] = []
+    monkeypatch.setattr(
+        m.generate_thumbnail, "delay", lambda image_id: delayed.append(image_id)
+    )
+
+    result = m.backfill_thumbnails()
+    db_sync.expire_all()
+    assert result == {"enqueued": 1, "ok": 0, "regenerated": 1}
+    assert delayed == [rec.id]
+    assert db_sync.get(ImageRecord, rec.id).thumbnail_path is None
+
+
+def test_backfill_mixed_aggregate(db_sync, tmp_path, monkeypatch):
+    from backend.app.tasks import thumbnail as m
+
+    src_null = tmp_path / "src_null.bin"
+    src_null.write_bytes(b"x")
+    src_jpeg = tmp_path / "src_jpeg.bin"
+    src_jpeg.write_bytes(b"x")
+    src_png = tmp_path / "src_png.bin"
+    src_png.write_bytes(b"x")
+    src_missing = tmp_path / "src_missing.bin"
+    src_missing.write_bytes(b"x")
+    src_bad = tmp_path / "src_bad.bin"
+    src_bad.write_bytes(b"x")
+
+    jpeg = _write_jpeg(tmp_path / "thumbs" / "ok.jpg")
+    png = _write_png(tmp_path / "thumbs" / "ok.png")
+    bad = _write_garbage(tmp_path / "thumbs" / "bad.jpg")
+
+    r_null = _rec(db_sync, src_null, sha=_sha("aa"), thumb_path=None)
+    _rec(db_sync, src_jpeg, sha=_sha("bb"), thumb_path=jpeg)
+    _rec(db_sync, src_png, sha=_sha("cc"), thumb_path=png)
+    r_missing = _rec(
+        db_sync, src_missing, sha=_sha("dd"),
+        thumb_path=tmp_path / "thumbs" / "missing.jpg",
+    )
+    r_bad = _rec(db_sync, src_bad, sha=_sha("ee"), thumb_path=bad)
+    db_sync.commit()
+
+    monkeypatch.setattr(m, "_sync_session_factory", lambda: _sf(db_sync))
+    delayed: list[int] = []
+    monkeypatch.setattr(
+        m.generate_thumbnail, "delay", lambda image_id: delayed.append(image_id)
+    )
+
+    result = m.backfill_thumbnails()
+    assert result == {"enqueued": 3, "ok": 2, "regenerated": 2}
+    assert sorted(delayed) == sorted([r_null.id, r_missing.id, r_bad.id])
diff --git a/tests/test_tasks_register.py b/tests/test_tasks_register.py
index a8d4f72..11f457c 100644
--- a/tests/test_tasks_register.py
+++ b/tests/test_tasks_register.py
@@ -23,3 +23,7 @@ def test_import_media_file_registered():
 
 def test_generate_thumbnail_registered():
     assert "backend.app.tasks.thumbnail.generate_thumbnail" in celery.tasks
+
+
+def test_backfill_thumbnails_registered():
+    assert "backend.app.tasks.thumbnail.backfill_thumbnails" in celery.tasks

From 983da9e5b144634874ddc252b479f6293b6b4333 Mon Sep 17 00:00:00 2001
From: Bryan Van Deusen <bryan.vandeusen@fabledsword.com>
Date: Tue, 26 May 2026 10:36:38 -0400
Subject: [PATCH 03/20] =?UTF-8?q?feat(thumb-backfill):=20/api/thumbnails/b?=
 =?UTF-8?q?ackfill=20endpoint=20=E2=80=94=20POST=20=E2=86=92=20202=20+=20c?=
 =?UTF-8?q?elery=5Ftask=5Fid?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---
 backend/app/api/__init__.py   |  2 ++
 backend/app/api/thumbnails.py | 13 +++++++++++++
 tests/test_api_thumbnails.py  | 32 ++++++++++++++++++++++++++++++++
 3 files changed, 47 insertions(+)
 create mode 100644 backend/app/api/thumbnails.py
 create mode 100644 tests/test_api_thumbnails.py

diff --git a/backend/app/api/__init__.py b/backend/app/api/__init__.py
index 4a88c1e..b8dc7c4 100644
--- a/backend/app/api/__init__.py
+++ b/backend/app/api/__init__.py
@@ -38,6 +38,7 @@ def all_blueprints() -> list[Blueprint]:
     from .system_activity import system_activity_bp
     from .system_backup import system_backup_bp
     from .tags import tags_bp
+    from .thumbnails import thumbnails_bp
     return [
         api_bp,
         attachments_bp,
@@ -58,6 +59,7 @@ def all_blueprints() -> list[Blueprint]:
         allowlist_bp,
         aliases_bp,
         ml_admin_bp,
+        thumbnails_bp,
         sources_bp,
         platforms_bp,
         posts_bp,
diff --git a/backend/app/api/thumbnails.py b/backend/app/api/thumbnails.py
new file mode 100644
index 0000000..eff1491
--- /dev/null
+++ b/backend/app/api/thumbnails.py
@@ -0,0 +1,13 @@
+"""Thumbnail admin API: backfill trigger."""
+
+from quart import Blueprint, jsonify
+
+thumbnails_bp = Blueprint("thumbnails", __name__, url_prefix="/api/thumbnails")
+
+
+@thumbnails_bp.route("/backfill", methods=["POST"])
+async def trigger_backfill():
+    from ..tasks.thumbnail import backfill_thumbnails
+
+    r = backfill_thumbnails.delay()
+    return jsonify({"celery_task_id": r.id}), 202
diff --git a/tests/test_api_thumbnails.py b/tests/test_api_thumbnails.py
new file mode 100644
index 0000000..b9d07a4
--- /dev/null
+++ b/tests/test_api_thumbnails.py
@@ -0,0 +1,32 @@
+import pytest
+
+from backend.app import create_app
+from backend.app.celery_app import celery
+
+pytestmark = pytest.mark.integration
+
+
+@pytest.fixture(autouse=True)
+def eager():
+    celery.conf.task_always_eager = True
+    yield
+    celery.conf.task_always_eager = False
+
+
+@pytest.fixture
+async def app():
+    return create_app()
+
+
+@pytest.fixture
+async def client(app):
+    async with app.test_client() as c:
+        yield c
+
+
+@pytest.mark.asyncio
+async def test_trigger_thumbnail_backfill(client):
+    r = await client.post("/api/thumbnails/backfill")
+    assert r.status_code == 202
+    body = await r.get_json()
+    assert "celery_task_id" in body

From ecd0199799df45d955ce87e9bb475b21fdd12d24 Mon Sep 17 00:00:00 2001
From: Bryan Van Deusen <bryan.vandeusen@fabledsword.com>
Date: Tue, 26 May 2026 10:36:45 -0400
Subject: [PATCH 04/20] =?UTF-8?q?feat(thumb-backfill):=20Pinia=20store=20?=
 =?UTF-8?q?=E2=80=94=20triggerBackfill()=20POSTs=20/api/thumbnails/backfil?=
 =?UTF-8?q?l?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---
 frontend/src/stores/thumbnails.js | 12 ++++++++++++
 1 file changed, 12 insertions(+)
 create mode 100644 frontend/src/stores/thumbnails.js

diff --git a/frontend/src/stores/thumbnails.js b/frontend/src/stores/thumbnails.js
new file mode 100644
index 0000000..5f86f2b
--- /dev/null
+++ b/frontend/src/stores/thumbnails.js
@@ -0,0 +1,12 @@
+import { defineStore } from 'pinia'
+import { useApi } from '../composables/useApi.js'
+
+export const useThumbnailsStore = defineStore('thumbnails', () => {
+  const api = useApi()
+
+  async function triggerBackfill () {
+    await api.post('/api/thumbnails/backfill')
+  }
+
+  return { triggerBackfill }
+})

From b7f693b15ebf27f11dd38809554059052f788eec Mon Sep 17 00:00:00 2001
From: Bryan Van Deusen <bryan.vandeusen@fabledsword.com>
Date: Tue, 26 May 2026 10:36:55 -0400
Subject: [PATCH 05/20] =?UTF-8?q?feat(thumb-backfill):=20ThumbnailBackfill?=
 =?UTF-8?q?Card=20=E2=80=94=20'Run=20backfill=20now'=20button,=20mirrors?=
 =?UTF-8?q?=20MLBackfillCard=20pattern?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---
 .../settings/ThumbnailBackfillCard.vue        | 30 +++++++++++++++++++
 1 file changed, 30 insertions(+)
 create mode 100644 frontend/src/components/settings/ThumbnailBackfillCard.vue

diff --git a/frontend/src/components/settings/ThumbnailBackfillCard.vue b/frontend/src/components/settings/ThumbnailBackfillCard.vue
new file mode 100644
index 0000000..489c45f
--- /dev/null
+++ b/frontend/src/components/settings/ThumbnailBackfillCard.vue
@@ -0,0 +1,30 @@
+<template>
+  <v-card>
+    <v-card-title>Thumbnail backfill</v-card-title>
+    <v-card-text>
+      <p class="text-body-2 mb-3">
+        Scan the library for images with no thumbnail, or whose thumbnail file
+        is missing or corrupt on disk. Repair candidates are re-enqueued for
+        thumbnail generation. Safe to re-run.
+      </p>
+      <v-btn color="primary" rounded="pill" :loading="busy" @click="run">
+        <v-icon start>mdi-image-refresh</v-icon> Run backfill now
+      </v-btn>
+      <span v-if="done" class="ml-3 text-caption">Enqueued.</span>
+    </v-card-text>
+  </v-card>
+</template>
+
+<script setup>
+import { ref } from 'vue'
+import { useThumbnailsStore } from '../../stores/thumbnails.js'
+const store = useThumbnailsStore()
+const busy = ref(false)
+const done = ref(false)
+async function run () {
+  busy.value = true
+  try { await store.triggerBackfill(); done.value = true }
+  catch (e) { window.__fcToast?.({ text: e.message, type: 'error' }) }
+  finally { busy.value = false }
+}
+</script>

From 06913eba8e5a28b614d69ec63a5aa5a89c95309b Mon Sep 17 00:00:00 2001
From: Bryan Van Deusen <bryan.vandeusen@fabledsword.com>
Date: Tue, 26 May 2026 10:37:09 -0400
Subject: [PATCH 06/20] =?UTF-8?q?feat(thumb-backfill):=20MaintenancePanel?=
 =?UTF-8?q?=20=E2=80=94=20wire=20ThumbnailBackfillCard=20into=20grid,=20br?=
 =?UTF-8?q?oaden=20intro=20to=20cover=20non-ML=20backfills?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---
 frontend/src/components/settings/MaintenancePanel.vue | 9 ++++++---
 1 file changed, 6 insertions(+), 3 deletions(-)

diff --git a/frontend/src/components/settings/MaintenancePanel.vue b/frontend/src/components/settings/MaintenancePanel.vue
index 90c410b..42c775d 100644
--- a/frontend/src/components/settings/MaintenancePanel.vue
+++ b/frontend/src/components/settings/MaintenancePanel.vue
@@ -1,13 +1,15 @@
 <template>
   <div class="fc-maint">
     <p class="text-body-2 mb-4">
-      Machine-assisted tagging controls. Backfill and centroid recompute run
-      nightly automatically; the allowlist auto-applies accepted tags to new
-      and existing images.
+      Operational backfills and tagging controls. The ML backfill and centroid
+      recompute run nightly automatically; the allowlist auto-applies accepted
+      tags to new and existing images. Use the cards below to trigger a
+      one-off pass.
     </p>
     <div class="fc-maint__grid">
       <MLBackfillCard />
       <CentroidRecomputeCard />
+      <ThumbnailBackfillCard />
     </div>
     <MLThresholdSliders class="mt-4" />
     <AllowlistTable class="mt-4" />
@@ -23,6 +25,7 @@
 <script setup>
 import MLBackfillCard from './MLBackfillCard.vue'
 import CentroidRecomputeCard from './CentroidRecomputeCard.vue'
+import ThumbnailBackfillCard from './ThumbnailBackfillCard.vue'
 import MLThresholdSliders from './MLThresholdSliders.vue'
 import AllowlistTable from './AllowlistTable.vue'
 import AliasTable from './AliasTable.vue'

From 4e1f208a9ff46c84870e86930239561d53d47d70 Mon Sep 17 00:00:00 2001
From: Bryan Van Deusen <bryan.vandeusen@fabledsword.com>
Date: Tue, 26 May 2026 11:16:32 -0400
Subject: [PATCH 07/20] =?UTF-8?q?fix(ui-copy):=20copyText=20utility=20with?=
 =?UTF-8?q?=20execCommand=20fallback=20=E2=80=94=20navigator.clipboard=20i?=
 =?UTF-8?q?s=20gated=20by=20Secure=20Context=20(HTTPS-only)=20and=20is=20u?=
 =?UTF-8?q?ndefined=20on=20plain-HTTP=20self-hosted=20deployments.=20Apply?=
 =?UTF-8?q?=20to=20ExtensionKeyBar=20+=20BrowserExtensionCard.?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---
 .../credentials/ExtensionKeyBar.vue           |  3 +-
 .../settings/BrowserExtensionCard.vue         |  3 +-
 frontend/src/utils/clipboard.js               | 35 +++++++++++++++++++
 3 files changed, 39 insertions(+), 2 deletions(-)
 create mode 100644 frontend/src/utils/clipboard.js

diff --git a/frontend/src/components/credentials/ExtensionKeyBar.vue b/frontend/src/components/credentials/ExtensionKeyBar.vue
index cf56b3c..2563ec6 100644
--- a/frontend/src/components/credentials/ExtensionKeyBar.vue
+++ b/frontend/src/components/credentials/ExtensionKeyBar.vue
@@ -28,6 +28,7 @@
 <script setup>
 import { onMounted, ref } from 'vue'
 import { useCredentialsStore } from '../../stores/credentials.js'
+import { copyText } from '../../utils/clipboard.js'
 
 const store = useCredentialsStore()
 const showRotateConfirm = ref(false)
@@ -37,7 +38,7 @@ onMounted(() => store.loadKey())
 async function copyKey() {
   if (!store.extensionKey) return
   try {
-    await navigator.clipboard.writeText(store.extensionKey)
+    await copyText(store.extensionKey)
     globalThis.window?.__fcToast?.({ text: 'Copied', type: 'success' })
   } catch {
     globalThis.window?.__fcToast?.({ text: 'Copy failed', type: 'error' })
diff --git a/frontend/src/components/settings/BrowserExtensionCard.vue b/frontend/src/components/settings/BrowserExtensionCard.vue
index db5d3a3..2a0bedb 100644
--- a/frontend/src/components/settings/BrowserExtensionCard.vue
+++ b/frontend/src/components/settings/BrowserExtensionCard.vue
@@ -110,6 +110,7 @@
 <script setup>
 import { computed, onMounted, ref } from 'vue'
 import { useApi } from '../../composables/useApi.js'
+import { copyText } from '../../utils/clipboard.js'
 
 const api = useApi()
 
@@ -172,7 +173,7 @@ async function rotateKey() {
 
 async function copy(text, label) {
   try {
-    await navigator.clipboard.writeText(text)
+    await copyText(text)
     window.__fcToast?.({ text: `${label} copied.`, type: 'success' })
   } catch (e) {
     window.__fcToast?.({ text: `Copy failed: ${e.message}`, type: 'error' })
diff --git a/frontend/src/utils/clipboard.js b/frontend/src/utils/clipboard.js
new file mode 100644
index 0000000..8246f27
--- /dev/null
+++ b/frontend/src/utils/clipboard.js
@@ -0,0 +1,35 @@
+// Clipboard write that works on plain-HTTP self-hosted deployments.
+//
+// navigator.clipboard is gated by the browser's Secure Context restriction
+// (HTTPS or localhost only). FabledCurator runs over plain HTTP per the
+// homelab posture, so the modern API is undefined in production. We fall
+// back to the legacy execCommand('copy') path via a temporary off-screen
+// textarea — wide browser support, no HTTPS requirement.
+
+export async function copyText (text) {
+  const str = text == null ? '' : String(text)
+  if (typeof navigator !== 'undefined' && navigator.clipboard?.writeText) {
+    try {
+      await navigator.clipboard.writeText(str)
+      return
+    } catch {
+      // Fall through to the legacy path. Some browsers throw even when
+      // the API exists (permission denied, focus loss, etc.).
+    }
+  }
+  const ta = document.createElement('textarea')
+  ta.value = str
+  ta.setAttribute('readonly', '')
+  ta.style.position = 'fixed'
+  ta.style.top = '0'
+  ta.style.left = '0'
+  ta.style.opacity = '0'
+  ta.style.pointerEvents = 'none'
+  document.body.appendChild(ta)
+  ta.select()
+  ta.setSelectionRange(0, str.length)
+  let ok = false
+  try { ok = document.execCommand('copy') } catch { ok = false }
+  document.body.removeChild(ta)
+  if (!ok) throw new Error('clipboard copy not supported')
+}

From 6de84d0d60d459c2b65fac2d14929c154f435587 Mon Sep 17 00:00:00 2001
From: Bryan Van Deusen <bryan.vandeusen@fabledsword.com>
Date: Tue, 26 May 2026 11:16:37 -0400
Subject: [PATCH 08/20] =?UTF-8?q?feat(ui):=20ErrorDetailModal=20=E2=80=94?=
 =?UTF-8?q?=20context=20panel=20(task/queue/target/duration/started/retrie?=
 =?UTF-8?q?s/worker/celery-id/args)=20+=20contrast=20fix=20(background-tok?=
 =?UTF-8?q?en=20bg=20vs=20surface-variant=20pale-on-pale)=20+=20copyText?=
 =?UTF-8?q?=20helper=20for=20Copy=20button?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---
 .../components/common/ErrorDetailModal.vue    | 113 +++++++++++++++---
 .../components/settings/SystemActivityTab.vue |  13 +-
 2 files changed, 104 insertions(+), 22 deletions(-)

diff --git a/frontend/src/components/common/ErrorDetailModal.vue b/frontend/src/components/common/ErrorDetailModal.vue
index d8281a5..021f3da 100644
--- a/frontend/src/components/common/ErrorDetailModal.vue
+++ b/frontend/src/components/common/ErrorDetailModal.vue
@@ -4,14 +4,20 @@
     <v-card>
       <v-card-title class="d-flex align-center" style="gap: 12px;">
         <v-icon icon="mdi-alert-circle-outline" color="error" />
-        <span>{{ title }}</span>
+        <span>{{ displayTitle }}</span>
         <v-spacer />
         <v-btn icon variant="text" size="small" @click="close">
           <v-icon>mdi-close</v-icon>
         </v-btn>
       </v-card-title>
       <v-card-text>
-        <pre class="fc-err-pre">{{ message || '(no error message)' }}</pre>
+        <dl v-if="contextRows.length" class="fc-err-context">
+          <template v-for="(row, idx) in contextRows" :key="idx">
+            <dt>{{ row[0] }}</dt>
+            <dd>{{ row[1] }}</dd>
+          </template>
+        </dl>
+        <pre class="fc-err-pre">{{ displayMessage || '(no error message)' }}</pre>
       </v-card-text>
       <v-card-actions>
         <v-btn
@@ -27,12 +33,21 @@
 </template>
 
 <script setup>
-import { ref, watch } from 'vue'
+import { computed, ref, watch } from 'vue'
+
+import { copyText } from '../../utils/clipboard.js'
 
 const props = defineProps({
   modelValue: { type: Boolean, default: false },
+  // Legacy mode: pass title + message strings. Used by callers whose row
+  // shape lacks structured context (e.g. ImportTaskList where `error` is
+  // a plain string on the import_task row, not a TaskRun).
   title: { type: String, default: 'Error details' },
   message: { type: String, default: '' },
+  // Row mode: pass the full TaskRun-shaped dict from /api/system_activity.
+  // When set, displayTitle/displayMessage derive from the row and a context
+  // panel of task_name/queue/target/duration/etc. renders above the error.
+  row: { type: Object, default: null },
 })
 
 const emit = defineEmits(['update:modelValue'])
@@ -40,6 +55,47 @@ const emit = defineEmits(['update:modelValue'])
 const copied = ref(false)
 let copiedTimer = null
 
+const displayTitle = computed(() => {
+  if (props.row) return props.row.error_type || 'Error details'
+  return props.title
+})
+
+const displayMessage = computed(() => {
+  if (props.row) return props.row.error_message || ''
+  return props.message
+})
+
+function _shortTaskName (name) {
+  if (!name) return ''
+  const parts = String(name).split('.')
+  return parts[parts.length - 1]
+}
+
+function _formatDuration (ms) {
+  if (ms == null) return null
+  if (ms < 1000) return `${ms} ms`
+  if (ms < 60_000) return `${(ms / 1000).toFixed(1)} s`
+  return `${(ms / 60_000).toFixed(1)} min`
+}
+
+const contextRows = computed(() => {
+  const r = props.row
+  if (!r) return []
+  const rows = []
+  if (r.task_name) rows.push(['Task', _shortTaskName(r.task_name)])
+  if (r.queue) rows.push(['Queue', r.queue])
+  if (r.target_id != null) rows.push(['Target', r.target_id])
+  const dur = _formatDuration(r.duration_ms)
+  if (dur != null) rows.push(['Duration', dur])
+  if (r.started_at) rows.push(['Started', r.started_at])
+  if (r.finished_at) rows.push(['Finished', r.finished_at])
+  if (r.retry_count) rows.push(['Retries', r.retry_count])
+  if (r.worker_hostname) rows.push(['Worker', r.worker_hostname])
+  if (r.celery_task_id) rows.push(['Celery ID', r.celery_task_id])
+  if (r.args_summary) rows.push(['Args', r.args_summary])
+  return rows
+})
+
 watch(() => props.modelValue, (open) => {
   if (!open) {
     copied.value = false
@@ -47,13 +103,18 @@ watch(() => props.modelValue, (open) => {
   }
 })
 
-function close() {
+function close () {
   emit('update:modelValue', false)
 }
 
-async function onCopy() {
+async function onCopy () {
+  let text = displayMessage.value || ''
+  if (contextRows.value.length) {
+    const header = contextRows.value.map(([k, v]) => `${k}: ${v}`).join('\n')
+    text = `${header}\n\nError: ${displayTitle.value}\n${text}`
+  }
   try {
-    await navigator.clipboard.writeText(props.message || '')
+    await copyText(text)
     copied.value = true
     if (copiedTimer) clearTimeout(copiedTimer)
     copiedTimer = setTimeout(() => { copied.value = false }, 1500)
@@ -64,19 +125,43 @@ async function onCopy() {
 </script>
 
 <style scoped>
-/* The full error often contains a SQLAlchemy statement + parameters
-   block + multi-line traceback. Pre-wrap keeps long lines readable;
-   monospace + tabular layout keeps the structure scannable. The
-   max-height + overflow-auto prevents a 50-line traceback from
-   pushing the Close button off-screen. Operator-flagged 2026-05-26:
-   the prior :title="..." tooltip was unusable for content this long. */
+/* Context panel: muted labels in vellum, crisp values in parchment. The
+   2-column key/value grid keeps rows visually scannable when there are
+   many fields (Task, Queue, Target, Duration, Started, Finished, Retries,
+   Worker, Celery ID, Args). */
+.fc-err-context {
+  display: grid;
+  grid-template-columns: max-content 1fr;
+  column-gap: 14px;
+  row-gap: 4px;
+  margin: 0 0 14px 0;
+  font-size: 13px;
+}
+.fc-err-context dt {
+  color: rgb(var(--v-theme-on-surface-variant));
+  font-weight: 500;
+  white-space: nowrap;
+}
+.fc-err-context dd {
+  color: rgb(var(--v-theme-on-surface));
+  margin: 0;
+  font-variant-numeric: tabular-nums;
+  word-break: break-all;
+}
+
+/* Error pre block: high-contrast pairing. The page's `background` token
+   (obsidian #14171A) is darker than the modal card's `surface` (iron
+   #1E2228), so parchment text reads crisply against it. The prior pairing
+   used `surface-variant` which Vuetify auto-derives to a near-parchment
+   light value in this theme — pale-on-pale and unreadable. Operator-
+   flagged 2026-05-26 ("ui contrast is poor"). */
 .fc-err-pre {
-  font-family: ui-monospace, SFMono-Regular, Menlo, Consolas, monospace;
+  font-family: 'JetBrains Mono', ui-monospace, SFMono-Regular, Menlo, Consolas, monospace;
   font-size: 12px;
   line-height: 1.5;
   white-space: pre-wrap;
   word-break: break-word;
-  background: rgb(var(--v-theme-surface-variant, 38 36 41));
+  background: rgb(var(--v-theme-background));
   color: rgb(var(--v-theme-on-surface));
   padding: 12px 14px;
   border-radius: 6px;
diff --git a/frontend/src/components/settings/SystemActivityTab.vue b/frontend/src/components/settings/SystemActivityTab.vue
index f623bb0..0d6485c 100644
--- a/frontend/src/components/settings/SystemActivityTab.vue
+++ b/frontend/src/components/settings/SystemActivityTab.vue
@@ -62,7 +62,7 @@
               <td>
                 <button
                   type="button" class="fc-err-link"
-                  @click="openError(r.error_type, r.error_message)"
+                  @click="openError(r)"
                   :title="'Click for full error'"
                 >{{ r.error_type }}</button>
               </td>
@@ -145,8 +145,7 @@
 
     <ErrorDetailModal
       v-model="showErrorModal"
-      :title="errorModalTitle"
-      :message="errorModalMessage"
+      :row="errorModalRow"
     />
   </div>
 </template>
@@ -163,12 +162,10 @@ import QueuesTable from './QueuesTable.vue'
 // rollback + traceback content rendered as a cramped browser tooltip
 // you couldn't copy from or scroll within).
 const showErrorModal = ref(false)
-const errorModalTitle = ref('')
-const errorModalMessage = ref('')
+const errorModalRow = ref(null)
 
-function openError(title, message) {
-  errorModalTitle.value = title || 'Error details'
-  errorModalMessage.value = message || ''
+function openError(row) {
+  errorModalRow.value = row
   showErrorModal.value = true
 }
 

From 110c1c0e513fe71fdcfec86d220882ec69bef9fb Mon Sep 17 00:00:00 2001
From: Bryan Van Deusen <bryan.vandeusen@fabledsword.com>
Date: Tue, 26 May 2026 11:46:06 -0400
Subject: [PATCH 09/20] =?UTF-8?q?fix(maintenance):=20recover=5Finterrupted?=
 =?UTF-8?q?=5Ftasks=20=E2=80=94=20fold=20SELECT=20into=20UPDATE=E2=80=A6WH?=
 =?UTF-8?q?ERE=E2=80=A6RETURNING=20so=20the=20IN-list=20no=20longer=20blow?=
 =?UTF-8?q?s=20past=20psycopg's=2065535-parameter=20ceiling=20(operator-hi?=
 =?UTF-8?q?t=202026-05-26=20after=20deep=20scan=20orphan=20pile)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---
 backend/app/tasks/maintenance.py | 59 ++++++++++++++++----------------
 1 file changed, 29 insertions(+), 30 deletions(-)

diff --git a/backend/app/tasks/maintenance.py b/backend/app/tasks/maintenance.py
index 4ef590a..df854a3 100644
--- a/backend/app/tasks/maintenance.py
+++ b/backend/app/tasks/maintenance.py
@@ -54,41 +54,40 @@ def recover_interrupted_tasks() -> int:
     processing_cutoff = now - timedelta(minutes=STUCK_THRESHOLD_MINUTES)
     orphan_cutoff = now - timedelta(minutes=ORPHAN_PENDING_THRESHOLD_MINUTES)
     with SessionLocal() as session:
-        stuck_ids = session.execute(
-            select(ImportTask.id)
+        # Both sweeps used to be SELECT ids → UPDATE WHERE id IN (...) which
+        # blew past psycopg's 65535-parameter ceiling once a sweep covered
+        # tens of thousands of rows (operator hit it 2026-05-26 after the
+        # /import deep scan piled up orphans). Folding the SELECT into the
+        # UPDATE eliminates the IN-list entirely. RETURNING gives us back
+        # exactly the ids that flipped so the stuck sweep can still
+        # .delay() each one.
+        stuck_result = session.execute(
+            update(ImportTask)
             .where(ImportTask.status == "processing")
             .where(ImportTask.started_at < processing_cutoff)
-        ).scalars().all()
+            .values(
+                status="queued",
+                started_at=None,
+                error="recovered from stuck state",
+            )
+            .returning(ImportTask.id)
+        )
+        stuck_ids = [row[0] for row in stuck_result.all()]
 
-        orphan_ids = session.execute(
-            select(ImportTask.id)
+        orphan_result = session.execute(
+            update(ImportTask)
             .where(ImportTask.status.in_(["pending", "queued"]))
             .where(ImportTask.created_at < orphan_cutoff)
-        ).scalars().all()
-
-        if not stuck_ids and not orphan_ids:
-            return 0
-
-        if stuck_ids:
-            session.execute(
-                update(ImportTask)
-                .where(ImportTask.id.in_(stuck_ids))
-                .values(status="queued", started_at=None, error="recovered from stuck state")
-            )
-
-        if orphan_ids:
-            session.execute(
-                update(ImportTask)
-                .where(ImportTask.id.in_(orphan_ids))
-                .values(
-                    status="failed",
-                    error=(
-                        "orphan pending/queued swept by recover_interrupted_tasks "
-                        "(scanner likely crashed mid-enqueue); retry via "
-                        "/api/import/retry-failed"
-                    ),
-                )
+            .values(
+                status="failed",
+                error=(
+                    "orphan pending/queued swept by recover_interrupted_tasks "
+                    "(scanner likely crashed mid-enqueue); retry via "
+                    "/api/import/retry-failed"
+                ),
             )
+        )
+        orphan_count = orphan_result.rowcount or 0
 
         session.commit()
 
@@ -97,7 +96,7 @@ def recover_interrupted_tasks() -> int:
             for tid in stuck_ids:
                 import_media_file.delay(tid)
 
-        return len(stuck_ids) + len(orphan_ids)
+        return len(stuck_ids) + orphan_count
 
 
 @celery.task(name="backend.app.tasks.maintenance.cleanup_old_tasks")

From b0bfbc585a1cd7d5e885fc2ed7c0544c8372ea0c Mon Sep 17 00:00:00 2001
From: Bryan Van Deusen <bryan.vandeusen@fabledsword.com>
Date: Tue, 26 May 2026 11:47:40 -0400
Subject: [PATCH 10/20] =?UTF-8?q?fix(import-admin):=20retry-failed=20+=20c?=
 =?UTF-8?q?lear-stuck=20=E2=80=94=20same=20UPDATE=E2=80=A6WHERE=20pattern?=
 =?UTF-8?q?=20as=20the=20maintenance=20sweep,=20so=20neither=20endpoint=20?=
 =?UTF-8?q?can=20hit=20psycopg's=2065535-parameter=20ceiling=20once=20accu?=
 =?UTF-8?q?mulated=20row=20counts=20exceed=20~65k?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---
 backend/app/api/import_admin.py | 61 +++++++++++++++++----------------
 1 file changed, 32 insertions(+), 29 deletions(-)

diff --git a/backend/app/api/import_admin.py b/backend/app/api/import_admin.py
index 367305b..02f65d2 100644
--- a/backend/app/api/import_admin.py
+++ b/backend/app/api/import_admin.py
@@ -103,17 +103,22 @@ async def list_tasks():
 
 @import_admin_bp.route("/retry-failed", methods=["POST"])
 async def retry_failed():
+    # Fold SELECT into UPDATE…WHERE…RETURNING — the prior SELECT-then-
+    # UPDATE-WHERE-id-IN pattern blew past psycopg's 65535-parameter
+    # ceiling once failed_ids exceeded ~65k rows.
     async with get_session() as session:
-        failed_ids = (
-            await session.execute(select(ImportTask.id).where(ImportTask.status == "failed"))
-        ).scalars().all()
+        result = await session.execute(
+            update(ImportTask)
+            .where(ImportTask.status == "failed")
+            .values(
+                status="queued", error=None,
+                started_at=None, finished_at=None,
+            )
+            .returning(ImportTask.id)
+        )
+        failed_ids = [row[0] for row in result.all()]
         if not failed_ids:
             return jsonify({"retried": 0})
-        await session.execute(
-            update(ImportTask)
-            .where(ImportTask.id.in_(failed_ids))
-            .values(status="queued", error=None, started_at=None, finished_at=None)
-        )
         await session.commit()
 
     from ..tasks.import_file import import_media_file
@@ -138,28 +143,26 @@ async def clear_stuck():
     autoretry-looped for 2 days after a corrupt-data PIL OSError.
     """
     async with get_session() as session:
-        stuck_ids = (
-            await session.execute(
-                select(ImportTask.id).where(
-                    ImportTask.status.in_(["pending", "queued", "processing"])
-                )
+        # Fold SELECT into UPDATE…WHERE — see /retry-failed for the
+        # 65535-parameter ceiling rationale. rowcount is enough here
+        # because we don't need the ids afterward (no .delay()).
+        clear_result = await session.execute(
+            update(ImportTask)
+            .where(
+                ImportTask.status.in_(["pending", "queued", "processing"])
             )
-        ).scalars().all()
-        if stuck_ids:
-            await session.execute(
-                update(ImportTask)
-                .where(ImportTask.id.in_(stuck_ids))
-                .values(
-                    status="failed",
-                    finished_at=datetime.now(UTC),
-                    error=(
-                        "manually cleared via /api/import/clear-stuck "
-                        "— stuck in non-terminal state; retry once "
-                        "underlying cause (corrupt file, missing model, "
-                        "etc.) is resolved"
-                    ),
-                )
+            .values(
+                status="failed",
+                finished_at=datetime.now(UTC),
+                error=(
+                    "manually cleared via /api/import/clear-stuck "
+                    "— stuck in non-terminal state; retry once "
+                    "underlying cause (corrupt file, missing model, "
+                    "etc.) is resolved"
+                ),
             )
+        )
+        tasks_failed = clear_result.rowcount or 0
 
         # Finalize any 'running' ImportBatch that no longer has any
         # active children. The "Scanning..." banner is driven by
@@ -195,7 +198,7 @@ async def clear_stuck():
         await session.commit()
 
     return jsonify({
-        "tasks_failed": len(stuck_ids),
+        "tasks_failed": tasks_failed,
         "batches_finalized": finalized_batches,
     })
 

From 397021dcbd377d6b335437cdf323d1ad94ee459d Mon Sep 17 00:00:00 2001
From: Bryan Van Deusen <bryan.vandeusen@fabledsword.com>
Date: Tue, 26 May 2026 13:47:55 -0400
Subject: [PATCH 11/20] =?UTF-8?q?fix(importer):=20ImageProvenance=20(image?=
 =?UTF-8?q?=5Frecord=5Fid,=20post=5Fid)=20race-safe=20via=20savepoint=20+?=
 =?UTF-8?q?=20alembic=200021=20UNIQUE=20=E2=80=94=20closes=20the=20SELECT-?=
 =?UTF-8?q?then-INSERT=20window=20that=20planted=20duplicates=20and=20brok?=
 =?UTF-8?q?e=20.scalar=5Fone=5For=5Fnone()=20on=20every=20later=20deep-sca?=
 =?UTF-8?q?n=20rederive=20(MultipleResultsFound).=20Migration=20dedupes=20?=
 =?UTF-8?q?existing=20rows=20(min(id)=20per=20pair);=20model=20gains=20=5F?=
 =?UTF-8?q?=5Ftable=5Fargs=5F=5F;=20gallery-filter=20test=20that=20seeded?=
 =?UTF-8?q?=20duplicates=20dropped.?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---
 .../versions/0021_image_provenance_unique.py  |  54 +++++
 backend/app/models/image_provenance.py        |  20 +-
 backend/app/services/importer.py              |  29 ++-
 tests/test_gallery_provenance_filter.py       |  21 +-
 tests/test_importer_provenance_race.py        | 186 ++++++++++++++++++
 5 files changed, 284 insertions(+), 26 deletions(-)
 create mode 100644 alembic/versions/0021_image_provenance_unique.py
 create mode 100644 tests/test_importer_provenance_race.py

diff --git a/alembic/versions/0021_image_provenance_unique.py b/alembic/versions/0021_image_provenance_unique.py
new file mode 100644
index 0000000..b9be941
--- /dev/null
+++ b/alembic/versions/0021_image_provenance_unique.py
@@ -0,0 +1,54 @@
+"""provenance-race: dedupe + UNIQUE(image_record_id, post_id) on image_provenance
+
+Revision ID: 0021
+Revises: 0020
+Create Date: 2026-05-26
+
+Closes the race in Importer._apply_sidecar's existence-check + INSERT pattern.
+Two workers writing for the same (image, post) pair both saw no existing row
+and both inserted, leaving duplicates that then broke .scalar_one_or_none()
+on every subsequent deep-scan rederive against those images
+(MultipleResultsFound). Most plausibly seeded when the 5-min recovery sweep
+re-enqueued a still-running long-import task and the second worker collided
+with the first inside _apply_sidecar.
+
+Migration steps:
+  1. DELETE all but min(id) per (image_record_id, post_id) pair. Operator's
+     DB had 2 affected pairs at write-time; harmless no-op if zero.
+  2. Add UNIQUE constraint so the importer's new savepoint+IntegrityError
+     recovery path can trip on collision and re-select, mirroring
+     uq_source_artist_platform_url and uq_post_source_external_id.
+"""
+from typing import Sequence, Union
+
+from alembic import op
+
+revision: str = "0021"
+down_revision: Union[str, None] = "0020"
+branch_labels: Union[str, Sequence[str], None] = None
+depends_on: Union[str, Sequence[str], None] = None
+
+
+def upgrade() -> None:
+    op.execute(
+        """
+        DELETE FROM image_provenance ip1
+        USING image_provenance ip2
+        WHERE ip1.image_record_id = ip2.image_record_id
+          AND ip1.post_id = ip2.post_id
+          AND ip1.id > ip2.id
+        """
+    )
+    op.create_unique_constraint(
+        "uq_image_provenance_image_post",
+        "image_provenance",
+        ["image_record_id", "post_id"],
+    )
+
+
+def downgrade() -> None:
+    op.drop_constraint(
+        "uq_image_provenance_image_post",
+        "image_provenance",
+        type_="unique",
+    )
diff --git a/backend/app/models/image_provenance.py b/backend/app/models/image_provenance.py
index cd50887..ec3e756 100644
--- a/backend/app/models/image_provenance.py
+++ b/backend/app/models/image_provenance.py
@@ -1,14 +1,18 @@
 """ImageProvenance — links an ImageRecord to a Post.
 
-Many-to-one (one image, many provenance rows) enables the enrich-on-duplicate
-rule (spec §3): when a downloaded image is a pHash dupe of an existing
-record, we append a new provenance row to the existing record rather than
-dropping the metadata.
+One image can have many provenance rows — different posts each contribute
+metadata (enrich-on-duplicate rule, spec §3: a downloaded image that is a
+pHash dupe of an existing record gets a NEW provenance row for the new post
+appended, rather than the metadata being dropped). But the (image, post)
+pair is unique — alembic 0021 enforces uq_image_provenance_image_post
+after operator-flagged 2026-05-26 saw _apply_sidecar's existence-check +
+INSERT race plant duplicates that then broke .scalar_one_or_none() on
+every later deep-scan rederive (MultipleResultsFound).
 """
 
 from datetime import datetime
 
-from sqlalchemy import JSON, DateTime, ForeignKey, Integer, func
+from sqlalchemy import JSON, DateTime, ForeignKey, Integer, UniqueConstraint, func
 from sqlalchemy.orm import Mapped, mapped_column
 
 from .base import Base
@@ -16,6 +20,12 @@ from .base import Base
 
 class ImageProvenance(Base):
     __tablename__ = "image_provenance"
+    __table_args__ = (
+        UniqueConstraint(
+            "image_record_id", "post_id",
+            name="uq_image_provenance_image_post",
+        ),
+    )
 
     id: Mapped[int] = mapped_column(Integer, primary_key=True)
     image_record_id: Mapped[int] = mapped_column(
diff --git a/backend/app/services/importer.py b/backend/app/services/importer.py
index 3223ec7..997a853 100644
--- a/backend/app/services/importer.py
+++ b/backend/app/services/importer.py
@@ -784,6 +784,15 @@ class Importer:
             post.attachment_count = sd.attachment_count
         post.raw_metadata = sd.raw
 
+        # Race-safe (image_record_id, post_id) upsert — mirrors the
+        # _find_or_create_source/post savepoint pattern. The plain
+        # SELECT-then-INSERT pattern lost a race when two workers ran
+        # _apply_sidecar on the same (image, post) pair (e.g. the 5-min
+        # recovery sweep re-enqueued a still-running long import), planting
+        # duplicates that then broke .scalar_one_or_none() on every later
+        # deep-scan rederive (MultipleResultsFound). Alembic 0021 adds the
+        # uq_image_provenance_image_post UNIQUE so this savepoint actually
+        # trips on collision.
         exists = self.session.execute(
             select(ImageProvenance.id).where(
                 ImageProvenance.image_record_id == record.id,
@@ -791,14 +800,20 @@ class Importer:
             )
         ).scalar_one_or_none()
         if exists is None:
-            self.session.add(
-                ImageProvenance(
-                    image_record_id=record.id,
-                    post_id=post.id,
-                    source_id=src.id,
-                    captured_metadata=sd.raw,
+            sp = self.session.begin_nested()
+            try:
+                self.session.add(
+                    ImageProvenance(
+                        image_record_id=record.id,
+                        post_id=post.id,
+                        source_id=src.id,
+                        captured_metadata=sd.raw,
+                    )
                 )
-            )
+                self.session.flush()
+                sp.commit()
+            except IntegrityError:
+                sp.rollback()
         if record.primary_post_id is None:
             record.primary_post_id = post.id
         self.session.flush()
diff --git a/tests/test_gallery_provenance_filter.py b/tests/test_gallery_provenance_filter.py
index a7f2ba4..2eb93aa 100644
--- a/tests/test_gallery_provenance_filter.py
+++ b/tests/test_gallery_provenance_filter.py
@@ -69,20 +69,13 @@ async def test_scroll_post_id_filter(db):
     assert {x.id for x in page.images} == {i1.id, i2.id}
 
 
-@pytest.mark.asyncio
-async def test_scroll_post_id_dedups_multi_rows(db):
-    i1 = await _img(db, 1)
-    _, s, p = await _post(db, "A", "a", "10")
-    # two provenance rows, same image+post (enrich-on-duplicate shape)
-    db.add(ImageProvenance(image_record_id=i1.id, post_id=p.id,
-                           source_id=s.id))
-    await db.flush()
-    db.add(ImageProvenance(image_record_id=i1.id, post_id=p.id,
-                           source_id=s.id))
-    await db.flush()
-    svc = GalleryService(db)
-    page = await svc.scroll(cursor=None, limit=10, post_id=p.id)
-    assert [x.id for x in page.images] == [i1.id]  # appears once
+# test_scroll_post_id_dedups_multi_rows removed 2026-05-26: it deliberately
+# inserted two ImageProvenance rows with the same (image_record_id, post_id),
+# now prevented at the DB layer by uq_image_provenance_image_post (alembic
+# 0021). The EXISTS-based dedup in _provenance_clause is still useful for the
+# artist-id filter (one image legitimately joins many provenance rows via
+# different posts under the same artist), so the gallery_service logic is
+# unchanged.
 
 
 @pytest.mark.asyncio
diff --git a/tests/test_importer_provenance_race.py b/tests/test_importer_provenance_race.py
new file mode 100644
index 0000000..8c4d956
--- /dev/null
+++ b/tests/test_importer_provenance_race.py
@@ -0,0 +1,186 @@
+"""Race-safe ImageProvenance insert in Importer._apply_sidecar.
+
+Operator-flagged 2026-05-26: the prior SELECT-then-INSERT pattern lost a
+race when two workers ran _apply_sidecar on the same (image, post) pair
+(plausibly seeded when the 5-min recovery sweep re-enqueued a still-running
+long import). Duplicates then broke .scalar_one_or_none() on every later
+deep-scan rederive (MultipleResultsFound). Alembic 0021 added
+uq_image_provenance_image_post; the importer's new savepoint+IntegrityError
+recovery path now trips on collision and gracefully recovers.
+
+Tests cover:
+  - idempotent: re-running _apply_sidecar via _deep_rederive produces
+    exactly one ImageProvenance row.
+  - IntegrityError recovery: pre-seed a provenance row, force the first
+    SELECT to return None (simulating the race window where two workers
+    both observed no row), call _apply_sidecar — the savepoint INSERT
+    trips uq_image_provenance_image_post, gets rolled back, no exception
+    escapes, still exactly one row.
+"""
+
+import json
+from pathlib import Path
+
+import pytest
+from PIL import Image
+from sqlalchemy import func, select
+
+from backend.app.models import (
+    ImageProvenance,
+    ImageRecord,
+    ImportSettings,
+    Post,
+    Source,
+)
+from backend.app.services.importer import Importer
+from backend.app.services.thumbnailer import Thumbnailer
+
+pytestmark = pytest.mark.integration
+
+
+@pytest.fixture
+def import_layout(tmp_path):
+    import_root = tmp_path / "import"
+    images_root = tmp_path / "images"
+    import_root.mkdir()
+    images_root.mkdir()
+    return import_root, images_root
+
+
+@pytest.fixture
+def importer(db_sync, import_layout):
+    import_root, images_root = import_layout
+    settings = db_sync.execute(
+        select(ImportSettings).where(ImportSettings.id == 1)
+    ).scalar_one()
+    return Importer(
+        session=db_sync,
+        images_root=images_root,
+        import_root=import_root,
+        thumbnailer=Thumbnailer(images_root=images_root),
+        settings=settings,
+    )
+
+
+@pytest.fixture
+def deep_importer(db_sync, import_layout):
+    import_root, images_root = import_layout
+    settings = db_sync.execute(
+        select(ImportSettings).where(ImportSettings.id == 1)
+    ).scalar_one()
+    return Importer(
+        session=db_sync,
+        images_root=images_root,
+        import_root=import_root,
+        thumbnailer=Thumbnailer(images_root=images_root),
+        settings=settings,
+        deep=True,
+    )
+
+
+def _split(path: Path, orient, size=(256, 256)):
+    path.parent.mkdir(parents=True, exist_ok=True)
+    w, h = size
+    im = Image.new("L", size, 0)
+    px = im.load()
+    for y in range(h):
+        for x in range(w):
+            if (x / w if orient == "v" else y / h) >= 0.5:
+                px[x, y] = 255
+    im.convert("RGB").save(path, "JPEG")
+
+
+def _sidecar(media: Path, payload: dict):
+    media.with_suffix(".json").write_text(json.dumps(payload))
+
+
+def test_apply_sidecar_idempotent_on_deep_rederive(
+    importer, deep_importer, import_layout,
+):
+    """Normal-flow path: deep rederive on an already-imported image finds
+    the existing provenance row via .scalar_one_or_none() and skips the
+    insert. Exactly one ImageProvenance row after both runs."""
+    import_root, _ = import_layout
+    m = import_root / "Alice" / "a.jpg"
+    _split(m, "v")
+    _sidecar(m, {
+        "category": "patreon", "id": 555,
+        "url": "https://patreon.com/posts/555", "title": "Set 1",
+    })
+    r = importer.import_one(m)
+    assert r.status == "imported"
+
+    # Re-import via deep mode — sha matches → _deep_rederive → _apply_sidecar.
+    r2 = deep_importer.import_one(m)
+    assert r2.status == "refreshed"
+
+    count = importer.session.execute(
+        select(func.count()).select_from(ImageProvenance)
+    ).scalar_one()
+    assert count == 1
+
+
+def test_apply_sidecar_recovers_from_integrity_error(
+    importer, deep_importer, import_layout, db_sync, monkeypatch,
+):
+    """Race recovery: a row already exists for (image, post). We force the
+    importer's existence-check SELECT to return None for one call, mimicking
+    the race window where two workers both saw no row. The savepoint INSERT
+    then trips uq_image_provenance_image_post; the helper rolls the
+    savepoint back, no exception escapes, and the row count stays at 1.
+    """
+    import_root, _ = import_layout
+    m = import_root / "Bob" / "b.jpg"
+    _split(m, "v")
+    _sidecar(m, {
+        "category": "patreon", "id": 777,
+        "url": "https://patreon.com/posts/777", "title": "Set 2",
+    })
+    # First import lays the canonical provenance row.
+    r = importer.import_one(m)
+    assert r.status == "imported"
+    rec = importer.session.get(ImageRecord, r.image_id)
+    post = importer.session.execute(select(Post)).scalar_one()
+    src = importer.session.execute(select(Source)).scalar_one()
+    assert rec is not None
+    assert src is not None
+
+    # Monkeypatch session.execute so the FIRST select inside _apply_sidecar's
+    # existence-check returns a "no row" wrapper. Subsequent selects (e.g.
+    # the find_or_create_source / find_or_create_post existence checks
+    # earlier in _apply_sidecar) all run normally; we intercept only the
+    # ImageProvenance lookup, identified by the SELECT's target columns
+    # mentioning image_provenance.
+    real_execute = db_sync.execute
+    intercepted = [False]
+
+    def _intercepting_execute(stmt, *args, **kwargs):
+        text = str(stmt)
+        if (
+            not intercepted[0]
+            and "image_provenance" in text
+            and "image_record_id" in text
+            and "post_id" in text
+        ):
+            intercepted[0] = True
+
+            class _ForcedMiss:
+                def scalar_one_or_none(self):
+                    return None
+            return _ForcedMiss()
+        return real_execute(stmt, *args, **kwargs)
+
+    monkeypatch.setattr(db_sync, "execute", _intercepting_execute)
+
+    # Re-import via deep mode → _deep_rederive → _apply_sidecar. With the
+    # provenance-SELECT forced to miss, the helper will attempt the INSERT,
+    # trip uq_image_provenance_image_post, catch IntegrityError, and recover.
+    r2 = deep_importer.import_one(m)
+    assert r2.status == "refreshed"
+
+    # Lift the intercept, then verify the row count.
+    monkeypatch.undo()
+    count = db_sync.execute(
+        select(func.count()).select_from(ImageProvenance)
+    ).scalar_one()
+    assert count == 1

From 0bc5767a2bb848bffbfd42f416ab9e8e9a3855f1 Mon Sep 17 00:00:00 2001
From: Bryan Van Deusen <bryan.vandeusen@fabledsword.com>
Date: Tue, 26 May 2026 14:19:29 -0400
Subject: [PATCH 12/20] =?UTF-8?q?fix(importer):=20Source=20=3D=20one=20per?=
 =?UTF-8?q?=20(artist,=20platform),=20not=20one=20per=20post=20=E2=80=94?=
 =?UTF-8?q?=20filesystem=20importer's=20sidecar=20paths=20now=20reuse=20th?=
 =?UTF-8?q?e=20artist's=20existing=20subscription=20Source=20(or=20create?=
 =?UTF-8?q?=20one=20synthetic=20anchor=20with=20enabled=3DFalse)=20instead?=
 =?UTF-8?q?=20of=20fabricating=20a=20new=20Source=20per=20post=20URL.=20Al?=
 =?UTF-8?q?embic=200022=20consolidates=20existing=20per-post=20Sources=20t?=
 =?UTF-8?q?o=20canonical=20(prefers=20campaign=20URL;=20falls=20back=20to?=
 =?UTF-8?q?=20sidecar:<platform>:<slug>)=20and=20re-parents=20Posts=20+=20?=
 =?UTF-8?q?ImageProvenance,=20merging=20Post=20collisions.?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Operator-flagged 2026-05-26: Atole artist detail page showed 406 Sources where 1 was right.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---
 .../0022_source_per_artist_platform.py        | 181 ++++++++++++++++++
 backend/app/services/importer.py              |  67 ++++++-
 tests/test_importer_upsert_helpers.py         |  65 +++++++
 3 files changed, 307 insertions(+), 6 deletions(-)
 create mode 100644 alembic/versions/0022_source_per_artist_platform.py

diff --git a/alembic/versions/0022_source_per_artist_platform.py b/alembic/versions/0022_source_per_artist_platform.py
new file mode 100644
index 0000000..eaee97f
--- /dev/null
+++ b/alembic/versions/0022_source_per_artist_platform.py
@@ -0,0 +1,181 @@
+"""source-collapse: one Source per (artist, platform) — consolidate junk per-post Sources
+
+Revision ID: 0022
+Revises: 0021
+Create Date: 2026-05-26
+
+Closes the operator-flagged 2026-05-26 issue where the filesystem importer
+called _find_or_create_source(url=sd.post_url), creating one Source row per
+imported post URL. Operator's Atole artist had 406 Source rows where there
+should have been 1 (the /cw/Atole subscription Source).
+
+Source represents a subscription feed (one per artist+platform — the
+gallery-dl URL polled by the FC-3 downloader). Posts hang off it. The
+filesystem importer was misusing Source as a per-post key.
+
+Migration steps per (artist_id, platform) group with >1 Source:
+  1. Pick canonical — prefer a URL NOT matching '/posts/<id>$' (real
+     campaign URL like /cw/Atole); else min(id).
+  2. Reparent Posts and ImageProvenance off the other Sources onto canonical.
+  3. Merge any Posts that collide on (canonical_source_id, external_post_id)
+     by repointing ImageProvenance + ImageRecord.primary_post_id to the
+     earliest Post, dedupe ImageProvenance, delete the loser Posts.
+  4. Delete the orphan Source rows.
+  5. If the canonical Source's URL still looks like a per-post URL (no
+     campaign URL existed among candidates), rewrite it to
+     'sidecar:<platform>:<artist_slug>' so the artist detail page shows
+     something readable.
+"""
+from typing import Sequence, Union
+
+from alembic import op
+from sqlalchemy import text
+
+revision: str = "0022"
+down_revision: Union[str, None] = "0021"
+branch_labels: Union[str, Sequence[str], None] = None
+depends_on: Union[str, Sequence[str], None] = None
+
+_POST_URL_RE = r"/posts/[^/]+$"
+
+
+def upgrade() -> None:
+    conn = op.get_bind()
+
+    # Find (artist_id, platform) groups with > 1 Source row.
+    groups = conn.execute(text("""
+        SELECT artist_id, platform
+        FROM source
+        GROUP BY artist_id, platform
+        HAVING COUNT(*) > 1
+    """)).fetchall()
+
+    for artist_id, platform in groups:
+        rows = conn.execute(
+            text("""
+                SELECT id, url FROM source
+                WHERE artist_id = :a AND platform = :p
+                ORDER BY id ASC
+            """),
+            {"a": artist_id, "p": platform},
+        ).fetchall()
+
+        # Canonical: first row whose URL doesn't look like a per-post URL;
+        # else min(id).
+        canonical_id = None
+        for sid, url in rows:
+            if not _matches_post_url(url):
+                canonical_id = sid
+                break
+        if canonical_id is None:
+            canonical_id = rows[0][0]
+
+        other_ids = [sid for sid, _ in rows if sid != canonical_id]
+        if not other_ids:
+            continue
+
+        # Reparent Posts off the other Sources.
+        conn.execute(
+            text("""
+                UPDATE post SET source_id = :canonical
+                WHERE source_id = ANY(:others)
+            """),
+            {"canonical": canonical_id, "others": other_ids},
+        )
+        # Reparent ImageProvenance.source_id similarly (denormalized FK).
+        conn.execute(
+            text("""
+                UPDATE image_provenance SET source_id = :canonical
+                WHERE source_id = ANY(:others)
+            """),
+            {"canonical": canonical_id, "others": other_ids},
+        )
+
+        # Merge any Posts colliding on (canonical, external_post_id) after
+        # reparent. In practice within a single (artist, platform) group
+        # these should be rare — each gallery-dl post has a unique id — but
+        # safe to handle.
+        post_collisions = conn.execute(
+            text("""
+                SELECT external_post_id, ARRAY_AGG(id ORDER BY id) AS ids
+                FROM post
+                WHERE source_id = :canonical
+                GROUP BY external_post_id
+                HAVING COUNT(*) > 1
+            """),
+            {"canonical": canonical_id},
+        ).fetchall()
+        for _epid, post_ids in post_collisions:
+            keep = post_ids[0]
+            drops = post_ids[1:]
+            conn.execute(
+                text("""
+                    UPDATE image_provenance SET post_id = :keep
+                    WHERE post_id = ANY(:drops)
+                """),
+                {"keep": keep, "drops": drops},
+            )
+            conn.execute(
+                text("""
+                    UPDATE image_record SET primary_post_id = :keep
+                    WHERE primary_post_id = ANY(:drops)
+                """),
+                {"keep": keep, "drops": drops},
+            )
+            # Repointed provenance rows may now collide on
+            # uq_image_provenance_image_post (alembic 0021). Same dedupe
+            # pattern: keep min(id) per (image_record_id, post_id).
+            conn.execute(text("""
+                DELETE FROM image_provenance ip1
+                USING image_provenance ip2
+                WHERE ip1.image_record_id = ip2.image_record_id
+                  AND ip1.post_id = ip2.post_id
+                  AND ip1.id > ip2.id
+            """))
+            conn.execute(
+                text("DELETE FROM post WHERE id = ANY(:drops)"),
+                {"drops": drops},
+            )
+
+        # Drop the orphan Sources.
+        conn.execute(
+            text("DELETE FROM source WHERE id = ANY(:others)"),
+            {"others": other_ids},
+        )
+
+        # If the canonical's URL still looks per-post (no campaign URL
+        # existed among the candidates), rewrite to a synthetic anchor so
+        # the artist detail page renders something readable.
+        canonical_url = conn.execute(
+            text("SELECT url FROM source WHERE id = :id"),
+            {"id": canonical_id},
+        ).scalar_one()
+        if _matches_post_url(canonical_url):
+            slug = conn.execute(
+                text("SELECT slug FROM artist WHERE id = :id"),
+                {"id": artist_id},
+            ).scalar_one()
+            conn.execute(
+                text("""
+                    UPDATE source
+                    SET url = :new_url, enabled = false
+                    WHERE id = :id
+                """),
+                {
+                    "id": canonical_id,
+                    "new_url": f"sidecar:{platform}:{slug}",
+                },
+            )
+
+
+def downgrade() -> None:
+    # Lossy migration — orphan Sources deleted, Posts reparented, Posts
+    # merged. No safe downgrade. If you need to roll back the schema
+    # invariant, fork from 0021 and re-run filesystem imports.
+    pass
+
+
+def _matches_post_url(url: str) -> bool:
+    """True if url ends with /posts/<token> (gallery-dl-style per-post URL)."""
+    import re
+    return bool(re.search(_POST_URL_RE, url or ""))
diff --git a/backend/app/services/importer.py b/backend/app/services/importer.py
index 997a853..e2b94eb 100644
--- a/backend/app/services/importer.py
+++ b/backend/app/services/importer.py
@@ -247,6 +247,62 @@ class Importer:
                 )
             ).scalar_one()
 
+    def _source_for_sidecar(
+        self, *, artist_id: int, platform: str, artist_slug: str,
+    ) -> Source:
+        """Filesystem-import sidecar Source resolver.
+
+        Source represents a subscription feed (one per artist+platform — the
+        gallery-dl URL polled by the FC-3 downloader). The filesystem importer
+        used to call _find_or_create_source(url=sd.post_url), which created
+        one Source row per post URL — 100s of junk Sources per artist, all
+        with enabled=True, polluting the artist detail page and tricking the
+        subscription checker into trying to poll patreon post URLs as feeds.
+        Operator-flagged 2026-05-26.
+
+        New behaviour: if any Source row exists for (artist_id, platform),
+        reuse it regardless of its URL — the artist's real subscription Source
+        (created by the downloader / extension / UI) is the canonical
+        attachment point for filesystem-imported posts. If none exists, create
+        ONE synthetic anchor with url='sidecar:<platform>:<artist_slug>' and
+        enabled=False (so the subscription checker doesn't poll it).
+        """
+        existing = self.session.execute(
+            select(Source)
+            .where(
+                Source.artist_id == artist_id,
+                Source.platform == platform,
+            )
+            .order_by(Source.id.asc())
+            .limit(1)
+        ).scalar_one_or_none()
+        if existing is not None:
+            return existing
+        synthetic_url = f"sidecar:{platform}:{artist_slug}"
+        sp = self.session.begin_nested()
+        try:
+            row = Source(
+                artist_id=artist_id,
+                platform=platform,
+                url=synthetic_url,
+                enabled=False,
+            )
+            self.session.add(row)
+            self.session.flush()
+            sp.commit()
+            return row
+        except IntegrityError:
+            sp.rollback()
+            return self.session.execute(
+                select(Source)
+                .where(
+                    Source.artist_id == artist_id,
+                    Source.platform == platform,
+                )
+                .order_by(Source.id.asc())
+                .limit(1)
+            ).scalar_one()
+
     def _find_or_create_post(
         self, *, source_id: int, external_post_id: str,
     ) -> Post:
@@ -315,9 +371,8 @@ class Importer:
             return None
         sd = parse_sidecar(data)
         platform = sd.platform or "unknown"
-        url = sd.post_url or f"sidecar:{platform}"
-        src = self._find_or_create_source(
-            artist_id=artist.id, platform=platform, url=url,
+        src = self._source_for_sidecar(
+            artist_id=artist.id, platform=platform, artist_slug=artist.slug,
         )
         epid = sd.external_post_id or sc.stem
         return self._find_or_create_post(
@@ -763,9 +818,9 @@ class Importer:
             src = explicit_source
         else:
             platform = sd.platform or "unknown"
-            url = sd.post_url or f"sidecar:{platform}"
-            src = self._find_or_create_source(
-                artist_id=artist.id, platform=platform, url=url,
+            src = self._source_for_sidecar(
+                artist_id=artist.id, platform=platform,
+                artist_slug=artist.slug,
             )
 
         epid = sd.external_post_id or sc.stem
diff --git a/tests/test_importer_upsert_helpers.py b/tests/test_importer_upsert_helpers.py
index ca8a3bc..8472031 100644
--- a/tests/test_importer_upsert_helpers.py
+++ b/tests/test_importer_upsert_helpers.py
@@ -145,3 +145,68 @@ def test_find_or_create_source_recovers_from_integrity_error(
         artist_id=artist_row.id, platform="patreon", url=canonical_url,
     )
     assert recovered.id == pre_existing.id
+
+
+def test_source_for_sidecar_reuses_existing_subscription(
+    importer, artist_row, db_sync,
+):
+    """The filesystem-import sidecar resolver should attach to whatever
+    Source already exists for (artist, platform) — the canonical subscription
+    Source — regardless of its URL. Without this, every imported post
+    spawned its own Source row.
+    """
+    canonical = Source(
+        artist_id=artist_row.id, platform="patreon",
+        url="https://www.patreon.com/cw/testartist", enabled=True,
+    )
+    db_sync.add(canonical)
+    db_sync.flush()
+
+    resolved = importer._source_for_sidecar(
+        artist_id=artist_row.id, platform="patreon",
+        artist_slug=artist_row.slug,
+    )
+    assert resolved.id == canonical.id
+
+
+def test_source_for_sidecar_creates_synthetic_anchor_when_none_exists(
+    importer, artist_row, db_sync,
+):
+    """No subscription Source for this (artist, platform) yet. The helper
+    creates one synthetic anchor (enabled=False, url='sidecar:<plat>:<slug>')
+    so subsequent imports reuse it instead of spawning per-post Sources.
+    """
+    resolved = importer._source_for_sidecar(
+        artist_id=artist_row.id, platform="pixiv",
+        artist_slug=artist_row.slug,
+    )
+    assert resolved.url == f"sidecar:pixiv:{artist_row.slug}"
+    assert resolved.enabled is False
+    assert resolved.artist_id == artist_row.id
+    assert resolved.platform == "pixiv"
+
+    # Second call returns the same row (no new Source spawned).
+    again = importer._source_for_sidecar(
+        artist_id=artist_row.id, platform="pixiv",
+        artist_slug=artist_row.slug,
+    )
+    assert again.id == resolved.id
+
+
+def test_source_for_sidecar_distinct_platforms_distinct_anchors(
+    importer, artist_row, db_sync,
+):
+    """One synthetic anchor per (artist, platform). Different platforms get
+    different anchors even when no campaign Source exists for either.
+    """
+    p = importer._source_for_sidecar(
+        artist_id=artist_row.id, platform="patreon",
+        artist_slug=artist_row.slug,
+    )
+    x = importer._source_for_sidecar(
+        artist_id=artist_row.id, platform="pixiv",
+        artist_slug=artist_row.slug,
+    )
+    assert p.id != x.id
+    assert p.platform == "patreon"
+    assert x.platform == "pixiv"

From 992f38ec20fec7a9d04010f03799a2e2acc05208 Mon Sep 17 00:00:00 2001
From: Bryan Van Deusen <bryan.vandeusen@fabledsword.com>
Date: Tue, 26 May 2026 14:39:57 -0400
Subject: [PATCH 13/20] fix(test): drop unused Post binding in
 test_importer_provenance_race (ruff F841)

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---
 tests/test_importer_provenance_race.py | 2 --
 1 file changed, 2 deletions(-)

diff --git a/tests/test_importer_provenance_race.py b/tests/test_importer_provenance_race.py
index 8c4d956..68ba96c 100644
--- a/tests/test_importer_provenance_race.py
+++ b/tests/test_importer_provenance_race.py
@@ -29,7 +29,6 @@ from backend.app.models import (
     ImageProvenance,
     ImageRecord,
     ImportSettings,
-    Post,
     Source,
 )
 from backend.app.services.importer import Importer
@@ -140,7 +139,6 @@ def test_apply_sidecar_recovers_from_integrity_error(
     r = importer.import_one(m)
     assert r.status == "imported"
     rec = importer.session.get(ImageRecord, r.image_id)
-    post = importer.session.execute(select(Post)).scalar_one()
     src = importer.session.execute(select(Source)).scalar_one()
     assert rec is not None
     assert src is not None

From 2e8d7c960c0408bd470181d39e5d30024c318b17 Mon Sep 17 00:00:00 2001
From: Bryan Van Deusen <bryan.vandeusen@fabledsword.com>
Date: Tue, 26 May 2026 15:59:28 -0400
Subject: [PATCH 14/20] =?UTF-8?q?feat(artist):=20post=5Fcount=20on=20the?=
 =?UTF-8?q?=20artist=20overview=20response=20=E2=80=94=20drives=20the=20Po?=
 =?UTF-8?q?sts/Gallery=20default-tab=20fallback=20in=20the=20upcoming=20Ar?=
 =?UTF-8?q?tistView=20redesign?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---
 backend/app/services/artist_service.py | 10 ++++++++++
 tests/test_api_artist.py               | 24 ++++++++++++++++++++++++
 2 files changed, 34 insertions(+)

diff --git a/backend/app/services/artist_service.py b/backend/app/services/artist_service.py
index 271778e..c6aff2b 100644
--- a/backend/app/services/artist_service.py
+++ b/backend/app/services/artist_service.py
@@ -111,12 +111,22 @@ class ArtistService:
             )
         ).all()
 
+        post_count = (
+            await self.session.execute(
+                select(func.count(func.distinct(Post.id)))
+                .select_from(Post)
+                .join(Source, Source.id == Post.source_id)
+                .where(Source.artist_id == aid)
+            )
+        ).scalar_one()
+
         return {
             "id": artist.id,
             "name": artist.name,
             "slug": artist.slug,
             "is_subscription": bool(artist.is_subscription),
             "image_count": int(image_count),
+            "post_count": int(post_count),
             "date_range": {
                 "min": dmin.isoformat() if dmin else None,
                 "max": dmax.isoformat() if dmax else None,
diff --git a/tests/test_api_artist.py b/tests/test_api_artist.py
index 599093f..0a61d3b 100644
--- a/tests/test_api_artist.py
+++ b/tests/test_api_artist.py
@@ -29,6 +29,30 @@ async def test_artist_overview_ok(client, db):
     body = await resp.get_json()
     assert body["name"] == "Mira"
     assert body["image_count"] == 0
+    assert body["post_count"] == 0
+
+
+@pytest.mark.asyncio
+async def test_artist_overview_post_count(client, db):
+    from backend.app.models import Post, Source
+
+    a = Artist(name="Lyra", slug="lyra")
+    db.add(a)
+    await db.flush()
+    s = Source(
+        artist_id=a.id, platform="patreon",
+        url="https://patreon.com/cw/lyra", enabled=True,
+    )
+    db.add(s)
+    await db.flush()
+    db.add(Post(source_id=s.id, external_post_id="p1"))
+    db.add(Post(source_id=s.id, external_post_id="p2"))
+    await db.flush()
+    await db.commit()
+    resp = await client.get("/api/artist/lyra")
+    assert resp.status_code == 200
+    body = await resp.get_json()
+    assert body["post_count"] == 2
 
 
 @pytest.mark.asyncio

From a36f72b3834b61896114588488d95f43e8ece408 Mon Sep 17 00:00:00 2001
From: Bryan Van Deusen <bryan.vandeusen@fabledsword.com>
Date: Tue, 26 May 2026 15:59:46 -0400
Subject: [PATCH 15/20] =?UTF-8?q?feat(artist-view):=20ArtistHeader=20?=
 =?UTF-8?q?=E2=80=94=20sticky=20frosted=20bar=20(top:64px)=20matching=20To?=
 =?UTF-8?q?pNav=20recipe,=20hosts=20name=20+=20image/post=20stats=20+=20ta?=
 =?UTF-8?q?b=20strip?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---
 .../src/components/artist/ArtistHeader.vue    | 116 ++++++++++++++++++
 1 file changed, 116 insertions(+)
 create mode 100644 frontend/src/components/artist/ArtistHeader.vue

diff --git a/frontend/src/components/artist/ArtistHeader.vue b/frontend/src/components/artist/ArtistHeader.vue
new file mode 100644
index 0000000..8b39cca
--- /dev/null
+++ b/frontend/src/components/artist/ArtistHeader.vue
@@ -0,0 +1,116 @@
+<template>
+  <header class="fc-artist-header">
+    <div class="fc-artist-header__left">
+      <h1 class="fc-artist-header__name">{{ name }}</h1>
+      <span v-if="stats" class="fc-artist-header__stats">{{ stats }}</span>
+    </div>
+    <v-tabs
+      :model-value="modelValue"
+      color="accent"
+      density="compact"
+      class="fc-artist-header__tabs"
+      @update:model-value="$emit('update:modelValue', $event)"
+    >
+      <v-tab value="posts">
+        Posts
+        <span v-if="postCount != null" class="fc-artist-header__tab-count">
+          ({{ postCount }})
+        </span>
+      </v-tab>
+      <v-tab value="gallery">
+        Gallery
+        <span v-if="imageCount != null" class="fc-artist-header__tab-count">
+          ({{ imageCount }})
+        </span>
+      </v-tab>
+      <v-tab value="management">Management</v-tab>
+    </v-tabs>
+  </header>
+</template>
+
+<script setup>
+import { computed } from 'vue'
+
+const props = defineProps({
+  name:        { type: String, required: true },
+  imageCount:  { type: Number, default: null },
+  postCount:   { type: Number, default: null },
+  lastAdded:   { type: String, default: null },
+  modelValue:  { type: String, required: true },
+})
+
+defineEmits(['update:modelValue'])
+
+const stats = computed(() => {
+  const parts = []
+  if (props.imageCount != null) {
+    parts.push(`${props.imageCount} image${props.imageCount === 1 ? '' : 's'}`)
+  }
+  if (props.lastAdded) {
+    parts.push(`last added ${props.lastAdded.slice(0, 10)}`)
+  }
+  return parts.join(' · ')
+})
+</script>
+
+<style scoped>
+/* Matches TopNav.vue's frosted recipe exactly — top:64px parks it under
+   the 64px-tall TopNav with no visible seam. */
+.fc-artist-header {
+  position: sticky;
+  top: 64px;
+  z-index: 4;
+  display: flex;
+  align-items: center;
+  gap: 1rem;
+  padding: 0.5rem 1rem;
+  background: linear-gradient(
+    to bottom,
+    rgba(20, 23, 26, 0.92) 0%,
+    rgba(20, 23, 26, 0.65) 60%,
+    rgba(20, 23, 26, 0) 100%
+  );
+  backdrop-filter: blur(2px);
+  -webkit-backdrop-filter: blur(2px);
+}
+
+.fc-artist-header__left {
+  flex: 1 1 0;
+  min-width: 0;
+  display: flex;
+  align-items: baseline;
+  gap: 12px;
+  overflow: hidden;
+}
+
+.fc-artist-header__name {
+  font-family: 'Fraunces', Georgia, serif;
+  font-size: 24px;
+  font-weight: 500;
+  margin: 0;
+  color: rgb(var(--v-theme-on-surface));
+  white-space: nowrap;
+  overflow: hidden;
+  text-overflow: ellipsis;
+}
+
+.fc-artist-header__stats {
+  font-size: 13px;
+  color: rgb(var(--v-theme-on-surface-variant));
+  font-variant-numeric: tabular-nums;
+  white-space: nowrap;
+  overflow: hidden;
+  text-overflow: ellipsis;
+}
+
+.fc-artist-header__tabs {
+  flex: 0 0 auto;
+}
+
+.fc-artist-header__tab-count {
+  margin-left: 4px;
+  font-size: 12px;
+  color: rgb(var(--v-theme-on-surface-variant));
+  font-variant-numeric: tabular-nums;
+}
+</style>

From c07effb59331c4808e29ce6e4b672d7a14947299 Mon Sep 17 00:00:00 2001
From: Bryan Van Deusen <bryan.vandeusen@fabledsword.com>
Date: Tue, 26 May 2026 16:00:02 -0400
Subject: [PATCH 16/20] =?UTF-8?q?feat(artist-view):=20ArtistPostsTab=20?=
 =?UTF-8?q?=E2=80=94=20PostCard=20infinite-scroll=20list,=20artist=5Fid=20?=
 =?UTF-8?q?pinned,=20no=20filter=20bar;=20mirrors=20PostsView=20mechanics?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---
 .../src/components/artist/ArtistPostsTab.vue  | 90 +++++++++++++++++++
 1 file changed, 90 insertions(+)
 create mode 100644 frontend/src/components/artist/ArtistPostsTab.vue

diff --git a/frontend/src/components/artist/ArtistPostsTab.vue b/frontend/src/components/artist/ArtistPostsTab.vue
new file mode 100644
index 0000000..b575531
--- /dev/null
+++ b/frontend/src/components/artist/ArtistPostsTab.vue
@@ -0,0 +1,90 @@
+<template>
+  <div class="fc-artist-posts">
+    <v-alert v-if="store.error" type="error" variant="tonal" closable class="mb-3">
+      {{ String(store.error) }}
+    </v-alert>
+
+    <div v-if="store.loading && store.items.length === 0" class="fc-artist-posts__loading">
+      <v-progress-circular indeterminate color="accent" size="36" />
+    </div>
+
+    <div v-else-if="store.items.length === 0 && store.done" class="fc-artist-posts__empty">
+      <p>No posts for this artist yet. Switch to
+        <a href="#" @click.prevent="$emit('switch-tab', 'gallery')">Gallery</a>
+        to see imported images, or visit
+        <RouterLink to="/subscriptions">Subscriptions</RouterLink>
+        to start capturing posts.
+      </p>
+    </div>
+
+    <div v-else>
+      <PostCard v-for="p in store.items" :key="p.id" :post="p" />
+      <div ref="sentinel" class="fc-artist-posts__sentinel">
+        <v-progress-circular v-if="store.loading" indeterminate color="accent" size="24" />
+        <span v-else-if="store.done" class="fc-artist-posts__end">End of stream</span>
+      </div>
+    </div>
+  </div>
+</template>
+
+<script setup>
+import { onMounted, onUnmounted, ref, watch } from 'vue'
+import { RouterLink } from 'vue-router'
+
+import { usePostsStore } from '../../stores/posts.js'
+import PostCard from '../posts/PostCard.vue'
+
+const props = defineProps({
+  artistId: { type: Number, required: true },
+})
+
+defineEmits(['switch-tab'])
+
+const store = usePostsStore()
+const sentinel = ref(null)
+let observer = null
+
+async function reload () {
+  await store.loadInitial({ artist_id: props.artistId, platform: null })
+}
+
+watch(() => props.artistId, reload)
+
+onMounted(async () => {
+  await reload()
+  observer = new IntersectionObserver((entries) => {
+    if (entries.some(e => e.isIntersecting)) {
+      store.loadMore()
+    }
+  }, { rootMargin: '400px 0px' })
+  if (sentinel.value) observer.observe(sentinel.value)
+})
+
+onUnmounted(() => {
+  if (observer) observer.disconnect()
+})
+</script>
+
+<style scoped>
+.fc-artist-posts {
+  max-width: 900px;
+  margin: 0 auto;
+}
+.fc-artist-posts__loading,
+.fc-artist-posts__empty {
+  display: flex;
+  justify-content: center;
+  padding: 2rem;
+  color: rgb(var(--v-theme-on-surface-variant));
+}
+.fc-artist-posts__sentinel {
+  display: flex;
+  justify-content: center;
+  padding: 1.5rem 0;
+  min-height: 2rem;
+}
+.fc-artist-posts__end {
+  color: rgb(var(--v-theme-on-surface-variant));
+  font-size: 0.85rem;
+}
+</style>

From cce014be3ad839da33181f19041cd16ea79558a8 Mon Sep 17 00:00:00 2001
From: Bryan Van Deusen <bryan.vandeusen@fabledsword.com>
Date: Tue, 26 May 2026 16:00:14 -0400
Subject: [PATCH 17/20] =?UTF-8?q?feat(artist-view):=20ArtistGalleryTab=20?=
 =?UTF-8?q?=E2=80=94=20MasonryGrid=20wired=20to=20the=20artist=20store's?=
 =?UTF-8?q?=20existing=20loadMoreImages;=20modal-open=20preserves=20=3Ftab?=
 =?UTF-8?q?=3D.=20No=20global=20gallery-store=20coupling=20(avoids=20cross?=
 =?UTF-8?q?-pollution=20into=20/gallery)?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---
 .../components/artist/ArtistGalleryTab.vue    | 48 +++++++++++++++++++
 1 file changed, 48 insertions(+)
 create mode 100644 frontend/src/components/artist/ArtistGalleryTab.vue

diff --git a/frontend/src/components/artist/ArtistGalleryTab.vue b/frontend/src/components/artist/ArtistGalleryTab.vue
new file mode 100644
index 0000000..0cc6b7c
--- /dev/null
+++ b/frontend/src/components/artist/ArtistGalleryTab.vue
@@ -0,0 +1,48 @@
+<template>
+  <div class="fc-artist-gallery">
+    <MasonryGrid
+      :items="store.images"
+      :loading="store.imagesLoading"
+      :has-more="store.hasMoreImages"
+      @load-more="store.loadMoreImages(props.slug)"
+      @open="openImage"
+    />
+  </div>
+</template>
+
+<script setup>
+import { onMounted, watch } from 'vue'
+import { useRoute, useRouter } from 'vue-router'
+
+import { useArtistStore } from '../../stores/artist.js'
+import { useModalStore } from '../../stores/modal.js'
+import MasonryGrid from '../discovery/MasonryGrid.vue'
+
+const props = defineProps({
+  slug: { type: String, required: true },
+})
+
+const store = useArtistStore()
+const modal = useModalStore()
+const route = useRoute()
+const router = useRouter()
+
+onMounted(() => {
+  const initial = parseInt(route.query.image, 10)
+  if (!isNaN(initial)) modal.open(initial)
+})
+
+watch(() => route.query.image, (q) => {
+  const id = parseInt(q, 10)
+  if (!isNaN(id) && id !== modal.currentImageId) modal.open(id)
+  else if (isNaN(id) && modal.currentImageId !== null) modal.close()
+})
+
+function openImage (id) {
+  router.push({ query: { ...route.query, image: id } })
+}
+</script>
+
+<style scoped>
+.fc-artist-gallery { min-width: 0; }
+</style>

From ca55d92c680af4224bea6ddad747b2f1fd29a32a Mon Sep 17 00:00:00 2001
From: Bryan Van Deusen <bryan.vandeusen@fabledsword.com>
Date: Tue, 26 May 2026 16:00:47 -0400
Subject: [PATCH 18/20] =?UTF-8?q?feat(artist-view):=20ArtistManagementTab?=
 =?UTF-8?q?=20=E2=80=94=20Overview=20chips=20(Subscription=20badge=20+=20s?=
 =?UTF-8?q?ubscription=20count)=20+=20Frequent=20tags=20+=20Activity=20spa?=
 =?UTF-8?q?rkline=20+=20Subscriptions=20table=20+=20Danger=20zone.=20'View?=
 =?UTF-8?q?=20posts'=20chip=20and=20'Credential=20health=20=C2=B7=20FC-3b'?=
 =?UTF-8?q?=20placeholder=20chip=20dropped;=20'Sources'=20section=20rename?=
 =?UTF-8?q?d=20to=20'Subscriptions'?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---
 .../components/artist/ArtistManagementTab.vue | 133 ++++++++++++++++++
 1 file changed, 133 insertions(+)
 create mode 100644 frontend/src/components/artist/ArtistManagementTab.vue

diff --git a/frontend/src/components/artist/ArtistManagementTab.vue b/frontend/src/components/artist/ArtistManagementTab.vue
new file mode 100644
index 0000000..b78c2a1
--- /dev/null
+++ b/frontend/src/components/artist/ArtistManagementTab.vue
@@ -0,0 +1,133 @@
+<template>
+  <div class="fc-artist-mgmt">
+    <section class="fc-artist-mgmt__sec">
+      <h2 class="fc-h2">Overview</h2>
+      <div class="fc-artist-mgmt__chips">
+        <v-chip
+          size="small"
+          :variant="overview.is_subscription ? 'flat' : 'outlined'"
+          :color="overview.is_subscription ? 'accent' : undefined"
+          prepend-icon="mdi-rss"
+        >{{ overview.is_subscription ? 'Subscription' : 'One-off' }}</v-chip>
+        <v-chip
+          size="small" variant="outlined" prepend-icon="mdi-link-variant"
+          :to="`/subscriptions?artist_id=${overview.id}`"
+        >{{ overview.sources.length }} subscription{{ overview.sources.length === 1 ? '' : 's' }}</v-chip>
+      </div>
+    </section>
+
+    <section v-if="overview.cooccurring_tags.length" class="fc-artist-mgmt__sec">
+      <h2 class="fc-h2">Frequent tags</h2>
+      <div class="fc-artist-mgmt__tags">
+        <v-chip
+          v-for="t in overview.cooccurring_tags" :key="t.id"
+          size="small" @click="openTag(t.id)"
+        >{{ t.name }} <span class="fc-artist-mgmt__tagc">{{ t.count }}</span></v-chip>
+      </div>
+    </section>
+
+    <section v-if="overview.activity.length" class="fc-artist-mgmt__sec">
+      <h2 class="fc-h2">Activity</h2>
+      <svg
+        class="fc-artist-mgmt__spark" :viewBox="`0 0 ${sparkW} ${sparkH}`"
+        preserveAspectRatio="none" role="img" aria-label="posts over time"
+      >
+        <polyline :points="sparkPoints" fill="none"
+                  stroke="rgb(var(--v-theme-accent))" stroke-width="2" />
+      </svg>
+    </section>
+
+    <section v-if="overview.sources.length" class="fc-artist-mgmt__sec">
+      <div class="fc-artist-mgmt__sec-head">
+        <h2 class="fc-h2">Subscriptions</h2>
+        <RouterLink
+          :to="`/subscriptions?artist_id=${overview.id}`"
+          class="fc-artist-mgmt__manage"
+        >Manage subscriptions →</RouterLink>
+      </div>
+      <v-table density="compact">
+        <thead>
+          <tr><th>Platform</th><th>URL</th><th class="text-right">Images</th></tr>
+        </thead>
+        <tbody>
+          <tr v-for="s in overview.sources" :key="s.id">
+            <td>{{ s.platform }}</td>
+            <td class="fc-artist-mgmt__url">{{ s.url }}</td>
+            <td class="text-right">{{ s.image_count }}</td>
+          </tr>
+        </tbody>
+      </v-table>
+    </section>
+
+    <section class="fc-artist-mgmt__sec">
+      <h2 class="fc-h2">Danger zone</h2>
+      <ArtistDangerZone
+        :slug="overview.slug"
+        :artist-id="overview.id"
+        :artist-name="overview.name"
+      />
+    </section>
+  </div>
+</template>
+
+<script setup>
+import { computed } from 'vue'
+import { useRouter, RouterLink } from 'vue-router'
+
+import ArtistDangerZone from './ArtistDangerZone.vue'
+
+const props = defineProps({
+  overview: { type: Object, required: true },
+})
+
+const router = useRouter()
+
+const sparkW = 600
+const sparkH = 80
+const sparkPoints = computed(() => {
+  const a = props.overview.activity ?? []
+  if (a.length === 0) return ''
+  const max = Math.max(...a.map(p => p.count), 1)
+  const stepX = a.length > 1 ? sparkW / (a.length - 1) : 0
+  return a.map((p, i) => {
+    const x = i * stepX
+    const y = sparkH - (p.count / max) * (sparkH - 4) - 2
+    return `${x.toFixed(1)},${y.toFixed(1)}`
+  }).join(' ')
+})
+
+function openTag (tagId) {
+  router.push({ name: 'gallery', query: { tag_id: tagId } })
+}
+</script>
+
+<style scoped>
+.fc-artist-mgmt { padding-top: 1rem; }
+.fc-h2 {
+  font-family: 'Fraunces', Georgia, serif;
+  font-size: 20px; font-weight: 500; margin-bottom: 8px;
+}
+.fc-artist-mgmt__sec { margin-bottom: 28px; }
+.fc-artist-mgmt__chips { display: flex; gap: 8px; flex-wrap: wrap; }
+.fc-artist-mgmt__tags { display: flex; flex-wrap: wrap; gap: 6px; }
+.fc-artist-mgmt__tagc {
+  color: rgb(var(--v-theme-on-surface-variant));
+  margin-left: 4px;
+  font-variant-numeric: tabular-nums;
+}
+.fc-artist-mgmt__spark { width: 100%; height: 80px; }
+.fc-artist-mgmt__url {
+  max-width: 380px; overflow: hidden; text-overflow: ellipsis;
+  white-space: nowrap;
+}
+.fc-artist-mgmt__sec-head {
+  display: flex; align-items: baseline; justify-content: space-between;
+  margin-bottom: 0.25rem;
+}
+.fc-artist-mgmt__manage {
+  font-size: 0.85rem;
+  color: rgb(var(--v-theme-accent));
+  text-decoration: none;
+}
+.fc-artist-mgmt__manage:hover { text-decoration: underline; }
+</style>

From 7d84990f6dcb5b8556e050d7b20449d8578f2857 Mon Sep 17 00:00:00 2001
From: Bryan Van Deusen <bryan.vandeusen@fabledsword.com>
Date: Tue, 26 May 2026 16:01:14 -0400
Subject: [PATCH 19/20] =?UTF-8?q?feat(artist-view):=20ArtistView=20rewrite?=
 =?UTF-8?q?=20=E2=80=94=20sticky=20frosted=20ArtistHeader=20(name=20+=20st?=
 =?UTF-8?q?ats=20+=20tabs)=20replaces=20the=20in-body=20h1;=20three=20lazy?=
 =?UTF-8?q?=20tabs=20(Posts=20default,=20Gallery=20fallback,=20Management)?=
 =?UTF-8?q?;=20=3Ftab=3D=20URL=20state;=20cross-artist=20store=20reset;=20?=
 =?UTF-8?q?document.title=20set=20on=20slug=20change?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---
 frontend/src/stores/artist.js     |  16 +-
 frontend/src/views/ArtistView.vue | 254 ++++++++----------------------
 2 files changed, 78 insertions(+), 192 deletions(-)

diff --git a/frontend/src/stores/artist.js b/frontend/src/stores/artist.js
index a8e250e..18dee7e 100644
--- a/frontend/src/stores/artist.js
+++ b/frontend/src/stores/artist.js
@@ -1,6 +1,7 @@
 import { defineStore } from 'pinia'
 import { ref, computed } from 'vue'
 import { useApi } from '../composables/useApi.js'
+import { usePostsStore } from './posts.js'
 
 const PAGE = 60
 
@@ -15,7 +16,11 @@ export const useArtistStore = defineStore('artist', () => {
   const notFound = ref(false)
   let started = false
 
-  async function load(slug) {
+  async function load (slug) {
+    // Cross-artist reset: clear this store AND the posts store so the new
+    // artist doesn't briefly render with the previous artist's content
+    // when the user is on the Posts tab. (Gallery tab uses this artist
+    // store's own images list — cleared above.)
     overview.value = null
     images.value = []
     nextCursor.value = null
@@ -23,6 +28,7 @@ export const useArtistStore = defineStore('artist', () => {
     started = false
     error.value = null
     loading.value = true
+    usePostsStore().$reset?.()
     try {
       overview.value = await api.get(`/api/artist/${encodeURIComponent(slug)}`)
       await loadMoreImages(slug)
@@ -34,7 +40,7 @@ export const useArtistStore = defineStore('artist', () => {
     }
   }
 
-  async function loadMoreImages(slug) {
+  async function loadMoreImages (slug) {
     if (imagesLoading.value) return
     if (started && nextCursor.value === null) return
     imagesLoading.value = true
@@ -55,9 +61,13 @@ export const useArtistStore = defineStore('artist', () => {
   }
 
   const hasMoreImages = computed(() => !started || nextCursor.value !== null)
+  const postCount = computed(() => overview.value?.post_count ?? null)
+  const imageCount = computed(() => overview.value?.image_count ?? null)
+  const lastAdded = computed(() => overview.value?.date_range?.max ?? null)
 
   return {
     overview, images, loading, imagesLoading, error, notFound,
-    hasMoreImages, load, loadMoreImages
+    hasMoreImages, postCount, imageCount, lastAdded,
+    load, loadMoreImages,
   }
 })
diff --git a/frontend/src/views/ArtistView.vue b/frontend/src/views/ArtistView.vue
index 8750e70..76eca6b 100644
--- a/frontend/src/views/ArtistView.vue
+++ b/frontend/src/views/ArtistView.vue
@@ -1,220 +1,96 @@
 <template>
-  <v-container fluid class="py-6">
-    <div v-if="store.loading && !store.overview" class="fc-artist__loading">
-      <v-progress-circular indeterminate color="accent" size="36" />
-    </div>
+  <div v-if="store.loading && !store.overview" class="fc-artist__loading">
+    <v-progress-circular indeterminate color="accent" size="36" />
+  </div>
 
-    <v-alert v-else-if="store.notFound" type="warning" variant="tonal">
-      Artist not found.
-    </v-alert>
+  <v-container v-else-if="store.notFound" class="py-6">
+    <v-alert type="warning" variant="tonal">Artist not found.</v-alert>
+  </v-container>
 
-    <v-alert v-else-if="store.error" type="error" variant="tonal" closable>
-      {{ store.error }}
-    </v-alert>
-
-    <template v-else-if="store.overview">
-      <header class="fc-artist__head">
-        <h1 class="fc-h1">{{ store.overview.name }}</h1>
-        <div class="fc-artist__stats">
-          <span>{{ store.overview.image_count }} images</span>
-          <span v-if="dateRange">· {{ dateRange }}</span>
-        </div>
-      </header>
-
-      <div class="fc-artist__fc4">
-        <v-chip
-          size="small"
-          :variant="store.overview.is_subscription ? 'flat' : 'outlined'"
-          :color="store.overview.is_subscription ? 'accent' : undefined"
-          prepend-icon="mdi-rss"
-        >{{ store.overview.is_subscription ? 'Subscription' : 'One-off' }}</v-chip>
-
-        <v-chip
-          size="small" variant="outlined" prepend-icon="mdi-link-variant"
-          :to="`/subscriptions?artist_id=${store.overview.id}`"
-        >{{ store.overview.sources.length }} source{{ store.overview.sources.length === 1 ? '' : 's' }}</v-chip>
-
-        <v-chip
-          size="small" variant="outlined" prepend-icon="mdi-rss"
-          :to="`/posts?artist_id=${store.overview.id}`"
-        >View posts</v-chip>
-
-        <v-chip
-          size="small" variant="outlined" disabled
-          prepend-icon="mdi-clock-outline"
-        >Credential health · FC-3b</v-chip>
-      </div>
-
-      <!-- Tabs split (2026-05-25): Settings was previously slotted at the
-           bottom of the page after the infinite-scroll image grid, which
-           made it effectively unreachable for any artist with more than
-           a couple of pages of content. The Settings tab now hosts
-           destructive admin actions (artist+content cascade-delete) and
-           any future per-artist management UI. v-tabs is `position:
-           sticky; top: 64px` (under the 64px AppShell TopNav) so it
-           stays parked while the gallery scrolls. -->
-      <v-tabs
-        v-model="tab" color="accent" class="mb-4"
-        style="position: sticky; top: 64px; z-index: 4;
-               background: rgb(var(--v-theme-surface));"
-      >
-        <v-tab value="overview">Overview</v-tab>
-        <v-tab value="settings">Settings</v-tab>
-      </v-tabs>
+  <v-container v-else-if="store.error && !store.overview" class="py-6">
+    <v-alert type="error" variant="tonal" closable>{{ store.error }}</v-alert>
+  </v-container>
 
+  <template v-else-if="store.overview">
+    <ArtistHeader
+      v-model="tab"
+      :name="store.overview.name"
+      :image-count="store.imageCount"
+      :post-count="store.postCount"
+      :last-added="store.lastAdded"
+    />
+    <v-container fluid class="py-4">
       <v-window v-model="tab">
-        <v-window-item value="overview">
-          <section v-if="store.overview.cooccurring_tags.length" class="fc-artist__sec">
-            <h2 class="fc-h2">Frequent tags</h2>
-            <div class="fc-artist__tags">
-              <v-chip
-                v-for="t in store.overview.cooccurring_tags" :key="t.id"
-                size="small" @click="openTag(t.id)"
-              >{{ t.name }} <span class="fc-artist__tagc">{{ t.count }}</span></v-chip>
-            </div>
-          </section>
-
-          <section v-if="store.overview.activity.length" class="fc-artist__sec">
-            <h2 class="fc-h2">Activity</h2>
-            <svg class="fc-artist__spark" :viewBox="`0 0 ${sparkW} ${sparkH}`"
-                 preserveAspectRatio="none" role="img" aria-label="posts over time">
-              <polyline :points="sparkPoints" fill="none"
-                        stroke="rgb(var(--v-theme-accent))" stroke-width="2" />
-            </svg>
-          </section>
-
-          <section v-if="store.overview.sources.length" class="fc-artist__sec">
-            <div class="fc-artist__sec-head">
-              <h2 class="fc-h2">Sources</h2>
-              <RouterLink
-                :to="`/subscriptions?artist_id=${store.overview.id}`"
-                class="fc-artist__manage"
-              >Manage subscriptions →</RouterLink>
-            </div>
-            <v-table density="compact">
-              <thead>
-                <tr><th>Platform</th><th>URL</th><th class="text-right">Images</th></tr>
-              </thead>
-              <tbody>
-                <tr v-for="s in store.overview.sources" :key="s.id">
-                  <td>{{ s.platform }}</td>
-                  <td class="fc-artist__url">{{ s.url }}</td>
-                  <td class="text-right">{{ s.image_count }}</td>
-                </tr>
-              </tbody>
-            </v-table>
-          </section>
-
-          <section class="fc-artist__sec">
-            <h2 class="fc-h2">Images</h2>
-            <MasonryGrid
-              :items="store.images"
-              :loading="store.imagesLoading"
-              :has-more="store.hasMoreImages"
-              @load-more="store.loadMoreImages(slug)"
-              @open="openImage"
-            />
-          </section>
-        </v-window-item>
-
-        <v-window-item value="settings">
-          <ArtistDangerZone
-            :slug="slug"
+        <v-window-item value="posts">
+          <ArtistPostsTab
             :artist-id="store.overview.id"
-            :artist-name="store.overview.name"
+            @switch-tab="(t) => tab = t"
           />
         </v-window-item>
+        <v-window-item value="gallery">
+          <ArtistGalleryTab :slug="slug" />
+        </v-window-item>
+        <v-window-item value="management">
+          <ArtistManagementTab :overview="store.overview" />
+        </v-window-item>
       </v-window>
-    </template>
-  </v-container>
+    </v-container>
+  </template>
 </template>
 
 <script setup>
 import { computed, ref, watch } from 'vue'
-import { useRoute, useRouter, RouterLink } from 'vue-router'
+import { useRoute, useRouter } from 'vue-router'
+
 import { useArtistStore } from '../stores/artist.js'
-import { useModalStore } from '../stores/modal.js'
-import MasonryGrid from '../components/discovery/MasonryGrid.vue'
-import ArtistDangerZone from '../components/artist/ArtistDangerZone.vue'
+import ArtistHeader from '../components/artist/ArtistHeader.vue'
+import ArtistPostsTab from '../components/artist/ArtistPostsTab.vue'
+import ArtistGalleryTab from '../components/artist/ArtistGalleryTab.vue'
+import ArtistManagementTab from '../components/artist/ArtistManagementTab.vue'
+
+const VALID_TABS = ['posts', 'gallery', 'management']
 
 const route = useRoute()
 const router = useRouter()
 const store = useArtistStore()
-const modal = useModalStore()
 
 const slug = computed(() => route.params.slug)
-// Per-artist tab — defaults to Overview. Settings tab hosts destructive
-// admin actions (DangerZone). Switching artists resets to Overview so the
-// destructive surface isn't re-shown by accident when navigating between
-// artists.
-const tab = ref('overview')
+const tab = ref('posts')
 
-watch(slug, (s) => {
-  if (s) {
-    store.load(s)
-    tab.value = 'overview'
-  }
+function resolveDefaultTab () {
+  const fromUrl = route.query.tab
+  if (VALID_TABS.includes(fromUrl)) return fromUrl
+  if ((store.postCount ?? 0) > 0) return 'posts'
+  return 'gallery'
+}
+
+watch(slug, async (s) => {
+  if (!s) return
+  await store.load(s)
+  document.title = store.overview
+    ? `${store.overview.name} — FabledCurator`
+    : 'FabledCurator'
+  tab.value = resolveDefaultTab()
 }, { immediate: true })
 
-const dateRange = computed(() => {
-  const r = store.overview?.date_range
-  if (!r || !r.min) return null
-  const fmt = (iso) => iso.slice(0, 10)
-  return r.min === r.max ? fmt(r.min) : `${fmt(r.min)} → ${fmt(r.max)}`
+// Reflect tab changes back into the URL so refresh/back/forward work.
+watch(tab, (newTab) => {
+  if (route.query.tab === newTab) return
+  router.replace({
+    query: { ...route.query, tab: newTab },
+  })
 })
 
-const sparkW = 600
-const sparkH = 80
-const sparkPoints = computed(() => {
-  const a = store.overview?.activity ?? []
-  if (a.length === 0) return ''
-  const max = Math.max(...a.map(p => p.count), 1)
-  const stepX = a.length > 1 ? sparkW / (a.length - 1) : 0
-  return a.map((p, i) => {
-    const x = i * stepX
-    const y = sparkH - (p.count / max) * (sparkH - 4) - 2
-    return `${x.toFixed(1)},${y.toFixed(1)}`
-  }).join(' ')
+// React to URL-tab changes (e.g., back/forward).
+watch(() => route.query.tab, (q) => {
+  if (q && VALID_TABS.includes(q) && tab.value !== q) {
+    tab.value = q
+  }
 })
-
-function openImage(id) {
-  modal.open(id)
-}
-function openTag(tagId) {
-  router.push({ name: 'gallery', query: { tag_id: tagId } })
-}
 </script>
 
 <style scoped>
-.fc-h1 {
-  font-family: 'Fraunces', Georgia, serif;
-  font-size: 32px; font-weight: 500;
-  color: rgb(var(--v-theme-on-surface));
+.fc-artist__loading {
+  display: flex; justify-content: center; padding: 64px 0;
 }
-.fc-h2 {
-  font-family: 'Fraunces', Georgia, serif;
-  font-size: 20px; font-weight: 500; margin-bottom: 8px;
-}
-.fc-artist__loading { display: flex; justify-content: center; padding: 64px 0; }
-.fc-artist__head { margin-bottom: 12px; }
-.fc-artist__stats { opacity: 0.75; margin-top: 4px; }
-.fc-artist__fc4 { display: flex; gap: 8px; flex-wrap: wrap; margin-bottom: 24px; }
-.fc-artist__sec { margin-bottom: 28px; }
-.fc-artist__tags { display: flex; flex-wrap: wrap; gap: 6px; }
-.fc-artist__tagc { opacity: 0.6; margin-left: 4px; font-variant-numeric: tabular-nums; }
-.fc-artist__spark { width: 100%; height: 80px; }
-.fc-artist__url {
-  max-width: 380px; overflow: hidden; text-overflow: ellipsis;
-  white-space: nowrap;
-}
-.fc-artist__sec-head {
-  display: flex; align-items: baseline; justify-content: space-between;
-  margin-bottom: 0.25rem;
-}
-.fc-artist__manage {
-  font-size: 0.85rem;
-  color: rgb(var(--v-theme-accent));
-  text-decoration: none;
-}
-.fc-artist__manage:hover { text-decoration: underline; }
 </style>

From fb41b901109a58307c4f3cc2d554e78e948194e5 Mon Sep 17 00:00:00 2001
From: Bryan Van Deusen <bryan.vandeusen@fabledsword.com>
Date: Tue, 26 May 2026 16:27:51 -0400
Subject: [PATCH 20/20] =?UTF-8?q?fix(extension):=20=5Ffind=5For=5Fcreate?=
 =?UTF-8?q?=5Fartist=20+=20=5Ffind=5For=5Fcreate=5Fsource=20race-safe=20vi?=
 =?UTF-8?q?a=20savepoint=20+=20IntegrityError=20recovery=20=E2=80=94=20sam?=
 =?UTF-8?q?e=20pattern=20as=20importer's=20helpers.=20Two=20concurrent=20q?=
 =?UTF-8?q?uick-add-source=20calls=20on=20the=20same=20artist/url=20would?=
 =?UTF-8?q?=20have=20500'd=20on=20uq=5Fartist=5Fslug=20/=20uq=5Fsource=5Fa?=
 =?UTF-8?q?rtist=5Fplatform=5Furl;=20now=20the=20second=20one=20rolls=20th?=
 =?UTF-8?q?e=20savepoint=20back=20and=20returns=20the=20existing=20row.?=
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
---
 backend/app/services/extension_service.py | 51 +++++++++++++++++++----
 1 file changed, 44 insertions(+), 7 deletions(-)

diff --git a/backend/app/services/extension_service.py b/backend/app/services/extension_service.py
index ff88c9c..ce34ed9 100644
--- a/backend/app/services/extension_service.py
+++ b/backend/app/services/extension_service.py
@@ -11,6 +11,7 @@ from __future__ import annotations
 import re
 
 from sqlalchemy import select
+from sqlalchemy.exc import IntegrityError
 from sqlalchemy.ext.asyncio import AsyncSession
 
 from ..models import Artist, Source
@@ -97,20 +98,40 @@ class ExtensionService:
         raise UnknownPlatformError(f"no platform pattern matched {url!r}")
 
     async def _find_or_create_artist(self, raw_name: str) -> tuple[Artist, bool]:
+        """Race-safe find-or-create on Artist by slug. Mirrors the
+        savepoint + IntegrityError recovery pattern used in
+        Importer._find_or_create_source/post (see
+        reference_scalar_one_or_none_duplicates memory). Without this,
+        two concurrent quick-add-source calls hitting the same artist
+        would both miss the existence check and the second INSERT would
+        500 against uq_artist_slug.
+        """
         slug = slugify(raw_name)
         existing = (await self.session.execute(
             select(Artist).where(Artist.slug == slug)
         )).scalar_one_or_none()
         if existing is not None:
             return existing, False
-        artist = Artist(name=raw_name, slug=slug, is_subscription=True)
-        self.session.add(artist)
-        await self.session.flush()
-        return artist, True
+        sp = await self.session.begin_nested()
+        try:
+            artist = Artist(name=raw_name, slug=slug, is_subscription=True)
+            self.session.add(artist)
+            await self.session.flush()
+            await sp.commit()
+            return artist, True
+        except IntegrityError:
+            await sp.rollback()
+            recovered = (await self.session.execute(
+                select(Artist).where(Artist.slug == slug)
+            )).scalar_one()
+            return recovered, False
 
     async def _find_or_create_source(
         self, *, artist_id: int, platform: str, url: str,
     ) -> tuple[Source, bool]:
+        """Race-safe — same pattern as _find_or_create_artist above. The
+        uq_source_artist_platform_url constraint catches the duplicate
+        insert; we roll the savepoint back and re-select."""
         existing = (await self.session.execute(
             select(Source).where(
                 Source.artist_id == artist_id,
@@ -120,8 +141,24 @@ class ExtensionService:
         )).scalar_one_or_none()
         if existing is not None:
             return existing, False
-        src = Source(artist_id=artist_id, platform=platform, url=url, enabled=True)
-        self.session.add(src)
-        await self.session.flush()
+        sp = await self.session.begin_nested()
+        try:
+            src = Source(
+                artist_id=artist_id, platform=platform,
+                url=url, enabled=True,
+            )
+            self.session.add(src)
+            await self.session.flush()
+            await sp.commit()
+        except IntegrityError:
+            await sp.rollback()
+            recovered = (await self.session.execute(
+                select(Source).where(
+                    Source.artist_id == artist_id,
+                    Source.platform == platform,
+                    Source.url == url,
+                )
+            )).scalar_one()
+            return recovered, False
         await self.session.commit()
         return src, True