From 102234e1320f37b001752024050e07579c9baa3f Mon Sep 17 00:00:00 2001 From: kert Date: Thu, 24 Sep 2026 19:54:40 -0400 Subject: [PATCH] fix(llm): theme stamping issues its batch UPDATE through Connection.execute with a params list (Connection has no executemany) --- src/llm/themes.py | 2 +- tests/llm/test_themes.py | 5 +---- 2 files changed, 2 insertions(+), 5 deletions(-) diff --git a/src/llm/themes.py b/src/llm/themes.py index 28946fd..881e466 100644 --- a/src/llm/themes.py +++ b/src/llm/themes.py @@ -140,7 +140,7 @@ def stamp( patches.append({"id": cid, "patch": json.dumps(patch)}) if patches: with engine.begin() as conn: - conn.executemany(text(_UPDATE), patches) + conn.execute(text(_UPDATE), patches) # a list of params = executemany stats["stamped"] += len(patches) log.info("themes: scanned=%d stamped=%d", stats["scanned"], stats["stamped"]) return stats diff --git a/tests/llm/test_themes.py b/tests/llm/test_themes.py index bce602d..2503a75 100644 --- a/tests/llm/test_themes.py +++ b/tests/llm/test_themes.py @@ -34,13 +34,10 @@ def _engine(pages): r = MagicMock() r.fetchall.return_value = selects.pop(0) return r + updates.extend(params) # an UPDATE with a list of params (executemany) return MagicMock() - def executemany(sql, params): - updates.extend(params) - conn.execute.side_effect = execute - conn.executemany.side_effect = executemany return engine, updates