@d19n/youfibre-odin-sdk 2.0.116 → 2.0.118

package/dist/wholesale-provider/kci-effect-bridge.d.ts

@@ -107,6 +107,20 @@ export declare type KciDispatchDecision = {
     dateFrom: string;
     timeBlockFrom: string;
     supplierReferenceFrom: string;
+    /**
+     * Optional co-effect — when present, the caller must ALSO apply a
+     * PROPERTY_WRITE on the named entity + properties after the
+     * schedule-appointment dispatch succeeds. Used by REAPPOINTED /
+     * AMENDED rows whose dto carries a new `provideServiceOrder.committedDate`
+     * for the rebooked appointment — without this co-effect the Order's
+     * CommittedDate would stay frozen at the ORIGINAL committed date and
+     * downstream consumers (Supply→Active cron, customer-facing dashboards)
+     * make decisions on stale data.
+     */
+    propertyWrite?: {
+        entityKey: YfKciTargetEntity;
+        properties: ReadonlyArray<string>;
+    };
 } | {
     kind: 'PROPERTY_WRITE';
     row: KciEffectRow;

package/dist/wholesale-provider/kci-effect-bridge.js

@@ -193,12 +193,22 @@ exports.KCI_EFFECT_ROWS = [
                 timeBlockFrom: 'provideServiceOrder.appointmentTimeslot.classification',
                 supplierReferenceFrom: 'provideServiceOrder.appointmentReservationReference',
             },
+            // Co-effect: REAPPOINTED's dto also carries an updated
+            // `provideServiceOrder.committedDate` for the rebooked appointment.
+            // Update Order.CommittedDate atomically with the schedule-appointment
+            // dispatch so downstream consumers (Supply→Active cron, dashboards)
+            // see the new supplier-committed date rather than the original.
+            propertyWrite: {
+                entityKey: 'OrderModule:Order',
+                properties: ['CommittedDate'],
+            },
             notes: 'Supplier-confirmed reschedule. Bundled effect: WO Pending→Scheduled + ' +
-                'set appointment date + stamp supplier reservation reference. Single ' +
-                'scheduleAppointment service call handles all three (skipSupplierBooking=true ' +
-                'because KCI is the commit, not a request). Target WO resolved from ' +
-                'dto.tenantWorkOrderReference; field-service picks the right Scheduled-stage ' +
-                'key per WO type.',
+                'set appointment date + stamp supplier reservation reference + write the ' +
+                'new Order.CommittedDate. Single scheduleAppointment service call handles ' +
+                'the WO transition + appointment write (skipSupplierBooking=true because ' +
+                'KCI is the commit, not a request); a follow-on PATCH on the Order writes ' +
+                'CommittedDate. Target WO resolved from dto.tenantWorkOrderReference; ' +
+                'field-service picks the right Scheduled-stage key per WO type.',
         },
         logAgainstOrder: true,
     },
@@ -394,26 +404,26 @@ exports.KCI_EFFECT_ROWS = [
         reasonCode: 'AMENDED',
         serviceOrderKind: 'PROVIDE',
         yfEffect: {
-            kind: 'WO_SCHEDULE_APPOINTMENT',
-            scheduleAppointment: {
-                dateFrom: 'provideServiceOrder.appointmentTimeslot.timeslotStartDateTime',
-                timeBlockFrom: 'provideServiceOrder.appointmentTimeslot.classification',
-                supplierReferenceFrom: 'provideServiceOrder.appointmentReservationReference',
+            kind: 'PROPERTY_WRITE',
+            propertyWrite: {
+                entityKey: 'OrderModule:Order',
+                properties: ['CommittedDate'],
             },
-            notes: 'Supplier-confirmed amendment — new appointment locked in. Bundled ' +
-                'effect: WO Pending→Scheduled + set appointment date + stamp supplier ' +
-                'reservation reference. Single scheduleAppointment service call handles ' +
-                'all three (skipSupplierBooking=true because NET has already accepted ' +
-                'the amendment server-side). Target WO resolved from ' +
-                'dto.tenantWorkOrderReference; field-service picks the right ' +
-                'Scheduled-stage key per WO type. ' +
-                'AMENDED is the canonical "supplier has accepted an amendment with new ' +
-                'appointment details" signal — emitted whether the amendment was ' +
-                'YF-tenant-driven (customer rebooks via /schedule-appointment → YF ' +
-                'amends NET → NET responds AMENDED) or NET-autonomous (rare; NET ' +
-                'reserves + amends server-side). Same JSONPaths as REAPPOINTED — the ' +
-                'two rows differ only in semantics: REAPPOINTED = NET initiated, ' +
-                'AMENDED = amendment-accept confirmation (tenant or auto-amend path).',
+            notes: 'Supplier confirmation that an amendment has been accepted with a new ' +
+                'appointment. In production this KCI always arrives AFTER the tenant ' +
+                'has driven the amendment — customer triggers YF /schedule-appointment ' +
+                '→ YF moves its WO Pending→Scheduled locally + calls NET amendOrder ' +
+                '→ NET responds with AMENDED. By the time AMENDED arrives at YF, the ' +
+                'WO is already at Scheduled with the new appointment data populated, ' +
+                'so the KCI carries no new state-change responsibility for the WO. ' +
+                'Only effect: update Order.CommittedDate from ' +
+                'provideServiceOrder.committedDate so the parent order tracks the ' +
+                'supplier-committed date for the rebooked slot (the initial COMMITTED ' +
+                'KCI set this on first commit; AMENDED overwrites with the post-amend ' +
+                'commitment). Earlier in this session a WO_SCHEDULE_APPOINTMENT bridge ' +
+                'was tried here but is wrong for production — it would cause a ' +
+                'double-amend cycle (YF amends NET → NET emits AMENDED → YF amends ' +
+                'NET again).',
         },
         logAgainstOrder: true,
         specRef: 'fibrecafe-docs/pages/orderstate.md (AMENDED) + Frank 2026-05-12 spec quote',
@@ -499,13 +509,14 @@ function decideKciDispatch(dto) {
             if (!row.yfEffect.scheduleAppointment) {
                 throw new Error(`bridge invariant: WO_SCHEDULE_APPOINTMENT row ${row.id} missing scheduleAppointment`);
             }
-            return {
-                kind: 'WO_SCHEDULE_APPOINTMENT',
-                row,
-                dateFrom: row.yfEffect.scheduleAppointment.dateFrom,
-                timeBlockFrom: row.yfEffect.scheduleAppointment.timeBlockFrom,
-                supplierReferenceFrom: row.yfEffect.scheduleAppointment.supplierReferenceFrom,
-            };
+            return Object.assign({ kind: 'WO_SCHEDULE_APPOINTMENT', row, dateFrom: row.yfEffect.scheduleAppointment.dateFrom, timeBlockFrom: row.yfEffect.scheduleAppointment.timeBlockFrom, supplierReferenceFrom: row.yfEffect.scheduleAppointment.supplierReferenceFrom }, (row.yfEffect.propertyWrite
+                ? {
+                    propertyWrite: {
+                        entityKey: row.yfEffect.propertyWrite.entityKey,
+                        properties: row.yfEffect.propertyWrite.properties,
+                    },
+                }
+                : {}));
         case 'PROPERTY_WRITE':
             if (!row.yfEffect.propertyWrite) {
                 throw new Error(`bridge invariant: PROPERTY_WRITE row ${row.id} missing propertyWrite`);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@d19n/youfibre-odin-sdk",
-  "version": "2.0.116",
+  "version": "2.0.118",
   "description": "",
   "author": "@d19n",
   "license": "UNLICENSED",