docs: capture 4 todos - storage tests, image bugs, manufacturer entity

.planning/todos/pending/2026-04-10-add-manufacturer-entity-with-brand-details.md

@@ -0,0 +1,18 @@
+---
+created: 2026-04-10T09:23:46.394Z
+title: Add manufacturer entity with brand details
+area: database
+files:
+  - src/db/schema.ts
+  - src/server/services/global-item.service.ts
+---
+
+## Problem
+
+The manual item adding form doesn't include detailed manufacturer info. Currently `brand` is just a text field on items and globalItems. There's no structured manufacturer data (logo, website, country, description). Users can't select from existing manufacturers — they type freeform text which leads to inconsistencies ("Ortlieb" vs "ORTLIEB" vs "ortlieb").
+
+## Solution
+
+Create a `manufacturers` table with fields: `id`, `name` (unique), `logoUrl`, `websiteUrl`, `country`, `description`, `createdAt`. Add a `manufacturerId` FK on `globalItems` (and potentially `items`). Build a manufacturer picker component (like CategoryPicker) with search and inline create. Update the manual add form and catalog enrichment to use the manufacturer entity. This is a significant feature — likely needs its own phase.
+
+Note: This would also improve the MCP agent seeding workflow — agents could create manufacturers first, then reference them when creating catalog items.

.planning/todos/pending/2026-04-10-fix-item-image-not-showing-on-collection-overview.md

@@ -0,0 +1,16 @@
+---
+created: 2026-04-10T09:23:46.394Z
+title: Fix item image not showing on collection overview
+area: ui
+files:
+  - src/client/routes/collection.tsx
+  - src/client/components/ItemCard.tsx
+---
+
+## Problem
+
+When adding an image to an item, the image is not displayed on the collection overview page (grid/list view). The image only appears when viewing the item's detail page. This suggests the collection overview query or component isn't fetching/passing the image URL, or the card component isn't rendering it.
+
+## Solution
+
+Check the collection overview query — does it include `imageUrl`/`imageFilename` in the response? Check the ItemCard component — does it render the image when present? May need to ensure the presigned S3 URL is being generated for the collection list endpoint, not just the detail endpoint.

.planning/todos/pending/2026-04-10-fix-storage-service-tests.md

@@ -0,0 +1,16 @@
+---
+created: 2026-04-10T09:23:46.394Z
+title: Fix storage service tests
+area: testing
+files:
+  - tests/services/storage.service.test.ts
+  - src/server/services/storage.service.ts
+---
+
+## Problem
+
+15 tests in the storage service test file are failing with `SyntaxError: Export named 'withImageUrl' not found in module 'src/server/services/storage.service.ts'`. This is a pre-existing issue that predates Phase 25. The `withImageUrl` export was likely renamed or removed but tests still reference it.
+
+## Solution
+
+Investigate the `storage.service.ts` exports, find what `withImageUrl` was renamed to (possibly `withImageUrls` based on recent Phase 24 commit `e1afd54`), and update the test imports to match.

.planning/todos/pending/2026-04-10-investigate-slow-image-loading.md

@@ -0,0 +1,16 @@
+---
+created: 2026-04-10T09:23:46.394Z
+title: Investigate slow image loading
+area: ui
+files:
+  - src/server/services/storage.service.ts
+  - src/client/components/ItemCard.tsx
+---
+
+## Problem
+
+Images seem to load slowly across the app. This could be due to presigned URL generation happening on every request, large original images being served without resizing, or no caching headers on S3 responses.
+
+## Solution
+
+Investigate: Are presigned URLs being cached or regenerated on every API call? Are images being served at original resolution? Consider: presigned URL caching with TTL, image thumbnails for list views, appropriate Cache-Control headers on S3 objects, lazy loading for off-screen images.