@goodandready/dsh-moa 0.2.15 → 0.2.17

package/README.md

@@ -282,6 +282,13 @@ npm test
 
 ---
 
+
+## 🛠️ Internal Tooling & Development
+
+For local verification and package integrity validation:
+- `npm test`: runs the full test suite (95 tests)
+- `./deploy.sh`: local infrastructure validation script (verifies package size, identity parity in package.json/cordis.patch.yml/client.js, and tests). Excluded from the published npm package.
+
 ## 📄 License
 
 MIT © [GooDAnDReaDY](https://github.com/GooDAnDReaDY)

package/lib/best-effort.js

@@ -0,0 +1,38 @@
+/**
+ * Run a non-critical side effect; never throw.
+ * Logs at debug/warn when a logger is provided or via console.debug.
+ *
+ * @param {string} label - Context identifier for telemetry / diagnostics
+ * @param {Function} fn - Operation to execute
+ * @param {object} [logger] - Optional logger instance
+ */
+export function bestEffort(label, fn, logger) {
+  const log = (err) => {
+    try {
+      if (logger && typeof logger.debug === 'function') {
+        logger.debug(`[dsh-moa:bestEffort] ${label}:`, err?.message || err)
+      } else if (typeof console !== 'undefined' && typeof console.debug === 'function') {
+        console.debug(`[dsh-moa:bestEffort] ${label}:`, err?.message || err)
+      }
+    } catch {
+      // Logger itself must never crash the process
+    }
+  }
+
+  try {
+    const result = fn()
+    if (result && typeof result.then === 'function') {
+      return result.then(
+        (value) => value,
+        (err) => {
+          log(err)
+          return undefined
+        }
+      )
+    }
+    return result
+  } catch (err) {
+    log(err)
+    return undefined
+  }
+}

package/lib/client.js

@@ -113,6 +113,17 @@ window.__ModuleLoader__.load({
       'badge.disabled': 'MoA Disabled',
       'config.enabled_label': 'Enable MoA (slash command and turn routing)',
       'slash.desc': 'Run turn with Mixture of Agents ensemble synthesis (/moa [preset] <prompt>)',
+      'updater.title': 'Plugin Updates',
+      'updater.desc': 'Check npm registry for new versions and update with one click.',
+      'updater.btnCheck': 'Check Updates',
+      'updater.btnUpdate': 'Update to v{version}',
+      'updater.checking': 'Checking registry...',
+      'updater.updating': 'Updating plugin...',
+      'updater.current': 'Current version: v{version}',
+      'updater.available': 'Update available: v{version}',
+      'updater.upToDate': 'Up to date',
+      'updater.checkFailed': 'Registry check failed',
+      'updater.success': 'Successfully updated to v{version}. Restart DSH to apply.',
     }
 
     const zh = {
@@ -205,6 +216,17 @@ window.__ModuleLoader__.load({
       'badge.disabled': 'MoA 已禁用',
       'config.enabled_label': '启用 MoA（斜杠命令与轮次拦截路由）',
       'slash.desc': '使用智能体混合 (MoA) 协同综合运行此轮次 (/moa [预设] <提示词>)',
+      'updater.title': '插件在线更新',
+      'updater.desc': '检查 npm 镜像源上的最新版本并支持一键无感升级。',
+      'updater.btnCheck': '检查更新',
+      'updater.btnUpdate': '升级至 v{version}',
+      'updater.checking': '正在检查版本...',
+      'updater.updating': '正在执行更新...',
+      'updater.current': '当前版本：v{version}',
+      'updater.available': '发现新版本：v{version}',
+      'updater.upToDate': '已是最新版本',
+      'updater.checkFailed': '检查版本失败',
+      'updater.success': '成功更新至 v{version}，请重启 DSH 使其生效。',
     }
 
 
@@ -572,6 +594,9 @@ window.__ModuleLoader__.load({
       handleSave,
       saveStatus,
       reload,
+      updateState,
+      checkUpdates,
+      runUpdate,
     }) {
       const [selectedPresetName, setSelectedPresetName] = React.useState(defaultPreset || (presets[0] && presets[0].name) || 'default')
       const [isCreating, setIsCreating] = React.useState(false)
@@ -1381,6 +1406,84 @@ window.__ModuleLoader__.load({
           )
         ),
 
+        /* ── PLUGIN UPDATES ── */
+        React.createElement(
+          'div',
+          { className: 'moa-section-card' },
+          React.createElement(
+            'div',
+            { style: { display: 'flex', justifyContent: 'space-between', alignItems: 'center', marginBottom: 6 } },
+            React.createElement('div', { className: 'moa-sec-title' }, '✨ ' + (t('updater.title') || 'Plugin Updates')),
+            React.createElement(
+              'div',
+              { style: { display: 'flex', gap: 8 } },
+              React.createElement(
+                'button',
+                {
+                  type: 'button',
+                  className: 'moa-btn moa-btn-sub',
+                  disabled: updateState?.checking || updateState?.updating,
+                  onClick: checkUpdates,
+                },
+                updateState?.checking ? (t('updater.checking') || 'Checking...') : (t('updater.btnCheck') || 'Check Updates')
+              ),
+              updateState?.updateAvailable
+                ? React.createElement(
+                    'button',
+                    {
+                      type: 'button',
+                      className: 'moa-btn moa-btn-sub moa-btn-primary',
+                      disabled: updateState?.updating,
+                      onClick: runUpdate,
+                    },
+                    updateState?.updating
+                      ? (t('updater.updating') || 'Updating...')
+                      : (t('updater.btnUpdate') || 'Update to v{version}').replace('{version}', updateState.latestVersion || '')
+                  )
+                : null
+            )
+          ),
+          React.createElement(
+            'div',
+            { className: 'moa-sec-desc', style: { marginBottom: 8 } },
+            t('updater.desc') || 'Check npm registry for new versions and update with one click.'
+          ),
+          React.createElement(
+            'div',
+            { style: { display: 'flex', alignItems: 'center', gap: 10, fontSize: 12 } },
+            React.createElement(
+              'span',
+              { style: { color: 'var(--dsw-alias-label-secondary)' } },
+              (t('updater.current') || 'Current version: v{version}').replace('{version}', updateState?.currentVersion || '0.2.16')
+            ),
+            updateState?.checking
+              ? React.createElement('span', { style: { color: 'var(--dsw-alias-state-brand-primary, var(--dsw-alias-label-primary))' } }, t('updater.checking'))
+              : updateState?.updateAvailable
+                ? React.createElement(
+                    'span',
+                    { style: { color: 'var(--dsw-alias-state-error-primary)', fontWeight: 600 } },
+                    (t('updater.available') || 'Update available: v{version}').replace('{version}', updateState?.latestVersion || '')
+                  )
+                : updateState?.currentVersion && !updateState?.error
+                  ? React.createElement('span', { style: { color: 'var(--dsw-alias-state-success-primary)' } }, '✓ ' + (t('updater.upToDate') || 'Up to date'))
+                  : null
+          ),
+          updateState?.notice
+            ? React.createElement(
+                'div',
+                { className: 'moa-alert-ok', style: { marginTop: 8 } },
+                updateState.notice
+              )
+            : null,
+          updateState?.error
+            ? React.createElement(
+                'div',
+                { className: 'moa-alert-err', style: { marginTop: 8 } },
+                updateState.error
+              )
+            : null
+        ),
+
         /* ── ENABLED TOGGLE ── */
         React.createElement(
           'label',
@@ -1444,7 +1547,7 @@ window.__ModuleLoader__.load({
       const snapshot = React.useSyncExternalStore
         ? React.useSyncExternalStore(
             React.useMemo(() => (cb) => (scope ? scope.subscribe(cb) : () => {}), [scope]),
-            React.useCallback(() => (scope ? scope.getSnapshot() : { status: 'ready' }), [scope]),
+            React.useCallback(() => (scope ? scope.getSnapshot() : { status: 'unavailable' }), [scope]),
             React.useCallback(() => ({ status: 'loading' }), [])
           )
         : null
@@ -1454,6 +1557,65 @@ window.__ModuleLoader__.load({
       const [defaultPreset, setDefaultPreset] = React.useState('default')
       const [availableModels, setAvailableModels] = React.useState([])
       const [saveStatus, setSaveStatus] = React.useState('')
+      const [updateState, setUpdateState] = React.useState({
+        checking: false,
+        updating: false,
+        currentVersion: '0.2.16',
+        latestVersion: undefined,
+        updateAvailable: false,
+        notice: null,
+        error: null,
+      })
+
+      const checkUpdates = async () => {
+        setUpdateState((prev) => ({ ...prev, checking: true, error: null, notice: null }))
+        try {
+          const res = await fetch('/api/dsh-moa/update', { cache: 'no-store' })
+          const data = await res.json()
+          if (data && data.currentVersion) {
+            setUpdateState((prev) => ({
+              ...prev,
+              checking: false,
+              currentVersion: data.currentVersion,
+              latestVersion: data.latestVersion,
+              updateAvailable: Boolean(data.updateAvailable),
+              error: data.latestCheckFailed ? (t('updater.checkFailed') || 'Registry check failed') : null,
+            }))
+          } else {
+            setUpdateState((prev) => ({ ...prev, checking: false, error: data.error || 'Check failed' }))
+          }
+        } catch (err) {
+          setUpdateState((prev) => ({ ...prev, checking: false, error: String(err?.message || err) }))
+        }
+      }
+
+      const runUpdate = async () => {
+        setUpdateState((prev) => ({ ...prev, updating: true, error: null, notice: null }))
+        try {
+          const res = await fetch('/api/dsh-moa/update', {
+            method: 'POST',
+            headers: { 'Content-Type': 'application/json', 'x-dsh-plugin-update': '1' },
+          })
+          const data = await res.json()
+          if (data && data.updated) {
+            setUpdateState((prev) => ({
+              ...prev,
+              updating: false,
+              updateAvailable: false,
+              currentVersion: data.updatedVersion || data.latestVersion,
+              notice: (t('updater.success') || 'Successfully updated to v{version}. Restart DSH to apply.').replace('{version}', data.updatedVersion || data.latestVersion),
+            }))
+          } else {
+            setUpdateState((prev) => ({
+              ...prev,
+              updating: false,
+              error: data.error || data.message || 'Update failed',
+            }))
+          }
+        } catch (err) {
+          setUpdateState((prev) => ({ ...prev, updating: false, error: String(err?.message || err) }))
+        }
+      }
       const [enabled, setEnabled] = React.useState(true)
       const [hostStatus, setHostStatus] = React.useState('connecting')
       const [stats, setStats] = React.useState({ totalRuns: null, avgCostUsd: null })
@@ -1461,8 +1623,11 @@ window.__ModuleLoader__.load({
       const [leaderboardPresetFilter, setLeaderboardPresetFilter] = React.useState('all')
 
       React.useEffect(() => {
-        if (!scope || !snapshot) return
-        const snapStatus = snapshot.status || 'ready'
+        if (!scope || !snapshot) {
+          setStatus('unavailable')
+          return
+        }
+        const snapStatus = snapshot.status || 'unavailable'
         if (snapStatus === 'ready') {
           const val = snapshot.value || {}
           if (Array.isArray(val.presets)) setPresets(val.presets)
@@ -1616,6 +1781,9 @@ window.__ModuleLoader__.load({
         handleSave,
         saveStatus,
         reload,
+        updateState,
+        checkUpdates,
+        runUpdate,
       }
     }
 
@@ -1730,7 +1898,7 @@ window.__ModuleLoader__.load({
           return () => {
             for (const off of undo) {
               if (typeof off === 'function') {
-                try { off() } catch {}
+                try { off() } catch (offErr) { /* unsubscribe best-effort */ }
               }
             }
           }

package/lib/file-workspace.js

@@ -1,3 +1,4 @@
+import { bestEffort } from './best-effort.js'
 import vm from 'node:vm'
 import fs from 'node:fs/promises'
 import path from 'node:path'
@@ -94,16 +95,25 @@ function sanitizePath(filePath) {
  * Formats collected project files into structured Markdown code blocks
  * suitable for inclusion in candidate system prompts.
  */
-export function formatProjectContext(projectFiles = []) {
+export function formatProjectContext(projectFiles = [], options = {}) {
   if (!Array.isArray(projectFiles) || projectFiles.length === 0) return ''
+  const { skippedFiles = 0, skippedList = [] } = options
   const fence = '```'
-  return projectFiles
+  let formatted = projectFiles
     .map((f) => `### File: ${f.relativePath}\n${fence}\n${f.content}\n${fence}`)
     .join('\n\n')
+
+  if (skippedFiles > 0) {
+    const preview = skippedList.slice(0, 5).join(', ')
+    const more = skippedList.length > 5 ? ` and ${skippedList.length - 5} more` : ''
+    const warning = `> ⚠️ **Workspace Scan Notice:** ${skippedFiles} workspace file(s) were skipped or truncated due to size/budget limits or read errors (${preview}${more}). The workspace context below is partial.`
+    formatted = `${warning}\n\n${formatted}`
+  }
+  return formatted
 }
 
 export async function collectProjectContext(baseDir, maxCharBudget = 16000) {
-  if (!baseDir) return { files: [], totalChars: 0 }
+  if (!baseDir) return { files: [], totalChars: 0, skippedFiles: 0, skippedList: [] }
 
   const ignoreDirs = new Set(['node_modules', '.git', '.moa', '.worktrees', 'dist', 'build', '.dsh', '.next', '.nuxt', '.cache', 'coverage', '.turbo', '.idea', '.vscode'])
   const codeExts = new Set([
@@ -114,43 +124,66 @@ export async function collectProjectContext(baseDir, maxCharBudget = 16000) {
   const collected = []
   let totalChars = 0
 
+  let skippedFiles = 0
+  const skippedList = []
+
   async function scan(currentDir, relPrefix = '') {
     if (totalChars >= maxCharBudget) return
+    let list
     try {
-      const list = await fs.readdir(currentDir, { withFileTypes: true })
-      for (const item of list) {
-        if (totalChars >= maxCharBudget) break
-        // Skip all dotfiles, including .env and .env.* variants: their
-        // contents must never be shipped to third-party model providers.
-        if (item.name.startsWith('.')) continue
+      list = await fs.readdir(currentDir, { withFileTypes: true })
+    } catch (readDirErr) {
+      skippedFiles++
+      skippedList.push(relPrefix || '.')
+      return
+    }
 
-        const rel = relPrefix ? `${relPrefix}/${item.name}` : item.name
-        const full = path.join(currentDir, item.name)
+    for (const item of list) {
+      // Skip all dotfiles, including .env and .env.* variants: their
+      // contents must never be shipped to third-party model providers.
+      if (item.name.startsWith('.')) continue
 
-        if (item.isDirectory()) {
-          if (!ignoreDirs.has(item.name)) {
-            await scan(full, rel)
-          }
-        } else if (item.isFile()) {
-          const ext = path.extname(item.name).toLowerCase()
-          if (codeExts.has(ext)) {
-            try {
-              const stat = await fs.stat(full)
-              if (stat.size > 100000) continue // skip files > 100kb
-              const content = await fs.readFile(full, 'utf8')
-              const budgetRemaining = maxCharBudget - totalChars
-              const truncated = content.length > budgetRemaining ? content.slice(0, budgetRemaining) + '\n... [truncated]' : content
-              totalChars += truncated.length
-              collected.push({ relativePath: rel, content: truncated, size: stat.size })
-            } catch {}
+      const rel = relPrefix ? `${relPrefix}/${item.name}` : item.name
+      const full = path.join(currentDir, item.name)
+
+      if (totalChars >= maxCharBudget) {
+        if (item.isFile() && codeExts.has(path.extname(item.name).toLowerCase())) {
+          skippedFiles++
+          skippedList.push(`${rel} (budget limit)`)
+        }
+        continue
+      }
+
+      if (item.isDirectory()) {
+        if (!ignoreDirs.has(item.name)) {
+          await scan(full, rel)
+        }
+      } else if (item.isFile()) {
+        const ext = path.extname(item.name).toLowerCase()
+        if (codeExts.has(ext)) {
+          try {
+            const stat = await fs.stat(full)
+            if (stat.size > 100000) {
+              skippedFiles++
+              skippedList.push(`${rel} (>100KB)`)
+              continue
+            }
+            const content = await fs.readFile(full, 'utf8')
+            const budgetRemaining = maxCharBudget - totalChars
+            const truncated = content.length > budgetRemaining ? content.slice(0, budgetRemaining) + '\n... [truncated]' : content
+            totalChars += truncated.length
+            collected.push({ relativePath: rel, content: truncated, size: stat.size })
+          } catch (fileErr) {
+            skippedFiles++
+            skippedList.push(rel)
           }
         }
       }
-    } catch {}
+    }
   }
 
   await scan(baseDir)
-  return { files: collected, totalChars }
+  return { files: collected, totalChars, skippedFiles, skippedList }
 }
 
 /**
@@ -224,7 +257,7 @@ export async function promoteCandidateWorkspace(baseDir, winningIndex, options =
     const entries = await getFilesRecursively(winningDir)
     if (entries.length === 0) {
       if (!options.keepMoa) {
-        try { await fs.rm(moaRoot, { recursive: true, force: true }) } catch {}
+        await bestEffort('clean moaRoot on empty candidate', () => fs.rm(moaRoot, { recursive: true, force: true }))
       }
       return []
     }
@@ -247,7 +280,7 @@ export async function promoteCandidateWorkspace(baseDir, winningIndex, options =
     return promoted
   } catch (err) {
     if (!options.keepMoa) {
-      try { await fs.rm(moaRoot, { recursive: true, force: true }) } catch {}
+      await bestEffort('clean moaRoot on promote error', () => fs.rm(moaRoot, { recursive: true, force: true }))
     }
     return []
   }
@@ -259,9 +292,7 @@ export async function promoteCandidateWorkspace(baseDir, winningIndex, options =
 export async function cleanMoaWorkspaces(baseDir) {
   if (!baseDir) return
   const moaRoot = path.join(baseDir, '.moa')
-  try {
-    await fs.rm(moaRoot, { recursive: true, force: true })
-  } catch {}
+  await bestEffort('cleanMoaWorkspaces', () => fs.rm(moaRoot, { recursive: true, force: true }))
 }
 
 async function getFilesRecursively(dir) {
@@ -269,13 +300,14 @@ async function getFilesRecursively(dir) {
   try {
     const stat = await fs.stat(dir)
     if (!stat.isDirectory()) return []
-  } catch {
+  } catch (statErr) {
     return []
   }
 
   async function scan(current, relPrefix = '') {
+    let list
     try {
-      const list = await fs.readdir(current, { withFileTypes: true })
+      list = await fs.readdir(current, { withFileTypes: true })
       for (const item of list) {
         const rel = relPrefix ? `${relPrefix}/${item.name}` : item.name
         const full = path.join(current, item.name)
@@ -285,7 +317,9 @@ async function getFilesRecursively(dir) {
           results.push(rel)
         }
       }
-    } catch {}
+    } catch (readErr) {
+      return
+    }
   }
   await scan(dir)
   return results

package/lib/history.js

@@ -15,7 +15,7 @@ const DEFAULT_HISTORY_FILE = path.join(DEFAULT_HISTORY_DIR, 'moa-history.jsonl')
 /** Max history file size before rotation (10 MB) */
 export const MAX_HISTORY_BYTES = 10 * 1024 * 1024
 
-export function getHistoryFilePath(customDir) {
+function getHistoryFilePath(customDir) {
   if (customDir) {
     return path.join(customDir, '.moa-history.jsonl')
   }
@@ -33,7 +33,7 @@ export function rotateHistoryFileIfNeeded(filePath = DEFAULT_HISTORY_FILE, maxBy
     if (stat.size >= maxBytes) {
       const backupPath = `${filePath}.1`
       if (fs.existsSync(backupPath)) {
-        try { fs.unlinkSync(backupPath) } catch {}
+        try { fs.unlinkSync(backupPath) } catch (unlinkErr) { /* backup unlink best-effort */ }
       }
       fs.renameSync(filePath, backupPath)
       return true
@@ -241,7 +241,7 @@ export function getMoaRunById(runId, filePath = DEFAULT_HISTORY_FILE) {
       try {
         const item = JSON.parse(lines[i])
         if (item && item.id === runId) return item
-      } catch {}
+      } catch (parseErr) { /* corrupt line skipped */ }
     }
     return null
   } catch {
@@ -270,7 +270,7 @@ export function exportMoaHistory(filePath = DEFAULT_HISTORY_FILE, options = {})
         if (!item) continue
         if (presetFilter && presetFilter !== 'all' && item.preset !== presetFilter) continue
         records.push(item)
-      } catch {}
+      } catch (parseErr) { /* corrupt line skipped */ }
     }
 
     if (format === 'csv') {