From 557b5ae9c24a54e32e9507a4eb4000f8ff43b685 Mon Sep 17 00:00:00 2001 From: Hermes Agent Date: Tue, 19 May 2026 16:17:38 +0800 Subject: [PATCH] fix(qdreader): limit archive cron grace window --- docs/autman-plugin-rules.md | 2 +- js/qdreader_sign_autman.js | 11 ++++++----- plugins/qdreader/README.md | 2 +- plugins/qdreader/qdreader_sign_autman.js | 11 ++++++----- tests/qdreader_plugin.test.mjs | 17 +++++++++++++++-- 5 files changed, 29 insertions(+), 14 deletions(-) diff --git a/docs/autman-plugin-rules.md b/docs/autman-plugin-rules.md index b10eb9c..fe4b1bf 100644 --- a/docs/autman-plugin-rules.md +++ b/docs/autman-plugin-rules.md @@ -619,7 +619,7 @@ QDReader 这次踩坑说明：账号 ID 识别成功，不代表 Autman 会话 - `xxx查询` 的数据来源必须写清楚：如果没有实时接口，就命名为“最近签到记录/最后执行日志”，读取 `last_run_json` 和 `last_result_json`，不要让用户误以为是实时查询。 - 执行动作要同时写批次日志和单账号日志；成功、业务失败、请求异常都要落日志，异常不能只显示在当次回复里。 -- 如果插件需要按天判断执行情况，不要把归档和运行任务放在同一个 cron 入口。优先在同一个插件内声明两个独立 cron，并在入口根据触发时间/触发标识分流：业务时间 cron（如 `15 7 * * *`）只执行签到/业务动作并写 `last_run_json` / `last_result_json`；零点归档 cron（如 `0 0 * * *`）只把上一份批次日志按日期写入 `archive_json`，再清空 `last_run_json` / `last_result_json`，绝不执行签到/业务动作。若只能按当前本机时间识别系统触发，归档窗口要覆盖调度延迟风险（例如 00:00-00:59 都归档），避免 00:00 cron 延迟几分钟后误走业务签到。只有在当前 Autman 运行环境确认不可靠支持单插件多 `[cron]` 时，才退而拆成两个插件文件。查询菜单优先展示当日/当前批次；当日日志为空时回退最近归档，按归档结果反馈成功/失败。 +- 如果插件需要按天判断执行情况，不要把归档和运行任务放在同一个 cron 入口。优先在同一个插件内声明两个独立 cron，并在入口根据触发时间/触发标识分流：业务时间 cron（如 `15 7 * * *`）只执行签到/业务动作并写 `last_run_json` / `last_result_json`；零点归档 cron（如 `0 0 * * *`）只把上一份批次日志按日期写入 `archive_json`，再清空 `last_run_json` / `last_result_json`，绝不执行签到/业务动作。若只能按当前本机时间识别系统触发，归档窗口要覆盖合理调度延迟风险（例如 00:00-00:04 都归档），避免 00:00 cron 短延迟后误走业务签到，同时不要让窗口过长影响业务语义。只有在当前 Autman 运行环境确认不可靠支持单插件多 `[cron]` 时，才退而拆成两个插件文件。查询菜单优先展示当日/当前批次；当日日志为空时回退最近归档，按归档结果反馈成功/失败。 - 普通回复永远不要输出完整 Cookie；只展示脱敏 UID、备注、Cookie key 摘要或长度。 - 高风险的导出/批量导入功能默认不做，除非 BOSS 明确需要。 - Node VM 测试应模拟：`sender.reply`、`sender.listen`、`sender.getUserID`、`get/set`、`bucketGet/bucketSet`、`request`。 diff --git a/js/qdreader_sign_autman.js b/js/qdreader_sign_autman.js index b6e8d89..d68a267 100644 --- a/js/qdreader_sign_autman.js +++ b/js/qdreader_sign_autman.js @@ -4,7 +4,7 @@ //[author: Hermes] //[service: BOSS] //[class: 工具类] -//[version: 2.5.3] +//[version: 2.5.4] //[platform: web,qq,wx,tg,tb,fs,we] //[public: false] //[price: 0] @@ -261,7 +261,7 @@ function contentText() { } function usage() { return [ - "启点读书签到插件 v2.5.3", + "启点读书签到插件 v2.5.4", "【一级菜单】", "账号管理：启点账号", "执行查询：启点查询", @@ -455,9 +455,10 @@ function latestArchiveForVisible(uids) { function cronMode() { var d = new Date() var h = d.getHours ? d.getHours() : 7 - // Autman cron can be delayed by scheduler/load; any system-triggered run during 00:00-00:59 is treated as archive-only. - // This avoids a dangerous midnight sign if the 00:00 archive cron is not executed exactly at minute 0. - if (h === 0) return "archive" + var m = d.getMinutes ? d.getMinutes() : 15 + // Autman cron can be delayed by scheduler/load; allow a 5-minute archive-only window after 00:00. + // Outside 00:00-00:04, system-triggered runs follow the business sign path. + if (h === 0 && m < 5) return "archive" return "sign" } function formatArchiveReply(ret) { diff --git a/plugins/qdreader/README.md b/plugins/qdreader/README.md index f0feadd..7f5ea30 100644 --- a/plugins/qdreader/README.md +++ b/plugins/qdreader/README.md @@ -133,7 +133,7 @@ set(key, value) 4. 写入单账号当日结果 `qdreader_last_result_json`，包括 `ok/msg/time/source/runId`。 5. 写入当日批次日志 `qdreader_last_run_json`，包括触发来源、开始/结束时间、成功/失败/总数和每账号结果；即使账号执行异常也会记录失败日志。 6. 插件内有两个 cron：`0 0 * * *` 和 `15 7 * * *`。 -7. 每日 00:00 触发时只做日志归档；为防止调度器延迟，系统触发发生在 00:00-00:59 都按归档处理：把上一份 `qdreader_last_run_json` 按完成日期写入 `qdreader_archive_json`，再清空 `qdreader_last_result_json` / `qdreader_last_run_json`；不请求签名网关、不执行签到。 +7. 每日 00:00 触发时只做日志归档；为防止调度器延迟，系统触发发生在 00:00-00:04 都按归档处理：把上一份 `qdreader_last_run_json` 按完成日期写入 `qdreader_archive_json`，再清空 `qdreader_last_result_json` / `qdreader_last_run_json`；不请求签名网关、不执行签到。 8. 每日 07:15 触发时只执行签到并反馈结果，不做归档。 9. 汇总每个账号签到结果。 diff --git a/plugins/qdreader/qdreader_sign_autman.js b/plugins/qdreader/qdreader_sign_autman.js index b6e8d89..d68a267 100644 --- a/plugins/qdreader/qdreader_sign_autman.js +++ b/plugins/qdreader/qdreader_sign_autman.js @@ -4,7 +4,7 @@ //[author: Hermes] //[service: BOSS] //[class: 工具类] -//[version: 2.5.3] +//[version: 2.5.4] //[platform: web,qq,wx,tg,tb,fs,we] //[public: false] //[price: 0] @@ -261,7 +261,7 @@ function contentText() { } function usage() { return [ - "启点读书签到插件 v2.5.3", + "启点读书签到插件 v2.5.4", "【一级菜单】", "账号管理：启点账号", "执行查询：启点查询", @@ -455,9 +455,10 @@ function latestArchiveForVisible(uids) { function cronMode() { var d = new Date() var h = d.getHours ? d.getHours() : 7 - // Autman cron can be delayed by scheduler/load; any system-triggered run during 00:00-00:59 is treated as archive-only. - // This avoids a dangerous midnight sign if the 00:00 archive cron is not executed exactly at minute 0. - if (h === 0) return "archive" + var m = d.getMinutes ? d.getMinutes() : 15 + // Autman cron can be delayed by scheduler/load; allow a 5-minute archive-only window after 00:00. + // Outside 00:00-00:04, system-triggered runs follow the business sign path. + if (h === 0 && m < 5) return "archive" return "sign" } function formatArchiveReply(ret) { diff --git a/tests/qdreader_plugin.test.mjs b/tests/qdreader_plugin.test.mjs index f0a9074..d86e123 100644 --- a/tests/qdreader_plugin.test.mjs +++ b/tests/qdreader_plugin.test.mjs @@ -266,7 +266,7 @@ test('midnight cron archives previous run log and clears daily logs without runn }); -test('delayed midnight cron still archives instead of signing', () => { +test('five-minute delayed midnight cron still archives instead of signing', () => { const previousRun = { runId: '20260518_cron', source: 'cron', @@ -282,12 +282,25 @@ test('delayed midnight cron still archives instead of signing', () => { qdreader_last_run_json: JSON.stringify(previousRun), qdreader_last_result_json: JSON.stringify({ '12345': JSON.stringify({ ok: true, msg: '昨日签到成功' }) }), }; - const r = runPlugin({ content: '', store, userId: 'cron', globals: { Date: fixedDateClass('2026-05-19T00:23:00.000+08:00') } }); + const r = runPlugin({ content: '', store, userId: 'cron', globals: { Date: fixedDateClass('2026-05-19T00:04:00.000+08:00') } }); assert.equal(r.calls.length, 0); assert.match(r.replies[0], /日志归档完成/); assert.equal(JSON.parse(r.store.qdreader_archive_json)['2026-05-18'].runId, '20260518_cron'); }); + +test('after five-minute midnight window cron follows sign path', () => { + const store = { + qdreader_cookie_json: JSON.stringify({ '12345': cookie }), + qdreader_user_bind_json: JSON.stringify({ '12345': 'user-a' }), + qdreader_last_run_json: JSON.stringify({ source: 'cron', finishedAt: '2026-05-18T07:15:02.000Z', total: 1, okCount: 1, failCount: 0, items: [{ uid: '12345', ok: true }] }), + }; + const r = runPlugin({ content: '', store, userId: 'cron', globals: { Date: fixedDateClass('2026-05-19T00:05:00.000+08:00') } }); + assert.equal(r.calls.length, 2); + assert.match(r.replies[0], /启点读书签到结果/); + assert.equal(r.store.qdreader_archive_json, undefined); +}); + test('query falls back to archived sign results after daily log was cleared', () => { const archiveRun = { source: 'cron',