@goodandready/dsh-kanban 0.2.0 → 0.2.2

package/docs/design/DESIGN.md

@@ -47,7 +47,7 @@
 - Don't: Не переводить карточку в done агентом (колонка done фиксируется только по факту закрытия issue и ветки).
 
 ## Locked Design Decisions
-- 2026-08-20 — Карточка настроек монтируется в settings.plugin.item с резервом в settings.section.
+- 2026-08-20 — Карточка настроек монтируется в settings.plugin.item (резерв в settings.section исключён по #238).
 - 2026-08-25 — Использование дробной индексации ($O(1)$) для позиционирования карточек.
 - 2026-09-02 — Снимок доски объединён на маршруте /dsh-kanban/snapshot (GET: выгрузка, POST: загрузка).
 - 2026-09-03 — Изоляция сессий задач в git worktree ($DSH_HOME/worktrees/<repo-key>/<task-id>/ на ветке task/<id>-<slug>), регистрация воркдерева в workspaceRegistry, проверка dirtyFiles и безопасная очистка в фазе cleanup (#193).
@@ -66,6 +66,8 @@
   - Фильтрация structural noise в проверках dirtyFiles: в checkWorktreeDirty добавлен флаг --ignore-submodules=all, исключающий ложные блокировки удаления воркдеревьев из-за субмодулей и дрейфа gitlinks (#213).
   - Фильтрация сессий субагентов: сессии с маркером subagent (origin === 'subagent' или parentId !== undefined) исключаются из liveSessions и не перезаписывают состояние корневых задач (#215).
 ## Locked Decisions
+- **0.2.2 (Issue #238)**: Удалён устаревший фоллбэк на `settings.section` в пользу строгого размещения карточки в `settings.plugin.item`; все обращения к службам Cordis (в частности к `llm`) приведены к каноническому `ctx.get('...')`.
+- **0.2.1**: Карточка настроек плагина перенесена строго в слот `settings.plugin.item` (раздел «Настройки → Плагины → Настройки плагинов») без одновременной дублирующей регистрации в `settings.section` (Issue #236).
 
 ### 2026-09-08: Релиз v0.2.0 — Milestone (SSE, Chat Sync, Templates, Two-Column Drawer, Multi-Repo) (#210, #219, #220, #222, #225)
 - **Live SSE Channel (`/dsh-kanban/events`)**: Постоянное SSE-соединение с автоматическим переподключением для мгновенного пуша обновлений доски (`task:change`, `board:change`) без избыточного поллинга (#222).

package/docs/plans/2026-09-10-issue-238-settings-scope.md

@@ -0,0 +1,13 @@
+# Issue #238: Убрать settings.section fallback и перевести обращение к службам на ctx.get
+
+## Контекст
+Issue #238 (priority/low, type/tech-debt, scope/webui) указывает:
+1. Карточка настроек регистрирует запасной слот settings.section (строку в боковом списке настроек). Согласно дизайн-контракту и политике DSH-плагинов, боковой список настроек плоский и перегружен, а набор полей плагина должен жить исключительно как карточка внутри «Настройки → Плагины → Настройки плагинов» (settings.plugin.item).
+2. В коде обнаружено обращение ctx.llm вместо канонического ctx.get('llm'). Контекст Cordis оборачивается в прокси, и свойства-поля могут возвращать undefined, поэтому доступ к службам должен выполняться строго через ctx.get('...').
+
+## План работ
+- [ ] 1. Исправить вызовы ctx.llm на ctx.get('llm') в lib/index.js (GET /dsh-kanban/models).
+- [ ] 2. В lib/client.js убрать запасную регистрацию settings.section, оставив карточку только в слоте settings.plugin.item.
+- [ ] 3. Актуализировать тесты в test/board-ui.test.mjs и test/identity.test.mjs.
+- [ ] 4. Обновить docs/design/DESIGN.md (зафиксировать решение #238).
+- [ ] 5. Выполнить полный тестовый прогон npm test (все 651 тест) и npm run compat.

package/lib/client.js

@@ -4217,12 +4217,14 @@ window.__ModuleLoader__.load({
           if (typeof ctx.slots?.has === 'function' && !ctx.slots.has(entry.name)) {
             continue
           }
-          let ok = false
-          ctx.slots.inject(entry.name, () => {
+          let registeredInSlot = false
+          const res = ctx.slots.inject(entry.name, () => {
             ctx.slots.register(entry, component)
-            ok = true
+            registeredInSlot = true
           })
-          if (ok) return entry.name
+          if (registeredInSlot || (res !== null && res !== false && res !== undefined)) {
+            return entry.name
+          }
         } catch { /* слота нет в этой сборке — пробуем следующий */ }
       }
       return undefined
@@ -4233,12 +4235,14 @@ window.__ModuleLoader__.load({
     function apply(ctx) {
       ctx.effect(() => ctx.locale.register(NS, { en, zh }), 'dsh-kanban: словари')
 
-      // Карточка настроек. Ключ обязан совпадать с пространством настроек:
-      // вкладка перебирает объявленные пространства и рисует слот с entryKey,
+      // Карточка настроек в Настройки → Плагины → Настройки плагинов.
+      // Ключ обязан совпадать с пространством настроек: вкладка перебирает
+      // объявленные пространства и рисует слот settings.plugin.item с entryKey,
       // равным имени пространства.
+      // Собственный раздел верхнего уровня (settings.section) не регистрируется,
+      // чтобы не засорять плоский общий список настроек (#238).
       const cardSlot = registerFirst(ctx, [
         { name: 'settings.plugin.item', key: NS, locale: NS, inject: () => ({ ctx }) },
-        { name: 'settings.section', id: '@goodandready/dsh-kanban', order: 32, locale: NS, label: () => fallbackT('title'), inject: () => ({ ctx }) },
       ], KanbanSettingsCard)
 
       // Экран доски встраивается в оболочку, а не в слот: раздела верхнего

package/lib/index.js

@@ -1230,16 +1230,17 @@ export function apply(ctx, config) {
     path: '/dsh-kanban/models',
     handler: async (req, res) => {
       const current = ctx.get('agentDefaultModel')?.currentSelection?.() ?? null
+      const llmService = ctx.get('llm')
       let providers = []
       try {
-        providers = (ctx.llm.listProviders() ?? []).map((p) => ({ id: p.id, name: p.name ?? p.id }))
+        providers = ((llmService?.listProviders?.() ?? []).map((p) => ({ id: p.id, name: p.name ?? p.id })))
       } catch { providers = [] }
 
       const wanted = urlOf(req).searchParams.get('provider') || ''
       if (wanted === '') { writeJson(res, 200, { current, providers, models: [] }); return }
 
       try {
-        const rows = await ctx.llm.listModels(wanted)
+        const rows = await llmService?.listModels?.(wanted)
         writeJson(res, 200, {
           current, providers,
           models: (rows ?? []).map((m) => ({ id: m.id, name: m.name ?? m.id })),

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@goodandready/dsh-kanban",
-  "version": "0.2.0",
+  "version": "0.2.2",
   "description": "Visual Kanban board for DeepSeek Harness: Gitea-backed tasks, workflow columns, and a dedicated agent session per task.",
   "type": "module",
   "main": "lib/index.js",