@teincfood/core 0.9.3 → 0.9.4

package/dist/sync/command-queue.service.js

@@ -13,6 +13,7 @@ import { insertCommand, getPendingCommands, markCommandInFlight, updateCommandSt
 import { syncLogger } from "../utils/sync-logger";
 import { recordAttempt } from "../utils/attempt-stats";
 import { TransitionError, isReplayDeferred, TRANSITION_ERROR_CODES } from "./transition-errors";
+import { assertBusinessOpenForCreate } from "../utils/business-hours";
 import { extractApiErrorCode } from "../utils/error";
 import { generateEventId } from "../utils/uuid";
 import { generateLocalCalloutNumber, generateOrderNumberForBusiness, generateOrderNumber, } from "./order-number";
@@ -203,6 +204,14 @@ class CommandQueueService {
      * Returns an optimistic domain event so the caller can update the UI.
      */
     async enqueue(command) {
+        // Queue-time hours gate: a closed business can never legally sync an
+        // order:create, so fail fast with BusinessHoursError instead of queuing
+        // an optimistic order that would sit pending forever. Other command
+        // types (transitions, payments, delivery) are unaffected so in-progress
+        // orders can still be completed while closed.
+        if (command.command_type === "order:create") {
+            await assertBusinessOpenForCreate(command.business_id);
+        }
         const db = getSyncDatabase();
         const preparedCommand = injectLocalOrderNumber(command);
         await insertCommand(db, { command: preparedCommand, status: "pending" });

package/dist/sync/engine.js

@@ -149,6 +149,20 @@ export class SyncEngine {
                 return events;
             }
             catch (error) {
+                // Definitive hours-guard rejection (thrown by createLocalOrder before
+                // any HTTP): the business is closed. Surface it immediately — do not
+                // mark cloud unreachable and do not fall back to the outbox, which
+                // would either re-throw from the enqueue gate or mask a real decision
+                // as a connectivity problem.
+                const code = errorCodeOf(error);
+                if (code === "BUSINESS_CLOSED" || code === "HOURS_NOT_SYNCED") {
+                    syncLogger.warn("SyncEngine", "Command blocked: business closed", {
+                        command_type: command.command_type,
+                        command_id: command.command_id,
+                        code,
+                    });
+                    throw error;
+                }
                 // Definitive server rejection (4xx): the backend answered with a real
                 // decision (e.g. INVALID_TRANSITION, PAYMENT_REQUIRED, FORBIDDEN).
                 // Surface it instead of masking it as a connectivity problem and
@@ -159,7 +173,6 @@ export class SyncEngine {
                     ? error.status
                     : undefined;
                 if (rejectionStatus !== undefined && rejectionStatus >= 400 && rejectionStatus < 500) {
-                    const code = errorCodeOf(error);
                     syncLogger.warn("SyncEngine", "Cloud command rejected by server", {
                         command_type: command.command_type,
                         command_id: command.command_id,

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@teincfood/core",
-  "version": "0.9.3",
+  "version": "0.9.4",
   "description": "TeincFood shared offline-first core — types, sync engine, local DB, reference data, and repositories for mobile + desktop",
   "type": "module",
   "main": "dist/index.js",