二七王:前端删掉 9 处阶段/控制权推导,一律读服务端包字段
I-3 前端不再持有对局状态机(配合服务端上一提交的下发面补齐): - 删掉 5 处按 rpc 名硬编码的 aset.step(Deal=1 / Call=2 / Main=3 / Bury=5 / Result=6), 一律改 _apply 读包里的 step - 删掉 4 处 turn.seat = aset.banker(CallHandler / MainHandler / ResyncHandler×2), 增量路径读 nextseat、重连路径读分组里的 seat;同一条规则不再有 4 个写入处 I-2 ResyncHandler._applyBalance 由只接 aset 扩为接 aset/bottom/account, 结算面板开着时重连/硬刷新不再丢抠底明细与末局大结算。 一致性测试 EXCLUDE 随之从 4 条收缩为 2 条(去掉 result.bottom / result.account, 只留一次性事件 table.ancard3s 与归下一轮的 result.chupai),未新增任何豁免条目。 夹具按新包结构重跑(种子不变,牌局不变):diff 纯为 67 行新增字段、0 行删除; test_fixture 补上 step/nextseat/Balance 同源断言(68→82 checks)。 export_packets.js 里关于 playproc/seatlist 活引用的注释已过时,更正为仍存在的 Balance.readystate 与三份结算快照。 各 handler 单测补齐新字段并新增守卫用例:刻意构造 nextseat != banker、 step 为非预期值的包,使「退回硬编码/banker 反推」的写法必然转红(11/11 已验证)。 Co-Authored-By: Claude Opus 5 (1M context) <noreply@anthropic.com>
This commit is contained in:
@@ -63,6 +63,44 @@ Object.keys(step5BySeat).forEach(seat => {
|
||||
}
|
||||
});
|
||||
|
||||
// I-3:推进阶段的下发包必须自带 step,且与该时刻的 deskinfo.step 属于同一套取值。
|
||||
// 前端不得按 rpc 名硬编码阶段(前端红线「数据驱动、前端无对局状态机」)。
|
||||
const STEP_OF = { fapai: 1, jiaofen: 1, shangzhuang: 2, xuanzhu: 3, maipai: 5, jiesuan: 6 };
|
||||
const noStep = [], badStep = [];
|
||||
[0, 1, 2].forEach(s => fx.seats[s].forEach(p => {
|
||||
if (!STEP_OF.hasOwnProperty(p.rpc)) { return; }
|
||||
if (!p.data.hasOwnProperty('step')) { noStep.push(s + ':' + p.rpc); return; }
|
||||
if (p.data.step !== STEP_OF[p.rpc]) { badStep.push(s + ':' + p.rpc + '=' + p.data.step); }
|
||||
}));
|
||||
t.eq('推进阶段的下发包都带 step', noStep, []);
|
||||
t.eq('各包的 step 就是该时点的真实阶段', badStep, []);
|
||||
|
||||
// I-3:选主/埋牌阶段的「轮到谁」必须由服务端显式给出(增量 nextseat / 重连分组的 seat)
|
||||
const noNext = [];
|
||||
[0, 1, 2].forEach(s => fx.seats[s].forEach(p => {
|
||||
if ((p.rpc === 'shangzhuang' || p.rpc === 'xuanzhu') && !p.data.hasOwnProperty('nextseat')) {
|
||||
noNext.push(s + ':' + p.rpc);
|
||||
}
|
||||
}));
|
||||
t.eq('shangzhuang/xuanzhu 都带 nextseat(控制权)', noNext, []);
|
||||
fx.deskinfo.filter(d => d.step === 2).forEach(d => {
|
||||
t.eq('重连 ChooseMain 带控制权 seat (seat' + d.seat + ')', typeof d.info.ChooseMain.seat, 'number');
|
||||
});
|
||||
fx.deskinfo.filter(d => d.step === 3).forEach(d => {
|
||||
t.eq('重连 BuryCards 带控制权 seat (seat' + d.seat + ')', typeof d.info.BuryCards.seat, 'number');
|
||||
});
|
||||
|
||||
// I-2:结算阶段的重连包必须与 jiesuan 推送给同一份结算数据(抠底明细 bottom 恒有;
|
||||
// account 只在末局,本夹具是第 1/6 局,故两条路径都没有)
|
||||
const jiesuanPk = fx.seats[0].filter(p => p.rpc === 'jiesuan')[0];
|
||||
t.eq('夹具的 jiesuan 带抠底明细 bottom', !!(jiesuanPk && jiesuanPk.data.bottom), true);
|
||||
fx.deskinfo.filter(d => d.step === 6).forEach(d => {
|
||||
t.eq('重连 Balance.bottom == jiesuan.bottom (seat' + d.seat + ')',
|
||||
JSON.stringify(d.info.Balance.bottom), JSON.stringify(jiesuanPk.data.bottom));
|
||||
t.eq('非末局:两条路径都不带 account (seat' + d.seat + ')',
|
||||
[d.info.Balance.hasOwnProperty('account'), jiesuanPk.data.hasOwnProperty('account')], [false, false]);
|
||||
});
|
||||
|
||||
// 每个包都有 success(协议 §0.1:每个下发包的 data 必带 success)
|
||||
const noSuccess = [];
|
||||
[0,1,2].forEach(s => fx.seats[s].forEach(p => {
|
||||
|
||||
Reference in New Issue
Block a user