@goodandready/dsh-tts 0.4.16 → 0.4.18

package/CHANGELOG.md

@@ -0,0 +1,35 @@
+# Changelog
+
+Notable changes to `@goodandready/dsh-tts`.
+
+## 0.4.18
+
+### Fixed
+- **Settings reachable again on the plugin's own page**: the current DSH core
+  (0.1.6-alpha.2) renders a plugin's configuration page only for entries registered
+  in the plugin-list seat `plugins.item`. `TtsRowConfig` is now registered there too
+  (`id: 'dsh-tts'`, order 60, static label); the row seat and the legacy
+  `settings.plugin.item` card stay as fallbacks. Sources edited in `lib/client-src`,
+  `lib/client.js` rebuilt.
+
+## 0.4.17
+
+### Added
+
+- Settings now register into the Plugins page's row seat, `plugins.row.config`,
+  keyed `@goodandready/dsh-tts#dsh-tts` (row id as `cordis.patch.yml` declares
+  it): the plugin's row on the Plugins page gains a configure control whose page
+  is the same settings form (`view: 'page'`), with a one-line state under the row
+  title (`view: 'summary'`). The previous seat, `settings.plugin.item`, stays as
+  a fallback for older cores, so settings never become unreachable (#127).
+
+### Fixed
+
+- The row-seat page renders the settings form **bare**: the host page already
+  draws the title, icon and crumb and provides the content padding, so our own
+  card chrome would double the border and shift the form out of the content area.
+
+### Tests
+
+- `test/row-config-seat.test.mjs` guards the row-seat key, the row id against
+  `cordis.patch.yml`, the fallback seats, and the bare view-aware rendering.

package/lib/client-src/00-open.js

@@ -15,6 +15,10 @@ window.__ModuleLoader__.load({
       ChevronIcon = null
     }
     const NS = 'dsh-tts'
+    // Plugins page row seat: key = '<package name>#<row id>', row id as cordis.patch.yml declares it.
+    const PKG = '@goodandready/dsh-tts'
+    const ROW_ID = 'dsh-tts'
+    const ROW_CONFIG_KEY = PKG + '#' + ROW_ID
     const PROVIDERS = [
       'openai', 'elevenlabs', 'google', 'azure', 'groq', 'deepgram', 'openrouter',
       'edge', 'piper', 'espeak',

package/lib/client-src/10-styles.js

@@ -2,6 +2,7 @@
     const SET_CSS =
       '.dts-wrap{display:flex;flex-direction:column;gap:18px;padding:4px 0 24px;max-width:900px}' +
       '.dts-header{display:flex;flex-direction:column;gap:8px;padding-bottom:14px;border-bottom:1px solid var(--dsw-alias-border-l2)}' +
+      '.dts-page{padding:0}' +
       '.dts-page-title{font-size:20px;font-weight:700;color:var(--dsw-alias-label-primary);display:flex;align-items:center;gap:10px}' +
       '.dts-page-sub{font-size:13px;color:var(--dsw-alias-label-secondary);line-height:1.5}' +
       '.dts-block{border:1px solid var(--dsw-alias-border-l2);background:var(--dsw-alias-bg-layer-3);border-radius:12px;padding:16px 18px;display:flex;flex-direction:column;gap:12px}' +

package/lib/client-src/60-card-dock.js

@@ -118,6 +118,26 @@
       ))
     }
 
+    /**
+     * Plugins page row seat (`plugins.row.config`, current core).
+     *
+     * The host page draws the title, icon and crumb and provides the content
+     * padding, so the page view renders the settings form bare: our own card
+     * chrome would double the border and shift the form out of the content
+     * area. The summary view feeds the one-liner under the row title.
+     */
+    function TtsRowConfig(props) {
+      const t = (props && props.t) || ((key) => key)
+      if (props && props.view === 'summary') {
+        return React.createElement('div', { className: 'dts-sub' }, t('subtitle'))
+      }
+      return React.createElement('div', { className: 'dts-page' },
+        React.createElement(ErrorBoundary, null,
+          React.createElement(TtsSection, Object.assign({}, props, { ctx: props && props.ctx })),
+        ),
+      )
+    }
+
     function registerSettings(ctx) {
     // Dictionary packages may also register languages for other namespaces.
     // Core throws on duplicate namespace+language; an unguarded call used to
@@ -137,9 +157,44 @@
     // The sidebar draws the section label, not our component: props.t never
     // reaches it, so bind the translator to the namespace ourselves.
       const t = ctx.locale.bind(NS)
-    // Primary settings home is the plugin-settings tab card. The tab looks up
-    // a slot by entryKey equal to the namespace name: key must equal NS or
-    // the card never appears, with no log error.
+    // Current core: settings live on the plugin's own row on the Plugins page
+    // (`plugins.row.config`). Key = '<package name>#<row id>'; the row gains a
+    // configure control that asks for view:'summary' and view:'page'.
+      try {
+        ctx.slots.inject('plugins.row.config', () => ctx.slots.register(
+          {
+            name: 'plugins.row.config',
+            key: ROW_CONFIG_KEY,
+            locale: NS,
+            inject: () => ({ ctx: ctx }),
+          },
+          TtsRowConfig,
+        ))
+      } catch (rowSeatUnavailable) {
+        console.warn('[dsh-tts] row seat plugins.row.config not available:', rowSeatUnavailable && rowSeatUnavailable.message)
+      }
+    // Plugin-list seat (`plugins.item`): the seat the current core (0.1.6-alpha.2)
+    // renders as the plugin's own page with its configuration. The label is a static
+    // string on purpose — it is resolved while the page renders, and a locale lookup
+    // there would take the whole client batch down with it.
+      try {
+        ctx.slots.inject('plugins.item', () => ctx.slots.register(
+          {
+            name: 'plugins.item',
+            id: ROW_ID,
+            order: 60,
+            label: () => 'Text to Speech',
+            locale: NS,
+            inject: () => ({ ctx: ctx }),
+          },
+          TtsRowConfig,
+        ))
+      } catch (itemSeatUnavailable) {
+        console.warn('[dsh-tts] list seat plugins.item not available:', itemSeatUnavailable && itemSeatUnavailable.message)
+      }
+    // Fallback for hosts that render the older plugin-settings tab card. The tab
+    // looks up a slot by entryKey equal to the namespace name: key must equal NS
+    // or the card never appears, with no log error.
       ctx.slots.inject('settings.plugin.item', () => ctx.slots.register(
         {
           name: 'settings.plugin.item',

package/lib/client.js

@@ -15,6 +15,10 @@ window.__ModuleLoader__.load({
       ChevronIcon = null
     }
     const NS = 'dsh-tts'
+    // Plugins page row seat: key = '<package name>#<row id>', row id as cordis.patch.yml declares it.
+    const PKG = '@goodandready/dsh-tts'
+    const ROW_ID = 'dsh-tts'
+    const ROW_CONFIG_KEY = PKG + '#' + ROW_ID
     const PROVIDERS = [
       'openai', 'elevenlabs', 'google', 'azure', 'groq', 'deepgram', 'openrouter',
       'edge', 'piper', 'espeak',
@@ -87,6 +91,7 @@ window.__ModuleLoader__.load({
     const SET_CSS =
       '.dts-wrap{display:flex;flex-direction:column;gap:18px;padding:4px 0 24px;max-width:900px}' +
       '.dts-header{display:flex;flex-direction:column;gap:8px;padding-bottom:14px;border-bottom:1px solid var(--dsw-alias-border-l2)}' +
+      '.dts-page{padding:0}' +
       '.dts-page-title{font-size:20px;font-weight:700;color:var(--dsw-alias-label-primary);display:flex;align-items:center;gap:10px}' +
       '.dts-page-sub{font-size:13px;color:var(--dsw-alias-label-secondary);line-height:1.5}' +
       '.dts-block{border:1px solid var(--dsw-alias-border-l2);background:var(--dsw-alias-bg-layer-3);border-radius:12px;padding:16px 18px;display:flex;flex-direction:column;gap:12px}' +
@@ -1876,6 +1881,26 @@ async function pollPending() {
       ))
     }
 
+    /**
+     * Plugins page row seat (`plugins.row.config`, current core).
+     *
+     * The host page draws the title, icon and crumb and provides the content
+     * padding, so the page view renders the settings form bare: our own card
+     * chrome would double the border and shift the form out of the content
+     * area. The summary view feeds the one-liner under the row title.
+     */
+    function TtsRowConfig(props) {
+      const t = (props && props.t) || ((key) => key)
+      if (props && props.view === 'summary') {
+        return React.createElement('div', { className: 'dts-sub' }, t('subtitle'))
+      }
+      return React.createElement('div', { className: 'dts-page' },
+        React.createElement(ErrorBoundary, null,
+          React.createElement(TtsSection, Object.assign({}, props, { ctx: props && props.ctx })),
+        ),
+      )
+    }
+
     function registerSettings(ctx) {
     // Dictionary packages may also register languages for other namespaces.
     // Core throws on duplicate namespace+language; an unguarded call used to
@@ -1895,9 +1920,44 @@ async function pollPending() {
     // The sidebar draws the section label, not our component: props.t never
     // reaches it, so bind the translator to the namespace ourselves.
       const t = ctx.locale.bind(NS)
-    // Primary settings home is the plugin-settings tab card. The tab looks up
-    // a slot by entryKey equal to the namespace name: key must equal NS or
-    // the card never appears, with no log error.
+    // Current core: settings live on the plugin's own row on the Plugins page
+    // (`plugins.row.config`). Key = '<package name>#<row id>'; the row gains a
+    // configure control that asks for view:'summary' and view:'page'.
+      try {
+        ctx.slots.inject('plugins.row.config', () => ctx.slots.register(
+          {
+            name: 'plugins.row.config',
+            key: ROW_CONFIG_KEY,
+            locale: NS,
+            inject: () => ({ ctx: ctx }),
+          },
+          TtsRowConfig,
+        ))
+      } catch (rowSeatUnavailable) {
+        console.warn('[dsh-tts] row seat plugins.row.config not available:', rowSeatUnavailable && rowSeatUnavailable.message)
+      }
+    // Plugin-list seat (`plugins.item`): the seat the current core (0.1.6-alpha.2)
+    // renders as the plugin's own page with its configuration. The label is a static
+    // string on purpose — it is resolved while the page renders, and a locale lookup
+    // there would take the whole client batch down with it.
+      try {
+        ctx.slots.inject('plugins.item', () => ctx.slots.register(
+          {
+            name: 'plugins.item',
+            id: ROW_ID,
+            order: 60,
+            label: () => 'Text to Speech',
+            locale: NS,
+            inject: () => ({ ctx: ctx }),
+          },
+          TtsRowConfig,
+        ))
+      } catch (itemSeatUnavailable) {
+        console.warn('[dsh-tts] list seat plugins.item not available:', itemSeatUnavailable && itemSeatUnavailable.message)
+      }
+    // Fallback for hosts that render the older plugin-settings tab card. The tab
+    // looks up a slot by entryKey equal to the namespace name: key must equal NS
+    // or the card never appears, with no log error.
       ctx.slots.inject('settings.plugin.item', () => ctx.slots.register(
         {
           name: 'settings.plugin.item',

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@goodandready/dsh-tts",
-  "version": "0.4.16",
+  "version": "0.4.18",
   "description": "Text-to-speech for DeepSeek Harness: speak agent replies in the Web UI with a provider fallback chain (OpenAI, ElevenLabs, Google, Azure, Groq, Deepgram, OpenRouter, Edge, Piper, eSpeak).",
   "license": "MIT",
   "type": "module",
@@ -15,6 +15,7 @@
     "lib/",
     "scripts/",
     "cordis.patch.yml",
+    "CHANGELOG.md",
     "README.md",
     "LICENSE",
     "README.ru.md",