Code review - review.py directly violates the spec requirements that all metadata values derive from the woo sync per caching rules, and are never hardcoded #98

New Issue

2026-09-21T18:33:22Z

Ty commented

2026-09-21 18:33:22 +00:00

this file hardcodes values for attributes that are potentially dynamic and live in the SOT of Woocommerce. The current implementation is a violation of those requirements.

grokbuild_ty referenced this issue from a commit

2026-09-21 19:51:29 +00:00

fix(m3): Option A must-fix #79 #82 #83 #84 #89 #98

grokbot_ty referenced this issue from a commit

2026-09-21 19:55:28 +00:00

fix(m3): Option A must-fix #79 #82 #83 #84 #89 #98

grokbot_ty closed this issue

2026-09-21 19:59:39 +00:00

grokbot_ty referenced this issue

2026-09-21 19:59:53 +00:00

[M3] Publish path omits User-Agent and reports a WAF 403 as retryable — the b99cbdf fix does not cover product create #79

grokbot_ty referenced this issue

2026-09-21 19:59:54 +00:00

[M3] stale_field error returns the submitted value as current_options instead of the live taxonomy options (PUBLISH-20) #82

grokbot_ty referenced this issue

2026-09-21 19:59:55 +00:00

[M3] POST /slabs: an extra payload key breaks the INSERT binding count; unvalidated columns/values raise raw DB errors instead of 400 #83

grokbot_ty referenced this issue

2026-09-21 19:59:55 +00:00

[M3] /api/health hardcodes woo_reachable: false and the version string is still 0.5.0-m3-s0 #84

grokbot_ty referenced this issue

2026-09-21 19:59:55 +00:00

Bug: Call 1 + human lock of taxonomy must complete before deterministic title/short desc #89

grokbot_ty commented

2026-09-21 19:59:56 +00:00

Closed by M3 Option A: main 0c00d25 + UAT QUALITY-110 evidence.

Deploy: version 0.5.0-m3, woo_reachable: true
test-woo ok (taxonomy synced)
Inference off for the walk; SLAB-UAT-125846 → Woo draft product_id=48147 matching reviewed ready set
Must-fix covered: #79 UA/403, #82 live current_options, #83 unknown-field/taxonomy 4xx, #84 health probe+version, #89 Create title/short from review, #98 live-edge from synced edge term + attr slug resolve

Closed by M3 Option A: main `0c00d25` + UAT QUALITY-110 evidence. - Deploy: version `0.5.0-m3`, `woo_reachable: true` - `test-woo` ok (taxonomy synced) - Inference off for the walk; **SLAB-UAT-125846** → Woo draft `product_id=48147` matching reviewed ready set - Must-fix covered: #79 UA/403, #82 live `current_options`, #83 unknown-field/taxonomy 4xx, #84 health probe+version, #89 Create title/short from review, #98 live-edge from synced edge term + attr slug resolve

Sign in to join this conversation.

2 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: Ty_Tech/SlabUploader#98