@goodandready/dsh-goal 0.2.1 → 0.2.3

package/README.md

@@ -225,6 +225,18 @@ npm test
 
 ---
 
+---
+
+## 📦 Release Notes
+
+### v0.2.3
+- **Auto-Updater**: Integrated one-click update API endpoint at `/api/dsh-goal/update` with fail-closed origin checks.
+- **Modular Architecture**: Decomposed server codebase into focused submodules (<600 lines each) for routes, tools, store, reports, and updater.
+- **Strict i18n Separation**: Purged hardcoded internal Russian strings/branches from `lib/*.js`; canonical English and Chinese core with Russian provided via `@goodandready/dsh-russian-lang`.
+- **Reliability & Resilience**: Eliminated all empty catch blocks, added contextual logging on deferred tool completions, and prune dead SSE streams on error.
+- **Design System Fidelity**: Replaced foreign CSS classes and hardcoded RGBA colors with native DSH theme design tokens (`--dsw-alias-*`).
+- **Settings Scope Protection**: Resolved null/loading status handling in the settings card to prevent false `ready` state.
+
 ## 📄 License
 
 MIT © [GooDAnDReaDY](https://github.com/GooDAnDReaDY)

package/README.ru.md

@@ -155,6 +155,18 @@ npm test
 
 ---
 
+---
+
+## 📦 История версий (Release Notes)
+
+### v0.2.3
+- **Автообновление**: Встроен эндпоинт API автообновления в один клик по адресу `/api/dsh-goal/update` с fail-closed проверкой источника.
+- **Модульная архитектура**: Серверная половина плагина полностью декомпозирована на специализированные модули (<600 строк каждый): маршруты, инструменты, хранилище, отчёты и модуль обновления.
+- **Чистое разделение локализации**: Из `lib/*.js` удалены все зашитые русские строки и ветвления; ядро поддерживает только `en` и `zh`, а русская локализация поставляется плагином `@goodandready/dsh-russian-lang`.
+- **Надёжность**: Устранены все пустые блоки `catch`, добавлено контекстное логирование завершения целей и автоматическая очистка отключившихся SSE-клиентов.
+- **Дизайн-система**: Чужеродные CSS-классы и явные цвета RGBA заменены на стандартные токены дизайн-системы DSH (`--dsw-alias-*`).
+- **Защита карточки настроек**: Исправлена обработка снимка настроек при загрузке или отсутствии сервиса, исключающая ложный статус `ready`.
+
 ## 📄 Лицензия
 
 MIT © [GooDAnDReaDY](https://github.com/GooDAnDReaDY)

package/README.zh.md

@@ -219,3 +219,15 @@ npm test
 ## 📄 开源许可
 
 MIT © [GooDAnDReaDY](https://github.com/GooDAnDReaDY)
+
+---
+
+## 📦 版本历史 (Release Notes)
+
+### v0.2.3
+- **自动更新**: 在 `/api/dsh-goal/update` 集成了一键更新 API 端点，并配备安全的原点验证。
+- **模块化架构**: 将服务端完整解耦为高内聚子模块（各模块均低于 600 行）：路由、工具、持久化存储、报告生成与更新器。
+- **严格的多语言规范**: 从 `lib/*.js` 中清除了硬编码的俄语字符串与分支；核心原生支持英文（默认）与中文，俄语由 `@goodandready/dsh-russian-lang` 插件提供。
+- **健壮性与错误处理**: 消除了所有空 catch 块，在目标完成与阻塞的上下文延迟中添加了日志记录，并自动清理断开的 SSE 连接。
+- **设计系统规范**: 将第三方 CSS 类与硬编码 RGBA 颜色替换为原生 DSH 主题设计变量（`--dsw-alias-*`）。
+- **设置卡片状态保护**: 修复了设置作用域未加载时错误显示 `ready` 状态的问题，未就绪时正确显示不可用/加载状态。

package/lib/card-form-state.js

@@ -1,33 +1,40 @@
 /**
- * Чистая логика состояния и валидации формы настроек dsh-goal.
- * Не зависит от DOM/React, полностью тестируема через node --test.
+ * Pure state and validation logic for dsh-goal settings form.
+ * Independent from DOM/React, 100% testable via node --test.
  */
 
 export const DEFAULT_SETTINGS = {
   maxIterations: 25,
   autoDrive: true,
   enableSound: true,
+  showQuickLaunchButton: true,
+  consecutiveToolFailureLimit: 3,
+  enableBrowserNotifications: true,
+  storagePath: '',
 };
 
 /**
- * Парсинг числового значения настроек:
- * - пустая строка / null / undefined -> undefined (сброс к базовому значению)
- * - валидное число >= 1 -> number
- * - иначе -> NaN (invalid)
+ * Parse numeric settings field:
+ * - empty string / null / undefined -> undefined (reset override)
+ * - valid integer >= min -> number
+ * - otherwise -> NaN (invalid)
+ * @param {any} raw
+ * @param {number} [min=1]
+ * @returns {number|undefined}
  */
-export function parseNumberField(raw) {
+export function parseNumberField(raw, min = 1) {
   if (raw === '' || raw === null || raw === undefined) {
     return undefined;
   }
   const n = Number(raw);
-  if (!Number.isFinite(n) || n < 1 || !Number.isInteger(n)) {
+  if (!Number.isFinite(n) || n < min || !Number.isInteger(n)) {
     return NaN;
   }
   return n;
 }
 
 /**
- * Вычисление состояния конкретного поля:
+ * Compute status for a specific setting field
  * @param {string} field
  * @param {any} draftValue
  * @param {Object} snap - { value, base, user }
@@ -46,11 +53,10 @@ export function getFieldStatus(field, draftValue, snap) {
   if (field === 'maxIterations') {
     if (draftValue !== undefined) {
       if (draftValue === '') {
-        // Пустая строка = сброс к базовому значению (не ошибка)
         invalid = false;
-        isDirty = userVal !== undefined; // dirty если было переопределено
+        isDirty = userVal !== undefined;
       } else {
-        const parsed = parseNumberField(draftValue);
+        const parsed = parseNumberField(draftValue, 1);
         if (Number.isNaN(parsed)) {
           invalid = true;
           isDirty = true;
@@ -59,10 +65,30 @@ export function getFieldStatus(field, draftValue, snap) {
         }
       }
     }
+  } else if (field === 'consecutiveToolFailureLimit') {
+    if (draftValue !== undefined) {
+      if (draftValue === '') {
+        invalid = false;
+        isDirty = userVal !== undefined;
+      } else {
+        const parsed = parseNumberField(draftValue, 0);
+        if (Number.isNaN(parsed)) {
+          invalid = true;
+          isDirty = true;
+        } else {
+          isDirty = parsed !== (userVal !== undefined ? userVal : baseVal);
+        }
+      }
+    }
+  } else if (field === 'storagePath') {
+    if (draftValue !== undefined) {
+      const cleanVal = typeof draftValue === 'string' ? draftValue.trim() : '';
+      isDirty = cleanVal !== (userVal !== undefined ? userVal : (baseVal || ''));
+    }
   } else {
-    // boolean fields (autoDrive, enableSound)
+    // boolean fields (autoDrive, enableSound, showQuickLaunchButton, enableBrowserNotifications)
     if (draftValue !== undefined) {
-      isDirty = draftValue !== (userVal !== undefined ? userVal : baseVal);
+      isDirty = Boolean(draftValue) !== (userVal !== undefined ? Boolean(userVal) : Boolean(baseVal));
     }
   }
 
@@ -77,7 +103,7 @@ export function getFieldStatus(field, draftValue, snap) {
 }
 
 /**
- * Вычисление списка операций записи (writes) для scope.set:
+ * Compute list of key-value writes for scope.set
  * @param {Object|null} draft
  * @param {Object} snap - { value, base, user }
  * @returns {Array<[string, any]>}
@@ -86,7 +112,15 @@ export function computeSavePlan(draft, snap) {
   if (!draft) return [];
 
   const writes = [];
-  const fields = ['maxIterations', 'autoDrive', 'enableSound'];
+  const fields = [
+    'maxIterations',
+    'autoDrive',
+    'enableSound',
+    'showQuickLaunchButton',
+    'consecutiveToolFailureLimit',
+    'enableBrowserNotifications',
+    'storagePath',
+  ];
 
   for (const f of fields) {
     if (draft[f] === undefined) continue;
@@ -95,11 +129,21 @@ export function computeSavePlan(draft, snap) {
     if (status.invalid) continue;
 
     if (status.isDirty) {
-      if (f === 'maxIterations' && draft[f] === '') {
-        // Сброс переопределения
-        writes.push([f, undefined]);
-      } else if (f === 'maxIterations') {
-        writes.push([f, parseNumberField(draft[f])]);
+      if (f === 'maxIterations') {
+        if (draft[f] === '') {
+          writes.push([f, undefined]);
+        } else {
+          writes.push([f, parseNumberField(draft[f], 1)]);
+        }
+      } else if (f === 'consecutiveToolFailureLimit') {
+        if (draft[f] === '') {
+          writes.push([f, undefined]);
+        } else {
+          writes.push([f, parseNumberField(draft[f], 0)]);
+        }
+      } else if (f === 'storagePath') {
+        const clean = typeof draft[f] === 'string' ? draft[f].trim() : '';
+        writes.push([f, clean ? clean : undefined]);
       } else {
         writes.push([f, Boolean(draft[f])]);
       }