Making scope filter by product id

app/crud/augmented_part.py

@@ -1,4 +1,4 @@
-from sqlalchemy import distinct, select
+from sqlalchemy import distinct, or_, select
 from sqlalchemy.ext.asyncio import AsyncSession
 
 from app.models.associations import augmented_part_imported_parts
@@ -89,10 +89,19 @@ async def list_filtered(
     direction: str = "asc",
     skip: int = 0,
     limit: int = 100,
+    match_sku: bool = False,
 ) -> list[AugmentedPart]:
     q = select(AugmentedPart)
     if description.strip():
-        q = q.where(AugmentedPart.description.ilike(f"%{description.strip()}%"))
+        term = f"%{description.strip()}%"
+        if match_sku:
+            q = q.where(
+                or_(
+                    AugmentedPart.description.ilike(term), AugmentedPart.sku.ilike(term)
+                )
+            )
+        else:
+            q = q.where(AugmentedPart.description.ilike(term))
     if colour_id is not None:
         q = q.where(AugmentedPart.colour_id == colour_id)
     if manufacturer_id is not None:

app/services/scope_guide.py

@@ -33,8 +33,9 @@ async def build(
     """Return the context for ``scopes.html``.
 
     The scope rows are narrowed by the scope-attribute filters (``description``
-    matches the scope's description; the ``*_id`` filters match exactly).
+    matches either the scope's description or its SKU/product ID; the ``*_id``
-    ``fits_part_type_id`` keeps only scopes that have at least one (non-``none``)
+    filters match exactly). ``fits_part_type_id`` keeps only scopes that have
+    at least one (non-``none``)
     fit for that part type. Callers that paginate should slice ``rows``
     themselves — this always returns the full filtered set, since
     ``fits_part_type_id`` is a post-filter that can't be pushed into the DB query.
@@ -71,6 +72,7 @@ async def build(
         sub_part_type_id=sub_part_type_id,
         part_type_id=scope_type.id,
         limit=1000,
+        match_sku=True,
     )
     scope_ids = {s.id for s in scopes}
     if not scope_ids:

app/templates/scopes.html

@@ -15,7 +15,7 @@
           hx-target="#scopes-table-card"
           hx-trigger="input delay:300ms, change"
           hx-swap="outerHTML">
-      <input type="text" name="description" placeholder="Search scopes…"
+      <input type="text" name="description" placeholder="Search by description or product ID…"
              value="{{ filters.description or '' }}">
       <select name="manufacturer_id">
         <option value="">All manufacturers</option>

tests/test_scope_guide.py

@@ -147,6 +147,26 @@ async def test_build_filters_by_description(db_sessionmaker):
     assert _descriptions(guide) == ["Alpha 3-18x50"]
 
 
+@pytest.mark.asyncio
+async def test_build_filters_by_sku(db_sessionmaker):
+    async with db_sessionmaker() as db:
+        ids = await _seed_guide_data(db)
+        ids["s1"].sku = "ARM-12345"
+        await db.commit()
+        guide = await scope_guide.build(db, description="ARM-12345")
+    assert _descriptions(guide) == ["Alpha 3-18x50"]
+
+
+@pytest.mark.asyncio
+async def test_build_filters_by_partial_sku(db_sessionmaker):
+    async with db_sessionmaker() as db:
+        ids = await _seed_guide_data(db)
+        ids["s1"].sku = "ARM-12345"
+        await db.commit()
+        guide = await scope_guide.build(db, description="12345")
+    assert _descriptions(guide) == ["Alpha 3-18x50"]
+
+
 @pytest.mark.asyncio
 async def test_build_filters_by_manufacturer(db_sessionmaker):
     async with db_sessionmaker() as db: