@goodandready/dsh-kanban 0.2.40 → 0.2.41

package/lib/client.js

@@ -493,7 +493,7 @@ window.__ModuleLoader__.load({
        gain.gain.exponentialRampToValueAtTime(0.01, now + 0.28)
        osc.start(now); osc.stop(now + 0.28)
       }
-     } catch (_) {}
+     } catch (_) { /* audio alert best effort */ }
     }
 
     function showDesktopNotification(title, body, onClick) {
@@ -501,7 +501,7 @@ window.__ModuleLoader__.load({
       if (typeof Notification === 'undefined' || Notification.permission !== 'granted') return
       const n = new Notification(title, { body, tag: 'dsh-kanban' })
       if (onClick) { n.onclick = () => { window.focus?.(); onClick() } }
-     } catch (_) {}
+     } catch (_) { /* desktop notification best effort */ }
     }
 
     function createErrorBoundary() {
@@ -2350,7 +2350,7 @@ window.__ModuleLoader__.load({
      method: 'POST', body: JSON.stringify({ title: tTitle, description: tDesc, board: currentBoard, excludeId: openTask?.id })
     })
     setDuplicates(res?.duplicates || [])
-    } catch (_) {} finally { setCheckingDup(false) }
+    } catch (_) { /* duplicate search best effort */ } finally { setCheckingDup(false) }
     }
     const runTaskAction = async (path, body, okMsg) => {
     const res = await api(path, { method: 'POST', body: JSON.stringify(body || {}) })

package/lib/pr-generator.js

@@ -98,6 +98,7 @@ export function parseDiffStat(statText) {
  * @returns {string} PR type
  */
 export function inferPrType(task, commits = []) {
+  const commitList = typeof commits === 'string' ? parseGitLog(commits) : (Array.isArray(commits) ? commits : [])
   const title = String(task?.title ?? '').toLowerCase()
   const labels = Array.isArray(task?.labels) ? task.labels.map((l) => String(l).toLowerCase()) : []
 
@@ -116,7 +117,7 @@ export function inferPrType(task, commits = []) {
   if (title.startsWith('docs:') || title.includes('document')) return 'docs'
   if (title.startsWith('chore:') || title.includes('cleanup')) return 'chore'
 
-  for (const c of commits) {
+  for (const c of commitList) {
     const msg = String(c?.message ?? '').toLowerCase()
     if (msg.startsWith('fix')) return 'fix'
     if (msg.startsWith('refactor')) return 'refactor'
@@ -134,7 +135,8 @@ export function inferPrType(task, commits = []) {
  * @returns {string} PR title
  */
 export function generatePrTitle(task, commits = []) {
-  const type = inferPrType(task, commits)
+  const commitList = typeof commits === 'string' ? parseGitLog(commits) : (Array.isArray(commits) ? commits : [])
+  const type = inferPrType(task, commitList)
   const scope = inferScope(task)
   let cleanTitle = String(task?.title ?? '').trim()
 
@@ -159,6 +161,8 @@ export function generatePrTitle(task, commits = []) {
  * @returns {string} Complete Markdown description for PR
  */
 export function generatePrDescription({ task, commits = [], diffStats = null, verification = null }) {
+  const commitList = typeof commits === 'string' ? parseGitLog(commits) : (Array.isArray(commits) ? commits : [])
+  const stats = typeof diffStats === 'string' ? parseDiffStat(diffStats) : diffStats
   const title = String(task?.title ?? '').trim()
   const body = String(task?.body ?? '').trim()
   const issueNum = typeof task?.issueNumber === 'number' ? task.issueNumber : null
@@ -193,17 +197,17 @@ export function generatePrDescription({ task, commits = [], diffStats = null, ve
   lines.push('')
 
   // 3. Changed components & diff statistics
-  if (diffStats && (diffStats.files?.length > 0 || diffStats.filesChanged > 0)) {
+  if (stats && (stats.files?.length > 0 || stats.filesChanged > 0)) {
     lines.push('### Changed Components & Files')
-    lines.push(`- Total files changed: ${diffStats.filesChanged || diffStats.files.length}`)
+    lines.push(`- Total files changed: ${stats.filesChanged || stats.files.length}`)
     if (diffStats.additions || diffStats.deletions) {
-      lines.push(`- Changes: +${diffStats.additions} / -${diffStats.deletions}`)
+      lines.push(`- Changes: +${stats.additions} / -${stats.deletions}`)
     }
-    if (Array.isArray(diffStats.files) && diffStats.files.length > 0) {
-      for (const f of diffStats.files.slice(0, 12)) {
+    if (Array.isArray(stats.files) && stats.files.length > 0) {
+      for (const f of stats.files.slice(0, 12)) {
         lines.push(`  - \`${f.path}\``)
       }
-      if (diffStats.files.length > 12) {
+      if (stats.files.length > 12) {
         lines.push(`  - ... and ${diffStats.files.length - 12} more files`)
       }
     }
@@ -213,7 +217,7 @@ export function generatePrDescription({ task, commits = [], diffStats = null, ve
   // 4. Commits list
   if (Array.isArray(commits) && commits.length > 0) {
     lines.push('### Commits')
-    for (const c of commits) {
+    for (const c of commitList) {
       const shaShort = c.sha ? `\`${c.sha.slice(0, 7)}\` ` : ''
       lines.push(`- ${shaShort}${c.message}`)
     }
@@ -252,14 +256,15 @@ export function generatePrDescription({ task, commits = [], diffStats = null, ve
  * @returns {string} Formatted release notes bullet
  */
 export function generateReleaseNotes({ task, commits = [] }) {
+  const commitList = typeof commits === 'string' ? parseGitLog(commits) : (Array.isArray(commits) ? commits : [])
   const title = String(task?.title ?? '').trim()
   const issueNum = typeof task?.issueNumber === 'number' ? ` (#${task.issueNumber})` : ''
   const cleanTitle = title.replace(/^[A-Z]:\s*(Feature|Bug|Fix|Refactor|Task):\s*/i, '')
 
   const lines = [`- **${cleanTitle}${issueNum}**:`]
 
-  if (Array.isArray(commits) && commits.length > 0) {
-    for (const c of commits.slice(0, 5)) {
+  if (Array.isArray(commitList) && commitList.length > 0) {
+    for (const c of commitList.slice(0, 5)) {
       lines.push(`  - ${c.message}`)
     }
   } else {

package/lib/remote-runner.js

@@ -112,7 +112,7 @@ export function createRemoteGitRunner(config, localGitRunner) {
  */
 export async function testRemoteConnection(config) {
   const root = String(config?.remoteRoot || '/mnt/external/Project/DEV').trim()
-  const probeCmd = `test -d ${quoteArg(root)} && echo REMOTE_ROOT_OK || echo REMOTE_ROOT_MISSING`
+  const probeCmd = 'echo REMOTE_OK'
   const sshArgs = buildSshArgs(config, probeCmd)
 
   const startTime = Date.now()
@@ -122,9 +122,11 @@ export async function testRemoteConnection(config) {
       timeout: 12000,
       env: { ...process.env, LC_ALL: 'C' },
     })
+    if (!(stdout || '').includes('REMOTE_OK')) {
+      return { ok: false, error: 'Remote probe failed' }
+    }
     const latencyMs = Date.now() - startTime
-    const trimmed = (stdout || '').trim()
-    const remoteRootFound = trimmed.includes('REMOTE_ROOT_OK')
+    const remoteRootFound = await remotePathExists(root, config)
 
     return {
       ok: true,

package/lib/routes/board.js

@@ -1,3 +1,4 @@
+import { checkCostLimit, calculateTokenCost, extractSessionUsage } from '../cost-telemetry.js'
 import { COLUMN_ORDER, columnsOf, wipLimitField, columnNamesOf } from '../config.js'
 import { parsePlan, planItems, planProgress, taskState, queuePosition } from '../plan.js'
 import { columnStats, doneWithin, stale } from '../metrics.js'
@@ -18,9 +19,19 @@ export function withPlan(task, liveOf, config, now, sameSession) {
     ? (live.step || live.activity || live.stateDetail || '')
     : ''
 
-  const costMetrics = task.costMetrics || null
+  let costMetrics = task.costMetrics || null
+  if (!costMetrics && (task.sessionUsage || live?.usage || live?.projectionView)) {
+    const usage = extractSessionUsage(task.sessionUsage || live?.usage || live?.projectionView)
+    if (usage.totalTokens > 0) {
+      costMetrics = {
+        ...usage,
+        ...calculateTokenCost({ usage, usdRate: config?.usdRate, currency: config?.currency }),
+      }
+    }
+  }
+
   const costLimitExceeded = (config?.costMeterEnabled !== false && Number(config?.taskCostLimit) > 0 && costMetrics?.costUsd)
-    ? costMetrics.costUsd >= Number(config.taskCostLimit)
+    ? checkCostLimit({ costUsd: costMetrics.costUsd, limit: Number(config.taskCostLimit) }).exceeded
     : Boolean(task.costLimitExceeded)
 
   return {

package/lib/routes/task-actions.js

@@ -1,6 +1,6 @@
 import { isTimeMachineAvailable, listTaskSnapshots, createCheckpointSnapshot, rollbackToSnapshot } from '../time-machine.js'
 import { generateTechnicalSpec, parseSpecMarkdown } from '../plan-generator.js'
-import { generatePrTitle, generatePrDescription, generateReleaseNotes } from '../pr-generator.js'
+import { generatePrTitle, generatePrDescription, generateReleaseNotes, parseGitLog, parseDiffStat } from '../pr-generator.js'
 import { getTaskCommits, getTaskDiffInspector } from '../worktree.js'
 
 /**
@@ -484,9 +484,11 @@ export async function handleTaskPrDescription({ store, id, method, body, gitRunn
     }
   }
 
-  const generatedTitle = generatePrTitle(task, commits)
-  const generatedDescription = generatePrDescription({ task, commits, diffStats })
-  const releaseNotes = generateReleaseNotes({ task, commits })
+  const parsedCommits = typeof commits === 'string' ? parseGitLog(commits) : commits
+  const parsedDiff = typeof diffStats === 'string' ? parseDiffStat(diffStats) : diffStats
+  const generatedTitle = generatePrTitle(task, parsedCommits)
+  const generatedDescription = generatePrDescription({ task, commits: parsedCommits, diffStats: parsedDiff })
+  const releaseNotes = generateReleaseNotes({ task, commits: parsedCommits })
 
   return {
     status: 200,

package/lib/store.js

@@ -124,7 +124,11 @@ const WRITABLE = [
 /** Valid transition sources; unexpected values are rejected to preserve log integrity. */
 const SOURCES = new Set(['manual', 'session', 'gitea', 'github', 'tool', 'workflow', 'watchdog'])
 
-function rowToTask(row) {
+let corruptedCostMetricsCount = 0
+export function getCorruptedCostMetricsCount() { return corruptedCostMetricsCount }
+export function resetCorruptedCostMetricsCount() { corruptedCostMetricsCount = 0 }
+
+export function rowToTask(row) {
   if (row === undefined) return undefined
   const { col, labels, waiting, labelColors, runs, checklist, report, comments, rejectReason, rejectedAt, isEpic, parentId, stalled, blockedBy, inceptionSnapshotId, spec, prDescription, ...rest } = row
   let parsedChecklist = []
@@ -177,7 +181,10 @@ function rowToTask(row) {
         try {
           const val = JSON.parse(rest.costMetrics);
           if (val && typeof val === 'object') return val;
-        } catch {}
+        } catch (err) {
+          corruptedCostMetricsCount++;
+          // Corrupted telemetry JSON counted for observability without breaking task hydration
+        }
       }
       return rest.costMetrics && typeof rest.costMetrics === 'object' ? rest.costMetrics : null;
     })(),
@@ -952,6 +959,8 @@ export function openStore({ dir }) {
       secureStorePermissions(dir)
     },
 
+    getCorruptedMetricsCount: () => getCorruptedCostMetricsCount(),
+    resetCorruptedMetricsCount: () => resetCorruptedCostMetricsCount(),
     close() {
       try { secureStorePermissions(dir) } catch { /* ignore */ }
       db.close()

package/lib/worktree.js

@@ -1,3 +1,4 @@
+import { remotePathExists } from './remote-runner.js'
 // Managed worktrees for task agent sessions (Issue #193).
 //
 // Isolating agent sessions in git worktrees protects the main repository
@@ -78,8 +79,12 @@ export function safeRepoKey(task) {
  * @param {Function} [gitRunner]
  * @returns {Promise<boolean>}
  */
-export async function safeWorktreeExists(worktreePath, gitRunner) {
+export async function safeWorktreeExists(worktreePath, gitRunner, config = null) {
   if (!worktreePath) return false
+  if (config?.remoteMode) {
+    const existsRemotely = await remotePathExists(worktreePath, config)
+    if (existsRemotely) return true
+  }
   if (existsSync(worktreePath)) return true
   if (typeof gitRunner === 'function') {
     try {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@goodandready/dsh-kanban",
-  "version": "0.2.40",
+  "version": "0.2.41",
   "description": "Visual Kanban board for DeepSeek Harness: Gitea and GitHub-backed tasks, workflow columns, and a dedicated agent session per task.",
   "type": "module",
   "main": "lib/index.js",
@@ -93,5 +93,6 @@
     "@deepseek-ai/dsh-tools": {
       "optional": true
     }
-  }
+  },
+  "packageManager": "npm@10.9.8"
 }