Standardize tag ordering to alphabetical across all endpoints

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
This commit is contained in:
Luke Mino-Altherr 2026-03-12 15:36:10 -07:00
parent b3e04d88bb
commit b5dc70bede
2 changed files with 6 additions and 6 deletions

View File

@ -319,7 +319,7 @@ def list_references_page(
select(AssetReferenceTag.asset_reference_id, Tag.name) select(AssetReferenceTag.asset_reference_id, Tag.name)
.join(Tag, Tag.name == AssetReferenceTag.tag_name) .join(Tag, Tag.name == AssetReferenceTag.tag_name)
.where(AssetReferenceTag.asset_reference_id.in_(id_list)) .where(AssetReferenceTag.asset_reference_id.in_(id_list))
.order_by(AssetReferenceTag.added_at) .order_by(AssetReferenceTag.tag_name.asc())
) )
for ref_id, tag_name in rows.all(): for ref_id, tag_name in rows.all():
tag_map[ref_id].append(tag_name) tag_map[ref_id].append(tag_name)

View File

@ -75,9 +75,9 @@ def get_reference_tags(session: Session, reference_id: str) -> list[str]:
tag_name tag_name
for (tag_name,) in ( for (tag_name,) in (
session.execute( session.execute(
select(AssetReferenceTag.tag_name).where( select(AssetReferenceTag.tag_name)
AssetReferenceTag.asset_reference_id == reference_id .where(AssetReferenceTag.asset_reference_id == reference_id)
) .order_by(AssetReferenceTag.tag_name.asc())
) )
).all() ).all()
] ]
@ -120,7 +120,7 @@ def set_reference_tags(
) )
session.flush() session.flush()
return SetTagsResult(added=to_add, removed=to_remove, total=desired) return SetTagsResult(added=sorted(to_add), removed=sorted(to_remove), total=sorted(desired))
def add_tags_to_reference( def add_tags_to_reference(
@ -364,7 +364,7 @@ def list_tag_counts_for_filtered_assets(
) )
.where(AssetReferenceTag.asset_reference_id.in_(select(ref_sq.c.id))) .where(AssetReferenceTag.asset_reference_id.in_(select(ref_sq.c.id)))
.group_by(AssetReferenceTag.tag_name) .group_by(AssetReferenceTag.tag_name)
.order_by(func.count(AssetReferenceTag.asset_reference_id).desc()) .order_by(func.count(AssetReferenceTag.asset_reference_id).desc(), AssetReferenceTag.tag_name.asc())
.limit(limit) .limit(limit)
) )