收缩 warning 白名单 #673

Open
opened 2026-10-07 20:25:25 +08:00 by suzmii · 2 comments
Member

当前还有一些warning正在不断产生..希望原作者可以维护一下,当前留了一个warning白名单把这些warning记录下来不影响CI,但是未来希望能清理掉..

其中 12 条根因是 b04aceb57(@王德宇,2026-10-07「AGC 陶泥儿打包链路落地与 exports 全退役」)——它删掉了 project/export.rs、commands/desktop.rs 里的调用点,这些入口就没人用了;另 2 条由 06e6cf852、c17180015/6ccb9f457 收尾。

现状:这 14 条已入白名单 ⇒ 不阻断任何人,只是记录在案。

文件:行 lint 符号 判断
src/project/bootstrap.rs:4/7/16/32 dead_code BOOTSTRAP_PACKAGE_MAX_BYTES、ProjectBootstrapResult、read_bootstrap_file、run_project_bootstrap_at 疑似发布链路半成品
src/command_exec.rs:645、:2500 dead_code resolve_project_bootstrap_spec_at、run_project_bootstrap_command_at 同上(bootstrap 链)
src/game_package_upload.rs:73 dead_code ensure_staging_path_in_dir 发布链暂存路径守卫
src/project/verification.rs:662 dead_code run_project_verification_with_commit_at 薄包装,可直接删
src/game_package_upload.rs:19、src/project.rs:30、src/export/draft/taonier/mod.rs:25 unused_imports GAME_PACKAGE_UPLOAD_PROGRESS_EVENT、bootstrap::*、TaonierArtifactPackage(均为再导出) 纯死项,直接删
src/game_package_upload/runtime.rs:11 dead_code GAME_PACKAGE_UPLOAD_PROGRESS_EVENT(常量) 纯死项,直接删
src/export/draft/taonier/registry.rs:88 dead_code read_form 纯死项,直接删
src/export/draft/taonier/script.rs:22 dead_code NpmScript.package_json_relative(字段,仅用例读) 纯死项,直接删

清理后:跑 npm run check:rust-warnings 会打印 [已消除·可更新基线];可以call下,我把白名单收缩掉。

当前还有一些warning正在不断产生..希望原作者可以维护一下,当前留了一个warning白名单把这些warning记录下来不影响CI,但是未来希望能清理掉.. 其中 **12 条根因是 `b04aceb57`(@王德宇,2026-10-07「AGC 陶泥儿打包链路落地与 exports 全退役」)**——它删掉了 `project/export.rs`、`commands/desktop.rs` 里的调用点,这些入口就没人用了;另 2 条由 `06e6cf852`、`c17180015`/`6ccb9f457` 收尾。 **现状**:这 14 条已入白名单 ⇒ **不阻断任何人**,只是记录在案。 | 文件:行 | lint | 符号 | 判断 | | --- | --- | --- | --- | | `src/project/bootstrap.rs:4/7/16/32` | dead_code | `BOOTSTRAP_PACKAGE_MAX_BYTES`、`ProjectBootstrapResult`、`read_bootstrap_file`、`run_project_bootstrap_at` | 疑似发布链路半成品 | | `src/command_exec.rs:645`、`:2500` | dead_code | `resolve_project_bootstrap_spec_at`、`run_project_bootstrap_command_at` | 同上(bootstrap 链) | | `src/game_package_upload.rs:73` | dead_code | `ensure_staging_path_in_dir` | 发布链暂存路径守卫 | | `src/project/verification.rs:662` | dead_code | `run_project_verification_with_commit_at` | 薄包装,可直接删 | | `src/game_package_upload.rs:19`、`src/project.rs:30`、`src/export/draft/taonier/mod.rs:25` | unused_imports | `GAME_PACKAGE_UPLOAD_PROGRESS_EVENT`、`bootstrap::*`、`TaonierArtifactPackage`(均为再导出) | 纯死项,直接删 | | `src/game_package_upload/runtime.rs:11` | dead_code | `GAME_PACKAGE_UPLOAD_PROGRESS_EVENT`(常量) | 纯死项,直接删 | | `src/export/draft/taonier/registry.rs:88` | dead_code | `read_form` | 纯死项,直接删 | | `src/export/draft/taonier/script.rs:22` | dead_code | `NpmScript.package_json_relative`(字段,仅用例读) | 纯死项,直接删 | **清理后**:跑 `npm run check:rust-warnings` 会打印 `[已消除·可更新基线]`;可以call下,我把白名单收缩掉。
suzmii changed title from AGC 壳 Windows 生产口径新报 14 条告警(Linux CI 看不到)——请确认是随退役删除还是要重新接线 to 收缩 warning 白名单 2026-10-07 20:33:14 +08:00
Author
Member

这 14 条告警的来龙去脉,以及想请你确认的一点

  • 它们来自本轮的 b04aceb57(AGC 陶泥儿打包链路落地与 exports 全退役):它删掉了 project/export.rs、commands/desktop.rs 里的调用点,于是这些入口变成"没人用"。共 14 条,其中 12 条根因是这个提交。
  • 其中 7 项看着像发布链路的半成品/预留入口(project/bootstrap.rs 整模块 + command_exec.rs 两个入口 + ensure_staging_path_in_dir):是"这条链先不做、随退役删掉",还是"发布会重新接线"?只有你最清楚,你说一句我就照办。
  • 我这边不代改的理由很朴素:这些文件你正在动,我去删容易删到你手上正在写的东西;之前试过一次顺手修,合并后立刻被 Linux 用例暴露成 E0425(那条链仍被 Linux-only 测试调用)。所以方向交给你定,风险最小。
  • 现在它们不阻断任何人:已登记在告警白名单 scripts/warning-baseline.json(agc-windows 14 条、agc-linux 69 条),检查只拦"白名单之外的新增"。
  • 你处理完(或确认保留)后跑一下 npm run check:rust-warnings,输出里会出现 [已消除·可更新基线] 的那几行;回我一下就把白名单收缩掉。
**这 14 条告警的来龙去脉,以及想请你确认的一点** - 它们来自本轮的 `b04aceb57`(AGC 陶泥儿打包链路落地与 exports 全退役):它删掉了 `project/export.rs`、`commands/desktop.rs` 里的调用点,于是这些入口变成"没人用"。共 14 条,其中 12 条根因是这个提交。 - **其中 7 项看着像发布链路的半成品/预留入口**(`project/bootstrap.rs` 整模块 + `command_exec.rs` 两个入口 + `ensure_staging_path_in_dir`):是"这条链先不做、随退役删掉",还是"发布会重新接线"?**只有你最清楚**,你说一句我就照办。 - 我这边不代改的理由很朴素:这些文件你正在动,我去删容易删到你手上正在写的东西;之前试过一次顺手修,合并后立刻被 Linux 用例暴露成 `E0425`(那条链仍被 Linux-only 测试调用)。所以方向交给你定,风险最小。 - **现在它们不阻断任何人**:已登记在告警白名单 `scripts/warning-baseline.json`(`agc-windows` 14 条、`agc-linux` 69 条),检查只拦"白名单之外的新增"。 - 你处理完(或确认保留)后跑一下 `npm run check:rust-warnings`,输出里会出现 `[已消除·可更新基线]` 的那几行;回我一下就把白名单收缩掉。
k88936 self-assigned this 2026-10-08 11:13:12 +08:00
Member

善

善
Sign in to join this conversation.
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: GenarrativeAI/Genarrative#673