Repository navigation
normalizer: a Watch band's size can key a watch title (follow-up to #1113) #1124
Description
Follow-up from the review of #1121 (#1113), left open there as P2.
Since #1121 a Watch Series or SE title keys with its case size (apple:watch-series-11-46mm:64gb). readAppleSize takes the first NNmm after the word "Watch", so a title naming a band size before the case size keys at the band's:
Apple Watch Series 10 dây Sport Band 45mm cho mặt 42mm->apple:watch-series-10-45mm:...
It is ambiguous whether such a title sells the band or the watch; if it is the band, it should be rejected as an accessory rather than keyed.
Task
- Decide from real listing titles whether a size after "dây"/"band"/"strap" belongs to the band, and skip it (or reject the title as an accessory) when it does.
- Add labeled fixtures in
apps/importer/src/shopee/__tests__/open-core-intelligence.test.ts.
Acceptance
- The title above does not key with
-45mm. - Existing
appleSizeKeys1113fixtures still pass.
Activity
JOY commentedon Oct 27, 2026
Claimed by Core (sub-session of the Rate Core PO). Plan: read real Watch titles read-only (public rate.vn API and existing fixtures), decide whether a size after "dây"/"band"/"strap" belongs to the band, then skip it or reject the title as an accessory in
readAppleSize, with labeled fixtures inopen-core-intelligence.test.ts.fix(normalizer): keep a lone band size as the Watch case size (#1124)
test(normalizer): pin each Watch band word and separator (#1124)
fix(normalizer): a Watch band's size no longer keys the watch (#1124) (…
JOY commentedon Oct 27, 2026
[Agent: PO] Released to production in #1128 (main c23ab85). verify-live: https://rate.vn serves main c23ab85, database ok, 5 routes 200.