From 19bae94f4c2b8daf621bd5b597f91b720b2ea894 Mon Sep 17 00:00:00 2001 From: Agent Date: Sat, 13 Jun 2026 16:48:11 -0400 Subject: [PATCH] remove plan file from tracking --- .opencode/plans/server-thresholds.md | 256 --------------------------- 1 file changed, 256 deletions(-) delete mode 100644 .opencode/plans/server-thresholds.md diff --git a/.opencode/plans/server-thresholds.md b/.opencode/plans/server-thresholds.md deleted file mode 100644 index 3c439f5..0000000 --- a/.opencode/plans/server-thresholds.md +++ /dev/null @@ -1,256 +0,0 @@ -# Plan: Umbrales de notificación por servidor - -## Archivos a modificar/crear - -### 1. `database/migrations/012_server_thresholds.sql` — CREAR - -```sql -ALTER TABLE `servers` - ADD COLUMN `threshold_cpu_warning` DECIMAL(5,2) DEFAULT 80.00 AFTER `agent_last_seen_at`, - ADD COLUMN `threshold_cpu_critical` DECIMAL(5,2) DEFAULT 95.00 AFTER `threshold_cpu_warning`, - ADD COLUMN `threshold_ram_warning` DECIMAL(5,2) DEFAULT 80.00 AFTER `threshold_cpu_critical`, - ADD COLUMN `threshold_ram_critical` DECIMAL(5,2) DEFAULT 95.00 AFTER `threshold_ram_warning`, - ADD COLUMN `threshold_disk_warning` DECIMAL(5,2) DEFAULT 85.00 AFTER `threshold_disk_warning`, - ADD COLUMN `threshold_disk_critical` DECIMAL(5,2) DEFAULT 95.00 AFTER `threshold_disk_critical`; -``` - -### 2. `src/Core/Database.php` — MODIFICAR - -Agregar método `fetchCol()` después de `fetchAll()`: - -```php - public function fetchCol(string $sql, array $params = []): array - { - return $this->query($sql, $params)->fetchAll(\PDO::FETCH_COLUMN); - } -``` - -### 3. `src/Models/Server.php` — MODIFICAR - -Agregar método `getAccessibleUserIds()` antes de `findByAgentKey()`: - -```php - public function getAccessibleUserIds(int $serverId): array - { - $direct = $this->db->fetchCol( - 'SELECT user_id FROM server_user WHERE server_id = ? AND status = \'active\'', - [$serverId] - ); - - $team = $this->db->fetchCol( - 'SELECT DISTINCT tu.user_id FROM team_user tu - JOIN team_server ts ON tu.team_id = ts.team_id - WHERE ts.server_id = ? AND tu.status = \'active\' AND ts.status = \'active\'', - [$serverId] - ); - - $owner = $this->db->fetchCol( - 'SELECT created_by FROM servers WHERE id = ? AND status = \'active\'', - [$serverId] - ); - - return array_unique(array_merge($direct, $team, $owner)); - } -``` - -### 4. `routes/web.php` — MODIFICAR - -Agregar después de línea 67 (`/servers/:id/metrics`): - -```php -$router->post('/servers/:id/thresholds', [ServerController::class, 'saveThresholds'], [AuthMiddleware::class, CSRFMiddleware::class]); -``` - -### 5. `src/Controllers/ServerController.php` — MODIFICAR - -Agregar en `use` imports (si no está ya) + nuevo método después de `metrics()`: - -```php - public function saveThresholds(int $id): void - { - $server = $this->requireServerAccess($id, 'manager'); - $role = $this->serverModel->getUserRole($id, (int) Session::get('user_id')); - - if ($role !== 'owner') { - $this->view->json(['success' => false, 'error' => 'Only the server owner can modify notification thresholds.'], 403); - } - - $fields = ['cpu_warning', 'cpu_critical', 'ram_warning', 'ram_critical', 'disk_warning', 'disk_critical']; - $data = []; - foreach ($fields as $f) { - $key = 'threshold_' . $f; - $data[$key] = isset($_POST[$f]) ? max(0, min(100, (float) $_POST[$f])) : null; - } - - $this->serverModel->update($id, $data); - $this->auditService->log('thresholds_updated', 'server', $id, $data); - $this->view->json(['success' => true, 'message' => 'Notification thresholds updated.']); - } -``` - -### 6. `src/Controllers/ApiController.php` — MODIFICAR - -En `pushMetrics()`, después de `updateAgentLastSeen()` (línea 435) y antes de `$this->view->json(['success' => true])` (línea 437): - -```php - // --- Verificar umbrales de notificación --- - $checks = [ - 'cpu' => ['val' => $cpu, 'warn' => $server['threshold_cpu_warning'], 'crit' => $server['threshold_cpu_critical']], - 'ram' => ['val' => $ram, 'warn' => $server['threshold_ram_warning'], 'crit' => $server['threshold_ram_critical']], - 'disk' => ['val' => $disk, 'warn' => $server['threshold_disk_warning'], 'crit' => $server['threshold_disk_critical']], - ]; - - $breaches = []; - foreach ($checks as $metric => $c) { - if ($c['crit'] !== null && $c['val'] >= (float) $c['crit']) { - $breaches[] = ['metric' => $metric, 'level' => 'critical', 'current' => $c['val'], 'threshold' => $c['crit']]; - } elseif ($c['warn'] !== null && $c['val'] >= (float) $c['warn']) { - $breaches[] = ['metric' => $metric, 'level' => 'warning', 'current' => $c['val'], 'threshold' => $c['warn']]; - } - } - - if (!empty($breaches)) { - $userIds = $serverModel->getAccessibleUserIds((int) $server['id']); - $notificationModel = new \ServerManager\Models\Notification(); - foreach ($breaches as $b) { - $title = strtoupper($b['metric']) . " {$b['level']} on {$server['name']}"; - $message = "{$server['name']}: {$b['metric']} at {$b['current']}% ({$b['level']} threshold: {$b['threshold']}%)"; - foreach ($userIds as $uid) { - $notificationModel->create([ - 'user_id' => (int) $uid, - 'title' => $title, - 'message' => $message, - 'type' => $b['level'], - ]); - } - } - } -``` - -Agregar import de Notification en los `use` del ApiController (si no está): -```php -use ServerManager\Models\Notification; -``` - -### 7. `views/servers/show.php` — MODIFICAR - -**a) Botón en Management Actions** (después de línea 272, antes de ``): - -```php - - - -``` - -**b) Modal** (después del `#agentConfirmModal`, antes de `
`): - -```php -
-``` - -**c) JavaScript** (al final del bloque ``): - -```javascript -function showThresholdModal() { - document.getElementById('thresholdModal').style.display = 'flex'; - document.getElementById('thresholdModal').onclick = function(e) { - if (e.target === this) closeThresholdModal(); - }; -} - -function closeThresholdModal() { - document.getElementById('thresholdModal').style.display = 'none'; -} - -function saveThresholds() { - const form = document.getElementById('thresholdForm'); - const data = new URLSearchParams(new FormData(form)); - data.append('_csrf_token', getCsrfToken()); - - document.querySelector('#thresholdModal .btn-primary').disabled = true; - - fetch('/servers//thresholds', { - method: 'POST', - headers: { - 'Content-Type': 'application/x-www-form-urlencoded', - 'X-CSRF-Token': getCsrfToken(), - }, - body: data.toString(), - }) - .then(r => r.json()) - .then(d => { - showToast(d.success ? 'success' : 'error', d.message ?? d.error); - if (d.success) closeThresholdModal(); - }) - .catch(e => showToast('error', 'Request failed: ' + e.message)) - .finally(() => { - document.querySelector('#thresholdModal .btn-primary').disabled = false; - }); -} -``` - -### 8. Ejecutar migración - -```bash -mysql -u root -p"servermanager2024" servermanager < /var/www/servermanager/database/migrations/012_server_thresholds.sql -``` - -### 9. Commit y push - -```bash -git add -A && git commit -m "feat: per-server notification thresholds with alert on agent metrics push" && git push origin feat/fcm-apk-v1.8.2 -``` - -## Variables de umbral en la vista - -Las columnas nuevas (`threshold_cpu_warning`, etc.) viajan automáticamente en `$server` porque `findById()` hace `SELECT *`. No requiere cambios en `ServerController::show()`.