hvp-shared 6.85.0 → 6.87.0

package/dist/constants/client-billing.enums.d.ts

@@ -15,7 +15,22 @@ export declare enum ClientInvoiceStatus {
     /** CFDI stamped successfully, PDF/XML available */
     stamped = "stamped",
     /** Processing failed (see errorMessage) */
-    failed = "failed"
+    failed = "failed",
+    /** Previous month invoice — requires admin/manager authorization */
+    requires_authorization = "requires_authorization",
+    /** CFDI cancelled with SAT */
+    cancelled = "cancelled"
+}
+/**
+ * Date rule applied to an invoice based on ticket date vs emission date.
+ */
+export declare enum InvoiceDateRule {
+    /** Ticket date within 72h — stamp with original date */
+    within_72h = "within_72h",
+    /** Ticket date >72h ago, same fiscal month — stamp with today's date */
+    same_month = "same_month",
+    /** Ticket date >72h ago, previous fiscal month — requires authorization */
+    previous_month = "previous_month"
 }
 /**
  * All ClientInvoiceStatus values as array (for dropdowns/filters)
@@ -25,3 +40,10 @@ export declare const CLIENT_INVOICE_STATUS_VALUES: ClientInvoiceStatus[];
  * ClientInvoiceStatus labels in Spanish (for UI)
  */
 export declare const CLIENT_INVOICE_STATUS_LABELS: Record<ClientInvoiceStatus, string>;
+/**
+ * Petco (MASCOTAS Y COMPAÑIA) — different invoice email per branch.
+ * Only case that requires branch-specific emails.
+ */
+export declare const PETCO_RFC = "MAS121116E51";
+export declare const PETCO_BRANCH_EMAILS: Record<string, string>;
+export declare const INVOICE_DATE_RULE_LABELS: Record<InvoiceDateRule, string>;

package/dist/constants/client-billing.enums.js

@@ -6,7 +6,7 @@
  * request CFDI income invoices for their veterinary visits.
  */
 Object.defineProperty(exports, "__esModule", { value: true });
-exports.CLIENT_INVOICE_STATUS_LABELS = exports.CLIENT_INVOICE_STATUS_VALUES = exports.ClientInvoiceStatus = void 0;
+exports.INVOICE_DATE_RULE_LABELS = exports.PETCO_BRANCH_EMAILS = exports.PETCO_RFC = exports.CLIENT_INVOICE_STATUS_LABELS = exports.CLIENT_INVOICE_STATUS_VALUES = exports.InvoiceDateRule = exports.ClientInvoiceStatus = void 0;
 /**
  * Status of a client invoice request through the processing pipeline.
  */
@@ -20,7 +20,23 @@ var ClientInvoiceStatus;
     ClientInvoiceStatus["stamped"] = "stamped";
     /** Processing failed (see errorMessage) */
     ClientInvoiceStatus["failed"] = "failed";
+    /** Previous month invoice — requires admin/manager authorization */
+    ClientInvoiceStatus["requires_authorization"] = "requires_authorization";
+    /** CFDI cancelled with SAT */
+    ClientInvoiceStatus["cancelled"] = "cancelled";
 })(ClientInvoiceStatus || (exports.ClientInvoiceStatus = ClientInvoiceStatus = {}));
+/**
+ * Date rule applied to an invoice based on ticket date vs emission date.
+ */
+var InvoiceDateRule;
+(function (InvoiceDateRule) {
+    /** Ticket date within 72h — stamp with original date */
+    InvoiceDateRule["within_72h"] = "within_72h";
+    /** Ticket date >72h ago, same fiscal month — stamp with today's date */
+    InvoiceDateRule["same_month"] = "same_month";
+    /** Ticket date >72h ago, previous fiscal month — requires authorization */
+    InvoiceDateRule["previous_month"] = "previous_month";
+})(InvoiceDateRule || (exports.InvoiceDateRule = InvoiceDateRule = {}));
 /**
  * All ClientInvoiceStatus values as array (for dropdowns/filters)
  */
@@ -33,4 +49,21 @@ exports.CLIENT_INVOICE_STATUS_LABELS = {
     [ClientInvoiceStatus.processing]: "Procesando",
     [ClientInvoiceStatus.stamped]: "Facturada",
     [ClientInvoiceStatus.failed]: "Error",
+    [ClientInvoiceStatus.requires_authorization]: "Requiere autorización",
+    [ClientInvoiceStatus.cancelled]: "Cancelada",
+};
+/**
+ * Petco (MASCOTAS Y COMPAÑIA) — different invoice email per branch.
+ * Only case that requires branch-specific emails.
+ */
+exports.PETCO_RFC = "MAS121116E51";
+exports.PETCO_BRANCH_EMAILS = {
+    URBAN: "7132ger@petco.com.mx",
+    HARBOR: "7178ger@petco.com.mx",
+    MONTEJO: "7154ger@petco.com.mx",
+};
+exports.INVOICE_DATE_RULE_LABELS = {
+    [InvoiceDateRule.within_72h]: "Dentro de 72 horas",
+    [InvoiceDateRule.same_month]: "Mismo mes (>72h)",
+    [InvoiceDateRule.previous_month]: "Mes anterior",
 };

package/dist/constants/sat-income-invoice.d.ts

@@ -79,9 +79,9 @@ export type SATPaymentFormCode = (typeof SATPaymentForm)[keyof typeof SATPayment
 export declare const SAT_PAYMENT_FORM_LABELS: Record<SATPaymentFormCode, string>;
 /**
  * Maps QVET payment method strings to SAT payment form codes.
- * QVET uses Spanish descriptors; SAT uses numeric codes.
+ * QVET uses Spanish descriptors (UPPERCASE from collections report); SAT uses numeric codes.
  *
- * TODO: Validate actual QVET payment method strings from qvet_collection_details data.
+ * Source: qvet_collections.paymentMethod actual values (verified from production data).
  */
 export declare const QVET_PAYMENT_METHOD_TO_SAT: Record<string, SATPaymentFormCode>;
 /**

package/dist/constants/sat-income-invoice.js

@@ -116,17 +116,22 @@ exports.SAT_PAYMENT_FORM_LABELS = {
 };
 /**
  * Maps QVET payment method strings to SAT payment form codes.
- * QVET uses Spanish descriptors; SAT uses numeric codes.
+ * QVET uses Spanish descriptors (UPPERCASE from collections report); SAT uses numeric codes.
  *
- * TODO: Validate actual QVET payment method strings from qvet_collection_details data.
+ * Source: qvet_collections.paymentMethod actual values (verified from production data).
  */
 exports.QVET_PAYMENT_METHOD_TO_SAT = {
-    Efectivo: "01",
-    "Tarjeta de Crédito": "04",
-    "Tarjeta de Débito": "28",
-    Transferencia: "03",
-    "Tarjeta crédito": "04",
-    "Tarjeta débito": "28",
+    // Exact matches from QVET collections (report 11) — all uppercase
+    EFECTIVO: "01",
+    "TARJETA CREDITO": "04",
+    "TARJETA DEBITO": "28",
+    TRANSFERENCIA: "03",
+    "AMERICAN EXPRESS": "04", // AmEx is credit card
+    CHEQUE: "02",
+    TARJETA: "04", // Generic "card" → credit card
+    PUNTOS: "05", // Points → electronic wallet
+    "RECIBO DOMICILIADO": "03", // Direct debit → electronic transfer
+    // Ignore: RECTIFICADO, VIDEOCONSULTA, FormaPago_MigracionEntregasACuentaAVale → "99"
 };
 /**
  * Default SAT payment form when QVET payment method is unknown
@@ -140,14 +145,21 @@ function resolvePaymentForm(qvetPaymentMethod) {
     if (!qvetPaymentMethod)
         return exports.DEFAULT_SAT_PAYMENT_FORM;
     const normalized = qvetPaymentMethod.trim();
-    // Direct match
+    // Direct match (QVET collections uses uppercase)
     if (exports.QVET_PAYMENT_METHOD_TO_SAT[normalized]) {
         return exports.QVET_PAYMENT_METHOD_TO_SAT[normalized];
     }
-    // Case-insensitive partial match
+    // Uppercase match (in case input comes from other source)
+    const upper = normalized.toUpperCase();
+    if (exports.QVET_PAYMENT_METHOD_TO_SAT[upper]) {
+        return exports.QVET_PAYMENT_METHOD_TO_SAT[upper];
+    }
+    // Case-insensitive partial match as last resort
     const lc = normalized.toLowerCase();
     if (lc.includes("efectivo"))
         return "01";
+    if (lc.includes("american") || lc.includes("amex"))
+        return "04";
     if (lc.includes("crédito") || lc.includes("credito"))
         return "04";
     if (lc.includes("débito") || lc.includes("debito"))
@@ -156,5 +168,7 @@ function resolvePaymentForm(qvetPaymentMethod) {
         return "03";
     if (lc.includes("cheque"))
         return "02";
+    if (lc.includes("tarjeta"))
+        return "04";
     return exports.DEFAULT_SAT_PAYMENT_FORM;
 }

package/dist/constants/sat-income-invoice.test.d.ts

@@ -0,0 +1 @@
+export {};

package/dist/constants/sat-income-invoice.test.js

@@ -0,0 +1,90 @@
+"use strict";
+Object.defineProperty(exports, "__esModule", { value: true });
+const sat_income_invoice_1 = require("./sat-income-invoice");
+describe("resolvePaymentForm", () => {
+    describe("exact matches from QVET collections (uppercase)", () => {
+        it("should resolve EFECTIVO to 01 (Efectivo)", () => {
+            expect((0, sat_income_invoice_1.resolvePaymentForm)("EFECTIVO")).toBe("01");
+        });
+        it("should resolve TARJETA CREDITO to 04 (Tarjeta de crédito)", () => {
+            expect((0, sat_income_invoice_1.resolvePaymentForm)("TARJETA CREDITO")).toBe("04");
+        });
+        it("should resolve TARJETA DEBITO to 28 (Tarjeta de débito)", () => {
+            expect((0, sat_income_invoice_1.resolvePaymentForm)("TARJETA DEBITO")).toBe("28");
+        });
+        it("should resolve TRANSFERENCIA to 03 (Transferencia electrónica)", () => {
+            expect((0, sat_income_invoice_1.resolvePaymentForm)("TRANSFERENCIA")).toBe("03");
+        });
+        it("should resolve AMERICAN EXPRESS to 04 (Tarjeta de crédito)", () => {
+            expect((0, sat_income_invoice_1.resolvePaymentForm)("AMERICAN EXPRESS")).toBe("04");
+        });
+        it("should resolve CHEQUE to 02 (Cheque nominativo)", () => {
+            expect((0, sat_income_invoice_1.resolvePaymentForm)("CHEQUE")).toBe("02");
+        });
+        it("should resolve TARJETA to 04 (generic card → credit)", () => {
+            expect((0, sat_income_invoice_1.resolvePaymentForm)("TARJETA")).toBe("04");
+        });
+        it("should resolve PUNTOS to 05 (Monedero electrónico)", () => {
+            expect((0, sat_income_invoice_1.resolvePaymentForm)("PUNTOS")).toBe("05");
+        });
+        it("should resolve RECIBO DOMICILIADO to 03 (Transferencia electrónica)", () => {
+            expect((0, sat_income_invoice_1.resolvePaymentForm)("RECIBO DOMICILIADO")).toBe("03");
+        });
+    });
+    describe("case-insensitive matching", () => {
+        it("should resolve 'Efectivo' (mixed case) to 01", () => {
+            expect((0, sat_income_invoice_1.resolvePaymentForm)("Efectivo")).toBe("01");
+        });
+        it("should resolve 'tarjeta credito' (lowercase) to 04", () => {
+            expect((0, sat_income_invoice_1.resolvePaymentForm)("tarjeta credito")).toBe("04");
+        });
+        it("should resolve 'Tarjeta Debito' (title case) to 28", () => {
+            expect((0, sat_income_invoice_1.resolvePaymentForm)("Tarjeta Debito")).toBe("28");
+        });
+        it("should resolve 'transferencia' (lowercase) to 03", () => {
+            expect((0, sat_income_invoice_1.resolvePaymentForm)("transferencia")).toBe("03");
+        });
+        it("should resolve 'american express' (lowercase) to 04", () => {
+            expect((0, sat_income_invoice_1.resolvePaymentForm)("american express")).toBe("04");
+        });
+    });
+    describe("partial matching (fallback)", () => {
+        it("should resolve 'Tarjeta de Crédito' (with accents) to 04", () => {
+            expect((0, sat_income_invoice_1.resolvePaymentForm)("Tarjeta de Crédito")).toBe("04");
+        });
+        it("should resolve 'Tarjeta de Débito' (with accents) to 28", () => {
+            expect((0, sat_income_invoice_1.resolvePaymentForm)("Tarjeta de Débito")).toBe("28");
+        });
+        it("should resolve 'Pago en efectivo' to 01", () => {
+            expect((0, sat_income_invoice_1.resolvePaymentForm)("Pago en efectivo")).toBe("01");
+        });
+        it("should resolve 'Transferencia bancaria' to 03", () => {
+            expect((0, sat_income_invoice_1.resolvePaymentForm)("Transferencia bancaria")).toBe("03");
+        });
+    });
+    describe("unknown/edge cases → 99 (Por definir)", () => {
+        it("should resolve RECTIFICADO to 99", () => {
+            expect((0, sat_income_invoice_1.resolvePaymentForm)("RECTIFICADO")).toBe("99");
+        });
+        it("should resolve VIDEOCONSULTA to 99", () => {
+            expect((0, sat_income_invoice_1.resolvePaymentForm)("VIDEOCONSULTA")).toBe("99");
+        });
+        it("should resolve FormaPago_MigracionEntregasACuentaAVale to 99", () => {
+            expect((0, sat_income_invoice_1.resolvePaymentForm)("FormaPago_MigracionEntregasACuentaAVale")).toBe("99");
+        });
+        it("should resolve empty string to 99", () => {
+            expect((0, sat_income_invoice_1.resolvePaymentForm)("")).toBe("99");
+        });
+        it("should resolve unknown method to 99", () => {
+            expect((0, sat_income_invoice_1.resolvePaymentForm)("BITCOIN")).toBe("99");
+        });
+    });
+    describe("whitespace handling", () => {
+        it("should trim whitespace", () => {
+            expect((0, sat_income_invoice_1.resolvePaymentForm)("  EFECTIVO  ")).toBe("01");
+        });
+        it("should handle extra spaces in method name", () => {
+            expect((0, sat_income_invoice_1.resolvePaymentForm)("TARJETA  CREDITO")).toBe("04");
+        });
+    });
+});

package/dist/contracts/client-billing/requests.d.ts

@@ -39,13 +39,14 @@ export interface UpdateClientProfileRequest {
     cfdiUse?: string;
 }
 /**
- * Look up a ticket by albarán number and amount for verification.
+ * Look up a ticket by invoice number (e.g. "H-4543") and amount for verification.
  * Amount is required as security measure (prevents facturing others' tickets).
+ * Accepts variants: "H4543", "H-4543", "h-4543", "h 4543".
  *
  * @example POST /api/client-billing/invoices/lookup
  */
 export interface LookupTicketRequest {
-    deliveryNoteNumber: number;
+    invoiceNumber: string;
     amount: number;
 }
 /**
@@ -55,6 +56,6 @@ export interface LookupTicketRequest {
  * @example POST /api/client-billing/invoices
  */
 export interface CreateClientInvoiceRequest {
-    deliveryNoteNumber: number;
+    invoiceNumber: string;
     amount: number;
 }

package/dist/contracts/client-billing/responses.d.ts

@@ -47,12 +47,14 @@ export interface InvoiceLineItemResponse {
     satUnitCode?: string;
 }
 /**
- * Result of looking up a ticket by albarán number.
+ * Result of looking up a ticket by invoice number (e.g. "H-4543", "U-5434").
  * Returned before the client confirms invoice creation.
  *
  * @example POST /api/client-billing/invoices/lookup
  */
 export interface TicketLookupResponse {
+    invoiceNumber: string;
+    /** @deprecated Use invoiceNumber instead */
     deliveryNoteNumber: number;
     date: string;
     customerName: string;
@@ -69,7 +71,7 @@ export interface TicketLookupResponse {
  */
 export interface ClientInvoiceListItemResponse {
     id: string;
-    deliveryNoteNumber: number;
+    invoiceNumber: string;
     date: string;
     total: number;
     status: ClientInvoiceStatus;
@@ -83,7 +85,7 @@ export interface ClientInvoiceListItemResponse {
  */
 export interface ClientInvoiceDetailResponse {
     id: string;
-    deliveryNoteNumber: number;
+    invoiceNumber: string;
     date: string;
     customerName: string;
     subtotal: number;

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "hvp-shared",
-  "version": "6.85.0",
+  "version": "6.87.0",
   "description": "Shared types and utilities for HVP backend and frontend",
   "main": "dist/index.js",
   "types": "dist/index.d.ts",