Skip to content

fix(utility): 补充 ActorId、LNumber、FP 正确性修复回归测试 - #931

Merged
AlianBlank merged 8 commits into
mainfrom
feature/auto-server-source-worker-run-12738-20261031T0900
Nov 1, 2026
Merged

AlianBlank merged 8 commits into
mainfrom
feature/auto-server-source-worker-run-12738-20261031T0900

Conversation

@AlianBlank

Copy link
Copy Markdown
Collaborator

Fixes GFX-1174
Closes #920

新增服务器ID超界、ActorId 布局损坏、LNumber 除零、敏感词表损坏、
敏感词并发初始化与热更退役清理失败共六个键,中英文资源同步。

Linear: GFX-1174
生成侧拒绝超过 MaxServerId 的 serverId 与 ActorTypeNone,杜绝左移溢出
产生负 ID 或字段串位;解析侧对多 Actor 布局校验服务器ID合法域,损坏
布局在解析点抛异常而非返回错误答案。

Linear: GFX-1174
计数器以墙钟秒初始化,重启或跨进程可能重复;修正文档承诺,提示持久
唯一键改用雪花 ID。

Linear: GFX-1174
负向大被除数同样走 BigInteger 路径,避免移位溢出静默错值;除数为零
改抛 DivideByZeroException,与取模运算语义一致,并删除固化旧行为的
除零返回 0 用例。

Linear: GFX-1174
负数除零返回 MinValue 而非 MaxValue;FPVector2/3/4 归一化对零向量
直接返回零向量,ClampMagnitude 随之不再产出哨兵垃圾值。

Linear: GFX-1174
入链与出链在同一把锁内更新过渡标志,保证 InTransition 恒等于退役链
非空;退役表改用递增票据作键,消除 GetHashCode 碰撞导致的模块永不
卸载;清理异常记录 Error 日志。

Linear: GFX-1174
Init 先清空全部注册结构,删词热更后旧词即时失效;注册期间检测直接
放行并拒绝并发 Init;耗时统计统一 UtcNow;词表字段长度非法时显式
失败;检测入口改用与注册一致的 ASCII 折叠,修复 tr-TR 文化漏检。

Linear: GFX-1174
@linear-code

linear-code Bot commented Oct 31, 2026

Copy link
Copy Markdown

GFX-1174

@coderabbitai

coderabbitai Bot commented Oct 31, 2026

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 11 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: ec2d52ad-5eeb-49f1-9420-9d9ce61048c2
Commits

Reviewing files that changed from the base of the PR and between ca12a17 and e3ecd48.

Files selected for processing: 16
  • GameFrameX.Localization/Keys.Utility.cs
  • GameFrameX.Localization/Localization/Messages/Resources.resx
  • GameFrameX.Localization/Localization/Messages/Resources.zh-CN.resx
  • GameFrameX.Utility/ActorIdGenerator.cs
  • GameFrameX.Utility/Hotload/CollectibleAssemblyHost.cs
  • GameFrameX.Utility/IdGenerator.cs
  • GameFrameX.Utility/IllegalWordDetection.cs
  • GameFrameX.Utility/LNumber.cs
  • GameFrameX.Utility/Math/FP.cs
  • GameFrameX.Utility/Math/FPVector2.cs
  • GameFrameX.Utility/Math/FPVector3.cs
  • GameFrameX.Utility/Math/FPVector4.cs
  • Tests/GameFrameX.Tests/Utility/Hotload/CollectibleAssemblyHostRetireTicketTests.cs
  • Tests/GameFrameX.Tests/Utility/IllegalWordDetectionTests.cs
  • Tests/GameFrameX.Tests/Utility/LNumberTests.cs
  • Tests/GameFrameX.Tests/Utility/UtilityCorrectnessFixTests.cs
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

Share

@sonarqubecloud

Copy link
Copy Markdown

@AlianBlank
AlianBlank merged commit 601d280 into main Nov 1, 2026
4 checks passed
@AlianBlank
AlianBlank deleted the feature/auto-server-source-worker-run-12738-20261031T0900 branch November 1, 2026 07:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

1 participant