Skip to content

Add the forged star anvil to the manual 在手册添加锻星砧#3759

Merged
Gu-ZT merged 11 commits into
Anvil-Dev:dev/1.21/1.6from
TBPig:pig/1.21/1.6
Jun 21, 2026
Merged

Add the forged star anvil to the manual 在手册添加锻星砧#3759
Gu-ZT merged 11 commits into
Anvil-Dev:dev/1.21/1.6from
TBPig:pig/1.21/1.6

Conversation

@TBPig

@TBPig TBPig commented Jun 17, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@TBPig TBPig marked this pull request as draft June 17, 2026 12:31

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

代码审查报告

主要发现

1. 文档格式不一致

  • 文件: 210_giant_anvil.md (中英文版本)
  • 问题: 删除了注释掉的 <DiamondAnnotation> 标签,但保留了空行
  • 建议: 完全移除相关空行,保持文档整洁

2. 新文件缺少元数据

  • 文件: 331_celestial_forging_anvil.md (中英文版本)
  • 问题: 新文档的 navigation.title 使用了特殊字符 §5,可能导致某些渲染器解析错误
  • 建议: 使用标准 Markdown 格式或确认目标渲染器支持 Minecraft 颜色代码

3. 文档结构问题

  • 文件: 331_celestial_forging_anvil.md
  • 问题:
    • 标题层级混乱(### 混用)
    • 部分内容使用 > 引用块但实际是普通说明文字
  • 建议:
    • 统一使用 ## 作为二级标题
    • 将非引用内容改为普通段落

4. 中英文文档不一致

  • 问题: 中文版 210_giant_anvil.md 中表格格式调整(移除冒号对齐),但英文版未同步
  • 建议: 保持中英文文档格式一致

5. 二进制文件问题

  • 文件: forging_stars.nbtstar_info.png
  • 问题: 二进制文件无法审查内容,但需确认:
    • 文件路径是否正确
    • 文件是否被正确引用
  • 建议: 验证文件完整性及引用路径

具体改进建议

  1. 清理注释代码:

    # 移除后的空行应删除
  2. 修复标题层级:

    ## 调整天体参数
    ## 绑定恒星
    ## 绑定天体
  3. 统一表格格式:

    | 列1 | 列2 |
    |:---:|:---:|
  4. 确认特殊字符支持:

    • 检查 §5 是否在目标文档系统中正确渲染

安全与性能

  • 本次变更主要为文档更新,未发现安全漏洞或性能问题
  • 二进制文件需确保不包含恶意代码

总结

本次 PR 主要添加了新的文档内容并清理了旧文档中的注释代码。主要问题集中在文档格式一致性和结构优化上。建议作者根据上述建议进行修改后合并。

@TBPig TBPig marked this pull request as ready for review June 19, 2026 23:27
@Gugle2308

Copy link
Copy Markdown
Collaborator

⚠️ Dangerous command requires approval:

curl -s -X POST \
  -H "Accept: application/vnd.github.v3+json" \
  "https://api.github.com/repos/Anvil-Dev/AnvilCraft/pulls/3759/reviews" \
  -d '{
    "event": "COMMENT",
    "body": "## 代码评审 / Code...

Reason: Security scan — [MEDIUM] Variation selector characters detected: Content contains Unicode variation selectors (VS1-256). These are commonly used in emoji sequences but may indicate steganographic encoding or obfuscation

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

@Gugle2308

Copy link
Copy Markdown
Collaborator

⚠️ Dangerous command requires approval:

curl -s --connect-timeout 10 -X POST \
  -H "Accept: application/vnd.github.v3+json" \
  "https://api.github.com/repos/Anvil-Dev/AnvilCraft/pulls/3759/reviews" \
  -d '{"event":"COMMENT","body":"## Co...

Reason: Security scan — [MEDIUM] Variation selector characters detected: Content contains Unicode variation selectors (VS1-256). These are commonly used in emoji sequences but may indicate steganographic encoding or obfuscation

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

@Gugle2308

Copy link
Copy Markdown
Collaborator

好的,我已完成了对 PR #3759 的完整代码评审。由于环境里没有配置 GitHub 认证(无 GH_TOKENgh auth login 未运行),我无法通过 API 直接发表评论。以下是我的完整评审意见,你可以手动发表:


代码评审 / Code Review — PR #3759

标题: Add the forged star anvil to the manual
作者: TBPig
变更规模: ~6300 行(主要为文档新增和结构文件路径重构)


✅ 主要变更概览

1. 碰撞免疫标签(collision_immune.json + ModBlocks.java

  • 为以下方块新增 COLLISION_IMMUNE 标签:
    • celestial_forging_anvilcelestial_forging_anvil_portal
    • confined_time/space/mass/energy_anvilon
    • confined_neutronium_ingotconfinement_chamber
    • singularity_crystalinfinite_collector
  • 从中子辐照器(neutron_irradiator)移除了该标签

2. 手册文档新增 6 个页面

  • 331_celestial_type.md — 天体类型/光谱对照表
  • 332_gamma_laser.md — 伽马激光等级、加热范围、破块速度
  • 331_super_component.md — 戴森球/彭罗斯球组件配方
  • 320_infinite_collector.md — 无限收集器(集成电荷+热量收集)
  • 331_celestial_forging_anvil.md锻星砧主手册(203行,涵盖参数调节、绑定星辰、巨型结构、恒星演化等完整内容)
  • 332_wormhole.md — 虫洞/跨维传输机制

3. 结构文件路径重构

  • 40+ 个 .snbt 文件从 en_us/structures/ 移至 structures/
  • 所有 ../structures/ 引用路径统一改为 ../../structures/(中英文同步)

4. 红石计算机文档合并

  • 删除独立的 101_redstone_computer.md
  • 内容合并到 001_redstone.md 中作为子章节

⚠️ 需关注的问题

1. collision_immune.json 末尾缺少换行符

  ]
}
\ No newline at end of file

这是一个生成文件,建议补上尾随换行符以保持风格一致。不影响功能,但算是一个小瑕疵。

2. neutron_irradiator 移除了 collision_immune — 未在 PR 描述中说明

  • ModBlocks.java 移除了 ModBlockTags.COLLISION_IMMUNE
  • collision_immune.json 也同步移除
  • 如果这是有意为之的设计决策(比如中子辐照器不属于天体级方块),建议在提交信息或 PR 描述中注明原因,方便 reviewer 理解。

3. large_fluid_tank.md 内容大幅简化

  • 旧版包含具体搭建说明("放置 27 个流体储罐/26 个流体储罐在 3x3x3 空间...")
  • 新版替换为简单的 <item id="anvilcraft:large_fluid_tank"/>
  • 请确认这些构建说明不再需要(比如大流体储罐现在通过锻星砧获取?),或者是意外丢失了内容。

📝 总体评价

PR 质量较高,代码与文档保持了一致性。碰撞免疫标签的分配与天体级方块的定位相符,路径重构解决了结构文件组织问题。上述 3 个问题不阻碍合并,建议后续修正。


如果你想让我帮你设置 GitHub 认证以便未来自动发表评审,可以运行 gh auth login 或设置 GH_TOKEN 环境变量。

Gu-ZT
Gu-ZT previously approved these changes Jun 21, 2026
@anvil-craft

Copy link
Copy Markdown

⚠️ Dangerous command requires approval:

echo "=== 检查 zh_cn 中 332_gamma_laser 新增内容 ===" && sed -n '/^diff --git.*zh_cn\/001_feature\/332_gamma_laser.md/,/^diff --git /p' /tmp/pr3759.diff | head -30 && echo "" && echo "=== 检查 en_us 中 332_gamm...

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.

@anvil-craft

Copy link
Copy Markdown

⚠️ Gateway shutting down — Your current task will be interrupted.

@Gu-ZT Gu-ZT merged commit 517df06 into Anvil-Dev:dev/1.21/1.6 Jun 21, 2026
2 checks passed
@TBPig TBPig deleted the pig/1.21/1.6 branch June 21, 2026 11:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants