diff --git a/backend/src/jobs/warehouseLogisticsExceptionNotifyJob.js b/backend/src/jobs/warehouseLogisticsExceptionNotifyJob.js index 0663c70..07ce039 100644 --- a/backend/src/jobs/warehouseLogisticsExceptionNotifyJob.js +++ b/backend/src/jobs/warehouseLogisticsExceptionNotifyJob.js @@ -9,6 +9,9 @@ * ② 不发 .xlsx 附件 —— 群机器人 webhook 发不了文件,业务也确认群里看文本就够。 * 内容 = 汇总(发货/异常/异常率/分类型/分仓)+ 前 N 条明细 + 页面链接。 * N 由页面配置(默认 30,上限 50);要全部明细就点链接去页面导出 Excel。 + * 明细一行 = 仓|快递单号|快递|异常类型(**只有 4 段**)。 + * ⚠️ **不带发货时间**、**不带待处理小时数**(业务 2026-10-22 要求),时间类都到页面看。 + * ⚠️ 第二段是**快递单号**不是内部单号(同 10-22 二次要求;无单号才退回内部单号)。 * ③ 消息里**必须带「异常」字样**:钉钉群机器人若用「自定义关键词」安全设置, * 关键词一般是「异常」,缺了会被拒(errcode 310000)。 * ④ 一天只推一次:状态存 system_config,按「推送日(北京时间)」去重, @@ -74,30 +77,20 @@ function cleanCell(value, fallback = '') { return text || fallback; } -/** '2026-10-21 16:00:00' → '10-21 16:00';空值给「—」 */ -function shortTime(value) { - const text = cleanCell(value); - if (!text) return '—'; - const match = text.match(/^(\d{4})-(\d{2})-(\d{2})[ T](\d{2}):(\d{2})/); - if (!match) return text.slice(0, 16); - return `${match[2]}-${match[3]} ${match[4]}:${match[5]}`; -} - -function formatHours(value) { - const num = Number(value); - if (!Number.isFinite(num)) return '—'; - return `${Math.round(num * 10) / 10}h`; -} - -/** 明细一行:仓|内部单号|快递|异常类型|发货时间|待处理 */ +/** + * 明细一行:仓|快递单号|快递|异常类型 + * ⚠️ 业务明确要求明细**不带发货时间**(10-22 反馈),别再加 sendTime 回去 —— + * 仓库看单子只需知道「哪个单、什么异常」,时间类信息一律到页面上看。 + * ⚠️ 第二段用**快递单号**(`trackingNo` = 上游 `logistic_id`),不是内部单号 `oId`(10-22 二次要求)。 + * 「未揽收 / 无物流单」这类包裹本来就没有快递单号 → 退回内部单号,否则群里这条无法定位。 + * ⚠️ 末尾的「待处理小时数」(`pendingHours`)也已按 10-22 三次要求去掉,别再加回来。 + */ function buildDetailLine(row, index) { const parts = [ cleanCell(row?.warehouseName, '未知仓'), - cleanCell(row?.oId, '无单号'), + cleanCell(row?.trackingNo || row?.oId, '无单号'), cleanCell(row?.carrier, '未知快递'), cleanCell(row?.exceptionLabel, '异常'), - shortTime(row?.sendTime), - formatHours(row?.pendingHours), ]; return `${index}. ${parts.join('|')}`; } @@ -428,6 +421,5 @@ module.exports = { cleanCell, pushWarehouseLogisticsExceptionNotify, runWarehouseLogisticsExceptionNotifyJob, - shortTime, startWarehouseLogisticsExceptionNotifyScheduler, }; diff --git a/backend/test/warehouseLogisticsExceptionNotify.test.js b/backend/test/warehouseLogisticsExceptionNotify.test.js index 1864c31..50c7995 100644 --- a/backend/test/warehouseLogisticsExceptionNotify.test.js +++ b/backend/test/warehouseLogisticsExceptionNotify.test.js @@ -118,10 +118,11 @@ test('群消息正文:抬头带「异常」关键词、数字与分类型、 assert.equal(built.truncated, false); }); -test('明细行:仓|内部单号|快递|异常类型|发货时间|待处理,字段里换行要压平', () => { +test('明细行:仓|快递单号|快递|异常类型(4 段,不带发货时间和待处理时长),字段里换行要压平', () => { const line = job.buildDetailLine({ warehouseName: '崔桥工厂仓', oId: '9845001', + trackingNo: 'JDVA00123456789', carrier: '京东快递', exceptionLabel: '已揽收未中转', sendTime: '2026-10-21 16:00:00', @@ -129,14 +130,27 @@ test('明细行:仓|内部单号|快递|异常类型|发货时间| lastLogisticInfo: '第一行\n第二行', }, 3); - assert.equal(line, '3. 崔桥工厂仓|9845001|京东快递|已揽收未中转|10-21 16:00|17.3h'); + assert.equal(line, '3. 崔桥工厂仓|JDVA00123456789|京东快递|已揽收未中转'); + // 业务要求:第二段是快递单号,不是内部单号 + assert.ok(!line.includes('9845001'), '明细行不该再显示内部单号'); + // 业务要求:明细不带发货时间(哪怕传了 sendTime 也不能出现在消息里) + assert.ok(!line.includes('10-21'), '明细行不该再带发货时间'); + assert.ok(!line.includes('16:00'), '明细行不该再带发货时间'); + // 业务要求:末尾的待处理小时数也去掉(哪怕传了 pendingHours 也不能出现在消息里) + assert.ok(!line.includes('17.3h'), '明细行不该再带待处理时长'); + assert.equal(line.split('|').length, 4, '明细必须正好 4 段'); // 物流描述里的换行不能漏进消息(会把一条明细劈成两行,看着像两条异常) assert.equal(job.cleanCell('甲\n乙\t丙 丁'), '甲 乙 丙 丁'); + // 未揽收/无物流单没有快递单号 → 退回内部单号(否则群里这条无法定位) + assert.equal( + job.buildDetailLine({ oId: '9845002', carrier: '中通' }, 1), + '1. 未知仓|9845002|中通|异常' + ); // 空值兜底:不能让消息里出现「undefined|null」 assert.equal( job.buildDetailLine({}, 1), - '1. 未知仓|无单号|未知快递|异常|—|—' + '1. 未知仓|无单号|未知快递|异常' ); });