adjust model of rings

[XeKr]

No description provided.

[Gugle2308]

⚠️ Dangerous command requires approval:

curl -sL "https://api.github.com/repos/Anvil-Dev/AnvilCraft/pulls/3799" 2>&1 | python3 -c "
import json, sys
data = json.load(sys.stdin)
print('Title:', data.get('title'))
print('State:', data.get('st...

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/AnvilCraft/pulls/3799 — or: vet https://api.github.com/repos/Anvil-Dev/AnvilCraft/pulls/3799 (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]

⚠️ Dangerous command requires approval:

curl -s -X POST \
  -H "Accept: application/vnd.github.v3+json" \
  -H "Authorization: token $GH_TOKEN" \
  -d '{"body":"## 代码审查 — PR #3799: adjust model of rings\n\n**作者:** XeKr　|　**分支:** `assets/mod...

Reason: Security scan — [HIGH] Confusable Unicode characters in text: Content contains Unicode characters visually identical to ASCII (math alphanumerics, Cyrillic/Greek lookalikes) appearing near ASCII text, which may indicate a homoglyph attack

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

[Gugle2308]

代码审查 — PR #3799: adjust model of rings

作者: XeKr　|　分支: assets/model-adjust → dev/1.21/1.6

变更概述

修改了 4 个文件（3 个 JSON 模型 + 1 个纹理贴图）：

文件	说明
ring_4_penrose_sphere_laser.json	大幅重构：镜像元素位置、新增几何体、调整 UV 映射、旋转角度从 -45° 改为 +45°
ring_5_stellar_evolution_accelerator.json	所有臂/环元素向外扩展 +0.25 单位
ring_6_stellar_evolution_accelerator.json	所有臂/环元素向外扩展 +0.25 单位
celestial_forging_anvil_rings_x.png	二进制纹理文件同步更新

审查结论：✅ LGTM（无阻塞性问题）

1. 正确性 — JSON 结构合法，坐标变化在 Ring 5/6 中遵循一致的 +0.25 向外偏移模式；Ring 4 的镜像和新增几何体与纹理更新相配套，均为合理的模型视觉调整。
2. 安全性 — 仅修改 Minecraft 资源模型文件（JSON/Png），无代码逻辑变更，无安全风险。
3. "scope": 0 — Blockbench 编辑器属性，不影响游戏内渲染，属于编辑器遗留元数据，可放心保留。
4. Ring 4 的变更量较大 — 新增了多个元素块并调整了 UV 坐标，建议在游戏内实际加载确认视觉结果。

建议： 无阻塞性问题，可直接合并。

[Gugle2308]

评审意见已成功发布于 PR #3799。

评论链接： #3799 (comment)

审查结论速览

维度	结论
变更范围	4 个文件：3 个 JSON 模型 + 1 个纹理 PNG
正确性	✅ Ring 5/6 遵循一致的 +0.25 向外偏移；Ring 4 为镜像 + 新增几何体，配套纹理已更新
安全性	✅ 纯资源文件变更，无代码逻辑，无安全隐患
"scope": 0	✅ Blockbench 编辑器属性，不影响游戏内渲染
建议	无阻塞性问题，可直接合并

[anvil-craft]

Now I have enough data for a thorough review. Here is the analysis:

---

代码审查摘要 · PR #3799

标题: adjust model of rings
作者: XeKr
分支: assets/model-adjust → dev/1.21/1.6
状态: 已关闭（已合入）
差异规模: +882 / -356 行，4 个文件

---

文件变更总览

文件	类型	变更说明
...ring_4_penrose_sphere_laser.json	模型 JSON	重大重构 — 元素镜像、旋转方向反转、新增元素、分组层级重组
...ring_5_stellar_evolution_accelerator.json	模型 JSON	微调 — 所有元素向外偏移 0.25 单位
...ring_6_stellar_evolution_accelerator.json	模型 JSON	微调 — 所有元素向外偏移 0.25 单位
...celestial_forging_anvil_rings_x.png	纹理 PNG	二进制更新 — 纹理已配合模型调整修改

---

✅ 看起来不错

1. 变更范围清晰，类型一致
所有更改均为 src/main/resources/assets/ 下的模型/纹理文件，完全没有涉及 Java 代码、数据生成器、配方或语言文件。这表明本次 PR 是纯粹的视觉资产调整，不改变任何游戏逻辑。

2. Ring 4 — 模型大幅优化

• 旧模型有 76 个元素（索引 0–75），新模型扩展到 108 个元素（索引 0–107），新增了 32 个元素，环面更密集、细节更丰富。
• 元素坐标从正 X 轴镜像到负 X 轴（例如 [1.5, -0.025, -2.125] → [-5.75, -0.025, -2.625]），同时旋转角度反转（-45° → 45°），表明环的整体朝向进行了重新设计。
• UV 映射同步调整，纹理坐标顺序翻转（例如 [0.75, 0.25, 1, 2.5] → [0.75, 2.5, 1, 0.25]），确保纹理方向正确。
• 分组结构大幅优化：原来第 3 个 group 直接包含 16 个元素（children: [60–75]），现在拆分为 8 个 subgroup × 6 个元素的层级结构。这种分组有利于 Blockbench 中按段独立编辑动画或旋转。

3. Ring 5 & 6 — 统一向外扩张 0.25 单位

• 所有元素的 from/to 坐标均沿轴向向外移动了 0.25 单位（例如 [7.25] → [7.5]、[8.25] → [8.5]），内外环的 cube_outline 也同步位移。
• 这种一致性的偏移说明是整体环径的放大调整，使得星体加速器的环圈看起来更大/更厚。

4. "scope": 0 属性统一新增
3 个 JSON 文件中共新增了 31 处 "scope": 0。这是 Blockbench 导出时的标准行为——scope 控制变换作用域，0 表示作用于所有子元素。这个是工具自动生成的，无安全隐患。

5. 文本结构验证

• JSON 文件末尾修复了 missing newline（\ No newline at end of file 修复为正常结尾）。
• 所有 group 元素均正确配置了 "shade": false，环面元素不会受光照 shading 影响，符合模型资产惯例。

---

🔍 审查清单快速检查

类别	结果
正确性	✅ 坐标、旋转、UV 映射一致性良好，分组索引无重叠/遗漏
安全性	✅ 纯资产文件，无代码，无注入风险
代码质量	✅ JSON 格式规范，分组层级合理，符合 Blockbench 导出标准
测试	✅ 无需测试，资产变更不影响游戏逻辑
性能	⚠️ 环 4 元素从 76 增加到 108（+42%），渲染顶点数增加，但在 Minecraft 现代渲染引擎下影响极小
文档	✅ 资产变更无需 README 更新

---

💡 备注

• 本次 PR 是铁砧工艺模组天体锻造环系列的视觉优化：环 4（潘罗斯球面激光器）进行了大幅重构，环 5/6（恒星演化加速器）做了统一放大调整。作为 XeKr 主导的纯资产调整 PR，代码层面无任何问题。
• 由于已合入且无 Java 代码变更，无需请求更改或额外操作。