@goodandready/dsh-goal 0.1.1 → 0.1.2

package/README.md

@@ -26,6 +26,20 @@
 dsh plugin add @goodandready/dsh-goal
 ```
 
+Requires DeepSeek Harness core **0.1.2-rc.1** or newer: the client half declares no external client modules (`dsh.client.inject: []`) because `slots` / `locale` / `settingsScope` are kernel services in rc.1, and plugin settings are registered through a schemastery schema.
+
+## Settings
+
+Settings live in the plugin card under **Settings → Plugins → Plugin settings** ("Goal Mode & Autonomous Loop"). The card reads the live settings snapshot (and renders nothing while the namespace is not `ready`) and saves only changed fields; a failed write keeps your drafts on screen.
+
+| Field | Type | Default | Purpose |
+|---|---|---|---|
+| `maxIterations` | number | `25` | Safety limit: max autonomous iterations per goal |
+| `autoDrive` | boolean | `true` | Keep the autonomous loop running after each turn |
+| `enableSound` | boolean | `true` | Play a sound when a goal completes |
+
+Settings namespace: `dsh-goal`.
+
 ## Quick Start
 
 1. Start a goal in chat:

package/README.ru.md

@@ -30,6 +30,20 @@
 dsh plugin add @goodandready/dsh-goal
 ```
 
+Требуется ядро DeepSeek Harness **0.1.2-rc.1** или новее: клиентская половина не объявляет внешних клиентских модулей (`dsh.client.inject: []`), потому что в rc.1 `slots` / `locale` / `settingsScope` — встроенные службы ядра, а настройки плагина регистрируются schemastery-схемой.
+
+## Настройки
+
+Настройки живут в карточке плагина: **Настройки → Плагины → Настройки плагинов** («Цели и автономный режим (Goal Mode)»). Карточка читает живой снимок пространства настроек (пока пространство не в статусе `ready`, карточка не рисуется) и сохраняет только изменённые поля; неудачная запись не стирает введённые значения.
+
+| Поле | Тип | По умолчанию | Назначение |
+|---|---|---|---|
+| `maxIterations` | число | `25` | Safety-лимит: максимум автономных итераций на цель |
+| `autoDrive` | булево | `true` | Продолжать автономный цикл после каждого хода |
+| `enableSound` | булево | `true` | Звуковой сигнал по завершении цели |
+
+Пространство настроек: `dsh-goal`.
+
 ## Использование
 
 1. Поставьте цель сообщением в чате:

package/docs/design/DESIGN.md

@@ -30,12 +30,14 @@
 - **Футер**: Кнопки ручного добавления шага, паузы и закрытия.
 
 ### 2.3 Settings Card (`settings.plugin.item`)
-- Размещение во вкладке «Настройки → Плагины».
-- Параметры:
+- Размещение во вкладке «Настройки → Плагины → Настройки плагинов», свёрнута по умолчанию, раскрытие по клику по заголовку (`aria-expanded`).
+- Ключ слота (`key`) — пространство настроек `dsh-goal`; пока снимок пространства не в статусе `ready`, карточка не рисуется; при `writable: false` поля и кнопки заблокированы.
+- Параметры (фактический набор, регистрируется schemastery-схемой):
   - `maxIterations`: максимальное число автономных шагов до принудительной остановки (дефолт: 25);
-  - `autoDrive`: автоматически вызывать следующий шаг модели (дефолт: true);
-  - `enableSound`: звуковой сигнал по завершении цели (дефолт: true);
-  - `promptTemplate`: шаблон системного инжекта в контекст модели.
+  - `autoDrive`: автоматически продолжать цикл после каждого хода (дефолт: true);
+  - `enableSound`: звуковой сигнал по завершении цели (дефолт: true).
+- Поля читаются из живого снимка пространства (`ctx.settingsScope.bind({ namespace })`), кнопки «Сбросить» / «Сохранить» в футере; сохраняются только изменённые поля, все записи выполняются подряд (не до первой ошибки), результат сверяется обратным чтением; неудачная запись не стирает введённые значения.
+- `Changed in ядро 0.1.2-rc.1`: поле `promptTemplate` никогда не было реализовано в коде и из схемы удалено из описания; контракт карточки переехал на встроенные службы ядра (`slots` / `locale` / `settingsScope`), `dsh.client.inject` пуст.
 
 ---
 
@@ -70,3 +72,5 @@
 1. **Единый префикс классов**: `.dsh-goal-*` во всех CSS-правилах во избежание конфликтов стилей.
 2. **Точный макет баннера**: скругление углов 8–10px, шапка с отступами `8px 16px`, выравнивание по центру, кнопки действий справа с `gap: 10px`.
 3. **Безопасность импортов**: все `require('@deepseek-ai/dsh-client-ui-primitives')` обёрнуты в `try/catch` с SVG-фоллбеками.
+4. **2026-09-04 — rc.1-совместимость клиента**: `dsh.client.inject: []` — в boot-графе клиента ядра 0.1.2-rc.1 нет package-строк `dsh-client-runtime` (модуля в ядре нет вовсе) и `dsh-client-ui-slots`; `slots`/`locale`/`settingsScope` — встроенные службы ядра. `react`, `@deepseek-ai/dsh-client-ui-slots`, `@deepseek-ai/dsh-client-ui-primitives` отвечаются статической seed-таблицей шелла, но динамических package-строк под них в boot-графе нет — объявлять их в inject нельзя. Условие пересмотра: появление этих package-строк в boot-таблице ядра.
+5. **2026-09-04 — контракт карточки настроек**: key слота = пространство настроек (`dsh-goal`), чтение через `ctx.settingsScope.bind`, решение по статусу снимка (loading/unavailable/ready + writable), сохранение всех изменённых полей с обратной сверкой. Причина: контракт вкладки «Настройки → Плагины» ядра rc.1 (BashCard/AgentLoopCard как эталон). Пересмотр — при изменении контракта слота в ядре.

package/lib/card-form-state.js

@@ -0,0 +1,110 @@
+/**
+ * Чистая логика состояния и валидации формы настроек dsh-goal.
+ * Не зависит от DOM/React, полностью тестируема через node --test.
+ */
+
+export const DEFAULT_SETTINGS = {
+  maxIterations: 25,
+  autoDrive: true,
+  enableSound: true,
+};
+
+/**
+ * Парсинг числового значения настроек:
+ * - пустая строка / null / undefined -> undefined (сброс к базовому значению)
+ * - валидное число >= 1 -> number
+ * - иначе -> NaN (invalid)
+ */
+export function parseNumberField(raw) {
+  if (raw === '' || raw === null || raw === undefined) {
+    return undefined;
+  }
+  const n = Number(raw);
+  if (!Number.isFinite(n) || n < 1 || !Number.isInteger(n)) {
+    return NaN;
+  }
+  return n;
+}
+
+/**
+ * Вычисление состояния конкретного поля:
+ * @param {string} field
+ * @param {any} draftValue
+ * @param {Object} snap - { value, base, user }
+ */
+export function getFieldStatus(field, draftValue, snap) {
+  const baseVal = snap?.base?.[field] ?? DEFAULT_SETTINGS[field];
+  const userVal = snap?.user?.[field];
+  const effectiveVal = snap?.value?.[field] ?? baseVal;
+
+  const isOverridden = userVal !== undefined && userVal !== null;
+  const currentVal = draftValue !== undefined ? draftValue : effectiveVal;
+
+  let invalid = false;
+  let isDirty = false;
+
+  if (field === 'maxIterations') {
+    if (draftValue !== undefined) {
+      if (draftValue === '') {
+        // Пустая строка = сброс к базовому значению (не ошибка)
+        invalid = false;
+        isDirty = userVal !== undefined; // dirty если было переопределено
+      } else {
+        const parsed = parseNumberField(draftValue);
+        if (Number.isNaN(parsed)) {
+          invalid = true;
+          isDirty = true;
+        } else {
+          isDirty = parsed !== (userVal !== undefined ? userVal : baseVal);
+        }
+      }
+    }
+  } else {
+    // boolean fields (autoDrive, enableSound)
+    if (draftValue !== undefined) {
+      isDirty = draftValue !== (userVal !== undefined ? userVal : baseVal);
+    }
+  }
+
+  return {
+    value: currentVal,
+    baseValue: baseVal,
+    userValue: userVal,
+    isOverridden,
+    isDirty,
+    invalid,
+  };
+}
+
+/**
+ * Вычисление списка операций записи (writes) для scope.set:
+ * @param {Object|null} draft
+ * @param {Object} snap - { value, base, user }
+ * @returns {Array<[string, any]>}
+ */
+export function computeSavePlan(draft, snap) {
+  if (!draft) return [];
+
+  const writes = [];
+  const fields = ['maxIterations', 'autoDrive', 'enableSound'];
+
+  for (const f of fields) {
+    if (draft[f] === undefined) continue;
+
+    const status = getFieldStatus(f, draft[f], 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])]);
+      } else {
+        writes.push([f, Boolean(draft[f])]);
+      }
+    }
+  }
+
+  return writes;
+}