test(fe): 补充守卫注释与架构闸门,防新增内联金额计算
- 生产接线处补注释,追溯到具体历史 bug (dcf649f2/20ea93d8/3c5f25a5/f873239a) - swapCalc.js 注明哪些函数已接入生产、哪些是纯交叉校验规格 - 新增 guard_arch.js 架构闸门:扫描提交新增行,拦截 Vue 组件内联金额计算 - pre-commit hook 串行运行 单元测试 + 架构闸门
This commit is contained in:
@@ -0,0 +1,122 @@
|
||||
#!/usr/bin/env node
|
||||
/**
|
||||
* guard_arch.js — 前端架构闸门(零依赖,纯 Node)
|
||||
* ============================================================================
|
||||
* 目的:防止在 Vue 组件方法里【新增】手写「金额 / 精度」计算(toFixed / _.round /
|
||||
* Math.round / × multiplier 缩放等),避免又长出一份「不可测的内联公式」。
|
||||
*
|
||||
* 规则:仅扫描【本次提交改动中新增/修改的行】(git diff 的 + 行),且位于
|
||||
* wwwroot/Scripts/app/ 并定义 new Vue(...) 的文件。对其中每一行「金额算术」,
|
||||
* 若该行没有路由到某个 *Calc 模块(含 SwapCalc.),则判为违规 → 退出码 1。
|
||||
*
|
||||
* 设计要点:
|
||||
* - 只查「新增/修改的行」,不查历史存量。这样不会阻断对存量文件(如仍含内联计算的
|
||||
* unwindSwapTrade.js)的正常改动,只拦「新写的内联金额公式」。
|
||||
* - 已外置计算逻辑的组件(incomeSwapTrade.js / swapTradeEdit.js)通过调用 SwapCalc.*
|
||||
* 保持合规;新代码必须把金额计算放进 *Calc 模块(参考 swapCalc.js)。
|
||||
* - 注释行会被剥离后再判断,避免注释里的样例文字误报。
|
||||
* - 无需 jest / npm install,和 _shim_run.js 同属零依赖守卫。
|
||||
*
|
||||
* 用法:node guard_arch.js (一般在 pre-commit / CI 中自动调用)
|
||||
* ============================================================================
|
||||
*/
|
||||
'use strict';
|
||||
|
||||
const { execSync } = require('child_process');
|
||||
const fs = require('fs');
|
||||
const path = require('path');
|
||||
|
||||
// fe-tests -> YLErpWeb -> zszq-trs(仓库根)
|
||||
const ROOT = path.resolve(__dirname, '..', '..');
|
||||
|
||||
// 金额 / 精度算术模式(命中即怀疑)
|
||||
const MONEY_PATTERNS = [
|
||||
/\.toFixed\s*\(/, // 显示/入库精度
|
||||
/_\.round\s*\(/, // lodash 四舍五入
|
||||
/Math\.round\s*\(/, // 原生四舍五入
|
||||
/\*\s*this\.multiplier/, // 价格 × 乘数缩放(dcf649f2 / 20ea93d8 类)
|
||||
/\*\s*thisObj\.multiplier/,
|
||||
];
|
||||
|
||||
// 合规标记:行内引用了某个 *Calc 模块(如 SwapCalc.),视为已外置
|
||||
const COMPLIANT = /Calc\./;
|
||||
|
||||
function tryCmd(cmd) {
|
||||
try {
|
||||
return execSync(cmd, { cwd: ROOT }).toString().trim();
|
||||
} catch (e) {
|
||||
return '';
|
||||
}
|
||||
}
|
||||
|
||||
function changedAppVueFiles() {
|
||||
const listCmd = 'git diff --cached --name-only --diff-filter=ACM -- "*.js"';
|
||||
let out = tryCmd(listCmd);
|
||||
if (!out) out = tryCmd('git diff --name-only --diff-filter=ACM -- "*.js"');
|
||||
if (!out) return [];
|
||||
return out.split('\n').filter((f) => {
|
||||
if (!/wwwroot[\\/]Scripts[\\/]app[\\/]/.test(f)) return false;
|
||||
const full = path.join(ROOT, f);
|
||||
if (!fs.existsSync(full)) return false;
|
||||
try {
|
||||
return /new\s+Vue\s*\(/.test(fs.readFileSync(full, 'utf8'));
|
||||
} catch (e) {
|
||||
return false;
|
||||
}
|
||||
});
|
||||
}
|
||||
|
||||
// 取文件在本次提交中【新增/修改】的行(git diff 的 + 行,排除 +++ 文件头)
|
||||
function addedLines(file) {
|
||||
const quoted = JSON.stringify(file);
|
||||
let out = tryCmd(`git diff --cached -U0 -- ${quoted}`);
|
||||
if (!out) out = tryCmd(`git diff -U0 -- ${quoted}`);
|
||||
if (!out) return [];
|
||||
return out.split('\n')
|
||||
.filter((l) => l.startsWith('+') && !l.startsWith('+++'))
|
||||
.map((l) => l.slice(1));
|
||||
}
|
||||
|
||||
function stripComment(line) {
|
||||
// 去掉 /* */ 块注释与 // 行内注释,避免注释里的样例文字误报
|
||||
return line.replace(/\/\*.*?\*\//g, '').replace(/\/\/.*$/, '').trim();
|
||||
}
|
||||
|
||||
function main() {
|
||||
const files = changedAppVueFiles();
|
||||
if (files.length) {
|
||||
console.log(`[guard_arch] 扫描 ${files.length} 个改动的 Vue 组件文件的新增/修改行`);
|
||||
}
|
||||
|
||||
let violations = 0;
|
||||
for (const file of files) {
|
||||
const added = addedLines(file);
|
||||
const found = [];
|
||||
added.forEach((rawLine) => {
|
||||
const code = stripComment(rawLine);
|
||||
if (!code) return;
|
||||
const hasMoney = MONEY_PATTERNS.some((p) => p.test(code));
|
||||
if (hasMoney && !COMPLIANT.test(code)) {
|
||||
found.push(` + ${rawLine.trim()}`);
|
||||
}
|
||||
});
|
||||
|
||||
if (found.length) {
|
||||
violations += found.length;
|
||||
console.log(`[guard_arch] 违规 ${file} 发现【新增】内联金额计算(应路由到 *Calc 模块):`);
|
||||
found.forEach((f) => console.log(f));
|
||||
}
|
||||
}
|
||||
|
||||
if (violations > 0) {
|
||||
console.log(
|
||||
`\n[guard_arch] 发现 ${violations} 处【新增】内联金额计算, 提交被阻断。` +
|
||||
`请把金额/精度计算抽到 *Calc 模块(参考 swapCalc.js)。`
|
||||
);
|
||||
process.exit(1);
|
||||
}
|
||||
console.log('[guard_arch] OK: 改动的 Vue 组件未新增内联金额计算。');
|
||||
process.exit(0);
|
||||
}
|
||||
|
||||
main();
|
||||
@@ -72,6 +72,8 @@ const vue = new Vue({
|
||||
$(this.$refs.incomeValueDatePicker.$el).val(MaxIncomeValueDate);
|
||||
return false;
|
||||
},
|
||||
// 守卫: 价格缩放因子(债券 multiplier=100 时界面为百分比态, 计算用相对价需 ÷100)
|
||||
// 计算已外置到 swapCalc.getPriceScale; 改动需同步 swapCalc.test.js
|
||||
getPriceScale() {
|
||||
return SwapCalc.getPriceScale(this.multiplier);
|
||||
},
|
||||
@@ -84,6 +86,8 @@ const vue = new Vue({
|
||||
this.initPosiGrossPrice = this.floatPosition.PosiGrossPrice;
|
||||
// 互换标的价格固定为期初净价,与平仓不同不需要用户填写
|
||||
// 期初净价入库为相对价(如1.02),需转换为界面百分比形态(102),与平仓页保持一致
|
||||
// 守卫: 期末全价必须用 PosiGrossPrice(全价) 且债券 ×100 转界面态
|
||||
// 对应历史 bug dcf649f2(错用净价) / 20ea93d8(×100 缩放丢失); 外置到 swapCalc.deriveTradingAmountAvg
|
||||
this.floatPosition.TradingAmountAvg = SwapCalc.deriveTradingAmountAvg(this.initPosiGrossPrice, this.multiplier);
|
||||
this.interestList = model.FlowEvents.filter((item) => {
|
||||
return item.InterestMode == 1 || item.InterestMode == 2 || item.InterestMode == 7 || item.InterestMode == 8 || item.InterestMode == 9;
|
||||
@@ -176,6 +180,8 @@ const vue = new Vue({
|
||||
//thisObj.floatPosition.MarkClosePnl = thisObj.deal.CloseNotionalValue * (thisObj.floatPosition.TradingAmountAvg * scale - thisObj.initPosiNetPrice) * floatRatio;
|
||||
thisObj.floatPosition.MarkClosePnl = thisObj.deal.CloseNotionalValue * (thisObj.floatPosition.TradingAmountAvg * scale - thisObj.initPosiGrossPrice) * floatRatio;
|
||||
thisObj.floatPosition.MarkClosePnl = otcformat.trading.StockEqvNotional(thisObj.floatPosition.MarkClosePnl);//MarkClosePnl 纯盯市不要计算交易费用和分红
|
||||
// 守卫: 浮动盈亏合计必须保留 2 位小数 → 对应历史 bug 3c5f25a5(原代码缺精度保留)
|
||||
// 数值由 swapCalc.calcFloatPnlSum 计算, 此处 .toFixed(2) 仅保留字符串类型以兼容下游
|
||||
thisObj.floatPosition.FloatPnlSum = SwapCalc.calcFloatPnlSum(thisObj.floatPosition.MarkClosePnl, TradingFee, TradingFeePending, DividendIn).toFixed(2);
|
||||
thisObj.calcCloseAmount();
|
||||
},
|
||||
|
||||
@@ -7,6 +7,11 @@
|
||||
* - Node: module.exports(UMD 包装),供 fe-tests/*.test.js 使用。
|
||||
* - 公式与 YLErpDAL/Helpers/FrontendCalcReference.cs 保持一致,是前后端同一份金标准。
|
||||
*
|
||||
* 生产接线状态(tested == used):incomeSwapTrade.js / swapTradeEdit.js 已调用
|
||||
* getPriceScale / deriveTradingAmountAvg / calcFloatPnlSum / calcStockEqvNotional
|
||||
* 这 4 个叶子函数(对应真实出过的 4 个 bug:20ea93d8 / dcf649f2 / 3c5f25a5 / f873239a)。
|
||||
* calcUnwind / calcIncome 仅用于 swapCalc.test.js 的前后端金标准交叉校验,未接入生产代码。
|
||||
*
|
||||
* 守卫的 bug(见 git 历史):
|
||||
* - 20ea93d8 / dcf649f2:deriveTradingAmountAvg 必须用 PosiGrossPrice(全价) 且债券 ×100
|
||||
* - 3c5f25a5:calcFloatPnlSum 必须 .toFixed(2)(保留 2 位小数)
|
||||
@@ -69,7 +74,12 @@
|
||||
return roundHalfAwayFromZero(step, 2);
|
||||
}
|
||||
|
||||
// ---- 组合函数:对齐 C# CalcUnwind / CalcIncome,作为前端与后端金标准的交叉校验 ----
|
||||
// ---- 组合函数:对齐 C# FrontendCalcReference.CalcUnwind / CalcIncome ----
|
||||
// 用途:作为「前端 JS 完整盈亏聚合公式」与「后端 C# 金标准」的交叉校验
|
||||
// (见 swapCalc.test.js 的 FC_001~FC_008 八个冻结场景)。
|
||||
// 注意:以下 calcUnwind / calcIncome **未接入生产代码**——生产 Vue 组件只调用上方
|
||||
// 4 个叶子函数。它们是冻结完整聚合逻辑的参考规格;若要让生产聚合逻辑也被自动守卫,
|
||||
// 需把 incomeSwapTrade.js / swapTradeEdit.js / unwindSwapTrade.js 的聚合计算也改调它们。
|
||||
|
||||
function parseOrZero(s) {
|
||||
return (s === undefined || s === null || s === '') ? 0 : Number(s);
|
||||
|
||||
@@ -357,6 +357,7 @@ const vue = new Vue({
|
||||
this.getSpotPrice(payItem.UnderlyingCode, this.trade.StartDate, payItem);
|
||||
}
|
||||
var national = payItem.PosiQuantity * payItem.ContractSize;
|
||||
// 守卫: 名义本金必须 round 到 2 位 → 对应历史 bug f873239a(缺 _.round); 外置到 swapCalc.calcStockEqvNotional
|
||||
var stockEqvNotional = SwapCalc.calcStockEqvNotional(payItem.PosiGrossPrice, national);//名义本金=期初价格*数量*乘数
|
||||
this.trade.StockEqvNotional = otcformat.trading.stockEqvNotional(stockEqvNotional);
|
||||
payItem.PosiNotionalValue = this.trade.StockEqvNotional;
|
||||
|
||||
Reference in New Issue
Block a user