sap-wm-mcp 0.3.0 → 0.3.2

package/README.md

@@ -13,7 +13,7 @@ Connect AI agents — Claude, Copilot, or any MCP-compatible client — directly
 >
 > This project ships a custom RAP OData V4 service that exposes classic WM operations as a proper API — and wraps it in an MCP server so AI agents can drive it. Large portions of the SAP install base are still on classic WM. This fills the gap.
 
-The **npm package** ships 22 tools covering core operations, analytics, shift management, anomaly detection, audit history, proactive replenishment, and interim zone reconciliation. This repository contains the 9 open-sourced tool implementations — the ABAP RAP service source and additional tool source are available separately (see [ABAP Service Installation](#abap-service-installation)).
+The **npm package** ships 23 tools covering core operations, analytics, shift management, anomaly detection, audit history, proactive replenishment, interim zone reconciliation, and cycle count management. This repository contains the 9 open-sourced tool implementations — the ABAP RAP service source and additional tool source are available separately (see [ABAP Service Installation](#abap-service-installation)).
 
 ---
 
@@ -374,7 +374,7 @@ For write operations:
 
 ## Tools Reference
 
-The npm package ships **22 tools** across five capability areas. The 9 tools below are open-sourced in this repository. Analytics, shift management, anomaly detection, and operations tools are available in the published package.
+The npm package ships **23 tools** across five capability areas. The 9 tools below are open-sourced in this repository. Analytics, shift management, anomaly detection, cycle count management, and operations tools are available in the published package.
 
 ---
 
@@ -541,6 +541,8 @@ The following tools are available in the published npm package and fully functio
 | `get_transfer_order_history` | Full TO history — creator, executor (resolved from LTAP.QNAME), and item detail; filterable by date range, status, movement type, material, `createdBy`, or `executedBy` |
 | `get_replenishment_needs` | Find forward-pick bins at or below a stock threshold — `defaultReplenishQty` param (default 50) used as fallback when no bin max qty is configured; flags bins with an open replenishment TO to avoid duplicate moves |
 | `get_interim_zone_anomalies` | Detect positive stock stranded in interim/staging zones (types 999, 998, 902) — surfaces same-day, overnight, and multi-day strandings with likely cause per zone; use `minDaysStranded` to filter noise during active shifts |
+| `get_goods_issue_monitor` | Open outbound deliveries — delivery qty vs picked qty, GI status, overdue flag, and stock still in GI zone (type 999). Tracks picking progress and stalled shipments. |
+| `create_cycle_count_document` | Create a WM inventory document (LI01N equivalent) for a specific bin. Activates the bin immediately by default (`LAGP.KZINV = 'IL'`). Use after `get_cycle_count_candidates` to trigger a count. |
 
 ---
 
@@ -566,11 +568,17 @@ The ABAP source is not included in this public repository. To obtain the ABAP pa
 | `ZR_WMCYCLECOUNTBIN` | CDS View | View over LAGP (cycle count indicators) |
 | `ZR_WMTRANSFERORDER` | BDEF | Behavior definition — defines actions |
 | `ZBP_R_WMTRANSFERORDER` | Class | RAP behavior implementation |
-| `ZWM_MFG` | Function Group | Contains RFC wrapper FM |
+| `ZR_WMGOODSISSUEDELIVERY` | CDS View | View over LIKP + LIPS (outbound deliveries for GI monitor) |
+| `ZR_WMTRANSFERORDER` | BDEF | Behavior definition — defines actions |
+| `ZBP_R_WMTRANSFERORDER` | Class | RAP behavior implementation |
+| `ZWM_MFG` | Function Group | Contains all RFC wrapper FMs |
 | `ZWM_TO_CREATE` | Function Module | RFC-enabled wrapper for `L_TO_CREATE_SINGLE` |
-| `ZA_WMCREATETOPARAM` | Structure | Parameter type for CreateTransferOrder action |
-| `ZA_WMCONFIRMTOSU` | Structure | Parameter type for ConfirmTransferOrderSU action |
-| `ZSD_WMMCPSERVICE` | Service Def | OData V4 service definition (7 entity sets) |
+| `ZWM_TO_CANCEL` | Function Module | RFC-enabled wrapper for `L_TO_CANCEL` |
+| `ZWM_INV_CREATE` | Function Module | RFC-enabled wrapper — writes directly to LINK/LINP/LINV and locks bin in LAGP |
+| `ZA_WMCREATETOPARAM` | Abstract Entity | Parameter type for CreateTransferOrder action |
+| `ZA_WMCONFIRMTOSUPARAM` | Abstract Entity | Parameter type for ConfirmTransferOrderSU action |
+| `ZA_WMCREATEINVDOCPARAM` | Abstract Entity | Parameter type for CreateInventoryDocument action |
+| `ZSD_WMMCPSERVICE` | Service Def | OData V4 service definition (8 entity sets) |
 | `ZSB_WMMCPSERVICE_ODATA4_UI` | Service Binding | OData V4 UI binding |
 
 ### abapGit compatibility
@@ -616,7 +624,8 @@ Classic WM has no standard OData APIs. This project builds one using **ABAP REST
 | `CreateTransferOrder` | `L_TO_CREATE_SINGLE` | Called via RFC wrapper `ZWM_TO_CREATE` with `DESTINATION 'NONE'` |
 | `ConfirmTransferOrder` | `L_TO_CONFIRM` | Called directly — no COMMIT needed |
 | `ConfirmTransferOrderSU` | `L_TO_CONFIRM_SU` | Called directly — no COMMIT needed |
-| `CancelTransferOrder` | `L_TO_CANCEL` | Called directly — no COMMIT needed |
+| `CancelTransferOrder` | `L_TO_CANCEL` | Called via RFC wrapper `ZWM_TO_CANCEL` with `DESTINATION 'NONE'` |
+| `CreateInventoryDocument` | Direct writes to LINK/LINP/LINV | Called via RFC wrapper `ZWM_INV_CREATE` with `DESTINATION 'NONE'` |
 
 ### Why the RFC wrapper?
 
@@ -683,7 +692,9 @@ sap-wm-mcp/
 │   ├── cancelTransferOrder.js        ← cancel_transfer_order
 │   ├── transferOrderHistory.js       ← get_transfer_order_history
 │   ├── replenishmentNeeds.js         ← get_replenishment_needs
-│   └── interimZoneAnomalies.js       ← get_interim_zone_anomalies
+│   ├── interimZoneAnomalies.js       ← get_interim_zone_anomalies
+│   ├── goodsIssueMonitor.js          ← get_goods_issue_monitor
+│   └── createCycleCountDoc.js        ← create_cycle_count_document
 ├── .env.example
 └── package.json
 ```
@@ -708,7 +719,7 @@ Running both side-by-side shows the contrast directly: same tools, same question
 | Phase | Status | Description |
 |---|---|---|
 | Phase 0 — RAP Service | ✅ Complete | Custom OData V4 service with 7 entity sets over classic WM tables |
-| Phase 1 — Local MCP | ✅ Complete | 22 tools working, security hardened, published to npm |
+| Phase 1 — Local MCP | ✅ Complete | 23 tools working, security hardened, published to npm |
 | Phase 2 — BTP CF | 🔜 Planned | Deploy to SAP BTP Cloud Foundry with SSE transport + XSUAA + Cloud Connector |
 | Phase 3 — Joule Agent | 💡 Future | Native Joule Studio agent using the same RAP service |
 

package/index.js

@@ -29,8 +29,10 @@ import { getTransferOrderHistory } from './tools/transferOrderHistory.js';
 import { cancelTransferOrder } from './tools/cancelTransferOrder.js';
 import { getReplenishmentNeeds } from './tools/replenishmentNeeds.js';
 import { getInterimZoneAnomalies } from './tools/interimZoneAnomalies.js';
+import { getGoodsIssueMonitor } from './tools/goodsIssueMonitor.js';
+import { createCycleCountDoc } from './tools/createCycleCountDoc.js';
 
-const server = new McpServer({ name: 'sap-wm-mcp', version: '0.3.0' });
+const server = new McpServer({ name: 'sap-wm-mcp', version: '0.3.2' });
 
 // Tool 1 — get_bin_status
 server.tool(
@@ -40,7 +42,7 @@ server.tool(
     warehouse: z.string().describe('Warehouse number e.g. 102'),
     storageType: z.string().optional().describe('Storage type e.g. 001'),
     bin: z.string().optional().describe('Specific bin number e.g. 01-01-01'),
-    top: z.number().optional().default(20).describe('Max records to return')
+    top: z.coerce.number().optional().default(20).describe('Max records to return')
   },
   async (params) => {
     try {
@@ -60,7 +62,7 @@ server.tool(
     warehouse: z.string().describe('Warehouse number e.g. 102'),
     material: z.string().optional().describe('Material number e.g. TG0001'),
     storageType: z.string().optional().describe('Filter by storage type'),
-    top: z.number().optional().default(20).describe('Max records to return')
+    top: z.coerce.number().optional().default(20).describe('Max records to return')
   },
   async (params) => {
     try {
@@ -80,7 +82,7 @@ server.tool(
     warehouse:   z.string().describe('Warehouse number e.g. 102'),
     storageType: z.string().optional().describe('Storage type to filter e.g. 001'),
     binType:     z.string().optional().describe('Bin type to filter e.g. E1, E2 — use when destination bin must match a specific storage unit type'),
-    top:         z.number().optional().default(50).describe('Max records to return')
+    top:         z.coerce.number().optional().default(50).describe('Max records to return')
   },
   async (params) => {
     try {
@@ -99,7 +101,7 @@ server.tool(
   {
     warehouse: z.string().describe('Warehouse number e.g. 102'),
     storageType: z.string().optional().describe('Filter by storage type'),
-    top: z.number().optional().default(100).describe('Max bins to analyze')
+    top: z.coerce.number().optional().default(100).describe('Max bins to analyze')
   },
   async (params) => {
     try {
@@ -120,7 +122,7 @@ server.tool(
     movementType:   z.string().describe('WM movement type e.g. 999'),
     material:       z.string().describe('Material number e.g. TG0001'),
     plant:          z.string().describe('Plant e.g. 1710'),
-    quantity:       z.number().describe('Quantity to move'),
+    quantity:       z.coerce.number().describe('Quantity to move'),
     unitOfMeasure:  z.string().optional().default('').describe('Unit of measure — leave empty to use material base UOM (recommended)'),
     sourceType:         z.string().optional().default('').describe('Source storage type e.g. 001'),
     sourceBin:          z.string().optional().default('').describe('Source bin e.g. 01-02-01'),
@@ -185,7 +187,7 @@ server.tool(
     storageType: z.string().optional().describe('Filter by source or destination storage type e.g. 999'),
     bin:         z.string().optional().describe('Filter by source or destination bin e.g. AUFNAHME'),
     material:    z.string().optional().describe('Filter by material number e.g. TG0001'),
-    top:         z.number().optional().default(50).describe('Max TO headers to return')
+    top:         z.coerce.number().optional().default(50).describe('Max TO headers to return')
   },
   async (params) => {
     try {
@@ -205,7 +207,7 @@ server.tool(
     warehouse:   z.string().describe('Warehouse number e.g. 102'),
     storageType: z.string().optional().describe('Storage type e.g. 001. Omit to see all types.'),
     bin:         z.string().optional().describe('Narrow to a specific bin within the type'),
-    top:         z.number().optional().default(100).describe('Max records to return')
+    top:         z.coerce.number().optional().default(100).describe('Max records to return')
   },
   async (params) => {
     try {
@@ -226,7 +228,7 @@ server.tool(
     status:      z.enum(['open', 'partial', 'to-created', 'completed']).optional().describe('TR status filter. Defaults to open + partial + to-created. Use to-created to see TRs where a TO exists but is not yet confirmed.'),
     material:    z.string().optional().describe('Filter by material number e.g. TG0001'),
     storageType: z.string().optional().describe('Filter by source or destination storage type'),
-    top:         z.number().optional().default(50).describe('Max records to return')
+    top:         z.coerce.number().optional().default(50).describe('Max records to return')
   },
   async (params) => {
     try {
@@ -247,7 +249,7 @@ server.tool(
     plant:           z.string().describe('Plant e.g. 1010'),
     storageLocation: z.string().optional().describe('Storage location (LGORT) linked to this warehouse e.g. 0002 — required for accurate results, otherwise MARD returns all plant stock'),
     material:        z.string().optional().describe('Narrow to a specific material'),
-    threshold:       z.number().optional().default(0).describe('Ignore variances smaller than this quantity')
+    threshold:       z.coerce.number().optional().default(0).describe('Ignore variances smaller than this quantity')
   },
   async (params) => {
     try {
@@ -266,8 +268,8 @@ server.tool(
   {
     warehouse:          z.string().describe('Warehouse number e.g. 102'),
     storageType:        z.string().optional().describe('Limit to a specific storage type e.g. 001'),
-    daysSinceLastCount: z.number().optional().default(180).describe('Flag bins not counted within this many days (default 180)'),
-    top:                z.number().optional().default(100).describe('Max bins to return')
+    daysSinceLastCount: z.coerce.number().optional().default(180).describe('Flag bins not counted within this many days (default 180)'),
+    top:                z.coerce.number().optional().default(100).describe('Max bins to return')
   },
   async (params) => {
     try {
@@ -287,8 +289,8 @@ server.tool(
     warehouse:        z.string().describe('Warehouse number e.g. 102'),
     storageType:      z.string().optional().describe('Narrow to a specific storage type'),
     material:         z.string().optional().describe('Narrow to a specific material'),
-    daysSinceLastMove: z.number().optional().default(90).describe('Flag stock not moved in this many days (default 90)'),
-    top:              z.number().optional().default(100).describe('Max quants to scan')
+    daysSinceLastMove: z.coerce.number().optional().default(90).describe('Flag stock not moved in this many days (default 90)'),
+    top:              z.coerce.number().optional().default(100).describe('Max quants to scan')
   },
   async (params) => {
     try {
@@ -343,8 +345,8 @@ server.tool(
   {
     warehouse:   z.string().describe('Warehouse number e.g. 102'),
     storageType: z.string().optional().describe('Limit to a specific storage type e.g. 003'),
-    threshold:   z.number().optional().default(3).describe('Flag bin+material combos with this many quants or more (default 3)'),
-    top:         z.number().optional().default(200).describe('Max stock records to scan')
+    threshold:   z.coerce.number().optional().default(3).describe('Flag bin+material combos with this many quants or more (default 3)'),
+    top:         z.coerce.number().optional().default(200).describe('Max stock records to scan')
   },
   async (params) => {
     try {
@@ -363,8 +365,8 @@ server.tool(
   {
     warehouse:   z.string().describe('Warehouse number e.g. 102'),
     storageType: z.string().optional().describe('Specific zone type to check e.g. 999. Defaults to 999 + 998.'),
-    minAgeDays:  z.number().optional().default(7).describe('Only show negatives older than this many days (default 7). Set to 0 to see all.'),
-    top:         z.number().optional().default(500).describe('Max stock records to scan')
+    minAgeDays:  z.coerce.number().optional().default(7).describe('Only show negatives older than this many days (default 7). Set to 0 to see all.'),
+    top:         z.coerce.number().optional().default(500).describe('Max stock records to scan')
   },
   async (params) => {
     try {
@@ -407,7 +409,7 @@ server.tool(
     createdBy:    z.string().optional().describe('Filter by the SAP user who created the TO e.g. NOMANH'),
     executedBy:   z.string().optional().describe('Filter by the SAP user who executed/confirmed the TO'),
     material:     z.string().optional().describe('Filter by material number e.g. TG0001'),
-    top:          z.number().optional().default(50).describe('Max number of TOs to return (default 50)')
+    top:          z.coerce.number().optional().default(50).describe('Max number of TOs to return (default 50)')
   },
   async (params) => {
     try {
@@ -445,10 +447,10 @@ server.tool(
     warehouse:       z.string().describe('Warehouse number e.g. 102'),
     storageType:     z.string().optional().default('P01').describe('Forward pick storage type to check (default P01)'),
     material:        z.string().optional().describe('Narrow to a specific material e.g. TG0001'),
-    minimumQuantity:     z.number().optional().default(0).describe('Flag bins at or below this level. Default 0 = empty/negative only. Overridden per bin when LAGP min qty (MNMNG) is configured.'),
-    targetQuantity:      z.number().optional().describe('Fill-to target — overrides bin master MaximumQuantity when set.'),
-    defaultReplenishQty: z.number().optional().default(50).describe('Fallback replenishment qty when bin master max qty (MXMNG) is 0 and no targetQuantity given. Default 50.'),
-    top:                 z.number().optional().default(50).describe('Max records to return')
+    minimumQuantity:     z.coerce.number().optional().default(0).describe('Flag bins at or below this level. Default 0 = empty/negative only. Overridden per bin when LAGP min qty (MNMNG) is configured.'),
+    targetQuantity:      z.coerce.number().optional().describe('Fill-to target — overrides bin master MaximumQuantity when set.'),
+    defaultReplenishQty: z.coerce.number().optional().default(50).describe('Fallback replenishment qty when bin master max qty (MXMNG) is 0 and no targetQuantity given. Default 50.'),
+    top:                 z.coerce.number().optional().default(50).describe('Max records to return')
   },
   async (params) => {
     try {
@@ -467,9 +469,9 @@ server.tool(
   {
     warehouse:       z.string().describe('Warehouse number e.g. 102'),
     interimTypes:    z.array(z.string()).optional().default(['999', '998', '902']).describe('Interim storage types to check — default [999, 998, 902]'),
-    minDaysStranded: z.number().optional().default(0).describe('Only return stock stranded for at least this many days. Default 0 = include same-day strandings.'),
+    minDaysStranded: z.coerce.number().optional().default(0).describe('Only return stock stranded for at least this many days. Default 0 = include same-day strandings.'),
     material:        z.string().optional().describe('Narrow to a specific material e.g. TG0001'),
-    top:             z.number().optional().default(100).describe('Max quants to scan (scans top * 3 internally to account for client-side filtering)')
+    top:             z.coerce.number().optional().default(100).describe('Max quants to scan (scans top * 3 internally to account for client-side filtering)')
   },
   async (params) => {
     try {
@@ -481,6 +483,47 @@ server.tool(
   }
 );
 
+// Tool 23 — get_goods_issue_monitor
+server.tool(
+  'get_goods_issue_monitor',
+  'Show open outbound deliveries in a classic WM warehouse — delivery qty vs picked qty, GI status, overdue flag, and which items still have stock in the GI zone (type 999). Use to track picking progress, identify stalled shipments, and understand the outbound delivery context behind TOs sourced from the GI zone.',
+  {
+    warehouse:        z.string().describe('Warehouse number e.g. 102'),
+    shippingPoint:    z.string().optional().describe('Filter by shipping point e.g. 1000'),
+    includeCompleted: z.boolean().optional().default(false).describe('Include deliveries where GI has already been posted (default false — open only)'),
+    material:         z.string().optional().describe('Narrow to a specific material e.g. TG0001'),
+    top:              z.coerce.number().optional().default(50).describe('Max delivery items to fetch (default 50)')
+  },
+  async (params) => {
+    try {
+      const result = await getGoodsIssueMonitor(params);
+      return { content: [{ type: 'text', text: JSON.stringify(result, null, 2) }] };
+    } catch (err) {
+      return { content: [{ type: 'text', text: `Error: ${err.message}` }], isError: true };
+    }
+  }
+);
+
+// Tool 24 — create_cycle_count_document
+server.tool(
+  'create_cycle_count_document',
+  'Create a classic WM inventory document (cycle count) for a specific bin — equivalent to LI01N. Activates the bin for counting immediately by default. Use after get_cycle_count_candidates to trigger a count on flagged bins.',
+  {
+    warehouse:   z.string().describe('Warehouse number e.g. 102'),
+    storageType: z.string().describe('Storage type of the bin to count e.g. 001'),
+    bin:         z.string().describe('Storage bin to count e.g. 01-02-03'),
+    activateNow: z.boolean().optional().default(true).describe('Activate the inventory document immediately so the bin is ready for counting (default true)')
+  },
+  async (params) => {
+    try {
+      const result = await createCycleCountDoc(params);
+      return { content: [{ type: 'text', text: JSON.stringify(result, null, 2) }] };
+    } catch (err) {
+      return { content: [{ type: 'text', text: `Error: ${err.message}` }], isError: true };
+    }
+  }
+);
+
 const transport = new StdioServerTransport();
 await server.connect(transport);
 console.error('SAP WM MCP Server running (stdio)...');

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "sap-wm-mcp",
-  "version": "0.3.0",
+  "version": "0.3.2",
   "description": "MCP server for SAP Classic Warehouse Management — connects AI agents to S/4HANA WM via a custom RAP OData V4 service. For systems where EWM is not active.",
   "type": "module",
   "main": "index.js",

package/tools/createCycleCountDoc.js

@@ -0,0 +1,54 @@
+import { s4hPost, s4hGet } from '../lib/s4hClient.js';
+import { esc } from '../lib/sanitize.js';
+
+const BASE = `/sap/opu/odata4/iwbep/all/srvd/sap/zsd_wmmcpservice/0001`;
+const NS   = `com.sap.gateway.srvd.zsd_wmmcpservice.v0001`;
+
+export async function createCycleCountDoc({
+  warehouse,
+  storageType,
+  bin,
+  activateNow = true
+}) {
+  const path = `${BASE}/WMTransferOrder/${NS}.CreateInventoryDocument`;
+
+  const body = {
+    WarehouseNumber: esc(warehouse),
+    StorageType:     esc(storageType),
+    StorageBin:      esc(bin),
+    ActivateNow:     activateNow ? 'X' : ' '
+  };
+
+  await s4hPost(path, body);
+
+  // RAP static action returns $self (WMTransferOrder) — field assignment does not
+  // propagate back through the OData layer. Instead, read the inventory document
+  // number from the bin master record (LAGP.IVNUM) via a follow-up GET on
+  // WMStorageBin. When activateNow=true the bin is locked and IVNUM is set.
+  const binFilter = `WarehouseNumber eq '${esc(warehouse)}' and StorageType eq '${esc(storageType)}' and StorageBin eq '${esc(bin)}'`;
+  const binData = await s4hGet(`${BASE}/WMStorageBin?$filter=${encodeURIComponent(binFilter)}&$select=InventoryDocNumber,InventoryStatus`);
+  const binRow  = binData?.value?.[0];
+  const invDocNumber = binRow?.InventoryDocNumber?.trim() || null;
+
+  if (!invDocNumber) {
+    return {
+      success:    true,
+      warning:    'Inventory document created but number unavailable via OData — verify in SAP via LI04.',
+      warehouse,
+      storageType,
+      bin,
+      activated:  activateNow
+    };
+  }
+
+  return {
+    success:            true,
+    warehouse,
+    storageType,
+    bin,
+    inventoryDocNumber: invDocNumber,
+    inventoryStatus:    binRow?.InventoryStatus ?? '',
+    activated:          activateNow,
+    message:            `Inventory document ${invDocNumber} created${activateNow ? ' and activated' : ''} for bin ${storageType}/${bin}`
+  };
+}

package/tools/goodsIssueMonitor.js

@@ -0,0 +1,92 @@
+import { s4hGet } from '../lib/s4hClient.js';
+import { esc } from '../lib/sanitize.js';
+
+const BASE = `/sap/opu/odata4/iwbep/all/srvd/sap/zsd_wmmcpservice/0001/WMGoodsIssueDelivery`;
+
+const GI_STATUS = {
+  'A': 'Not started',
+  'B': 'Partially picked',
+  'C': 'Goods issue posted',
+  ' ': 'Not started'
+};
+
+export async function getGoodsIssueMonitor({
+  warehouse,
+  shippingPoint,
+  includeCompleted = false,
+  material,
+  top = 50
+}) {
+  const filters = [
+    `WarehouseNumber eq '${esc(warehouse)}'`
+  ];
+  if (!includeCompleted) filters.push(`GIStatus ne 'C'`);
+  if (shippingPoint) filters.push(`ShippingPoint eq '${esc(shippingPoint)}'`);
+  if (material) filters.push(`Material eq '${esc(material)}'`);
+
+  const path = `${BASE}?$filter=${encodeURIComponent(filters.join(' and '))}&$top=${top}&$orderby=PlannedGIDate asc`;
+  const data = await s4hGet(path);
+  const rows = data.value ?? [];
+
+  const today = new Date();
+  const todayStr = today.toISOString().slice(0, 10);
+
+  // Group items by delivery number
+  const deliveryMap = {};
+  for (const r of rows) {
+    const del = r.DeliveryNumber;
+    if (!deliveryMap[del]) {
+      deliveryMap[del] = {
+        deliveryNumber: del,
+        shippingPoint:  r.ShippingPoint,
+        customer:       r.Customer?.trimStart?.() ?? r.Customer,
+        plannedGIDate:  r.PlannedGIDate,
+        actualGIDate:   r.ActualGIDate || null,
+        deliveryDate:   r.DeliveryDate,
+        giStatus:       r.GIStatus,
+        giStatusText:   GI_STATUS[r.GIStatus] ?? r.GIStatus,
+        isOverdue:      r.PlannedGIDate && r.PlannedGIDate < todayStr && r.GIStatus !== 'C',
+        items:          []
+      };
+    }
+    const deliveryQty = parseFloat(r.DeliveryQuantity ?? 0);
+    const pickedQty   = parseFloat(r.PickedQuantity ?? 0);
+    deliveryMap[del].items.push({
+      item:          r.DeliveryItem,
+      material:      r.Material?.trimStart?.() ?? r.Material,
+      plant:         r.Plant,
+      storageType:   r.StorageType,
+      bin:           r.StorageBin,
+      wmMovementType: r.WMMovementType,
+      wmTORequired:  r.WMTORequired,
+      deliveryQty,
+      pickedQty,
+      remainingQty:  Math.max(0, deliveryQty - pickedQty),
+      uom:           r.DeliveryUOM,
+      pickComplete:  deliveryQty > 0 && pickedQty >= deliveryQty
+    });
+  }
+
+  const deliveries = Object.values(deliveryMap);
+  const overdue    = deliveries.filter(d => d.isOverdue);
+
+  return {
+    warehouse,
+    filters: {
+      shippingPoint:    shippingPoint ?? 'all',
+      includeCompleted,
+      material:         material ?? 'all'
+    },
+    summary: {
+      totalDeliveries:  deliveries.length,
+      overdueDeliveries: overdue.length,
+      byStatus: {
+        notStarted: deliveries.filter(d => d.giStatus === 'A' || d.giStatus === ' ').length,
+        partial:    deliveries.filter(d => d.giStatus === 'B').length,
+        complete:   deliveries.filter(d => d.giStatus === 'C').length
+      }
+    },
+    truncated: rows.length === top,
+    deliveries
+  };
+}