Compatible with Youer Server 兼容 Youer 端

[WhereisFff]

No description provided.

[Gugle2308]

⚠️ Dangerous command requires approval:

cd /tmp && curl -sL -H "Accept: application/vnd.github.raw" "https://api.github.com/repos/Anvil-Dev/AnvilLib/git/blobs/a48a5e92e062f249ba3f830bdd9e99d970c4f0f5" | head -60; echo "=== roseau.yaml ===";...

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 #135 状态为 closed，按约束 1 跳过审查，不发布任何评论。

（无需 diff 分析、无需 gh pr edit；本次会话到此结束。）

[Gugle2308]

已完成核查（含 base 树 sha 对齐 + Youer 1.21.1 真实源码交叉验证）。以下为审查意见：

---

代码审查摘要 — PR #135

操作: opened（另经 HTML 复核：data-status="pullOpened"，未合并、无 draft、mergeable: true）
范围: 2 文件 / 57 行 diff（1 个新增 Java、1 个 mixins.json 修改；ghost 0、删除 0、EOF 缺失 0）
diff 完整性对账: 本地 md5 183293e6...、diff --git 计数 2、+39/-1 与 API changed_files=2 / additions=39 / deletions=1 完全一致 → 无静默截断
核查基准: 拉取 PR base 树（892ef231 = API base.sha）+ Youer 官方仓库 MohistMC/Youer 分支 1.21.1 源码逐点比对（PaperUnsupportedSettings.java、patches/.../PistonBaseBlock.java.patch）

✅ 注入点验证（逐项，附证据）

检查项	结果	证据
@Pseudo 目标类存在	✅	Youer 1.21.1 分支确有 src/main/java/com/mohistmc/youer/util/PaperUnsupportedSettings.java
目标方法存在且为静态	✅	该文件 L27 public static void end_allowPistonDuplication(BlockPos, List<BlockPos>, int, Level, Map<BlockPos,BlockState>, BlockState, List<BlockState>, Direction, boolean) → handler 必须 static，写法正确
注入点唯一 / 无需 ordinal	✅	该方法体内仅 L33 一处 p_60182_.setBlockEntity(...)
at 描述符匹配	✅	L33 p_60182_.setBlockEntity(MovingPistonBlock.newMovingBlockEntity(...)) → Lnet/minecraft/world/level/Level;setBlockEntity(Lnet/minecraft/world/level/block/entity/BlockEntity;)V，与 diff 一字不差
被读取的前置 BE 确实存在	✅	PistonBaseBlock 补丁 L142-144 顺序为 setBlock(blockpos3, MOVING_PISTON, 68) → vanilla setBlockEntity(newMovingBlockEntity(...)) → end_allowPistonDuplication(...)。故 wrap 执行时 level.getBlockEntity(pos) 拿到的正是「刚放入、且已被 PistonBaseBlockMixin 的 newMovingBlockEntity wrap 填好 carried BE」的 PMBE_A，条件链成立
原版 setBlock 与 movedState 语义不冲突	✅	!allowDesync 分支取 iblockdata1 = level.getBlockState(oldPos)，而 oldPos 是 .relative(direction) 之前的源位置（补丁 L138/139），读到的是被推动方块的原始状态 → getMovedState().getBlock() instanceof IMoveableEntityBlock 守卫对 AnvilCraft 方块成立
接口/方法可用	✅	interface_injections.json 把 IPistonMovingBlockEntityExtension 注入 PistonMovingBlockEntity，anvillib$get/setBlockEntity 存在
注册正确	✅	mixins.json 加入 common mixins 数组（非 client），package 一致
类型 API 存在	✅	PistonMovingBlockEntity#getMovedState()、BlockEntityType#isValid(BlockState)、BlockEntity#getBlockPos() 在 1.21.1 均存在（且 Level#setBlockEntity 亦被本 PR 同模块 PistonMovingBlockEntityMixin 使用）

结论：修复动机与注入点均正确。 该 compat mixin 解决的是一个真实缺口：Youer 把 newMovingBlockEntity 的调用搬进了 PaperUnsupportedSettings，导致本模块 PistonBaseBlockMixin 的 @WrapOperation(newMovingBlockEntity, ordinal=0) 只作用于 L143 那次（会被 L144 覆盖的）调用，而真正生效的 L144 新 PMBE 拿不到 carried BE → 推动到位后 BE 丢失。

⚠️ 警告

1. defaultRequire: 1 且未设 require = 0 — 版本漂移会崩服
   anvillib_moveable_entity_block.mixins.json 显式声明 "injectors": { "defaultRequire": 1 }，@Pseudo 只兜「类不存在」，不兜「类在但方法改名/被删」。end_allowPistonDuplication 是第三方 fork 的私有内部名，无稳定性承诺；一旦 Youer 重命名或重构该方法，所有装载 AnvilLib 的 Youer 服会在启动阶段 MixinApplyError 崩服，而不是退化为「不生效」。建议：

   @WrapOperation(method = "end_allowPistonDuplication", require = 0, at = @At(...))

   让漂移只损失兼容性、不阻断开服（代价是静默失效，可在注释里注明）。

2. 隐式依赖链需注释固化
   本修复成立的前提是「Youer 在 end_allowPistonDuplication 之前仍保留 vanilla 的 setBlockEntity(newMovingBlockEntity(...))」（PistonBaseBlock 补丁 L143）。该调用在当前 Youer 里是被 L144 覆盖的冗余调用——将来 Youer 若顺手删掉它，则 level.getBlockEntity() 返回 null，同时 AnvilLib 的 newMovingBlockEntity wrap 也失去目标（defaultRequire=1 → 崩服），两处同时失效且难以定位。建议：在 mixin 顶部加注释写明依赖的 Youer 版本/构建与「必须先有 vanilla setBlockEntity」这一前提。

💡 建议

• 转移语义建议改用 anvillib$clearBlockEntity()：模块内其它三处（PistonBaseBlockMixin 的 share 传递、PMBEMixin.tick/finalTick）都是「清空并接管」的单所有权语义；这里用 getBlockEntity() 会让新旧两个 PMBE 同时持有同一 BE 实例。当前旧实例会被同位置的 setBlockEntity 覆盖、不会重复 tick/存档，所以不是 bug，但换成 clear 更稳、也更一致。
• replacement instanceof IPistonMovingBlockEntityExtension replacementExtension 冗余：replacement 已作为 moving（PMBE）匹配，而该接口正是注入在 PistonMovingBlockEntity 上的，可合并判断，少一层 pattern 匹配。
• carried.getType().isValid(moving.getMovedState()) 未通过时会静默丢 BE：主路径（PistonMovingBlockEntityMixin）没有这个校验，仅此处有；若真出现不匹配（方块/BE 类型注册不一致），结果是玩家方块实体无声消失、无任何日志。建议不通过时补一条 warn 日志。
• 命名一致性：模块内 wrap handler 惯例跟随被包裹目标（isPushable、setBlock、newMovingBlockEntity），preserveBlockEntity 既非目标方法名也非被包裹方法名；建议改为 setBlockEntity。（PistonBaseBlockMixin.setBlock 那种「名字≠目标方法」的 hanlder 都带了 @SuppressWarnings("NameDoesntMatchTargetClass")，本次未加；如无 IDE 检查门禁可忽略。）
• PR 描述为空：建议补上验证环境（Youer 版本/构建号）与 unsupported-settings.allow-piston-duplication 取 true/false 两种配置下的实测结果——两种配置都会走 end_allowPistonDuplication 的 setBlockEntity，都应覆盖到。

🟢 看起来不错

• Youer 特有逻辑被完整隔离进 @Pseudo mixin，vanilla/NeoForge 路径零侵入。
• 注入范围用 targets= 精确锁定到 Youer 类，不会误伤 PistonBaseBlock.moveBlocks 中 L143 那次 vanilla 调用。
• level.getBlockEntity(...) 无额外同步加载/死锁风险：同一位置的 chunk 已被紧邻其前的 vanilla setBlockEntity touch 过。
• 只在 original.call 之前读取、之后再提交，时序正确（若放到之后，旧 PMBE 已被覆盖，就取不到了）。
• CI 会因 module.*/src/** 命中而自动跑 build + roseau 门禁。

📋 声称验证表

声称	状态	对应实现
«兼容 Youer 端»	✅	YouerPistonCompatibilityMixin（注入点已对 Youer 1.21.1 源码核实）+ mixins.json 注册

结论: COMMENT（实质倾向 APPROVE） — 逻辑与注入点经真实源码验证正确，无功能性缺陷；建议合并前处理「require = 0（防 Youer 漂移崩服）」与「依赖注释」两项，其余为可选优化。

🧪 测试建议

被测目标	推荐场景	优先级
YouerPistonCompatibilityMixin.preserveBlockEntity	Youer 1.21.1 上推/拉 AnvilCraft 可移动 BE 方块（含带库存/多格），到位后校验库存与 NBT 未丢；allow-piston-duplication = true / false 各跑一遍	🔴
非 Youer 环境（NeoForge 原版服务端/客户端）	启动不崩、@Pseudo 正常跳过（检查告警日志不含异常）	🔴
跨区块推动	跨 chunk 边界推动 BE 方块不丢 BE、无同步加载死锁	🟡
BE 类型不匹配	人为让 BlockEntityType#isValid 返回 false，确认是「丢 BE」而非崩溃	🟢

标题

gh auth status 显示 token 已失效（The token in /opt/data/home/.config/gh/hosts.yml is invalid），按规范未执行 gh pr edit（也不会去 export token）。现标题 Compatible with Youer Server 兼容 Youer 端 具备中英双语、不算错误，但建议改为更具体的：

Fix moveable block entity loss on Youer server 修复 Youer 端活塞推动可移动方块实体丢失

---

由 Hermes Agent 审查

[Gugle2308]

💾 Self-improvement review: Patched SKILL.md in skill 'mixin-injection-verification' (1 replacement).