@mzzsfy/dsh-cron-board 0.3.1 → 0.3.2

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@mzzsfy/dsh-cron-board",
   "description": "轻量级定时任务看板:环境变量集中管理与定时任务调度,执行体支持本地脚本与 dsh 会话任务",
-  "version": "0.3.1",
+  "version": "0.3.2",
   "type": "module",
   "main": "src/index.js",
   "exports": {

package/src/client.js

@@ -679,7 +679,7 @@ if (typeof window !== 'undefined' && window.__ModuleLoader__) {
           h('button', { className: 'cb-button cb-button--danger', onClick: onConfirm }, '删除')))
     }
 
-    function JobsTab({ jobs, status, now, reload, wsModel }) {
+    function JobsTab({ jobs, status, now, reload, applyJobPatch, wsModel }) {
       const [formJob, setFormJob] = useState(null)
       const [notice, setNotice] = useState(null)
       const [kindFilter, setKindFilter] = useState('all')
@@ -691,7 +691,13 @@ if (typeof window !== 'undefined' && window.__ModuleLoader__) {
         setTimeout(reload, 300)
       }
       const toggleJob = async (job) => {
-        await request('PATCH', 'jobs/' + job.id, { enabled: !job.enabled })
+        // 乐观翻转:受控开关即时到位;失败回滚并提示,不静默
+        applyJobPatch(job.id, { enabled: !job.enabled })
+        const outcome = await request('PATCH', 'jobs/' + job.id, { enabled: !job.enabled })
+        if (!outcome.ok) {
+          applyJobPatch(job.id, { enabled: job.enabled })
+          setNotice('切换失败:' + outcome.error)
+        }
         reload()
       }
       const deleteJob = async (job) => {
@@ -811,14 +817,20 @@ if (typeof window !== 'undefined' && window.__ModuleLoader__) {
           h('button', { className: 'cb-button cb-button--danger', onClick: () => doApply(true) }, '覆盖导入')))
     }
 
-    function EnvsTab({ envs, reload }) {
+    function EnvsTab({ envs, reload, applyEnvPatch }) {
       const [formRow, setFormRow] = useState(null)
       const [importing, setImporting] = useState(false)
       const [exportText, setExportText] = useState(null)
       const [notice, setNotice] = useState(null)
 
       const toggleEnv = async (row) => {
-        await request('PATCH', 'envs/' + row.id, { enabled: !row.enabled })
+        // 乐观翻转:受控开关即时到位;失败回滚并提示,不静默
+        applyEnvPatch(row.id, { enabled: !row.enabled })
+        const outcome = await request('PATCH', 'envs/' + row.id, { enabled: !row.enabled })
+        if (!outcome.ok) {
+          applyEnvPatch(row.id, { enabled: row.enabled })
+          setNotice('切换失败:' + outcome.error)
+        }
         reload()
       }
       const deleteEnv = async (row) => {
@@ -921,6 +933,13 @@ if (typeof window !== 'undefined' && window.__ModuleLoader__) {
       const [now, setNow] = useState(Date.now())
       const [reloadFlag, setReloadFlag] = useState(0)
       const reload = useCallback(() => setReloadFlag((value) => value + 1), [])
+      // 乐观更新:受控开关点击即时到位,PATCH 失败由调用方回滚;reload 拉权威态对账
+      const applyJobPatch = useCallback((id, patch) => {
+        setJobs((prev) => prev.map((job) => (job.id === id ? { ...job, ...patch } : job)))
+      }, [])
+      const applyEnvPatch = useCallback((id, patch) => {
+        setEnvs((prev) => prev.map((row) => (row.id === id ? { ...row, ...patch } : row)))
+      }, [])
 
       useEffect(() => {
         if (!active) return undefined
@@ -949,8 +968,8 @@ if (typeof window !== 'undefined' && window.__ModuleLoader__) {
           status ? h('span', { className: 'cb-meta' },
             '运行中 ' + status.scheduler.active + ' · 队列 ' + status.scheduler.queued +
             (status.nextAt ? ' · 下次 ' + relativeTime(status.nextAt, now) : '')) : null),
-        tab === 'jobs' ? h(JobsTab, { key: 'jobs', jobs, status, now, reload, wsModel }) : null,
-        tab === 'envs' ? h(EnvsTab, { key: 'envs', envs, reload }) : null,
+        tab === 'jobs' ? h(JobsTab, { key: 'jobs', jobs, status, now, reload, applyJobPatch, wsModel }) : null,
+        tab === 'envs' ? h(EnvsTab, { key: 'envs', envs, reload, applyEnvPatch }) : null,
         tab === 'logs' ? h(LogsTab, { key: 'logs', jobs, reload, reloadFlag }) : null)
     }
 

package/test/switch-guard.test.mjs

@@ -60,6 +60,24 @@ test('client.js 无顶层词法声明(经典 script 书挡内安全)', () => {
   assert.ok(trimmed.endsWith('})()'), '必须以 IIFE 书挡结尾')
 })
 
+test('启停开关乐观更新契约:即时翻转 + 失败回滚提示,禁止等待往返才反馈', () => {
+  // Given 受控 switch(checked 锁定 props)点击后无本地 state 变化不产生视觉反馈
+  // When toggleJob/toggleEnv 发起 PATCH
+  // Then 先乐观翻转,失败回滚并 notice,最后 reload 对账权威态(顺序锁定)
+  for (const fn of ['toggleJob', 'toggleEnv']) {
+    const start = source.indexOf('const ' + fn + ' = async')
+    assert.ok(start > 0, `缺少 ${fn}`)
+    const body = source.slice(start, source.indexOf('reload()', start))
+    assert.match(body, /apply(Job|Env)Patch\([^)]*, \{ enabled: !/m, `${fn} 必须先乐观翻转`)
+    assert.match(body, /if \(!outcome\.ok\)/m, `${fn} 必须检查 PATCH 结果`)
+    assert.match(body, /apply(Job|Env)Patch\([^)]*, \{ enabled: (job|row)\.enabled \}\)/m, `${fn} 失败必须回滚`)
+    assert.match(body, /切换失败:/m, `${fn} 失败必须提示,禁止静默`)
+  }
+  // 两个 Tab 均接收乐观更新通道(Board 下传)
+  assert.match(source, /applyJobPatch, wsModel/)
+  assert.match(source, /envs, reload, applyEnvPatch/)
+})
+
 test('client.js 主页面双形态挂载契约(默认主界面;设置开关手动移入侧边栏;无自动回退)', () => {
   // Given 用户要求:默认永远主界面;检测到 better-sidebar 提供设置项,由用户手动移入;
   //        移入后页签关闭/禁用不再自动返回主界面(挂载权完全交给用户与侧边栏)