Migrate features and fixes from dev/26.1 to port/1.21.1

[Gu-ZT]

No description provided.

[Gugle2308]

⚠️ Dangerous command requires approval:

curl -sL -H "Accept: application/vnd.github+json" "https://api.github.com/repos/Anvil-Dev/AnvilLib/pulls/97" | python3 -c "
import json,sys
d=json.load(sys.stdin)
print('title:', d.get('title'))
print...

Reason: Security scan — [HIGH] Pipe to interpreter: curl | python3: Command pipes output from 'curl' directly to interpreter 'python3'. Downloaded content will be executed without inspection.
Safer: tirith run https://api.github.com/repos/Anvil-Dev/AnvilLib/pulls/97 — or: vet https://api.github.com/repos/Anvil-Dev/AnvilLib/pulls/97 (https://getvet.sh); script execution via -e/-c flag

Reply /approve to execute, /approve session to approve this pattern for the session, /approve always to approve permanently, or /deny to cancel.

[Gugle2308]

🌿 Roseau API Breaking Change Report

Module	Status	Breaking Changes
codec	⚪ Skipped	—
collision	⚪ Skipped	—
config	⚪ Skipped	—
integration	⚪ Skipped	—
moveable-entity-block	⚪ Skipped	—
network	⚪ Skipped	—
rendering	⚪ Skipped	—
space-select	⚪ Skipped	—
font	⚪ Skipped	—
util	⚪ Skipped	—
explosion	⚪ Skipped	—
rpc	⚪ Skipped	—
multiblock	⚪ Skipped	—
recipe	⚪ Skipped	—
registrum	⚪ Skipped	—
sync	⚪ Skipped	—
wheel	⚪ Skipped	—
main	⚪ Skipped	—

Full CSVs: see the Artifacts section of this workflow run.

[Gugle2308]

⚠️ Dangerous command requires approval:

cd /tmp && curl -sL "https://api.github.com/repos/Anvil-Dev/AnvilLib/pulls/97" 2>/dev/null | python3 -c "import json,sys; d=json.load(sys.stdin); print('base:', d['base']['ref'], d['base']['sha']); pr...

Reason: Security scan — [HIGH] Pipe to interpreter: curl | python3: Command pipes output from 'curl' directly to interpreter 'python3'. Downloaded content will be executed without inspection.
Safer: tirith run https://api.github.com/repos/Anvil-Dev/AnvilLib/pulls/97 — or: vet https://api.github.com/repos/Anvil-Dev/AnvilLib/pulls/97 (https://getvet.sh); script execution via -e/-c flag

Reply /approve to execute, /approve session to approve this pattern for the session, /approve always to approve permanently, or /deny to cancel.

[Gugle2308]

🌿 Roseau API Breaking Change Report

Module	Status	Breaking Changes
codec	⚪ Skipped	—
collision	⚪ Skipped	—
config	⚪ Skipped	—
integration	⚪ Skipped	—
moveable-entity-block	⚪ Skipped	—
network	⚪ Skipped	—
rendering	⚪ Skipped	—
space-select	⚪ Skipped	—
font	⚪ Skipped	—
util	⚪ Skipped	—
explosion	⚪ Skipped	—
rpc	⚪ Skipped	—
multiblock	⚪ Skipped	—
recipe	⚪ Skipped	—
registrum	⚪ Skipped	—
sync	⚪ Skipped	—
wheel	⚪ Skipped	—
main	⚪ Skipped	—

Full CSVs: see the Artifacts section of this workflow run.

[Gugle2308]

⚠️ Dangerous command requires approval:

cd /tmp && echo "=== version manifest for 1.21.1 and 26.1.2 ===" && curl -sL "https://piston-meta.mojang.com/mc/game/version_manifest_v2.json" -o /tmp/vm.json --max-time 30 && python3 -c "
import json...

Reason: script execution via -e/-c flag

Reply /approve to execute, /approve session to approve this pattern for the session, /approve always to approve permanently, or /deny to cancel.

[Gugle2308]

信息收集完毕。综合所有分析，输出审查意见：

代码审查摘要 — PR #97

操作: converted_to_draft（草稿状态 — 仅报严重问题，comment 级别）
范围: 212 个文件（172 新增 / 0 删除）/ 20,066 行 diff
内容: dev/26.1 → port/1.21.1 回移植（rpc / sync / explosion / collision / space-select / registrum 增强 / wheel 环形扇区效果等，#P1–#P19 迁移计划）

---

🔴 关键

• settings.gradle — include 了两个不存在的模块目录 module.rendering 和 module.font（第 3、4 行新增）
  已用 git tree 双重验证：fork 分支（544c9eb，962 个 path）和 GitHub API 根目录列表中均无 module.rendering / module.font 目录，diff 中也没有它们的任何文件（迁移计划 #P6/#P7 尚未落地）。Gradle 配置阶段会对 include 的每个项目检查 projectDir 存在性，报 Project directory '.../module.rendering' does not exist → 任何 gradlew 命令（build / runData / 导入 IDE）都会在配置阶段失败。
  建议：若这两个模块不在本 PR 范围内，先移除这两行 include（及对应的 project(':module.rendering').name / project(':module.font').name），待 #P6/#P7 落地时再补回；否则当前分支无法构建。

---

⚠️ 警告

• @NullMarked / jspecify 与迁移计划文档自相矛盾 — 文档 docs/migration-plan-port-1.21.1.md §2.4 明确声明「@NullMarked（jspecify）→ 保持 port 侧 @MethodsReturnNonnullByDefault + javax.annotation.ParametersAreNonnullByDefault，不迁移」，但实际代码中打入了 29 处 @NullMarked + 55 处 jspecify import（涉及 rpc / sync / explosion / collision / registrum / space-select 模块）。jspecify 1.0.0 已作为 compileOnly 依赖声明，编译不受影响，但与文档约定和 port 侧既有风格不一致。需二选一：更新文档声明（允许 jspecify），或按文档约定改写为 port 侧注解。

• SingleItemRecipeAccessor（explosion） — 26.1 源是 @Accessor ItemStackTemplate getResult()，移植后改为 @Accessor ItemStack getResult()。1.21.1 的 SingleItemRecipe 已含 public ItemStack getResult() 方法（Recipe 体系自带），accessor 可能与既有方法语义重叠/冗余，建议确认是否有必要保留（若保留，确认 Mixin 注入不会与既有 getter 冲突）。

---

🟢 看起来不错

• 26.1 API 泄漏扫描全部干净：Java 源码中 Identifier / RecipeMap / ItemStackTemplate / ValueInput / ValueOutput / RenderPipeline / GpuDevice / FMLLoader.getCurrent() / ResourceKey::identifier 泄漏均为 0（文档中的命中均为说明文字）✅
• sync processor CoreMod 重写正确：SyncClassProcessor 用 ICoreMod + ITransformer<ClassNode> 替代 26.1 的 transformation SPI，services 文件注册正确 ✅
• 渲染适配正确：DistrictRenderer 用 LevelRenderer.renderVoxelShape（1.21.1 API）、WheelScreen/LibShaders 用 RegisterShadersEvent + ShaderInstance + POSITION_COLOR、shader json/fsh 资源路径与 AnvilLibWheel.of()（anvillib 命名空间）匹配 ✅
• DynamicMultiblockManager 移植完整：异步快照检测（主线程建快照 → 线程池测试 → 主线程回调）、isLoaded 守卫、pendingChecks 防重、SavedData 序列化对称 ✅
• RpcPendingCalls 超时/清理逻辑：ConcurrentHashMap + tick 超时 + clear() 取消，线程安全 ✅
• 语言文件同步：en_us.json（generated）与 zh_cn.json 7 个 key 完全一致 ✅
• 注册完整性：5 个新模块的 mod_id 各不相同、jarJar project 依赖路径与 settings.gradle 重命名一致、jspecify 依赖在各 build.gradle 齐全 ✅

---

📋 迁移计划落地核对（#P1–#P19）

计划项	状态	说明
#P1 rpc	✅	30 文件完整
#P2 sync + processor	✅	39 文件，CoreMod 重写
#P3 explosion	✅	16 文件
#P4 collision	✅	7 文件
#P5 space-select	✅	23 文件
#P6 font	❌	未落地（settings.gradle 已 include）
#P7 rendering	❌	未落地（settings.gradle 已 include）
#P8 renderdoc-loader	✅	4 文件
#P9 config	✅	6 文件
#P10 integration	✅	3 文件
#P11 network	✅	3 文件
#P12 util	✅	7 文件 + 测试
#P13 registrum	✅	46 文件
#P14 registrum datagen	✅	RecipeProvider 改动
#P15 multiblock	✅	7 文件
#P16 recipe	✅	7 文件
#P17 wheel	✅	8 文件
#P18 test	⚠️	未见集成测试落地
#P19 构建体系	✅	roseau.yaml + roseau.gradle

---

结论: COMMENT（草稿 PR，需修复构建阻断后转 ready） — 核心问题是 settings.gradle 引用了未包含的 module.rendering / module.font，当前分支无法通过 Gradle 配置阶段；建议先移除这两个 include（或补齐模块），并统一 @NullMarked 与迁移文档的约定。其余模块移植质量高，26.1 API 泄漏为零，可待构建验证后进入正式审查。

---

由 Hermes Agent 审查

[Gugle2308]

⚠️ Dangerous command requires approval:

cd /tmp && python3 -c "
import json
vm = json.load(open('/tmp/vm.json'))
for v in ['1.21.1','26.1.2']:
    for ver in vm['versions']:
        if ver['id'] == v:
            print(v, ver['url'])
" 2>/d...

Reason: script execution via -e/-c flag

Reply /approve to execute, /approve session to approve this pattern for the session, /approve always to approve permanently, or /deny to cancel.

[Gugle2308]

💾 Self-improvement review: Patched SKILL.md in skill 'minecraft-mod-pr-review' (1 replacement).

[Gugle2308]

I now have a complete picture. Let me compile the final review.

---

代码审查摘要 — PR #97

操作: opened
范围: 212 个文件（171 Java, 170 新增, 0 删除）/ 20,066 行 diff
性质: dev/26.1 → port/1.21.1 回移植（目标平台 MC 1.21.1 / NeoForge 21.1.226 / Java 21），覆盖迁移计划 #P1–#P19 中的大部分项

迁移健康度（1.21.1 反向适配检查）

指标	结果	状态
Identifier（26.1 新 API）泄漏到 Java 代码	0	✅ 全部保持 ResourceLocation
Java 22+ 语法（lambda _、accessFlags）	0	✅ 已改写为 Java 21 兼容
neoforgespi 使用	仅 ICoreMod/ModFileScanData/IModFile（1.21.1 存在）	✅ 合法
jspecify @NullMarked	46 个文件使用 + 各模块新增 compileOnly 依赖	⚠️ 与计划文档 §2.4 相悖（见下）
渲染层 API	Tesselator/ShaderInstance/RegisterShadersEvent/LevelRenderer.renderVoxelShape	✅ 1.21.1 等价物
翻译同步（explosion en_us ↔ zh_cn）	7/7 key 一致	✅
EOF 缺换行	17 个文件（build.gradle / mixins.json / roseau.gradle 等）	⚠️ 低优先级

🔴 关键（必须修复后合并）

• settings.gradle — 注册了 PR 中不存在的模块 module.rendering 和 module.font，整个仓库的 Gradle 构建直接失败
  已用 git ls-tree 核实 PR head 树中不存在 module.rendering/ 与 module.font/ 目录（diff 中相关文件数为 0），但 settings.gradle 却加入了 include 'module.rendering' + include 'module.font' 及其 project 改名。Gradle 在配置阶段会报 Project directory .../module.rendering does not exist，连现有模块（如 :module.config:build）都无法构建。这两个模块对应计划 #P6/#P7（高难度、未随本 PR 交付），settings 注册属于提前注册。修复：删除这两组 include/rename 行，待 #P6/#P7 落地时再加回。

⚠️ 警告

• jspecify @NullMarked 与随 PR 提交的迁移计划文档 §2.4 直接矛盾 — 计划明确要求「@NullMarked（jspecify）→ 保持 port 侧 @MethodsReturnNonnullByDefault + javax.annotation.ParametersAreNonnullByDefault」，但实际代码在 46 个文件中保留了 jspecify，并为每个新模块加了 compileOnly "org.jspecify:jspecify:1.0.0"。注意：下游 AnvilCraft 的 AGENTS.md 恰恰强制要求只用 JSpecify，因此保留 jspecify 可能是有意为之且更正确——但计划文档必须同步更新，否则文档与代码互相矛盾。另外这造成仓库内注解风格分裂：旧 config 代码用 javax.annotation.Nullable，新模块用 jspecify。
• module.multiblock DynamicMultiblockManager — roseau.gradle 被应用到所有子项目且配置期执行网络 I/O — 根 build.gradle projectsEvaluated { subprojects { apply roseau.gradle } } 使每次 gradlew 调用、每个子项目（21 个）都在配置阶段访问 maven-metadata.xml（5s 超时 × 2 仓库）。离线/CI 沙箱环境下每次构建可能卡顿数十秒甚至失败；建议改为惰性解析（provider）或仅 roseauCheck 任务触发。
• module.sync SyncManager.getValue() 是空实现 — SyncProxy.getValue() 调用 this.manager.getValue(...)（方法体为空）后直接返回本地 this.value；「读」路径实际不向 manager 拉取任何东西。若设计是纯推送式可接受，但空方法 + 调用点的存在容易误导，建议确认意图（26.1 原版此处应有逻辑）或删除空调用。
• module.multiblock 未加载区块的快照条目 test() 恒为 true — MultiblockCheckSnapshot.Entry.test() 在 blockState == null 时返回 true，配合 buildSnapshot 的未加载区块复用逻辑，意味着组件方块位于未加载区块时多方块结构不会解体（即使方块已被破坏，也要等区块加载后的下一轮检测才生效）。这是计划 #P15「懒解析/快照复用」的刻意取舍（避免同步加载区块死锁），但属于行为变化，建议在文档中明确此语义。
• module.registrum OneTimeEventReceiver — if (pairs == null) return; 静默丢弃监听器注册 — 原代码在 contains+put 之后 get 不可能为 null，新加的空判断若触发会静默不注册监听器（原为 NPE 显式暴露）。低风险但建议改抛异常或去掉。

💡 建议

• module.sync processor 被 jarJar 进主模块 — processor jar 无 mods.toml，仅靠 META-INF/services/...ICoreMod 注册。开发环境（project 依赖在 classpath）可工作；生产 jarJar 嵌套后 FML 能否在 CoreMod 发现阶段读到该 services 文件建议做一次运行时验证（这是 #P2 最关键的运行时风险点）。
• module.collision CollisionTest 放在 main 源集 — static void main() 无 String[] 参数，非标准 JVM 入口，文档注明是手动自测入口；建议挪到 test 源集并给标准签名。
• 17 处 EOF 缺换行（多为新 build.gradle/mixins.json），顺手补上。
• module.explosion mixins.json 的 compatibilityLevel 是 JAVA_21，其余新模块是 JAVA_8，风格不一致，统一即可。

🟢 看起来不错

• CoreMod 重写（#P2 核心风险项）质量高 — SyncClassProcessor 用 ICoreMod + ITransformer<ClassNode> 正确替代 26.1 的 transformation SPI，META-INF/services 注册正确，时序注释（LoadingModList 就绪后才建索引）清晰合理。
• PacketData 泛型检查修复（#P11）正确 — 原 declaringClass.isAssignableFrom(Type.class) 方向反了，修复为 Type.class.isAssignableFrom(fieldType) 并新增 isMatchingTypeArgument 精确匹配泛型实参。
• MultiblockState 懒解析 + 不可变键 — 构造器即初始化快照（无 NPE），Map<BlockPos, MultiblockState> 全部用 pos.immutable()，getDefinition(registryAccess) 惰性缓存；DynamicMultiblockEvent.Form/Unform 取消语义在服务端 updateFormed 与客户端两个 packet handler 中一致（取消则回滚 formed 并跳过广播）。
• #P16 修复到位 — SpawnItem 零数量守卫（原 copyWithCount(0) 会产出空栈）、SetBlock nbt 默认值 new CompoundTag()。
• RPC 模块工程性良好 — RpcPendingCalls 用 ConcurrentHashMap + 100 tick 超时清理，ServerStoppedEvent/客户端断开时 clear，静态缓存均并发安全。
• 翻译 key 同步、VillagerProfessionBuilder 按 1.21.1 6 元 record 适配、applyData → applyClientData/applyServerData 拆分（破坏性 API 已在计划中声明）均符合预期。

📋 迁移计划对照表（#P1–#P19）

计划项	状态	证据
#P1 module.rpc	✅	30 文件，网络/配置/客户端注册齐全
#P2 module.sync + processor	✅	39 文件，CoreMod 重写正确（运行时 jarJar 待验证）
#P3 module.explosion	✅	16 文件，mixins.json 已注册 Accessor
#P4 module.collision	✅	纯数学，零 MC 依赖
#P5 module.space-select	✅	渲染层已改 LevelRenderer.renderVoxelShape
#P6 module.font	❌	未交付，但 settings.gradle 已注册（构建阻断）
#P7 module.rendering	❌	未交付，但 settings.gradle 已注册（构建阻断）
#P8 renderdoc-loader	✅	Agent + build.gradle
#P9 config group/TranslatableEnum/点分隔	✅	Config/ConfigManager/ConfigRecord/FormattingUtil 联动一致
#P10 integration 拆分	✅	CLIENT_DATA/SERVER_DATA + meter 递增
#P11 network 三处	✅	PacketData 泛型修复 / Included 方法 / PacketProtocol public
#P12 util 演进	✅	OutlineUtil + HolderGetter 重构（连带 recipe 4 调用方）
#P13 registrum 14 种 builder	✅	AbstractRegistrum +438 行，VillagerProfession 6 元 record
#P14 registrum datagen	✅	RecipeProvider public 化 + dataMap(provider)
#P15 multiblock 四项	✅	事件 + 快照复用 + BlockPos 键化 + 懒解析
#P16 recipe 两修复	✅	已核对
#P17 wheel 环形扇区	✅	fsh + json + LibShaders + Widget
#P18 module.test	⚠️	仅 module.util 的 ShapeUtilJoinTimingTest 落地
#P19 roseauCheck	⚠️	文件齐备，但全子项目配置期网络 I/O 需惰性化

结论: REQUEST_CHANGES — settings.gradle 引用了两个不存在的模块目录，导致仓库任何 Gradle 构建（含现有模块）在配置阶段直接失败，这是必须修复的阻断项；其余为文档一致性（jspecify vs 计划 §2.4）、roseau 配置期网络 I/O、SyncManager.getValue 空实现等警告级问题。功能迁移本身质量很高，修复后值得合并。

🧪 测试建议

被测目标	推荐测试场景	优先级
SyncClassProcessor（CoreMod）	生产 jar（jarJar 嵌套）下 @Sync 字段是否被注入	🔴
DynamicMultiblockManager.buildSnapshot	未加载区块 + 快照复用 + 首次检测（old 为空 Map）三条路径	🟡
PacketData.isMatchingTypeArgument	泛型实参为通配符/非 Class/类型参数时的 false 分支	🟢
RpcPendingCalls.tick()	100 tick 超时边界（99/100/101）与已移除条目	🟢
ConfigData.enumValueAdd	TranslatableEnum + SerializedName 覆盖	🟢

---

由 Hermes Agent 审查

[Gugle2308]

⚠️ Dangerous command requires approval:

curl -sL "https://api.github.com/repos/Anvil-Dev/AnvilLib/pulls/97" | python3 -c "import json,sys; d=json.load(sys.stdin); print('title:', d['title']); print('state:', d['state']); print('base:', d['b...

Reason: Security scan — [HIGH] Pipe to interpreter: curl | python3: Command pipes output from 'curl' directly to interpreter 'python3'. Downloaded content will be executed without inspection.
Safer: tirith run https://api.github.com/repos/Anvil-Dev/AnvilLib/pulls/97 — or: vet https://api.github.com/repos/Anvil-Dev/AnvilLib/pulls/97 (https://getvet.sh); script execution via -e/-c flag

Reply /approve to execute, /approve session to approve this pattern for the session, /approve always to approve permanently, or /deny to cancel.

[Gugle2308]

所有检查完成。@mod dist 隔离正确（font/rendering/rpc-client/space-select-client/sync-client 均为 Dist.CLIENT，同 jar 双 @mod 标准模式）。输出审查：

---

代码审查摘要 — PR #97

操作: synchronize（反向移植复查：dev/26.1 → port/1.21.1）
范围: 419 个文件（337 Java，319 新增，0 删除）/ 35,213 行 diff
分支: base port/1.21.1@05235aa ← head dev/1.21.1@fab551b；state: open，mergeable_state: unstable（CI 未全绿）

对比上次 synchronize（418 文件/33.8K 行），本次 diff 再次增大（419 文件/35.2K 行）。已按复查要求重跑全部扫描（headers 重建、反向 API、ghost、mixin 交叉、接线验证），未沿用上次计数。

✅ 反向 API 健康度（重扫干净）

指标（仅 Java）	计数	状态
Identifier / org.jspecify / ResourceHandler / Capabilities.Fluid	0	✅ 干净（全部命中都在 docs 规划文档）
builtInRegistryHolder	10	✅ 合法 1.21.1 用法（勿误报）
TODO/FIXME / 硬编码凭据	0 / 0	✅
Ghost 文件	0	✅
EOF 缺换行	31	✅ 低于异常阈值（多为 package-info/mixins.json/lang）

✅ 结构性复查（接线全部到位）

• Mixin 交叉检查：8 个 mixins.json 中非空配置（explosion/font/rendering×2）的 6 个条目全部与 diff 中类文件一一对应；collision/rpc/space-select/sync 的空数组为模板残留，无害。
• jarJar 链：explosion→config、font→rendering、rpc→network、space-select→network、sync→processor+codec+network+util，全部 jarJar(api(...latest.release)) + jarJar(implementation project(...)) 成对；module.main 汇总全部 9 个模块 ✅
• @mod dist 隔离：font/rendering/rpc-client/space-select-client/sync-client 均为 Dist.CLIENT，同 jar 双 @mod（主+Client）标准模式 ✅
• CI workflow 引用：build_and_test.yml / publish_maven_central.yml 在 base SHA 和 head SHA 均存在（200）。裸分支名 raw 端点 404 是斜杠分支已知误报，按 SHA 验证通过 ✅
• 声称清单：migration-todo #P1–#P20 全部 ✅，逐项关键词→文件计数映射吻合（rpc 25 / sync 31+processor / rendering 79 / registrum 44 / font 25 / space-select 18 / recipe 26 / test 16 等）

🔴 关键（需关注后合并）

• module.sync — ICoreMod 时序风险依旧无防护：SyncClassProcessor.getTransformers() → SyncTransformer.targets() → SyncTargetIndex.allTargets() / LazySyncTargetIndex.allTargets() → 无 try/catch 的静态块直接调 LoadingModList.get().getModFiles()。Javadoc 声称 getTransformers() 在 FMLServiceProvider.launch()（mod 扫描后）调用故安全，但：① 没有防御性改写兜底；② todo 中 module.sync 验收仅 compileJava，无 runClient 首次启动实测。两种失败形态：LoadingModList.get() 未就绪 → 启动崩溃；时序恰好允许 → targets() 拿到空目标集 → @Sync/@lazysync 静默失效 → 运行时 NPE。建议：runClient 首次启动验证 + 静态块 try/catch 容错 + 索引构建延后到 FML 事件时机显式刷新。

⚠️ 警告

• roseau.gradle 急切网络访问（此前已标记，未修复）：roseauBaselineVersion = findProperty(...) ?: { ...HTTP... }() 闭包在配置期急切执行；配合 root build.gradle gradle.projectsEvaluated { subprojects { apply from ... } }，20+ 子项目每次构建（含 gradlew help）都会向 2 个 Maven 仓库发起请求（5s connect + 5s read 超时 ×2）。离线/受限网络下每次构建阻塞数分钟。建议改 lazy Provider 或仅任务执行时解析。
• #P19 roseauCheck 验收缺失：todo 状态为「已完成（运行待补验）」，与 ICoreMod 一样属"无运行验证"信号。
• mergeable_state = unstable：CI 存在未通过/未完成检查，建议 CI 全绿后再合并。

💡 建议

• module.collision/CollisionTest.java — 类名带 Test 却位于 src/main，且含大量 System.out.println 基准/断言输出。作为 standalone 调试 harness 可接受，但建议移入 src/test 或加 @ApiStatus.Internal，避免随 mod jar 发布。
• EOF 缺换行（31 处，非阻塞）：少数 Java 类（UnlimitedItemStack、ItemPredicate、SyncDirection、HasItem 等）缺末尾换行，属风格问题，可顺手补齐。

🟢 看起来不错

• 反向移植质量高：26.1 独有 API（Identifier/jspecify/ResourceHandler/Capabilities.Fluid）在 Java 代码中零残留，1.21.1 目标 API 使用正确。
• 多模块 wiring 完整且一致（jarJar 双声明、interface injection 声明、AT、@mod dist），上一次复查的结构性发现均确认仍成立。
• 网络层对称性（RegistryFriendlyByteBuf 1.21.1 签名、配置握手模式）、ExplosionSession 异步纯数学预计算 + 主线程 isLoaded 守卫等要点保持正确。

结论: COMMENT（倾向 APPROVE，但建议先补两处运行验证） — 反向移植本身干净、接线完整；阻塞项集中在运行验证缺口：① module.sync CoreMod 首次启动实测（最高优先），② roseauCheck 运行验证。若 CI 变绿且 runClient 启动通过，可合并。

---

由 Hermes Agent 审查

⚠️ File-mutation verifier: 2 file(s) were NOT modified this turn despite any wording above that may suggest otherwise. Run git status or read_file to confirm.
• /tmp/mixin_check.sh — [write_file] Write denied: '/tmp/mixin_check.sh' is a protected system/credential file.
• /tmp/wiring_check.sh — [write_file] Write denied: '/tmp/wiring_check.sh' is a protected system/credential file.

[Gugle2308]

💾 Self-improvement review: Patched SKILL.md in skill 'reverse-port-pr-review' (1 replacement). · Patched references/reverse-port-review-detail.md in skill 'reverse-port-pr-review' (1 replacement).

[Gugle2308]

🌿 Roseau API Breaking Change Report

Module	Status	Breaking Changes
codec	⚪ Skipped	—
collision	⚪ Skipped	—
config	⚪ Skipped	—
integration	⚪ Skipped	—
moveable-entity-block	⚪ Skipped	—
network	⚪ Skipped	—
rendering	⚪ Skipped	—
space-select	⚪ Skipped	—
font	⚪ Skipped	—
util	⚪ Skipped	—
explosion	⚪ Skipped	—
rpc	⚪ Skipped	—
multiblock	⚪ Skipped	—
recipe	⚪ Skipped	—
registrum	⚪ Skipped	—
sync	⚪ Skipped	—
wheel	⚪ Skipped	—
main	⚪ Skipped	—

Full CSVs: see the Artifacts section of this workflow run.

[Gugle2308]

⚠️ Iteration budget exhausted (90/90) — asking model to summarise

[Gugle2308]

代码审查摘要 — PR #97

操作: synchronize（反向移植复查）
方向: dev/26.1（高）→ port/1.21.1（低）✅ 确认
范围: 419 文件 (337 Java, 21 json, 16 gradle, 5 docs) / 35,213 行 diff / 319 新增 / 0 删除
PR 状态: mergeable=true，mergeable_state=unstable（CI 未全绿）⚠️
EOF 缺换行: 31（正常）｜Ghost 文件: 0（修正版 awk 检测干净）

---

✅ 反向 API 残留扫描（干净）

源版本 API（应=0）	Java 命中	归属
Identifier	1	纯 Javadoc 移植说明（"Ported from 26.1…"），非代码 ✅
org.jspecify / ResourceHandler / Capabilities.Fluid	0	✅
GpuDevice / RenderPipeline	7 / 4	全部为 Javadoc 注释，执行层已改写 ✅

1.21.1 正常指标：ResourceLocation 119、RegisterShadersEvent 23、ShaderInstance 12、builtInRegistryHolder 10（合法用法，不误报）。

✅ Mixin 配置 ↔ 类文件交叉检查（全匹配）

• rendering.integration：CachedBlockEntityRenderingPipelineMixin + RebuildTaskMixin + plugin ALRIntegrationCompatMixinPlugin（类存在）✅
• rendering：GameRendererMixin / MinecraftMixin / RenderTypeMixin ✅
• explosion：SingleItemRecipeAccessor、font：GuiGraphicsExtractorMixin ✅
• collision / rpc / space-select / sync 空数组 = 模板残留，无害 ✅

✅ 接线验证

• jarJar 链完整：sync→processor+codec+network+util 四联成对声明 ✅
• @mod dist 分离：Rpc/SpaceSelect/SyncClient 均 dist = Dist.CLIENT ✅
• 7 个 neoforge.mods.toml 齐全；settings.gradle 9 个新模块注册完整 ✅
• CI workflow 引用（build_and_test.yml / publish_maven_central.yml）按 head SHA 验证存在 ✅

📋 声称清单映射（todo 20 项全对应）

#P	声称	文件数	状态
#P1 rpc / #P6 font	25 / 25	✅
#P2 sync+processor	31+6（ICoreMod 重写）	✅
#P3/#P4/#P5	explosion 10 / collision 3 / space-select 18	✅
#P7 rendering	79（#P7a–e 分批）	✅
#P8–#P20	renderdoc/config/integration/network/util/registrum/multiblock/recipe/wheel/test/roseau/codec	✅

---

🔴 关键问题（建议修复后再合并）

1. SyncTargetIndex / LazySyncTargetIndex 静态块直接调 LoadingModList.get()，无容错

   • CoreMod 时序风险：targets() 在 getTransformers() 时求值，依赖静态块已填充索引；若加载顺序早于 mod 扫描完成 → 空目标集 → @Sync/@lazysync 注入静默失效 → 运行时 NPE（parent 为 null）。
   • todo 文档自认 V-9（jar-in-jar services 可见性）/ V-10（GETSTATIC 类可解析性）均为「未验证（需打包启动实测）」，验收仅 compileJava。建议静态块 try/catch 容错 + allTargets() 空集防御。这是多次复查一致确认的结构性风险。

2. gradle/scripts/roseau.gradle 配置期急切网络（#P19 标注「运行待补验」）

   • findProperty(...) ?: { ... }() 在配置期立即执行：对 repo1 + cjsah 两仓库各发起 HTTP 请求（5s+5s 超时×2），且经 gradle.projectsEvaluated { subprojects { apply from } } 应用到全部子项目——每次构建（含 gradlew help）都会触发，离线/受限网络下 20+ 子项目阻塞数分钟。建议改 lazy Provider 或任务执行期解析。

⚠️ 警告

• CI 未全绿（mergeable_state=unstable），合并前等 CI 通过。
• 运行验证缺失：V-4/V-16（District.color 颜色语义）、L-2/L-3/L-4（RPC 往返、@sync 注入生效）均需 runClient 实测；Iris 集成改为反射访问（V-2/V-6 已签名核实 1.8.12）但运行时行为未验证。建议合并后补一次打包启动验证。

💡 非阻塞建议

• 多块结构 Long→BlockPos 键化（#P15）兼容旧存档（load 路径 fromTag 不变，仅容器键类型变化）；未加载区块快照复用路径有 Entry.test() null 守卫、old 快照由构造器初始化非 null，实现正确 ✅
• SpawnItem 零数量守卫 + SetBlock optionalFieldOf("nbt", new CompoundTag()) 修复移植正确。
• RPC 配置握手（ICustomConfigurationTask→索引 map→client adopt）编解码对称 ✅

🟢 看起来不错

• 反向 API 残留完全为注释、无真实代码残留；渲染降级架构正确（GpuDevice/RenderPipeline → RegisterShadersEvent + ShaderInstance + RenderTarget）。
• Mixin/plugin/jarJar/dist 接线全部闭合，无 ghost 文件，无 TODO/FIXME/硬编码凭据。

---

结论: APPROVE（建议合并前处理 2 项关键问题或至少记录后续验证计划） — 移植质量整体很高，代码层残留为零；剩余风险集中在「未做 runClient 启动实测」的 ICoreMod 时序与 roseau 急切网络，均为已记录但未销账的验证项。建议合并时要求补跑一次打包启动（验证 V-9/V-10），roseau 网络问题可在后续独立修复。

---

由 Hermes Agent 审查（reverse-port-pr-review 流程，复查重跑全部扫描）

[Gugle2308]

💾 Self-improvement review: Patched SKILL.md in skill 'reverse-port-pr-review' (1 replacement). · Patched references/reverse-port-review-detail.md in skill 'reverse-port-pr-review' (1 replacement).

[Gugle2308]

🌿 Roseau API Breaking Change Report

Module	Status	Breaking Changes
codec	⚪ Skipped	—
collision	⚪ Skipped	—
config	⚪ Skipped	—
integration	⚪ Skipped	—
moveable-entity-block	⚪ Skipped	—
network	⚪ Skipped	—
rendering	⚪ Skipped	—
space-select	⚪ Skipped	—
font	⚪ Skipped	—
util	⚪ Skipped	—
explosion	⚪ Skipped	—
rpc	⚪ Skipped	—
multiblock	⚪ Skipped	—
recipe	⚪ Skipped	—
registrum	⚪ Skipped	—
sync	⚪ Skipped	—
wheel	⚪ Skipped	—
main	⚪ Skipped	—

Full CSVs: see the Artifacts section of this workflow run.

[Gugle2308]

所有扫描完成。以下是审查结果：

---

代码审查摘要 — PR #97

操作: synchronize
范围: 314 个文件 (256 Java, 214 新增, 0 删除) / 27,655 行 diff
分支: dev/1.21.1 (head, Gu-ZT) → port/1.21.1 (base, Anvil-Dev) — 反向移植（dev/26.1 → port/1.21.1）
状态: draft ⚠️ | mergeable_state: unstable（CI 未全绿，建议等 CI 绿后再合并）

📌 本次为 synchronize 复查。diff 规模从上次的 419 文件/35.2K 行收缩至 314 文件/27.6K 行——与 todo 表 #P7（rendering）/ #P8（renderdoc-loader）「已移除」状态一致（render 模块 79 文件 + renderdoc 整体剔除），属预期变化。所有扫描均重新执行（headers 重建、反向 API 扫描、mixin 交叉、ghost 检测），未沿用上次计数。

✅ 反向 API 健康度（26.1 残留扫描）

指标	计数	状态
Identifier（Java 代码）	0	✅
org.jspecify.*（Java 代码）	0	✅
ResourceHandler / Capabilities.Fluid（Java 代码）	0	✅
GpuDevice/RenderPipeline/ShaderDefines（Java 代码）	0	✅
builtInRegistryHolder	9 处 Java	✅ 均为合法 1.21.1 用法
-26.1 坐标（build.gradle 实际代码）	0	✅（命中全在 docs）
Ghost 文件（修正版 awk）	0	✅
EOF 缺换行	29	⚠️ 均为手写源文件，非 generated，无碍

所有 26.1 API 命中均落在 docs 规划文档（合法提及），Java 真实残留 = 0。

🔴 关键（建议合并前处理）

• module.sync/processor — ICoreMod 时序风险仍未验证（V-9/V-10）
  SyncTargetIndex/LazySyncTargetIndex 静态块急切执行 onLoad() → LoadingModList.get().getModFiles()。targets() 求值时机若早于 mod 扫描完成，两种失败形态：启动异常崩溃，或拿到空目标集 → @Sync/@LazySync 静默失效 → 运行时 NPE（parent 为 null）。todo 表 V-9（jar-in-jar services 可见性）/V-10（GETSTATIC 类可解析性）仍标「未验证」，验收仅到 compileJava。建议：打包后 runClient 启动实测，并为静态块加 try/catch 容错 + 索引构建改 FML 事件时机显式刷新。

⚠️ 警告

• gradle/scripts/roseau.gradle — 急切网络反模式（#P19）
  def roseauBaselineVersion = project.findProperty(...) ?: { ...HTTP... }() 闭包立即调用，在配置期对每个子项目×2 个 Maven 仓库发请求（5s 连接 + 5s 读取超时）。根 build.gradle 通过 gradle.projectsEvaluated { subprojects { apply from } } 全量应用——离线/受限网络下每次构建（含 gradlew help）会阻塞数分钟。todo L-1 已记录本地运行 roseauCheck 时 Gradle daemon 崩溃（roseau-cli 0.6.0 与 Java 21 兼容问题），此项「运行待补验」。建议改 lazy Provider 或仅任务执行时解析。
• module.collision/src/.../CollisionTest.java — 18 处 System.out：@ApiStatus.Internal 自测类，可接受，但建议换 @Slf4j logger。
• mixins.json compatibilityLevel 不一致：anvillib_explosion 为 JAVA_21，其余 5 个为 JAVA_8（模板复制残留）。1.21.1 运行于 Java 21，JAVA_8 无害但建议统一。

💡 建议（非阻塞）

• 各模块 EOF 缺换行共 29 处（util/recipe/network 等手写源文件），建议统一补换行符。
• 语言文件 en_us/zh_cn 键完全对称 ✅（explosion 7 键逐一对应，section key 与 TOML 点分隔一致）。
• todo 表中 V-4（renderVoxelShape 颜色语义）/V-16（District.color() int 打包颜色）仍需 runClient 目测，本 PR 已无 rendering 模块，若 space-select/wheel 独立验证通过可销账。

🟢 看起来不错

• Mixin JSON ↔ 类文件全匹配：6 个 mixins.json 中 2 个非空条目（explosion SingleItemRecipeAccessor、font client GuiGraphicsExtractorMixin）均对应存在；4 个空数组为模板残留，无害。
• interface_injections 接线正确：font 模块 interface_injections.json（GuiGraphics → GuiGraphicsExtractorExtension）在 build.gradle 有 from+publish 声明，GuiGraphicsExtractorMixin implements 该接口 ✅。
• jarJar 链完整：module.main 聚合 8 个新模块（api + project 成对）；sync→processor+codec+network+util 四联；rpc/space-select→network；explosion→config；font 已按 todo 切断 rendering 依赖 ✅。
• @mod dist 分离正确：font 纯 CLIENT；rpc/sync/space-select 主 @mod + Client 类 dist = Dist.CLIENT；AnvilLibSync 构造器保留 ModContainer 参数为 1.21.1 合法签名 ✅。
• settings.gradle 注册完整：include 'module.sync:processor' 等全部新模块 + 坐标后缀 -neoforge-1.21.1 ✅。
• CI workflow 引用验证：build_and_test.yml/publish_maven_central.yml 按 head SHA（819199e）均 HTTP 200 ✅。
• RPC 编解码对称（writeVarInt 索引 + 参数 codec 数组 / decode 对称）；ExplosionSession 并行切片 endX=(taskIdx==numTasks-1)?layer:startX+sliceSize-1 无空洞；isLoaded 守卫 + per-tick 钳制 ✅。测试代码 Thread.sleep 均仅出现在注释/Javadoc 中（实际用 tick 调度器避免主线程阻塞）✅。

📋 声称验证表（#P1–#P20）

声称	状态	对应模块文件数
#P1 module.rpc (25)	✅	25 Java
#P2 module.sync+processor (31)	✅	31 Java（主 25 + processor 6）
#P3 module.explosion (10)	✅	10
#P4 module.collision (3)	✅	3
#P5 module.space-select (18)	✅	18
#P6 module.font (25)	✅	25
#P7 module.rendering	✅ 已移除	diff 中 0 文件
#P8 renderdoc-loader	✅ 已移除	diff 中 0 文件
#P9–#P20（config/integration/network/util/registrum×2/multiblock/recipe/wheel/test/roseau/codec）	✅	8/2/6/11/44/13/26/12/16/roseau.yaml+gradle/2

V-9/V-10/L-1 仍为「未验证/运行待补验」，其余 V 项已销账或编译级验证通过。

结论: COMMENT — 代码质量与移植完整性优秀（反向 API 残留 0、mixin/injection/jarJar 接线全对、声称清单全兑现）。合并前请：① 完成 V-9/V-10 的打包启动实测并补防御性改写（ICoreMod 时序是最大结构性风险）；② 处理 roseau.gradle 急切网络；③ 等 CI 绿（当前 unstable）。PR 仍为 draft，转 ready 后再合入。

---

由 Hermes Agent 审查

[Gugle2308]

💾 Self-improvement review: Patched SKILL.md in skill 'reverse-port-pr-review' (1 replacement). · Patched references/reverse-port-review-detail.md in skill 'reverse-port-pr-review' (1 replacement).

[Gugle2308]

🌿 Roseau API Breaking Change Report

Module	Status	Breaking Changes
codec	⚪ Skipped	—
collision	⚪ Skipped	—
config	⚪ Skipped	—
integration	⚪ Skipped	—
moveable-entity-block	⚪ Skipped	—
network	⚪ Skipped	—
rendering	⚪ Skipped	—
space-select	⚪ Skipped	—
font	⚪ Skipped	—
util	⚪ Skipped	—
explosion	⚪ Skipped	—
rpc	⚪ Skipped	—
multiblock	⚪ Skipped	—
recipe	⚪ Skipped	—
registrum	⚪ Skipped	—
sync	⚪ Skipped	—
wheel	⚪ Skipped	—
main	⚪ Skipped	—

Full CSVs: see the Artifacts section of this workflow run.

[Gugle2308]

所有扫描完成。汇总数据：

• 反向 API 扫描：Identifier/jspecify/ResourceHandler/Capabilities.Fluid/GpuDevice/RenderPipeline 在 Java 代码中均为 0（排除 Javadoc 注释）✅
• Ghost 文件：0；EOF 缺换行：0
• Mixin：6 个 mixins.json，font 的 GuiGraphicsExtractorMixin 在 client 数组且类存在 ✅，其余空数组为模板残留无害；compatibilityLevel 混用（explosion=JAVA_21，其余 JAVA_8）
• 模块接线：jarJar 链完整（explosion→config、rpc→network、space-select→network、sync→processor+codec+network+util），main 聚合 8 模块成对
• CI workflow 引用：build_and_test.yml / publish_maven_central.yml 按 head SHA 5392d42 均返回 200 ✅
• 声称清单：#P1–#P20 全 ✅；V-4/V-9/V-10/V-16 仍未验证（需 runClient）；L-1 roseauCheck、L-2~L-9 运行验证未销账

---

代码审查摘要 — PR #97（第 6 次 synchronize 复查）

操作: synchronize
范围: 313 个文件（255 Java, 213 新增, 0 删除）/ 27,632 行 diff
分支: dev/1.21.1 → port/1.21.1（反向移植：26.1 → 1.21.1）
状态: open / draft / mergeable=true / mergeable_state=unstable（CI 未全绿）
规模对比: 上次 314 文件/27,655 行 → 本次 313/27,632，基本持平（rendering/renderdoc 移除后保持稳定，非回归）

✅ 反向 API 健康度（本次全量重扫）

指标（26.1 独有，应=0）	Java 代码命中	状态
Identifier	0（Javadoc 注释命中已排除）	✅
org.jspecify	0	✅
ResourceHandler / Capabilities.Fluid	0	✅
GpuDevice / RenderPipeline	0	✅

1.21.1 基线指标正常：ResourceLocation 96、org.jetbrains 58、IFluidHandler 23、builtInRegistryHolder 10（均为 1.21.1 合法用法，非 26.1 残留）。

🔴 关键

无新增关键问题。结构性风险复查后依旧成立：

• module.sync CoreMod 时序（V-9/V-10，未销账） — SyncTargetIndex 静态块直接调 LoadingModList.get().getModFiles() 扫描注解。若该类在 ModLauncher 早期阶段被加载（早于 mod 索引构建），有两种失败形态：启动抛异常崩溃，或 targets() 拿到空目标集导致 @Sync/@LazySync 注入静默失效。todo 验收只到 compileJava，无 runClient 启动日志实测。建议：静态块 try/catch 容错 + 允许空索引 + 在 FML 事件时机显式刷新。

⚠️ 警告

• roseau.gradle 急切网络请求（L-1，未销账） — roseauBaselineVersion = project.findProperty(...) ?: { ...HTTP... }() 在配置期急切执行，每个子项目每次构建（含 gradlew help）向 repo1.maven.org + server.cjsah.net 发起请求（5s×2 超时）。20+ 子项目离线/受限网络时每次构建阻塞数分钟。建议改 lazy Provider 或仅任务执行时解析。
• mergeable_state=unstable — CI 未全绿，且 PR 处于 draft。建议等 CI 绿后再合并。

💡 建议

• compatibilityLevel 混用 — anvillib_explosion.mixins.json = JAVA_21，其余 5 个 = JAVA_8，模板复制残留。1.21.1 运行于 Java 21，JAVA_8 无害，但建议统一。
• V-4/V-16（space-select 渲染语义） — renderVoxelShape 颜色、District.color() 打包语义仍需 runClient 目测验证。
• ExplosionSession — THREAD_EXECUTOR 为静态 newWorkStealingPool()（注释写 "virtual thread executor" 名不副实，小瑕疵）；预计算 future 在 session 结束时不 cancel（纯数学无副作用，浪费可忽略）。
• L-2~L-9（RPC/sync/explosion/multiblock/wheel 行为验证） — 均为运行类验收，todo 已如实登记，合并前建议至少跑一次 runClient 冒烟。

🟢 看起来不错

• 反向 API 扫描零残留（排除 Javadoc 移植说明注释后确认），1.21.1 目标 API 使用规范。
• Ghost 文件 0、EOF 缺换行 0，213 个新文件全部有实际内容。
• Mixin 交叉检查：font 的 GuiGraphicsExtractorMixin（client 数组）+ interface_injections.json（from+publish 声明）↔ GuiGraphicsExtractorExtension 三线对齐 ✅。
• jarJar 依赖链完整：explosion→config、rpc→network、space-select→network、sync→processor+codec+network+util，main 聚合 8 模块 api+project 成对；settings.gradle 注册含 module.sync:processor 子模块。
• @mod dist 分离：font、rpc-client 均为 @Mod(value=..., dist=Dist.CLIENT)，符合 1.21.1 标准。
• CI workflow 引用：新增 codec/collision 等任务引用的 build_and_test.yml / publish_maven_central.yml 按 head SHA 5392d42 验证 200 存在。
• 关键逻辑抽查：ExplosionSession 并行切片 endX=(taskIdx==numTasks-1)?layer:startX+sliceSize-1 无空洞无重叠、layer≥96 才并行、主线程 tick 内 isLoaded 守卫；Multiblock snapshot 非 null 初始化 + 未加载区块 fallback 保守通过 + Form/Unform 可取消事件回滚对称；PacketData 泛型类型实参匹配修复；SpawnItem 零数量守卫；SetBlock nbt 默认 CompoundTag()；OneTimeEventReceiver 双重空值修复；RpcRequestPayload 编解码对称。
• 语言文件：explosion/font 的 en_us ↔ zh_cn 键完全对称。

📋 声称验证表（todo → diff 映射）

声称	状态	对应模块文件数
#P1 module.rpc	✅	30
#P2 module.sync + processor	✅	39 + 9
#P3 module.explosion	✅	15
#P4 module.collision	✅	7
#P5 module.space-select	✅	23
#P6 module.font	✅	36
#P7 module.rendering	✅（已移除）	0 残留确认
#P8 renderdoc-loader	✅（已移除）	0 残留确认
#P9 module.config	✅	8
#P10 module.integration	✅	2
#P11 module.network	✅	6
#P12 module.util	✅	12
#P13/#P14 module.registrum	✅	44
#P15 module.multiblock	✅	14
#P16 module.recipe	✅	26
#P17 module.wheel	✅	14
#P18 module.test	✅	17
#P19 构建体系 roseau	✅（运行待补验 L-1）	roseau.yaml + roseau.gradle
#P20 module.codec CodecUtil	✅	2（32 个 create/mapCodec 重载）

结论: COMMENT — 反向移植质量良好（API 残留 0、接线完整、逻辑抽查无新问题），规模收缩与 todo 移除项一致。但 PR 仍为 draft、CI unstable，且 V-9/V-10（CoreMod 时序）与 L-1（roseau 急切网络）两项结构性风险未销账、无 runClient 实测。建议：CI 转绿后合并；合并前至少完成一次启动冒烟验证，并优先处理 roseau.gradle 的急切网络反模式。

🧪 测试建议

被测目标	推荐测试场景	优先级
SyncClassProcessor / SyncTargetIndex	启动日志验证注入点 + LoadingModList 就绪时序	🔴
roseau.gradle roseauCheck	离线/无网络环境构建不阻塞	🔴
ExplosionSession	并行切片边界（layer=96/97）、radius 边界、per-tick 上限	🟡
MultiblockState	未加载区块快照复用、事件取消回滚	🟡
WheelWidget	环形扇区角度归一化（负角/跨 2π）、shader null 回退	🟡

---

由 Hermes Agent 审查

[Gugle2308]

💾 Self-improvement review: Patched references/reverse-port-review-detail.md in skill 'reverse-port-pr-review' (1 replacement). · Patched SKILL.md in skill 'reverse-port-pr-review' (1 replacement).

[Gugle2308]

🌿 Roseau API Breaking Change Report

Module	Status	Breaking Changes
codec	⚪ Skipped	—
collision	⚪ Skipped	—
config	⚪ Skipped	—
integration	⚪ Skipped	—
moveable-entity-block	⚪ Skipped	—
network	⚪ Skipped	—
rendering	⚪ Skipped	—
space-select	⚪ Skipped	—
font	⚪ Skipped	—
util	⚪ Skipped	—
explosion	⚪ Skipped	—
rpc	⚪ Skipped	—
multiblock	⚪ Skipped	—
recipe	⚪ Skipped	—
registrum	⚪ Skipped	—
sync	⚪ Skipped	—
wheel	⚪ Skipped	—
main	⚪ Skipped	—

Full CSVs: see the Artifacts section of this workflow run.

[Gugle2308]

🌿 Roseau API Breaking Change Report

Module	Status	Breaking Changes
codec	✅ Compatible	0
collision	⚪ Skipped	—
config	🔴 BC detected	1
integration	✅ Compatible	0
moveable-entity-block	✅ Compatible	0
network	✅ Compatible	0
rendering	⚪ Skipped	—
space-select	⚪ Skipped	—
font	⚪ Skipped	—
util	🔴 BC detected	6
explosion	⚪ Skipped	—
rpc	⚪ Skipped	—
multiblock	🔴 BC detected	4
recipe	✅ Compatible	0
registrum	✅ Compatible	0
sync	⚪ Skipped	—
wheel	🔴 BC detected	1
main	✅ Compatible	0

🔴 config — 1 breaking change(s)

dev/anvilcraft/lib/v2/config/ConfigRecord.java:25

dev.anvilcraft.lib.v2.config.ConfigRecord
dev.anvilcraft.lib.v2.config.ConfigRecord.<init>(java.lang.String,net.neoforged.fml.config.ModConfig$Type,net.neoforged.neoforge.common.ModConfigSpec,java.lang.Object,java.util.List<dev.anvilcraft.lib.v2.config.ConfigField>)
CONSTRUCTOR_REMOVED
✓ binary-compatible
✓ source-compatible

🔴 util — 6 breaking change(s)

dev/anvilcraft/lib/v2/util/stack/UnlimitedItemStack.java:-1

dev.anvilcraft.lib.v2.util.stack.UnlimitedItemStack
dev.anvilcraft.lib.v2.util.stack.UnlimitedItemStack
SUPERTYPE_REMOVED
✓ binary-compatible
✓ source-compatible

dev/anvilcraft/lib/v2/util/stack/UnlimitedItemStack.java:124

dev.anvilcraft.lib.v2.util.stack.UnlimitedItemStack
dev.anvilcraft.lib.v2.util.stack.UnlimitedItemStack.deserializeNBT(net.minecraft.core.HolderLookup$Provider,net.minecraft.nbt.CompoundTag)
METHOD_REMOVED
✓ binary-compatible
✓ source-compatible

dev/anvilcraft/lib/v2/util/stack/UnlimitedItemStack.java:128

dev.anvilcraft.lib.v2.util.stack.UnlimitedItemStack
dev.anvilcraft.lib.v2.util.stack.UnlimitedItemStack.parse(net.minecraft.core.HolderLookup$Provider,net.minecraft.nbt.Tag)
METHOD_REMOVED
✓ binary-compatible
✓ source-compatible

dev/anvilcraft/lib/v2/util/stack/UnlimitedItemStack.java:119

dev.anvilcraft.lib.v2.util.stack.UnlimitedItemStack
dev.anvilcraft.lib.v2.util.stack.UnlimitedItemStack.serializeNBT(net.minecraft.core.HolderLookup$Provider)
METHOD_REMOVED
✓ binary-compatible
✓ source-compatible

dev/anvilcraft/lib/v2/util/stack/UnlimitedItemStack.java:218

dev.anvilcraft.lib.v2.util.stack.UnlimitedItemStack
dev.anvilcraft.lib.v2.util.stack.UnlimitedItemStack.split(int)
METHOD_RETURN_TYPE_CHANGED
✓ binary-compatible
✓ source-compatible

dev/anvilcraft/lib/v2/util/stack/UnlimitedItemStack.java:210

dev.anvilcraft.lib.v2.util.stack.UnlimitedItemStack
dev.anvilcraft.lib.v2.util.stack.UnlimitedItemStack.splitUnlimited(int)
METHOD_REMOVED
✓ binary-compatible
✓ source-compatible

🔴 multiblock — 4 breaking change(s)

dev/anvilcraft/lib/v2/multiblock/dynamic/MultiblockCheckSnapshot.java:20

dev.anvilcraft.lib.v2.multiblock.dynamic.MultiblockCheckSnapshot
dev.anvilcraft.lib.v2.multiblock.dynamic.MultiblockCheckSnapshot.<init>(long,java.util.Map<net.minecraft.core.BlockPos,dev.anvilcraft.lib.v2.multiblock.dynamic.MultiblockCheckSnapshot$Entry>)
CONSTRUCTOR_REMOVED
✓ binary-compatible
✓ source-compatible

dev/anvilcraft/lib/v2/multiblock/dynamic/MultiblockCheckSnapshot.java:20

dev.anvilcraft.lib.v2.multiblock.dynamic.MultiblockCheckSnapshot
dev.anvilcraft.lib.v2.multiblock.dynamic.MultiblockCheckSnapshot.controllerPosLong()
METHOD_REMOVED
✓ binary-compatible
✓ source-compatible

dev/anvilcraft/lib/v2/multiblock/dynamic/MultiblockState.java:47

dev.anvilcraft.lib.v2.multiblock.dynamic.MultiblockState
dev.anvilcraft.lib.v2.multiblock.dynamic.MultiblockState.<init>(net.minecraft.core.BlockPos,net.minecraft.core.Holder$Reference<dev.anvilcraft.lib.v2.multiblock.dynamic.definition.MultiblockDefinition>,boolean)
CONSTRUCTOR_REMOVED
✓ binary-compatible
✓ source-compatible

dev/anvilcraft/lib/v2/multiblock/dynamic/MultiblockState.java:38

dev.anvilcraft.lib.v2.multiblock.dynamic.MultiblockState
dev.anvilcraft.lib.v2.multiblock.dynamic.MultiblockState.<init>(net.minecraft.core.BlockPos,net.minecraft.core.Holder<dev.anvilcraft.lib.v2.multiblock.dynamic.definition.MultiblockDefinition>)
CONSTRUCTOR_REMOVED
✓ binary-compatible
✓ source-compatible

🔴 wheel — 1 breaking change(s)

dev/anvilcraft/lib/v2/wheel/client/gui/component/WheelWidget.java:143

dev.anvilcraft.lib.v2.wheel.client.gui.component.WheelWidget
dev.anvilcraft.lib.v2.wheel.client.gui.component.WheelWidget.<init>(int,int,int,int,float,float,int,int,int,int,int,int,float,int,float,java.util.List<dev.anvilcraft.lib.v2.wheel.client.gui.component.WheelWidget$RawSection>,int)
CONSTRUCTOR_REMOVED
✓ binary-compatible
✓ source-compatible

Full CSVs: see the Artifacts section of this workflow run.

[Gugle2308]

所有扫描完成。汇总第 7 次 synchronize 复查结果：

代码审查摘要 — PR #97

操作: synchronize（第 7 次复查）
范围: 327 文件（221 新增 / 0 删除 / +20,755 -1,176 行）— 上次 313 文件/27.6K 行，本次 +14 文件、-2K 行
方向: dev/26.1 → port/1.21.1（反向移植）✅
状态: open / draft / mergeable_state=unstable（CI 未全绿）/ head SHA f107a889

📊 本次规模变化来源

新增内容	说明
.github/modules.json + generate-matrix.js	CI 矩阵化：Kahn 拓扑分层（l0/l1/l2），替代逐模块手写 job（ci.yml 因此 -2K 行）
roseau.yaml + gradle/scripts/roseau.gradle + roseau_check.yml / roseau_comment.yml	新增 Roseau API 兼容性检查基础设施
module.moveable-entity-block ×3	仅 package-info 注解迁移（@FieldsAreNonnullByDefault）
module.integration/IntegrationManager	1 行：meter.increment() 进度计数

✅ 反向 API 残留扫描（核心检查）

API	Java 代码残留
Identifier / org.jspecify / ResourceHandler / Capabilities.Fluid / GpuDevice / RenderPipeline	0 / 0 / 0 / 0 / 0 / 0（含注释命中 0）

Ghost 文件 0、硬编码密钥 0、TODO/FIXME 0、lang 键对称（font 6/6、explosion 7/7）✅

🔴 关键

• ICoreMod 时序风险（V-9/V-10，复查后依旧成立，未销账） — SyncTargetIndex 静态块 static { onLoad(); } 直接调 LoadingModList.get().getModFiles()；SyncClassProcessor（ICoreMod）的 targets() 调 SyncTargetIndex.allTargets() 触发静态初始化。Javadoc 声称「mod 发现完成后就绪」是推断而非实测——两种失败形态依旧存在（早期崩溃 / 空目标集静默失效 → @sync 注入失效运行时 NPE）。验收仅到 compileJava，无 runClient 启动冒烟。

⚠️ 警告

• roseau.gradle 配置期急切网络（L-1，部分缓解未销账） — def roseauBaselineVersion = project.findProperty(...) ?: { ...HTTP... }() 在配置期急切执行，且根 build.gradle projectsEvaluated { subprojects { apply from } } 使每个子项目每次构建（含 gradlew help）都向 2 个 Maven 仓库发请求。改进：本次加了 try/catch + fallback（warn + 返回当前 version），不再「阻塞数分钟」，但网络请求仍在配置期发生。建议改为 lazy Provider 或仅 roseauCheck 任务执行时解析。
• EOF 缺换行 29 处（上次 0，本次新增） — 多为 package-info.java（注解迁移产物，Java 惯例无害）、lang JSON、mixins.json；SyncDirection.java（新文件）也缺。29 < 100 阈值不算异常，建议顺手补全。

💡 建议

• compatibilityLevel 混用（模板复制残留）：anvillib_explosion.mixins.json=JAVA_21，其余 5 个=JAVA_8。1.21.1 运行于 Java 21，JAVA_8 无害，建议统一。
• CollisionTest.java 18 处 System.out/printStackTrace（@ApiStatus.Internal 自测类，可接受，无需改）。

🟢 看起来不错

• CI 矩阵化设计合理：modules.json 声明 17 模块依赖，generate-matrix.js Kahn 拓扑排序生成 3 层并行矩阵；全部被引 workflow（build_and_test / publish_maven_central / ci / release / pull_request / roseau_check / roseau_comment）按 head SHA 验证均 200。
• mixin 交叉检查：6 个 mixins.json 全通过（font 的 GuiGraphicsExtractorMixin OK，其余空数组=模板残留无害）。
• jarJar 链完整：main 聚合 8 模块 api+project 成对；sync→processor+codec+network+util 四联。
• @mod dist 分离正确：font/rpc-client/space-select-client/sync-client 均 Dist.CLIENT。
• IntegrationManager.meter.increment()、moveable-entity-block 注解迁移均为小且正确的同步修正。

📋 声称验证

todo 文档（docs/migration-todo）已不在 diff（净变化为零——#P7 rendering / #P8 renderdoc-loader 移除确认，diff 中 0 文件）。V-9/V-10（runClient 启动验证）状态无法从 todo 文档核对，维持「未验证」。

结论: COMMENT — 第 7 次复查：反向 API 残留持续为 0，CI 矩阵化 + Roseau 检查是实质改进；但 ICoreMod 时序（V-9/V-10）与 roseau 配置期网络（L-1）两个结构性风险复查后依旧成立。等 CI 转绿（当前 unstable）+ 至少一次 runClient 启动冒烟后再合并。

---

由 Hermes Agent 审查

[Gugle2308]

💾 Self-improvement review: Patched SKILL.md in skill 'reverse-port-pr-review' (1 replacement). · Patched references/reverse-port-review-detail.md in skill 'reverse-port-pr-review' (1 replacement).

[Gugle2308]

🌿 Roseau API Breaking Change Report

Module	Status	Breaking Changes
codec	✅ Compatible	0
collision	⚪ Skipped	—
config	🔴 BC detected	1
integration	✅ Compatible	0
moveable-entity-block	✅ Compatible	0
network	✅ Compatible	0
rendering	⚪ Skipped	—
space-select	⚪ Skipped	—
font	⚪ Skipped	—
util	🔴 BC detected	6
explosion	⚪ Skipped	—
rpc	⚪ Skipped	—
multiblock	🔴 BC detected	4
recipe	✅ Compatible	0
registrum	✅ Compatible	0
sync	⚪ Skipped	—
wheel	🔴 BC detected	3
main	✅ Compatible	0

🔴 config — 1 breaking change(s)

dev/anvilcraft/lib/v2/config/ConfigRecord.java:25

dev.anvilcraft.lib.v2.config.ConfigRecord
dev.anvilcraft.lib.v2.config.ConfigRecord.<init>(java.lang.String,net.neoforged.fml.config.ModConfig$Type,net.neoforged.neoforge.common.ModConfigSpec,java.lang.Object,java.util.List<dev.anvilcraft.lib.v2.config.ConfigField>)
CONSTRUCTOR_REMOVED
✓ binary-compatible
✓ source-compatible

🔴 util — 6 breaking change(s)

dev/anvilcraft/lib/v2/util/stack/UnlimitedItemStack.java:-1

dev.anvilcraft.lib.v2.util.stack.UnlimitedItemStack
dev.anvilcraft.lib.v2.util.stack.UnlimitedItemStack
SUPERTYPE_REMOVED
✓ binary-compatible
✓ source-compatible

dev/anvilcraft/lib/v2/util/stack/UnlimitedItemStack.java:124

dev.anvilcraft.lib.v2.util.stack.UnlimitedItemStack
dev.anvilcraft.lib.v2.util.stack.UnlimitedItemStack.deserializeNBT(net.minecraft.core.HolderLookup$Provider,net.minecraft.nbt.CompoundTag)
METHOD_REMOVED
✓ binary-compatible
✓ source-compatible

dev/anvilcraft/lib/v2/util/stack/UnlimitedItemStack.java:128

dev.anvilcraft.lib.v2.util.stack.UnlimitedItemStack
dev.anvilcraft.lib.v2.util.stack.UnlimitedItemStack.parse(net.minecraft.core.HolderLookup$Provider,net.minecraft.nbt.Tag)
METHOD_REMOVED
✓ binary-compatible
✓ source-compatible

dev/anvilcraft/lib/v2/util/stack/UnlimitedItemStack.java:119

dev.anvilcraft.lib.v2.util.stack.UnlimitedItemStack
dev.anvilcraft.lib.v2.util.stack.UnlimitedItemStack.serializeNBT(net.minecraft.core.HolderLookup$Provider)
METHOD_REMOVED
✓ binary-compatible
✓ source-compatible

dev/anvilcraft/lib/v2/util/stack/UnlimitedItemStack.java:218

dev.anvilcraft.lib.v2.util.stack.UnlimitedItemStack
dev.anvilcraft.lib.v2.util.stack.UnlimitedItemStack.split(int)
METHOD_RETURN_TYPE_CHANGED
✓ binary-compatible
✓ source-compatible

dev/anvilcraft/lib/v2/util/stack/UnlimitedItemStack.java:210

dev.anvilcraft.lib.v2.util.stack.UnlimitedItemStack
dev.anvilcraft.lib.v2.util.stack.UnlimitedItemStack.splitUnlimited(int)
METHOD_REMOVED
✓ binary-compatible
✓ source-compatible

🔴 multiblock — 4 breaking change(s)

dev/anvilcraft/lib/v2/multiblock/dynamic/MultiblockCheckSnapshot.java:20

dev.anvilcraft.lib.v2.multiblock.dynamic.MultiblockCheckSnapshot
dev.anvilcraft.lib.v2.multiblock.dynamic.MultiblockCheckSnapshot.<init>(long,java.util.Map<net.minecraft.core.BlockPos,dev.anvilcraft.lib.v2.multiblock.dynamic.MultiblockCheckSnapshot$Entry>)
CONSTRUCTOR_REMOVED
✓ binary-compatible
✓ source-compatible

dev/anvilcraft/lib/v2/multiblock/dynamic/MultiblockCheckSnapshot.java:20

dev.anvilcraft.lib.v2.multiblock.dynamic.MultiblockCheckSnapshot
dev.anvilcraft.lib.v2.multiblock.dynamic.MultiblockCheckSnapshot.controllerPosLong()
METHOD_REMOVED
✓ binary-compatible
✓ source-compatible

dev/anvilcraft/lib/v2/multiblock/dynamic/MultiblockState.java:47

dev.anvilcraft.lib.v2.multiblock.dynamic.MultiblockState
dev.anvilcraft.lib.v2.multiblock.dynamic.MultiblockState.<init>(net.minecraft.core.BlockPos,net.minecraft.core.Holder$Reference<dev.anvilcraft.lib.v2.multiblock.dynamic.definition.MultiblockDefinition>,boolean)
CONSTRUCTOR_REMOVED
✓ binary-compatible
✓ source-compatible

dev/anvilcraft/lib/v2/multiblock/dynamic/MultiblockState.java:38

dev.anvilcraft.lib.v2.multiblock.dynamic.MultiblockState
dev.anvilcraft.lib.v2.multiblock.dynamic.MultiblockState.<init>(net.minecraft.core.BlockPos,net.minecraft.core.Holder<dev.anvilcraft.lib.v2.multiblock.dynamic.definition.MultiblockDefinition>)
CONSTRUCTOR_REMOVED
✓ binary-compatible
✓ source-compatible

🔴 wheel — 3 breaking change(s)

dev/anvilcraft/lib/v2/wheel/client/gui/component/WheelWidget.java:143

dev.anvilcraft.lib.v2.wheel.client.gui.component.WheelWidget
dev.anvilcraft.lib.v2.wheel.client.gui.component.WheelWidget.<init>(int,int,int,int,float,float,int,int,int,int,int,int,float,int,float,java.util.List<dev.anvilcraft.lib.v2.wheel.client.gui.component.WheelWidget$RawSection>,int)
CONSTRUCTOR_REMOVED
✓ binary-compatible
✓ source-compatible

dev/anvilcraft/lib/v2/wheel/client/gui/component/WheelWidget.java:36

dev.anvilcraft.lib.v2.wheel.client.gui.component.WheelWidget
dev.anvilcraft.lib.v2.wheel.client.gui.component.WheelWidget.<init>(int,int,int,int,net.minecraft.network.chat.Component,float,float,int,int,int,int,int,int,float,int,float,float,java.util.List<dev.anvilcraft.lib.v2.wheel.client.gui.component.WheelWidget$RawSection>,int)
CONSTRUCTOR_REMOVED
✓ binary-compatible
✓ source-compatible

dev/anvilcraft/lib/v2/wheel/client/gui/component/WheelWidget.java:223

dev.anvilcraft.lib.v2.wheel.client.gui.component.WheelWidget
dev.anvilcraft.lib.v2.wheel.client.gui.component.WheelWidget.getSectionCircleDiameter()
METHOD_REMOVED
✓ binary-compatible
✓ source-compatible

Full CSVs: see the Artifacts section of this workflow run.

[Gugle2308]

All scans complete. Compiling the review — this ready_for_review trigger matches the known-stable diff (327 files / 25,592 lines, same as last session), but with a major state change: CI is now green (mergeable_state: clean, previously unstable) and the PR is out of draft.

代码审查摘要 — PR #97

操作: ready_for_review（PR 已从 draft 转为就绪，CI 全绿 ✅）
方向: dev/26.1 → port/1.21.1（反向移植，base=port/1.21.1，head=dev/1.21.1）
范围: 327 个文件（259 Java，221 新增，0 删除）/ 25,592 行 diff

✅ 健康度扫描（全量重跑）

检查项	结果
反向 API 残留（Identifier / org.jspecify / ResourceHandler / Capabilities.Fluid / GpuDevice / RenderPipeline）	0 处（Java 代码，含注释命中全排除）
1.21.1 正常用法（ResourceLocation 39 / org.jetbrains 59 / builtInRegistryHolder 9）	全部合法，无误报
Ghost 文件（无 hunk 的 diff 头）	0
Mixin 6 配置 ↔ 类文件	全匹配（仅 font 有实际条目 GuiGraphicsExtractorMixin ✅，其余 5 个空数组 = 模板残留，无害）
jarJar 链（main 聚合 12 模块，api(project) + api(latest.release) 成对）	完整
@mod dist 分离（font/rpc/space-select/sync 均 Dist.CLIENT 变体）	正确
interface_injections	仅 font 实际接线（from + publish 成对，其余模块注释模板残留）
settings.gradle ↔ modules.json 三方一致	✅（8 新模块 include + 17 模块依赖声明）
CI workflow 引用（build_and_test / publish_maven_central / ci / release / pull_request / roseau_check / roseau_comment）	全部 200（按 head SHA f107a889 验证）
lang 键对称	font 6/6、explosion 7/7 ✅
EOF 缺换行	29 处，归因全为 package-info.java（Java 惯例无害）、lang/mixins JSON 及少量新文件

🔴 关键（待验证项，非新问题）

• SyncClassProcessor（ICoreMod）启动时序 —— 仍是唯一未销账的运行风险（V-9/V-10）
  SyncTargetIndex 静态块直接调 LoadingModList.get().getModFiles() 构建注解索引，targets() 依赖该索引求值。若 targets 在索引构建前被求值 → 空目标集 → @Sync 静默不注入 → 运行时 NPE。代码中未见 try/catch 防御性改写。
  CI 转绿只能证明编译与单元测试通过，覆盖不到 ModLauncher 早期阶段的注入路径。 合并前请至少跑一次 runClient 启动冒烟，确认 @Sync 注解类被实际注入（或补防御性改写：静态块容错 + 允许空索引）。

⚠️ 警告

• roseau.gradle 配置期网络请求（L-1，部分缓解） — 已重写为 try/catch + fallback，不再阻塞构建，但根 build.gradle 仍对每个子项目在配置期发 2 仓库请求。建议改 lazy Provider 或仅任务执行时解析（非阻塞）。

🟢 看起来不错

• 反向 API 残留 0，移植方向干净；最近提交的 revert（HolderGetter 重构、BlockBuilder lang 修复、integration 拆分）均为「26.1 专属改动，1.21.1 无需移植」的正确取舍
• 空注解体系迁移至 jsr305（@FieldsAreNonnullByDefault 等），符合 1.21.1 生态
• CI 矩阵化（modules.json + generate-matrix.js Kahn 分层）+ Roseau 检查基础设施，构建链比此前更健壮

📋 声称验证

PR body 为 null（todo 文档在 PR 历史内 add+remove 净零，未进入 base），验证转向结构检查：模块分布（registrum 44 / font 34 / sync 28+9 / rpc 28 / recipe 26 / wheel 23 / test 22 / space-select 21 / multiblock 13 / explosion 13 / util 11 / config 8 / network 6 / collision 5 / moveable-entity-block 3 / integration 2 / codec 2）与历次 todo 映射一致。

结论: COMMENT — 本 PR 已到合并门槛的最后一步：CI 全绿、结构检查全过、API 健康度干净。剩余唯一阻塞点是 V-9/V-10 runClient 启动冒烟（ICoreMod @sync 注入时序无法被 CI 覆盖）。跑一次本地 runClient 确认注入正常即可合并；建议完成后以评论销账。

---

由 Hermes Agent 审查

[Gugle2308]

💾 Self-improvement review: Patched SKILL.md in skill 'reverse-port-pr-review' (1 replacement). · Patched references/reverse-port-review-detail.md in skill 'reverse-port-pr-review' (1 replacement).

[Gugle2308]

PR #97 已关闭（closed），按 webhook 约束停止审查，不发布评论。

操作: closed — 跳过审查流程（closed/labeled 事件不触发代码审查）。