@spaceinvoices/react-ui 0.4.2 → 0.4.4

package/src/generated/schemas/webhook.ts

@@ -43,6 +43,7 @@ const createWebhookSchemaDefinition = z.object({
         "estimate.restored",
         "credit_note.created",
         "credit_note.issued",
+        "credit_note.voided",
         "credit_note.deleted",
         "credit_note.restored",
         "advance_invoice.created",
@@ -83,6 +84,11 @@ const createWebhookSchemaDefinition = z.object({
         "order_integration.created",
         "order_integration.updated",
         "order_integration.deleted",
+        "delivery_note.created",
+        "delivery_note.sent",
+        "delivery_note.cancelled",
+        "delivery_note.deleted",
+        "delivery_note.restored",
       ])
     )
     .min(1),
@@ -130,6 +136,7 @@ const updateWebhookSchemaDefinition = z
           "estimate.restored",
           "credit_note.created",
           "credit_note.issued",
+          "credit_note.voided",
           "credit_note.deleted",
           "credit_note.restored",
           "advance_invoice.created",
@@ -170,6 +177,11 @@ const updateWebhookSchemaDefinition = z
           "order_integration.created",
           "order_integration.updated",
           "order_integration.deleted",
+          "delivery_note.created",
+          "delivery_note.sent",
+          "delivery_note.cancelled",
+          "delivery_note.deleted",
+          "delivery_note.restored",
         ])
       )
       .min(1),

package/src/hooks/use-duplicate-document.ts

@@ -2,9 +2,11 @@ import type {
   AdvanceInvoice,
   CreateAdvanceInvoiceRequest,
   CreateCreditNoteRequest,
+  CreateDeliveryNoteRequest,
   CreateEstimateRequest,
   CreateInvoiceRequest,
   CreditNote,
+  DeliveryNote,
   Estimate,
   Invoice,
 } from "@spaceinvoices/js-sdk";
@@ -13,13 +15,14 @@ import { useQuery } from "@tanstack/react-query";
 import { useEntities } from "@/ui/providers/entities-context";
 import { useSDK } from "@/ui/providers/sdk-provider";
 
-export type DocumentType = "invoice" | "estimate" | "credit_note" | "advance_invoice";
-type Document = Invoice | Estimate | CreditNote | AdvanceInvoice;
+export type DocumentType = "invoice" | "estimate" | "credit_note" | "advance_invoice" | "delivery_note";
+type Document = Invoice | Estimate | CreditNote | AdvanceInvoice | DeliveryNote;
 type CreateRequest =
   | CreateInvoiceRequest
   | CreateEstimateRequest
   | CreateCreditNoteRequest
-  | CreateAdvanceInvoiceRequest;
+  | CreateAdvanceInvoiceRequest
+  | CreateDeliveryNoteRequest;
 
 /**
  * Get document type from ID prefix
@@ -29,6 +32,7 @@ export function getDocumentTypeFromId(id: string): DocumentType | null {
   if (id.startsWith("est_")) return "estimate";
   if (id.startsWith("cn_")) return "credit_note";
   if (id.startsWith("adv_")) return "advance_invoice";
+  if (id.startsWith("del_")) return "delivery_note";
   return null;
 }
 
@@ -45,6 +49,8 @@ export function getAllowedDuplicateTargets(sourceType: DocumentType): DocumentTy
       return ["credit_note"];
     case "advance_invoice":
       return ["advance_invoice", "invoice"];
+    case "delivery_note":
+      return ["delivery_note", "invoice"];
     default:
       return [];
   }
@@ -54,7 +60,7 @@ export function getAllowedDuplicateTargets(sourceType: DocumentType): DocumentTy
  * Transform a source document into form-compatible initial values
  * Copies relevant fields and resets computed/generated ones
  */
-function transformDocumentForDuplication(source: Document, _targetType: DocumentType): Partial<CreateRequest> {
+function transformDocumentForDuplication(source: Document, targetType: DocumentType): Partial<CreateRequest> {
   // Transform items - copy only the fields needed for creation
   // Use type assertion for items since all document item types share the same shape
   const sourceItems = source.items as Array<{
@@ -92,6 +98,10 @@ function transformDocumentForDuplication(source: Document, _targetType: Document
       }
     : undefined;
 
+  // When converting to a different type, link back to the source document
+  const sourceType = getDocumentTypeFromId(source.id);
+  const isConversion = sourceType && sourceType !== targetType;
+
   // Build base duplicate data
   const baseData: Partial<CreateRequest> = {
     // Customer - always pass both customer_id AND customer data when available
@@ -109,6 +119,8 @@ function transformDocumentForDuplication(source: Document, _targetType: Document
     date: new Date().toISOString(),
     // Number - leave empty for auto-generation
     // Do NOT copy: number, totals, taxes, payments, furs, eslog, vies, shareable_id
+    // Link back to source document when converting (e.g., delivery note → invoice)
+    ...(isConversion ? { linked_documents: [source.id] } : {}),
   };
 
   return baseData;
@@ -123,9 +135,20 @@ export type UseDuplicateDocumentOptions = {
   enabled?: boolean;
 };
 
+export type LinkedDocumentSummary = {
+  id: string;
+  type: string;
+  number: string;
+  date: string;
+  total_with_tax: number;
+  currency_code: string;
+};
+
 export type UseDuplicateDocumentResult = {
   /** Transformed initial values for the form */
   initialValues: Partial<CreateRequest> | undefined;
+  /** Source documents linked to this document (populated for conversions) */
+  sourceDocuments: LinkedDocumentSummary[];
   /** Loading state */
   isLoading: boolean;
   /** Error if fetch failed */
@@ -170,6 +193,8 @@ export function useDuplicateDocument({
         source = await sdk.estimates.get(sourceId, undefined, { entity_id: activeEntity.id });
       } else if (sourceType === "advance_invoice") {
         source = await sdk.advanceInvoices.get(sourceId, undefined, { entity_id: activeEntity.id });
+      } else if (sourceType === "delivery_note") {
+        source = await sdk.deliveryNotes.get(sourceId, undefined, { entity_id: activeEntity.id });
       } else {
         // Credit note
         source = await sdk.creditNotes.get(sourceId, undefined, { entity_id: activeEntity.id });
@@ -179,13 +204,31 @@ export function useDuplicateDocument({
         throw new Error("Source document not found");
       }
 
-      return transformDocumentForDuplication(source, targetType);
+      const initialValues = transformDocumentForDuplication(source, targetType);
+
+      // Build source document summaries for conversions (different source → target type)
+      const isConversion = sourceType !== targetType;
+      const sourceDocuments: LinkedDocumentSummary[] = isConversion
+        ? [
+            {
+              id: source.id,
+              type: sourceType,
+              number: (source as any).number || "",
+              date: (source as any).date || "",
+              total_with_tax: (source as any).total_with_tax ?? 0,
+              currency_code: (source as any).currency_code || "",
+            },
+          ]
+        : [];
+
+      return { initialValues, sourceDocuments };
     },
     enabled: enabled && !!sourceId && !!activeEntity?.id && !!sourceType,
   });
 
   return {
-    initialValues: query.data,
+    initialValues: query.data?.initialValues,
+    sourceDocuments: query.data?.sourceDocuments ?? [],
     isLoading: query.isLoading,
     error: query.error,
     sourceType,

package/src/hooks/use-next-document-number.ts

@@ -4,7 +4,7 @@ import { useSDK } from "@/ui/providers/sdk-provider";
 
 export const NEXT_DOCUMENT_NUMBER_CACHE_KEY = "next-document-number";
 
-export type DocumentType = "invoice" | "estimate" | "credit_note" | "advance_invoice";
+export type DocumentType = "invoice" | "estimate" | "credit_note" | "advance_invoice" | "delivery_note";
 
 /** Response type for next document number preview */
 export type NextDocumentNumberResponse = {
@@ -43,7 +43,7 @@ export function useNextDocumentNumber(
     queryFn: async () => {
       const response = await sdk.documents.getNextNumber(
         {
-          type,
+          type: type as "invoice",
           business_premise_name: options?.businessPremiseName,
           electronic_device_name: options?.electronicDeviceName,
         },

package/src/lib/fiscalization.ts

@@ -0,0 +1,12 @@
+/**
+ * Checks if a document has a failed fiscalization status.
+ * Returns the provider type that failed, or null if no failure.
+ */
+export function getFiscalizationFailureType(doc: {
+  furs?: { status?: string } | null;
+  fina?: { status?: string } | null;
+}): "furs" | "fina" | null {
+  if (doc.furs?.status === "failed") return "furs";
+  if (doc.fina?.status === "failed") return "fina";
+  return null;
+}

package/src/lib/schemas/advance-invoice.ts

@@ -11,7 +11,6 @@ import { customerSchema, lineItemSchema, transformItemsForApi } from "./shared";
 export const advanceInvoiceFormSchema = z.object({
   number: z.string().max(100).optional(),
   date: z.string().optional(),
-  date_due: z.string().optional(),
   customer_id: z.string().nullish(),
   customer: customerSchema,
   items: z.array(lineItemSchema).min(1),

package/src/providers/sdk-provider.tsx

@@ -1,6 +1,6 @@
 import SDK from "@spaceinvoices/js-sdk";
 import type { ReactNode } from "react";
-import { createContext, useContext, useEffect, useMemo, useState } from "react";
+import { createContext, useCallback, useContext, useEffect, useMemo, useState } from "react";
 
 import { AUTH_COOKIES } from "@/ui/lib/auth";
 import { flushCookies, getCookie } from "@/ui/lib/browser-cookies";
@@ -59,7 +59,7 @@ export function SDKProvider({
   /**
    * Initialize the SDK with the current auth token
    */
-  const initializeSDK = async () => {
+  const initializeSDK = useCallback(async () => {
     setIsLoading(true);
     setError(null);
 
@@ -92,17 +92,15 @@ export function SDKProvider({
     } finally {
       setIsLoading(false);
     }
-  };
+  }, [onUnauthorized]);
 
   // Initialize SDK on component mount
-  // biome-ignore lint/correctness/useExhaustiveDependencies: initializeSDK is intentionally omitted - should only run once on mount
   useEffect(() => {
     initializeSDK();
-  }, []); // Empty dependency array - only run on mount
+  }, [initializeSDK]);
 
   // IMPORTANT: useMemo must be called BEFORE any conditional returns
   // to satisfy React's Rules of Hooks
-  // biome-ignore lint/correctness/useExhaustiveDependencies: initializeSDK is stable (doesn't change) and including it would cause unnecessary re-renders
   const value = useMemo(
     () => ({
       sdk: sdk as SDK, // Will be non-null when children render (checked below)
@@ -111,7 +109,7 @@ export function SDKProvider({
       error,
       reinitialize: initializeSDK,
     }),
-    [sdk, isInitialized, isLoading, error], // Don't include initializeSDK - it's stable but causes re-renders
+    [sdk, isInitialized, isLoading, error, initializeSDK],
   );
 
   // Render appropriate UI based on SDK state

package/src/components/table/sortable-header.tsx

@@ -1,56 +0,0 @@
-import { ArrowDown, ArrowUp, ArrowUpDown } from "lucide-react";
-
-import { Button } from "@/ui/components/ui/button";
-import { cn } from "@/ui/lib/utils";
-
-type SortableHeaderProps = {
-  children: React.ReactNode;
-  field: string;
-  currentOrder?: string;
-  align?: "left" | "center" | "right";
-  onSort?: (order: string | null) => void;
-};
-
-/**
- * Sortable column header with visual indicators
- * Cycles through: none -> asc -> desc -> none
- */
-export function SortableHeader({ children, field, currentOrder, align = "left", onSort }: SortableHeaderProps) {
-  const isActive = currentOrder?.replace(/^-/, "") === field;
-  const isDesc = currentOrder?.startsWith("-");
-
-  // Determine next sort state
-  const getNextOrder = () => {
-    if (!isActive) return field; // Not active -> ascending
-    if (!isDesc) return `-${field}`; // Ascending -> descending
-    return null; // Descending -> none
-  };
-
-  const nextOrder = getNextOrder();
-
-  return (
-    <Button
-      variant="ghost"
-      size="sm"
-      onClick={() => onSort?.(nextOrder)}
-      className={cn(
-        "h-8 gap-1",
-        isActive && "font-semibold",
-        align === "left" && "-ml-3",
-        align === "right" && "-mr-3",
-      )}
-      aria-label={`Sort by ${field}${isActive ? (isDesc ? " descending" : " ascending") : ""}`}
-    >
-      {children}
-      {isActive ? (
-        isDesc ? (
-          <ArrowDown className="h-4 w-4" aria-hidden="true" />
-        ) : (
-          <ArrowUp className="h-4 w-4" aria-hidden="true" />
-        )
-      ) : (
-        <ArrowUpDown className="h-4 w-4 opacity-50" aria-hidden="true" />
-      )}
-    </Button>
-  );
-}