From e566be6a4bba397ae6345a38c0d480992b4085af Mon Sep 17 00:00:00 2001 From: Thor Whalen <1906276+thorwhalen@users.noreply.github.com> Date: Wed, 14 Oct 2026 12:43:02 +0000 Subject: [PATCH] artifact_store: refresh stale RowKey comment in from_sql (#64, #65) Co-Authored-By: Claude Sonnet 5.5 --- lacing/artifact_store.py | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/lacing/artifact_store.py b/lacing/artifact_store.py index 049929d8..e3dbd853 100644 --- a/lacing/artifact_store.py +++ b/lacing/artifact_store.py @@ -739,8 +739,10 @@ def content_hash_of(record: BaseModel) -> str | None: ) def _key_of_id(row_or_key): - # ``wrap_kvs`` hands this the key dict on writes and the ORM row on - # iteration (sqldol's ``__iter__`` yields rows, not keys). + # ``wrap_kvs`` hands this a key dict on writes and a ``RowKey`` on + # iteration (sqldol's ``__iter__`` yields ``RowKey``s, which are + # dicts that also read as attributes). The attribute branch is only + # for older sqldol releases that iterated ORM rows. if isinstance(row_or_key, dict): return row_or_key["artifact_id"] return row_or_key.artifact_id