Remove unnecessary console logs and improve code readability across multiple components

frontend/src/lib/components/dashboard/export/manifest/manifest-form.svelte

@@ -234,7 +234,6 @@
 			return;
 		}
 		
-		console.log('Fetching assigned invoices for:', formData.manifest_number);
 		lastFetchAttempt = new Date().toLocaleTimeString();
 		invoicesLoading = true;
 		const companyId = companyStore.activeCompany.id;
@@ -253,10 +252,6 @@
 			// Mark all as selected for visual consistency (since they ARE linked)
 			allInvoices.forEach((inv) => (inv.is_selected = true));			
 
-			console.log(
-				`Loaded ${allInvoices.length} export invoices for manifest ${formData.manifest_number}`
-			);
-
 			// Helper to normalize system
 			const getSystem = (s: string | undefined) => {
 				if (!s) return '';
@@ -297,13 +292,6 @@
 					isRepar(inv.invoice_type ?? undefined)
 			);
 
-			console.log('Invoices loaded:', {
-				total: allInvoices.length,
-				scaiiExp: scaiiExpInvoices.length,
-				scaiiRepar: scaiiReparInvoices.length,
-				scafExp: scafExpInvoices.length,
-				scafRepar: scafReparInvoices.length
-			});
 		} catch (e) {
 			console.error('Error fetching invoices:', e);
 			toast.error('Error al cargar facturas');
@@ -313,9 +301,7 @@
 	}
 
 	$effect(() => {
-		console.log('Active tab changed:', activeTab);
 		if (activeTab === 'facturas_expo') {
-			console.log('Triggering fetchInvoiceLists');
 			fetchInvoiceLists();
 		}
 	});
@@ -614,36 +600,25 @@
 
 				const updatePromises = [];				
 
-				console.log('Starting batch update for invoices:', allTrackedInvoices.length);
-
 				for (const inv of allTrackedInvoices) {
 					const currentLink = inv.compliance_mx?.manifest_number;
 					const shouldLink = inv.is_selected;
 
-					console.log(
-						`Invoice ${inv.invoice_number}: current=${currentLink}, selected=${shouldLink}, target=${formData.manifest_number}`
-					);
-
 					// Case 1: Needs Linking (Selected but not currently linked to this manifest)
 					if (shouldLink && currentLink !== formData.manifest_number) {					
-						console.log(`Linking invoice ${inv.id} to ${formData.manifest_number}`);
 						updatePromises.push(
 							manifestApi.updateInvoiceCompliance(inv.id, companyId, formData.manifest_number)
 						);
 					}
 					// Case 2: Needs Unlinking (Not Selected but currently linked to this manifest)
 					else if (!shouldLink && currentLink === formData.manifest_number) {						
-						console.log(`Unlinking invoice ${inv.id}`);
 						updatePromises.push(manifestApi.updateInvoiceCompliance(inv.id, companyId, null));
 					}
 				}
 
 				if (updatePromises.length > 0) {					
-					console.log('Sending update promises:', updatePromises.length);
 					await Promise.all(updatePromises);
 					toast.success(`${updatePromises.length} facturas actualizadas`);									
-				} else {
-					console.log('No invoices needed update');
 				}
 			}
 
@@ -1138,7 +1113,7 @@
 											<Label class="text-xs font-bold uppercase">Estatus Vehículo</Label>
 											<Select.Root
 												type="single"
-												bind:value={formData.vehicle_status}
+												value={formData.vehicle_status}
 												onValueChange={(v) => (formData.vehicle_status = v)}
 											>
 												<Select.Trigger class="h-9 px-3 text-xs">

frontend/src/lib/components/dashboard/goods/modales/currency-selector-dialog.svelte

@@ -40,11 +40,9 @@
 
     async function loadCurrencies() {
         loading = true;        
-        console.log("CurrencySelectorDialog: loading currencies (public)...");
         try {
             // FIX: Usar API pública, sin company_id
             const response = await currencyTypesApi.list(1, 100);            
-            console.log("Respuesta Monedas Public FULL:", response);
             
             if (response.error) {
                  console.error("CurrencySelectorDialog Error:", response.error);
@@ -55,7 +53,6 @@
             if (response.data?.items) {
                 items = response.data.items;
                 loaded = true;                
-                console.log("CurrencySelectorDialog: loaded items", items.length);
             } else {
                 console.warn("No se encontraron monedas (public):", response);
                 toast.error("No se encontraron monedas");

frontend/src/lib/components/dashboard/goods/parts/partForm.svelte

@@ -253,20 +253,12 @@
                 delete commonData.origin_country;
             }        
 
-            console.log("Submitting Part Data:", { 
-                isEdit, 
-                partId, 
-                commonData 
-            });
-
             if (isEdit && partId) {
                 // TODO: Verify partId is number/string as expected
                 const res = await partsApi.update(Number(partId), commonData, activeCompanyId);                
-                console.log("Update Response:", res);
                 if (res.error) throw new Error(res.error);
             } else {
                 const result = await partsApi.create({ ...commonData, company_id: activeCompanyId }, activeCompanyId);                
-                console.log("Create Response:", result);
                 if (result.error) { error = result.error; return; }
             }
             toast.success(isEdit ? "Parte actualizada" : "Parte creada");
@@ -421,7 +413,7 @@
                                             <Label for="fa_weight">Peso Unitario</Label>
                                             <div class="flex gap-2">
                                                 <Input type="number" step="0.0001" id="fa_weight" bind:value={formData.unit_weight} placeholder="0.0000" />
-                                                <Select.Root type="single" bind:value={formData.weight_type}>
+                                                <Select.Root type="single" value={formData.weight_type} onValueChange={(v) => formData.weight_type = v}>
                                                     <Select.Trigger class="w-[85px]">{formData.weight_type}</Select.Trigger>
                                                     <Select.Content>
                                                         <Select.Item value="KG">KG</Select.Item>
@@ -640,7 +632,7 @@
                                             <Label for="unit_weight">Peso Unitario</Label>
                                             <div class="flex gap-2">
                                                 <Input type="number" step="0.0001" id="unit_weight" bind:value={formData.unit_weight} placeholder="0.0000" class="bg-background"/>
-                                                <Select.Root type="single" bind:value={formData.weight_type}>
+                                                <Select.Root type="single" value={formData.weight_type} onValueChange={(v) => formData.weight_type = v}>
                                                     <Select.Trigger class="w-[90px] bg-background">{formData.weight_type}</Select.Trigger>
                                                     <Select.Content>
                                                         <Select.Item value="KG">KG</Select.Item>

frontend/src/lib/components/dashboard/invoices/create-edit-dialog.svelte

@@ -42,8 +42,6 @@
 		path_pdf: "",
 		path_xml: "",
 		// Compliance MX fields
-		pedimento: "",
-		pedimento_code: "",
 		remesa: null as number | null,
 		aduana: "",
 		customs_broker_id: "",
@@ -64,7 +62,7 @@
 		freight: null as number | null,
 		insurance: null as number | null,
 		iva_mn: null as number | null,
-		iva_factor: null as number | null,
+		iva_factor: null as string | null,
 		total_quantity: null as number | null,
 		gross_weight: null as number | null,
 		net_weight: null as number | null,
@@ -95,15 +93,13 @@
 				cfdi_uuid: item.cfdi_uuid || "",
 				path_pdf: item.path_pdf || "",
 				path_xml: item.path_xml || "",
-				pedimento: item.compliance_mx?.pedimento || "",
-				pedimento_code: item.compliance_mx?.pedimento_code || "",
 				remesa: item.compliance_mx?.remesa || null,
 				aduana: item.compliance_mx?.aduana || "",
-				customs_broker_id: item.compliance_mx?.customs_broker_id || "",
+				customs_broker_id: item.compliance_mx?.customs_broker_id?.toString() || "",
-				provider_id: item.compliance_mx?.provider_id || "",
+				provider_id: item.compliance_mx?.provider_id?.toString() || "",
-				sold_to_id: item.compliance_mx?.sold_to_id || "",
+				sold_to_id: item.compliance_mx?.sold_to_id?.toString() || "",
-				shipped_to_id: item.compliance_mx?.shipped_to_id || "",
+				shipped_to_id: item.compliance_mx?.shipped_to_id?.toString() || "",
-				shipped_by_id: item.compliance_mx?.shipped_by_id || "",
+				shipped_by_id: item.compliance_mx?.shipped_by_id?.toString() || "",
 				is_mixed: item.compliance_mx?.is_mixed || false,
 				waste_type: item.compliance_mx?.waste_type || "",
 				appendix_17: item.compliance_mx?.appendix_17 || null,
@@ -145,8 +141,6 @@
 			cfdi_uuid: "",
 			path_pdf: "",
 			path_xml: "",
-			pedimento: "",
-			pedimento_code: "",
 			remesa: null,
 			aduana: "",
 			customs_broker_id: "",
@@ -198,6 +192,7 @@
 			let response;
 			if (isEditing && item) {
 				const payload: UpdateInvoiceData = {
+					id: item.id,
 					operation_type: formData.operation_type,
 					invoice_type: formData.invoice_type || null,
 					invoice_number: formData.invoice_number || null,
@@ -213,18 +208,16 @@
 					path_pdf: formData.path_pdf || null,
 					path_xml: formData.path_xml || null,
 					compliance_mx: {
-						pedimento: formData.pedimento || null,
-						pedimento_code: formData.pedimento_code || null,
 						remesa: formData.remesa,
 						aduana: formData.aduana || null,
-						customs_broker_id: formData.customs_broker_id || null,
+						customs_broker_id: formData.customs_broker_id ? parseInt(formData.customs_broker_id) : null,
-						provider_id: formData.provider_id || null,
+						provider_id: formData.provider_id ? parseInt(formData.provider_id) : null,
-						sold_to_id: formData.sold_to_id || null,
+						sold_to_id: formData.sold_to_id ? parseInt(formData.sold_to_id) : null,
-						shipped_to_id: formData.shipped_to_id || null,
+						shipped_to_id: formData.shipped_to_id ? parseInt(formData.shipped_to_id) : null,
-						shipped_by_id: formData.shipped_by_id || null,
+						shipped_by_id: formData.shipped_by_id ? parseInt(formData.shipped_by_id) : null,
 						is_mixed: formData.is_mixed,
 						waste_type: formData.waste_type || null,
-						appendix_17: formData.appendix_17,
+						appendix_17: formData.appendix_17 || null,
 						edocument: formData.edocument || null
 					},
 					financials: {
@@ -246,9 +239,11 @@
 				response = await invoicesApi.update(item.id, companyStore.activeCompany.id, payload);
 			} else {
 				const payload: CreateInvoiceData = {					
+					system: "a76",
+					document_type: "invoice",					
 					operation_type: formData.operation_type,
-					invoice_type: formData.invoice_type || null,
+					invoice_type: formData.invoice_type,
-					invoice_number: formData.invoice_number || null,
+					invoice_number: formData.invoice_number,
 					project_number: formData.project_number || null,
 					purchase_order: formData.purchase_order || null,
 					related_doc_id: formData.related_doc_id,
@@ -261,18 +256,16 @@
 					path_pdf: formData.path_pdf || null,
 					path_xml: formData.path_xml || null,
 					compliance_mx: {
-						pedimento: formData.pedimento || null,
-						pedimento_code: formData.pedimento_code || null,
 						remesa: formData.remesa,
 						aduana: formData.aduana || null,
-						customs_broker_id: formData.customs_broker_id || null,
+						customs_broker_id: formData.customs_broker_id ? parseInt(formData.customs_broker_id) : null,
-						provider_id: formData.provider_id || null,
+						provider_id: formData.provider_id ? parseInt(formData.provider_id) : null,
-						sold_to_id: formData.sold_to_id || null,
+						sold_to_id: formData.sold_to_id ? parseInt(formData.sold_to_id) : null,
-						shipped_to_id: formData.shipped_to_id || null,
+						shipped_to_id: formData.shipped_to_id ? parseInt(formData.shipped_to_id) : null,
-						shipped_by_id: formData.shipped_by_id || null,
+						shipped_by_id: formData.shipped_by_id ? parseInt(formData.shipped_by_id) : null,
 						is_mixed: formData.is_mixed,
 						waste_type: formData.waste_type || null,
-						appendix_17: formData.appendix_17,
+						appendix_17: formData.appendix_17 || null,
 						edocument: formData.edocument || null
 					},
 					financials: {
@@ -308,7 +301,6 @@
                         
                     if (errorStr.includes('No existe un Tipo de Cambio registrado') || errorStr.includes('financials.exchange_rate')) {
                         // Interceptar error de tipo de cambio                        
-                        console.log("Interceptor: Exchange rate missing error caught (Invoice).");
                         error = null;
                         
                         const dateMatch = errorStr.match(/(\d{4}-\d{2}-\d{2})/);
@@ -510,24 +502,6 @@
 				<!-- Compliance Tab -->
 				<Tabs.Content value="compliance" class="space-y-4">
 					<div class="grid grid-cols-2 gap-4">
-						<div class="space-y-2">
-							<Label for="pedimento">Pedimento</Label>
-							<Input
-								id="pedimento"
-								bind:value={formData.pedimento}
-								placeholder="Número de pedimento"
-							/>
-						</div>
-
-						<div class="space-y-2">
-							<Label for="pedimento_code">Código de Pedimento</Label>
-							<Input
-								id="pedimento_code"
-								bind:value={formData.pedimento_code}
-								placeholder="R1, K1, etc."
-							/>
-						</div>
-
 						<div class="space-y-2">
 							<Label for="remesa">Remesa</Label>
 							<Input

frontend/src/lib/components/dashboard/invoices/edit/items/fa/country-dialog.svelte

@@ -27,8 +27,6 @@
 			});
 			if (response.ok) {
 				const data = await response.json();
-				console.log('Countries data received:', data);
-				console.log('First country sample:', data[0]);
 				if (Array.isArray(data)) {
 					countries = data;
 				} else if (data.items && Array.isArray(data.items)) {
@@ -38,7 +36,6 @@
 					countries = [];
 				}
 				filteredCountries = countries;				
-				console.log('Total countries loaded:', countries.length);
 			} else {
 				error = `Error: ${response.status} - ${response.statusText}`;
 				console.error('Error response:', await response.text());

frontend/src/lib/components/dashboard/invoices/edit/items/fa/tariff-fraction-dialog.svelte

@@ -48,7 +48,6 @@
 
 			if (response.ok) {
 				const data = await response.json();				
-				console.log('Tariff fractions data received:', data);
 
 				if (data.items && Array.isArray(data.items)) {
 					if (append) {
@@ -59,7 +58,6 @@
 					currentPage = data.page;
 					totalPages = data.pages;
 					hasMore = currentPage < totalPages;					
-					console.log(`Loaded page ${currentPage}/${totalPages}, total items: ${fractions.length}`);
 				} else {
 					console.error('Unexpected data format:', data);
 					if (!append) fractions = [];

frontend/src/lib/components/dashboard/invoices/edit/items/fa/unit-of-measure-dialog.svelte

@@ -28,7 +28,6 @@
 			});
 			if (response.ok) {
 				const data = await response.json();				
-				console.log('Units data received:', data);
 				// El backend puede devolver { items: [...] } o directamente un array
 				if (Array.isArray(data)) {
 					units = data;

frontend/src/lib/components/dashboard/invoices/edit/others-tab-form.svelte

@@ -100,15 +100,6 @@
 		// Función para cargar información
 		console.log('Cargar información');
 	}
-
-	$effect(() => {
-		console.log('OthersTabForm props:', {
-			invoiceType,
-			operationType,
-			isMixed: formData.is_mixed,
-			rule: formData.rule_3121_parties_ii
-		});
-	});
 </script>
 
 {#if invoiceType !== 'MEX'}

frontend/src/routes/dashboard/admin/roles/+page.svelte

@@ -85,8 +85,6 @@
 		try {
 			const response = await permissionsAPI.list();
 			allPermissions = response.items || [];
-			console.log('Total permissions loaded:', allPermissions.length);
-			console.log('Unique permission IDs:', new Set(allPermissions.map(p => p.id)).size);
 			if (selectedRole) {
 				updateAvailablePermissions();
 			}
@@ -118,17 +116,12 @@
 		const assignedIds = rolePermissions.map((rp) => rp.permission_id);
 		availablePermissions = allPermissions.filter((p) => !assignedIds.includes(p.id));
 		
-		console.log('Available permissions:', availablePermissions.length);
-		console.log('Unique available IDs:', new Set(availablePermissions.map(p => p.id)).size);
-		
 		updatePermissionsByModule();
 	}
 
 	function updatePermissionsByModule() {
 		const grouped = new Map<string, Permission[]>();
 		
-		console.log('Grouping permissions:', availablePermissions.length);
-		
 		// Filtrar por búsqueda
 		const filtered = availablePermissions.filter((p) => {
 			if (!permissionSearchQuery.trim()) return true;
@@ -147,8 +140,6 @@
 			grouped.get(p.module)!.push(p);
 		});
 		
-		console.log('Grouped by module:', Array.from(grouped.entries()).map(([k, v]) => `${k}: ${v.length}`));
-		
 		permissionsByModule = grouped;
 	}
 

frontend/src/routes/dashboard/general_catalogs/doda/edit/[[id]]/+page.svelte

@@ -115,13 +115,10 @@
 		const currentId = $page.params.id;
 		const companyId = companyStore.activeCompany?.id;
 
-		console.log('DEBUG: Effect triggered', { currentId, companyId });
 
 		if (currentId && companyId) {
-			console.log('DEBUG: Calling loadDoda with', currentId);
 			loadDoda(Number(currentId));
 		} else if (!currentId) {
-			console.log('DEBUG: No ID, resetting form');
 			formData = getEmptyForm();
 			error = null;
 		}
@@ -131,15 +128,8 @@
 		loading = true;
 		try {
 			const companyId = companyStore.activeCompany?.id;
-			console.log('DEBUG: loadDoda executing', { dodaId, companyId });
-
-			if (!companyId) {
-				console.error('DEBUG: No company ID available in loadDoda');
-				return;
-			}
 
 			const data = await getDoda(dodaId, companyId);
-			console.log('DEBUG: getDoda response', data);
 
 			if (data) {
 				formData = {

frontend/src/routes/dashboard/invoices/edit/[id]/+page.svelte

@@ -183,6 +183,7 @@
 		cantidad_guias_embarque: null,
 		destino_origen: '',
 		puerto_entrada: '',
+		vehicle_data: '',
 		fue_revisado_equipo: false,
 		sub_division: false,
 		funge_como_cd: false,
@@ -191,7 +192,16 @@
 		semaforo_verde_aduana_mexicana: false,
 		semaforo_verde_aduana_americana: false,
 		semaforo_rojo_aduana_mexicana: false,
-		semaforo_rojo_aduana_americana: false
+		semaforo_rojo_aduana_americana: false,
+		// Export fields
+		is_mixed: false,
+		reason_export: '1',
+		purchase_order: '',
+		payment_terms: '',
+		handling_fees: 0,
+		cfdi_uuid: '',
+		path_pdf: '',
+		path_xml: ''
 	};
 
 	function ensureItemsFormData(initial?: any) {

frontend/src/routes/dashboard/pedimentos/+page.svelte

@@ -86,14 +86,6 @@
 	function handleRowClick(pedimento: Pedimento) {
 		// Toggle: si ya está seleccionado, deseleccionar; si no, seleccionar
 		selectedId = selectedId === pedimento.id ? null : pedimento.id;		
-		console.log(
-			'🔘 Row clicked, pedimento.id:',
-			pedimento.id,
-			'selectedId:',
-			selectedId,
-			'hasSelection:',
-			hasSelection
-		);
 	}
 
 	function handleEditSelected() {