@goodandready/dsh-goal 0.1.6 → 0.1.7

package/docs/design/DESIGN.md

@@ -90,3 +90,9 @@
    - **Crash Hydration**: При перезапуске DSH цели, оставшиеся в статусе `RUNNING`, автоматически гидрируются в `PAUSED` с отметкой времени и пояснением оператору для ручного возобновления в 1 клик.
    - **Web Audio Синтез**: Синтезированные пентатонические колокольчики через `window.AudioContext` при завершении цели (`COMPLETED` — мажорный аккорд, `FAILED` — минорный аккорд) с соблюдением настройки `enableSound`.
    - **Milestone Integrity**: Статус выполнения вех строго контролируется агентом; чекбоксы ручной отметки в UI исключены в соответствии с архитектурным контрактом.
+
+8. **2026-09-12 — Сессионная изоляция /goal, устойчивость SSE и строгая REST валидация (v0.1.7)**:
+   - **Сквозная сессионная изоляция в слеш-команде**: В `command-handler.js` метод `executeGoalSlashCommand` пробрасывает `sessionId` во все вызовы `engine` (`getSnapshot`, `startGoal`, `pause`, `resume`, `clear`), исключая мутацию глобальной сессии при работе в конкретном чате.
+   - **Устойчивость SSE с Exponential Backoff**: На клиенте внедрена автоматическая схема повторного подключения `EventSource` с возрастающей задержкой (от 2s до 30s) и рандомизированным джиттером для защиты от шторма переподключений.
+   - **Строгая REST валидация и Enum Guard**: Обработчик `POST /dsh-goal/action` валидирует непустой `title` при старте цели и проверяет допустимость статуса вехи по `MilestoneStatus` enum, возвращая внятный HTTP 400 Bad Request / 404 Not Found.
+   - **Безопасность файловой системы**: `GoalEngine.writeStateToDiskSync()` гарантированно создаёт отсутствующие родительские директории через `fs.mkdirSync(dir, { recursive: true })` перед созданием временного файла и атомарным переименованием.

package/lib/client.js

@@ -869,8 +869,10 @@ window.__ModuleLoader__.load({
       useEffect(() => {
         let es = null;
         let timer = null;
+        let reconnectTimer = null;
         let isDisposed = false;
         let sseActive = false;
+        let retryCount = 0;
 
         const scheduleNextPoll = () => {
           if (isDisposed || sseActive) return;
@@ -891,18 +893,33 @@ window.__ModuleLoader__.load({
           }, delay);
         };
 
-        // Запуск SSE соединения
-        if (typeof window !== 'undefined' && typeof window.EventSource !== 'undefined') {
+        const connectSSE = () => {
+          if (isDisposed) return;
+          if (typeof window === 'undefined' || typeof window.EventSource === 'undefined') {
+            scheduleNextPoll();
+            return;
+          }
+
           try {
+            if (es) {
+              es.close();
+              es = null;
+            }
+
             const query = sid && sid !== 'default' ? `?sessionId=${encodeURIComponent(sid)}` : '';
             es = new window.EventSource(`/dsh-goal/events${query}`);
 
             es.onopen = () => {
               sseActive = true;
+              retryCount = 0;
               if (timer) {
                 clearTimeout(timer);
                 timer = null;
               }
+              if (reconnectTimer) {
+                clearTimeout(reconnectTimer);
+                reconnectTimer = null;
+              }
             };
 
             es.onmessage = (event) => {
@@ -916,22 +933,38 @@ window.__ModuleLoader__.load({
             };
 
             es.onerror = () => {
-              // При сбое соединения SSE переключаемся на адаптивный fallback поллинг
               sseActive = false;
-              if (!isDisposed && !timer) {
-                scheduleNextPoll();
+              if (es) {
+                es.close();
+                es = null;
+              }
+              if (!isDisposed) {
+                // Запускаем немедленный опрос для непрерывности работы UI
+                if (!timer) scheduleNextPoll();
+
+                // Экспоненциальный backoff для повторного подключения SSE (от 2s до 30s) с джиттером
+                retryCount = Math.min(retryCount + 1, 5);
+                const backoffBase = Math.min(30000, 2000 * Math.pow(1.8, retryCount - 1));
+                const jitter = Math.random() * 1000;
+                const reconnectDelay = Math.round(backoffBase + jitter);
+
+                if (reconnectTimer) clearTimeout(reconnectTimer);
+                reconnectTimer = setTimeout(() => {
+                  reconnectTimer = null;
+                  connectSSE();
+                }, reconnectDelay);
               }
             };
           } catch (_) {
             sseActive = false;
             scheduleNextPoll();
           }
-        } else {
-          scheduleNextPoll();
-        }
+        };
+
+        connectSSE();
 
         // Первоначальная загрузка состояния
-        fetchState().then((data) => {
+        fetchState().then(() => {
           if (!sseActive) scheduleNextPoll();
         });
 
@@ -942,6 +975,10 @@ window.__ModuleLoader__.load({
               fetchState().then(() => {
                 scheduleNextPoll();
               });
+              // При возвращении на вкладку можно попытаться сразу переподключить SSE
+              if (!reconnectTimer && !sseActive) {
+                connectSSE();
+              }
             }
           }
         };
@@ -957,6 +994,7 @@ window.__ModuleLoader__.load({
             es = null;
           }
           if (timer) clearTimeout(timer);
+          if (reconnectTimer) clearTimeout(reconnectTimer);
           if (typeof document !== 'undefined' && document.removeEventListener) {
             document.removeEventListener('visibilitychange', onVisibilityChange);
           }

package/lib/command-handler.js

@@ -1,192 +1,193 @@
-import { randomUUID } from 'node:crypto';
-import { GoalState } from './goal-engine.js';
-
-export const USAGE = 'Использование: /goal [<цель>|clear|pause|resume]';
-
-/**
- * Парсинг строки ввода для команды /goal
- * @param {string} rawInput
- * @returns {{ action: string, text?: string }}
- */
-export function parseGoalInput(rawInput = '') {
-  const input = String(rawInput || '').trim();
-  if (!input) {
-    return { action: 'show' };
-  }
-
-  const lower = input.toLowerCase();
-  if (lower === 'clear') return { action: 'clear' };
-  if (lower === 'pause') return { action: 'pause' };
-  if (lower === 'resume') return { action: 'resume' };
-
-  return { action: 'start', text: input };
-}
-
-/**
- * Создание валидного identified сообщения пользователя для отправки агенту через followup
- * @param {string} text
- * @returns {Object}
- */
-export function createGoalUserMessage(text) {
-  return {
-    id: randomUUID(),
-    role: 'user',
-    content: [{ type: 'text', text }],
-    source: { kind: 'user' },
-  };
-}
-
-/**
- * Формирование стартового промпта с жестким требованием составить план работ
- * @param {string} goalText
- * @returns {string}
- */
-export function formatGoalStartPrompt(goalText) {
-  return `🎯 Активирован режим цели (Goal Mode): "${goalText}"\n\n` +
-    `СТРОГИЙ КОНТРАКТ АВТОНОМНОГО РЕЖИМА:\n` +
-    `1. ОБЯЗАТЕЛЬНЫЙ ШАГ №1: Твоим ПЕРВЫМ действием ДО выполнения любой другой работы или ответа должен быть вызов инструмента goal_set_milestones с массивом из 3-7 конкретных, последовательных пунктов плана работ. Пользователь видит этот план в интерфейсе в реальном времени.\n` +
-    `2. ШАГ №2: После установки плана переходи к выполнению шагов. Перед началом работы над каждым шагом отметь его статус как in_progress через goal_update_progress(milestone_id, "in_progress").\n` +
-    `3. ШАГ №3: После успешного завершения шага отметь его статус как completed через goal_update_progress(milestone_id, "completed", "краткие итоги шага").\n` +
-    `4. ШАГ №4: Когда все пункты плана выполнены, вызови инструмент goal_finish(summary) с подробным резюме достигнутых результатов.`;
-}
-
-/**
- * Форматирование текстового ответа для UI и постановка/управление задачей агента
- * @param {import('./goal-engine.js').GoalEngine} engine
- * @param {{ action: string, text?: string }} parsed
- * @param {Object} [config]
- * @param {Object} [agent]
- * @returns {{ kind: 'success' | 'error', text: string }}
- */
-export function executeGoalSlashCommand(engine, parsed, config = {}, agent = null, sessionId = 'default') {
-  const sid = sessionId || 'default';
-  const snap = engine.getSnapshot(sid);
-
-  switch (parsed.action) {
-    case 'show': {
-      if (!snap.hasActiveGoal) {
-        return {
-          kind: 'success',
-          text: `🎯 Режим цели (Goal Mode): цель не установлена.\n${USAGE}`,
-        };
-      }
-
-      const milestonesInfo = snap.milestones.length > 0
-        ? `\nПлан работ:\n` + snap.milestones.map((m, i) => `  ${i + 1}. [${m.status}] ${m.title}`).join('\n')
-        : '';
-
-      return {
-        kind: 'success',
-        text: `🎯 Цель: «${snap.title}»\n` +
-          `Статус: ${snap.state}\n` +
-          `Время: ${snap.formattedElapsed}\n` +
-          `Итераций: ${snap.iterationsCount}/${snap.maxIterations}` +
-          milestonesInfo +
-          `\n\nКоманды: /goal pause, /goal resume, /goal clear, /goal <новая цель>`,
-      };
-    }
-
-    case 'clear': {
-      if (!snap.hasActiveGoal) {
-        return {
-          kind: 'success',
-          text: 'Цель не была установлена.',
-        };
-      }
-      engine.clear(sid);
-      if (agent && typeof agent.cancel === 'function') {
-        try {
-          agent.cancel({ kind: 'user', reason: 'Goal cleared by user' });
-        } catch (err) {
-          console.warn('[dsh-goal] Failed to cancel agent on clear:', err);
-        }
-      }
-      return {
-        kind: 'success',
-        text: '🎯 Цель сброшена.',
-      };
-    }
-
-    case 'pause': {
-      if (!snap.hasActiveGoal) {
-        return {
-          kind: 'error',
-          text: `Нельзя приостановить: активная цель отсутствует.\n${USAGE}`,
-        };
-      }
-      if (snap.state === GoalState.PAUSED) {
-        return {
-          kind: 'success',
-          text: '⏸ Цель уже на паузе.',
-        };
-      }
-      engine.pause('Пауза по команде пользователя (/goal pause)', sid);
-      if (agent && typeof agent.cancel === 'function') {
-        try {
-          agent.cancel({ kind: 'user', reason: 'Goal paused by user' });
-        } catch (err) {
-          console.warn('[dsh-goal] Failed to cancel agent on pause:', err);
-        }
-      }
-      return {
-        kind: 'success',
-        text: `⏸ Цель приостановлена: «${snap.title}». Работа агента остановлена. Используйте /goal resume или кнопку ▶️ для продолжения.`,
-      };
-    }
-
-    case 'resume': {
-      if (!snap.hasActiveGoal) {
-        return {
-          kind: 'error',
-          text: `Нельзя возобновить: активная цель отсутствует.\n${USAGE}`,
-        };
-      }
-      if (snap.state === GoalState.RUNNING) {
-        return {
-          kind: 'success',
-          text: '▶️ Цель уже выполняется.',
-        };
-      }
-      engine.resume(sid);
-      if (agent && typeof agent.followup === 'function') {
-        try {
-          const resumePrompt = `▶️ Цель возобновлена пользователем. Продолжай выполнение плана работ с того места, где остановился. Отмечай шаги через goal_update_progress.`;
-          agent.followup(createGoalUserMessage(resumePrompt));
-        } catch (err) {
-          console.warn('[dsh-goal] Failed to followup agent on resume:', err);
-        }
-      }
-      return {
-        kind: 'success',
-        text: `▶️ Цель возобновлена: «${snap.title}». Агент продолжает работу.`,
-      };
-    }
-
-    case 'start': {
-      const maxIterations = config.maxIterations ?? snap.maxIterations ?? 25;
-      const newSnap = engine.startGoal(parsed.text, { maxIterations }, sid);
-
-      // Если команда вызвана в контексте агента, отправляем цель в очередь модели
-      if (agent && typeof agent.followup === 'function') {
-        try {
-          const userMsg = createGoalUserMessage(formatGoalStartPrompt(parsed.text));
-          agent.followup(userMsg);
-        } catch (err) {
-          console.warn('[dsh-goal] Failed to dispatch goal to agent followup:', err);
-        }
-      }
-
-      return {
-        kind: 'success',
-        text: `🎯 Активирована цель: «${newSnap.title}» (макс. ${newSnap.maxIterations} итераций).\n` +
-          `Для паузы: /goal pause | Для сброса: /goal clear`,
-      };
-    }
-
-    default:
-      return {
-        kind: 'error',
-        text: `Неизвестная команда. ${USAGE}`,
-      };
-  }
-}
+import { randomUUID } from 'node:crypto';
+import { GoalState } from './goal-engine.js';
+
+export const USAGE = 'Использование: /goal [<цель>|clear|pause|resume]';
+
+/**
+ * Парсинг строки ввода для команды /goal
+ * @param {string} rawInput
+ * @returns {{ action: string, text?: string }}
+ */
+export function parseGoalInput(rawInput = '') {
+  const input = String(rawInput || '').trim();
+  if (!input) {
+    return { action: 'show' };
+  }
+
+  const lower = input.toLowerCase();
+  if (lower === 'clear') return { action: 'clear' };
+  if (lower === 'pause') return { action: 'pause' };
+  if (lower === 'resume') return { action: 'resume' };
+
+  return { action: 'start', text: input };
+}
+
+/**
+ * Создание валидного identified сообщения пользователя для отправки агенту через followup
+ * @param {string} text
+ * @returns {Object}
+ */
+export function createGoalUserMessage(text) {
+  return {
+    id: randomUUID(),
+    role: 'user',
+    content: [{ type: 'text', text }],
+    source: { kind: 'user' },
+  };
+}
+
+/**
+ * Формирование стартового промпта с жестким требованием составить план работ
+ * @param {string} goalText
+ * @returns {string}
+ */
+export function formatGoalStartPrompt(goalText) {
+  return `🎯 Активирован режим цели (Goal Mode): "${goalText}"\n\n` +
+    `СТРОГИЙ КОНТРАКТ АВТОНОМНОГО РЕЖИМА:\n` +
+    `1. ОБЯЗАТЕЛЬНЫЙ ШАГ №1: Твоим ПЕРВЫМ действием ДО выполнения любой другой работы или ответа должен быть вызов инструмента goal_set_milestones с массивом из 3-7 конкретных, последовательных пунктов плана работ. Пользователь видит этот план в интерфейсе в реальном времени.\n` +
+    `2. ШАГ №2: После установки плана переходи к выполнению шагов. Перед началом работы над каждым шагом отметь его статус как in_progress через goal_update_progress(milestone_id, "in_progress").\n` +
+    `3. ШАГ №3: После успешного завершения шага отметь его статус как completed через goal_update_progress(milestone_id, "completed", "краткие итоги шага").\n` +
+    `4. ШАГ №4: Когда все пункты плана выполнены, вызови инструмент goal_finish(summary) с подробным резюме достигнутых результатов.`;
+}
+
+/**
+ * Форматирование текстового ответа для UI и постановка/управление задачей агента
+ * @param {import('./goal-engine.js').GoalEngine} engine
+ * @param {{ action: string, text?: string }} parsed
+ * @param {Object} [config]
+ * @param {Object} [agent]
+ * @param {string} [sessionId='default']
+ * @returns {{ kind: 'success' | 'error', text: string }}
+ */
+export function executeGoalSlashCommand(engine, parsed, config = {}, agent = null, sessionId = 'default') {
+  const sid = sessionId || 'default';
+  const snap = engine.getSnapshot(sid);
+
+  switch (parsed.action) {
+    case 'show': {
+      if (!snap.hasActiveGoal) {
+        return {
+          kind: 'success',
+          text: `🎯 Режим цели (Goal Mode): цель не установлена.\n${USAGE}`,
+        };
+      }
+
+      const milestonesInfo = snap.milestones.length > 0
+        ? `\nПлан работ:\n` + snap.milestones.map((m, i) => `  ${i + 1}. [${m.status}] ${m.title}`).join('\n')
+        : '';
+
+      return {
+        kind: 'success',
+        text: `🎯 Цель: «${snap.title}»\n` +
+          `Статус: ${snap.state}\n` +
+          `Время: ${snap.formattedElapsed}\n` +
+          `Итераций: ${snap.iterationsCount}/${snap.maxIterations}` +
+          milestonesInfo +
+          `\n\nКоманды: /goal pause, /goal resume, /goal clear, /goal <новая цель>`,
+      };
+    }
+
+    case 'clear': {
+      if (!snap.hasActiveGoal) {
+        return {
+          kind: 'success',
+          text: 'Цель не была установлена.',
+        };
+      }
+      engine.clear(sid);
+      if (agent && typeof agent.cancel === 'function') {
+        try {
+          agent.cancel({ kind: 'user', reason: 'Goal cleared by user' });
+        } catch (err) {
+          console.warn('[dsh-goal] Failed to cancel agent on clear:', err);
+        }
+      }
+      return {
+        kind: 'success',
+        text: '🎯 Цель сброшена.',
+      };
+    }
+
+    case 'pause': {
+      if (!snap.hasActiveGoal) {
+        return {
+          kind: 'error',
+          text: `Нельзя приостановить: активная цель отсутствует.\n${USAGE}`,
+        };
+      }
+      if (snap.state === GoalState.PAUSED) {
+        return {
+          kind: 'success',
+          text: '⏸ Цель уже на паузе.',
+        };
+      }
+      engine.pause('Пауза по команде пользователя (/goal pause)', sid);
+      if (agent && typeof agent.cancel === 'function') {
+        try {
+          agent.cancel({ kind: 'user', reason: 'Goal paused by user' });
+        } catch (err) {
+          console.warn('[dsh-goal] Failed to cancel agent on pause:', err);
+        }
+      }
+      return {
+        kind: 'success',
+        text: `⏸ Цель приостановлена: «${snap.title}». Работа агента остановлена. Используйте /goal resume или кнопку ▶️ для продолжения.`,
+      };
+    }
+
+    case 'resume': {
+      if (!snap.hasActiveGoal) {
+        return {
+          kind: 'error',
+          text: `Нельзя возобновить: активная цель отсутствует.\n${USAGE}`,
+        };
+      }
+      if (snap.state === GoalState.RUNNING) {
+        return {
+          kind: 'success',
+          text: '▶️ Цель уже выполняется.',
+        };
+      }
+      engine.resume(sid);
+      if (agent && typeof agent.followup === 'function') {
+        try {
+          const resumePrompt = `▶️ Цель возобновлена пользователем. Продолжай выполнение плана работ с того места, где остановился. Отмечай шаги через goal_update_progress.`;
+          agent.followup(createGoalUserMessage(resumePrompt));
+        } catch (err) {
+          console.warn('[dsh-goal] Failed to followup agent on resume:', err);
+        }
+      }
+      return {
+        kind: 'success',
+        text: `▶️ Цель возобновлена: «${snap.title}». Агент продолжает работу.`,
+      };
+    }
+
+    case 'start': {
+      const maxIterations = config.maxIterations ?? snap.maxIterations ?? 25;
+      const newSnap = engine.startGoal(parsed.text, { maxIterations }, sid);
+
+      // Если команда вызвана в контексте агента, отправляем цель в очередь модели
+      if (agent && typeof agent.followup === 'function') {
+        try {
+          const userMsg = createGoalUserMessage(formatGoalStartPrompt(parsed.text));
+          agent.followup(userMsg);
+        } catch (err) {
+          console.warn('[dsh-goal] Failed to dispatch goal to agent followup:', err);
+        }
+      }
+
+      return {
+        kind: 'success',
+        text: `🎯 Активирована цель: «${newSnap.title}» (макс. ${newSnap.maxIterations} итераций).\n` +
+          `Для паузы: /goal pause | Для сброса: /goal clear`,
+      };
+    }
+
+    default:
+      return {
+        kind: 'error',
+        text: `Неизвестная команда. ${USAGE}`,
+      };
+  }
+}

package/lib/goal-engine.js

@@ -163,6 +163,10 @@ export class GoalEngine {
         sessions: sessionsObj,
         ...(this.goals.has('default') ? this.goals.get('default') : {}),
       };
+      const dir = path.dirname(this.storagePath);
+      if (!fs.existsSync(dir)) {
+        fs.mkdirSync(dir, { recursive: true });
+      }
       const tmp = `${this.storagePath}.tmp.${Date.now()}`;
       fs.writeFileSync(tmp, JSON.stringify(payload, null, 2), 'utf8');
       fs.renameSync(tmp, this.storagePath);

package/lib/index.js

@@ -526,12 +526,18 @@ export function apply(ctx, config = {}) {
 
               let result = null;
               switch (action) {
-                case 'start':
-                  result = engine.startGoal(title || 'Новая цель', {
-                    description,
+                case 'start': {
+                  const cleanTitle = typeof title === 'string' ? title.trim() : '';
+                  if (!cleanTitle) {
+                    res.statusCode = 400;
+                    return res.end(JSON.stringify({ error: 'Goal title cannot be empty' }));
+                  }
+                  result = engine.startGoal(cleanTitle, {
+                    description: typeof description === 'string' ? description.trim() : '',
                     maxIterations: getConfig().maxIterations,
                   }, sid);
                   break;
+                }
                 case 'pause':
                   result = engine.pause(reason || 'Пауза по кнопке интерфейса', sid);
                   stopRunningAgents(sid);
@@ -549,14 +555,24 @@ export function apply(ctx, config = {}) {
                   stopRunningAgents(sid);
                   sessionAgents.delete(sid);
                   break;
-                case 'update_milestone':
+                case 'update_milestone': {
                   if (!milestoneId || !status) {
                     res.statusCode = 400;
                     return res.end(JSON.stringify({ error: 'milestoneId and status are required' }));
                   }
-                  engine.updateMilestone(milestoneId, status, notes, sid);
+                  const validStatuses = Object.values(MilestoneStatus);
+                  if (!validStatuses.includes(status)) {
+                    res.statusCode = 400;
+                    return res.end(JSON.stringify({ error: `Invalid status: ${status}. Must be one of: ${validStatuses.join(', ')}` }));
+                  }
+                  const ok = engine.updateMilestone(milestoneId, status, notes, sid);
+                  if (!ok) {
+                    res.statusCode = 404;
+                    return res.end(JSON.stringify({ error: `Milestone with id "${milestoneId}" not found` }));
+                  }
                   result = engine.getSnapshot(sid);
                   break;
+                }
                 default:
                   res.statusCode = 400;
                   return res.end(JSON.stringify({ error: `Unknown action: ${action}` }));

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@goodandready/dsh-goal",
-  "version": "0.1.6",
+  "version": "0.1.7",
   "description": "Goal mode & autonomous execution plugin for DeepSeek Harness with sticky top banner",
   "type": "module",
   "main": "./lib/index.js",