Deploy AIOS web (React glide grid + FastAPI slice)

RELEASES.json

@@ -1,5 +1,5 @@
 {
- "current": "76fb302",
+ "current": "4089c02",
  "releases": [
   {
    "version": "v53",

VERSION

@@ -1 +1 @@
-76fb302
+4089c02

api/routes_products.py

@@ -309,6 +309,21 @@ def _measure_condition_sets(views, offer, rows_src, team_id, today):
             for rid, codes in sets.items()}
 
 
+def _hydrate_effective_choice_options(fields, rows_src, overlays, derived):
+    """Project selectable values from this caller's effective Product rows.
+
+    The Product table applies supplier-master defaults after overlay rows are built.  Choice
+    hydration must use that same effective value, otherwise the picker misses historic master
+    data even while the cell visibly contains it.
+    """
+    import aios_grid
+
+    effective_rows = _seed_shared(
+        aios_grid.rows_from_pool(rows_src, fields, overlays, derived=derived), rows_src, fields)
+    return aios_grid.hydrate_choice_options_from_rows(
+        fields, effective_rows, ignored_by_field={"supplier": {"(none)"}})
+
+
 def product_assembly(session: Session, scope: str = "product", storage_key: str = "",
                      consume_corrections: bool = True):
     """The product topic's mirror of `routes_customers.grid_assembly` — SAME g-dict keys, so
@@ -422,6 +437,10 @@ def product_assembly(session: Session, scope: str = "product", storage_key: str
 
     from core import user_tables as _zones
     fields = _zones.with_zone_field("product_data", fields, st=session.runtime)
+    # Existing supplier-master defaults and imported overlay cells predate W43's write-time
+    # choice hydration.  Build their effective, already-walled row values before returning the
+    # field contract so a populated select never opens as "no choices yet".
+    fields = _hydrate_effective_choice_options(fields, rows_src, ws.get("overlays"), derived)
     return {"rows_src": rows_src, "pids": pids, "ws": ws, "workspace": workspace,
             "fields": fields, "views": views, "lists": lists,
             "derived": derived,

platform/aios_grid.py

@@ -257,6 +257,49 @@ def merge_choice_options(*sources):
     return _clean_options(merged)
 
 
+def _choice_values_from_cell(field, value):
+    """The individual labels represented by one select-family cell value."""
+    if not isinstance(field, dict) or field.get("type") not in ("select", "multiselect"):
+        return []
+    raw_values = (str(value or "").split(",")
+                  if field.get("type") == "multiselect" else [value])
+    return _clean_options(raw_values)
+
+
+def hydrate_choice_options_from_rows(fields, rows, ignored_by_field=None):
+    """Return field copies whose choice vocabularies include their served cell values.
+
+    A field may already contain real values from an import or a read-time source default before
+    anybody edits it through ``grid_events``.  Those values are still choices: omitting them
+    makes the picker claim that a populated select has no options.  This is a read projection,
+    deliberately; it cannot write a value a caller is not permitted to read into tenant schema.
+    """
+    row_maps = [row for row in (rows or ()) if isinstance(row, dict)]
+    ignored = {}
+    for key, raw_values in (ignored_by_field or {}).items():
+        values = [raw_values] if isinstance(raw_values, (str, int, float)) else raw_values
+        ignored[str(key)] = {str(value).strip().casefold() for value in (values or ())
+                             if str(value).strip()}
+
+    out = []
+    for raw in fields or ():
+        field = dict(raw) if isinstance(raw, dict) else raw
+        if not isinstance(field, dict) or field.get("type") not in ("select", "multiselect"):
+            out.append(field)
+            continue
+        key = str(field.get("key") or "")
+        values = []
+        for row in row_maps:
+            if key not in row:
+                continue
+            values.extend(value for value in _choice_values_from_cell(field, row.get(key))
+                          if value.casefold() not in ignored.get(key, set()))
+        if values:
+            field["options"] = merge_choice_options(field.get("options"), values)
+        out.append(field)
+    return out
+
+
 def apply_choice_overrides(fields, overrides):
     """Return field copies with durable select-family option overrides applied.
 

platform/aios_grid_fields.json

@@ -359,7 +359,7 @@
         "type": "multiselect",
         "source": "overlay",
         "default": true,
-        "description": "Who makes it. Editable here and shared with everyone in the workspace; seeded from the inventory mastersheet.",
+        "description": "Supplier or manufacturer for this product.",
         "shared": true
       },
       {
@@ -368,7 +368,7 @@
         "type": "text",
         "source": "overlay",
         "default": false,
-        "description": "Country of origin. Editable here and shared with everyone; seeded from the inventory mastersheet.",
+        "description": "Country where this product was made.",
         "shared": true
       },
       {
@@ -377,7 +377,7 @@
         "type": "int",
         "source": "overlay",
         "default": true,
-        "description": "Order-to-arrival days for this supplier. Drives the buy signal. Editable and shared with everyone.",
+        "description": "Days from placing an order to receiving it.",
         "shared": true
       },
       {
@@ -386,7 +386,7 @@
         "type": "currency",
         "source": "overlay",
         "default": false,
-        "description": "Quoted unit cost at origin, before freight and duty. Editable and shared with everyone.",
+        "description": "Quoted unit cost before freight and duty.",
         "shared": true
       },
       {
@@ -403,7 +403,7 @@
         "type": "text",
         "source": "odoo",
         "default": false,
-        "description": "Package label for Price 1. Blank means Odoo has no package reference for that price."
+        "description": "Package label for Price 1."
       },
       {
         "key": "price_2",
@@ -671,7 +671,7 @@
         "label": "Days of supply",
         "type": "int",
         "source": "odoo",
-        "description": "Days of supply at the trailing twelve month rate, counting ON HAND PLUS INBOUND units as stock. A blank means there is no honest number to print, and Stock status beside it says which of two reasons applies: 'No recent sales' is stock on the shelf that sold nothing in the window, so there is no rate to divide by; 'No stock record' is a product the inventory read returned no row for, so its shelf is unknown rather than empty. A product that sold nothing and holds nothing reads 0, not blank. Same stock figure as the cover gap columns, so the two cannot disagree about how much you have. They still divide by different rates: this one is the trailing average, the cover gap uses the forward 8 month forecast, so a seasonal SKU reads differently in each. A business unit caller gets this column too, re-scoped: the shelf is the whole warehouse's, the rate is that unit's."
+        "description": "Estimated days of supply at the trailing twelve-month sales rate."
       },
       {
         "key": "cover_gap_d",
@@ -687,7 +687,7 @@
         "type": "int",
         "source": "odoo",
         "default": false,
-        "description": "The recommended reorder quantity: units of forecast demand over the lead time that on-hand plus inbound does not cover. POSITIVE means buy this many; negative is surplus units; blank means we cannot say, because the SKU has no lead time on file or no forecast demand. Rounded AWAY from zero, so a real shortfall never rounds down to nothing."
+        "description": "Recommended reorder quantity after on-hand and inbound stock."
       },
       {
         "key": "demand_fwd",
@@ -732,7 +732,7 @@
           "Yes"
         ],
         "shared": true,
-        "description": "Team-maintained. A SKU carries “Yes” when it appears on the NEEDS PRICING tab of the 2027 catalog workbook; no value means it is not on that list. Shared with everyone in the workspace — the whole point of R8 is that the TEAM's work lands in the system, not one importer's private column."
+        "description": "Marks products that need pricing."
       },
       {
         "key": "march_pricelist",
@@ -744,7 +744,7 @@
           "Yes"
         ],
         "shared": true,
-        "description": "Team-maintained. A SKU carries “Yes” when it appears on the March Pricelist tab of the 2027 catalog workbook; no value means it is not on that list. Shared with everyone in the workspace — the whole point of R8 is that the TEAM's work lands in the system, not one importer's private column."
+        "description": "Marks products included in the March pricelist."
       },
       {
         "key": "price_changes",
@@ -756,7 +756,7 @@
           "Yes"
         ],
         "shared": true,
-        "description": "Team-maintained. A SKU carries “Yes” when it appears on the Price Changes tab of the 2027 catalog workbook; no value means it is not on that list. Shared with everyone in the workspace — the whole point of R8 is that the TEAM's work lands in the system, not one importer's private column."
+        "description": "Marks products with planned price changes."
       },
       {
         "key": "closeouts",
@@ -768,7 +768,7 @@
           "Yes"
         ],
         "shared": true,
-        "description": "Team-maintained. A SKU carries “Yes” when it appears on the Closeouts tab of the 2027 catalog workbook; no value means it is not on that list. Shared with everyone in the workspace — the whole point of R8 is that the TEAM's work lands in the system, not one importer's private column."
+        "description": "Marks products designated for closeout."
       },
       {
         "key": "notes",
@@ -777,7 +777,7 @@
         "source": "overlay",
         "default": false,
         "shared": true,
-        "description": "Team-maintained. The 2027 workbook's own non-Odoo columns (Product Description, Packing) folded into one field. Shared with everyone in the workspace."
+        "description": "Team notes about this product."
       },
       {
         "key": "sub_category",
@@ -791,7 +791,7 @@
         "permissions": {
           "edit": "collaborative"
         },
-        "description": "Fine grained product class from the Daily Mastersheet. Owned by the Administration account and shared with everyone in the workspace. Blank means the mastersheet carries no value for this SKU."
+        "description": "Detailed product classification."
       },
       {
         "key": "main_category",
@@ -805,7 +805,7 @@
         "permissions": {
           "edit": "collaborative"
         },
-        "description": "Top level product class from the Daily Mastersheet. Owned by the Administration account and shared with everyone in the workspace. Blank means the mastersheet carries no value for this SKU."
+        "description": "Top-level product classification."
       },
       {
         "key": "color",
@@ -819,7 +819,7 @@
         "permissions": {
           "edit": "collaborative"
         },
-        "description": "Colors this SKU is offered in, from the Daily Mastersheet. Multi-select, because the sheet stores several colors in one comma separated cell and each one becomes its own value. Owned by the Administration account and shared with everyone in the workspace."
+        "description": "Colors available for this product."
       },
       {
         "key": "shape_style",
@@ -833,7 +833,7 @@
         "permissions": {
           "edit": "collaborative"
         },
-        "description": "Shape or styling of the piece, from the Daily Mastersheet. Owned by the Administration account and shared with everyone in the workspace. Blank means the mastersheet carries no value for this SKU."
+        "description": "Shape or styling of this product."
       },
       {
         "key": "material",
@@ -847,7 +847,7 @@
         "permissions": {
           "edit": "collaborative"
         },
-        "description": "What the piece is made of, from the Daily Mastersheet. Owned by the Administration account and shared with everyone in the workspace. Blank means the mastersheet carries no value for this SKU."
+        "description": "Material this product is made from."
       },
       {
         "key": "finish",
@@ -861,7 +861,7 @@
         "permissions": {
           "edit": "collaborative"
         },
-        "description": "Surface finish of the piece, from the Daily Mastersheet. Owned by the Administration account and shared with everyone in the workspace. Blank means the mastersheet carries no value for this SKU."
+        "description": "Surface finish of this product."
       },
       {
         "key": "occassion",
@@ -875,7 +875,7 @@
         "permissions": {
           "edit": "collaborative"
         },
-        "description": "Occasions this SKU is bought for, from the Daily Mastersheet. Multi-select, because the sheet stores several occasions in one comma separated cell and each one becomes its own value. The label keeps the mastersheet's own spelling. Owned by the Administration account and shared with everyone in the workspace."
+        "description": "Occasions this product is bought for."
       },
       {
         "key": "collection",
@@ -889,7 +889,7 @@
         "permissions": {
           "edit": "collaborative"
         },
-        "description": "The merchandising collection this SKU belongs to, from the Daily Mastersheet. Owned by the Administration account and shared with everyone in the workspace. Blank means the mastersheet carries no value for this SKU."
+        "description": "Merchandising collection this product belongs to."
       },
       {
         "key": "size",
@@ -903,7 +903,7 @@
         "permissions": {
           "edit": "collaborative"
         },
-        "description": "Size band from the Daily Mastersheet. Owned by the Administration account and shared with everyone in the workspace. Blank means the mastersheet carries no value for this SKU."
+        "description": "Size band for this product."
       },
       {
         "key": "upc",
@@ -911,7 +911,7 @@
         "type": "text",
         "source": "odoo",
         "default": false,
-        "description": "The SKU's barcode, read live from Odoo as product.product.barcode. Blank when Odoo carries no barcode for this product. The Daily Mastersheet has a UPC column of its own and it is deliberately not used: three quarters of it is a placeholder rather than a number."
+        "description": "Barcode for this product."
       }
     ]
   }

platform/model/topics/odoo_products.yml

@@ -174,22 +174,22 @@ fields:
     label: "Supplier"
     type: multiselect
     kind: data
-    means: "Who makes it. Editable here and shared with everyone in the workspace; seeded from the inventory mastersheet."
+    means: "Supplier or manufacturer for this product."
   - key: origin_country
     label: "Country"
     type: text
     kind: data
-    means: "Country of origin. Editable here and shared with everyone; seeded from the inventory mastersheet."
+    means: "Country where this product was made."
   - key: lead_days
     label: "Lead time (days)"
     type: int
     kind: data
-    means: "Order-to-arrival days for this supplier. Drives the buy signal. Editable and shared with everyone."
+    means: "Days from placing an order to receiving it."
   - key: first_cost
     label: "First cost"
     type: currency
     kind: data
-    means: "Quoted unit cost at origin, before freight and duty. Editable and shared with everyone."
+    means: "Quoted unit cost before freight and duty."
   - key: price_1
     label: "Price 1"
     type: currency