@goodandready/dsh-cron 0.2.0 → 0.2.1

package/docs/design/DESIGN.md

@@ -16,7 +16,7 @@
   - Поисковая строка; сводная статистика (активные задачи, запуски, токены, стоимость).
   - Карточки задач: статус-переключатель, название, расписание (человекочитаемое + raw cron), действия (запуск, редактирование, удаление).
   - Блок «Рекомендуемые задачи»: готовые шаблоны (Daily digest, Weekly review, Follow-up monitor) в один клик.
-  - Карточка настроек в слоте settings.plugin.item, key = namespace `dsh-cron`; запасной путь — settings.section.
+  - Карточка настроек в слоте settings.plugin.item, key = namespace `dsh-cron`; отдельный раздел настроек не используется (#102).
 - LLM Tools: cron_create_task (+ alias cron_schedule_task), cron_list_tasks, cron_pause_task, cron_resume_task, cron_delete_task, cron_run_task.
 - API: HTTP эндпоинты /dsh-cron/* (tasks, models, chat/start, settings, telegram/test, kanban/test, tasks/:id/actions, legacy action/:id/:action). Мутирующие эндпоинты отклоняют cross-origin запросы; script-задачи по HTTP требуют заголовок x-dsh-cron-confirm.
 - Chat / Slash Commands: отсутствуют (ранее заявленные /cron-команды не были реализованы и удалены из документации; решение 2026-09-09).
@@ -46,7 +46,7 @@
   - SettingsModal: настройки Telegram/Kanban с тестами доставки.
   - TaskItem: строка задачи с переключателем состояния и действиями.
   - RecommendationCard: плашка с готовым шаблоном.
-  - CronSettingsCard: карточка параметров плагина в настройках.
+  - CronSettingsCard: карточка параметров плагина в настройках; свёрнута по умолчанию, шапка-кнопка с шевроном разворачивает тело (aria-expanded), кнопка «Открыть панель задач» — внутри раскрытого тела (решение 2026-09-10, #100).
 - States:
   - Loading: индикатор загрузки списка/истории.
   - Empty: дружелюбный пустой экран со списком рекомендаций.
@@ -63,7 +63,8 @@
 ## Locked Design Decisions
 - 2026-09-09 — Пакет надёжности ядра (v0.1.24): буфер shell-задач 10МБ, атомарное сохранение с PID, аудит пропущенных запусков при рестарте (missed), фоновый поллинг UI (8с).
 - 2026-09-03 — Публичный скоуп @goodandready/dsh-cron; оверлей через mountSidebarEntry/mountScreen аналогично dsh-kanban; двойная кнопка «Создать ⌄».
-- 2026-09-09 — Слот карточки настроек: settings.plugin.item с key/namespace `dsh-cron` (совпадение с серверной регистрацией); settings.section — только запасной путь. Причина: контракт слота настроек (#85).
+- 2026-09-09 — Слот карточки настроек: settings.plugin.item с key/namespace `dsh-cron` (совпадение с серверной регистрацией). Причина: контракт слота настроек (#85). Changed 2026-09-10 (#102): settings.section fallback удалён — карточка только во вкладке плагинов.
+- 2026-09-10 — Значения настроек пишутся через зарегистрированный settings-скоуп (`scope.set` → `watch` → store); REST /dsh-cron/settings остаётся транспортом для карточки и headless-сценариев; карточка проверяет статус снапшота и не рисует поля до его получения (#102).
 - 2026-09-09 — Английский — канонический язык строк; словарь STRINGS.en регистрируется в ctx.locale; русский — через translation-плагин. Причина: стандарт DSH-плагинов (#87).
 - 2026-09-09 — Same-origin проверка мутирующих эндпоинтов, лимит тела 1 МБ, confirm-заголовок для script-задач. Причина: закрытие CSRF→RCE поверхности (#86).
 - 2026-09-09 — Стилевая изоляция: динамические <style> с data-dsh-plugin="dsh-cron"; цвета только через токены темы + единый блок plugin-переменных. Причина: защита от очистки стилей соседями и поддержка светлой темы (#91).

package/lib/client.js

@@ -119,7 +119,9 @@ window.__ModuleLoader__.load({
         'form.notifyOnlyFailure': 'Only on failures (onlyOnFailure / silent on success)',
         'form.nextRunLabel': 'Next run: ',
         'form.notScheduled': 'not scheduled',
-        'settings.sectionLabel': 'Cron tasks',
+        'settings.loading': 'Loading settings...',
+        'settings.unavailable': 'Settings are unavailable right now.',
+        'settings.retry': 'Retry',
         'settings.cardTitle': '⏰ Task scheduler (Cron)',
         'settings.cardDesc': 'Scheduled task execution (cron/intervals), background agent scenarios and Telegram/Kanban integration.',
         'settings.openPanel': 'Open task panel',
@@ -170,6 +172,14 @@ window.__ModuleLoader__.load({
     };
     const T = (key, vars) => translate(key, vars);
 
+    // Core chevron primitive when the build ships it; own SVG fallback otherwise.
+    let CoreChevron = null;
+    try {
+      const primitives = require('@deepseek-ai/dsh-client-ui-primitives');
+      CoreChevron = primitives && primitives.IconChevronDownOutline14;
+    } catch (e) {}
+    const chevronNode = () => (CoreChevron ? React.createElement(CoreChevron) : React.createElement('span', { style: { display: 'flex', alignItems: 'center' }, dangerouslySetInnerHTML: { __html: ICON_CHEVRON_DOWN } }));
+
     // Status colors: core semantic tokens first, plugin fallbacks as a single
     // point of change (Issue #91 — no scattered hardcoded hex).
     const STYLES = `
@@ -271,9 +281,14 @@ window.__ModuleLoader__.load({
       .dsh-cron-send-btn:hover:not(:disabled) { transform: translateY(-1px); }
       .dsh-cron-send-btn:disabled { opacity: 0.4; cursor: not-allowed; }
 
-      .dsh-cron-card { background: var(--dsw-alias-bg-layer-2, #1f1f1f); border: 1px solid var(--dsw-alias-border-l1, #333); border-radius: 12px; padding: 18px 20px; margin-bottom: 16px; }
-      .dsh-cron-card-head { display: flex; justify-content: space-between; align-items: center; margin-bottom: 12px; }
+      .dsh-cron-card { background: var(--dsw-alias-bg-layer-2, #1f1f1f); border: 1px solid var(--dsw-alias-border-l1, #333); border-radius: 12px; padding: 14px 20px; margin-bottom: 16px; }
+      .dsh-cron-card-head-btn { appearance: none; width: 100%; font: inherit; color: inherit; text-align: left; cursor: pointer; background: 0 0; border: 0; border-radius: 12px; display: flex; align-items: center; gap: 12px; padding: 4px 0; }
+      .dsh-cron-card-head-btn .dsh-cron-card-title { flex: 1; }
       .dsh-cron-card-title { font-size: 15px; font-weight: 600; color: var(--dsw-alias-label-primary, #fff); }
+      .dsh-cron-chev { margin-left: auto; flex: none; color: var(--dsw-alias-label-tertiary, #888); transition: transform 0.16s; display: flex; align-items: center; }
+      .dsh-cron-chev-open { transform: rotate(180deg); }
+      .dsh-cron-card-body { border-top: 1px solid var(--dsw-alias-border-l1, #333); margin-top: 12px; padding-top: 14px; }
+      .dsh-cron-card-head { display: flex; justify-content: space-between; align-items: center; margin-bottom: 12px; }
       .dsh-cron-card-desc { font-size: 13px; color: var(--dsw-alias-label-secondary, #888); margin-bottom: 16px; line-height: 1.4; }
 
       .dsh-cron-modal-nav { display: flex; gap: 4px; border-bottom: 1px solid var(--dsw-alias-border-l2, #2e2e2e); margin-bottom: 18px; padding-bottom: 2px; }
@@ -1355,24 +1370,36 @@ window.__ModuleLoader__.load({
       const [hasDefault, setHasDefault] = React.useState(false);
       const [saved, setSaved] = React.useState(false);
       const [testInfo, setTestInfo] = React.useState(null);
+      // Collapsed by default, head is the toggle (canonical card contract, #100)
+      const [cardOpen, setCardOpen] = React.useState(false);
+
+      // Settings snapshot status (#102): never render phantom inputs before
+      // the snapshot arrives; offer a retry when it is unavailable.
+      const [loadState, setLoadState] = React.useState('loading');
+      const [reloadTick, setReloadTick] = React.useState(0);
 
       React.useEffect(() => {
         let active = true;
+        setLoadState('loading');
         fetch('/dsh-cron/settings')
           .then(r => r.json())
           .then(data => {
-            if (active && data && data.ok && data.settings) {
+            if (!active) return;
+            if (data && data.ok && data.settings) {
               setToken(data.settings.botToken || '');
               setChatId(data.settings.chatId || '');
               setNotify(Boolean(data.settings.notifyTelegram));
               setOnlyFailure(Boolean(data.settings.onlyOnFailure));
               setKanbanUrl(data.settings.kanbanBaseUrl || 'http://127.0.0.1:3000');
               setHasDefault(Boolean(data.settings.hasDefaultCredentials));
+              setLoadState('ready');
+            } else {
+              setLoadState('error');
             }
           })
-          .catch(() => {});
+          .catch(() => { if (active) setLoadState('error'); });
         return () => { active = false; };
-      }, []);
+      }, [reloadTick]);
 
       const saveSettings = async (e) => {
         if (e) e.preventDefault();
@@ -1434,19 +1461,44 @@ window.__ModuleLoader__.load({
         }
       };
 
+      if (loadState !== 'ready') {
+        return React.createElement('div', { className: 'dsh-cron-card' },
+          React.createElement('div', { className: 'dsh-cron-card-title' }, T_KEY(t, 'settings.cardTitle')),
+          React.createElement('div', {
+            className: 'dsh-cron-status-banner',
+            'data-type': loadState === 'error' ? 'error' : 'info',
+            style: { marginTop: '10px', display: 'flex', justifyContent: 'space-between', alignItems: 'center', gap: '10px' }
+          },
+            React.createElement('span', null, loadState === 'error' ? T_KEY(t, 'settings.unavailable') : T_KEY(t, 'settings.loading')),
+            loadState === 'error' && React.createElement('button', {
+              type: 'button',
+              className: 'dsh-cron-btn-secondary',
+              onClick: () => setReloadTick(x => x + 1)
+            }, T_KEY(t, 'settings.retry'))
+          )
+        );
+      }
+
       return React.createElement('div', { className: 'dsh-cron-card' },
-        React.createElement('div', { className: 'dsh-cron-card-head' },
+        React.createElement('button', {
+          type: 'button',
+          className: 'dsh-cron-card-head-btn',
+          'aria-expanded': cardOpen ? 'true' : 'false',
+          onClick: () => setCardOpen(!cardOpen)
+        },
           React.createElement('span', { className: 'dsh-cron-card-title' }, T_KEY(t, 'settings.cardTitle')),
-          React.createElement('button', {
-            type: 'button',
-            className: 'dsh-cron-btn-primary',
-            onClick: () => { if (toggle) toggle.set(true); }
-          }, T_KEY(t, 'settings.openPanel'))
-        ),
-        React.createElement('div', { className: 'dsh-cron-card-desc', style: { marginBottom: '16px' } },
-          T_KEY(t, 'settings.cardDesc')
+          React.createElement('span', { className: 'dsh-cron-chev' + (cardOpen ? ' dsh-cron-chev-open' : '') }, chevronNode())
         ),
-        React.createElement('form', { onSubmit: saveSettings, style: { display: 'flex', flexDirection: 'column', gap: '12px' } },
+        cardOpen && React.createElement('div', { className: 'dsh-cron-card-body' },
+          React.createElement('div', { className: 'dsh-cron-card-desc' }, T_KEY(t, 'settings.cardDesc')),
+          React.createElement('div', { style: { display: 'flex', justifyContent: 'flex-end', marginBottom: '12px' } },
+            React.createElement('button', {
+              type: 'button',
+              className: 'dsh-cron-btn-primary',
+              onClick: () => { if (toggle) toggle.set(true); }
+            }, T_KEY(t, 'settings.openPanel'))
+          ),
+          React.createElement('form', { onSubmit: saveSettings, style: { display: 'flex', flexDirection: 'column', gap: '12px' } },
           React.createElement('div', { className: 'dsh-cron-form-group', style: { marginBottom: 0 } },
             React.createElement('label', null, T_KEY(t, 'settings.tokenLabel')),
             React.createElement('input', {
@@ -1516,7 +1568,8 @@ window.__ModuleLoader__.load({
             }, T_KEY(t, 'settings.testKanban'))
           )
         )
-      );
+      )
+    );
     }
 
     function nativeButton() {
@@ -1696,6 +1749,9 @@ window.__ModuleLoader__.load({
 
       const toggle = createToggle();
 
+      // The settings card lives ONLY in the plugins settings tab (#102): no
+      // top-level section fallback — the flat sidebar list is a shared core
+      // resource, and the card slot is proven on current builds (#85).
       const cardOk = registerIntoMount(ctx, 'settings.plugin.item', {
         name: 'settings.plugin.item',
         key: NS,
@@ -1703,16 +1759,7 @@ window.__ModuleLoader__.load({
         inject: () => ({ ctx, toggle }),
       }, CronSettingsCard);
       if (!cardOk) {
-        // Fallback for builds without the settings.plugin.item mount: a
-        // dedicated section so the settings are never lost (#85).
-        registerIntoMount(ctx, 'settings.section', {
-          name: 'settings.section',
-          id: PLUGIN_ID,
-          order: 34,
-          locale: NS,
-          label: () => translate('settings.sectionLabel'),
-          inject: () => ({ ctx, toggle }),
-        }, CronSettingsCard);
+        console.warn('[dsh-cron] settings.plugin.item slot unavailable — settings card not registered');
       }
 
       const chipOk = registerIntoMount(ctx, 'conversation.session.header.utilities', {
@@ -1722,7 +1769,7 @@ window.__ModuleLoader__.load({
         locale: NS,
         inject: () => ({ ctx, toggle }),
       }, CronChip);
-      const cardSlot = cardOk ? 'settings.plugin.item' : 'settings.section';
+      const cardSlot = cardOk ? 'settings.plugin.item' : undefined;
       const chipSlot = chipOk ? 'conversation.session.header.utilities' : undefined;
 
       function CronChip(props) {

package/lib/index.js

@@ -66,6 +66,25 @@ async function readBody(req, res) {
   }
 }
 
+/**
+ * Apply settings writes through the registered settings scope so the core
+ * snapshot stays authoritative (#102). Returns null when the settings
+ * service is unavailable (headless profiles fall back to store-only writes).
+ */
+async function applySettingsToScope(settingsScope, body) {
+  if (!settingsScope || typeof settingsScope.set !== 'function') return null;
+  const errors = [];
+  for (const key of ['botToken', 'chatId', 'notifyTelegram', 'onlyOnFailure', 'kanbanBaseUrl']) {
+    if (!Object.prototype.hasOwnProperty.call(body, key)) continue;
+    try {
+      await settingsScope.set(key, body[key]);
+    } catch (e) {
+      errors.push(key + ': ' + ((e && e.message) || e));
+    }
+  }
+  return errors;
+}
+
 /**
  * Shared implementation for the /dsh-cron/tasks... and /dsh-cron/action...
  * route families — the #66 alias is kept alive by the same code path instead
@@ -303,14 +322,18 @@ export function apply(ctx, config) {
   const runner = new SessionRunner(ctx);
   const scheduler = new TaskScheduler(store, (task, opts) => runner.execute(task, opts));
 
-  // Register settings in Harness settings service with Config schema and initial state from store (#71)
+  // Register settings in the Harness settings service (#71, #102): the scope
+  // is the source of truth for the core, scope.watch mirrors values into the
+  // plugin store, and REST writes go through scope.set so nothing bypasses
+  // the settings service.
+  let cronSettingsScope = null;
   ctx.inject(['settings'], (sctx) => {
     try {
-      const scope = sctx.settings?.register?.('dsh-cron', Config, {
+      cronSettingsScope = sctx.settings.register('dsh-cron', Config, {
         base: store.getSettings()
       });
-      if (scope && typeof scope.watch === 'function') {
-        scope.watch((updated) => {
+      if (cronSettingsScope && typeof cronSettingsScope.watch === 'function') {
+        cronSettingsScope.watch((updated) => {
           if (updated && typeof updated === 'object') {
             store.saveSettings(updated);
           }
@@ -407,8 +430,17 @@ export function apply(ctx, config) {
           if (rejectCrossOrigin(req, res)) return;
           const { body, error } = await readBody(req, res);
           if (error) return;
-          store.saveSettings(body);
-          sendJson(res, 200, { ok: true, settings: store.getClientSettings() });
+          // Masked secrets echoed back by the client must never overwrite the
+          // stored token (store.saveSettings guards this too, #86-era).
+          const payload = Object.assign({}, body);
+          if (typeof payload.botToken === 'string' && (payload.botToken.includes('••') || payload.botToken.includes('****'))) {
+            delete payload.botToken;
+          }
+          const scopeErrors = await applySettingsToScope(cronSettingsScope, payload);
+          if (scopeErrors === null) {
+            store.saveSettings(payload);
+          }
+          sendJson(res, 200, { ok: true, settings: store.getClientSettings(), scopeErrors: scopeErrors || undefined });
           return;
         }
         sendJson(res, 405, { ok: false, error: 'Method not allowed' });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@goodandready/dsh-cron",
-  "version": "0.2.0",
+  "version": "0.2.1",
   "description": "Scheduled cron tasks, background automation and agent execution for DeepSeek Harness.",
   "type": "module",
   "main": "lib/index.js",