toga-ai 1.0.592 → 1.0.594

package/knowledge/2.0/apps/api2/INDEX.md

@@ -18,7 +18,7 @@
 | [TOGa IQ Sprint Dashboard API (Record Scripts)](features/sprint-dashboard-api.md) | The internal **TOGa IQ sprint dashboard** is served in production by **six api2 Record Scripts** on `_Model_Team_Sprint` (`_underscore/Model/Team/Sprint.php`), | _underscore/Model/Team/Sprint.php, api2/Component/Api/V2/V2.php, dbchanges2/Core/2026-07-24a - SprintDashboardRecordScripts.sql, dbchanges2/Client_True/2026-07-24b - SprintDashboardScriptAcl.sql |
 | [Surface action-state via the surface=<slug> request option (M2M-safe)](features/surface-meta-option.md) | An opt-in V2 engine request option, `surface=<slug>`, that attaches per-record UI action state (`isVisible`/`isEnabled`) to a GET response **under `meta.surface | api2/Component/Api/V2/V2.php, _underscore/Model/Core/Surface.php |
 | [TableView row-filtering via apiWhereClause (options.where grammar, end to end)](features/tableview-apiwhereclause-row-filtering.md) | `TableViews.apiWhereClause` (TEXT, nullable) is the sanctioned, code-free way to restrict or exclude rows from a 2.0 table view. | api2/Component/Api/V2/V2.php, _underscore/Model/Client/TableView.php, toga2-supply/src/api/toga.ts |
-| [TableView field/column metadata (TableViewFields, hidden projected columns)](features/tableview-field-metadata.md) | The columns of a 2.0 table view are defined by DB metadata, not code. | _underscore/Model/Client/TableView.php, api2/Component/Api/V2/V2.php, dbchanges2/Client/2026-07-20 - ItemsUuidForPurchaseOrderItemsTableView.sql |
+| [TableView field/column metadata (TableViewFields, hidden projected columns)](features/tableview-field-metadata.md) | The columns of a 2.0 table view are defined by DB metadata, not code. | _underscore/Model/Client/TableView.php, api2/Component/Api/V2/V2.php, dbchanges2/Client/2026-07-20 - ItemsUuidForPurchaseOrderItemsTableView.sql, dbchanges2/Core/2026-08-17a - ItemFulfillmentQuantityFieldTypeNumber.sql, dbchanges2/Client/2026-08-17a - ItemFulfillmentColumnsCopyable.sql |
 | [Tickets API (/v2/tickets)](features/tickets-api.md) | The generic ticket endpoint of the 2.0 REST API. | Component/Api/V2/V2.php |
 | [V2 API error/message codes (EV/EZ troubleshooting map)](features/v2-api-error-codes.md) | The V2 JSON engine (`Component/Api/V2/V2.php`) returns short **message codes** in the response `error` field, grouped by family: `EN-*` authentication, `EZ-*` a | api2/Component/Api/V2/V2.php, api2/Component/Api/V2/Response/Response.php, api2/Component/Api/V2/Response/Oauth/Oauth.php, api2/Controller/Index.php, toga2-supply/src/globalTypes.ts, toga2-supply/src/pages/Orders/api/OrdersApi.ts, toga2-supply/src/api/toga.ts, _underscore/Model/Client/TrackingNumber.php |
 | [V2 request deadlock-retry — route-scoped in-process replay](features/v2-deadlock-retry.md) | api2's front controller can **detect a MySQL deadlock (1213) / lock-wait timeout (1205) and replay the whole request in-process**, so a transient lock collision | api2/Controller/Index.php, _underscore/Database.php, _underscore/Query.php |

package/knowledge/2.0/apps/api2/features/tableview-field-metadata.md

@@ -12,6 +12,8 @@ files:
   - _underscore/Model/Client/TableView.php
   - api2/Component/Api/V2/V2.php
   - dbchanges2/Client/2026-07-20 - ItemsUuidForPurchaseOrderItemsTableView.sql
+  - dbchanges2/Core/2026-08-17a - ItemFulfillmentQuantityFieldTypeNumber.sql
+  - dbchanges2/Client/2026-08-17a - ItemFulfillmentColumnsCopyable.sql
 related:
   - tableview-apiwhereclause-row-filtering.md
   - ../../toga25-supply/features/meta-driven-table-data.md
@@ -33,7 +35,20 @@ it.
 - **`Client_*.TableViews`** — one row per view (per client DB, e.g. `Client_Nychh`).
 - **`Client_*.TableViewFields`** — one row per **column** of a view. Key columns: `recordFieldId`
   (which field this column shows), `tableViewJoinId` (which join in the view the field is reached
-  through), `slug` (the frontend/tanstack column id), `isVisible` (0 = projected-but-hidden).
+  through), `slug` (the frontend/tanstack column id), `isVisible` (0 = projected-but-hidden),
+  and the per-view **presentation flags** `isCopyable` / `isSortable` / `isFilterable`. These
+  four flags **are** per-view — the same underlying field can be copyable in one view and not in
+  another.
+
+### A column's `type` / `precision` are NOT per-view — they come from `Core.RecordFields`
+
+A field's rendered **type and precision are canonical**, read from the shared
+`Core.RecordFields.type` / `Core.RecordFields.precision` — there is **no per-view type/precision
+override** on `TableViewFields`. Changing `Core.RecordFields.type`/`precision` therefore changes
+the column's type/format **everywhere that field appears, for every client**. That is the
+intended canonical behavior — if you only want the change in one view, this mechanism cannot do
+it. (Reference: `Core.RecordFields` id 369 = `ItemFulfillmentItems.quantity` set to
+`type=NUMBER`, `precision=0`, which normalizes that column across all fulfillment views/clients.)
 - **`Client_*.TableViewJoins`** — the view's joins; also drive `options.join`/`ojoin` on the data
   request (see [tableview-apiwhereclause-row-filtering](tableview-apiwhereclause-row-filtering.md)).
 
@@ -87,9 +102,17 @@ per-client folder — the reference migration
 - **Make column migrations idempotent** (`NOT EXISTS` guard) — they run per client DB.
 - **`isVisible=0` still costs a fetch** — the field is selected into the projection; only the render
   is suppressed.
+- **`type`/`precision` are canonical (Core-wide), not per-view.** Editing `Core.RecordFields`
+  changes every view/client that shows the field; there is no per-view type override. `isCopyable`,
+  `isSortable`, `isFilterable`, `isVisible` are the per-view knobs (on `TableViewFields`).
 
 ## Change history
 
+- 2026-08-17 — Documented that a column's `type`/`precision` are canonical from `Core.RecordFields`
+  (no per-view override — a change hits every view/client) while `isCopyable`/`isSortable`/
+  `isFilterable`/`isVisible` are per-view `TableViewFields` flags. Confirmed setting
+  `Core.RecordFields` id 369 (`ItemFulfillmentItems.quantity`) to `type=NUMBER precision=0` and
+  toggling `isCopyable` per-view on the shared item-fulfillment table views. (apeterson)
 - 2026-07-20 — Documented the backend TableView column-metadata model (per-client
   `TableViews`/`TableViewFields`/`TableViewJoins`, `recordFieldId` → shared `Core.RecordFields`
   cross-DB ref, `isVisible=0` projected-but-hidden columns) and the dbchanges2 recipe for adding a

package/knowledge/2.0/apps/toga-blox/features/primary-table-templates.md

@@ -6,7 +6,7 @@ project: TOGa Blox
 client: shared
 type: feature
 status: active
-updated: 2026-06-23
+updated: 2026-08-17
 owners: [apeterson]
 files:
   - toga-blox/src/templates/PrimaryTable/PrimaryTable.tsx
@@ -80,7 +80,13 @@ Sticky columns use a `--sticky-offset` CSS variable summed from neighboring colu
 - Frozen widths measure only **mounted** rows; virtualized tables size from the first paint set.
 - `headerSpan` adds a separate top row (`colSpan`); only the leaf header row maps to colgroup.
 - Resize is non-persistent (local state, resets when columns change).
+- **A visible column can look "missing" under `shrinkColumns`.** `shrinkColumns` (modal tables)
+  freezes each column to its measured header+body content width, so a column whose content is
+  minimal collapses to a sliver and appears absent even though it built. If a column seems gone,
+  confirm `isVisible=1` first (that is the only real gate), then look at its shrink/freeze width —
+  not the column set.
 
 ## Change history
+- 2026-08-17 — Noted that under `shrinkColumns` a genuinely visible column can render at a sliver width and look "missing" — check `isVisible` and the measured/frozen width, not the column set (apeterson).
 - 2026-06-23 — Documented the three Primary Table entry points, measure-and-freeze sizing, virtualization, and sticky columns (apeterson).
 - 2026-06-22 — Added `shrinkColumns` / `minColumnWidth` shrink-to-fit for modal tables; freeze logic uses natural content width (apeterson).

package/knowledge/2.0/apps/toga-blox/features/table.md

@@ -6,7 +6,7 @@ project: TOGa Blox
 client: shared
 type: feature
 status: active
-updated: 2026-06-23
+updated: 2026-08-17
 owners: [apeterson]
 files:
   - toga-blox/src/components/Table/index.ts
@@ -96,7 +96,10 @@ Per-column header controls, each a popover trigger with `data-active` / `data-op
   is hidden while the input is focused; both the trigger button and the icon span carry
   `data-active`. The clear button must not widen the input row (fixed-width menu).
 - **`HeaderFilterRange`** — numeric, modes exactly/moreThan/lessThan + a range toggle
-  (min/max); emits `{ key, value }[]` (`eq`/`min`/`max`/`between`).
+  (min/max); emits `{ key, value }[]` (`eq`/`min`/`max`/`between`). Reads its persisted
+  operator/values back from a `filterParams` prop (not `filterValue`, which collapses to the
+  literal `"active"` when a range filter is set); `PrimaryTableHeaderCell` must pass
+  `filterParams` through.
 - **`HeaderFilterMultiselect`** — checkbox list with "Select All"; emits comma-joined slugs.
 - **`HeaderFilterDate`** — calendar picker, modes exactly/before/after + range; converts
   `MM/DD/YYYY` ↔ URL `YYYY-MM-DD`; calendar + mode dropdown both portalled.
@@ -134,8 +137,50 @@ Per-column header controls, each a popover trigger with `data-active` / `data-op
 - The filter `includes` mode treats comma-separated values as OR.
 - `EditableCell` currency formats **on blur**, phone formats **on keystroke**.
 - Cell type config (`columnTypeConfig` prop) overrides the hardcoded `COLUMN_TYPE_CONFIG`.
+- **Field `type`/`precision` come from the backend `Core.RecordFields`, not from the table-view
+  field.** `resolveCellType` keys off `tableViewField.type` but that value is populated upstream
+  from the shared Core RecordField — there is no per-view type override. Changing a column's type
+  or precision is a `Core.RecordFields` DB change (see
+  [api2 tableview-field-metadata](../../api2/features/tableview-field-metadata.md)), not a blox
+  change. `isCopyable`/`isVisible`/`isSortable`/`isFilterable` *are* per-view flags.
+- **The only column-visibility gate is `field.isVisible`.** `buildTanstackColumns` builds a column
+  for every visible field. The `accessorFn` walks the `tableViewField` dot-path
+  (`SalesOrderItems.lineNumber` → `row.SalesOrderItems.lineNumber`), so a joined/context field is
+  blank unless `tableViewMeta.joins` actually provides the join. `settings.contextTable`/
+  `contextField` are used **only** to resolve the column *label* (`useAssignTableFieldLabels`),
+  never for data fetching. A visible column can still *look* missing when `shrinkColumns` freezes
+  it to a tiny width because its measured header+body content is minimal (see
+  [primary-table-templates](primary-table-templates.md)).
+- **Filter presentational components must seed their internal mode/value state from props.**
+  The filter mode/operator survives the URL round-trip — it is persisted as a URL key suffix
+  (`_starts`/`_ends`/`_eq`/`_excludes` for text; `_min`/`_max`/`_eq`/`_between` for range),
+  rehydrated into TanStack `columnFilters` by toga25-supply `useServerTableUrlState`, and
+  carried through `PrimaryTableHeaderCell`. But `HeaderFilterSearch`/`HeaderFilterRange` are
+  remounted fresh each time a filter popover opens, so if they don't seed local state from
+  their props (`filterMode` / `filterParams`) they hard-default (`includes` for text,
+  `exactly` for range) and reopening a filter *shows the wrong mode* even though the applied
+  filter is correct. Fixed via a `useEffect` seeding `filterMode` from `filterModeProp` in
+  `HeaderFilterSearch`, and a rehydration effect keyed on `JSON.stringify(filterParams)` in
+  `HeaderFilterRange` that reconstructs mode + value(s) from the operator keys
+  (`eq`→Exactly, `min`→MoreThan, `max`→LessThan, `between`→range min/max).
+- **Table/surface meta is cached with `Infinity` staleTime/gcTime** (`useFetchTablePageMeta`), so
+  after a DB metadata change (`TableViewFields`, `Core.RecordFields`) a **hard reload** is required
+  to see it. And the app consumes toga-blox's built **`dist/`**, not `src/` — editing blox source
+  requires `npm run build` before it takes effect at runtime.
 
 ## Change history
+- 2026-08-17 — Fixed: column filter **mode** was not restored when a filter popover was reopened
+  (text reverted to `includes`, range to `exactly`). Root cause: the URL round-trip and
+  `PrimaryTableHeaderCell` preserved the mode, but `HeaderFilterSearch` (destructured
+  `filterMode` as `filterModeProp` and never used it) and `HeaderFilterRange` (received no
+  operator info) never seeded internal state from props. Seeded both from props; added a
+  `filterParams` prop to `HeaderFilterRange` and passed it through `PrimaryTableHeaderCell`.
+  (apeterson)
+- 2026-08-17 — Added gotchas on meta→column mapping: field `type`/`precision` come from
+  `Core.RecordFields` (not per-view) while `isCopyable`/`isVisible`/`isSortable`/`isFilterable` are
+  per-view; `isVisible` is the only column gate; `accessorFn` dot-walks and needs the join;
+  `contextTable`/`contextField` drive labels only; meta is cached `Infinity` (hard reload after DB
+  meta change) and the app runs `dist/` (rebuild required). (apeterson)
 - 2026-06-23 — Documented the Table component layers, cell types, filters/sorts, hooks, and theming (apeterson).
 - 2026-06-23 — `HeaderFilterSearch`: search icon hidden on input focus; `data-active` added to the icon span; clear button no longer widens the input (fixed-width menu); mode dropdown excludes the selected mode (apeterson).
 - 2026-06-22 — Column sizing reworked to measure-and-freeze using natural content width; `MIN_COLUMN_WIDTH` floor (120px) applies only when no explicit width is set (apeterson).

package/knowledge/2.0/apps/toga25-supply/INDEX.md

@@ -9,7 +9,7 @@
 | [Column Visibility (URL-driven show/hide columns)](features/column-visibility.md) | A "Columns" header button that opens a modal listing every column from the table meta, lets the user show/hide columns, adjusts the table live, and persists the | toga25-supply/src/components/ColumnVisibilityModal/, toga25-supply/src/pages/SalesOrders/SalesOrders.tsx, toga25-supply/src/pages/SalesOrders/viewModel/useSalesOrdersPageViewModel.tsx, toga25-supply/src/pages/SalesOrders/hooks/useSalesOrdersTableData.tsx |
 | [Force Logout on Deployment (useDeploymentGuard)](features/force-logout-on-deployment.md) | On large deployments the backend bumps the Core parameter `META_LAST_REFRESH_DATETIME`. | toga25-supply/src/hooks/useDeploymentGuard.tsx, toga25-supply/src/App.tsx |
 | [Meta-Driven Page & Table Setup](features/meta-driven-table-data.md) | A page in this app is **meta-driven end to end**: the page view model fetches *page meta* (labels, sections, ACL) and *table meta* (the columns/fields + table s | toga25-supply/src/pages/SalesOrders/viewModel/useSalesOrdersPageViewModel.tsx, toga25-supply/src/pages/SalesOrders/hooks/useSalesOrdersTableState.ts, toga25-supply/src/hooks/useTablePageMeta.ts, toga-blox-npm/dist/hooks/useFetchPageMeta.d.ts, toga-blox-npm/dist/hooks/useFetchTablePageMeta.d.ts, toga-blox-npm/dist/hooks/useAssignTableFieldLabels.d.ts, toga-blox-npm/dist/components/Table/hooks/useTableData.d.ts |
-| [Record Modals & Nested Tables](features/record-modals-and-nested-tables.md) | The repo's family of modal + nested-table patterns layered over toga-blox `TableRecordModal` and `PrimaryTable*Layout`. | toga25-supply/src/layout/ItemRecordModalLayout/, toga25-supply/src/layout/SalesOrderRecordModalLayout/, toga25-supply/src/layout/SalesOrderItemsTableLayout/, toga25-supply/src/layout/ItemFulfillmentModal/, toga25-supply/src/layout/GenericNestedTables/GenericNestedTables.tsx, toga25-supply/src/layout/GenericNestedTables/GenericTableLayout.tsx, toga25-supply/src/pages/Inventory/viewModel/useInventoryPageViewModel.tsx, toga25-supply/src/pages/Inventory/viewModel/FIELDS/DEFAULT/inventoryGroupings.json, toga25-supply/src/hooks/useTableCellInteractions.ts, toga25-supply/src/hooks/useServerTableUrlState.ts, toga25-supply/src/layout/RecordApprovalModal/helpers/handleFormatApprovalWorkflowPayload.ts, toga25-supply/src/layout/RecordApprovalModal/api/approvalDecisionsApi.ts |
+| [Record Modals & Nested Tables](features/record-modals-and-nested-tables.md) | The repo's family of modal + nested-table patterns layered over toga-blox `TableRecordModal` and `PrimaryTable*Layout`. | toga25-supply/src/layout/PrimaryTableServerLayout/PrimaryTableServerLayout.tsx, toga25-supply/src/layout/PrimaryTableServerLayout/types.ts, toga25-supply/src/layout/ItemRecordModalLayout/, toga25-supply/src/layout/SalesOrderRecordModalLayout/, toga25-supply/src/layout/SalesOrderItemsTableLayout/, toga25-supply/src/layout/ItemFulfillmentModal/, toga25-supply/src/layout/GenericNestedTables/GenericNestedTables.tsx, toga25-supply/src/layout/GenericNestedTables/GenericTableLayout.tsx, toga25-supply/src/pages/Inventory/viewModel/useInventoryPageViewModel.tsx, toga25-supply/src/pages/Inventory/viewModel/FIELDS/DEFAULT/inventoryGroupings.json, toga25-supply/src/hooks/useTableCellInteractions.ts, toga25-supply/src/hooks/useServerTableUrlState.ts, toga25-supply/src/layout/RecordApprovalModal/helpers/handleFormatApprovalWorkflowPayload.ts, toga25-supply/src/layout/RecordApprovalModal/api/approvalDecisionsApi.ts |
 | [Surface Frontend (DB-driven UI consumption, src/surface/)](features/surface-frontend.md) | The frontend consumer of the platform-wide Surface layer — DB-driven UI config fetched from `GET /v2/surfaces/meta?slug=<slug>` instead of statically-imported J | toga25-supply/src/surface/applyColSpan.ts, toga25-supply/src/pages/SalesOrders/view/SalesOrderRecordModalLayout/helpers/sectionRenderers.tsx, toga25-supply/src/pages/SalesOrders/view/SalesOrderRecordModalLayout/helpers/getVisibleSections.ts, toga25-supply/src/pages/SalesOrders/view/SalesOrderRecordModalLayout/view/layoutComponents/SalesOrderApprovalSummaryGrid.tsx, toga25-supply/src/surface/useFetchSurfaceMeta.ts, toga25-supply/src/pages/SalesOrders/helpers/surfaceBundleToTenantFields.ts, toga25-supply/src/pages/SalesOrders/helpers/buildPatchedTenantFields.ts, toga25-supply/src/pages/SalesOrders/view/SalesOrderRecordModalLayout/viewModel/useSalesOrderRecordModalLayoutModel.tsx, toga25-supply/src/pages/SalesOrders/view/SalesOrderRecordModalLayout/view/SalesOrderView.tsx, toga25-supply/src/pages/SalesOrders/view/SalesOrderRecordModalLayout/view/layoutComponents/SalesOrderSummaryGrid.tsx, toga25-supply/src/pages/SalesOrders/view/SalesOrderRecordModalLayout/helpers/getDetailSections.tsx, toga25-supply/src/pages/SalesOrders/view/SalesOrderRecordModalLayout/view/sections/SalesOrderNotesSection.tsx, toga25-supply/src/pages/SalesOrders/helpers/cleanOrder.ts, toga25-supply/src/pages/SalesOrders/view/SalesOrderRecordModalLayout/view/sections/AdminNotesSection.tsx, toga25-supply/src/pages/SalesOrders/view/SalesOrderRecordModalLayout/helpers/getAdminNotes.ts, toga25-supply/src/pages/SalesOrders/view/SalesOrderRecordModalLayout/helpers/index.ts, toga25-supply/src/surface/evaluateSurfaceRule.ts, toga25-supply/src/surface/resolveElementState.ts, toga25-supply/src/pages/SalesOrders/helpers/evaluateEnableRule.ts, toga25-supply/src/pages/SalesOrders/hooks/useSalesOrderRowRecordState.ts, toga25-supply/src/surface/actionRegistry.ts, toga25-supply/src/surface/componentRegistry.tsx, toga25-supply/src/surface/SurfaceActionBar.tsx, toga25-supply/src/surface/SurfaceSection.tsx, toga25-supply/src/surface/resolve.ts, toga25-supply/src/surface/types.ts, toga25-supply/src/surface/index.ts, toga25-supply/src/pages/Login/LoginPage.tsx, toga25-supply/src/pages/SalesOrders/SalesOrders.tsx, toga25-supply/src/pages/SalesOrders/view/SurfaceRowActions.tsx, toga25-supply/src/pages/SalesOrders/view/SalesOrderRecordModalLayout/view/sections/SalesOrderTopBar.tsx, toga25-supply/src/pages/Items/ItemsPage.tsx, toga25-supply/src/pages/Items/viewModel/useItemsPageViewModel.tsx, toga25-supply/src/pages/VendorItems/VendorItemsPage.tsx, toga25-supply/src/pages/VendorItems/viewModel/useVendorItemsPageViewModel.tsx, toga25-supply/src/pages/Inventory/Inventory.tsx, toga25-supply/src/pages/Inventory/viewModel/useInventoryPageViewModel.tsx, toga25-supply/src/pages/Inventory/viewModel/FIELDS/index.ts, toga25-supply/src/fieldsConfig/index.ts, toga25-supply/src/layout/ItemRecordModalLayout/helpers/surfaceBundleToItemFields.ts, toga25-supply/src/layout/ItemRecordModalLayout/helpers/index.ts, toga25-supply/src/layout/ItemRecordModalLayout/viewModel/useItemRecordModalViewModel.tsx, toga25-supply/src/layout/ItemRecordModalLayout/ItemRecordModalLayout.tsx, toga25-supply/src/layout/ItemRecordModalLayout/components/ItemRecordView.tsx, toga25-supply/src/surface/useStatusColors.ts, toga25-supply/src/surface/SurfaceHeader.tsx, toga25-supply/src/pages/SalesOrders/view/SalesOrderApprovalModalsLayout/helpers/surfaceBundlesToDecisionFields.ts, toga25-supply/src/pages/SalesOrders/view/SalesOrderApprovalModalsLayout/viewModel/useApprovalModalViewModel.tsx |
 | [AWS Amplify Multi-Environment Deployment](workflows/amplify-deployment.md) | How `toga25-supply` deploys to **all** of its environments on AWS Amplify from a **single shared `amplify.yml`**. | toga25-supply/amplify.yml, toga25-supply/src/api/api.ts, toga25-supply/src/hooks/useAuthenticationFlow.ts, toga25-supply/vite.config.ts, toga25-supply/package.json |
 | [Cypress Testing Harness (component + e2e)](workflows/cypress-testing.md) | The Cypress test harness for the `toga25-supply` frontend, bootstrapped from scratch (`cypress` was already a dependency but there was no config, no `cypress/` | toga25-supply/cypress.config.ts, toga25-supply/cypress/tsconfig.json, toga25-supply/cypress/support/component.tsx, toga25-supply/cypress/support/component-index.html, toga25-supply/cypress/support/e2e.ts, toga25-supply/cypress/support/commands.ts, toga25-supply/cypress/support/fixtures.ts, toga25-supply/cypress/support/mocks/useApprovalModalViewModel.ts, toga25-supply/cypress/component/SalesOrderApprovalModalsLayout.cy.tsx, toga25-supply/cypress/component/RecordApprovalModalLayout.cy.tsx, toga25-supply/cypress/component/EnterPoNumberModal.cy.tsx, toga25-supply/cypress/e2e/salesOrderApproval.cy.ts |

package/knowledge/2.0/apps/toga25-supply/features/record-modals-and-nested-tables.md

@@ -6,9 +6,11 @@ project: TOGa 2.5 Supply
 client: shared
 type: feature
 status: active
-updated: 2026-08-14
+updated: 2026-08-17
 owners: [apeterson]
 files:
+  - toga25-supply/src/layout/PrimaryTableServerLayout/PrimaryTableServerLayout.tsx
+  - toga25-supply/src/layout/PrimaryTableServerLayout/types.ts
   - toga25-supply/src/layout/ItemRecordModalLayout/
   - toga25-supply/src/layout/SalesOrderRecordModalLayout/
   - toga25-supply/src/layout/SalesOrderItemsTableLayout/
@@ -92,6 +94,13 @@ caches the unfiltered first render (uuid null) and never refetches. Guard with `
 when uuid is null, `additionalData` returns `{}`. Typically `isPaginationEnabled: false` +
 `isInfiniteScrollingEnabled: true` (but see [meta-driven-table-data](meta-driven-table-data.md)).
 
+**Read-only (non-clickable) modal tables — `disableRowClick`.** `PrimaryTableServerLayout` takes
+an optional `disableRowClick?: boolean` (its `types.ts`); when set it passes `onRowClick={undefined}`
+to `PrimaryTableServerTemplate` so rows render non-interactive (no cursor/hover affordance, no
+navigation). `ItemFulfillmentModal` sets it because the fulfillment table is read-only. It is
+scoped so all other `PrimaryTableServerLayout` consumers keep normal row-click behavior — pass it
+only for genuinely read-only tables.
+
 ## Pattern 5 — `GenericNestedTables` (config-driven multi-level / swap tables)
 
 A config-driven renderer: declare an array of `TableLevel` objects instead of hand-wiring each
@@ -204,6 +213,9 @@ Scaffold per the `ItemRecordModalLayout` pattern (`src/layout/ItemRecordModalLay
   calling `apiDelete(route)`.
 
 ## Change history
+- 2026-08-17 — Added `disableRowClick?: boolean` to `PrimaryTableServerLayout` (passes
+  `onRowClick={undefined}` to the template) for read-only modal tables; `ItemFulfillmentModal` uses
+  it. Scoped so other consumers keep normal row-click behavior. (apeterson)
 - 2026-08-14 — Fixed unassigning an approval-workflow stage: clearing the assignee now emits a
   DELETE `/approval-decisions/{uuid}` (the assignment lives on the `ApprovalDecisions` record)
   instead of silently dropping the field. Gated to fire only for a still-undecided stage

package/knowledge/clients/compass-canada/profile.md

@@ -6,6 +6,7 @@ apps:
   - api2
   - toga2-supply
   - toga25-supply
+  - toga-blox
   - toga2-commerce
   - worker
   - worker2
@@ -15,7 +16,7 @@ project: _Underscore
 client: compass-canada
 type: profile
 status: active
-updated: 2026-08-12
+updated: 2026-08-17
 owners: [jcardinal, bala, tcox, apeterson]
 files: []
 related:

package/knowledge/clients/compass-usa/profile.md

@@ -6,6 +6,7 @@ apps:
   - api2
   - toga2-supply
   - toga25-supply
+  - toga-blox
   - toga2-commerce
   - worker
   - worker1.5
@@ -17,7 +18,7 @@ project: _Underscore
 client: compass-usa
 type: profile
 status: active
-updated: 2026-08-13
+updated: 2026-08-17
 owners: [jcardinal, bala, tcox, apeterson, dfranks]
 files: []
 related:

package/knowledge/clients/nychh/profile.md

@@ -6,6 +6,7 @@ apps:
   - api2
   - toga2-supply
   - toga25-supply
+  - toga-blox
   - dbchanges2
   - worker2
   - worker
@@ -14,7 +15,7 @@ project: _Underscore
 client: nychh
 type: profile
 status: active
-updated: 2026-08-05
+updated: 2026-08-17
 owners: ["jcardinal", "apeterson", "bala"]
 files: []
 related:

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "toga-ai",
-  "version": "1.0.592",
+  "version": "1.0.594",
   "description": "TOGA Technology Team Claude Knowledge System — shared AI coding harness with skills, knowledge base CLI, and project installer for Claude Code.",
   "keywords": [
     "claude",