@spaceinvoices/react-ui 0.4.3 → 0.4.5

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

@@ -32,7 +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("dn_")) return "delivery_note";
+  if (id.startsWith("del_")) return "delivery_note";
   return null;
 }
 
@@ -50,7 +50,7 @@ export function getAllowedDuplicateTargets(sourceType: DocumentType): DocumentTy
     case "advance_invoice":
       return ["advance_invoice", "invoice"];
     case "delivery_note":
-      return ["delivery_note"];
+      return ["delivery_note", "invoice"];
     default:
       return [];
   }
@@ -60,10 +60,11 @@ 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<{
+    type?: string | null;
     name: string;
     description: string | null;
     quantity: number;
@@ -72,15 +73,21 @@ function transformDocumentForDuplication(source: Document, _targetType: Document
     taxes: Array<{ tax_id?: string }>;
   }>;
   const items = sourceItems?.map((item) => ({
+    type: item.type ?? undefined,
     name: item.name,
     description: item.description,
-    quantity: item.quantity,
-    // Use gross_price if set, otherwise use price. The form uses is_gross_price as a UI toggle.
-    price: item.gross_price ?? item.price,
-    // Copy tax references (tax_id), not computed tax data
-    taxes: item.taxes?.map((tax) => ({ tax_id: tax.tax_id })),
-    // Derive is_gross_price from whether gross_price is set
-    gross_price: item.gross_price ?? undefined,
+    // Separator items skip financial fields
+    ...(item.type !== "separator"
+      ? {
+          quantity: item.quantity,
+          // Use gross_price if set, otherwise use price. The form uses is_gross_price as a UI toggle.
+          price: item.gross_price ?? item.price,
+          // Copy tax references (tax_id), not computed tax data
+          taxes: item.taxes?.map((tax) => ({ tax_id: tax.tax_id })),
+          // Derive is_gross_price from whether gross_price is set
+          gross_price: item.gross_price ?? undefined,
+        }
+      : {}),
   }));
 
   // Build customer data - always copy if available (form needs this for display)
@@ -98,14 +105,18 @@ 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
     // The form needs customer data for display, even when customer_id is set
     ...(source.customer_id ? { customer_id: source.customer_id } : {}),
     ...(customerData ? { customer: customerData } : {}),
-    // Items
-    items,
+    // Items (cast needed: separator items omit financial fields like quantity)
+    items: items as any,
     // Currency
     currency_code: source.currency_code,
     // Notes
@@ -115,6 +126,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;
@@ -129,9 +142,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 */
@@ -187,13 +211,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-vies-check.ts

@@ -42,6 +42,8 @@ export interface UseViesCheckResult {
   reverseChargeApplies: boolean;
   /** Transaction type determined by VIES check */
   transactionType: ViesCheckResponse["transaction_type"] | undefined;
+  /** Customer country code returned by VIES check */
+  customerCountryCode: string | null;
   /** Warning message from VIES validation */
   warning: string | null;
   /** Whether VIES validation was successful */
@@ -124,6 +126,7 @@ export function useViesCheck({
     error: query.error,
     reverseChargeApplies: query.data?.reverse_charge_applies ?? false,
     transactionType: query.data?.transaction_type,
+    customerCountryCode: query.data?.customer_country_code ?? null,
     warning: query.data?.warning ?? null,
     viesValid: query.data?.vies_valid ?? null,
   };

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/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>
-  );
-}