@goodandready/dsh-moa 0.2.14 → 0.2.16

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

@@ -1444,7 +1444,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
@@ -1461,8 +1461,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)
@@ -1730,7 +1733,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'
@@ -90,6 +91,18 @@ function sanitizePath(filePath) {
  * Collects readable text files from current workspace to feed as context for candidate models (#6).
  * Ignores node_modules, .git, .moa, binary files, large bundles.
  */
+/**
+ * Formats collected project files into structured Markdown code blocks
+ * suitable for inclusion in candidate system prompts.
+ */
+export function formatProjectContext(projectFiles = []) {
+  if (!Array.isArray(projectFiles) || projectFiles.length === 0) return ''
+  const fence = '```'
+  return projectFiles
+    .map((f) => `### File: ${f.relativePath}\n${fence}\n${f.content}\n${fence}`)
+    .join('\n\n')
+}
+
 export async function collectProjectContext(baseDir, maxCharBudget = 16000) {
   if (!baseDir) return { files: [], totalChars: 0 }
 
@@ -102,51 +115,63 @@ 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) {
+      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
 
-        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 (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 (fileErr) {
+            skippedFiles++
+            skippedList.push(rel)
           }
         }
       }
-    } catch {}
+    }
   }
 
   await scan(baseDir)
-  return { files: collected, totalChars }
+  return { files: collected, totalChars, skippedFiles, skippedList }
 }
 
 /**
  * Checks if a task prompt is an iterative refinement / modification
  * of an existing project rather than a greenfield project creation (#4).
  */
-export function isRefinementTask(userPrompt = '', projectFiles = []) {
-  if (!Array.isArray(projectFiles) || projectFiles.length === 0) return false
+export function isRefinementTask(userPrompt = '', projectFiles = null) {
+  if (Array.isArray(projectFiles) && projectFiles.length === 0) return false
   const p = userPrompt.toLowerCase().trim()
   if (!p) return false
 
@@ -212,7 +237,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 []
     }
@@ -235,7 +260,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 []
   }
@@ -247,9 +272,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) {
@@ -257,13 +280,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)
@@ -273,7 +297,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') {
@@ -298,3 +298,13 @@ export function exportMoaHistory(filePath = DEFAULT_HISTORY_FILE, options = {})
     return options?.format === 'csv' ? '' : '[]'
   }
 }
+
+export function candidatesForHistory(referenceOutputs) {
+  return (referenceOutputs || []).map((r) => ({
+    provider: r.slot?.provider || "",
+    model: r.slot?.model || "",
+    files: (r.files || []).map((f) => f.relativePath),
+    usage: r.usage || { inputTokens: 0, outputTokens: 0 },
+    costUsd: r.costUsd || 0,
+  }))
+}