wiggleverse/wiggleverse-ecomm
2
0
Fork 0
Code Issues 5 Pull Requests Actions Packages Projects Releases Wiki Activity

SLICE-5: products import spine — canonical CSV → preview → confirm (SD-0002 §7.2) #26

Merged
ben.stull merged 21 commits from worktree-slice-5-import-spine into main 2026-06-12 03:50:21 +00:00
Conversation 0 Commits 21 Files Changed 50
+6735 -31
ben.stull commented 2026-06-12 00:07:42 +00:00
Owner
Copy Link
Copy Source

Implements SD-0002 §7.2 SLICE-5 (anchor #13, epic #12): the products import spine, canonical dialect only, images stubbed.

Ships

  • products domain + §6.3 tables (migration 0002; images table present, fetch phase stubbed to no-op)
  • Canonical CSV codec (INV-18 caps), §6.5.1 row validation (INV-15 sanitization), diff engine with INV-11 fingerprint + apply plan
  • One-transaction confirm/apply (INV-10 upsert-only), import drafts/runs, TEL-1/2/6 telemetry
  • §6.4 endpoints: imports, draft records/confirm/cancel, runs, summary, sample.csv (DOC-3)
  • Admin Products section (§5.2–5.5): Products page + history, upload, preview with drill-in diffs (the BUC-2 consent gate), run detail; hash routing
  • First Playwright E2E suite: e2e_import_preview_confirm, e2e_import_errors_actionable, e2e_import_file_rejected, e2e_import_cancel_no_trace — all green
  • DOC-1 docs/OPERATIONS.md (RB-2 + ALR-2 gesture), DOC-4 docs/products-domain.md; version 0.5.0

DoD (§7.2): BUC-1/1a/2 canonical path ✓ · INV-10/11/13/14/15/18 enforced + tested (147 backend / 10 vitest / 4 E2E green) · TEL-1/2/6 emitting ✓ · ALR-2 gesture documented, run at PPE deploy ✓

Session: ecomm 0032. Export/round-trip = SLICE-6; images pipeline = SLICE-7; Shopify dialect = SLICE-8.

🤖 Generated with Claude Code

Implements SD-0002 §7.2 SLICE-5 (anchor #13, epic #12): the products import spine, canonical dialect only, images stubbed. **Ships** - `products` domain + §6.3 tables (migration 0002; images table present, fetch phase stubbed to no-op) - Canonical CSV codec (INV-18 caps), §6.5.1 row validation (INV-15 sanitization), diff engine with INV-11 fingerprint + apply plan - One-transaction confirm/apply (INV-10 upsert-only), import drafts/runs, TEL-1/2/6 telemetry - §6.4 endpoints: imports, draft records/confirm/cancel, runs, summary, sample.csv (DOC-3) - Admin Products section (§5.2–5.5): Products page + history, upload, preview with drill-in diffs (the BUC-2 consent gate), run detail; hash routing - First Playwright E2E suite: `e2e_import_preview_confirm`, `e2e_import_errors_actionable`, `e2e_import_file_rejected`, `e2e_import_cancel_no_trace` — all green - DOC-1 `docs/OPERATIONS.md` (RB-2 + ALR-2 gesture), DOC-4 `docs/products-domain.md`; version 0.5.0 **DoD (§7.2)**: BUC-1/1a/2 canonical path ✓ · INV-10/11/13/14/15/18 enforced + tested (147 backend / 10 vitest / 4 E2E green) · TEL-1/2/6 emitting ✓ · ALR-2 gesture documented, run at PPE deploy ✓ Session: ecomm 0032. Export/round-trip = SLICE-6; images pipeline = SLICE-7; Shopify dialect = SLICE-8. 🤖 Generated with [Claude Code](https://claude.com/claude-code)
ben.stull added 21 commits 2026-06-12 00:07:43 +00:00
docs: SLICE-5 implementation plan (SD-0002 §7.2 import spine) 1267d4f29d 
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
feat(products): SD-0002 §6.3 schema — migration 0002 + nh3/multipart deps (SLICE-5) 0ee948b34d 
Also updates test_migrate_from_empty_applies_0001 → test_migrate_from_empty_applies_all
to assert both migrations apply on a fresh DB (mechanical consequence of adding 0002).

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
feat(products): domain skeleton — errors, column registry, canonical row model c39bbd4728 
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
feat(products): CSV codec — file-level parse + INV-18 caps (PUC-5a) f64c3fddf9 
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
feat(products): row validation — §6.5.1 rules + INV-15 sanitization 19ee695c20 
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
feat(products): diff engine — add/update/unchanged/error + INV-11 fingerprint 9bc6e4dbd2 
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
fix(products): honest position compare for matched variants in diff 667a462e0b 
A Variant Position column previously produced a spurious update with a
false before:None (CatalogVariant keeps position as an attribute, not in
fields{}). Compare against the attribute, like images already do.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
feat(products): repo — catalog snapshot, draft/run SQL, upsert primitives 138126ab17 
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
feat(products): import_validate → draft, preview records, discard + TEL-1 fcbf1393f5 
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
feat(products): confirm/apply in one transaction, runs, summary — INV-10/11/14 + TEL-2/6 0c7865e9e1 
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
fix(products): cleared Variant Position resolves to file order, not NULL a8538d3ecc 
A blank Variant Position cell previewed position->null and aborted the
confirm transaction (variant.position is NOT NULL). Resolve the clear to
the variant's file order at diff time so preview and apply stay in
lockstep; carry file_order on VariantPlan instead of an identity-keyed
map; release the read snapshot before PreviewStale/NothingToApply.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
feat(products): /api/products/* BFF endpoints + sample.csv (DOC-3) 4dacc2dafd 
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
feat(products-ui): typed products API client + admin hash routing b5dac8886f 
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
feat(products-ui): admin nav + Products page with import history (§5.2) 26f84cb916 
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
feat(products-ui): import upload screen (§5.3, PUC-2/5a) 188494272a 
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
feat(products-ui): import preview — tiles, drill-in diffs, confirm/cancel (§5.4) 4141285b89 
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
feat(products-ui): import run detail (§5.5, PUC-4/5/8) 87a56a66c4 
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
fix(products-ui): review pass — show-more race, keyboard history access, a11y glyphs/aria-live, label consistency d18a84e135 
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
test(e2e): stand up Playwright — fresh-DB server harness + sign-up helper a58f42cf86 
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
test(e2e): SLICE-5 scenarios — preview/confirm, errors, rejection, cancel (§6.8) ad738adbc0 
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
docs(products): operator guide (DOC-1) + domain notes (DOC-4); version 0.5.0
ci / check (push) Has been cancelled
Details
ci / check (pull_request) Has been cancelled
Details
2606fbf826 
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
ben.stull merged commit 011f4d5dc1 into main 2026-06-12 03:50:21 +00:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
priority/P0
Priority: highest — drop everything
 priority/P1
Priority: high
 priority/P2
Priority: medium
 priority/P3
Priority: low / someday
 type/bug
Type: a defect in shipped behaviour — small leaf work, straight to a plan
 type/epic
Type: a large body of work spanning many features
 type/feature
Type: a deliverable capability under an epic
 type/story
Type: a user story — the smallest valuable increment
 type/task
Type: a concrete unit of work under a story
No Label
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
ben.stull (Ben Stull) ohm-bot
No Assignees
1 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: wiggleverse/wiggleverse-ecomm#26
Delete Branch "worktree-slice-5-import-spine"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?