Bryce
5126f3799d
Recategorizes 131 Sysco line-item descriptions per the client-reviewed "Product Category Verification with codes" sheet: 96 existing rows re-coded and 35 new rows appended (ids 1796-1830). Root cause this addresses: get-line-account matches on exact description string and silently defaults anything unmapped to 50000 Food Costs. Only 147 of the sheet's 321 reviewed rows coded the way the client expected. Note the sheet's "change" column understates the work -- 20 of its 53 change rows are no-ops (Paper -> Paper, confirming the gloves/liners/hairnets) and 3 were already fixed in38575aa5/7a0e256f, while 214,398 lines of movement come from rows the client ticked as correct against a suggestion that already differed from production. Moves, replayed over all 1,022,732 DET lines in sysco-poller: 50000 -> 51500 Dry Goods 90,481 ln $5,215,011.82 105 clients 50000 -> 51450 Dressing & Sauce 58,191 ln $4,445,726.56 98 50000 -> 51400 Bread and Bun 37,705 ln $3,964,462.24 96 50000 -> 52000 Soft Beverage 36,244 ln $1,033,497.52 94 50000 -> 51200 Produce 6,519 ln $460,629.32 98 55000 -> 51500 Dry Goods 6,516 ln $259,750.60 97 50000 -> 74100 Cleaning Supplies 5,630 ln $205,233.19 98 55000 -> 74100 Cleaning Supplies 5,222 ln $125,946.32 99 50000 -> 51120 Chicken/Poultry 265 ln $42,482.50 8 50000 -> 51300 Dairy 36 ln $5,941.69 6 50000 -> 55000 Paperware 54 ln $1,751.22 18 54400 -> 51450 Dressing & Sauce 24 ln $1,413.26 1 total 246,887 ln $15,761,846.24 Only three source accounts are touched: 50000 and 55000 (the two silent defaults) plus the single intended 54400 -> 51450 vinaigrette row. Nothing else leaves a deliberately assigned account. The 7 Misc Charges descriptions are deliberately left alone per Bryce, including PICKLE CHIP KOSH 1/4 KK, which therefore stays at the 50000 default rather than moving to Produce as the sheet originally suggested. Also corrects the PAPER & DISP fallback comment in sysco.clj: 440 of its 455 mapped descriptions point at 55000, not all 454. The 15 exceptions (foil pans -> 51500, scour pads -> 74100) are mapped explicitly, so the description map still wins ahead of the fallback. No logic changed. Affects only clients with the code-sysco-items feature flag, and only at import time -- already-imported invoices keep their existing splits. Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>