[futurebuildai]

A yard's stock is its own inventory row plus every zone and bin whose branch_id is that yard. The dashboard's Inventory Alerts read one row per location, so a yard holding 112 on its own row and 300 in a zone under it (412 against a reorder point of 150) was reported "Current: 112, Low Stock".

The fix

GetInventoryAlerts groups by product and yard, sums the quantity, compares the sum with the reorder point and reports it. location_id is now the yard (the branch). The serve wire test that read it expects the yard; the contract description and the CONTRACT-CHANGES row say so.

Red first

TestGetInventoryAlerts_SumsTheYardsOwnRowAndItsZones (commit 1, alone): a yard with 112 on its own row and 300 in a zone under it against a reorder point of 150 gives no alert; cut the zone to 20 and the alert reports 132 as LOW_STOCK at the yard. At commit 1 it fails: yard holds 412 against a reorder point of 150, want no alert, got {... CurrentQty:112 ... LOW_STOCK}.

Readers and verdicts

Reader	Verdict
Dashboard inventory alerts (dashboard.GetInventoryAlerts)	Fixed here, with its leg
Counter search, tiles, catalogue (pos counterStockScope)	Already right: sums rows by locations.branch_id
Product list and product page stock (product stockJoin)	Already right: sums on hand and allocated over the caller's branches
Reorder list (product.ListBelowReorder)	Already right: sums every row in the caller's branches against the reorder point
Reorder recommendations and purchasing suggestions (purchase_order recommendations, via inventory.ListByProduct)	Already right: sums on hand less allocated over the rows in the caller's reach
Portal availability (portal catalog, via inventory.ListByProduct)	Already right: the same sum
Stock lookup, levels list (inventory.ListLevelsPage, ListInventoryByProduct) and the bin picker	Per location by design (a row per bin); not a yard total
Stock across yards (the product's Stock & Locations tab)	Already right: sums the levels rows of the caller's yard
Market check's first pick	No stock reader of its own in the server; the picker lists products through the product list's summed stock
Reporting builder's inventory source	Per location rows by design (the report groups or sums as the author defines it)
Write paths (GetInventory, allocate, fulfil, LockBranchInventory)	Per row by design; an absolute count with no location is now refused where the product sits in more than one location (round 1)
Cycle count page (pages/yard/CycleCount.ts)	Read the yard's sum and wrote it to one row: wrong with zone stock. Fixed in round 1: it counts by location and names each location
Seeds and product/unitset.go (FROM inventory)	Not stock readers for a person: seed deletes and a unit change lock

Gates

Go build, vet, gofmt; dashboard, inventory, product, pos, purchase_order, portal, reporting, apicontract, serve (race) and characterization packages against a fresh database; merge -check, cmd/pending, api-client generate, drift and typecheck; check-shape.

Review round 1

• P2-1, the count by location. The Cycle count lists each location with its expected figure and writes each count to its own location (a found pallet may be added); the adjust route refuses an absolute count with no location where the product is in several, 422 location_required. Red legs: go test ./internal/inventory -run TestCycleCount (count with no location = 200, want 422) and vitest src/pages/yard/CycleCount.test.ts.
• P3s. The alert leg's second yard is stocked and an unwalled call proves two rows each at its own yard; a legacy row stays its own alert; each alert carries yard_name and the widget shows it; the contract text says what a branch caller sees for a product its yard holds no row of. Whether reorder is per yard or per company is left to a decision.

Review round 2

• P2-1: an absolute count that names no location is refused 422 location_required whenever the caller has no branch context (administrator with no header, unbound key, granted user with no header), and where the product sits in several locations in reach; the guard reads through the caller's branch or grants. Legs: go test ./internal/inventory -run TestCount_NoLocationIsRefused (three callers 200, want 422 at their first commit); a caller bound to the yard with one row still counts.
• P3s: a delta the oldest row cannot take is 422 insufficient_stock (was a bare 500); a count under the allocated figure is 422 below_allocated, and the page warns and holds the submit; a scan adds to the location the person chose (asks which when there are several); "Found elsewhere" offers only the yard's own locations; the failed-adjustment toast carries the server's words; the alert grouping drops a term that changed nothing.

🤖 Generated with Claude Code