dsh-comfyui-canvas 0.1.6 → 0.1.7

package/README.md

@@ -3,7 +3,7 @@
 > [中文](README.zh.md) · English
 
 [![License: MIT](https://img.shields.io/badge/license-MIT-blue.svg)](LICENSE)
-[![Version](https://img.shields.io/badge/version-0.1.6-brightgreen.svg)](https://github.com/wbin0001/dsh-comfyui-canvas/releases)
+[![Version](https://img.shields.io/badge/version-0.1.7-brightgreen.svg)](https://github.com/wbin0001/dsh-comfyui-canvas/releases)
 [![GitHub Stars](https://img.shields.io/github/stars/wbin0001/dsh-comfyui-canvas.svg?style=social)](https://github.com/wbin0001/dsh-comfyui-canvas)
 [![DSH](https://img.shields.io/badge/DSH-v0.1.x-blueviolet.svg)](https://github.com/DeepSeek-Harness/DSH)
 [![ComfyUI](https://img.shields.io/badge/ComfyUI-0.34+-orange.svg)](https://github.com/comfyanonymous/ComfyUI)

package/README.zh.md

@@ -3,7 +3,7 @@
 > 中文 · [English](README.md)
 
 [![License: MIT](https://img.shields.io/badge/license-MIT-blue.svg)](LICENSE)
-[![Version](https://img.shields.io/badge/version-0.1.6-brightgreen.svg)](https://github.com/wbin0001/dsh-comfyui-canvas/releases)
+[![Version](https://img.shields.io/badge/version-0.1.7-brightgreen.svg)](https://github.com/wbin0001/dsh-comfyui-canvas/releases)
 [![GitHub Stars](https://img.shields.io/github/stars/wbin0001/dsh-comfyui-canvas.svg?style=social)](https://github.com/wbin0001/dsh-comfyui-canvas)
 [![DSH](https://img.shields.io/badge/DSH-v0.1.x-blueviolet.svg)](https://github.com/DeepSeek-Harness/DSH)
 [![ComfyUI](https://img.shields.io/badge/ComfyUI-0.34+-orange.svg)](https://github.com/comfyanonymous/ComfyUI)

package/comfyui-bridge/ComfyUI-DSH-Canvas/__init__.py

@@ -9,7 +9,7 @@
 
 # Bridge release version. The DSH plugin compares this against its embedded
 # copy (see version.json) to decide whether an update is needed.
-__version__ = "0.1.6"
+__version__ = "0.1.7"
 
 import os
 import time

package/comfyui-bridge/ComfyUI-DSH-Canvas/version.json

@@ -1,4 +1,4 @@
 {
   "name": "ComfyUI-DSH-Canvas",
-  "version": "0.1.6"
+  "version": "0.1.7"
 }

package/lib/index.js

@@ -303,6 +303,8 @@ export function apply(ctx) {
   // Function-level handle to the settings scope (used by comfyui_config's
   // activeView read); undefined when the settings service is unavailable.
   let settingsScope = undefined
+  // Assigned after a successful settings.register; no-op until then.
+  let safeUpdate = () => undefined
   // The background process handle of the ComfyUI launched by this plugin, so
   // the canvas header's stop/restart buttons can kill it directly.
   let comfyProc = null
@@ -344,6 +346,12 @@ export function apply(ctx) {
     try {
       const scope = settings.register(NAMESPACE, ConfigSchema, { applies: 'live' })
       settingsScope = scope
+      // Every queued settings write is a promise. try/catch CANNOT see its
+      // rejection, and an unhandled rejection is fatal to boot (installFailLoud
+      // reports it as "fatal load failure"). Funnel all writes through here.
+      safeUpdate = (patch) => {
+        try { void Promise.resolve(scope.update(patch)).catch(() => undefined) } catch { /* best effort */ }
+      }
       resolveBase = () => {
         const value = scope.get()
         const base = (value?.baseUrl || DEFAULT_BASE).replace(/\/+$/, '')
@@ -362,7 +370,7 @@ export function apply(ctx) {
         void (async () => {
           const fail = (message) => {
             console.error(`[dsh-comfyui-canvas] ${message}`)
-            try { scope.update({ launchError: message }) } catch { /* best effort */ }
+            safeUpdate({ launchError: message })
           }
           try {
             const cmd = (value?.launchCommand || '').trim()
@@ -371,7 +379,7 @@ export function apply(ctx) {
               return
             }
             // Clear any stale error from a previous attempt before launching.
-            try { scope.update({ launchError: '' }) } catch { /* best effort */ }
+            safeUpdate({ launchError: '' })
             const workdir = (value?.comfyuiDir || '').trim() || undefined
             const shell = ctx.get?.('shell')
             if (shell && typeof shell.start === 'function') {
@@ -405,7 +413,7 @@ export function apply(ctx) {
             }
           } finally {
             launching = false
-            try { scope.update({ launchRequested: false }) } catch { /* best effort */ }
+            safeUpdate({ launchRequested: false })
           }
         })()
       })
@@ -426,7 +434,7 @@ export function apply(ctx) {
             const base = resolveBase()
             const result = await probeBridgeAuth(base, token)
             try {
-              scope.update({ bridgeProbe: result, probeRequested: false })
+              safeUpdate({ bridgeProbe: result, probeRequested: false })
             } catch { /* best effort */ }
           } finally {
             probing = false
@@ -439,7 +447,7 @@ export function apply(ctx) {
       try {
         const pkgPath = fileURLToPath(new URL('../package.json', import.meta.url))
         const version = String(JSON.parse(readFileSync(pkgPath, 'utf8')).version || '')
-        if (version) scope.update({ pluginVersion: version })
+        if (version) safeUpdate({ pluginVersion: version })
       } catch { /* best effort */ }
 
       // Plugin-update watcher: the settings page sets updateRequest with
@@ -455,7 +463,7 @@ export function apply(ctx) {
         updatingPlugin = true
         void (async () => {
           const setState = (patch) => {
-            try { scope.update(patch) } catch { /* best effort */ }
+            safeUpdate(patch)
           }
           try {
             const action = req.action === 'update' ? 'update' : 'check'
@@ -545,7 +553,7 @@ export function apply(ctx) {
         controlling = true
         void (async () => {
           const setState = (patch) => {
-            try { scope.update(patch) } catch { /* best effort */ }
+            safeUpdate(patch)
           }
           try {
             const action = ctrl.action === 'restart' ? 'restart' : 'stop'
@@ -810,6 +818,9 @@ export function apply(ctx) {
       },
       matrix: {
         type: 'object',
+        // Required by the tool-schema compiler: object params must declare
+        // additionalProperties explicitly (see packages/core/tools/src/schema.ts).
+        additionalProperties: true,
         description: 'Declarative parameter sweep: {fields: [{nodeId, key, values}], mode: "zip"|"product"}. Expanded into runs. Use this OR runs, not both.',
       },
     },