Fixing serialization for manufacturers in schema

.claude/settings.local.json

@@ -37,7 +37,9 @@
       "Bash(sqlite3 app.db \"SELECT COUNT\\(*\\) FROM augmented_parts WHERE scope_model_id=\\(SELECT id FROM scope_models WHERE name='Viper'\\);\")",
       "Bash(sqlite3 app.db \"PRAGMA wal_checkpoint\\(FULL\\);\")",
       "Bash(curl -s -o /dev/null -w \"%{http_code}\\\\n\" http://127.0.0.1:8000/login)",
-      "Bash(curl -s -o /dev/null -w \"%{http_code}\\\\n\" http://127.0.0.1:8000/admin/imported-parts)"
+      "Bash(curl -s -o /dev/null -w \"%{http_code}\\\\n\" http://127.0.0.1:8000/admin/imported-parts)",
+      "Bash(rg -n \"hx-delete|router.delete\" app/web/admin.py app/templates/admin/*.html)",
+      "Bash(rg -n \"is_superuser\" tests/conftest.py tests/*.py)"
     ]
   }
 }

.env.example

@@ -25,6 +25,9 @@ AUTO_CREATE_TABLES=true
 COOKIE_NAME=access_token
 # Set true when serving over HTTPS.
 COOKIE_SECURE=false
+# How long a browser login lasts before being bounced back to /login.
+# Independent of ACCESS_TOKEN_EXPIRE_MINUTES, which governs JSON API tokens.
+SESSION_TIMEOUT_HOURS=24
 
 # --- LDAP (optional; direct-bind) ---
 # When enabled, the login page and /api/auth/ldap-login accept a directory

Makefile

@@ -1,4 +1,4 @@
-VERSION := $(shell git describe --tags --always)
+VERSION := $(shell git tag | tail -1)
 IMAGE_NAME := fitguide
 
 sync:
@@ -8,7 +8,7 @@ seed:
 	uv run alembic upgrade head
 	uv run python -m scripts.seed
 
-dev:
+dev: migrate
 	uv run uvicorn app.main:app --reload
 	open http://localhost:8000
 

README.md

@@ -149,8 +149,9 @@ Every setting is read from an environment variable first, then `.env`
 |----------|---------|-------|
 | `SECRET_KEY` | dev placeholder | **set a strong value in production** |
 | `DATABASE_URL` | `sqlite+aiosqlite:///./app.db` | use `postgresql+asyncpg://...` for Postgres |
-| `ACCESS_TOKEN_EXPIRE_MINUTES` | `15` | |
+| `ACCESS_TOKEN_EXPIRE_MINUTES` | `15` | JSON API bearer token lifetime |
 | `REFRESH_TOKEN_EXPIRE_DAYS` | `7` | |
+| `SESSION_TIMEOUT_HOURS` | `24` | browser login (HTMX UI cookie) lifetime |
 | `AUTO_CREATE_TABLES` | `true` | create tables on startup (dev); use Alembic in prod |
 | `COOKIE_SECURE` | `false` | set `true` behind HTTPS |
 | `FIRST_SUPERUSER_EMAIL` / `_PASSWORD` | admin@example.com / changethis | used by seed |

alembic/versions/04425ce3d4d5_add_jobs_table.py

@@ -5,16 +5,17 @@ Revises: a3f7c19b2d40
 Create Date: 2026-08-05 15:21:09.527442
 """
 
-from typing import Sequence, Union
+from collections.abc import Sequence
+
+import sqlalchemy as sa
 
 from alembic import op
-import sqlalchemy as sa
 
 # revision identifiers, used by Alembic.
 revision: str = "04425ce3d4d5"
-down_revision: Union[str, None] = "a3f7c19b2d40"
+down_revision: str | None = "a3f7c19b2d40"
-branch_labels: Union[str, Sequence[str], None] = None
+branch_labels: str | Sequence[str] | None = None
-depends_on: Union[str, Sequence[str], None] = None
+depends_on: str | Sequence[str] | None = None
 
 
 def upgrade() -> None:

alembic/versions/0c3d1a5cff1f_add_part_colours.py

@@ -5,25 +5,28 @@ Revises: 5b6a7817b408
 Create Date: 2026-07-02 12:22:24.106795
 """
 
-from typing import Sequence, Union
+from collections.abc import Sequence
+
+import sqlalchemy as sa
 
 from alembic import op
-import sqlalchemy as sa
 
 # revision identifiers, used by Alembic.
 revision: str = "0c3d1a5cff1f"
-down_revision: Union[str, None] = "5b6a7817b408"
+down_revision: str | None = "5b6a7817b408"
-branch_labels: Union[str, Sequence[str], None] = None
+branch_labels: str | Sequence[str] | None = None
-depends_on: Union[str, Sequence[str], None] = None
+depends_on: str | Sequence[str] | None = None
 
 
 def upgrade() -> None:
-    # ### commands auto generated by Alembic - please adjust! ###
+    op.create_table(
-    pass
+        "part_colours",
-    # ### end Alembic commands ###
+        sa.Column("id", sa.Integer(), nullable=False),
+        sa.Column("name", sa.String(), nullable=False),
+        sa.PrimaryKeyConstraint("id"),
+        sa.UniqueConstraint("name"),
+    )
 
 
 def downgrade() -> None:
-    # ### commands auto generated by Alembic - please adjust! ###
+    op.drop_table("part_colours")
-    pass
-    # ### end Alembic commands ###

alembic/versions/1b99be8a9b1e_add_fits_tables.py

@@ -5,16 +5,17 @@ Revises: adf4a5b9493e
 Create Date: 2026-07-14 15:00:00.000000
 """
 
-from typing import Sequence, Union
+from collections.abc import Sequence
+
+import sqlalchemy as sa
 
 from alembic import op
-import sqlalchemy as sa
 
 # revision identifiers, used by Alembic.
 revision: str = "1b99be8a9b1e"
-down_revision: Union[str, None] = "adf4a5b9493e"
+down_revision: str | None = "adf4a5b9493e"
-branch_labels: Union[str, Sequence[str], None] = None
+branch_labels: str | Sequence[str] | None = None
-depends_on: Union[str, Sequence[str], None] = None
+depends_on: str | Sequence[str] | None = None
 
 
 def upgrade() -> None:

alembic/versions/235d823a56ee_drop_objective_thread_pitch_from_scope_.py

@@ -0,0 +1,30 @@
+"""drop_objective_thread_pitch_from_scope_infos
+
+Revision ID: 235d823a56ee
+Revises: 04425ce3d4d5
+Create Date: 2026-08-17 15:18:26.763831
+"""
+
+from collections.abc import Sequence
+
+import sqlalchemy as sa
+
+from alembic import op
+
+# revision identifiers, used by Alembic.
+revision: str = "235d823a56ee"
+down_revision: str | None = "04425ce3d4d5"
+branch_labels: str | Sequence[str] | None = None
+depends_on: str | Sequence[str] | None = None
+
+
+def upgrade() -> None:
+    with op.batch_alter_table("scope_infos", schema=None) as batch_op:
+        batch_op.drop_column("objective_thread_pitch")
+
+
+def downgrade() -> None:
+    with op.batch_alter_table("scope_infos", schema=None) as batch_op:
+        batch_op.add_column(
+            sa.Column("objective_thread_pitch", sa.String(), nullable=True)
+        )

alembic/versions/5b6a7817b408_add_static_connections.py

@@ -5,25 +5,29 @@ Revises: 7be5ef599d3f
 Create Date: 2026-07-02 11:57:13.095417
 """
 
-from typing import Sequence, Union
+from collections.abc import Sequence
+
+import sqlalchemy as sa
 
 from alembic import op
-import sqlalchemy as sa
 
 # revision identifiers, used by Alembic.
 revision: str = "5b6a7817b408"
-down_revision: Union[str, None] = "7be5ef599d3f"
+down_revision: str | None = "7be5ef599d3f"
-branch_labels: Union[str, Sequence[str], None] = None
+branch_labels: str | Sequence[str] | None = None
-depends_on: Union[str, Sequence[str], None] = None
+depends_on: str | Sequence[str] | None = None
 
 
 def upgrade() -> None:
-    # ### commands auto generated by Alembic - please adjust! ###
+    op.create_table(
-    pass
+        "static_connections",
-    # ### end Alembic commands ###
+        sa.Column("id", sa.Integer(), nullable=False),
+        sa.Column("od_mm", sa.Float(), nullable=False),
+        sa.Column("od_in", sa.Float(), nullable=False),
+        sa.PrimaryKeyConstraint("id"),
+        sa.UniqueConstraint("od_mm"),
+    )
 
 
 def downgrade() -> None:
-    # ### commands auto generated by Alembic - please adjust! ###
+    op.drop_table("static_connections")
-    pass
-    # ### end Alembic commands ###

alembic/versions/627d4460a94f_add_iso_threads_table.py

@@ -6,16 +6,15 @@ Create Date: 2026-06-24 21:50:05.340235
 """
 
 from collections.abc import Sequence
-from typing import Union
 
 import sqlalchemy as sa
 
 from alembic import op
 
 revision: str = "627d4460a94f"
-down_revision: Union[str, None] = "0001_initial"
+down_revision: str | None = "0001_initial"
-branch_labels: Union[str, Sequence[str], None] = None
+branch_labels: str | Sequence[str] | None = None
-depends_on: Union[str, Sequence[str], None] = None
+depends_on: str | Sequence[str] | None = None
 
 
 def upgrade() -> None:

alembic/versions/70e48bda8829_add_attachment_point_names.py

@@ -5,25 +5,28 @@ Revises: c7882cedc9ae
 Create Date: 2026-07-02 13:54:58.335394
 """
 
-from typing import Sequence, Union
+from collections.abc import Sequence
+
+import sqlalchemy as sa
 
 from alembic import op
-import sqlalchemy as sa
 
 # revision identifiers, used by Alembic.
 revision: str = "70e48bda8829"
-down_revision: Union[str, None] = "c7882cedc9ae"
+down_revision: str | None = "c7882cedc9ae"
-branch_labels: Union[str, Sequence[str], None] = None
+branch_labels: str | Sequence[str] | None = None
-depends_on: Union[str, Sequence[str], None] = None
+depends_on: str | Sequence[str] | None = None
 
 
 def upgrade() -> None:
-    # ### commands auto generated by Alembic - please adjust! ###
+    op.create_table(
-    pass
+        "attachment_point_names",
-    # ### end Alembic commands ###
+        sa.Column("id", sa.Integer(), nullable=False),
+        sa.Column("name", sa.String(), nullable=False),
+        sa.PrimaryKeyConstraint("id"),
+        sa.UniqueConstraint("name"),
+    )
 
 
 def downgrade() -> None:
-    # ### commands auto generated by Alembic - please adjust! ###
+    op.drop_table("attachment_point_names")
-    pass
-    # ### end Alembic commands ###

alembic/versions/7be5ef599d3f_add_imported_parts_tables.py

@@ -5,25 +5,42 @@ Revises: b60d0aa3ceda
 Create Date: 2026-06-25 15:29:20.685050
 """
 
-from typing import Sequence, Union
+from collections.abc import Sequence
+
+import sqlalchemy as sa
 
 from alembic import op
-import sqlalchemy as sa
 
 # revision identifiers, used by Alembic.
 revision: str = "7be5ef599d3f"
-down_revision: Union[str, None] = "b60d0aa3ceda"
+down_revision: str | None = "b60d0aa3ceda"
-branch_labels: Union[str, Sequence[str], None] = None
+branch_labels: str | Sequence[str] | None = None
-depends_on: Union[str, Sequence[str], None] = None
+depends_on: str | Sequence[str] | None = None
 
 
 def upgrade() -> None:
-    # ### commands auto generated by Alembic - please adjust! ###
+    op.create_table(
-    pass
+        "imported_parts",
-    # ### end Alembic commands ###
+        sa.Column("InventoryID", sa.String(), nullable=False),
+        sa.Column("Description", sa.String(), nullable=True),
+        sa.Column("TBXColour", sa.String(), nullable=True),
+        sa.Column("ProductMaxThreadDepthmm", sa.String(), nullable=True),
+        sa.Column("ProductMaxThreadDepthin", sa.String(), nullable=True),
+        sa.Column("ProductODmm", sa.String(), nullable=True),
+        sa.Column("ProductODin", sa.String(), nullable=True),
+        sa.Column("TBXThreadAttribute", sa.String(), nullable=True),
+        sa.PrimaryKeyConstraint("InventoryID"),
+    )
+    op.create_table(
+        "imported_part_config",
+        sa.Column("id", sa.Integer(), nullable=False),
+        sa.Column("odata_url", sa.String(), nullable=True),
+        sa.Column("username", sa.String(), nullable=True),
+        sa.Column("password", sa.String(), nullable=True),
+        sa.PrimaryKeyConstraint("id"),
+    )
 
 
 def downgrade() -> None:
-    # ### commands auto generated by Alembic - please adjust! ###
+    op.drop_table("imported_part_config")
-    pass
+    op.drop_table("imported_parts")
-    # ### end Alembic commands ###

alembic/versions/8012c00b0c04_add_augmented_part_notes_table.py

@@ -5,16 +5,17 @@ Revises: d1e2f3a4b5c6
 Create Date: 2026-07-31 09:07:06.137175
 """
 
-from typing import Sequence, Union
+from collections.abc import Sequence
+
+import sqlalchemy as sa
 
 from alembic import op
-import sqlalchemy as sa
 
 # revision identifiers, used by Alembic.
 revision: str = "8012c00b0c04"
-down_revision: Union[str, None] = "d1e2f3a4b5c6"
+down_revision: str | None = "d1e2f3a4b5c6"
-branch_labels: Union[str, Sequence[str], None] = None
+branch_labels: str | Sequence[str] | None = None
-depends_on: Union[str, Sequence[str], None] = None
+depends_on: str | Sequence[str] | None = None
 
 
 def upgrade() -> None:

alembic/versions/8bc411b5814e_add_augmented_parts.py

@@ -5,25 +5,48 @@ Revises: 0c3d1a5cff1f
 Create Date: 2026-07-02 12:37:59.457033
 """
 
-from typing import Sequence, Union
+from collections.abc import Sequence
+
+import sqlalchemy as sa
 
 from alembic import op
-import sqlalchemy as sa
 
 # revision identifiers, used by Alembic.
 revision: str = "8bc411b5814e"
-down_revision: Union[str, None] = "0c3d1a5cff1f"
+down_revision: str | None = "0c3d1a5cff1f"
-branch_labels: Union[str, Sequence[str], None] = None
+branch_labels: str | Sequence[str] | None = None
-depends_on: Union[str, Sequence[str], None] = None
+depends_on: str | Sequence[str] | None = None
 
 
 def upgrade() -> None:
-    # ### commands auto generated by Alembic - please adjust! ###
+    op.create_table(
-    pass
+        "augmented_parts",
-    # ### end Alembic commands ###
+        sa.Column("id", sa.Integer(), nullable=False),
+        sa.Column("sku", sa.String(), nullable=True),
+        sa.Column("description", sa.String(), nullable=False),
+        sa.Column("colour_id", sa.Integer(), nullable=False),
+        sa.Column("manufacturer_id", sa.Integer(), nullable=False),
+        sa.Column("scope_model_id", sa.Integer(), nullable=True),
+        sa.Column("part_type_id", sa.Integer(), nullable=False),
+        sa.Column("sub_part_type_id", sa.Integer(), nullable=True),
+        sa.ForeignKeyConstraint(
+            ["colour_id"], ["part_colours.id"], ondelete="RESTRICT"
+        ),
+        sa.ForeignKeyConstraint(
+            ["manufacturer_id"], ["manufacturers.id"], ondelete="RESTRICT"
+        ),
+        sa.ForeignKeyConstraint(
+            ["scope_model_id"], ["scope_models.id"], ondelete="SET NULL"
+        ),
+        sa.ForeignKeyConstraint(
+            ["part_type_id"], ["part_types.id"], ondelete="RESTRICT"
+        ),
+        sa.ForeignKeyConstraint(
+            ["sub_part_type_id"], ["sub_part_types.id"], ondelete="SET NULL"
+        ),
+        sa.PrimaryKeyConstraint("id"),
+    )
 
 
 def downgrade() -> None:
-    # ### commands auto generated by Alembic - please adjust! ###
+    op.drop_table("augmented_parts")
-    pass
-    # ### end Alembic commands ###

alembic/versions/9a1c2f6e0b3d_add_archived_to_augmented_parts.py

@@ -0,0 +1,32 @@
+"""add_archived_to_augmented_parts
+
+Revision ID: 9a1c2f6e0b3d
+Revises: 235d823a56ee
+Create Date: 2026-08-17 21:30:00.000000
+"""
+
+from collections.abc import Sequence
+
+import sqlalchemy as sa
+
+from alembic import op
+
+# revision identifiers, used by Alembic.
+revision: str = "9a1c2f6e0b3d"
+down_revision: str | None = "235d823a56ee"
+branch_labels: str | Sequence[str] | None = None
+depends_on: str | Sequence[str] | None = None
+
+
+def upgrade() -> None:
+    with op.batch_alter_table("augmented_parts", schema=None) as batch_op:
+        batch_op.add_column(
+            sa.Column(
+                "archived", sa.Boolean(), nullable=False, server_default=sa.false()
+            )
+        )
+
+
+def downgrade() -> None:
+    with op.batch_alter_table("augmented_parts", schema=None) as batch_op:
+        batch_op.drop_column("archived")

alembic/versions/a192abbf4091_add_manufacturers_table.py

@@ -5,16 +5,17 @@ Revises: 627d4460a94f
 Create Date: 2026-06-25 10:32:11.398529
 """
 
-from typing import Sequence, Union
+from collections.abc import Sequence
+
+import sqlalchemy as sa
 
 from alembic import op
-import sqlalchemy as sa
 
 # revision identifiers, used by Alembic.
 revision: str = "a192abbf4091"
-down_revision: Union[str, None] = "627d4460a94f"
+down_revision: str | None = "627d4460a94f"
-branch_labels: Union[str, Sequence[str], None] = None
+branch_labels: str | Sequence[str] | None = None
-depends_on: Union[str, Sequence[str], None] = None
+depends_on: str | Sequence[str] | None = None
 
 
 def upgrade() -> None:
@@ -27,12 +28,19 @@ def upgrade() -> None:
         sa.Column("logo_url", sa.String(), nullable=True),
         sa.PrimaryKeyConstraint("id"),
     )
+    # batch_alter_table's alter_column doesn't emit CREATE TYPE for the new
+    # Enum on its own (unlike create_table, which ties enum creation to the
+    # table-create DDL event) - create it explicitly first, or Postgres fails
+    # with `type "uncseries" does not exist`. checkfirst / no-op on SQLite.
+    uncseries = sa.Enum("UNC", "UNF", "UNEF", "UN", "UNS", name="uncseries")
+    uncseries.create(op.get_bind(), checkfirst=True)
     with op.batch_alter_table("unc_threads", schema=None) as batch_op:
         batch_op.alter_column(
             "series",
             existing_type=sa.VARCHAR(length=8),
-            type_=sa.Enum("UNC", "UNF", "UNEF", "UN", "UNS", name="uncseries"),
+            type_=uncseries,
             existing_nullable=False,
+            postgresql_using="series::uncseries",
         )
 
     # ### end Alembic commands ###
@@ -46,7 +54,9 @@ def downgrade() -> None:
             existing_type=sa.Enum("UNC", "UNF", "UNEF", "UN", "UNS", name="uncseries"),
             type_=sa.VARCHAR(length=8),
             existing_nullable=False,
+            postgresql_using="series::varchar(8)",
         )
+    sa.Enum(name="uncseries").drop(op.get_bind(), checkfirst=True)
 
     op.drop_table("manufacturers")
     # ### end Alembic commands ###

alembic/versions/a3f7c19b2d40_add_objective_lens_diameter_to_scope_infos.py

@@ -5,16 +5,17 @@ Revises: 8012c00b0c04
 Create Date: 2026-08-04 00:00:00.000000
 """
 
-from typing import Sequence, Union
+from collections.abc import Sequence
+
+import sqlalchemy as sa
 
 from alembic import op
-import sqlalchemy as sa
 
 # revision identifiers, used by Alembic.
 revision: str = "a3f7c19b2d40"
-down_revision: Union[str, None] = "8012c00b0c04"
+down_revision: str | None = "8012c00b0c04"
-branch_labels: Union[str, Sequence[str], None] = None
+branch_labels: str | Sequence[str] | None = None
-depends_on: Union[str, Sequence[str], None] = None
+depends_on: str | Sequence[str] | None = None
 
 
 def upgrade() -> None:

alembic/versions/a52e7ca7a925_add_scope_models_table.py

@@ -5,16 +5,17 @@ Revises: a192abbf4091
 Create Date: 2026-06-25 10:35:40.227253
 """
 
-from typing import Sequence, Union
+from collections.abc import Sequence
+
+import sqlalchemy as sa
 
 from alembic import op
-import sqlalchemy as sa
 
 # revision identifiers, used by Alembic.
 revision: str = "a52e7ca7a925"
-down_revision: Union[str, None] = "a192abbf4091"
+down_revision: str | None = "a192abbf4091"
-branch_labels: Union[str, Sequence[str], None] = None
+branch_labels: str | Sequence[str] | None = None
-depends_on: Union[str, Sequence[str], None] = None
+depends_on: str | Sequence[str] | None = None
 
 
 def upgrade() -> None:

alembic/versions/adf4a5b9493e_add_attachment_points.py

@@ -5,25 +5,52 @@ Revises: 70e48bda8829
 Create Date: 2026-07-02 14:03:23.561253
 """
 
-from typing import Sequence, Union
+from collections.abc import Sequence
+
+import sqlalchemy as sa
 
 from alembic import op
-import sqlalchemy as sa
 
 # revision identifiers, used by Alembic.
 revision: str = "adf4a5b9493e"
-down_revision: Union[str, None] = "70e48bda8829"
+down_revision: str | None = "70e48bda8829"
-branch_labels: Union[str, Sequence[str], None] = None
+branch_labels: str | Sequence[str] | None = None
-depends_on: Union[str, Sequence[str], None] = None
+depends_on: str | Sequence[str] | None = None
 
 
 def upgrade() -> None:
-    # ### commands auto generated by Alembic - please adjust! ###
+    op.create_table(
-    pass
+        "attachment_points",
-    # ### end Alembic commands ###
+        sa.Column("id", sa.Integer(), nullable=False),
+        sa.Column("augmented_part_id", sa.Integer(), nullable=False),
+        sa.Column("attachment_point_name_id", sa.Integer(), nullable=False),
+        sa.Column("iso_thread_id", sa.Integer(), nullable=True),
+        sa.Column("unc_thread_id", sa.Integer(), nullable=True),
+        sa.Column("clamp_connection_id", sa.Integer(), nullable=True),
+        sa.Column("static_connection_id", sa.Integer(), nullable=True),
+        sa.ForeignKeyConstraint(
+            ["augmented_part_id"], ["augmented_parts.id"], ondelete="CASCADE"
+        ),
+        sa.ForeignKeyConstraint(
+            ["attachment_point_name_id"],
+            ["attachment_point_names.id"],
+            ondelete="RESTRICT",
+        ),
+        sa.ForeignKeyConstraint(
+            ["iso_thread_id"], ["iso_threads.id"], ondelete="RESTRICT"
+        ),
+        sa.ForeignKeyConstraint(
+            ["unc_thread_id"], ["unc_threads.id"], ondelete="RESTRICT"
+        ),
+        sa.ForeignKeyConstraint(
+            ["clamp_connection_id"], ["clamp_connections.id"], ondelete="RESTRICT"
+        ),
+        sa.ForeignKeyConstraint(
+            ["static_connection_id"], ["static_connections.id"], ondelete="RESTRICT"
+        ),
+        sa.PrimaryKeyConstraint("id"),
+    )
 
 
 def downgrade() -> None:
-    # ### commands auto generated by Alembic - please adjust! ###
+    op.drop_table("attachment_points")
-    pass
-    # ### end Alembic commands ###

alembic/versions/b60d0aa3ceda_add_sub_part_types_table.py

@@ -5,25 +5,32 @@ Revises: fd5379986da4
 Create Date: 2026-06-25 11:29:20.420056
 """
 
-from typing import Sequence, Union
+from collections.abc import Sequence
+
+import sqlalchemy as sa
 
 from alembic import op
-import sqlalchemy as sa
 
 # revision identifiers, used by Alembic.
 revision: str = "b60d0aa3ceda"
-down_revision: Union[str, None] = "fd5379986da4"
+down_revision: str | None = "fd5379986da4"
-branch_labels: Union[str, Sequence[str], None] = None
+branch_labels: str | Sequence[str] | None = None
-depends_on: Union[str, Sequence[str], None] = None
+depends_on: str | Sequence[str] | None = None
 
 
 def upgrade() -> None:
-    # ### commands auto generated by Alembic - please adjust! ###
+    op.create_table(
-    pass
+        "sub_part_types",
-    # ### end Alembic commands ###
+        sa.Column("id", sa.Integer(), nullable=False),
+        sa.Column("name", sa.String(), nullable=False),
+        sa.Column("part_type_id", sa.Integer(), nullable=False),
+        sa.ForeignKeyConstraint(
+            ["part_type_id"], ["part_types.id"], ondelete="RESTRICT"
+        ),
+        sa.PrimaryKeyConstraint("id"),
+        sa.UniqueConstraint("name", "part_type_id"),
+    )
 
 
 def downgrade() -> None:
-    # ### commands auto generated by Alembic - please adjust! ###
+    op.drop_table("sub_part_types")
-    pass
-    # ### end Alembic commands ###

alembic/versions/bfbce324cd5f_add_augmented_part_imported_parts.py

@@ -5,25 +5,33 @@ Revises: 8bc411b5814e
 Create Date: 2026-07-02 12:46:27.493942
 """
 
-from typing import Sequence, Union
+from collections.abc import Sequence
+
+import sqlalchemy as sa
 
 from alembic import op
-import sqlalchemy as sa
 
 # revision identifiers, used by Alembic.
 revision: str = "bfbce324cd5f"
-down_revision: Union[str, None] = "8bc411b5814e"
+down_revision: str | None = "8bc411b5814e"
-branch_labels: Union[str, Sequence[str], None] = None
+branch_labels: str | Sequence[str] | None = None
-depends_on: Union[str, Sequence[str], None] = None
+depends_on: str | Sequence[str] | None = None
 
 
 def upgrade() -> None:
-    # ### commands auto generated by Alembic - please adjust! ###
+    op.create_table(
-    pass
+        "augmented_part_imported_parts",
-    # ### end Alembic commands ###
+        sa.Column("augmented_part_id", sa.Integer(), nullable=False),
+        sa.Column("imported_part_id", sa.String(), nullable=False),
+        sa.ForeignKeyConstraint(
+            ["augmented_part_id"], ["augmented_parts.id"], ondelete="CASCADE"
+        ),
+        sa.ForeignKeyConstraint(
+            ["imported_part_id"], ["imported_parts.InventoryID"], ondelete="CASCADE"
+        ),
+        sa.PrimaryKeyConstraint("augmented_part_id", "imported_part_id"),
+    )
 
 
 def downgrade() -> None:
-    # ### commands auto generated by Alembic - please adjust! ###
+    op.drop_table("augmented_part_imported_parts")
-    pass
-    # ### end Alembic commands ###

alembic/versions/c7882cedc9ae_add_clamp_connections.py

@@ -5,25 +5,30 @@ Revises: bfbce324cd5f
 Create Date: 2026-07-02 13:31:57.064381
 """
 
-from typing import Sequence, Union
+from collections.abc import Sequence
+
+import sqlalchemy as sa
 
 from alembic import op
-import sqlalchemy as sa
 
 # revision identifiers, used by Alembic.
 revision: str = "c7882cedc9ae"
-down_revision: Union[str, None] = "bfbce324cd5f"
+down_revision: str | None = "bfbce324cd5f"
-branch_labels: Union[str, Sequence[str], None] = None
+branch_labels: str | Sequence[str] | None = None
-depends_on: Union[str, Sequence[str], None] = None
+depends_on: str | Sequence[str] | None = None
 
 
 def upgrade() -> None:
-    # ### commands auto generated by Alembic - please adjust! ###
+    op.create_table(
-    pass
+        "clamp_connections",
-    # ### end Alembic commands ###
+        sa.Column("id", sa.Integer(), nullable=False),
+        sa.Column("min_mm", sa.Float(), nullable=True),
+        sa.Column("max_mm", sa.Float(), nullable=True),
+        sa.Column("min_in", sa.Float(), nullable=True),
+        sa.Column("max_in", sa.Float(), nullable=True),
+        sa.PrimaryKeyConstraint("id"),
+    )
 
 
 def downgrade() -> None:
-    # ### commands auto generated by Alembic - please adjust! ###
+    op.drop_table("clamp_connections")
-    pass
-    # ### end Alembic commands ###

alembic/versions/d1e2f3a4b5c6_add_scope_infos_table.py

@@ -5,16 +5,17 @@ Revises: c9f1a2b3d4e5
 Create Date: 2026-07-16 20:00:00.000000
 """
 
-from typing import Sequence, Union
+from collections.abc import Sequence
+
+import sqlalchemy as sa
 
 from alembic import op
-import sqlalchemy as sa
 
 # revision identifiers, used by Alembic.
 revision: str = "d1e2f3a4b5c6"
-down_revision: Union[str, None] = "c9f1a2b3d4e5"
+down_revision: str | None = "c9f1a2b3d4e5"
-branch_labels: Union[str, Sequence[str], None] = None
+branch_labels: str | Sequence[str] | None = None
-depends_on: Union[str, Sequence[str], None] = None
+depends_on: str | Sequence[str] | None = None
 
 
 def upgrade() -> None:

alembic/versions/e23aa8dd2536_add_part_types_table.py

@@ -5,16 +5,17 @@ Revises: a52e7ca7a925
 Create Date: 2026-06-25 10:49:56.463466
 """
 
-from typing import Sequence, Union
+from collections.abc import Sequence
+
+import sqlalchemy as sa
 
 from alembic import op
-import sqlalchemy as sa
 
 # revision identifiers, used by Alembic.
 revision: str = "e23aa8dd2536"
-down_revision: Union[str, None] = "a52e7ca7a925"
+down_revision: str | None = "a52e7ca7a925"
-branch_labels: Union[str, Sequence[str], None] = None
+branch_labels: str | Sequence[str] | None = None
-depends_on: Union[str, Sequence[str], None] = None
+depends_on: str | Sequence[str] | None = None
 
 
 def upgrade() -> None:

alembic/versions/fd5379986da4_add_audit_logs_table.py

@@ -5,25 +5,43 @@ Revises: e23aa8dd2536
 Create Date: 2026-06-25 11:06:13.028067
 """
 
-from typing import Sequence, Union
+from collections.abc import Sequence
+
+import sqlalchemy as sa
 
 from alembic import op
-import sqlalchemy as sa
 
 # revision identifiers, used by Alembic.
 revision: str = "fd5379986da4"
-down_revision: Union[str, None] = "e23aa8dd2536"
+down_revision: str | None = "e23aa8dd2536"
-branch_labels: Union[str, Sequence[str], None] = None
+branch_labels: str | Sequence[str] | None = None
-depends_on: Union[str, Sequence[str], None] = None
+depends_on: str | Sequence[str] | None = None
 
 
 def upgrade() -> None:
-    # ### commands auto generated by Alembic - please adjust! ###
+    op.create_table(
-    pass
+        "audit_logs",
-    # ### end Alembic commands ###
+        sa.Column("id", sa.Integer(), nullable=False),
+        sa.Column(
+            "created_at",
+            sa.DateTime(timezone=True),
+            server_default=sa.func.now(),
+            nullable=False,
+        ),
+        sa.Column("user_id", sa.Integer(), nullable=True),
+        sa.Column("user_email", sa.String(length=255), nullable=False),
+        sa.Column("action", sa.String(length=16), nullable=False),
+        sa.Column("resource_type", sa.String(length=64), nullable=False),
+        sa.Column("resource_id", sa.Integer(), nullable=True),
+        sa.Column("description", sa.String(), nullable=False),
+        sa.ForeignKeyConstraint(["user_id"], ["users.id"], ondelete="SET NULL"),
+        sa.PrimaryKeyConstraint("id"),
+    )
+    op.create_index("ix_audit_logs_created_at", "audit_logs", ["created_at"])
+    op.create_index("ix_audit_logs_user_id", "audit_logs", ["user_id"])
 
 
 def downgrade() -> None:
-    # ### commands auto generated by Alembic - please adjust! ###
+    op.drop_index("ix_audit_logs_user_id", table_name="audit_logs")
-    pass
+    op.drop_index("ix_audit_logs_created_at", table_name="audit_logs")
-    # ### end Alembic commands ###
+    op.drop_table("audit_logs")

app/api/attachment_points.py

@@ -27,7 +27,7 @@ def _connection_label(ap) -> str:
         return f"{cc.min_in}–{cc.max_in} in"
     if ap.static_connection:
         return f"{ap.static_connection.od_mm} mm OD"
-    return "—"
+    return "Manual"
 
 
 @router.get("", response_model=list[AttachmentPointRead])
@@ -54,10 +54,14 @@ async def create_attachment_point(
         payload.clamp_connection_id,
         payload.static_connection_id,
     ]
-    if sum(v is not None for v in conn_ids) != 1:
+    if sum(v is not None for v in conn_ids) > 1:
         raise HTTPException(
             status_code=400,
-            detail="Exactly one connection (iso_thread, unc_thread, clamp_connection, or static_connection) must be provided.",
+            detail=(
+                "At most one connection (iso_thread, unc_thread, clamp_connection, "
+                "or static_connection) may be provided. Leave all unset for a "
+                "manual attachment point."
+            ),
         )
     ap = await attachment_point_crud.create(db, **payload.model_dump())
     await audit_crud.log(

app/api/augmented_parts.py

@@ -24,10 +24,13 @@ router = APIRouter()
 async def list_augmented_parts(
     skip: int = 0,
     limit: int = 100,
+    include_archived: bool = False,
     db: AsyncSession = Depends(get_db),
     _: User = Depends(require_permission("augmented_parts:read")),
 ):
-    return await augmented_part_crud.list_all(db, skip=skip, limit=limit)
+    return await augmented_part_crud.list_all(
+        db, skip=skip, limit=limit, include_archived=include_archived
+    )
 
 
 @router.post("", response_model=AugmentedPartRead, status_code=status.HTTP_201_CREATED)
@@ -72,7 +75,7 @@ async def sync_augmented_parts(
         resource_type="AugmentedPart",
         resource_id=None,
         description=(
-            f"Sync: {result.created} created, {result.deleted} deleted, "
+            f"Sync: {result.created} created, {result.archived} archived, "
             f"{result.skipped} skipped (no type match)"
         ),
     )

app/api/imported_parts.py

@@ -13,6 +13,7 @@ from app.schemas.imported_part import (
     ImportedPartRead,
     SyncResult,
 )
+from app.services import augmented_part_sync
 from app.services import odata as odata_service
 
 router = APIRouter()
@@ -128,7 +129,9 @@ async def delete_imported_part(
         raise HTTPException(
             status_code=status.HTTP_404_NOT_FOUND, detail="Imported part not found"
         )
-    await imported_part_crud.delete(db, part)
+    archived = await augmented_part_sync.delete_imported_part_and_archive_orphans(
+        db, part
+    )
     await audit_crud.log(
         db,
         actor=actor,
@@ -137,3 +140,15 @@ async def delete_imported_part(
         resource_id=None,
         description=audit_crud.describe_delete("ImportedPart", 0, inventory_id),
     )
+    if archived:
+        await audit_crud.log(
+            db,
+            actor=actor,
+            action="update",
+            resource_type="AugmentedPart",
+            resource_id=None,
+            description=(
+                f"Archived {archived} augmented part(s) left with no remaining "
+                f"imported parts after deleting {inventory_id!r}"
+            ),
+        )

app/core/config.py

@@ -33,9 +33,13 @@ class Settings(BaseSettings):
     database_url: str = "sqlite+aiosqlite:///./app.db"
     auto_create_tables: bool = True
 
-    # Web / cookies (the HTMX UI stores the access token in an httpOnly cookie)
+    # Web / cookies (the HTMX UI stores the access token in an httpOnly cookie).
+    # session_timeout_hours governs how long a browser login lasts before the
+    # user is bounced back to /login; it's independent of the JSON API's
+    # short-lived access_token_expire_minutes.
     cookie_name: str = "access_token"
     cookie_secure: bool = False
+    session_timeout_hours: int = 24
 
     # LDAP (direct-bind). When enabled, the login page offers an LDAP option.
     # At login the user's DN is built from ``ldap_bind_dn_template`` (with

app/core/security.py

@@ -47,10 +47,12 @@ def _create_token(
     return token, payload
 
 
-def create_access_token(subject: str | int) -> str:
+def create_access_token(
+    subject: str | int, expires_delta: timedelta | None = None
+) -> str:
     token, _ = _create_token(
         subject,
-        timedelta(minutes=settings.access_token_expire_minutes),
+        expires_delta or timedelta(minutes=settings.access_token_expire_minutes),
         ACCESS_TOKEN_TYPE,
     )
     return token

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
@@ -58,14 +58,15 @@ async def get_by_sku(db: AsyncSession, sku: str) -> AugmentedPart | None:
 
 
 async def list_all(
-    db: AsyncSession, skip: int = 0, limit: int = 100
+    db: AsyncSession,
+    skip: int = 0,
+    limit: int = 100,
+    include_archived: bool = False,
 ) -> list[AugmentedPart]:
-    result = await db.execute(
+    q = select(AugmentedPart).order_by(AugmentedPart.description)
-        select(AugmentedPart)
+    if not include_archived:
-        .order_by(AugmentedPart.description)
+        q = q.where(AugmentedPart.archived.is_(False))
-        .offset(skip)
+    result = await db.execute(q.offset(skip).limit(limit))
-        .limit(limit)
-    )
     return list(result.scalars().all())
 
 
@@ -85,6 +86,7 @@ async def list_filtered(
     scope_model_id: int | None = None,
     part_type_id: int | None = None,
     sub_part_type_id: int | None = None,
+    include_archived: bool = False,
     sort: str = "description",
     direction: str = "asc",
     skip: int = 0,
@@ -92,7 +94,10 @@ async def list_filtered(
 ) -> list[AugmentedPart]:
     q = select(AugmentedPart)
     if description.strip():
-        q = q.where(AugmentedPart.description.ilike(f"%{description.strip()}%"))
+        term = f"%{description.strip()}%"
+        q = q.where(
+            or_(AugmentedPart.description.ilike(term), AugmentedPart.sku.ilike(term))
+        )
     if colour_id is not None:
         q = q.where(AugmentedPart.colour_id == colour_id)
     if manufacturer_id is not None:
@@ -103,6 +108,8 @@ async def list_filtered(
         q = q.where(AugmentedPart.part_type_id == part_type_id)
     if sub_part_type_id is not None:
         q = q.where(AugmentedPart.sub_part_type_id == sub_part_type_id)
+    if not include_archived:
+        q = q.where(AugmentedPart.archived.is_(False))
     q = _apply_sort(q, sort, direction)
     result = await db.execute(q.offset(skip).limit(limit))
     return list(result.scalars().all())
@@ -116,12 +123,16 @@ async def count_filtered(
     scope_model_id: int | None = None,
     part_type_id: int | None = None,
     sub_part_type_id: int | None = None,
+    include_archived: bool = False,
 ) -> int:
     from sqlalchemy import func
 
     q = select(func.count()).select_from(AugmentedPart)
     if description.strip():
-        q = q.where(AugmentedPart.description.ilike(f"%{description.strip()}%"))
+        term = f"%{description.strip()}%"
+        q = q.where(
+            or_(AugmentedPart.description.ilike(term), AugmentedPart.sku.ilike(term))
+        )
     if colour_id is not None:
         q = q.where(AugmentedPart.colour_id == colour_id)
     if manufacturer_id is not None:
@@ -132,6 +143,8 @@ async def count_filtered(
         q = q.where(AugmentedPart.part_type_id == part_type_id)
     if sub_part_type_id is not None:
         q = q.where(AugmentedPart.sub_part_type_id == sub_part_type_id)
+    if not include_archived:
+        q = q.where(AugmentedPart.archived.is_(False))
     result = await db.execute(q)
     return result.scalar_one()
 
@@ -245,6 +258,15 @@ async def get_unlinked_imported_parts(db: AsyncSession) -> list[ImportedPart]:
     return list(result.scalars().all())
 
 
+async def set_archived(
+    db: AsyncSession, ap: AugmentedPart, archived: bool
+) -> AugmentedPart:
+    ap.archived = archived
+    await db.commit()
+    await db.refresh(ap, attribute_names=_RELATIONSHIPS)
+    return ap
+
+
 async def delete(db: AsyncSession, ap: AugmentedPart) -> None:
     await db.delete(ap)
     await db.commit()

app/crud/imported_part.py

@@ -1,4 +1,4 @@
-from sqlalchemy import distinct, insert, select, update
+from sqlalchemy import distinct, insert, or_, select, update
 from sqlalchemy.ext.asyncio import AsyncSession
 
 from app.models.imported_part import ImportedPart
@@ -61,7 +61,13 @@ async def list_filtered(
 ) -> list[ImportedPart]:
     q = select(ImportedPart)
     if description.strip():
-        q = q.where(ImportedPart.Description.ilike(f"%{description.strip()}%"))
+        term = f"%{description.strip()}%"
+        q = q.where(
+            or_(
+                ImportedPart.Description.ilike(term),
+                ImportedPart.InventoryID.ilike(term),
+            )
+        )
     q = _apply_sort(q, sort, direction)
     result = await db.execute(q.offset(skip).limit(limit))
     return list(result.scalars().all())
@@ -88,16 +94,17 @@ async def count_filtered(db: AsyncSession, description: str = "") -> int:
 
     q = select(func.count()).select_from(ImportedPart)
     if description.strip():
-        q = q.where(ImportedPart.Description.ilike(f"%{description.strip()}%"))
+        term = f"%{description.strip()}%"
+        q = q.where(
+            or_(
+                ImportedPart.Description.ilike(term),
+                ImportedPart.InventoryID.ilike(term),
+            )
+        )
     result = await db.execute(q)
     return result.scalar_one()
 
 
-async def delete(db: AsyncSession, part: ImportedPart) -> None:
-    await db.delete(part)
-    await db.commit()
-
-
 async def upsert_bulk(db: AsyncSession, records: list[dict]) -> tuple[int, int]:
     """Insert-or-update records keyed on InventoryID. Returns (created, updated)."""
     if not records:

app/crud/scope_info.py

@@ -37,7 +37,6 @@ async def create(
     magnification_max: float | None = None,
     model_suffix: str | None = None,
     objective_lens_diameter_mm: float | None = None,
-    objective_thread_pitch: str | None = None,
 ) -> ScopeInfo:
     info = ScopeInfo(
         augmented_part_id=augmented_part_id,
@@ -45,7 +44,6 @@ async def create(
         magnification_max=magnification_max,
         model_suffix=model_suffix,
         objective_lens_diameter_mm=objective_lens_diameter_mm,
-        objective_thread_pitch=objective_thread_pitch,
     )
     db.add(info)
     await db.commit()

app/main.py

@@ -49,7 +49,7 @@ def create_app() -> FastAPI:
         # Preserve the originally requested URL so we can return there after
         # login. Only meaningful for GET navigations (the ones users deep-link).
         login_url = "/login"
-        if request.method == "GET" and request.url.path not in ("/", "/login"):
+        if request.method == "GET" and request.url.path != "/login":
             target = request.url.path
             if request.url.query:
                 target += "?" + request.url.query

app/models/__init__.py

@@ -19,12 +19,12 @@ from app.models.manufacturer import Manufacturer
 from app.models.part_colour import PartColour
 from app.models.part_type import PartType
 from app.models.permission import Permission
+from app.models.refresh_token import RefreshToken
+from app.models.role import Role
 from app.models.scope_info import ScopeInfo
 from app.models.scope_model import ScopeModel
 from app.models.static_connection import StaticConnection
 from app.models.sub_part_type import SubPartType
-from app.models.refresh_token import RefreshToken
-from app.models.role import Role
 from app.models.unc_thread import UncThread
 from app.models.user import User
 

app/models/attachment_point.py

@@ -21,7 +21,10 @@ class AttachmentPoint(Base):
         ForeignKey("attachment_point_names.id", ondelete="RESTRICT"), nullable=False
     )
 
-    # Exactly one of the four below must be non-null (enforced in application layer)
+    # At most one of the four below is non-null (enforced in application layer).
+    # All four null means a "Manual" attachment point: no connection details,
+    # used to let a user assert a Fit by hand without the fit-matching algorithm
+    # (app.services.fit_matching) ever auto-matching it.
     iso_thread_id: Mapped[int | None] = mapped_column(
         ForeignKey("iso_threads.id", ondelete="RESTRICT"), nullable=True, default=None
     )

app/models/augmented_part.py

@@ -1,6 +1,6 @@
 from datetime import datetime
 
-from sqlalchemy import DateTime, ForeignKey, String, Text, func
+from sqlalchemy import Boolean, DateTime, ForeignKey, String, Text, func
 from sqlalchemy.orm import Mapped, mapped_column, relationship
 
 from app.core.database import Base
@@ -36,6 +36,10 @@ class AugmentedPart(Base):
         nullable=True,
         default=None,
     )
+    # Set instead of deleting the row when the last backing ImportedPart is
+    # removed, so fit/attachment-point history is preserved. Hidden from list
+    # views by default.
+    archived: Mapped[bool] = mapped_column(Boolean, nullable=False, default=False)
 
     # linked imported parts — not eagerly loaded; used by sync service only
     imported_parts: Mapped[list[ImportedPart]] = relationship(

app/models/fit.py

@@ -7,7 +7,7 @@ from sqlalchemy.orm import Mapped, mapped_column, relationship
 from app.core.database import Base
 
 
-class FitLabel(str, enum.Enum):
+class FitLabel(enum.StrEnum):
     POSSIBLE = "possible"
     EXACT = "exact"
     NONE = "none"

app/models/manufacturer.py

@@ -8,5 +8,5 @@ class Manufacturer(Base):
 
     id: Mapped[int] = mapped_column(primary_key=True)
     name: Mapped[str] = mapped_column(nullable=False)
-    url: Mapped[str] = mapped_column(nullable=False)
+    url: Mapped[str] = mapped_column(nullable=True)
     logo_url: Mapped[str | None] = mapped_column(nullable=True, default=None)

app/models/scope_info.py

@@ -32,8 +32,3 @@ class ScopeInfo(Base):
     objective_lens_diameter_mm: Mapped[float | None] = mapped_column(
         Float, nullable=True, default=None
     )
-    # Kept as the raw string from the sheet (e.g. "0.75", "1/16") so no precision
-    # or notation is lost; the parsed thread lives on its own AttachmentPoint.
-    objective_thread_pitch: Mapped[str | None] = mapped_column(
-        String, nullable=True, default=None
-    )

app/models/unc_thread.py

@@ -6,7 +6,7 @@ from sqlalchemy.orm import Mapped, mapped_column
 from app.core.database import Base
 
 
-class UncSeries(str, enum.Enum):
+class UncSeries(enum.StrEnum):
     UNC = "UNC"
     UNF = "UNF"
     UNEF = "UNEF"

app/schemas/augmented_part.py

@@ -53,11 +53,12 @@ class AugmentedPartUpdate(BaseModel):
     scope_model_id: int | None = None
     part_type_id: int | None = None
     sub_part_type_id: int | None = None
+    archived: bool | None = None
 
 
 class AugmentedPartSyncResult(BaseModel):
     created: int
-    deleted: int
+    archived: int
     skipped: int
 
 
@@ -65,6 +66,7 @@ class AugmentedPartRead(AugmentedPartBase):
     model_config = ConfigDict(from_attributes=True)
 
     id: int
+    archived: bool
     colour: PartColourRead
     manufacturer: ManufacturerRead
     scope_model: ScopeModelRead | None

app/schemas/manufacturer.py

@@ -3,7 +3,7 @@ from pydantic import BaseModel, ConfigDict, HttpUrl
 
 class ManufacturerBase(BaseModel):
     name: str
-    url: HttpUrl
+    url: HttpUrl | None = None
     logo_url: HttpUrl | None = None
 
 

app/schemas/scope_info.py

@@ -6,7 +6,6 @@ class ScopeInfoBase(BaseModel):
     magnification_max: float | None = None
     model_suffix: str | None = None
     objective_lens_diameter_mm: float | None = None
-    objective_thread_pitch: str | None = None
 
 
 class ScopeInfoRead(ScopeInfoBase):

app/services/augmented_part_sync.py

@@ -2,11 +2,12 @@
 
 Algorithm
 ---------
-Phase 1 – prune stale entries
+Phase 1 – archive stale entries
   • Find association rows whose imported_part_id no longer exists.
   • Delete those rows.
-  • Delete any AugmentedPart that ends up with zero remaining associations
+  • Archive (rather than delete) any AugmentedPart that ends up with zero
-    (i.e. it was created purely from now-deleted ImportedParts).
+    remaining associations (i.e. it was created purely from now-deleted
+    ImportedParts), preserving its fit/attachment-point history.
 
 Phase 2 – create entries for new ImportedParts
   • Any ImportedPart not yet linked to an AugmentedPart gets one created.
@@ -17,7 +18,7 @@ Phase 2 – create entries for new ImportedParts
   • Type          ← first PartType whose name is a substring of the description
                     (longest name wins; skipped if no match)
   • SubType       ← null
-  • Attachment point ← always named "Occular"; one of:
+  • Attachment point ← always named "Ocular"; one of:
                     - ISO/UNC thread parsed from TBXThreadAttribute (using the
                       matching ProductMaxThreadDepth mm/in if populated), or
                     - StaticConnection from ProductODmm/ProductODin (whichever
@@ -33,6 +34,7 @@ from sqlalchemy.ext.asyncio import AsyncSession
 
 from app.crud import attachment_point as attachment_point_crud
 from app.crud import attachment_point_name as attachment_point_name_crud
+from app.crud import clamp_connection as clamp_connection_crud
 from app.crud import iso_thread as iso_thread_crud
 from app.crud import manufacturer as manufacturer_crud
 from app.crud import part_colour as part_colour_crud
@@ -46,7 +48,7 @@ from app.models.part_type import PartType
 from app.models.unc_thread import UncSeries
 from app.schemas.augmented_part import AugmentedPartSyncResult
 
-ATTACHMENT_POINT_NAME = "Occular"
+ATTACHMENT_POINT_NAME = "Ocular"
 MM_PER_INCH = 25.4
 
 # e.g.
@@ -70,6 +72,11 @@ _UNC_THREAD_RE = re.compile(
     re.IGNORECASE,
 )
 
+_CLAMP_RANGE_RE = re.compile(
+    r"\((?P<min_diameter>\d*\.?\d+)\s*-\s*(?P<max_diameter>\d*\.?\d+)\s*mm\)",
+    re.IGNORECASE,
+)
+
 
 def _parse_float(value: str | None) -> float | None:
     if value is None:
@@ -101,6 +108,13 @@ def _parse_thread_attribute(value: str) -> tuple[str, dict] | None:
             "series": series,
         }
 
+    if m := _CLAMP_RANGE_RE.match(value):
+        # Source data often omits the series letters; UNS ("unified special") is the
+        # correct designation when the diameter/TPI pairing isn't a standard pitch.
+        return "clamp", {
+            "min_diametere": float(m.group("min_diameter")),
+            "max_diameter": float(m.group("max_diameter")),
+        }
     return None
 
 
@@ -144,6 +158,7 @@ async def _add_attachment_point(
                 attachment_point_name_id=apn.id,
                 iso_thread_id=thread.id,
             )
+            return
         elif kind == "unc":
             depth_in = _parse_float(ip.ProductMaxThreadDepthin)
             thread = await unc_thread_crud.get_by_values(
@@ -157,9 +172,21 @@ async def _add_attachment_point(
                 attachment_point_name_id=apn.id,
                 unc_thread_id=thread.id,
             )
-        else:
-            pass
             return
+        elif kind == "clamp":
+            clamp = await clamp_connection_crud.get_by_values(db, **fields)
+            if clamp is None:
+                clamp = await clamp_connection_crud.create(db, **fields)
+            await attachment_point_crud.create(
+                db,
+                augmented_part_id=ap.id,
+                attachment_point_name_id=apn.id,
+                clamp_connection_id=clamp.id,
+            )
+            return
+        else:
+            # Unable to determine
+            pass
 
     od_mm = _parse_float(ip.ProductODmm)
     od_in = _parse_float(ip.ProductODin)
@@ -242,6 +269,91 @@ def _infer_part_type(description: str, part_types: list[PartType]) -> PartType |
     return None
 
 
+async def _archive_orphans(db: AsyncSession, candidate_ap_ids: set[int]) -> int:
+    """Archive any of `candidate_ap_ids` left with zero linked ImportedParts.
+
+    Idempotent: already-archived parts and parts that still have a link are
+    left untouched. Does not commit — the caller commits as part of its own
+    transaction.
+    """
+    archived = 0
+    for ap_id in candidate_ap_ids:
+        result = await db.execute(
+            select(func.count())
+            .select_from(augmented_part_imported_parts)
+            .where(augmented_part_imported_parts.c.augmented_part_id == ap_id)
+        )
+        if result.scalar() == 0:
+            ap = await db.get(AugmentedPart, ap_id)
+            if ap is not None and not ap.archived:
+                ap.archived = True
+                archived += 1
+    return archived
+
+
+async def _detach_and_archive_orphans(
+    db: AsyncSession, imported_part_ids: set[str]
+) -> int:
+    """Detach `imported_part_ids` from any AugmentedPart association and archive
+    AugmentedParts left with zero remaining ImportedPart links as a result.
+
+    Deletes the association rows directly rather than relying on the
+    ImportedPart row's ON DELETE CASCADE, since SQLite (this app's dev/test
+    database) doesn't enforce FK actions by default. Safe to call whether or
+    not the ImportedPart row(s) still exist. Does not commit — the caller
+    commits as part of its own transaction.
+    """
+    if not imported_part_ids:
+        return 0
+
+    result = await db.execute(
+        select(augmented_part_imported_parts.c.augmented_part_id).where(
+            augmented_part_imported_parts.c.imported_part_id.in_(imported_part_ids)
+        )
+    )
+    candidate_ap_ids: set[int] = {row[0] for row in result.all()}
+
+    await db.execute(
+        augmented_part_imported_parts.delete().where(
+            augmented_part_imported_parts.c.imported_part_id.in_(imported_part_ids)
+        )
+    )
+    await db.flush()
+
+    return await _archive_orphans(db, candidate_ap_ids)
+
+
+async def delete_imported_part_and_archive_orphans(
+    db: AsyncSession, imported_part: ImportedPart
+) -> int:
+    """Delete `imported_part` and archive any AugmentedPart left with no
+    remaining ImportedPart links as a result.
+
+    Captures affected AugmentedPart ids before deleting so this is correct
+    regardless of whether the DB enforces the association table's ON DELETE
+    CASCADE. Returns the number of AugmentedParts newly archived.
+    """
+    inventory_id = imported_part.InventoryID
+    result = await db.execute(
+        select(augmented_part_imported_parts.c.augmented_part_id).where(
+            augmented_part_imported_parts.c.imported_part_id == inventory_id
+        )
+    )
+    candidate_ap_ids: set[int] = {row[0] for row in result.all()}
+
+    await db.delete(imported_part)
+    await db.execute(
+        augmented_part_imported_parts.delete().where(
+            augmented_part_imported_parts.c.imported_part_id == inventory_id
+        )
+    )
+    await db.flush()
+
+    archived = await _archive_orphans(db, candidate_ap_ids)
+    await db.commit()
+    return archived
+
+
 async def sync(db: AsyncSession) -> AugmentedPartSyncResult:
     # ------------------------------------------------------------------ phase 1
 
@@ -256,42 +368,10 @@ async def sync(db: AsyncSession) -> AugmentedPartSyncResult:
     current_ids: set[str] = {row[0] for row in result.all()}
 
     orphaned_import_ids = all_linked_ids - current_ids
-    deleted = 0
+    archived = 0
 
     if orphaned_import_ids:
-        # Which AugmentedParts are affected?
+        archived = await _detach_and_archive_orphans(db, orphaned_import_ids)
-        result = await db.execute(
-            select(augmented_part_imported_parts.c.augmented_part_id).where(
-                augmented_part_imported_parts.c.imported_part_id.in_(
-                    orphaned_import_ids
-                )
-            )
-        )
-        orphaned_ap_ids: set[int] = {row[0] for row in result.all()}
-
-        # Remove the stale association rows
-        await db.execute(
-            augmented_part_imported_parts.delete().where(
-                augmented_part_imported_parts.c.imported_part_id.in_(
-                    orphaned_import_ids
-                )
-            )
-        )
-        await db.flush()
-
-        # Delete AugmentedParts that now have no remaining associations
-        for ap_id in orphaned_ap_ids:
-            result = await db.execute(
-                select(func.count())
-                .select_from(augmented_part_imported_parts)
-                .where(augmented_part_imported_parts.c.augmented_part_id == ap_id)
-            )
-            if result.scalar() == 0:
-                ap = await db.get(AugmentedPart, ap_id)
-                if ap is not None:
-                    await db.delete(ap)
-                    deleted += 1
-
         await db.commit()
 
     # ------------------------------------------------------------------ phase 2
@@ -308,7 +388,7 @@ async def sync(db: AsyncSession) -> AugmentedPartSyncResult:
 
     if not unlinked_ids:
         return AugmentedPartSyncResult(
-            created=created, deleted=deleted, skipped=skipped
+            created=created, archived=archived, skipped=skipped
         )
 
     result = await db.execute(
@@ -359,4 +439,4 @@ async def sync(db: AsyncSession) -> AugmentedPartSyncResult:
 
     await db.commit()
 
-    return AugmentedPartSyncResult(created=created, deleted=deleted, skipped=skipped)
+    return AugmentedPartSyncResult(created=created, archived=archived, skipped=skipped)

app/services/auth.py

@@ -1,5 +1,7 @@
 """Token issuance / rotation shared by the JSON API and the HTMX UI."""
 
+from datetime import timedelta
+
 import jwt
 from sqlalchemy.ext.asyncio import AsyncSession
 
@@ -13,9 +15,16 @@ from app.crud import token as token_crud
 from app.models.user import User
 
 
-async def issue_token_pair(db: AsyncSession, user: User) -> tuple[str, str]:
+async def issue_token_pair(
-    """Create an access + refresh token and persist the refresh token for revocation."""
+    db: AsyncSession, user: User, access_expires_delta: timedelta | None = None
-    access = create_access_token(user.id)
+) -> tuple[str, str]:
+    """Create an access + refresh token and persist the refresh token for revocation.
+
+    ``access_expires_delta`` overrides the access token's default lifetime
+    (``settings.access_token_expire_minutes``) — used by the web login flow to
+    issue a longer-lived session cookie without affecting API bearer tokens.
+    """
+    access = create_access_token(user.id, access_expires_delta)
     refresh, payload = create_refresh_token(user.id)
     await token_crud.store(
         db,

app/services/fit_matching.py

@@ -76,9 +76,15 @@ def _static_clamp_match(ap_a: AttachmentPoint, ap_b: AttachmentPoint) -> bool:
 
 
 def attachment_points_match(ap_a: AttachmentPoint, ap_b: AttachmentPoint) -> bool:
-    """True if these two attachment points satisfy the fit-matching algorithm."""
+    """True if these two attachment points satisfy the fit-matching algorithm.
-    if ap_a.attachment_point_name_id == ap_b.attachment_point_name_id:
+
-        return False
+    A "Manual" attachment point (all four connections null) always returns
+    False here — it never participates in algorithmic matching, only in
+    fits a user asserts by hand.
+    """
+    # Disabled for now, the attachment_point_names should not be considered for the purposes of matching
+    # if ap_a.attachment_point_name_id == ap_b.attachment_point_name_id:
+    #     return False
     return (
         _iso_match(ap_a, ap_b)
         or _unc_match(ap_a, ap_b)

app/services/ldap.py

@@ -7,7 +7,6 @@ successful profile is provisioned locally on first sign-in.
 """
 
 import logging
-
 from dataclasses import dataclass
 
 from fastapi.concurrency import run_in_threadpool
@@ -43,7 +42,7 @@ def _bind_and_fetch(username: str, password: str) -> LdapIdentity | None:
     )
     try:
         conn = ldap3.Connection(server, user=user_dn, password=password, auto_bind=True)
-    except LDAPException as e:
+    except LDAPException:
         logging.exception("Failed to log into LDAP for user %s", user_dn)
         # Wrong credentials, unreachable server, or a refused bind.
         return None

app/services/scope_guide.py

@@ -29,13 +29,21 @@ async def build(
     scope_model_id: int | None = None,
     sub_part_type_id: int | None = None,
     fits_part_type_id: int | None = None,
+    only_exact: bool = False,
 ) -> dict:
     """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
-    fit for that part type.
+    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.
+
+    ``only_exact`` drops ``possible`` fits entirely, keeping only user-confirmed
+    ``exact`` ones — used for anonymous visitors, who shouldn't see unconfirmed
+    fit candidates.
 
     Shape::
 
@@ -77,6 +85,8 @@ async def build(
     fits = await fit_crud.list_for_parts(
         db, list(scope_ids), exclude_label=FitLabel.NONE
     )
+    if only_exact:
+        fits = [f for f in fits if f.label == FitLabel.EXACT]
 
     # Resolve the "other" (non-scope) part on each fit in a single batched query.
     other_ids: set[int] = set()
@@ -110,6 +120,8 @@ async def build(
         other = other_parts.get(other_id)
         if other is None or other.part_type_id == scope_type.id:
             continue  # scope-to-scope fits have no column
+        if other.archived:
+            continue  # archived parts don't show up as compatible accessories
         by_type = cell_map[scope_id].setdefault(other.part_type_id, {})
         existing = by_type.get(other.id)
         label = fit.label.value

app/services/scope_import.py

@@ -15,7 +15,6 @@ sub_part_type                       → SubPartType (get-or-create under "Scope"
 product_number                      → AugmentedPart.sku  (also the upsert / fit-lookup key)
 magnification_min / _max            → ScopeInfo
 objective_lens_diameter_mm          → ScopeInfo (descriptive only, no attachment point)
-objective_thread_pitch              → ScopeInfo
 ocular_outer_diameter_mm            → StaticConnection + AttachmentPoint "Ocular"
 objective_thread                    → ISO/UNC thread + AttachmentPoint "Objective"
 ocular_fit                          → Fit: scope's Ocular AP ↔ referenced part's first AP
@@ -26,8 +25,15 @@ and colour defaults to an "Unknown" PartColour, since the sheet carries neither.
 
 Fits are created with label POSSIBLE. The referenced part is looked up by
 ``product_number`` and linked to its first attachment point (chosen answer:
-"sole/first AP"). Both directions are resolved in a second pass so a fit between
+"sole/first AP"). If the referenced part has no attachment points at all, one
-two scopes in the same spreadsheet resolves regardless of row order.
+is created for it that mirrors the scope's attachment point's connection, so
+the fit has something valid to attach to. If the scope itself has no Ocular/
+Objective attachment point (no ocular_outer_diameter_mm/objective_thread, or
+an unparseable one), a Manual attachment point — no connection details, never
+picked up by the fit-matching algorithm — is created on the scope under the
+appropriate label so the fit still has somewhere to attach. Both directions
+are resolved in a second pass so a fit between two scopes in the same
+spreadsheet resolves regardless of row order.
 
 Bad rows (missing key, unknown fit target, unparseable thread) are skipped and
 reported per-row; good rows are committed as they go.
@@ -75,7 +81,6 @@ EXPECTED_COLUMNS = [
     "url",
     "ocular_outer_diameter_mm",
     "ocular_fit",
-    "objective_thread_pitch",
     "objective_thread",
     "objective_fit",
 ]
@@ -188,6 +193,54 @@ async def _thread_ap(db: AsyncSession, *, augmented_part_id: int, label: str, sp
     )
 
 
+async def _mirrored_ap(db: AsyncSession, *, augmented_part_id: int, scope_ap):
+    """Create an attachment point mirroring ``scope_ap``'s connection.
+
+    Used when the referenced part has no attachment point that already fits
+    the scope, so the explicit ocular_fit/objective_fit link has something
+    valid to attach to.
+    """
+    return await attachment_point_crud.create(
+        db,
+        augmented_part_id=augmented_part_id,
+        attachment_point_name_id=scope_ap.attachment_point_name_id,
+        iso_thread_id=scope_ap.iso_thread_id,
+        unc_thread_id=scope_ap.unc_thread_id,
+        clamp_connection_id=scope_ap.clamp_connection_id,
+        static_connection_id=scope_ap.static_connection_id,
+    )
+
+
+async def _ensure_manual_scope_ap(
+    db: AsyncSession,
+    *,
+    scope_ap,
+    augmented_part_id: int,
+    label: str,
+    messages: list[str],
+):
+    """Return ``scope_ap``, creating a Manual attachment point named ``label``
+    on the scope if it's None.
+
+    Used so an ocular_fit/objective_fit reference still has something to link
+    from even when the scope has no ocular_outer_diameter_mm/objective_thread
+    of its own.
+    """
+    if scope_ap is not None:
+        return scope_ap
+    apn = await apn_crud.get_by_name(db, label)
+    if apn is None:
+        apn = await apn_crud.create(db, name=label)
+    messages.append(
+        f"scope has no {label.lower()} attachment point; created a manual one to record the fit"
+    )
+    return await attachment_point_crud.create(
+        db,
+        augmented_part_id=augmented_part_id,
+        attachment_point_name_id=apn.id,
+    )
+
+
 async def _link_fit(
     db: AsyncSession,
     *,
@@ -197,12 +250,13 @@ async def _link_fit(
     scope_part_id: int,
     messages: list[str],
 ) -> None:
-    """Create a POSSIBLE fit from the scope's AP to the referenced part's first AP."""
+    """Create a POSSIBLE fit from the scope's AP to the referenced part's first AP.
-    if scope_ap is None:
+
-        messages.append(
+    ``scope_ap`` must already exist — callers ensure this via
-            f"{scope_role}_fit '{target_sku}' skipped: scope has no {scope_role} attachment point"
+    ``_ensure_manual_scope_ap`` before calling. If the referenced part has no
-        )
+    attachment points at all, one is created mirroring the scope's attachment
-        return
+    point's connection.
+    """
     target = await augmented_part_crud.get_by_sku(db, target_sku)
     if target is None:
         messages.append(f"{scope_role}_fit references unknown product '{target_sku}'")
@@ -211,12 +265,14 @@ async def _link_fit(
         messages.append(f"{scope_role}_fit '{target_sku}' skipped: self-reference")
         return
     target_aps = await attachment_point_crud.list_for_augmented_part(db, target.id)
-    if not target_aps:
+    if target_aps:
-        messages.append(
-            f"{scope_role}_fit '{target_sku}' skipped: referenced part has no attachment points"
-        )
-        return
         target_ap = target_aps[0]
+    else:
+        target_ap = await _mirrored_ap(db, augmented_part_id=target.id, scope_ap=scope_ap)
+        messages.append(
+            f"{scope_role}_fit '{target_sku}': referenced part had no attachment points, "
+            "created one matching the scope's attachment point"
+        )
     existing = await fit_crud.get_by_attachment_points(db, scope_ap, target_ap)
     if existing is None:
         await fit_crud.create(db, ap_a=scope_ap, ap_b=target_ap, label=FitLabel.POSSIBLE)
@@ -241,7 +297,21 @@ async def import_scopes(db: AsyncSession, rows: list[dict]) -> ScopeImportResult
         # +2: 1-based, and the header occupies row 1.
         row_num = offset + 2
         model_suffix = _text(row.get("model_suffix"))
-        default_product_number = f"{_text(row.get('model'))} {row.get('magnification_min')}-{row.get('magnification_max')}x{row.get('objective_lens_diameter_mm')} {row.get('model_suffix')}".strip()
+        model = _text(row.get("model"))
+        magnification_min = _num(row.get("magnification_min"))
+        magnification_max = _num(row.get("magnification_max"))
+        objective_lens_diameter_mm = _num(row.get("objective_lens_diameter_mm")) or _num(
+            row.get("objective_lens")
+        )
+
+        # Construct the model number
+        default_product_number = model
+        if magnification_min:
+            default_product_number += f" {magnification_min}-{magnification_max}"
+        if objective_lens_diameter_mm:
+            default_product_number += f"x{objective_lens_diameter_mm}"
+        if model_suffix:
+            default_product_number += f" {model_suffix}"
         product_number = _text(row.get("product_number")) or default_product_number
         result = ScopeImportRowResult(
             row=row_num, product_number=product_number, status="skipped", messages=[]
@@ -321,7 +391,6 @@ async def import_scopes(db: AsyncSession, rows: list[dict]) -> ScopeImportResult
                 magnification_max=_num(row.get("magnification_max")),
                 model_suffix=model_suffix,
                 objective_lens_diameter_mm=_num(row.get("objective_lens_diameter_mm")),
-                objective_thread_pitch=_text(row.get("objective_thread_pitch")),
             )
 
             # Rebuild attachment points from scratch so re-imports stay idempotent
@@ -368,6 +437,19 @@ async def import_scopes(db: AsyncSession, rows: list[dict]) -> ScopeImportResult
         try:
             ocular_fit = _text(row.get("ocular_fit"))
             if ocular_fit:
+                ocular_ap = await _ensure_manual_scope_ap(
+                    db,
+                    scope_ap=ocular_ap,
+                    augmented_part_id=part_id,
+                    label=OCULAR_LABEL,
+                    messages=result.messages,
+                )
+                fits = ocular_fit
+                if "/" in fits:
+                    fits = [x.strip() for x in fits.split("/")]
+                else:
+                    fits = [fits]
+                for ocular_fit in fits:
                     await _link_fit(
                         db,
                         scope_ap=ocular_ap,
@@ -378,6 +460,19 @@ async def import_scopes(db: AsyncSession, rows: list[dict]) -> ScopeImportResult
                     )
             objective_fit = _text(row.get("objective_fit"))
             if objective_fit:
+                objective_ap = await _ensure_manual_scope_ap(
+                    db,
+                    scope_ap=objective_ap,
+                    augmented_part_id=part_id,
+                    label=OBJECTIVE_LABEL,
+                    messages=result.messages,
+                )
+                fits = objective_fit
+                if "/" in fits:
+                    fits = [x.strip() for x in fits.split("/")]
+                else:
+                    fits = [fits]
+                for objective_fit in fits:
                     await _link_fit(
                         db,
                         scope_ap=objective_ap,

app/static/css/style.css

@@ -486,43 +486,89 @@ textarea:focus {
   margin-bottom: 0.5rem;
 }
 
-.ap-item-name { font-weight: 500; flex: 0 0 auto; }
+.ap-item-name {
-.ap-item-type { color: var(--muted); flex: 0 0 auto; font-size: 0.8rem; }
+  font-weight: 500;
-.ap-item-conn { flex: 1; }
+  flex: 0 0 auto;
-.ap-item .row-actions { margin-left: auto; }
-
-.fit-item {
-  padding: 0.5rem 0;
-  border-bottom: 1px solid var(--border);
-  font-size: 0.9rem;
 }
 
-.fit-item:last-of-type { border-bottom: none; }
+.ap-item-type {
-
+  color: var(--muted);
-.fit-item-main {
+  flex: 0 0 auto;
-  display: flex;
+  font-size: 0.8rem;
-  align-items: center;
-  gap: 0.5rem;
-  flex-wrap: wrap;
-  margin-bottom: 0.3rem;
 }
 
-.fit-item-arrow { color: var(--muted); }
+.ap-item-conn {
+  flex: 1;
+}
+
+.ap-item .row-actions {
+  margin-left: auto;
+}
+
+.fit-item-arrow {
+  color: var(--muted);
+  text-align: center;
+}
 
 .fit-badge {
-  font-size: 0.7rem;
   text-transform: uppercase;
-  letter-spacing: 0.03em;
-  padding: 0.1rem 0.5rem;
-  border-radius: 999px;
-  font-weight: 600;
 }
 
-.fit-badge-possible { background: var(--surface-2); color: var(--muted); }
+/* Fits table on the augmented part detail page */
-.fit-badge-exact { background: #16a34a26; color: #16a34a; }
+.fit-table select,
-.fit-badge-none { background: #dc262626; color: #dc2626; }
+.fit-table input[type="text"] {
+  min-width: 8rem;
+}
 
-.fit-notes { margin-top: 0.4rem; padding-left: 0.25rem; }
+.fit-table .fit-add-row {
+  background: var(--surface-2);
+}
+
+.fit-table .fit-add-row td {
+  vertical-align: top;
+}
+
+.fit-notes-cell {
+  min-width: 14rem;
+}
+
+.fit-badge-possible {
+  background: var(--surface-2);
+  color: var(--muted);
+}
+
+.fit-badge-exact {
+  background: #16a34a26;
+  color: #16a34a;
+}
+
+.fit-badge-none {
+  background: #dc262626;
+  color: #dc2626;
+}
+
+
+
+.fit-badge-possible a {
+  background: var(--surface-2);
+  color: var(--muted);
+}
+
+.fit-badge-exact a {
+  background: #16a34a26;
+  color: #16a34a;
+}
+
+.fit-badge-none a {
+  background: #dc262626;
+  color: #dc2626;
+}
+
+
+.fit-notes {
+  margin-top: 0.4rem;
+  padding-left: 0.25rem;
+}
 
 /* Scope guide: stacked matching-part entries within a fit column cell. */
 .fit-cell-item {
@@ -532,14 +578,33 @@ textarea:focus {
   font-size: 0.85rem;
   white-space: nowrap;
 }
-.fit-cell-item + .fit-cell-item { margin-top: 0.3rem; }
 
-.fit-note { font-size: 0.85rem; margin-bottom: 0.3rem; }
+.fit-cell-item+.fit-cell-item {
-.fit-note-meta { color: var(--muted); font-size: 0.75rem; }
+  margin-top: 0.3rem;
-.fit-note-body { margin: 0.1rem 0 0; }
+}
 
-.fit-note-form { margin-top: 0.3rem; }
+.fit-note {
-.fit-note-form input[type="text"] { flex: 1; min-width: 12rem; }
+  font-size: 0.85rem;
+  margin-bottom: 0.3rem;
+}
+
+.fit-note-meta {
+  color: var(--muted);
+  font-size: 0.75rem;
+}
+
+.fit-note-body {
+  margin: 0.1rem 0 0;
+}
+
+.fit-note-form {
+  margin-top: 0.3rem;
+}
+
+.fit-note-form input[type="text"] {
+  flex: 1;
+  min-width: 12rem;
+}
 
 /* Pagination */
 .pagination {
@@ -567,6 +632,18 @@ textarea:focus {
   margin-bottom: 0.75rem;
 }
 
+/* Bulk actions (e.g. delete-selected above a checkbox table) */
+.bulk-actions {
+  margin-bottom: 0.75rem;
+}
+
+.dashboard-sync-actions {
+  display: flex;
+  flex-wrap: wrap;
+  gap: 0.5rem;
+  margin-top: 0.75rem;
+}
+
 .filter-bar form {
   display: flex;
   flex-wrap: wrap;
@@ -665,7 +742,9 @@ textarea:focus {
   gap: 0.4rem;
 }
 
-.local-login-panel summary::-webkit-details-marker { display: none; }
+.local-login-panel summary::-webkit-details-marker {
+  display: none;
+}
 
 .local-login-panel summary::before {
   content: '›';

app/templates/_dashboard_sync_card.html

@@ -0,0 +1,32 @@
+<div id="dashboard-sync-card" class="card">
+  <h2>Data sync</h2>
+  <dl class="kv">
+    <dt>Scopes</dt>
+    <dd>{{ scope_count }}</dd>
+    <dt>Imported parts skipped</dt>
+    <dd>{% if can_sync_augmented_parts %}<a href="/admin/augmented-parts/skipped">{{ skipped_count }}</a>{% else %}{{ skipped_count }}{% endif %}</dd>
+  </dl>
+
+  {% if status_message %}<p class="muted">✅ {{ status_message }}</p>{% endif %}
+  {% if status_error %}<p class="muted">⚠️ {{ status_error }}</p>{% endif %}
+
+  {% if can_refresh_odata or can_sync_augmented_parts or can_recompute_fits %}
+  <div class="dashboard-sync-actions">
+    {% if can_refresh_odata %}
+    <form hx-post="/dashboard/refresh-odata" hx-target="#dashboard-sync-card" hx-swap="outerHTML">
+      <button type="submit" class="btn btn-primary">Refresh from OData</button>
+    </form>
+    {% endif %}
+    {% if can_sync_augmented_parts %}
+    <form hx-post="/dashboard/sync-augmented-parts" hx-target="#dashboard-sync-card" hx-swap="outerHTML">
+      <button type="submit" class="btn btn-primary">Sync augmented parts</button>
+    </form>
+    {% endif %}
+    {% if can_recompute_fits %}
+    <form hx-post="/dashboard/recompute-fits" hx-target="#dashboard-sync-card" hx-swap="outerHTML">
+      <button type="submit" class="btn btn-primary">Recompute possible fits</button>
+    </form>
+    {% endif %}
+  </div>
+  {% endif %}
+</div>

app/templates/_nav.html

@@ -1,11 +1,12 @@
-{% set su = user.is_superuser %}
+{% set su = user.is_superuser if user else False %}
-{% set p = user.permissions %}
+{% set p = user.permissions if user else [] %}
 {% set can_admin = su or "users:read" in p or "roles:read" in p or "audit_log:read" in p %}
 {% set can_connections = su or "clamp_connections:read" in p or "iso_threads:read" in p or "static_connections:read" in p or "unc_threads:read" in p %}
 {% set can_labels = su or "attachment_point_names:read" in p or "manufacturers:read" in p or "part_colours:read" in p or "part_types:read" in p or "scope_models:read" in p or "sub_part_types:read" in p %}
 {% set can_database = su or "imported_parts:read" in p or "augmented_parts:read" in p or "attachment_points:read" in p %}
 {% set can_parts = can_connections or can_labels or can_database %}
-{% set can_scopes = su or ("augmented_parts:read" in p and "fits:read" in p) %}
+{# /scopes is public, so anonymous visitors see the link too; logged-in users still need the read perms. #}
+{% set can_scopes = (not user) or su or ("augmented_parts:read" in p and "fits:read" in p) %}
 
 {# ── SVG icon sprite ───────────────────────────────────────────────────────── #}
 <svg xmlns="http://www.w3.org/2000/svg" style="display:none">
@@ -232,8 +233,12 @@
 
     <a href="/docs" target="_blank">{{ icon("book-open") }} API docs</a>
   </nav>
+  {% if user %}
   <form method="post" action="/logout" class="logout">
     <span class="muted">{{ user.email }}</span>
     <button type="submit" class="btn btn-ghost">Sign out</button>
   </form>
+  {% else %}
+  <a href="/login" class="btn btn-ghost">Sign in</a>
+  {% endif %}
 </header>

app/templates/_scopes_table_card.html

@@ -18,12 +18,14 @@
   {%- endif -%}
 {%- endmacro -%}
 <div id="scopes-table-card" class="card" style="overflow-x:auto">
+  {% include "admin/_pagination.html" %}
   <table class="table sortable">
     <thead>
       <tr>
+        {% if can_write_scopes %}
+          <th data-sort-off><input type="checkbox" id="scopes-select-all" title="Select all"></th>
+        {% endif %}
         <th>Scope</th>
-        <th>Manufacturer</th>
-        <th>Model</th>
         <th>Type</th>
         <th>Magnification</th>
         <th>Objective thread</th>
@@ -36,20 +38,19 @@
       {% for row in rows %}
         {% set scope = row.scope %}
         {% set info = row.info %}
-        <tr>
+        <tr id="scope-row-{{ scope.id }}">
+          {% if can_write_scopes %}
+            <td><input type="checkbox" name="scope_ids" value="{{ scope.id }}"></td>
+          {% endif %}
           <td>
             <strong><a href="/admin/augmented-parts/{{ scope.id }}">{{ scope.description }}</a></strong>
             {% if scope.sku %}<br><span class="muted">{{ scope.sku }}</span>{% endif %}
           </td>
-          <td>{{ scope.manufacturer.name if scope.manufacturer else "—" }}</td>
-          <td>{{ scope.scope_model.name if scope.scope_model else "—" }}</td>
           <td>{{ scope.sub_part_type.name if scope.sub_part_type else "—" }}</td>
           <td>{{ magnification(info) }}</td>
           <td>
             {% if row.objective_attachment_point %}
-              <code>{{ connection_spec(row.objective_attachment_point) }}</code>
+              {{ connection_spec(row.objective_attachment_point) }}
-            {% elif info and info.objective_thread_pitch %}
-              {{ info.objective_thread_pitch }}
             {% else %}<span class="muted">—</span>{% endif %}
           </td>
           {% for ct in column_types %}
@@ -58,8 +59,9 @@
               {% if entries %}
               {% for e in entries %}
                   <div class="fit-cell-item">
-                    <span class="fit-badge fit-badge-{{ e.label }}">{{ e.label | capitalize }}</span>
+                    <span class="fit-badge fit-badge-{{ e.label }}">
-                    <a href="/admin/augmented-parts/{{ e.part.id }}">{{ e.part.description | truncate(48) }}</a>
+                      <a href="/admin/augmented-parts/{{ e.part.id }}" title="{{ e.part.description | truncate(48) }}">{{ e.part.sku }}</a>
+                    </span>
                   </div>
                 {% endfor %}
               {% else %}
@@ -70,7 +72,7 @@
         </tr>
       {% else %}
         <tr>
-          <td colspan="{{ 6 + column_types | length }}" class="muted">
+          <td colspan="{{ 6 + column_types | length + (1 if can_write_scopes else 0) }}" class="muted">
             No scopes match your search.
           </td>
         </tr>

app/templates/admin/_attachment_point_connection_select.html

@@ -48,4 +48,6 @@
     {% endfor %}
   </select>
 </label>
+{% elif connection_type == "manual" %}
+<p>No connection details — fits for this attachment point must be asserted manually.</p>
 {% endif %}

app/templates/admin/_attachment_point_row.html

@@ -15,7 +15,7 @@
   {% set conn_type = "Static Connection" %}
   {% set conn_label = ap.static_connection.od_mm ~ " mm OD" %}
 {% else %}
-  {% set conn_type = "—" %}
+  {% set conn_type = "Manual" %}
   {% set conn_label = "—" %}
 {% endif %}
 <tr id="attachment-point-{{ ap.id }}">

app/templates/admin/_attachment_point_row_edit.html

@@ -2,7 +2,7 @@
 {% elif ap.unc_thread_id %}{% set current_type = "unc_thread" %}
 {% elif ap.clamp_connection_id %}{% set current_type = "clamp_connection" %}
 {% elif ap.static_connection_id %}{% set current_type = "static_connection" %}
-{% else %}{% set current_type = "" %}{% endif %}
+{% else %}{% set current_type = "manual" %}{% endif %}
 <tr id="attachment-point-{{ ap.id }}">
   <td colspan="6">
     <form hx-post="/admin/attachment-points/{{ ap.id }}"
@@ -41,6 +41,7 @@
           <option value="unc_thread" {% if current_type == "unc_thread" %}selected{% endif %}>UNC Thread</option>
           <option value="clamp_connection" {% if current_type == "clamp_connection" %}selected{% endif %}>Clamp Connection</option>
           <option value="static_connection" {% if current_type == "static_connection" %}selected{% endif %}>Static Connection</option>
+          <option value="manual" {% if current_type == "manual" %}selected{% endif %}>Manual</option>
         </select>
       </label>
       {# Render the current connection select pre-filled #}

app/templates/admin/_augmented_part_ap_row.html

@@ -15,7 +15,7 @@
   {% set conn_type = "Static" %}
   {% set conn_label = apoint.static_connection.od_mm ~ " mm OD" %}
 {% else %}
-  {% set conn_type = "—" %}
+  {% set conn_type = "Manual" %}
   {% set conn_label = "—" %}
 {% endif %}
 <div id="augmented-part-ap-{{ apoint.id }}" class="ap-item">

app/templates/admin/_augmented_part_ap_row_edit.html

@@ -2,7 +2,7 @@
 {% elif apoint.unc_thread_id %}{% set current_type = "unc_thread" %}
 {% elif apoint.clamp_connection_id %}{% set current_type = "clamp_connection" %}
 {% elif apoint.static_connection_id %}{% set current_type = "static_connection" %}
-{% else %}{% set current_type = "" %}{% endif %}
+{% else %}{% set current_type = "manual" %}{% endif %}
 {# _attachment_point_connection_select.html expects `ap` for pre-selection #}
 {% set ap = apoint %}
 {% set connection_type = current_type %}
@@ -29,6 +29,7 @@
         <option value="unc_thread" {{ "selected" if current_type == "unc_thread" }}>UNC Thread</option>
         <option value="clamp_connection" {{ "selected" if current_type == "clamp_connection" }}>Clamp Connection</option>
         <option value="static_connection" {{ "selected" if current_type == "static_connection" }}>Static Connection</option>
+        <option value="manual" {{ "selected" if current_type == "manual" }}>Manual</option>
       </select>
     </label>
     <div id="edit-ap-{{ apoint.id }}-connection-options">

app/templates/admin/_augmented_part_attachment_points.html

@@ -31,6 +31,7 @@
         <option value="unc_thread">UNC Thread</option>
         <option value="clamp_connection">Clamp Connection</option>
         <option value="static_connection">Static Connection</option>
+        <option value="manual">Manual</option>
       </select>
     </label>
     <div id="ap-{{ augmented_part_id }}-add-connection-options"></div>

app/templates/admin/_augmented_part_fit_row.html

@@ -8,46 +8,22 @@
   {%- elif ap.static_connection -%}
     Static {{ ap.static_connection.od_mm }} mm OD
   {%- else -%}
-    —
+    Manual
   {%- endif -%}
 {%- endmacro %}
 
-<div id="fit-{{ fit.id }}" class="fit-item">
+<tr id="fit-{{ fit.id }}" class="fit-row">
-  <div class="fit-item-main">
+  <td><span class="fit-badge fit-badge-{{ fit.label.value }}">{{ fit.label.value | capitalize }}</span></td>
-    <span class="fit-badge fit-badge-{{ fit.label.value }}">{{ fit.label.value | capitalize }}</span>
+  <td>
-    <span class="fit-item-side"><strong>{{ own_ap.attachment_point_name.name }}</strong> ({{ conn_desc(own_ap) }})</span>
+    <strong>{{ own_ap.attachment_point_name.name }}</strong>
-    <span class="fit-item-arrow">↔</span>
+    <div class="muted" style="font-size:0.8rem">{{ conn_desc(own_ap) }}</div>
-    <span class="fit-item-side">{{ other_part.description if other_part else "Unknown part" }} — <strong>{{ other_ap.attachment_point_name.name }}</strong> ({{ conn_desc(other_ap) }})</span>
+  </td>
-  </div>
+  <td class="fit-item-arrow">↔</td>
-  {% if can_write %}
+  <td>
-  <div class="row-actions">
+    <a href="/admin/augmented-parts/{{ other_part.id }}" title="{{ other_part.description }}">{{ other_part.sku or other_part.description }}</a>
-    {% if fit.label.value != "exact" %}
+    <div class="muted" style="font-size:0.8rem">{{ other_part.part_type.name }}</div>
-      <button class="btn btn-ghost"
+  </td>
-              hx-post="/admin/augmented-parts/{{ augmented_part_id }}/fits/{{ fit.id }}/label"
+  <td class="fit-notes-cell">
-              hx-vals='{"label": "exact"}'
-              hx-target="#fit-{{ fit.id }}" hx-swap="outerHTML">Mark Exact</button>
-    {% endif %}
-    {% if fit.label.value != "possible" %}
-      <button class="btn btn-ghost"
-              hx-post="/admin/augmented-parts/{{ augmented_part_id }}/fits/{{ fit.id }}/label"
-              hx-vals='{"label": "possible"}'
-              hx-target="#fit-{{ fit.id }}" hx-swap="outerHTML">Mark Possible</button>
-    {% endif %}
-    {% if fit.label.value != "none" %}
-      <button class="btn btn-ghost"
-              hx-post="/admin/augmented-parts/{{ augmented_part_id }}/fits/{{ fit.id }}/label"
-              hx-vals='{"label": "none"}'
-              hx-target="#fit-{{ fit.id }}" hx-swap="outerHTML">Mark Impossible</button>
-    {% endif %}
-    <button class="btn btn-danger"
-            hx-delete="/admin/augmented-parts/{{ augmented_part_id }}/fits/{{ fit.id }}"
-            hx-target="#ap-{{ augmented_part_id }}-fits"
-            hx-swap="outerHTML"
-            hx-confirm="Delete this fit?">Delete</button>
-  </div>
-  {% endif %}
-
-  <div class="fit-notes">
     {% for note in fit.notes %}
       <div class="fit-note">
         <span class="fit-note-meta">{{ note.username }} · {{ note.created_at.strftime("%Y-%m-%d %H:%M") }}</span>
@@ -59,8 +35,37 @@
           hx-target="#fit-{{ fit.id }}" hx-swap="outerHTML"
           class="inline-edit fit-note-form">
       <input type="text" name="body" placeholder="Add a note…" required>
-      <button type="submit" class="btn btn-ghost">Add note</button>
+      <button type="submit" class="btn btn-ghost btn-sm">Add note</button>
     </form>
     {% endif %}
+  </td>
+  {% if can_write %}
+  <td>
+    <div class="row-actions">
+      {% if fit.label.value != "exact" %}
+        <button class="btn btn-ghost btn-sm"
+                hx-post="/admin/augmented-parts/{{ augmented_part_id }}/fits/{{ fit.id }}/label"
+                hx-vals='{"label": "exact"}'
+                hx-target="#fit-{{ fit.id }}" hx-swap="outerHTML">Mark Exact</button>
+      {% endif %}
+      {% if fit.label.value != "possible" %}
+        <button class="btn btn-ghost btn-sm"
+                hx-post="/admin/augmented-parts/{{ augmented_part_id }}/fits/{{ fit.id }}/label"
+                hx-vals='{"label": "possible"}'
+                hx-target="#fit-{{ fit.id }}" hx-swap="outerHTML">Mark Possible</button>
+      {% endif %}
+      {% if fit.label.value != "none" %}
+        <button class="btn btn-ghost btn-sm"
+                hx-post="/admin/augmented-parts/{{ augmented_part_id }}/fits/{{ fit.id }}/label"
+                hx-vals='{"label": "none"}'
+                hx-target="#fit-{{ fit.id }}" hx-swap="outerHTML">Mark Impossible</button>
+      {% endif %}
+      <button class="btn btn-danger btn-sm"
+              hx-delete="/admin/augmented-parts/{{ augmented_part_id }}/fits/{{ fit.id }}"
+              hx-target="#ap-{{ augmented_part_id }}-fits"
+              hx-swap="outerHTML"
+              hx-confirm="Delete this fit?">Delete</button>
     </div>
-</div>
+  </td>
+  {% endif %}
+</tr>

app/templates/admin/_augmented_part_fits.html

@@ -4,36 +4,46 @@
   {% if message %}<p class="alert alert-success">{{ message }}</p>{% endif %}
   {% if error %}<p class="alert alert-error">{{ error }}</p>{% endif %}
 
-  {% for view in fit_views %}
+  {% set add_form_id = "ap-" ~ augmented_part_id ~ "-fit-add-form" %}
-    {% with fit=view.fit, own_ap=view.own_ap, other_ap=view.other_ap, other_part=view.other_part %}
-      {% include "admin/_augmented_part_fit_row.html" %}
-    {% endwith %}
-  {% else %}
-    <p class="muted" style="font-size:0.85rem;margin:0 0 0.5rem">No fits recorded yet.</p>
-  {% endfor %}
-
   {% if can_write %}
-  <div class="row-actions" style="margin:0.5rem 0">
+  <form id="{{ add_form_id }}"
-    <form hx-post="/admin/augmented-parts/{{ augmented_part_id }}/fits/recompute"
+        hx-post="/admin/augmented-parts/{{ augmented_part_id }}/fits"
-          hx-target="#ap-{{ augmented_part_id }}-fits" hx-swap="outerHTML">
-      <button type="submit" class="btn btn-ghost">Recompute possible fits</button>
-    </form>
-  </div>
-
-  <form hx-post="/admin/augmented-parts/{{ augmented_part_id }}/fits"
         hx-target="#ap-{{ augmented_part_id }}-fits"
-        hx-swap="outerHTML"
+        hx-swap="outerHTML"></form>
-        class="inline-edit" style="margin-top:0.5rem">
+  {% endif %}
-    <label>This part's point
+
-      <select name="own_attachment_point_id" required>
+  <table class="table fit-table">
+    <thead>
+      <tr>
+        <th>Type</th>
+        <th>This part's point</th>
+        <th></th>
+        <th>Fits with part</th>
+        <th>Notes</th>
+        {% if can_write %}<th></th>{% endif %}
+      </tr>
+    </thead>
+    <tbody>
+      {% if can_write %}
+      <tr class="fit-add-row">
+        <td>
+          <select form="{{ add_form_id }}" name="label">
+            <option value="exact" selected>Exact</option>
+            <option value="possible">Possible</option>
+            <option value="none">None</option>
+          </select>
+        </td>
+        <td>
+          <select form="{{ add_form_id }}" name="own_attachment_point_id" required>
             <option value="">— select —</option>
             {% for p in own_points %}
               <option value="{{ p.id }}">{{ p.attachment_point_name.name }}</option>
             {% endfor %}
           </select>
-    </label>
+        </td>
-    <label>Fits with part
+        <td class="fit-item-arrow">↔</td>
-      <select name="other_augmented_part_id" required
+        <td>
+          <select form="{{ add_form_id }}" name="other_augmented_part_id" required
                   hx-get="/admin/augmented-parts/{{ augmented_part_id }}/fits/other-points"
                   hx-target="#ap-{{ augmented_part_id }}-fit-other-points"
                   hx-trigger="change"
@@ -43,19 +53,35 @@
               <option value="{{ p.id }}">{{ p.description | truncate(40) }}</option>
             {% endfor %}
           </select>
-    </label>
           <div id="ap-{{ augmented_part_id }}-fit-other-points"></div>
-    <label>Fit type
+        </td>
-      <select name="label">
+        <td>
-        <option value="exact" selected>Exact</option>
+          <input form="{{ add_form_id }}" type="text" name="note" placeholder="Add a note…">
-        <option value="possible">Possible</option>
+        </td>
-        <option value="none">None</option>
+        <td>
-      </select>
+          <button type="submit" form="{{ add_form_id }}" class="btn btn-primary btn-sm">Add fit</button>
-    </label>
+        </td>
-    <label>Note (optional)
+      </tr>
-      <input type="text" name="note" placeholder="Add a note…">
+      {% endif %}
-    </label>
+
-    <button type="submit" class="btn btn-primary">Add fit</button>
+      {% for view in fit_views %}
+        {% with fit=view.fit, own_ap=view.own_ap, other_ap=view.other_ap, other_part=view.other_part %}
+          {% include "admin/_augmented_part_fit_row.html" %}
+        {% endwith %}
+      {% else %}
+        <tr>
+          <td colspan="{{ 6 if can_write else 5 }}" class="muted" style="font-size:0.85rem">No fits recorded yet.</td>
+        </tr>
+      {% endfor %}
+    </tbody>
+  </table>
+
+  {% if can_write %}
+  <div class="row-actions" style="margin:0.5rem 0">
+    <form hx-post="/admin/augmented-parts/{{ augmented_part_id }}/fits/recompute"
+          hx-target="#ap-{{ augmented_part_id }}-fits" hx-swap="outerHTML">
+      <button type="submit" class="btn btn-ghost">Recompute possible fits</button>
     </form>
+  </div>
   {% endif %}
 </div>

app/templates/admin/_augmented_part_row.html

@@ -7,11 +7,24 @@
   <td>{{ ap.scope_model.name if ap.scope_model else '—' }}</td>
   <td>{{ ap.part_type.name }}</td>
   <td>{{ ap.sub_part_type.name if ap.sub_part_type else '—' }}</td>
+  <td>{% if ap.archived %}<span class="tag">Archived</span>{% else %}—{% endif %}</td>
   <td class="row-actions">
     <button class="btn btn-ghost"
             hx-get="/admin/augmented-parts/{{ ap.id }}/edit"
             hx-target="#augmented-part-{{ ap.id }}"
             hx-swap="outerHTML">Edit</button>
+    {% if ap.archived %}
+    <button class="btn btn-ghost"
+            hx-post="/admin/augmented-parts/{{ ap.id }}/unarchive"
+            hx-target="#augmented-part-{{ ap.id }}"
+            hx-swap="outerHTML">Unarchive</button>
+    {% else %}
+    <button class="btn btn-ghost"
+            hx-post="/admin/augmented-parts/{{ ap.id }}/archive"
+            hx-target="#augmented-part-{{ ap.id }}"
+            hx-swap="outerHTML"
+            hx-confirm="Archive {{ ap.sku or ap.description }}?">Archive</button>
+    {% endif %}
     <button class="btn btn-danger"
             hx-delete="/admin/augmented-parts/{{ ap.id }}"
             hx-target="#augmented-part-{{ ap.id }}"

app/templates/admin/_augmented_part_row_edit.html

@@ -1,5 +1,5 @@
 <tr id="augmented-part-{{ ap.id }}">
-  <td colspan="9">
+  <td colspan="10">
     <form id="augmented-part-edit-form-{{ ap.id }}"
           hx-post="/admin/augmented-parts/{{ ap.id }}"
           hx-target="#augmented-part-{{ ap.id }}"

app/templates/admin/_augmented_parts_table.html

@@ -12,7 +12,7 @@
         hx-swap="outerHTML">
     <input type="hidden" name="sort" value="{{ sort }}">
     <input type="hidden" name="dir" value="{{ dir }}">
-    <input type="text" name="description" placeholder="Filter by description…" value="{{ filter_description or '' }}">
+    <input type="text" name="description" placeholder="Filter by description or SKU…" value="{{ filter_description or '' }}">
     <select name="colour_id">
       <option value="">All colours</option>
       {% for pc in part_colours %}
@@ -43,6 +43,7 @@
         <option value="{{ spt.id }}" {{ "selected" if filter_sub_part_type_id and filter_sub_part_type_id|int == spt.id }}>{{ spt.name }}</option>
       {% endfor %}
     </select>
+    <label><input type="checkbox" name="include_archived" {{ "checked" if include_archived }}> Show archived</label>
   </form>
 </div>
 

app/templates/admin/_augmented_parts_table_card.html

@@ -12,6 +12,7 @@
         {{ sort_th("scope_model", "Model", sort_headers) }}
         {{ sort_th("part_type", "Type", sort_headers) }}
         {{ sort_th("sub_part_type", "Sub Type", sort_headers) }}
+        <th>Archived</th>
         <th></th>
       </tr>
     </thead>
@@ -19,7 +20,7 @@
       {% for ap in augmented_parts %}
         {% include "admin/_augmented_part_row.html" %}
       {% else %}
-        <tr><td colspan="9" class="muted">No augmented parts recorded.</td></tr>
+        <tr><td colspan="10" class="muted">No augmented parts recorded.</td></tr>
       {% endfor %}
     </tbody>
   </table>

app/templates/admin/_fit_other_points_select.html

@@ -1,12 +1,12 @@
 {% if points %}
-<select name="other_attachment_point_id" required>
+<select name="other_attachment_point_id" form="{{ form_id }}" required>
   <option value="">— select —</option>
   {% for p in points %}
     <option value="{{ p.id }}">{{ p.attachment_point_name.name }}</option>
   {% endfor %}
 </select>
 {% else %}
-<input type="hidden" name="other_attachment_point_id" value="">
+<input type="hidden" name="other_attachment_point_id" value="" form="{{ form_id }}">
 <p class="muted" style="font-size:0.85rem;margin:0">
   This part has no attachment points yet — one matching yours will be created automatically.
 </p>

app/templates/admin/_imported_parts_table.html

@@ -13,7 +13,7 @@
           hx-swap="outerHTML">
       <input type="hidden" name="sort" value="{{ sort }}">
       <input type="hidden" name="dir" value="{{ dir }}">
-      <input type="text" name="description" placeholder="Filter by description…" value="{{ filter_description or '' }}">
+      <input type="text" name="description" placeholder="Filter by description or SKU…" value="{{ filter_description or '' }}">
     </form>
   </div>
 

app/templates/admin/attachment_points.html

@@ -38,6 +38,7 @@
           <option value="unc_thread">UNC Thread</option>
           <option value="clamp_connection">Clamp Connection</option>
           <option value="static_connection">Static Connection</option>
+          <option value="manual">Manual</option>
         </select>
       </label>
       <div id="add-connection-options"></div>

app/templates/admin/augmented_part_detail.html

@@ -6,11 +6,12 @@
 <main class="container">
   <p><a href="/admin/augmented-parts" class="btn btn-ghost">← Augmented Parts</a></p>
 
-  <h1>{{ ap.description }}</h1>
+  <h1> {% if ap.sku %} {{ ap.sku }} {% else %} {{ ap.description }} {% endif %} </h1>
   <p class="muted">
     <span class="tag">{{ ap.part_type.name }}</span>
     {% if ap.sub_part_type %}<span class="tag">{{ ap.sub_part_type.name }}</span>{% endif %}
-    {% if ap.sku %}· SKU {{ ap.sku }}{% endif %}
+    {% if ap.archived %}<span class="tag">Archived</span>{% endif %}
+    {% if ap.description %}{{ ap.description }}{% endif %}
   </p>
 
   <div class="grid">
@@ -50,7 +51,6 @@
             {{ scope_info.objective_lens_diameter_mm }} mm
           {% else %}—{% endif %}
         </dd>
-        <dt>Objective thread</dt><dd>{{ scope_info.objective_thread_pitch or "—" }}</dd>
       </dl>
     </div>
     {% endif %}

app/templates/admin/augmented_parts.html

@@ -56,7 +56,7 @@
 
   <div class="card">
     <h2>Sync with Imported Parts</h2>
-    <p class="muted">Creates Augmented Part entries for new Imported Parts, and removes entries for deleted Imported Parts.</p>
+    <p class="muted">Creates Augmented Part entries for new Imported Parts, and archives entries for deleted Imported Parts.</p>
     <div class="row-actions">
       <form hx-post="/admin/augmented-parts/sync" hx-target="#augmented-part-list" hx-swap="outerHTML">
         <button type="submit" class="btn btn-primary">Sync now</button>

app/templates/dashboard.html

@@ -35,6 +35,8 @@
         <li><a href="/redoc" target="_blank">API reference (ReDoc)</a></li>
       </ul>
     </div>
+
+    {% include "_dashboard_sync_card.html" %}
   </div>
 </main>
 {% endblock %}

app/templates/scopes.html

@@ -7,7 +7,8 @@
   <p class="muted">
     Every scope on record, with the accessories that fit it. Each part-type
     column lists matching parts; <span class="fit-badge fit-badge-exact">Exact</span>
-    is a confirmed fit, <span class="fit-badge fit-badge-possible">Possible</span> a candidate.
+    is a confirmed fit{% if not only_exact %}, <span class="fit-badge fit-badge-possible">Possible</span> a candidate{% endif %}.
+    {% if only_exact %}<a href="/login">Sign in</a> to also see unconfirmed candidate fits.{% endif %}
   </p>
 
   <div class="filter-bar">
@@ -15,7 +16,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>
@@ -44,6 +45,50 @@
     </form>
   </div>
 
+  {% if can_write_scopes %}
+    <div class="bulk-actions">
+      <button id="delete-selected-scopes-btn" type="button" class="btn btn-danger" disabled
+              hx-post="/scopes/bulk-delete"
+              hx-include="[name='scope_ids']:checked, [name='description'], [name='manufacturer_id'], [name='scope_model_id'], [name='sub_part_type_id'], [name='fits_part_type_id']"
+              hx-target="#scopes-table-card"
+              hx-swap="outerHTML"
+              hx-confirm="Delete selected scopes?">Delete selected</button>
+    </div>
+  {% endif %}
+
   {% include "_scopes_table_card.html" %}
 </main>
+
+{% if can_write_scopes %}
+<script>
+(function () {
+  function updateDeleteButton() {
+    var count = document.querySelectorAll('input[name="scope_ids"]:checked').length;
+    var btn = document.getElementById('delete-selected-scopes-btn');
+    if (!btn) return;
+    btn.textContent = count > 0 ? 'Delete selected (' + count + ')' : 'Delete selected';
+    btn.disabled = count === 0;
+    btn.setAttribute('hx-confirm', 'Delete ' + count + ' selected scope' + (count === 1 ? '' : 's') + '?');
+  }
+
+  document.addEventListener('change', function (e) {
+    if (!e.target) return;
+
+    if (e.target.id === 'scopes-select-all') {
+      document.querySelectorAll('input[name="scope_ids"]').forEach(function (cb) {
+        cb.checked = e.target.checked;
+      });
+    }
+
+    if (e.target.id === 'scopes-select-all' || e.target.name === 'scope_ids') {
+      updateDeleteButton();
+    }
+  });
+
+  // Re-run after every HTMX swap (filter results, post-delete) so the button
+  // state reflects the freshly rendered checkboxes, which start unchecked.
+  document.addEventListener('htmx:afterSwap', updateDeleteButton);
+}());
+</script>
+{% endif %}
 {% endblock %}

app/web/admin.py

@@ -1,8 +1,5 @@
 """HTMX-driven admin dashboard for users, roles, and thread standards."""
 
-import math
-import urllib.parse
-
 from fastapi import (
     APIRouter,
     Depends,
@@ -17,9 +14,14 @@ from fastapi.responses import HTMLResponse, RedirectResponse, Response
 from sqlalchemy.ext.asyncio import AsyncSession
 
 from app.core.database import AsyncSessionLocal, get_db
+from app.crud import attachment_point as attachment_point_crud
+from app.crud import attachment_point_name as apn_crud
 from app.crud import audit as audit_crud
 from app.crud import augmented_part as augmented_part_crud
 from app.crud import augmented_part_note as augmented_part_note_crud
+from app.crud import clamp_connection as clamp_connection_crud
+from app.crud import fit as fit_crud
+from app.crud import fit_note as fit_note_crud
 from app.crud import imported_part as imported_part_crud
 from app.crud import imported_part_config as imported_part_config_crud
 from app.crud import iso_thread as iso_thread_crud
@@ -29,11 +31,6 @@ from app.crud import part_type as part_type_crud
 from app.crud import role as role_crud
 from app.crud import scope_info as scope_info_crud
 from app.crud import scope_model as scope_model_crud
-from app.crud import attachment_point as attachment_point_crud
-from app.crud import attachment_point_name as apn_crud
-from app.crud import clamp_connection as clamp_connection_crud
-from app.crud import fit as fit_crud
-from app.crud import fit_note as fit_note_crud
 from app.crud import static_connection as static_connection_crud
 from app.crud import sub_part_type as sub_part_type_crud
 from app.crud import unc_thread as unc_thread_crud
@@ -48,6 +45,8 @@ from app.services import fit_matching as fit_matching_service
 from app.services import jobs
 from app.services import odata as odata_service
 from app.services import scope_import as scope_import_service
+from app.web.pagination import paginate as _paginate
+from app.web.pagination import query_url as _query_url
 from app.web.templates import templates
 
 router = APIRouter(prefix="/admin", include_in_schema=False)
@@ -55,42 +54,6 @@ router = APIRouter(prefix="/admin", include_in_schema=False)
 _PAGE_SIZE = 50
 
 
-def _query_url(base_path: str, params: dict) -> str:
-    clean = {k: v for k, v in params.items() if v not in (None, "", 0)}
-    return f"{base_path}?{urllib.parse.urlencode(clean)}"
-
-
-def _paginate(
-    base_path: str,
-    filter_params: dict,
-    page: int,
-    page_size: int,
-    total: int,
-    target_id: str,
-) -> dict:
-    total_pages = max(1, math.ceil(total / page_size))
-    page = max(1, min(page, total_pages))
-    skip = (page - 1) * page_size
-
-    def _url(p: int) -> str:
-        return _query_url(
-            base_path, {**filter_params, "page": p, "page_size": page_size}
-        )
-
-    return {
-        "page": page,
-        "page_size": page_size,
-        "total": total,
-        "total_pages": total_pages,
-        "skip": skip,
-        "showing_from": skip + 1 if total > 0 else 0,
-        "showing_to": min(skip + page_size, total),
-        "prev_url": _url(page - 1) if page > 1 else None,
-        "next_url": _url(page + 1) if page < total_pages else None,
-        "target_id": target_id,
-    }
-
-
 def _sort_headers(
     base_path: str,
     filter_params: dict,
@@ -1697,6 +1660,7 @@ async def _default_augmented_parts_table_ctx(db: AsyncSession) -> dict:
         ),
         "sort": "description",
         "dir": "asc",
+        "include_archived": False,
     }
 
 
@@ -1705,17 +1669,24 @@ async def augmented_parts_page(
     request: Request,
     sort: str = "description",
     dir: str = "asc",
+    include_archived: bool = False,
     db: AsyncSession = Depends(get_db),
     user: User = Depends(web_require("augmented_parts:read")),
 ):
     ctx = await _augmented_part_form_ctx(db)
-    total = await augmented_part_crud.count_filtered(db)
+    total = await augmented_part_crud.count_filtered(
+        db, include_archived=include_archived
+    )
     ctx["augmented_parts"] = await augmented_part_crud.list_filtered(
-        db, sort=sort, direction=dir, limit=_PAGE_SIZE
+        db,
+        sort=sort,
+        direction=dir,
+        include_archived=include_archived,
+        limit=_PAGE_SIZE,
     )
     ctx["pagination"] = _paginate(
         "/admin/augmented-parts/filter",
-        {"sort": sort, "dir": dir},
+        {"sort": sort, "dir": dir, "include_archived": include_archived},
         1,
         _PAGE_SIZE,
         total,
@@ -1723,7 +1694,7 @@ async def augmented_parts_page(
     )
     ctx["sort_headers"] = _sort_headers(
         "/admin/augmented-parts/filter",
-        {},
+        {"include_archived": include_archived},
         _AUGMENTED_PARTS_SORT_COLUMNS,
         sort,
         dir,
@@ -1731,6 +1702,7 @@ async def augmented_parts_page(
     )
     ctx["sort"] = sort
     ctx["dir"] = dir
+    ctx["include_archived"] = include_archived
     ctx["user"] = user
     return templates.TemplateResponse(request, "admin/augmented_parts.html", ctx)
 
@@ -1801,7 +1773,7 @@ async def sync_augmented_parts(
                 resource_type="AugmentedPart",
                 resource_id=None,
                 description=(
-                    f"Sync: {result.created} created, {result.deleted} deleted, "
+                    f"Sync: {result.created} created, {result.archived} archived, "
                     f"{result.skipped} skipped (no type match)"
                 ),
             )
@@ -1809,7 +1781,7 @@ async def sync_augmented_parts(
                 job_id,
                 message=(
                     f"Sync complete: {result.created} created, "
-                    f"{result.deleted} deleted"
+                    f"{result.archived} archived"
                     + (
                         f", {result.skipped} skipped (type not recognised)"
                         if result.skipped
@@ -1883,6 +1855,7 @@ async def filter_augmented_parts(
     scope_model_id: str = "",
     part_type_id: str = "",
     sub_part_type_id: str = "",
+    include_archived: bool = False,
     sort: str = "description",
     dir: str = "asc",
     page: int = 1,
@@ -1903,6 +1876,7 @@ async def filter_augmented_parts(
         scope_model_id=scope_model_id_int,
         part_type_id=part_type_id_int,
         sub_part_type_id=sub_part_type_id_int,
+        include_archived=include_archived,
     )
     filter_params = {
         "description": description,
@@ -1911,6 +1885,7 @@ async def filter_augmented_parts(
         "scope_model_id": scope_model_id,
         "part_type_id": part_type_id,
         "sub_part_type_id": sub_part_type_id,
+        "include_archived": include_archived,
     }
     pagination = _paginate(
         "/admin/augmented-parts/filter",
@@ -1928,6 +1903,7 @@ async def filter_augmented_parts(
         scope_model_id=scope_model_id_int,
         part_type_id=part_type_id_int,
         sub_part_type_id=sub_part_type_id_int,
+        include_archived=include_archived,
         sort=sort,
         direction=dir,
         skip=pagination["skip"],
@@ -1950,6 +1926,7 @@ async def filter_augmented_parts(
             "sort_headers": sort_headers,
             "sort": sort,
             "dir": dir,
+            "include_archived": include_archived,
         },
     )
 
@@ -2247,6 +2224,60 @@ async def delete_augmented_part(
     return Response(status_code=status.HTTP_200_OK)
 
 
+async def _set_augmented_part_archived(
+    request: Request,
+    augmented_part_id: int,
+    archived: bool,
+    db: AsyncSession,
+    actor: User,
+):
+    ap = await augmented_part_crud.get(db, augmented_part_id)
+    if ap is None:
+        raise HTTPException(status_code=404, detail="Augmented part not found")
+    ap = await augmented_part_crud.set_archived(db, ap, archived)
+    await audit_crud.log(
+        db,
+        actor=actor,
+        action="update",
+        resource_type="AugmentedPart",
+        resource_id=augmented_part_id,
+        description=audit_crud.describe_update(
+            "AugmentedPart", augmented_part_id, {"archived": archived}
+        ),
+    )
+    return templates.TemplateResponse(
+        request, "admin/_augmented_part_row.html", {"ap": ap}
+    )
+
+
+@router.post(
+    "/augmented-parts/{augmented_part_id}/archive", response_class=HTMLResponse
+)
+async def archive_augmented_part(
+    request: Request,
+    augmented_part_id: int,
+    db: AsyncSession = Depends(get_db),
+    actor: User = Depends(web_require("augmented_parts:write")),
+):
+    return await _set_augmented_part_archived(
+        request, augmented_part_id, True, db, actor
+    )
+
+
+@router.post(
+    "/augmented-parts/{augmented_part_id}/unarchive", response_class=HTMLResponse
+)
+async def unarchive_augmented_part(
+    request: Request,
+    augmented_part_id: int,
+    db: AsyncSession = Depends(get_db),
+    actor: User = Depends(web_require("augmented_parts:write")),
+):
+    return await _set_augmented_part_archived(
+        request, augmented_part_id, False, db, actor
+    )
+
+
 def _can_write_attachment_points(user: User) -> bool:
     return user.is_superuser or "attachment_points:write" in user.permissions
 
@@ -2356,7 +2387,7 @@ async def edit_augmented_part_attachment_point(
     elif apoint.static_connection_id:
         ctx["connection_type"] = "static_connection"
     else:
-        ctx["connection_type"] = ""
+        ctx["connection_type"] = "manual"
     return templates.TemplateResponse(
         request, "admin/_augmented_part_ap_row_edit.html", ctx
     )
@@ -2651,7 +2682,12 @@ async def fit_other_points(
         else []
     )
     return templates.TemplateResponse(
-        request, "admin/_fit_other_points_select.html", {"points": points}
+        request,
+        "admin/_fit_other_points_select.html",
+        {
+            "points": points,
+            "form_id": f"ap-{augmented_part_id}-fit-add-form",
+        },
     )
 
 
@@ -3206,7 +3242,7 @@ def _ap_connection_label(ap) -> str:
         return f"{cc.min_in}–{cc.max_in} in"
     if ap.static_connection:
         return f"{ap.static_connection.od_mm} mm OD"
-    return "—"
+    return "Manual"
 
 
 @router.get("/attachment-points", response_class=HTMLResponse)
@@ -3303,7 +3339,7 @@ async def edit_attachment_point_row(
     elif ap.static_connection_id:
         ctx["connection_type"] = "static_connection"
     else:
-        ctx["connection_type"] = ""
+        ctx["connection_type"] = "manual"
     return templates.TemplateResponse(
         request, "admin/_attachment_point_row_edit.html", ctx
     )

app/web/pages.py

@@ -1,18 +1,33 @@
+from datetime import timedelta
+
 from fastapi import APIRouter, Depends, Form, HTTPException, Query, Request, status
 from fastapi.responses import HTMLResponse, RedirectResponse, Response
 from sqlalchemy.ext.asyncio import AsyncSession
 
 from app.core.config import settings
-from app.core.database import get_db
+from app.core.database import AsyncSessionLocal, get_db
+from app.crud import audit as audit_crud
+from app.crud import augmented_part as augmented_part_crud
+from app.crud import imported_part as imported_part_crud
+from app.crud import imported_part_config as imported_part_config_crud
+from app.crud import part_type as part_type_crud
 from app.crud import user as user_crud
-from app.deps import get_current_web_user
+from app.deps import get_optional_web_user
 from app.models.user import User
+from app.services import augmented_part_sync as augmented_part_sync_service
+from app.services import fit_matching as fit_matching_service
+from app.services import jobs
 from app.services import ldap as ldap_service
+from app.services import odata as odata_service
+from app.services import scope_guide as scope_guide_service
 from app.services.auth import issue_token_pair
+from app.web.admin import web_require
 from app.web.templates import templates
 
 router = APIRouter(include_in_schema=False)
 
+_SYNC_CARD_ID = "dashboard-sync-card"
+
 
 def _safe_next(next_url: str) -> str:
     """Return next_url only if it's a safe same-site path, else "/".
@@ -36,17 +51,344 @@ def _set_auth_cookie(response: Response, access_token: str) -> None:
         httponly=True,
         secure=settings.cookie_secure,
         samesite="lax",
-        max_age=settings.access_token_expire_minutes * 60,
+        max_age=settings.session_timeout_hours * 3600,
         path="/",
     )
 
 
+async def _dashboard_stats(db: AsyncSession) -> dict:
+    scope_type = await part_type_crud.get_by_name(
+        db, scope_guide_service.SCOPE_TYPE_NAME
+    )
+    scope_count = (
+        await augmented_part_crud.count_filtered(db, part_type_id=scope_type.id)
+        if scope_type is not None
+        else 0
+    )
+    return {
+        "scope_count": scope_count,
+        "skipped_count": await augmented_part_crud.count_unlinked_filtered(db),
+    }
+
+
+def _dashboard_permissions(user: User) -> dict:
+    return {
+        "can_refresh_odata": user.is_superuser
+        or "imported_parts:write" in user.permissions,
+        "can_sync_augmented_parts": user.is_superuser
+        or "augmented_parts:write" in user.permissions,
+        "can_recompute_fits": user.is_superuser or "fits:write" in user.permissions,
+    }
+
+
+async def _sync_card_ctx(
+    db: AsyncSession,
+    user: User,
+    status_message: str | None = None,
+    status_error: str | None = None,
+) -> dict:
+    return {
+        **(await _dashboard_stats(db)),
+        **_dashboard_permissions(user),
+        "status_message": status_message,
+        "status_error": status_error,
+    }
+
+
 @router.get("/", response_class=HTMLResponse)
 async def dashboard(
     request: Request,
-    user: User = Depends(get_current_web_user),
+    db: AsyncSession = Depends(get_db),
+    user: User | None = Depends(get_optional_web_user),
 ):
-    return templates.TemplateResponse(request, "dashboard.html", {"user": user})
+    # Anonymous visitors land on the public Scopes page instead of being
+    # bounced to login; the dashboard itself requires an account.
+    if user is None:
+        return RedirectResponse(url="/scopes", status_code=status.HTTP_302_FOUND)
+    return templates.TemplateResponse(
+        request,
+        "dashboard.html",
+        {"user": user, **(await _sync_card_ctx(db, user))},
+    )
+
+
+@router.post("/dashboard/refresh-odata", response_class=HTMLResponse)
+async def dashboard_refresh_odata(
+    request: Request,
+    db: AsyncSession = Depends(get_db),
+    actor: User = Depends(web_require("imported_parts:write")),
+):
+    config = await imported_part_config_crud.get(db)
+    if not config or not config.odata_url:
+        return templates.TemplateResponse(
+            request,
+            "_dashboard_sync_card.html",
+            await _sync_card_ctx(
+                db, actor, status_error="OData URL is not configured."
+            ),
+        )
+
+    actor_id = actor.id
+
+    async def _run(job_id: str) -> None:
+        async with AsyncSessionLocal() as db:
+            config = await imported_part_config_crud.get(db)
+            if not config or not config.odata_url:
+                await jobs.mark_error(job_id, "OData URL is not configured.")
+                return
+            try:
+                records = await odata_service.fetch(
+                    config.odata_url, config.username, config.password
+                )
+                created, updated = await imported_part_crud.upsert_bulk(db, records)
+            except Exception as exc:
+                await jobs.mark_error(job_id, f"Sync failed: {exc}")
+                return
+            actor_row = await user_crud.get(db, actor_id)
+            await audit_crud.log(
+                db,
+                actor=actor_row,
+                action="update",
+                resource_type="ImportedPart",
+                resource_id=None,
+                description=(
+                    f"OData sync: {created} created, {updated} updated "
+                    f"({len(records)} total records)"
+                ),
+            )
+            await jobs.mark_done(
+                job_id,
+                message=f"Sync complete: {created} created, {updated} updated.",
+            )
+
+    job = await jobs.start("imported_parts_refresh", _run)
+    return templates.TemplateResponse(
+        request,
+        "admin/_job_progress.html",
+        {
+            "container_id": _SYNC_CARD_ID,
+            "poll_url": f"/dashboard/refresh-odata/jobs/{job.id}",
+            "job": job,
+        },
+    )
+
+
+@router.get("/dashboard/refresh-odata/jobs/{job_id}", response_class=HTMLResponse)
+async def dashboard_refresh_odata_job_status(
+    request: Request,
+    job_id: str,
+    db: AsyncSession = Depends(get_db),
+    actor: User = Depends(web_require("imported_parts:write")),
+):
+    job = await jobs.get(db, job_id)
+    if job is None:
+        return templates.TemplateResponse(
+            request,
+            "_dashboard_sync_card.html",
+            await _sync_card_ctx(
+                db, actor, status_error="Refresh job not found — it may have expired."
+            ),
+        )
+    if job.status == "running":
+        return templates.TemplateResponse(
+            request,
+            "admin/_job_progress.html",
+            {
+                "container_id": _SYNC_CARD_ID,
+                "poll_url": f"/dashboard/refresh-odata/jobs/{job_id}",
+                "job": job,
+            },
+        )
+    return templates.TemplateResponse(
+        request,
+        "_dashboard_sync_card.html",
+        await _sync_card_ctx(
+            db,
+            actor,
+            status_message=job.message if job.status == "done" else None,
+            status_error=job.message if job.status == "error" else None,
+        ),
+    )
+
+
+@router.post("/dashboard/sync-augmented-parts", response_class=HTMLResponse)
+async def dashboard_sync_augmented_parts(
+    request: Request,
+    db: AsyncSession = Depends(get_db),
+    actor: User = Depends(web_require("augmented_parts:write")),
+):
+    actor_id = actor.id
+
+    async def _run(job_id: str) -> None:
+        async with AsyncSessionLocal() as db:
+            try:
+                result = await augmented_part_sync_service.sync(db)
+            except Exception as exc:
+                await jobs.mark_error(job_id, f"Sync failed: {exc}")
+                return
+            actor_row = await user_crud.get(db, actor_id)
+            await audit_crud.log(
+                db,
+                actor=actor_row,
+                action="update",
+                resource_type="AugmentedPart",
+                resource_id=None,
+                description=(
+                    f"Sync: {result.created} created, {result.archived} archived, "
+                    f"{result.skipped} skipped (no type match)"
+                ),
+            )
+            await jobs.mark_done(
+                job_id,
+                message=(
+                    f"Sync complete: {result.created} created, "
+                    f"{result.archived} archived"
+                    + (
+                        f", {result.skipped} skipped (type not recognised)"
+                        if result.skipped
+                        else ""
+                    )
+                    + "."
+                ),
+            )
+
+    job = await jobs.start("augmented_part_sync", _run)
+    return templates.TemplateResponse(
+        request,
+        "admin/_job_progress.html",
+        {
+            "container_id": _SYNC_CARD_ID,
+            "poll_url": f"/dashboard/sync-augmented-parts/jobs/{job.id}",
+            "job": job,
+        },
+    )
+
+
+@router.get(
+    "/dashboard/sync-augmented-parts/jobs/{job_id}", response_class=HTMLResponse
+)
+async def dashboard_sync_augmented_parts_job_status(
+    request: Request,
+    job_id: str,
+    db: AsyncSession = Depends(get_db),
+    actor: User = Depends(web_require("augmented_parts:write")),
+):
+    job = await jobs.get(db, job_id)
+    if job is None:
+        return templates.TemplateResponse(
+            request,
+            "_dashboard_sync_card.html",
+            await _sync_card_ctx(
+                db, actor, status_error="Sync job not found — it may have expired."
+            ),
+        )
+    if job.status == "running":
+        return templates.TemplateResponse(
+            request,
+            "admin/_job_progress.html",
+            {
+                "container_id": _SYNC_CARD_ID,
+                "poll_url": f"/dashboard/sync-augmented-parts/jobs/{job_id}",
+                "job": job,
+            },
+        )
+    return templates.TemplateResponse(
+        request,
+        "_dashboard_sync_card.html",
+        await _sync_card_ctx(
+            db,
+            actor,
+            status_message=job.message if job.status == "done" else None,
+            status_error=job.message if job.status == "error" else None,
+        ),
+    )
+
+
+@router.post("/dashboard/recompute-fits", response_class=HTMLResponse)
+async def dashboard_recompute_fits(
+    request: Request,
+    db: AsyncSession = Depends(get_db),
+    actor: User = Depends(web_require("fits:write")),
+):
+    actor_id = actor.id
+
+    async def _run(job_id: str) -> None:
+        async with AsyncSessionLocal() as db:
+            try:
+                result = await fit_matching_service.recompute_all(db)
+            except Exception as exc:
+                await jobs.mark_error(job_id, f"Recompute failed: {exc}")
+                return
+            actor_row = await user_crud.get(db, actor_id)
+            await audit_crud.log(
+                db,
+                actor=actor_row,
+                action="update",
+                resource_type="Fit",
+                resource_id=None,
+                description=(
+                    f"Recomputed fits for all parts: {result.created} created, "
+                    f"{result.removed} removed"
+                ),
+            )
+            await jobs.mark_done(
+                job_id,
+                message=(
+                    f"Recompute complete: {result.created} created, "
+                    f"{result.removed} removed."
+                ),
+            )
+
+    job = await jobs.start("fit_recompute_all", _run)
+    return templates.TemplateResponse(
+        request,
+        "admin/_job_progress.html",
+        {
+            "container_id": _SYNC_CARD_ID,
+            "poll_url": f"/dashboard/recompute-fits/jobs/{job.id}",
+            "job": job,
+        },
+    )
+
+
+@router.get("/dashboard/recompute-fits/jobs/{job_id}", response_class=HTMLResponse)
+async def dashboard_recompute_fits_job_status(
+    request: Request,
+    job_id: str,
+    db: AsyncSession = Depends(get_db),
+    actor: User = Depends(web_require("fits:write")),
+):
+    job = await jobs.get(db, job_id)
+    if job is None:
+        return templates.TemplateResponse(
+            request,
+            "_dashboard_sync_card.html",
+            await _sync_card_ctx(
+                db,
+                actor,
+                status_error="Recompute job not found — it may have expired.",
+            ),
+        )
+    if job.status == "running":
+        return templates.TemplateResponse(
+            request,
+            "admin/_job_progress.html",
+            {
+                "container_id": _SYNC_CARD_ID,
+                "poll_url": f"/dashboard/recompute-fits/jobs/{job_id}",
+                "job": job,
+            },
+        )
+    return templates.TemplateResponse(
+        request,
+        "_dashboard_sync_card.html",
+        await _sync_card_ctx(
+            db,
+            actor,
+            status_message=job.message if job.status == "done" else None,
+            status_error=job.message if job.status == "error" else None,
+        ),
+    )
 
 
 @router.get("/login", response_class=HTMLResponse)
@@ -79,7 +421,9 @@ async def login_submit(
             },
             status_code=status.HTTP_401_UNAUTHORIZED,
         )
-    access, _ = await issue_token_pair(db, user)
+    access, _ = await issue_token_pair(
+        db, user, timedelta(hours=settings.session_timeout_hours)
+    )
     response = RedirectResponse(
         url=_safe_next(next_url), status_code=status.HTTP_303_SEE_OTHER
     )
@@ -110,7 +454,9 @@ async def login_ldap_submit(
             },
             status_code=status.HTTP_401_UNAUTHORIZED,
         )
-    access, _ = await issue_token_pair(db, user)
+    access, _ = await issue_token_pair(
+        db, user, timedelta(hours=settings.session_timeout_hours)
+    )
     response = RedirectResponse(
         url=_safe_next(next_url), status_code=status.HTTP_303_SEE_OTHER
     )

app/web/pagination.py

@@ -0,0 +1,40 @@
+"""Shared pagination helpers for HTMX table cards (admin screens, scopes page)."""
+
+import math
+import urllib.parse
+
+
+def query_url(base_path: str, params: dict) -> str:
+    clean = {k: v for k, v in params.items() if v not in (None, "", 0)}
+    return f"{base_path}?{urllib.parse.urlencode(clean)}"
+
+
+def paginate(
+    base_path: str,
+    filter_params: dict,
+    page: int,
+    page_size: int,
+    total: int,
+    target_id: str,
+) -> dict:
+    total_pages = max(1, math.ceil(total / page_size))
+    page = max(1, min(page, total_pages))
+    skip = (page - 1) * page_size
+
+    def _url(p: int) -> str:
+        return query_url(
+            base_path, {**filter_params, "page": p, "page_size": page_size}
+        )
+
+    return {
+        "page": page,
+        "page_size": page_size,
+        "total": total,
+        "total_pages": total_pages,
+        "skip": skip,
+        "showing_from": skip + 1 if total > 0 else 0,
+        "showing_to": min(skip + page_size, total),
+        "prev_url": _url(page - 1) if page > 1 else None,
+        "next_url": _url(page + 1) if page < total_pages else None,
+        "target_id": target_id,
+    }

app/web/scopes.py

@@ -5,29 +5,67 @@ non-Scope part type listing the accessories that fit each scope. Supports HTMX
 search/filter over scope attributes and required accessory fits.
 """
 
-from fastapi import APIRouter, Depends, Request
+from fastapi import APIRouter, Depends, Form, HTTPException, Request, status
 from fastapi.responses import HTMLResponse
 from sqlalchemy.ext.asyncio import AsyncSession
 
 from app.core.database import get_db
+from app.crud import audit as audit_crud
+from app.crud import augmented_part as augmented_part_crud
 from app.crud import manufacturer as manufacturer_crud
 from app.crud import part_type as part_type_crud
 from app.crud import scope_model as scope_model_crud
 from app.crud import sub_part_type as sub_part_type_crud
+from app.deps import get_optional_web_user
 from app.models.user import User
 from app.services import scope_guide as scope_guide_service
 from app.web.admin import web_require
+from app.web.pagination import paginate
 from app.web.templates import templates
 
 router = APIRouter(include_in_schema=False)
 
+_PAGE_SIZE = 50
+_TABLE_TARGET = "#scopes-table-card"
+_FILTER_PATH = "/scopes/filter"
+_REQUIRED_PERMS = {"augmented_parts:read", "fits:read"}
+
 
 def _to_int(value: str) -> int | None:
     return int(value) if value.strip() else None
 
 
-async def _build(db: AsyncSession, filters: dict) -> dict:
+def _can_write_scopes(user: User | None) -> bool:
-    """Run the guide query for the given raw (string) filter values."""
+    return user is not None and (
+        user.is_superuser or "augmented_parts:write" in user.permissions
+    )
+
+
+async def _optional_scopes_user(
+    user: User | None = Depends(get_optional_web_user),
+) -> User | None:
+    """Anyone may view the scopes page; logged-in users still need the read perms.
+
+    Anonymous visitors are allowed through (and get an exact-fits-only view —
+    see ``only_exact`` in ``scope_guide.build``); a logged-in user missing the
+    permissions gets the usual 403, matching ``web_require``'s behaviour.
+    """
+    if user is not None and not (
+        user.is_superuser or _REQUIRED_PERMS.issubset(user.permissions)
+    ):
+        raise HTTPException(
+            status_code=status.HTTP_403_FORBIDDEN,
+            detail="You do not have access to this page.",
+        )
+    return user
+
+
+async def _build(db: AsyncSession, filters: dict, only_exact: bool = False) -> dict:
+    """Run the guide query for the given raw (string) filter values.
+
+    Always returns the full matching row set — pagination is applied by the
+    caller by slicing ``rows`` (see ``_paginated_guide``).
+    """
     return await scope_guide_service.build(
         db,
         description=filters["description"],
@@ -35,9 +73,30 @@ async def _build(db: AsyncSession, filters: dict) -> dict:
         scope_model_id=_to_int(filters["scope_model_id"]),
         sub_part_type_id=_to_int(filters["sub_part_type_id"]),
         fits_part_type_id=_to_int(filters["fits_part_type_id"]),
+        only_exact=only_exact,
     )
 
 
+async def _paginated_guide(
+    db: AsyncSession,
+    filters: dict,
+    page: int,
+    page_size: int,
+    only_exact: bool = False,
+) -> tuple[dict, dict]:
+    """Run the guide query, then slice its rows to the requested page.
+
+    Returns ``(guide, pagination)`` where ``guide["rows"]`` is already sliced.
+    """
+    guide = await _build(db, filters, only_exact=only_exact)
+    all_rows = guide["rows"]
+    pagination = paginate(
+        _FILTER_PATH, filters, page, page_size, len(all_rows), _TABLE_TARGET
+    )
+    guide["rows"] = all_rows[pagination["skip"] : pagination["skip"] + page_size]
+    return guide, pagination
+
+
 async def _filter_options(db: AsyncSession) -> dict:
     """Dropdown option lists for the filter bar."""
     scope_type = await part_type_crud.get_by_name(
@@ -64,7 +123,7 @@ async def scopes_page(
     sub_part_type_id: str = "",
     fits_part_type_id: str = "",
     db: AsyncSession = Depends(get_db),
-    user: User = Depends(web_require("augmented_parts:read", "fits:read")),
+    user: User | None = Depends(_optional_scopes_user),
 ):
     filters = {
         "description": description,
@@ -73,11 +132,22 @@ async def scopes_page(
         "sub_part_type_id": sub_part_type_id,
         "fits_part_type_id": fits_part_type_id,
     }
-    guide = await _build(db, filters)
+    only_exact = user is None
+    guide, pagination = await _paginated_guide(
+        db, filters, 1, _PAGE_SIZE, only_exact=only_exact
+    )
     return templates.TemplateResponse(
         request,
         "scopes.html",
-        {"user": user, "filters": filters, **guide, **(await _filter_options(db))},
+        {
+            "user": user,
+            "filters": filters,
+            "can_write_scopes": _can_write_scopes(user),
+            "only_exact": only_exact,
+            "pagination": pagination,
+            **guide,
+            **(await _filter_options(db)),
+        },
     )
 
 
@@ -89,8 +159,10 @@ async def scopes_filter(
     scope_model_id: str = "",
     sub_part_type_id: str = "",
     fits_part_type_id: str = "",
+    page: int = 1,
+    page_size: int = _PAGE_SIZE,
     db: AsyncSession = Depends(get_db),
-    _: User = Depends(web_require("augmented_parts:read", "fits:read")),
+    user: User | None = Depends(_optional_scopes_user),
 ):
     filters = {
         "description": description,
@@ -99,5 +171,63 @@ async def scopes_filter(
         "sub_part_type_id": sub_part_type_id,
         "fits_part_type_id": fits_part_type_id,
     }
-    guide = await _build(db, filters)
+    only_exact = user is None
-    return templates.TemplateResponse(request, "_scopes_table_card.html", guide)
+    guide, pagination = await _paginated_guide(
+        db, filters, page, page_size, only_exact=only_exact
+    )
+    return templates.TemplateResponse(
+        request,
+        "_scopes_table_card.html",
+        {
+            **guide,
+            "pagination": pagination,
+            "can_write_scopes": _can_write_scopes(user),
+        },
+    )
+
+
+@router.post("/scopes/bulk-delete", response_class=HTMLResponse)
+async def bulk_delete_scopes(
+    request: Request,
+    scope_ids: list[int] = Form([]),
+    description: str = Form(""),
+    manufacturer_id: str = Form(""),
+    scope_model_id: str = Form(""),
+    sub_part_type_id: str = Form(""),
+    fits_part_type_id: str = Form(""),
+    db: AsyncSession = Depends(get_db),
+    actor: User = Depends(web_require("augmented_parts:write")),
+):
+    for augmented_part_id in scope_ids:
+        ap = await augmented_part_crud.get(db, augmented_part_id)
+        if ap is None:
+            continue
+        label = ap.sku or ap.description
+        await augmented_part_crud.delete(db, ap)
+        await audit_crud.log(
+            db,
+            actor=actor,
+            action="delete",
+            resource_type="AugmentedPart",
+            resource_id=augmented_part_id,
+            description=audit_crud.describe_delete(
+                "AugmentedPart", augmented_part_id, label
+            ),
+        )
+    filters = {
+        "description": description,
+        "manufacturer_id": manufacturer_id,
+        "scope_model_id": scope_model_id,
+        "sub_part_type_id": sub_part_type_id,
+        "fits_part_type_id": fits_part_type_id,
+    }
+    guide, pagination = await _paginated_guide(db, filters, 1, _PAGE_SIZE)
+    return templates.TemplateResponse(
+        request,
+        "_scopes_table_card.html",
+        {
+            **guide,
+            "pagination": pagination,
+            "can_write_scopes": _can_write_scopes(actor),
+        },
+    )

data/TODO

@@ -0,0 +1,34 @@
+- [ ] AugmentedParts Listing
+	- [x] Add an `archived` attribute that indicates the part should not show up in default searches
+		- [x] Admins and users with permissions can opt into seeing archived parts
+	- [ ] Add bulk "mark archived"
+
+- [ ] Scope Import
+	- [ ] Test scopes with explicit fits populated
+
+- [ ] Dashboard
+	- [ ] Make this more useful.
+	- [ ] Number of
+		- [ ] Count of Augmented Parts by Type
+		- [ ] Skipped parts from last Sync
+	- [ ] Handy buttons to run the import / sync / recompute actions
+
+- [x] AugmentedPart Details
+	- [x] Title should be the SKU
+	- [x] Add new inference for attachment point "Clamp"
+		- [x] Appears as `(44.5-47.5mm)` in the description
+
+- [x] Better layout for Scopes page
+	- [x] Layout is too wide, requiring side-scrolling
+	- [x] Use product IDs instead of full descriptions
+	- [x] Descriptions available as a mouseover hint
+	- [x] Fit type (exact/possible/none) represented as a halo rather than a label
+- [X] ScopeInfo table
+	- [X] Remove "objective_thread", as it's not populated and confusing
+- [x] Part Import
+	- [x] Example query of `ZC5000` should return multiple parts
+	- [x] Filter text should search SKU and description
+- [x] Attachment Points
+	- [x] Remove objective / occular labels and matching logic
+
+

entrypoint.sh

@@ -2,6 +2,7 @@
 set -e
 
 # Idempotent: creates perms/roles/superuser only if missing.
+uv run alembic upgrade head
 uv run python -m scripts.seed
 
 exec uv run uvicorn app.main:app --host 0.0.0.0 --port 8000

scripts/seed.py

@@ -17,7 +17,7 @@ from app.crud import role as role_crud
 from app.crud import sub_part_type as sub_part_type_crud
 from app.crud import user as user_crud
 
-ATTACHMENT_POINT_NAMES = ["Objective", "Occular"]
+ATTACHMENT_POINT_NAMES = ["Objective", "Ocular", "Rail"]
 
 PART_TYPES = ["Flip Cover", "ARD", "Polarizer", "Adapter Ring", "Scope"]
 

tests/test_admin_sorting.py

@@ -90,6 +90,23 @@ async def test_augmented_parts_filter_sorts_by_sku_asc(
     ]
 
 
+@pytest.mark.asyncio
+async def test_augmented_parts_filter_matches_by_sku(
+    client, seeded, make_user, db_sessionmaker
+):
+    async with db_sessionmaker() as db:
+        await _seed_augmented_parts(db)
+    await make_user("reader2b@example.com", role_ids=[seeded["roles"]["user"].id])
+    await _web_login(client, "reader2b@example.com")
+
+    resp = await client.get("/admin/augmented-parts/filter?description=SKU-B")
+    assert resp.status_code == 200
+    body = resp.text
+    assert "Charlie scope" in body
+    assert "Alpha scope" not in body
+    assert "Bravo scope" not in body
+
+
 @pytest.mark.asyncio
 async def test_imported_parts_page_sorts_by_description_desc(
     client, seeded, make_user, db_sessionmaker
@@ -109,6 +126,23 @@ async def test_imported_parts_page_sorts_by_description_desc(
     ]
 
 
+@pytest.mark.asyncio
+async def test_imported_parts_filter_matches_by_inventory_id(
+    client, seeded, make_user, db_sessionmaker
+):
+    async with db_sessionmaker() as db:
+        await _seed_imported_parts(db)
+    await make_user("reader4@example.com", role_ids=[seeded["roles"]["user"].id])
+    await _web_login(client, "reader4@example.com")
+
+    resp = await client.get("/admin/imported-parts/filter?description=INV-3")
+    assert resp.status_code == 200
+    body = resp.text
+    assert "Bravo widget" in body
+    assert "Alpha widget" not in body
+    assert "Charlie widget" not in body
+
+
 @pytest.mark.asyncio
 async def test_list_filtered_sorts_by_joined_colour_name(db_sessionmaker):
     async with db_sessionmaker() as db:

tests/test_attachment_points_api.py

@@ -0,0 +1,106 @@
+from app.models.attachment_point_name import AttachmentPointName
+from app.models.augmented_part import AugmentedPart
+from app.models.iso_thread import IsoThread
+from app.models.manufacturer import Manufacturer
+from app.models.part_colour import PartColour
+from app.models.part_type import PartType
+
+
+async def _seed_part(db):
+    manufacturer = Manufacturer(name="Acme", url="https://acme.test")
+    colour = PartColour(name="Black")
+    part_type = PartType(name="Flip Cover")
+    apn = AttachmentPointName(name="Mount")
+    iso = IsoThread(diameter_mm=52.0, pitch_mm=0.75)
+    db.add_all([manufacturer, colour, part_type, apn, iso])
+    await db.flush()
+
+    part = AugmentedPart(
+        description="Flip cover",
+        colour_id=colour.id,
+        manufacturer_id=manufacturer.id,
+        part_type_id=part_type.id,
+    )
+    db.add(part)
+    await db.flush()
+    await db.commit()
+    return part.id, apn.id, iso.id
+
+
+async def _token(client, email, password="secret123"):
+    resp = await client.post(
+        "/api/auth/login", data={"username": email, "password": password}
+    )
+    assert resp.status_code == 200, resp.text
+    return resp.json()["access_token"]
+
+
+async def test_create_manual_attachment_point_with_no_connection_succeeds(
+    client, seeded, make_user, db_sessionmaker
+):
+    await make_user("root@example.com", is_superuser=True)
+    headers = {"Authorization": f"Bearer {await _token(client, 'root@example.com')}"}
+    async with db_sessionmaker() as db:
+        part_id, apn_id, _ = await _seed_part(db)
+
+    resp = await client.post(
+        "/api/attachment-points",
+        json={
+            "augmented_part_id": part_id,
+            "attachment_point_name_id": apn_id,
+        },
+        headers=headers,
+    )
+    assert resp.status_code == 201, resp.text
+    body = resp.json()
+    assert body["iso_thread_id"] is None
+    assert body["unc_thread_id"] is None
+    assert body["clamp_connection_id"] is None
+    assert body["static_connection_id"] is None
+
+
+async def test_create_attachment_point_with_two_connections_rejected(
+    client, seeded, make_user, db_sessionmaker
+):
+    await make_user("root2@example.com", is_superuser=True)
+    headers = {"Authorization": f"Bearer {await _token(client, 'root2@example.com')}"}
+    async with db_sessionmaker() as db:
+        part_id, apn_id, iso_id = await _seed_part(db)
+        from app.crud import static_connection as static_connection_crud
+
+        static = await static_connection_crud.create(db, od_mm=40.0, od_in=1.57)
+        static_id = static.id
+
+    resp = await client.post(
+        "/api/attachment-points",
+        json={
+            "augmented_part_id": part_id,
+            "attachment_point_name_id": apn_id,
+            "iso_thread_id": iso_id,
+            "static_connection_id": static_id,
+        },
+        headers=headers,
+    )
+    assert resp.status_code == 400
+    assert "at most one" in resp.json()["detail"].lower()
+
+
+async def test_create_attachment_point_with_one_connection_still_works(
+    client, seeded, make_user, db_sessionmaker
+):
+    await make_user("root3@example.com", is_superuser=True)
+    headers = {"Authorization": f"Bearer {await _token(client, 'root3@example.com')}"}
+    async with db_sessionmaker() as db:
+        part_id, apn_id, iso_id = await _seed_part(db)
+
+    resp = await client.post(
+        "/api/attachment-points",
+        json={
+            "augmented_part_id": part_id,
+            "attachment_point_name_id": apn_id,
+            "iso_thread_id": iso_id,
+        },
+        headers=headers,
+    )
+    assert resp.status_code == 201, resp.text
+    assert resp.json()["iso_thread_id"] == iso_id

tests/test_augmented_part_archiving.py

@@ -0,0 +1,257 @@
+import pytest
+
+from app.crud import augmented_part as augmented_part_crud
+from app.models.augmented_part import AugmentedPart
+from app.models.imported_part import ImportedPart
+from app.models.manufacturer import Manufacturer
+from app.models.part_colour import PartColour
+from app.models.part_type import PartType
+from app.services import augmented_part_sync
+
+
+async def _seed_reference_data(db):
+    manufacturer = Manufacturer(name="Acme", url="https://acme.test")
+    colour = PartColour(name="Black")
+    part_type = PartType(name="Scope")
+    db.add_all([manufacturer, colour, part_type])
+    await db.flush()
+    return manufacturer, colour, part_type
+
+
+async def _token(client, email, password="secret123"):
+    resp = await client.post(
+        "/api/auth/login", data={"username": email, "password": password}
+    )
+    assert resp.status_code == 200, resp.text
+    return resp.json()["access_token"]
+
+
+@pytest.mark.asyncio
+async def test_list_and_count_exclude_archived_by_default(db_sessionmaker):
+    async with db_sessionmaker() as db:
+        manufacturer, colour, part_type = await _seed_reference_data(db)
+        active = AugmentedPart(
+            description="Active part",
+            colour_id=colour.id,
+            manufacturer_id=manufacturer.id,
+            part_type_id=part_type.id,
+        )
+        archived = AugmentedPart(
+            description="Archived part",
+            colour_id=colour.id,
+            manufacturer_id=manufacturer.id,
+            part_type_id=part_type.id,
+            archived=True,
+        )
+        db.add_all([active, archived])
+        await db.commit()
+
+        assert [p.description for p in await augmented_part_crud.list_all(db)] == [
+            "Active part"
+        ]
+        assert await augmented_part_crud.count_filtered(db) == 1
+
+        all_descs = {
+            p.description
+            for p in await augmented_part_crud.list_all(db, include_archived=True)
+        }
+        assert all_descs == {"Active part", "Archived part"}
+        assert await augmented_part_crud.count_filtered(db, include_archived=True) == 2
+
+
+@pytest.mark.asyncio
+async def test_set_archived_toggles_flag(db_sessionmaker):
+    async with db_sessionmaker() as db:
+        manufacturer, colour, part_type = await _seed_reference_data(db)
+        ap = AugmentedPart(
+            description="Widget",
+            colour_id=colour.id,
+            manufacturer_id=manufacturer.id,
+            part_type_id=part_type.id,
+        )
+        db.add(ap)
+        await db.commit()
+
+        ap = await augmented_part_crud.set_archived(db, ap, True)
+        assert ap.archived is True
+
+        ap = await augmented_part_crud.set_archived(db, ap, False)
+        assert ap.archived is False
+
+
+@pytest.mark.asyncio
+async def test_sync_archives_part_only_when_all_backing_imports_are_gone(
+    db_sessionmaker,
+):
+    async with db_sessionmaker() as db:
+        manufacturer, colour, part_type = await _seed_reference_data(db)
+        ip_x = ImportedPart(InventoryID="INV-X", Description="Widget X")
+        ip_y = ImportedPart(InventoryID="INV-Y", Description="Widget Y")
+        db.add_all([ip_x, ip_y])
+        await db.flush()
+
+        # One AugmentedPart backed by two ImportedParts.
+        merged = AugmentedPart(
+            description="Merged widget",
+            colour_id=colour.id,
+            manufacturer_id=manufacturer.id,
+            part_type_id=part_type.id,
+            imported_parts=[ip_x, ip_y],
+        )
+        db.add(merged)
+        await db.commit()
+        merged_id = merged.id
+
+        # Delete one backing ImportedPart directly (simulating an external feed
+        # change) and sync: the AugmentedPart still has INV-Y, so it survives.
+        await db.delete(ip_x)
+        await db.commit()
+
+        result = await augmented_part_sync.sync(db)
+        assert result.archived == 0
+
+        refreshed = await augmented_part_crud.get(db, merged_id)
+        assert refreshed.archived is False
+
+        # Now delete the last remaining backing ImportedPart.
+        await db.delete(ip_y)
+        await db.commit()
+
+        result = await augmented_part_sync.sync(db)
+        assert result.archived == 1
+        assert result.created == 0
+
+        refreshed = await augmented_part_crud.get(db, merged_id)
+        assert refreshed.archived is True
+
+        # Re-running sync is a no-op — it must not double count or explode.
+        result = await augmented_part_sync.sync(db)
+        assert result.archived == 0
+
+
+@pytest.mark.asyncio
+async def test_delete_imported_part_via_api_archives_orphaned_augmented_part(
+    client, seeded, make_user, db_sessionmaker
+):
+    async with db_sessionmaker() as db:
+        manufacturer, colour, part_type = await _seed_reference_data(db)
+        ip = ImportedPart(InventoryID="INV-SOLE", Description="Solo widget")
+        db.add(ip)
+        await db.flush()
+        ap = AugmentedPart(
+            description="Solo widget",
+            sku="INV-SOLE",
+            colour_id=colour.id,
+            manufacturer_id=manufacturer.id,
+            part_type_id=part_type.id,
+            imported_parts=[ip],
+        )
+        db.add(ap)
+        await db.commit()
+        ap_id = ap.id
+
+    await make_user("root@example.com", is_superuser=True)
+    headers = {"Authorization": f"Bearer {await _token(client, 'root@example.com')}"}
+
+    resp = await client.delete("/api/imported-parts/INV-SOLE", headers=headers)
+    assert resp.status_code == 204
+
+    # ImportedPart is really gone.
+    assert (
+        await client.get("/api/imported-parts/INV-SOLE", headers=headers)
+    ).status_code == 404
+
+    # AugmentedPart is archived, not deleted — direct fetch still finds it.
+    get_resp = await client.get(f"/api/augmented-parts/{ap_id}", headers=headers)
+    assert get_resp.status_code == 200
+    assert get_resp.json()["archived"] is True
+
+    # Hidden from the default list...
+    default_list = await client.get("/api/augmented-parts", headers=headers)
+    assert ap_id not in [p["id"] for p in default_list.json()]
+
+    # ...but present when archived parts are explicitly requested.
+    full_list = await client.get(
+        "/api/augmented-parts?include_archived=true", headers=headers
+    )
+    assert ap_id in [p["id"] for p in full_list.json()]
+
+
+@pytest.mark.asyncio
+async def test_patch_can_unarchive_an_augmented_part(
+    client, seeded, make_user, db_sessionmaker
+):
+    async with db_sessionmaker() as db:
+        manufacturer, colour, part_type = await _seed_reference_data(db)
+        ap = AugmentedPart(
+            description="Previously archived",
+            colour_id=colour.id,
+            manufacturer_id=manufacturer.id,
+            part_type_id=part_type.id,
+            archived=True,
+        )
+        db.add(ap)
+        await db.commit()
+        ap_id = ap.id
+
+    await make_user("root2@example.com", is_superuser=True)
+    headers = {"Authorization": f"Bearer {await _token(client, 'root2@example.com')}"}
+
+    default_list = await client.get("/api/augmented-parts", headers=headers)
+    assert ap_id not in [p["id"] for p in default_list.json()]
+
+    resp = await client.patch(
+        f"/api/augmented-parts/{ap_id}", json={"archived": False}, headers=headers
+    )
+    assert resp.status_code == 200
+    assert resp.json()["archived"] is False
+
+    default_list = await client.get("/api/augmented-parts", headers=headers)
+    assert ap_id in [p["id"] for p in default_list.json()]
+
+
+async def _web_login(client, email, password="secret123"):
+    resp = await client.post("/login", data={"email": email, "password": password})
+    assert resp.status_code == 303
+
+
+@pytest.mark.asyncio
+async def test_admin_ui_hides_archived_unless_requested_and_can_toggle(
+    client, seeded, make_user, db_sessionmaker
+):
+    async with db_sessionmaker() as db:
+        manufacturer, colour, part_type = await _seed_reference_data(db)
+        ap = AugmentedPart(
+            description="Hidden gem",
+            colour_id=colour.id,
+            manufacturer_id=manufacturer.id,
+            part_type_id=part_type.id,
+            archived=True,
+        )
+        db.add(ap)
+        await db.commit()
+        ap_id = ap.id
+
+    await make_user("admin@example.com", role_ids=[seeded["roles"]["admin"].id])
+    await _web_login(client, "admin@example.com")
+
+    resp = await client.get("/admin/augmented-parts")
+    assert resp.status_code == 200
+    assert "Hidden gem" not in resp.text
+
+    resp = await client.get("/admin/augmented-parts/filter?include_archived=true")
+    assert resp.status_code == 200
+    assert "Hidden gem" in resp.text
+
+    resp = await client.post(f"/admin/augmented-parts/{ap_id}/unarchive")
+    assert resp.status_code == 200
+    assert "Archived" not in resp.text
+
+    resp = await client.get("/admin/augmented-parts")
+    assert "Hidden gem" in resp.text
+
+    resp = await client.post(f"/admin/augmented-parts/{ap_id}/archive")
+    assert resp.status_code == 200
+
+    resp = await client.get("/admin/augmented-parts")
+    assert "Hidden gem" not in resp.text

tests/test_augmented_part_detail.py

@@ -38,7 +38,6 @@ async def _seed_part_with_fit(db):
             augmented_part_id=scope.id,
             magnification_min=3,
             magnification_max=18,
-            objective_thread_pitch="M52x0.75",
         )
     )
     ap_s = AttachmentPoint(augmented_part_id=scope.id, attachment_point_name_id=apn.id)
@@ -80,7 +79,6 @@ async def test_detail_shows_attributes_and_fits_readonly(
     # Attributes
     assert "Alpha 3-18x50" in body
     assert "ALP-318" in body
-    assert "M52x0.75" in body  # scope info
     assert "Objective" in body  # attachment point
     # Fits are shown (the other part in the fit)
     assert "Flip cover 50mm" in body

tests/test_danger_zone.py

@@ -55,8 +55,12 @@ async def _seed_domain_data(db):
     db.add_all([scope, cover])
     await db.flush()
 
-    db.add(AugmentedPartNote(augmented_part_id=scope.id, username="tester", body="note"))
+    db.add(
-    db.add(ScopeInfo(augmented_part_id=scope.id, magnification_min=3, magnification_max=18))
+        AugmentedPartNote(augmented_part_id=scope.id, username="tester", body="note")
+    )
+    db.add(
+        ScopeInfo(augmented_part_id=scope.id, magnification_min=3, magnification_max=18)
+    )
 
     ap_s = AttachmentPoint(
         augmented_part_id=scope.id,
@@ -85,7 +89,9 @@ async def _seed_domain_data(db):
     imported = ImportedPart(InventoryID="INV-1", Description="Alpha 3-18x50")
     db.add(imported)
 
-    config = ImportedPartConfig(id=1, odata_url="https://example.test/odata", username="svc")
+    config = ImportedPartConfig(
+        id=1, odata_url="https://example.test/odata", username="svc"
+    )
     db.add(config)
 
     await db.commit()

tests/test_dashboard.py

@@ -0,0 +1,241 @@
+import asyncio
+import re
+
+import pytest
+import pytest_asyncio
+from sqlalchemy.ext.asyncio import async_sessionmaker, create_async_engine
+
+from app.core.database import Base
+from app.crud import imported_part_config as imported_part_config_crud
+from app.crud import role as role_crud
+from app.models.augmented_part import AugmentedPart
+from app.models.part_type import PartType
+
+_JOB_URL_RE = re.compile(r'hx-get="([^"]+/jobs/[^"]+)"')
+
+
+@pytest_asyncio.fixture
+async def db_sessionmaker(tmp_path):
+    """Override conftest's in-memory+StaticPool fixture for this module.
+
+    The dashboard's sync/recompute actions run a real background asyncio task
+    concurrently with polling requests, both hitting the DB at once. A real
+    file-backed DB (its own connection pool) supports that, unlike StaticPool's
+    single shared connection (see tests/test_admin_background_jobs.py).
+    """
+    db_path = tmp_path / "dashboard_test.db"
+    engine = create_async_engine(f"sqlite+aiosqlite:///{db_path}")
+    async with engine.begin() as conn:
+        await conn.run_sync(Base.metadata.create_all)
+    maker = async_sessionmaker(engine, expire_on_commit=False)
+    yield maker
+    await engine.dispose()
+
+
+async def _web_login(client, email, password="secret123"):
+    resp = await client.post("/login", data={"email": email, "password": password})
+    assert resp.status_code == 303
+
+
+async def _poll_until_done(client, url, max_tries=50):
+    """Follow the same htmx polling loop a browser would, until no further
+    ``hx-get`` job-status link is present in the response."""
+    body = ""
+    for _ in range(max_tries):
+        resp = await client.get(url)
+        assert resp.status_code == 200
+        body = resp.text
+        match = _JOB_URL_RE.search(body)
+        if not match:
+            return body
+        url = match.group(1)
+        await asyncio.sleep(0.02)
+    raise AssertionError(f"job did not finish after {max_tries} polls: {body!r}")
+
+
+@pytest.mark.asyncio
+async def test_dashboard_shows_scope_and_skipped_counts(
+    client, seeded, make_user, db_sessionmaker
+):
+    async with db_sessionmaker() as db:
+        scope_t = PartType(name="Scope")
+        db.add(scope_t)
+        await db.flush()
+        db.add_all(
+            [
+                AugmentedPart(
+                    description="Alpha 3-18x50",
+                    colour_id=1,
+                    manufacturer_id=1,
+                    part_type_id=scope_t.id,
+                ),
+                AugmentedPart(
+                    description="Zeta 1-6x24",
+                    colour_id=1,
+                    manufacturer_id=1,
+                    part_type_id=scope_t.id,
+                ),
+            ]
+        )
+        await db.commit()
+    await make_user("reader@example.com", role_ids=[seeded["roles"]["user"].id])
+    await _web_login(client, "reader@example.com")
+
+    resp = await client.get("/")
+    assert resp.status_code == 200
+    assert "Data sync" in resp.text
+    assert "<dd>2</dd>" in resp.text  # scope count
+    assert "<dd>0</dd>" in resp.text  # nothing imported/skipped yet
+
+
+@pytest.mark.asyncio
+async def test_dashboard_hides_action_buttons_without_permissions(
+    client, seeded, make_user
+):
+    await make_user("plain@example.com", role_ids=[seeded["roles"]["user"].id])
+    await _web_login(client, "plain@example.com")
+
+    resp = await client.get("/")
+    assert resp.status_code == 200
+    assert "Refresh from OData" not in resp.text
+    assert "Sync augmented parts" not in resp.text
+    assert "Recompute possible fits" not in resp.text
+
+
+@pytest.mark.asyncio
+async def test_dashboard_shows_action_buttons_for_superuser(client, make_user):
+    await make_user("boss@example.com", is_superuser=True)
+    await _web_login(client, "boss@example.com")
+
+    resp = await client.get("/")
+    assert resp.status_code == 200
+    assert "Refresh from OData" in resp.text
+    assert "Sync augmented parts" in resp.text
+    assert "Recompute possible fits" in resp.text
+
+
+@pytest.mark.asyncio
+async def test_dashboard_shows_only_permitted_button(
+    client, seeded, make_user, db_sessionmaker
+):
+    async with db_sessionmaker() as db:
+        perms = seeded["perms"]
+        editor = await role_crud.create(
+            db, name="fits-only-editor", permission_ids=[perms["fits:write"].id]
+        )
+        editor_id = editor.id
+    await make_user("fitsonly@example.com", role_ids=[editor_id])
+    await _web_login(client, "fitsonly@example.com")
+
+    resp = await client.get("/")
+    assert resp.status_code == 200
+    assert "Recompute possible fits" in resp.text
+    assert "Refresh from OData" not in resp.text
+    assert "Sync augmented parts" not in resp.text
+
+
+@pytest.mark.asyncio
+async def test_dashboard_refresh_odata_without_config_skips_job(
+    client, seeded, make_user, db_sessionmaker
+):
+    async with db_sessionmaker() as db:
+        perms = seeded["perms"]
+        editor = await role_crud.create(
+            db,
+            name="dash-refresh-noconfig-editor",
+            permission_ids=[perms["imported_parts:write"].id],
+        )
+        editor_id = editor.id
+    await make_user("dashrefresher2@example.com", role_ids=[editor_id])
+    await _web_login(client, "dashrefresher2@example.com")
+
+    resp = await client.post("/dashboard/refresh-odata")
+    assert resp.status_code == 200
+    assert "OData URL is not configured." in resp.text
+    assert _JOB_URL_RE.search(resp.text) is None
+
+
+@pytest.mark.asyncio
+async def test_dashboard_sync_augmented_parts_runs_in_background(
+    client, seeded, make_user, db_sessionmaker, monkeypatch
+):
+    monkeypatch.setattr("app.web.pages.AsyncSessionLocal", db_sessionmaker)
+    monkeypatch.setattr("app.services.jobs.AsyncSessionLocal", db_sessionmaker)
+    async with db_sessionmaker() as db:
+        perms = seeded["perms"]
+        editor = await role_crud.create(
+            db,
+            name="dash-sync-editor",
+            permission_ids=[perms["augmented_parts:write"].id],
+        )
+        editor_id = editor.id
+    await make_user("dashsyncer@example.com", role_ids=[editor_id])
+    await _web_login(client, "dashsyncer@example.com")
+
+    resp = await client.post("/dashboard/sync-augmented-parts")
+    assert resp.status_code == 200
+    match = _JOB_URL_RE.search(resp.text)
+    assert match, resp.text
+
+    final_body = await _poll_until_done(client, match.group(1))
+    assert "Sync complete" in final_body
+    assert "dashboard-sync-card" in final_body
+
+
+@pytest.mark.asyncio
+async def test_dashboard_recompute_fits_runs_in_background(
+    client, seeded, make_user, db_sessionmaker, monkeypatch
+):
+    monkeypatch.setattr("app.web.pages.AsyncSessionLocal", db_sessionmaker)
+    monkeypatch.setattr("app.services.jobs.AsyncSessionLocal", db_sessionmaker)
+    async with db_sessionmaker() as db:
+        perms = seeded["perms"]
+        editor = await role_crud.create(
+            db, name="dash-recompute-editor", permission_ids=[perms["fits:write"].id]
+        )
+        editor_id = editor.id
+    await make_user("dashrecomputer@example.com", role_ids=[editor_id])
+    await _web_login(client, "dashrecomputer@example.com")
+
+    resp = await client.post("/dashboard/recompute-fits")
+    assert resp.status_code == 200
+    match = _JOB_URL_RE.search(resp.text)
+    assert match, resp.text
+
+    final_body = await _poll_until_done(client, match.group(1))
+    assert "Recompute complete" in final_body
+
+
+@pytest.mark.asyncio
+async def test_dashboard_refresh_odata_runs_in_background(
+    client, seeded, make_user, db_sessionmaker, monkeypatch
+):
+    monkeypatch.setattr("app.web.pages.AsyncSessionLocal", db_sessionmaker)
+    monkeypatch.setattr("app.services.jobs.AsyncSessionLocal", db_sessionmaker)
+
+    async def _fake_fetch(url, username, password):
+        return [{"InventoryID": "INV-1", "Description": "Widget"}]
+
+    monkeypatch.setattr("app.web.pages.odata_service.fetch", _fake_fetch)
+
+    async with db_sessionmaker() as db:
+        await imported_part_config_crud.upsert(
+            db, odata_url="https://example.test/odata", username=None, password=None
+        )
+        perms = seeded["perms"]
+        editor = await role_crud.create(
+            db,
+            name="dash-refresh-editor",
+            permission_ids=[perms["imported_parts:write"].id],
+        )
+        editor_id = editor.id
+    await make_user("dashrefresher@example.com", role_ids=[editor_id])
+    await _web_login(client, "dashrefresher@example.com")
+
+    resp = await client.post("/dashboard/refresh-odata")
+    assert resp.status_code == 200
+    match = _JOB_URL_RE.search(resp.text)
+    assert match, resp.text
+
+    final_body = await _poll_until_done(client, match.group(1))
+    assert "Sync complete: 1 created, 0 updated." in final_body

tests/test_fit_matching.py

@@ -27,10 +27,10 @@ def test_iso_thread_mismatched_dimensions():
     assert not attachment_points_match(a, b)
 
 
-def test_same_label_never_matches_even_with_identical_threads():
+def test_same_label_still_matches_even_with_identical_threads():
     a = _ap(1, 1, iso_thread=IsoThread(diameter_mm=8, pitch_mm=1.25))
     b = _ap(2, 1, iso_thread=IsoThread(diameter_mm=8, pitch_mm=1.25))
-    assert not attachment_points_match(a, b)
+    assert attachment_points_match(a, b)
 
 
 def test_unc_thread_exact_match():

tests/test_ldap_login.py

@@ -92,6 +92,9 @@ async def test_ldap_login_bad_credentials(client, seeded, monkeypatch, ldap_enab
     assert resp.status_code == 401
 
 
+@pytest.mark.skip(
+    "Not required, can cause issues if running against a live LDAP server"
+)
 async def test_ldap_login_disabled_returns_404(client, seeded):
     # ldap_enabled defaults to False.
     resp = await client.post(

tests/test_login_redirect.py

@@ -3,23 +3,35 @@ import pytest
 
 @pytest.mark.asyncio
 async def test_deep_link_redirects_to_login_with_next(client):
-    resp = await client.get("/scopes")
+    resp = await client.get("/admin/users")
     assert resp.status_code == 302
-    assert resp.headers["location"] == "/login?next=%2Fscopes"
+    assert resp.headers["location"] == "/login?next=%2Fadmin%2Fusers"
 
 
 @pytest.mark.asyncio
 async def test_deep_link_preserves_query_string(client):
-    resp = await client.get("/scopes?description=alpha")
+    resp = await client.get("/admin/users?sort=email")
     assert resp.status_code == 302
-    assert resp.headers["location"] == "/login?next=%2Fscopes%3Fdescription%3Dalpha"
+    assert resp.headers["location"] == "/login?next=%2Fadmin%2Fusers%3Fsort%3Demail"
 
 
 @pytest.mark.asyncio
-async def test_root_redirect_has_no_next(client):
+async def test_root_redirects_anonymous_to_scopes(client):
     resp = await client.get("/")
     assert resp.status_code == 302
-    assert resp.headers["location"] == "/login"
+    assert resp.headers["location"] == "/scopes"
+
+
+@pytest.mark.asyncio
+async def test_root_renders_dashboard_when_logged_in(client, make_user):
+    await make_user("dash@example.com")
+    resp = await client.post(
+        "/login", data={"email": "dash@example.com", "password": "secret123"}
+    )
+    assert resp.status_code == 303
+    resp = await client.get("/")
+    assert resp.status_code == 200
+    assert "dash@example.com" in resp.text
 
 
 @pytest.mark.asyncio

tests/test_scope_guide.py

@@ -49,7 +49,6 @@ async def _seed_guide_data(db):
             magnification_min=3,
             magnification_max=18,
             objective_lens_diameter_mm=50,
-            objective_thread_pitch="M52x0.75",
         )
     )
 
@@ -147,6 +146,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:
@@ -202,7 +221,6 @@ async def test_scopes_page_renders_rows_and_fits(
     assert "Alpha 3-18x50" in body  # scope row
     assert "Flip cover 50mm" in body  # a matching part in a fit column
     assert "fit-badge-exact" in body  # the exact fit badge
-    assert "M52x0.75" in body  # scope info: objective thread
     assert "3-18" in body  # magnification range, integral values un-padded
 
 
@@ -274,9 +292,7 @@ async def test_scopes_page_renders_objective_attachment_point_thread(
     resp = await client.get("/scopes")
     assert resp.status_code == 200
     body = resp.text
-    # The attachment point's thread spec wins over ScopeInfo.objective_thread_pitch.
     assert "M44.0×0.5" in body
-    assert "M52x0.75" not in body
 
 
 @pytest.mark.asyncio
@@ -325,8 +341,212 @@ async def test_scopes_page_forbidden_without_permission(client, seeded, make_use
 
 
 @pytest.mark.asyncio
-async def test_scopes_page_redirects_when_anonymous(client):
+async def test_scopes_page_renders_for_anonymous(client, db_sessionmaker):
+    async with db_sessionmaker() as db:
+        await _seed_guide_data(db)
     resp = await client.get("/scopes")
-    assert resp.status_code == 302
+    assert resp.status_code == 200
-    # Deep link is preserved so the user lands back here after logging in.
+    body = resp.text
-    assert resp.headers["location"] == "/login?next=%2Fscopes"
+    assert "Alpha 3-18x50" in body
+    assert "fit-badge-exact" in body
+    # Possible fits (e.g. the ARD) are hidden entirely from anonymous visitors.
+    assert "ARD 50mm" not in body
+    assert "fit-badge-possible" not in body
+
+
+@pytest.mark.asyncio
+async def test_scopes_page_hides_write_controls_for_anonymous(client, db_sessionmaker):
+    async with db_sessionmaker() as db:
+        await _seed_guide_data(db)
+    resp = await client.get("/scopes")
+    assert resp.status_code == 200
+    assert 'name="scope_ids"' not in resp.text
+    assert "delete-selected-scopes-btn" not in resp.text
+
+
+@pytest.mark.asyncio
+async def test_scopes_filter_endpoint_works_for_anonymous(client, db_sessionmaker):
+    async with db_sessionmaker() as db:
+        await _seed_guide_data(db)
+    resp = await client.get("/scopes/filter", params={"description": "Alpha"})
+    assert resp.status_code == 200
+    assert "Alpha 3-18x50" in resp.text
+    assert "Zeta 1-6x24" not in resp.text
+
+
+@pytest.mark.asyncio
+async def test_build_only_exact_drops_possible_fits(db_sessionmaker):
+    async with db_sessionmaker() as db:
+        ids = await _seed_guide_data(db)
+        guide = await scope_guide.build(db, only_exact=True)
+
+    cover_id, ard_id = ids["cover"].id, ids["ard"].id
+    rows = {r["scope"].description: r for r in guide["rows"]}
+
+    s1 = rows["Alpha 3-18x50"]
+    assert [(e["part"].description, e["label"]) for e in s1["cells"][cover_id]] == [
+        ("Flip cover 50mm", "exact")
+    ]
+    assert ard_id not in s1["cells"]  # the ARD fit was only "possible"
+
+    s2 = rows["Zeta 1-6x24"]
+    assert cover_id not in s2["cells"]  # S2's flip-cover fit was only "possible"
+
+
+@pytest.mark.asyncio
+async def test_scopes_page_hides_checkboxes_for_read_only_user(
+    client, seeded, make_user, db_sessionmaker
+):
+    async with db_sessionmaker() as db:
+        await _seed_guide_data(db)
+    await make_user("reader6@example.com", role_ids=[seeded["roles"]["user"].id])
+    await _web_login(client, "reader6@example.com")
+    resp = await client.get("/scopes")
+    assert resp.status_code == 200
+    assert 'name="scope_ids"' not in resp.text
+    assert "delete-selected-scopes-btn" not in resp.text
+
+
+@pytest.mark.asyncio
+async def test_scopes_page_shows_checkboxes_for_write_user(
+    client, seeded, make_user, db_sessionmaker
+):
+    async with db_sessionmaker() as db:
+        ids = await _seed_guide_data(db)
+    await make_user("writer@example.com", is_superuser=True)
+    await _web_login(client, "writer@example.com")
+    resp = await client.get("/scopes")
+    assert resp.status_code == 200
+    assert f'name="scope_ids" value="{ids["s1"].id}"' in resp.text
+    assert "delete-selected-scopes-btn" in resp.text
+    assert 'hx-post="/scopes/bulk-delete"' in resp.text
+
+
+@pytest.mark.asyncio
+async def test_bulk_delete_scopes_removes_selected_augmented_parts(
+    client, seeded, make_user, db_sessionmaker
+):
+    async with db_sessionmaker() as db:
+        ids = await _seed_guide_data(db)
+    await make_user("writer2@example.com", is_superuser=True)
+    await _web_login(client, "writer2@example.com")
+
+    resp = await client.post(
+        "/scopes/bulk-delete",
+        data={"scope_ids": [str(ids["s1"].id), str(ids["s2"].id)]},
+    )
+    assert resp.status_code == 200
+
+    async with db_sessionmaker() as db:
+        from app.crud import augmented_part as augmented_part_crud
+
+        assert await augmented_part_crud.get(db, ids["s1"].id) is None
+        assert await augmented_part_crud.get(db, ids["s2"].id) is None
+
+
+@pytest.mark.asyncio
+async def test_bulk_delete_scopes_leaves_unselected_rows(
+    client, seeded, make_user, db_sessionmaker
+):
+    async with db_sessionmaker() as db:
+        ids = await _seed_guide_data(db)
+    await make_user("writer3@example.com", is_superuser=True)
+    await _web_login(client, "writer3@example.com")
+
+    resp = await client.post(
+        "/scopes/bulk-delete", data={"scope_ids": [str(ids["s1"].id)]}
+    )
+    assert resp.status_code == 200
+
+    async with db_sessionmaker() as db:
+        from app.crud import augmented_part as augmented_part_crud
+
+        assert await augmented_part_crud.get(db, ids["s1"].id) is None
+        assert await augmented_part_crud.get(db, ids["s2"].id) is not None
+
+
+@pytest.mark.asyncio
+async def test_scopes_page_shows_single_page_summary(
+    client, seeded, make_user, db_sessionmaker
+):
+    async with db_sessionmaker() as db:
+        await _seed_guide_data(db)  # 2 scopes, well under the default page size
+    await make_user("pager1@example.com", role_ids=[seeded["roles"]["user"].id])
+    await _web_login(client, "pager1@example.com")
+    resp = await client.get("/scopes")
+    assert resp.status_code == 200
+    assert "Showing 2 records." in resp.text
+    assert "Next" not in resp.text
+
+
+@pytest.mark.asyncio
+async def test_scopes_filter_paginates_results(
+    client, seeded, make_user, db_sessionmaker
+):
+    async with db_sessionmaker() as db:
+        scope_t = PartType(name="Scope")
+        db.add(scope_t)
+        await db.flush()
+        for i in range(5):
+            db.add(
+                AugmentedPart(
+                    description=f"Scope {i:02d}",
+                    colour_id=1,
+                    manufacturer_id=1,
+                    part_type_id=scope_t.id,
+                )
+            )
+        await db.commit()
+    await make_user("pager2@example.com", role_ids=[seeded["roles"]["user"].id])
+    await _web_login(client, "pager2@example.com")
+
+    resp = await client.get("/scopes/filter", params={"page_size": 2})
+    assert resp.status_code == 200
+    body = resp.text
+    assert "Scope 00" in body
+    assert "Scope 01" in body
+    assert "Scope 02" not in body
+    assert "Showing 1–2 of 5." in body
+
+    resp2 = await client.get("/scopes/filter", params={"page_size": 2, "page": 3})
+    assert resp2.status_code == 200
+    body2 = resp2.text
+    assert "Scope 04" in body2
+    assert "Scope 00" not in body2
+    assert "Showing 5–5 of 5." in body2
+
+
+@pytest.mark.asyncio
+async def test_scopes_filter_clamps_out_of_range_page(
+    client, seeded, make_user, db_sessionmaker
+):
+    async with db_sessionmaker() as db:
+        await _seed_guide_data(db)  # 2 scopes
+    await make_user("pager3@example.com", role_ids=[seeded["roles"]["user"].id])
+    await _web_login(client, "pager3@example.com")
+
+    resp = await client.get("/scopes/filter", params={"page_size": 1, "page": 99})
+    assert resp.status_code == 200
+    # Clamped to the last real page (2), not an empty page 99.
+    assert "Zeta 1-6x24" in resp.text
+    assert "Alpha 3-18x50" not in resp.text
+
+
+@pytest.mark.asyncio
+async def test_bulk_delete_scopes_forbidden_without_write_permission(
+    client, seeded, make_user, db_sessionmaker
+):
+    async with db_sessionmaker() as db:
+        ids = await _seed_guide_data(db)
+    await make_user("reader7@example.com", role_ids=[seeded["roles"]["user"].id])
+    await _web_login(client, "reader7@example.com")
+
+    resp = await client.post(
+        "/scopes/bulk-delete", data={"scope_ids": [str(ids["s1"].id)]}
+    )
+    assert resp.status_code == 403
+
+    async with db_sessionmaker() as db:
+        from app.crud import augmented_part as augmented_part_crud
+
+        assert await augmented_part_crud.get(db, ids["s1"].id) is not None

tests/test_scope_import.py

@@ -1,5 +1,6 @@
 import io
 
+import pytest
 from openpyxl import Workbook
 
 from app.crud import attachment_point as attachment_point_crud
@@ -26,7 +27,6 @@ HEADERS = [
     "url",
     "ocular_outer_diameter_mm",
     "ocular_fit",
-    "objective_thread_pitch",
     "objective_thread",
     "objective_fit",
 ]
@@ -59,7 +59,6 @@ def _scope_row(**overrides):
         "url": "https://example.com/viper",
         "ocular_outer_diameter_mm": 40,
         "ocular_fit": None,
-        "objective_thread_pitch": "0.75",
         "objective_thread": "M40x0.75",
         "objective_fit": None,
     }
@@ -142,7 +141,6 @@ async def test_import_creates_scope_with_info_and_attachment_points(db_sessionma
         assert info.magnification_max == 25
         assert info.model_suffix == "PST"
         assert info.objective_lens_diameter_mm == 56
-        assert info.objective_thread_pitch == "0.75"
 
         aps = await attachment_point_crud.list_for_augmented_part(db, part.id)
         by_label = {}
@@ -203,6 +201,9 @@ async def test_import_upserts_on_product_number(db_sessionmaker):
         assert len(aps) == 2
 
 
+@pytest.mark.skip(
+    "Product numbers are now autovivified from model and other attributes"
+)
 async def test_row_without_product_number_is_skipped(db_sessionmaker):
     async with db_sessionmaker() as db:
         data = _workbook([_scope_row(product_number=None)])
@@ -234,6 +235,90 @@ async def test_unknown_fit_reference_is_reported(db_sessionmaker):
         assert await fit_crud.list_for_part(db, part.id) == []
 
 
+async def _make_bare_part(db, sku):
+    """A non-scope part with no attachment points at all."""
+    colour = await part_colour_crud.get_by_name(db, "Black")
+    if colour is None:
+        colour = await part_colour_crud.create(db, name="Black")
+    manu = await manufacturer_crud.get_by_name(db, "Armament")
+    if manu is None:
+        manu = await manufacturer_crud.create(db, name="Armament", url="")
+    pt = await part_type_crud.get_by_name(db, "Flip Cover")
+    if pt is None:
+        pt = await part_type_crud.create(db, name="Flip Cover")
+    return await augmented_part_crud.create(
+        db,
+        sku=sku,
+        description=f"Accessory {sku}",
+        colour_id=colour.id,
+        manufacturer_id=manu.id,
+        scope_model_id=None,
+        part_type_id=pt.id,
+        sub_part_type_id=None,
+    )
+
+
+async def test_fit_reference_without_attachment_points_creates_matching_one(
+    db_sessionmaker,
+):
+    async with db_sessionmaker() as db:
+        await _make_bare_part(db, "ACC-BARE")
+
+        data = _workbook([_scope_row(ocular_fit="ACC-BARE")])
+        result = await scope_import_service.import_from_bytes(db, data)
+        assert result.created == 1
+        msgs = " ".join(result.rows[0].messages)
+        assert "created one matching the scope's attachment point" in msgs
+
+        acc = await augmented_part_crud.get_by_sku(db, "ACC-BARE")
+        acc_aps = await attachment_point_crud.list_for_augmented_part(db, acc.id)
+        assert len(acc_aps) == 1
+        assert acc_aps[0].static_connection is not None
+        assert acc_aps[0].static_connection.od_mm == 40
+        assert acc_aps[0].attachment_point_name.name == "Ocular"
+
+        part = await augmented_part_crud.get_by_sku(db, "SCOPE-A")
+        fits = await fit_crud.list_for_part(db, part.id)
+        assert len(fits) == 1
+        linked_ap_ids = {fits[0].attachment_point_a_id, fits[0].attachment_point_b_id}
+        assert acc_aps[0].id in linked_ap_ids
+
+
+async def test_scope_fit_without_scope_attachment_point_creates_manual_one(
+    db_sessionmaker,
+):
+    async with db_sessionmaker() as db:
+        _, acc_ap = await _make_accessory(db, "ACC-1", od_mm=40)
+
+        # No ocular_outer_diameter_mm, so the scope itself has no Ocular AP,
+        # but an ocular_fit reference is still given.
+        data = _workbook(
+            [_scope_row(ocular_outer_diameter_mm=None, ocular_fit="ACC-1")]
+        )
+        result = await scope_import_service.import_from_bytes(db, data)
+        assert result.created == 1
+        msgs = " ".join(result.rows[0].messages)
+        assert "created a manual one to record the fit" in msgs
+
+        part = await augmented_part_crud.get_by_sku(db, "SCOPE-A")
+        scope_aps = await attachment_point_crud.list_for_augmented_part(db, part.id)
+        by_label = {ap.attachment_point_name.name: ap for ap in scope_aps}
+        assert "Ocular" in by_label
+        ocular_ap = by_label["Ocular"]
+        assert ocular_ap.iso_thread is None
+        assert ocular_ap.unc_thread is None
+        assert ocular_ap.clamp_connection is None
+        assert ocular_ap.static_connection is None
+        # Objective still gets its normal thread-based AP.
+        assert "Objective" in by_label
+
+        fits = await fit_crud.list_for_part(db, part.id)
+        assert len(fits) == 1
+        linked_ap_ids = {fits[0].attachment_point_a_id, fits[0].attachment_point_b_id}
+        assert ocular_ap.id in linked_ap_ids
+        assert acc_ap.id in linked_ap_ids
+
+
 async def test_unc_objective_thread_creates_unc_attachment_point(db_sessionmaker):
     async with db_sessionmaker() as db:
         data = _workbook([_scope_row(objective_thread="1.750-32 UNS")])