feat: update gitignore and refactor callback query dispatch with error handling

This commit is contained in:
Marco Gallegos
2025-12-18 16:01:20 -06:00
parent 05922b1e0b
commit 611120cef6
2 changed files with 30 additions and 35 deletions

1
.gitignore vendored
View File

@@ -160,6 +160,7 @@ cython_debug/
# Google Service Account Credentials # Google Service Account Credentials
*.json *.json
!credentials.example.json !credentials.example.json
google_key.json
# Logs # Logs
*.log *.log

View File

@@ -73,13 +73,7 @@ async def button_dispatcher(update: Update, context: ContextTypes.DEFAULT_TYPE)
response_text = "Acción no reconocida." response_text = "Acción no reconocida."
reply_markup = None reply_markup = None
# Mapeo de `callback_data` a funciones `async` simple_handlers = {
async_handlers = {
'propose_activity': propose_activity_start,
}
# Mapeo para funciones síncronas que devuelven solo texto
sync_text_handlers = {
'view_agenda': get_agenda, 'view_agenda': get_agenda,
'view_requests_status': view_requests_status, 'view_requests_status': view_requests_status,
'schedule_appointment': request_appointment, 'schedule_appointment': request_appointment,
@@ -88,12 +82,12 @@ async def button_dispatcher(update: Update, context: ContextTypes.DEFAULT_TYPE)
'manage_users': lambda: "Función de gestión de usuarios no implementada.", 'manage_users': lambda: "Función de gestión de usuarios no implementada.",
} }
# Mapeo para funciones síncronas que devuelven texto y markup complex_handlers = {
sync_markup_handlers = {
'admin_menu': get_admin_secondary_menu, 'admin_menu': get_admin_secondary_menu,
'view_pending': view_pending, 'view_pending': view_pending,
} }
try:
if query.data in simple_handlers: if query.data in simple_handlers:
handler = simple_handlers[query.data] handler = simple_handlers[query.data]
logger.info(f"Ejecutando simple_handler para: {query.data}") logger.info(f"Ejecutando simple_handler para: {query.data}")
@@ -101,8 +95,6 @@ async def button_dispatcher(update: Update, context: ContextTypes.DEFAULT_TYPE)
response_text = await handler() response_text = await handler()
else: else:
response_text = handler() response_text = handler()
logger.info(f"Respuesta obtenida para {query.data}")
await query.edit_message_text(text=response_text, parse_mode='Markdown')
elif query.data in complex_handlers: elif query.data in complex_handlers:
handler = complex_handlers[query.data] handler = complex_handlers[query.data]
logger.info(f"Ejecutando complex_handler para: {query.data}") logger.info(f"Ejecutando complex_handler para: {query.data}")
@@ -110,8 +102,6 @@ async def button_dispatcher(update: Update, context: ContextTypes.DEFAULT_TYPE)
response_text, reply_markup = await handler() response_text, reply_markup = await handler()
else: else:
response_text, reply_markup = handler() response_text, reply_markup = handler()
logger.info(f"Respuesta obtenida para {query.data}")
await query.edit_message_text(text=response_text, reply_markup=reply_markup, parse_mode='Markdown')
elif query.data.startswith(('approve:', 'reject:')): elif query.data.startswith(('approve:', 'reject:')):
logger.info(f"Ejecutando acción de aprobación: {query.data}") logger.info(f"Ejecutando acción de aprobación: {query.data}")
response_text = handle_approval_action(query.data) response_text = handle_approval_action(query.data)
@@ -121,6 +111,10 @@ async def button_dispatcher(update: Update, context: ContextTypes.DEFAULT_TYPE)
logger.warning(f"Consulta no manejada por el despachador: {query.data}") logger.warning(f"Consulta no manejada por el despachador: {query.data}")
await query.edit_message_text(text=response_text) await query.edit_message_text(text=response_text)
return return
except Exception as exc:
logger.exception(f"Error al procesar la acción {query.data}: {exc}")
response_text = "❌ Ocurrió un error al procesar tu solicitud. Intenta de nuevo."
reply_markup = None
await query.edit_message_text(text=response_text, reply_markup=reply_markup, parse_mode='Markdown') await query.edit_message_text(text=response_text, reply_markup=reply_markup, parse_mode='Markdown')