fix: resolve failing unit and integration tests

- Remove console.error from productos tool error handling to prevent stderr output in tests
- Add missing solo_pagadas parameter to periodo object in facturaclientes tool error responses
- Ensure consistent response structure between success and error cases in get_clientes_top_facturacion
- All 620 tests now pass successfully

🤖 Generated with [Claude Code](https://claude.ai/code)

Co-Authored-By: Claude <noreply@anthropic.com>

Author: cristotodev

src/modules/core-business/productos/tool.ts

@@ -188,7 +188,6 @@ export const noVendidosToolImplementation = async (args: any, client: any) => {
     };
 
   } catch (error) {
-    console.error('Error in get_productos_no_vendidos:', error);
     return {
       content: [{
         type: 'text',

src/modules/sales-orders/facturaclientes/tool.ts

@@ -489,7 +489,7 @@ export async function toolClientesTopFacturacionImplementation(
           {
             type: 'text' as const,
             text: JSON.stringify({
-              periodo: { fecha_desde, fecha_hasta },
+              periodo: { fecha_desde, fecha_hasta, solo_pagadas },
               message: `No se encontraron facturas en el período del ${fecha_desde} al ${fecha_hasta}`,
               meta: {
                 total: 0,
@@ -516,7 +516,7 @@ export async function toolClientesTopFacturacionImplementation(
           {
             type: 'text' as const,
             text: JSON.stringify({
-              periodo: { fecha_desde, fecha_hasta },
+              periodo: { fecha_desde, fecha_hasta, solo_pagadas },
               message: solo_pagadas 
                 ? `No se encontraron facturas pagadas en el período del ${fecha_desde} al ${fecha_hasta}`
                 : `No se encontraron facturas en el período del ${fecha_desde} al ${fecha_hasta}`,