Commit 1be4e4e
authored
提取Minecraft版本列表到外部文件 (#123)
## 工作总结
本次任务成功将硬编码的 Minecraft 版本列表从 `rule/datapack.bzl`
中提取到外部文件,实现了动态加载支持。具体更改如下:
### 主要变更
1. **创建外部版本文件** `rule/minecraft_versions.bzl`
- 将原有的 `_ALL_MINECRAFT_VERSIONS` 硬编码列表完整迁移到独立文件
- 将变量名改为 `ALL_MINECRAFT_VERSIONS` 以符合 Bazel 的公共符号规范
- 添加了文件描述注释说明其用途
2. **修改主构建文件** `rule/datapack.bzl`
- 添加了 `load(":minecraft_versions.bzl", "ALL_MINECRAFT_VERSIONS")` 导入语句
- 删除了原有的硬编码版本列表(第211-258行)
- 将所有对 `_ALL_MINECRAFT_VERSIONS` 的引用更新为 `ALL_MINECRAFT_VERSIONS`
### 技术细节
- **变量可见性**:由于 Bazel 不允许导入以下划线开头的私有符号,将变量名从 `_ALL_MINECRAFT_VERSIONS` 改为
`ALL_MINECRAFT_VERSIONS`
- **向后兼容性**:`minecraft_versions_range()` 函数和其他依赖版本列表的功能保持不变
- **构建验证**:通过 `bazelisk build --nobuild
//subprojects/stone-disappearance:stone-disappearance` 验证了构建系统正常工作
### 影响范围
- **构建系统**:所有数据包的构建配置现在从外部文件加载 Minecraft 版本列表
- **维护性**:未来更新 Minecraft 版本只需修改 `rule/minecraft_versions.bzl`
文件,无需编辑主构建规则
- **动态支持**:为后续实现自动版本检测或从远程源加载版本信息奠定了基础
### 提交信息
已按照项目规范创建提交:`♻️ refactor: extract hardcoded Minecraft versions to
external file`
此项改进实现了问题的核心要求,将硬编码的 Minecraft
版本列表(datapack.bzl:11-57)成功提取到外部数据文件,支持了动态加载能力,同时保持了与现有构建系统的完全兼容性。
Closes #113
<a href="https://opencode.ai/s/Etz8Blcq"><img width="200"
alt="New%20session%20-%202026-01-24T17%3A05%3A52.555Z"
src="https://social-cards.sst.dev/opencode-share/TmV3IHNlc3Npb24gLSAyMDI2LTAxLTI0VDE3OjA1OjUyLjU1NVo=.png?model=deepseek/deepseek-reasoner&version=1.1.34&id=Etz8Blcq"
/></a>
[opencode
session](https://opencode.ai/s/Etz8Blcq) | [github
run](/TouchController/E1epack/actions/runs/21318549101)2 files changed
+97
-55
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
20 | 20 | | |
21 | 21 | | |
22 | 22 | | |
| 23 | + | |
23 | 24 | | |
24 | 25 | | |
25 | 26 | | |
| |||
271 | 272 | | |
272 | 273 | | |
273 | 274 | | |
274 | | - | |
275 | | - | |
276 | | - | |
277 | | - | |
278 | | - | |
279 | | - | |
280 | | - | |
281 | | - | |
282 | | - | |
283 | | - | |
284 | | - | |
285 | | - | |
286 | | - | |
287 | | - | |
288 | | - | |
289 | | - | |
290 | | - | |
291 | | - | |
292 | | - | |
293 | | - | |
294 | | - | |
295 | | - | |
296 | | - | |
297 | | - | |
298 | | - | |
299 | | - | |
300 | | - | |
301 | | - | |
302 | | - | |
303 | | - | |
304 | | - | |
305 | | - | |
306 | | - | |
307 | | - | |
308 | | - | |
309 | | - | |
310 | | - | |
311 | | - | |
312 | | - | |
313 | | - | |
314 | | - | |
315 | | - | |
316 | | - | |
317 | | - | |
318 | | - | |
319 | | - | |
320 | | - | |
321 | | - | |
| 275 | + | |
322 | 276 | | |
323 | 277 | | |
324 | 278 | | |
| |||
337 | 291 | | |
338 | 292 | | |
339 | 293 | | |
340 | | - | |
| 294 | + | |
341 | 295 | | |
342 | 296 | | |
343 | | - | |
| 297 | + | |
344 | 298 | | |
345 | 299 | | |
346 | 300 | | |
347 | | - | |
| 301 | + | |
348 | 302 | | |
349 | | - | |
| 303 | + | |
350 | 304 | | |
351 | 305 | | |
352 | | - | |
| 306 | + | |
353 | 307 | | |
354 | 308 | | |
355 | 309 | | |
356 | 310 | | |
357 | | - | |
| 311 | + | |
358 | 312 | | |
359 | 313 | | |
360 | 314 | | |
| |||
398 | 352 | | |
399 | 353 | | |
400 | 354 | | |
401 | | - | |
| 355 | + | |
402 | 356 | | |
403 | 357 | | |
404 | 358 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
| 1 | + | |
| 2 | + | |
| 3 | + | |
| 4 | + | |
| 5 | + | |
| 6 | + | |
| 7 | + | |
| 8 | + | |
| 9 | + | |
| 10 | + | |
| 11 | + | |
| 12 | + | |
| 13 | + | |
| 14 | + | |
| 15 | + | |
| 16 | + | |
| 17 | + | |
| 18 | + | |
| 19 | + | |
| 20 | + | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
| 25 | + | |
| 26 | + | |
| 27 | + | |
| 28 | + | |
| 29 | + | |
| 30 | + | |
| 31 | + | |
| 32 | + | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
| 54 | + | |
| 55 | + | |
| 56 | + | |
| 57 | + | |
| 58 | + | |
| 59 | + | |
| 60 | + | |
| 61 | + | |
| 62 | + | |
| 63 | + | |
| 64 | + | |
| 65 | + | |
| 66 | + | |
| 67 | + | |
| 68 | + | |
| 69 | + | |
| 70 | + | |
| 71 | + | |
| 72 | + | |
| 73 | + | |
| 74 | + | |
| 75 | + | |
| 76 | + | |
| 77 | + | |
| 78 | + | |
| 79 | + | |
| 80 | + | |
| 81 | + | |
| 82 | + | |
| 83 | + | |
| 84 | + | |
| 85 | + | |
| 86 | + | |
| 87 | + | |
| 88 | + | |
0 commit comments