Feat: Añadir panel de métricas con estadísticas por idioma, plataforma y servidor
- Crear página dedicada /metrics con gráficos usando Chart.js - Implementar función get_translation_stats() en database.py - Añadir endpoint /api/stats en panel/main.py - Mostrar métricas de traducciones por idioma, plataforma y servidor Discord - Agregar tarjeta de acceso rápido a Métricas en el Dashboard - Actualizar action_plan_pro.md con el progreso completado
This commit is contained in:
@@ -2,24 +2,32 @@
|
||||
|
||||
## 🎯 Progreso de Mejoras
|
||||
|
||||
- [ ] **1. Panel de Métricas (Analytics & Estadísticas)**
|
||||
- Agregar gráficos en el Dashboard (FastAPI/Jinja).
|
||||
- Contabilizar traducciones totales, por idioma y por plataforma.
|
||||
- [x] **1. Panel de Métricas (Analytics & Estadísticas)**
|
||||
- ✅ Página dedicada `/metrics` con gráficos usando Chart.js.
|
||||
- ✅ Métricas de traducciones totales, por idioma y por plataforma.
|
||||
- ✅ Métricas por servidor de Discord.
|
||||
- ✅ Función `get_translation_stats()` en `botdiscord/database.py`.
|
||||
- ✅ Endpoint `/api/stats` en `panel/main.py`.
|
||||
- ✅ Tarjeta de acceso rápido a Métricas en el Dashboard.
|
||||
|
||||
- [ ] **2. Rate Limiting (Prevención de Spam y Abusos)**
|
||||
- Limitar botones de Discord/Telegram a X usos por minuto por usuario.
|
||||
- Implementar mensajes efímeros de advertencia por spam.
|
||||
- [x] **2. Rate Limiting (Prevención de Spam y Abusos)**
|
||||
- ✅ Implementado en `botdiscord/ui.py`: máximo 1 clic por usuario/idioma cada 3 segundos.
|
||||
- ✅ Aviso efímero automático si el usuario excede el límite.
|
||||
|
||||
- [ ] **3. Sistema de Logging Real y Monitoreo (Observabilidad)**
|
||||
- Reemplazar `print()` por la librería estandar `logging` con guardado en disco rotatorio (archivos diarios).
|
||||
- Enmascarar errores y alertas.
|
||||
- [x] **3. Sistema de Logging Real y Monitoreo (Observabilidad)**
|
||||
- ✅ Módulo `utils/logger.py` con rotación de archivos diaria (14 días de historial).
|
||||
- ✅ Reemplazados todos los `print()` por `log.info()`, `log.warning()`, `log.error()`.
|
||||
|
||||
- [ ] **4. Reemplazo de Caché en RAM por Redis**
|
||||
- Configurar contenedor oficial de Redis en OMV (`docker-compose-redis.yaml`).
|
||||
- Adaptar `botdiscord/database.py` y `panel/main.py` para usar Redis si está disponible.
|
||||
- [x] **4. Reemplazo de Caché en RAM por Redis**
|
||||
- ✅ `utils/cache.py` creado con fallback a RAM si Redis no está disponible.
|
||||
- ✅ Caché en cascada en `panel/main.py` y `translate.py`: Redis → RAM → DB → LibreTranslate.
|
||||
- ✅ `docker-compose-redis.yaml` creado y Redis instalado en OMV.
|
||||
- ✅ Variables de entorno configuradas en `docker-compose.yml` y `.env.example`.
|
||||
|
||||
- [ ] **5. Sistema de "Health Check" de LibreTranslate**
|
||||
- Comprobar que el endpoint de traducción está vivo antes de lanzar errores, devolviendo mensaje de mantenimiento.
|
||||
- [x] **5. Sistema de "Health Check" de LibreTranslate**
|
||||
- ✅ Verificación de disponibilidad del endpoint `/languages` de LibreTranslate.
|
||||
- ✅ Resultado cacheado 30s para no sobrecargar el servidor.
|
||||
- ✅ Mensaje amigable `⚠️ Servicio de traducción en mantenimiento` cuando está caído.
|
||||
|
||||
---
|
||||
*Este documento guiará la transformación del ecosistema de bots a un entorno de producción masiva.*
|
||||
|
||||
@@ -974,3 +974,115 @@ def delete_discord_server(server_id: int):
|
||||
c.execute("DELETE FROM discord_servers WHERE server_id = ?", (server_id,))
|
||||
conn.commit()
|
||||
conn.close()
|
||||
|
||||
def get_translation_stats() -> dict:
|
||||
"""Obtiene estadísticas de traducciones totales, por idioma y por plataforma"""
|
||||
db_type = get_db_type()
|
||||
|
||||
if db_type == "mysql":
|
||||
conn = get_connection()
|
||||
cursor = conn.cursor(dictionary=True)
|
||||
|
||||
cursor.execute("SELECT COUNT(*) as total FROM translations")
|
||||
total_result = cursor.fetchone()
|
||||
total_translations = total_result['total'] if total_result else 0
|
||||
|
||||
cursor.execute("""
|
||||
SELECT target_lang, COUNT(*) as count
|
||||
FROM translations
|
||||
GROUP BY target_lang
|
||||
ORDER BY count DESC
|
||||
""")
|
||||
by_language = {row['target_lang']: row['count'] for row in cursor.fetchall()}
|
||||
|
||||
cursor.execute("""
|
||||
SELECT bot_type, COUNT(*) as count
|
||||
FROM translations t
|
||||
JOIN messages m ON t.message_id = m.message_id
|
||||
GROUP BY bot_type
|
||||
""")
|
||||
by_platform = {row['bot_type']: row['count'] for row in cursor.fetchall()}
|
||||
|
||||
cursor.execute("""
|
||||
SELECT DATE(created_at) as date, COUNT(*) as count
|
||||
FROM translations
|
||||
WHERE created_at >= DATE_SUB(NOW(), INTERVAL 30 DAY)
|
||||
GROUP BY DATE(created_at)
|
||||
ORDER BY date
|
||||
""")
|
||||
by_day = [{'date': str(row['date']), 'count': row['count']} for row in cursor.fetchall()]
|
||||
|
||||
cursor.execute("""
|
||||
SELECT COALESCE(s.server_name, CONCAT('Servidor ', m.guild_id)) as server_name,
|
||||
COUNT(*) as count
|
||||
FROM translations t
|
||||
JOIN messages m ON t.message_id = m.message_id
|
||||
LEFT JOIN discord_servers s ON m.guild_id = s.server_id
|
||||
WHERE m.guild_id IS NOT NULL
|
||||
GROUP BY m.guild_id
|
||||
ORDER BY count DESC
|
||||
""")
|
||||
by_server = [{'server': row['server_name'], 'count': row['count']} for row in cursor.fetchall()]
|
||||
|
||||
cursor.close()
|
||||
|
||||
return {
|
||||
'total': total_translations,
|
||||
'by_language': by_language,
|
||||
'by_platform': by_platform,
|
||||
'by_day': by_day,
|
||||
'by_server': by_server
|
||||
}
|
||||
else:
|
||||
conn = get_connection()
|
||||
c = conn.cursor()
|
||||
|
||||
c.execute("SELECT COUNT(*) FROM translations")
|
||||
total_translations = c.fetchone()[0] or 0
|
||||
|
||||
c.execute("""
|
||||
SELECT target_lang, COUNT(*) as count
|
||||
FROM translations
|
||||
GROUP BY target_lang
|
||||
ORDER BY count DESC
|
||||
""")
|
||||
by_language = {row[0]: row[1] for row in c.fetchall()}
|
||||
|
||||
c.execute("""
|
||||
SELECT m.bot_type, COUNT(*) as count
|
||||
FROM translations t
|
||||
JOIN messages m ON t.message_id = m.message_id
|
||||
GROUP BY m.bot_type
|
||||
""")
|
||||
by_platform = {row[0]: row[1] for row in c.fetchall()}
|
||||
|
||||
c.execute("""
|
||||
SELECT DATE(created_at) as date, COUNT(*) as count
|
||||
FROM translations
|
||||
WHERE created_at >= DATE('now', '-30 days')
|
||||
GROUP BY DATE(created_at)
|
||||
ORDER BY date
|
||||
""")
|
||||
by_day = [{'date': row[0], 'count': row[1]} for row in c.fetchall()]
|
||||
|
||||
c.execute("""
|
||||
SELECT COALESCE(s.server_name, 'Servidor ' || m.guild_id) as server_name,
|
||||
COUNT(*) as count
|
||||
FROM translations t
|
||||
JOIN messages m ON t.message_id = m.message_id
|
||||
LEFT JOIN discord_servers s ON m.guild_id = s.server_id
|
||||
WHERE m.guild_id IS NOT NULL
|
||||
GROUP BY m.guild_id
|
||||
ORDER BY count DESC
|
||||
""")
|
||||
by_server = [{'server': row[0], 'count': row[1]} for row in c.fetchall()]
|
||||
|
||||
conn.close()
|
||||
|
||||
return {
|
||||
'total': total_translations,
|
||||
'by_language': by_language,
|
||||
'by_platform': by_platform,
|
||||
'by_day': by_day,
|
||||
'by_server': by_server
|
||||
}
|
||||
|
||||
@@ -24,7 +24,7 @@ load_config() # Cargamos configuración inmediatamente
|
||||
from botdiscord.database import (
|
||||
init_db, get_ui_translation, save_ui_translation,
|
||||
get_admins, get_admin_by_username, add_admin, delete_admin,
|
||||
_normalize_text
|
||||
_normalize_text, get_translation_stats
|
||||
)
|
||||
init_db() # Aseguramos que las tablas existan antes de que FastAPI atienda peticiones
|
||||
from botdiscord.translate import translate_text
|
||||
@@ -684,6 +684,25 @@ async def diagnosis_page(request: Request):
|
||||
"config": config
|
||||
})
|
||||
|
||||
@app.get("/api/stats")
|
||||
async def get_stats(request: Request):
|
||||
if request.cookies.get("auth") != "ok":
|
||||
raise HTTPException(status_code=401)
|
||||
|
||||
stats = get_translation_stats()
|
||||
return stats
|
||||
|
||||
@app.get("/metrics")
|
||||
async def metrics_page(request: Request):
|
||||
if request.cookies.get("auth") != "ok":
|
||||
return RedirectResponse(url="/login")
|
||||
|
||||
username = request.cookies.get("username", "")
|
||||
return templates.TemplateResponse("metrics.html", {
|
||||
"request": request,
|
||||
"username": username
|
||||
})
|
||||
|
||||
if __name__ == "__main__":
|
||||
import uvicorn
|
||||
web_config = get_web_config()
|
||||
|
||||
@@ -87,6 +87,16 @@
|
||||
</div>
|
||||
</div>
|
||||
</div>
|
||||
|
||||
<div class="col-md-4">
|
||||
<div class="card text-white bg-danger mb-3">
|
||||
<div class="card-body">
|
||||
<h5 class="card-title"><i class="bi bi-bar-chart"></i> {{ "Métricas" | translate(lang) }}</h5>
|
||||
<p class="card-text">{{ "Ver estadísticas y análisis de traducciones" | translate(lang) }}</p>
|
||||
<a href="/metrics" class="btn btn-light btn-sm">{{ "Ver Métricas" | translate(lang) }}</a>
|
||||
</div>
|
||||
</div>
|
||||
</div>
|
||||
</div>
|
||||
|
||||
<div class="card mt-4">
|
||||
|
||||
244
panel/templates/metrics.html
Normal file
244
panel/templates/metrics.html
Normal file
@@ -0,0 +1,244 @@
|
||||
{% set lang = request.cookies.get('panel_lang', 'es') %}
|
||||
<!DOCTYPE html>
|
||||
<html lang="{{ lang }}">
|
||||
<head>
|
||||
<meta charset="UTF-8">
|
||||
<meta name="viewport" content="width=device-width, initial-scale=1.0">
|
||||
<title>{{ "Métricas - Bots de Traducción" | translate(lang) }}</title>
|
||||
<link href="https://cdn.jsdelivr.net/npm/bootstrap@5.3.0/dist/css/bootstrap.min.css" rel="stylesheet">
|
||||
<link rel="stylesheet" href="https://cdn.jsdelivr.net/npm/bootstrap-icons@1.11.0/font/bootstrap-icons.css">
|
||||
</head>
|
||||
<body>
|
||||
<nav class="navbar navbar-expand-lg navbar-dark bg-dark">
|
||||
<div class="container-fluid">
|
||||
<a class="navbar-brand" href="/dashboard">
|
||||
<i class="bi bi-translate"></i> {{ "Bots de Traducción" | translate(lang) }}
|
||||
</a>
|
||||
<a href="/dashboard" class="btn btn-outline-light btn-sm">{{ "Volver al Dashboard" | translate(lang) }}</a>
|
||||
</div>
|
||||
</nav>
|
||||
|
||||
<div class="container mt-5">
|
||||
<h2 class="mb-4">📊 {{ "Métricas y Estadísticas" | translate(lang) }}</h2>
|
||||
|
||||
<div class="row mb-4">
|
||||
<div class="col-md-3">
|
||||
<div class="card text-white bg-primary mb-3">
|
||||
<div class="card-body text-center">
|
||||
<h1 id="stat-total">-</h1>
|
||||
<small>{{ "Total Traducciones" | translate(lang) }}</small>
|
||||
</div>
|
||||
</div>
|
||||
</div>
|
||||
<div class="col-md-3">
|
||||
<div class="card text-white bg-success mb-3">
|
||||
<div class="card-body text-center">
|
||||
<h1 id="stat-discord">-</h1>
|
||||
<small>{{ "Discord" | translate(lang) }}</small>
|
||||
</div>
|
||||
</div>
|
||||
</div>
|
||||
<div class="col-md-3">
|
||||
<div class="card text-white bg-info mb-3">
|
||||
<div class="card-body text-center">
|
||||
<h1 id="stat-telegram">-</h1>
|
||||
<small>{{ "Telegram" | translate(lang) }}</small>
|
||||
</div>
|
||||
</div>
|
||||
</div>
|
||||
<div class="col-md-3">
|
||||
<div class="card text-white bg-warning mb-3">
|
||||
<div class="card-body text-center">
|
||||
<h1 id="stat-languages">-</h1>
|
||||
<small>{{ "Idiomas usados" | translate(lang) }}</small>
|
||||
</div>
|
||||
</div>
|
||||
</div>
|
||||
</div>
|
||||
|
||||
<div class="row">
|
||||
<div class="col-md-6">
|
||||
<div class="card mb-4">
|
||||
<div class="card-header">
|
||||
<h5 class="mb-0"><i class="bi bi-pie-chart"></i> {{ "Traducciones por Idioma" | translate(lang) }}</h5>
|
||||
</div>
|
||||
<div class="card-body">
|
||||
<canvas id="chart-by-language" height="300"></canvas>
|
||||
</div>
|
||||
</div>
|
||||
</div>
|
||||
<div class="col-md-6">
|
||||
<div class="card mb-4">
|
||||
<div class="card-header">
|
||||
<h5 class="mb-0"><i class="bi bi-server"></i> {{ "Traducciones por Servidor" | translate(lang) }}</h5>
|
||||
</div>
|
||||
<div class="card-body">
|
||||
<canvas id="chart-by-server" height="300"></canvas>
|
||||
</div>
|
||||
</div>
|
||||
</div>
|
||||
</div>
|
||||
|
||||
<div class="row">
|
||||
<div class="col-md-6">
|
||||
<div class="card">
|
||||
<div class="card-header">
|
||||
<h5 class="mb-0"><i class="bi bi-list-ul"></i> {{ "Detalle por Idioma" | translate(lang) }}</h5>
|
||||
</div>
|
||||
<div class="card-body">
|
||||
<table class="table table-striped" id="languages-table">
|
||||
<thead>
|
||||
<tr>
|
||||
<th>{{ "Idioma" | translate(lang) }}</th>
|
||||
<th>{{ "Cantidad" | translate(lang) }}</th>
|
||||
<th>{{ "Porcentaje" | translate(lang) }}</th>
|
||||
</tr>
|
||||
</thead>
|
||||
<tbody id="languages-body">
|
||||
</tbody>
|
||||
</table>
|
||||
</div>
|
||||
</div>
|
||||
</div>
|
||||
<div class="col-md-6">
|
||||
<div class="card">
|
||||
<div class="card-header">
|
||||
<h5 class="mb-0"><i class="bi bi-list-ul"></i> {{ "Detalle por Servidor" | translate(lang) }}</h5>
|
||||
</div>
|
||||
<div class="card-body">
|
||||
<table class="table table-striped" id="servers-table">
|
||||
<thead>
|
||||
<tr>
|
||||
<th>{{ "Servidor" | translate(lang) }}</th>
|
||||
<th>{{ "Cantidad" | translate(lang) }}</th>
|
||||
<th>{{ "Porcentaje" | translate(lang) }}</th>
|
||||
</tr>
|
||||
</thead>
|
||||
<tbody id="servers-body">
|
||||
</tbody>
|
||||
</table>
|
||||
</div>
|
||||
</div>
|
||||
</div>
|
||||
</div>
|
||||
|
||||
<div class="card mt-4">
|
||||
<div class="card-header">
|
||||
<h5 class="mb-0"><i class="bi bi-graph-up"></i> {{ "Actividad (últimos 30 días)" | translate(lang) }}</h5>
|
||||
</div>
|
||||
<div class="card-body">
|
||||
<canvas id="chart-by-day" height="100"></canvas>
|
||||
</div>
|
||||
</div>
|
||||
</div>
|
||||
<script src="https://cdn.jsdelivr.net/npm/bootstrap@5.3.0/dist/js/bootstrap.bundle.min.js"></script>
|
||||
<script src="https://cdn.jsdelivr.net/npm/chart.js@4.4.0/dist/chart.umd.min.js"></script>
|
||||
<script>
|
||||
async function loadStats() {
|
||||
try {
|
||||
const resp = await fetch('/api/stats');
|
||||
const data = await resp.json();
|
||||
|
||||
document.getElementById('stat-total').textContent = data.total || 0;
|
||||
document.getElementById('stat-discord').textContent = data.by_platform?.discord || 0;
|
||||
document.getElementById('stat-telegram').textContent = data.by_platform?.telegram || 0;
|
||||
document.getElementById('stat-languages').textContent = Object.keys(data.by_language || {}).length;
|
||||
|
||||
const langLabels = Object.keys(data.by_language || {});
|
||||
const langData = Object.values(data.by_language || {});
|
||||
const total = data.total || 1;
|
||||
|
||||
if (langLabels.length > 0) {
|
||||
new Chart(document.getElementById('chart-by-language'), {
|
||||
type: 'doughnut',
|
||||
data: {
|
||||
labels: langLabels,
|
||||
datasets: [{
|
||||
data: langData,
|
||||
backgroundColor: [
|
||||
'#0d6efd', '#198754', '#dc3545', '#ffc107', '#0dcaf0',
|
||||
'#6f42c1', '#d63384', '#fd7e14', '#20c997', '#795548',
|
||||
'#6610f2', '#e83e8c', '#fd7e14', '#17a2b8', '#28a745'
|
||||
]
|
||||
}]
|
||||
},
|
||||
options: {
|
||||
responsive: true,
|
||||
plugins: {
|
||||
legend: { position: 'right' },
|
||||
tooltip: {
|
||||
callbacks: {
|
||||
label: function(context) {
|
||||
const pct = ((context.raw / total) * 100).toFixed(1);
|
||||
return context.label + ': ' + context.raw + ' (' + pct + '%)';
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
}
|
||||
});
|
||||
|
||||
const tbody = document.getElementById('languages-body');
|
||||
langLabels.forEach((label, i) => {
|
||||
const pct = ((langData[i] / total) * 100).toFixed(1);
|
||||
tbody.innerHTML += '<tr><td>' + label + '</td><td>' + langData[i] + '</td><td>' + pct + '%</td></tr>';
|
||||
});
|
||||
}
|
||||
|
||||
const serverLabels = (data.by_server || []).map(d => d.server);
|
||||
const serverData = (data.by_server || []).map(d => d.count);
|
||||
if (serverLabels.length > 0) {
|
||||
new Chart(document.getElementById('chart-by-server'), {
|
||||
type: 'bar',
|
||||
data: {
|
||||
labels: serverLabels,
|
||||
datasets: [{
|
||||
label: '{{ "Traducciones" | translate(lang) }}',
|
||||
data: serverData,
|
||||
backgroundColor: '#198754'
|
||||
}]
|
||||
},
|
||||
options: {
|
||||
responsive: true,
|
||||
indexAxis: 'y',
|
||||
scales: { x: { beginAtZero: true } }
|
||||
}
|
||||
});
|
||||
|
||||
const tbody = document.getElementById('servers-body');
|
||||
serverData.forEach((count, i) => {
|
||||
const pct = ((count / total) * 100).toFixed(1);
|
||||
tbody.innerHTML += '<tr><td>' + serverLabels[i] + '</td><td>' + count + '</td><td>' + pct + '%</td></tr>';
|
||||
});
|
||||
}
|
||||
|
||||
const dayLabels = (data.by_day || []).map(d => d.date);
|
||||
const dayData = (data.by_day || []).map(d => d.count);
|
||||
if (dayLabels.length > 0) {
|
||||
new Chart(document.getElementById('chart-by-day'), {
|
||||
type: 'line',
|
||||
data: {
|
||||
labels: dayLabels,
|
||||
datasets: [{
|
||||
label: '{{ "Traducciones" | translate(lang) }}',
|
||||
data: dayData,
|
||||
borderColor: '#0d6efd',
|
||||
backgroundColor: 'rgba(13, 110, 253, 0.1)',
|
||||
fill: true,
|
||||
tension: 0.3
|
||||
}]
|
||||
},
|
||||
options: {
|
||||
responsive: true,
|
||||
scales: { y: { beginAtZero: true } }
|
||||
}
|
||||
});
|
||||
}
|
||||
} catch (e) {
|
||||
console.error('Error loading stats:', e);
|
||||
}
|
||||
}
|
||||
loadStats();
|
||||
</script>
|
||||
</body>
|
||||
</html>
|
||||
Reference in New Issue
Block a user