@linxin666/dsh-client-ui-task-board 0.1.9 → 0.1.11

package/lib/types/client/PluginSettingsCard.d.ts

@@ -29,7 +29,7 @@ export interface PluginSettingsCardProps {
 /**
  * Render one plugin settings card.
  * @param props - the plugin's copy keys, its form state, and its controls.
- * @returns the card, or nothing when the namespace is unavailable.
+ * @returns the card, or nothing while the namespace is still loading.
  */
 export declare function PluginSettingsCard(props: PluginSettingsCardProps): import("react").JSX.Element | null;
 /** Props every field control needs regardless of its value type. */

package/lib/types/client/locales.d.ts

@@ -83,6 +83,7 @@ export declare const zh: {
     'settings.off': string;
     'settings.overridden': string;
     'settings.reset': string;
+    'settings.notExposed': string;
     'settings.readOnly': string;
     'settings.expand': string;
     'settings.collapse': string;

package/lib/types/client/settings-form.d.ts

@@ -37,8 +37,16 @@ export interface FieldState {
 }
 /** Form state every plugin settings card shares. */
 export interface CardShell {
-    /** False while the namespace is not served to this client; the card renders nothing. */
+    /** False while the namespace is still loading; the card renders nothing. */
     available: boolean;
+    /**
+     * Whether the namespace is actually served to this client. False when the
+     * Host deployment does not expose it (e.g. the official apiproxy settings
+     * allowlist omits third-party namespaces): the card renders an explanation
+     * instead of its form, so a missing namespace never looks like a missing
+     * plugin.
+     */
+    exposed: boolean;
     /** Whether the Host document accepts writes. */
     writable: boolean;
     /** Whether the form holds edits that a save would write. */

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@linxin666/dsh-client-ui-task-board",
   "description": "Task board for the dsh web GUI: a sidebar entry plus a multi-column kanban view with local persistence, task details, and real execution through dsh sessions (session.prompt). Hot-pluggable — mounted via ~/.dsh/cordis.patch.yml + a profile node_modules symlink, no dsh source changes.",
-  "version": "0.1.9",
+  "version": "0.1.11",
   "type": "module",
   "main": "lib/index.js",
   "types": "lib/types/index.d.ts",

package/src/client/PluginSettingsCard.tsx

@@ -33,7 +33,7 @@ export interface PluginSettingsCardProps {
 /**
  * Render one plugin settings card.
  * @param props - the plugin's copy keys, its form state, and its controls.
- * @returns the card, or nothing when the namespace is unavailable.
+ * @returns the card, or nothing while the namespace is still loading.
  */
 export function PluginSettingsCard(props: PluginSettingsCardProps) {
   const [open, setOpen] = useState(false)
@@ -41,6 +41,36 @@ export function PluginSettingsCard(props: PluginSettingsCardProps) {
   if (!state.available) return null
   const title = props.t(props.titleKey)
   const blocked = !state.dirty || state.invalid || state.saving
+  // The namespace exists but the Host does not serve it to this client (the
+  // official settings allowlist omits third-party namespaces): show a card
+  // that explains the gap instead of vanishing, so a missing card never
+  // reads as a missing plugin.
+  if (!state.exposed) {
+    return (
+      <li className={css.card}>
+        <button
+          type="button"
+          className={css.header}
+          aria-expanded={open}
+          aria-label={`${props.t(open ? 'settings.collapse' : 'settings.expand')}: ${title}`}
+          onClick={() => { setOpen(!open) }}
+        >
+          <span className={css.headText}>
+            <span className={css.name}>{title}</span>
+            <span className={css.description}>{props.t(props.descriptionKey)}</span>
+          </span>
+          <span className={open ? css.chevronOpen : css.chevron}>▾</span>
+        </button>
+        {open
+          ? (
+            <div className={css.body}>
+              <p className={css.notExposed} role="status">{props.t('settings.notExposed')}</p>
+            </div>
+          )
+          : null}
+      </li>
+    )
+  }
   return (
     <li className={css.card}>
       <button

package/src/client/board.module.css

@@ -16,7 +16,7 @@
   position: absolute;
   inset: 0;
   display: none;
-  z-index: 5;
+  z-index: 60;
 }
 
 /* The center column is single-occupant; the :not() guards keep the two
@@ -27,9 +27,13 @@ html[data-dsh-taskboard-active]:not([data-dsh-ssh-active]) [data-dsh-taskboard-v
 }
 
 /* While the board is active, the conversation content underneath is hidden
-   (it stays mounted and stateful). */
+   (it stays mounted and stateful). The !important is required: the dsh shell
+   (0.1.0-rc.6) wraps the conversation view in a node with an inline
+   `display: contents`, and inline styles beat a plain stylesheet rule. Without
+   it the composer (input card) stays visible at the bottom and paints over the
+   board modals' footer, hiding the run/delete buttons (issue #76). */
 html[data-dsh-taskboard-active]:not([data-dsh-ssh-active]) [data-pane='conversation'] > :not([data-dsh-taskboard-view]) {
-  display: none;
+  display: none !important;
 }
 
 /* --- sidebar entry row ------------------------------------------------------- */
@@ -426,7 +430,7 @@ html[data-dsh-taskboard-active]:not([data-dsh-ssh-active]) [data-pane='conversat
 .modalBackdrop {
   position: fixed;
   inset: 0;
-  z-index: 40;
+  z-index: 1300;
   display: flex;
   align-items: center;
   justify-content: center;

package/src/client/locales.ts

@@ -85,6 +85,7 @@ export const zh = {
   'settings.off': '关',
   'settings.overridden': '已覆盖',
   'settings.reset': '恢复默认',
+  'settings.notExposed': '当前 DSH 版本未向设置页暴露本插件的配置命名空间，表单不可用。可编辑 ~/.dsh/settings.yaml 直接配置，或为 dsh-host-apiproxy 的 WEB_SETTINGS_NAMESPACES 白名单补充本命名空间后重启。',
   'settings.readOnly': '当前部署的设置只读。',
   'settings.expand': '展开设置',
   'settings.collapse': '收起设置',
@@ -176,6 +177,7 @@ export const en: Record<keyof typeof zh, string> = {
   'settings.off': 'Off',
   'settings.overridden': 'Overridden',
   'settings.reset': 'Reset to default',
+  'settings.notExposed': 'This DSH version does not expose this plugin\'s settings namespace to the configuration page, so the form is unavailable. Edit ~/.dsh/settings.yaml directly, or add the namespace to dsh-host-apiproxy\'s WEB_SETTINGS_NAMESPACES allowlist and restart.',
   'settings.readOnly': 'This deployment stores settings read-only.',
   'settings.expand': 'Show settings',
   'settings.collapse': 'Hide settings',

package/src/client/settings-card.module.css

@@ -74,6 +74,13 @@
   color: var(--dsw-alias-label-tertiary);
 }
 
+.notExposed {
+  margin: 0;
+  font-size: 12px;
+  line-height: 1.5;
+  color: var(--dsw-alias-state-warn-primary);
+}
+
 .footer {
   display: flex;
   align-items: center;

package/src/client/settings-form.ts

@@ -40,8 +40,16 @@ export interface FieldState {
 
 /** Form state every plugin settings card shares. */
 export interface CardShell {
-  /** False while the namespace is not served to this client; the card renders nothing. */
+  /** False while the namespace is still loading; the card renders nothing. */
   available: boolean
+  /**
+   * Whether the namespace is actually served to this client. False when the
+   * Host deployment does not expose it (e.g. the official apiproxy settings
+   * allowlist omits third-party namespaces): the card renders an explanation
+   * instead of its form, so a missing namespace never looks like a missing
+   * plugin.
+   */
+  exposed: boolean
   /** Whether the Host document accepts writes. */
   writable: boolean
   /** Whether the form holds edits that a save would write. */
@@ -157,7 +165,8 @@ export class CardForm<T> {
     const snapshot = this.scope.getSnapshot()
     const plan = this.plan()
     return {
-      available: snapshot.status === 'ready',
+      available: snapshot.status !== 'loading',
+      exposed: snapshot.status === 'ready',
       writable: snapshot.writable,
       dirty: plan.length > 0,
       invalid: plan.some(item => item.run === undefined),

package/src/client/sidebar-entry.ts

@@ -139,11 +139,15 @@ export function mountSidebarEntry(controller: BoardController): () => void {
     }
   })
 
-  // Reflect the board's open state on the row (active highlight).
-  const unsubscribe = controller.subscribe(() => {
-    entry.dataset.active = controller.getSnapshot().boardOpen ? 'true' : undefined
-  })
-  entry.dataset.active = controller.getSnapshot().boardOpen ? 'true' : undefined
+  // Reflect the board's open state on the row (active highlight). Note: assigning
+  // undefined to dataset.active materializes data-active="undefined" and keeps the
+  // row permanently highlighted — delete the attribute instead.
+  const syncActive = () => {
+    if (controller.getSnapshot().boardOpen) entry.dataset.active = 'true'
+    else delete entry.dataset.active
+  }
+  const unsubscribe = controller.subscribe(syncActive)
+  syncActive()
 
   tryPlace()