fix(registrum): 保持创造栏物品集合可变

[WhereisFff]

• fixed [Bug] AnvilLib makes CreativeModeTab entries immutable #110

[Gugle2308]

⚠️ Dangerous command requires approval:

curl -sL "https://api.github.com/repos/Anvil-Dev/AnvilLib/pulls/111/commits" -o /tmp/pr111_commits.json && python3 -c "
import json
data = json.load(open('/tmp/pr111_commits.json'))
for c in data:
   ...

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 head 相对基线只有 2 个文件的真实变更，mixin 未动。结论已完整：

核心事实链：

1. PR fix(registrum): 保持创造栏物品集合可变 #111（WhereisFff）：arrange/fold 返回可变集合，fold 空组早退返回 new ArrayList<>(original)。但 mixin 没改——arrange 的 layout==null 早退路径仍返回原始传入集合，而该传入集合是原版 displayItems（可变性取决于原版字段，通常是可变 ArrayList），所以那一路不构成新问题。
2. 关键盲区：fold 的 eligible.isEmpty() 早退现在返回 new ArrayList<>(original)，但 mixin 调用 fold(this.displayItems) 时传的是原版 displayItems 本身。这个新 ArrayList 与原 displayItems 无关，fold 内对 original 只读、折叠结果也来自 group.source()/原栈拷贝——但 arrange 紧随其后把整个集合重新赋给 displayItems，所以中间的可变/不可变差异被抹平了。真正要紧的是最终赋给 displayItems 的集合必须可变，而 PR 只改了 arrange 的非早退路径。
3. 最大的坑：Gugle 已在本地 bb260c8 用「mixin 层 new ArrayList<>(...) 包装」修复同一问题（commit message 明确提到 bits_n_bobs 的 removeIf 崩溃），该提交尚未合入 dev/1.21.1。PR fix(registrum): 保持创造栏物品集合可变 #111 的修复位置（工具方法内部）与 bb260c8 不同——如果两者都合入，会双重包装，且 PR fix(registrum): 保持创造栏物品集合可变 #111 未覆盖 arrange 的早退路径，修复不完整。存在提交冲突/重复修复风险，需协调确认。

审查意见已成型，直接输出（webhook 自动投递，不执行任何 gh 发表命令）。标题「fix(registrum): 保持创造栏物品集合可变」准确，无需修改。

代码审查摘要 — PR #111

操作: opened
范围: 2 个文件 (2 Java) / 35 行 diff
PR: fix(registrum): 保持创造栏物品集合可变 — WhereisFff → fff/1.21.1-fix3 → dev/1.21.1

变更内容

将 CreativeTabSections.arrange() 与 CreativeVariantPickerRegistry.fold() 的返回集合从 List.copyOf(...)（不可变）改为可变集合，目的是保持 CreativeModeTab.displayItems 可变，兼容其他 mod（如 bits_n_bobs）在同一 mixin 注入点对 displayItems 调用 removeIf 的操作，避免 UnsupportedOperationException 崩溃。

🔴 关键问题

1. 与已有未合并修复重复/冲突（需协调）

   • 本地克隆 AnvilLib 的 bb260c8（Gugle，2026-08-29）提交了同一问题的修复，但方案是改 CreativeModeTabMixin：this.displayItems = new ArrayList<>(fold(...)) / new ArrayList<>(arrange(...))，commit message 明确提到「bits_n_bobs 等 mod 的 mixin 对 displayItems 调用 removeIf 崩溃」。
   • 该提交尚未合入 dev/1.21.1（git merge-base --is-ancestor bb260c8 origin/dev/1.21.1 = NO）。
   • 本 PR 走的是「工具方法内部返回可变集合」路线，与 bb260c8 的「mixin 层包装」路线同时合入会双重包装（虽然无害，但冗余），且本 PR 未覆盖 arrange 的 layout == null 早退路径——两条路线需取舍，建议与 Gugle 协调：要么只合本 PR（并在 mixin 补 new ArrayList<> 兜底），要么只合 bb260c8。

2. arrange() 的 layout == null 早退路径未覆盖

   • if (layout == null) return items; 仍直接返回传入集合。当前基线 mixin 是 this.displayItems = CreativeTabSections.arrange(tabId, this.displayItems)，若该 tab 无 layout，返回的就是原始 displayItems——它本身可能是可变集合（原版字段），不构成新崩溃，但可变性契约依赖原版字段类型，未显式保证。若按 bb260c8 思路在 mixin 统一包 new ArrayList<>，则此路径也覆盖；本 PR 未处理。

⚠️ 警告

3. fold() 的 eligible.isEmpty() 早退返回 new ArrayList<>(original) 的语义
   • 该分支本可直接 return original（original 本身就是 List.copyOf(items) 的新列表），现在包一层 new ArrayList<> 只是复制一份，无实际收益（不改变可变性：List.copyOf 结果仍不可变，new ArrayList<>(不可变) 才可变）。真正保证可变的是 new ArrayList<>(original) 这个包装本身，逻辑正确但表达冗余——建议直接 return new ArrayList<>(original) 或改为先 new ArrayList<>(items) 再返回。不影响正确性，仅为可读性。

💡 建议

4. arrange() 返回的 arranged 是 ArrayList，但未用 List.copyOf 包装 — 现在直接 return arranged，调用方（mixin）拿到的是共享的可变列表。目前 mixin 是唯一调用方且随即赋给 displayItems，安全；但若未来其他代码复用该方法并对结果做修改，可能意外污染。可考虑在方法内明确返回 new ArrayList<>(arranged) 或加注释说明「返回值保持可变以符合 displayItems 契约」。

🟢 看起来不错

• 改动极小（2 文件 3 处），方向正确：修复了 fold/arrange 非早退路径返回不可变集合导致其他 mod removeIf 崩溃的根因。
• fold 的 emitted 去重 + group.source() 拷贝语义未受影响，折叠行为保持。

📋 声称验证表

声称	状态	说明
保持创造栏物品集合可变	✅	arrange/fold 主路径返回可变集合
兼容其他 mod 的 removeIf	⚠️	主路径已覆盖，arrange 早退路径未覆盖，且与 bb260c8 重复

结论

REQUEST_CHANGES — 核心修复方向正确，但需：

1. 与 Gugle 的 bb260c8（mixin 层 new ArrayList<> 方案）协调去重——该提交已在本地但未合入 dev/1.21.1，两条路线只保留一条；
2. 补齐 arrange 的 layout == null 早退路径的可变性保证（或在 mixin 统一兜底）；
3. 简化 fold 空组早退的冗余包装。

如仅合入本 PR，建议 mixin 保持 new ArrayList<>(...) 包装作为统一兜底，覆盖所有路径。

---

由 Hermes Agent 审查

[Gugle2308]

💾 Self-improvement review: Patched SKILL.md in skill 'pr-diff-scope-reconciliation' (1 replacement).

[Gugle2308]

🌿 Roseau API Breaking Change Report

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

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

[Gugle2308]

PR #111 的 action 为 closed，按约束停止处理，不下载 diff、不执行审查、不发布评论。

[Gugle2308]

⚠️ Dangerous command requires approval:

curl -sL "https://api.github.com/repos/Anvil-Dev/AnvilLib/issues/110" | python3 -c "import json,sys; d=json.load(sys.stdin); print(d['body'])" 2>/dev/null || curl -sL "https://api.github.com/repos/Anv...

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/issues/110 — or: vet https://api.github.com/repos/Anvil-Dev/AnvilLib/issues/110 (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]

代码审查摘要 — PR #111

操作: edited（标题/描述变更）｜当前状态: 已合并（merged，2026-09-14，closes #110）
范围: 2 个文件（2 Java，+3 / −3，无新增/删除文件）/ 35 行 diff

🔍 审查依据（已交叉验证）

验证项	结果
CreativeModeTabMixin 中 displayItems 唯一写入路径	✅ 只有 fold() + arrange() 两处（grep displayItems 全仓仅 mixin 3 处）
其余 List.copyOf 是否仍流入 displayItems	✅ 否——LAYOUTS/PLACEMENTS/GROUPS 均为内部缓存，不对外暴露（placedSections() 返回不可变副本是有意为之）
返回列表是否与内部状态别名（可变性修复是否引入新 bug）	✅ arranged / folded / new ArrayList<>(original) 全是方法内新建对象，外部修改不会污染 LAYOUTS/PLACEMENTS
fix(fixed #110) 声称	✅ 成立，见下

#110 崩溃根因与修复对齐（crash-2026-08-30_09.11.15-client.txt）:

java.lang.UnsupportedOperationException
  at ImmutableCollections$AbstractImmutableCollection.clear
  at CreativeModeTab.handler$bcf000$fabric_item_group_api_v1$getStacks(CreativeModeTab.java:1579)
  at CreativeModeTab.buildContents(CreativeModeTab.java:148)

Fabric ItemGroup API v1（ItemGroupMixin#getStacks）在 updateEntries 的 @At("TAIL") 里对 displayStacks 执行 displayStacks.clear(); displayStacks.addAll(...) 回写。AnvilLib 的 @At("TAIL") 注入在同一注入点先跑，把字段替换成 List.copyOf(...) 的不可变列表 → Fabric 的 clear() 直接 UOE。本次改动将返回值改为可变集合，正好修复该路径。✅

补充验证：Fabric 侧是先 new LinkedList<>(displayStacks) 快照、事件回调后再 clear()+addAll() 写回——只要求字段集合可变，不要求保留原实例，因此本修复完全满足其契约；且两种 TAIL 顺序都安全（AnvilLib 先→Fabric 清空重填仍保留 AnvilLib 的排列与占位符；Fabric 先→AnvilLib 在含模组增补的列表上重排）。searchTabStacks AnvilLib 从未改写，仍是原版可变列表，不受影响 ✅。

⚠️ 警告

• CreativeTabSections.arrange() 的 if (layout == null) return items;（未改动行） — 该分支把调用方传入的集合原样返回，并不保证可变。目前调用链恰好安全（CATEGORY 页先经 fold() 返回可变列表，其他页拿到的是原版可变 ArrayList），但方法签名 Collection<ItemStack> 无法表达「必然可变」这一契约。若有外部模组直接调用该 public util（dev.anvilcraft.lib.v2.registrum.util 为公开包），传入不可变集合时同一类 UOE 会复现。建议改为 return new ArrayList<>(items); 与其余两条出口保持一致。

💡 建议

• CreativeTabSections.arrange() / CreativeVariantPickerRegistry.fold() — 缺少说明「返回集合必须可变」的 Javadoc/注释。本次修复靠的是隐式约定，将来一次「顺手加上 List.copyOf 做防御性拷贝」的重构就会让 [Bug] AnvilLib makes CreativeModeTab entries immutable #110 回归。建议补注释并引用 #110（例如 // 必须可变：Fabric ItemGroup API 会对该字段执行 clear()/addAll()，见 #110）。
• fold() 的 if (eligible.isEmpty()) return new ArrayList<>(original); — 因为 original 已是 List.copyOf(items)，此处必然多一次拷贝。可改为 List<ItemStack> original = new ArrayList<>(items); ... if (eligible.isEmpty()) return original;，省掉一次全量拷贝（每页重建触发一次，收益微小，仅作可选清理）。
• 缺少回归测试 — module.test 目前无任何针对 registrum 创造栏的测试。建议补一条守卫测试，锁死「可变」这一契约（模拟 Fabric 的用法）：

  Collection<ItemStack> result = CreativeVariantPickerRegistry.fold(List.of(stack));
  assertDoesNotThrow(() -> { result.clear(); result.addAll(List.of(stack)); });
  // 对 CreativeTabSections.arrange(tabId, items) 同样断言（含 layout == null 的分支）

• PR 标题不含英文 — 建议改为 fix(registrum): Keep creative tab item collections mutable 保持创造栏物品集合可变。本次 gh auth status 显示 /opt/data/home/.config/gh/hosts.yml 中 token 已失效（Active account: true 但 invalid），按流程不自行 export 凭据，故未自动修改。

🟢 看起来不错

• 改动最小且精准：只动 3 行，把「不可变」这一副作用去掉，未顺手改动排列/折叠逻辑，回归风险低。
• 内部缓存仍保留 List.copyOf 防御性拷贝（LAYOUTS、PLACEMENTS），没有为了「可变」而把内部状态也暴露出去——这是本次修复处理得最好的地方。
• fold() 早退分支特意补 new ArrayList<>(original)，说明作者清楚「只有从不可变源来才需要再拷一层」，不是简单地把 List.copyOf 全文替换掉。
• fixes #110 的声称与崩溃栈完全对应，非「碰巧改对了」。

结论: COMMENT（无阻塞项） — 修复正确且已被上游 crash log 证实，PR 已合并无需回滚。两条非阻塞建议（arrange 的空 layout 分支统一返回可变副本；补注释/测试防止 List.copyOf 回归）建议以 follow-up commit 形式补上。

---

由 Hermes Agent 审查

[Gugle2308]

💾 Self-improvement review: Patched SKILL.md in skill 'api-exposure-pr-review' (1 replacement). · Patched SKILL.md in skill 'creative-tab-pr-review' (1 replacement). · Patched references/pr-class-routing.md in skill 'github-webhook-pr-review' (1 replacement).