处理编译大量warning问题 #503

Merged

lhk229 merged 21 commits from opt/compile-warning into master 2026-09-24 13:01:32 +08:00

Conversation 2 Commits 21 Files Changed 149

+918 -7145

lhk229 commented 2026-09-23 20:43:53 +08:00

Member

Copy Link

Copy Source

No description provided.

👍 2

lhk229 added 8 commits 2026-09-23 20:43:53 +08:00

记录主要构建入口编译警告清单 ... b7ae18e65c

记录各主要构建入口的验证结果、警告分类和未覆盖范围
整理 AGC 的 237 条编译器警告、ts-rs 提示及后续处理建议
登记前端构建警告和移动端检查错误，并更新文档分类索引

Merge remote-tracking branch 'origin/master' into opt/compile-warning 07feed7511

清理低风险编译警告并修复构建检查脚本 ... 018697203f

清理冗余导入、可变绑定和赋值，将 AGC 默认编译警告从 231 条降至 195 条
按测试和平台边界限定导入，保留兼容出口与待核查的业务参数
修复预览部署器嵌套 npm 警告和移动端检查脚本的 Windows 启动错误
更新警告清单、开发运维文档与共享开发流程，记录验证结果和剩余项

更新Direct审计计时退役契约 ... 5a86a64e48

核查未使用代码的保留依据并更新警告清单
将旧平行审计方案归为历史并同步文档入口
明确项目历史、界面计时与模型使用记录的现役边界

清理已退役的Direct审计与计时代码 ... 268f45015c

删除旧平行审计和请求计时模块及沿途参数
保留代理透传、模型记录与现役历史链路测试
同步契约和验证记录，消除12条编译警告

清理局部未使用代码并收紧测试编译边界 ... 600beb452b

删除无消费者的包装、声明和重导出
将Direct执行测试入口限定为测试构建
记录86条复核与33条清理结果，验证85项回归

清理成组未使用实现并更新现役契约 ... 36f1adac61

删除旧Home用户对话入口、附件sidecar和无调用包装
保留现役附件输入、内部Home通道及重生成保护
更新过期契约与文档索引，记录30条警告清理和67项测试结果

合并最新master到编译警告清理分支 ... e1b9008852

同步远端master最新变更并保留编译警告清理成果

suzmii commented 2026-09-23 20:49:26 +08:00

Member

Copy Link

Copy Source

🤓☝🏻好修兄弟，好修

🤓☝🏻好修兄弟，好修

lhk229 added 1 commit 2026-09-23 21:29:57 +08:00

清理测试用途代码并减少七十条编译警告 ... 1a4af3506c

收紧测试辅助代码的编译范围并删除退役实现和冗余包装
逐项核对原测试断言，仅保留现役契约覆盖并调整必要调用
移除未消费的Pending错误载荷，保留持久错误记录和恢复验证
保留四条待澄清契约项，更新警告清单与实施文档
验证一百七十八个定向Rust测试，默认构建警告从一百二十条降至五十条

lhk229 added 1 commit 2026-09-23 22:36:37 +08:00

Merge branch 'master' into opt/compile-warning 761cccbbf1

lhk229 added 1 commit 2026-09-23 22:52:49 +08:00

收紧测试与平台编译边界并清理冗余测试包装 ... df06ee003a

将十五条重导出诊断对应的测试出口限定编译范围并删除无消费者导出
将Provider交接故障注入结果及传递分支限定为测试编译
删除计数快照和旧回复流ready包装，保留有效测试断言
将项目目录句柄方法限定为Unix平台
更新二十条警告的处理记录，注明未重新编译验证数量

lhk229 added 5 commits 2026-09-24 02:09:37 +08:00

清理退役策划链路残留并统一文档口径 ... 00984be228

删除旧策划生命周期常量与无构造的恢复分支
合并无用锁参数透传，保留现役持锁范围和项目锁归属校验
移除旧身份参数与哨兵包装，保留通用工具规则及项目过滤
明确策划V1与V2均已退役，修正文档与共享记忆
删除四份过期退役计划并更新六条编译警告处理记录

清理九条旧包装与失效门禁编译警告 ... b39cd555ae

删除无调用的素材生成、Goal 与用户修订查询包装
清理恒假视觉门禁、专属提示词及不可达等待分支
移除 manifest 初始化未用参数并更新全部调用
以循环返回值替代 Direct 回合的冗余初始化，保持重试语义
同步现行行为文档、警告处理记录及未编译验证边界

收紧平台与编辑器编译范围并分离构建校验 ... c62a911745

分离 Codex 构建端元数据校验并保留原有测试
限定 Linux 沙箱与进程清理入口，删除无调用的平台占位实现
按实际 feature 收紧编辑器注册并删除冗余宿主包装
保留 Runner 备用端口算法的 Linux 与测试编译入口
同步警告清单、插件契约及定向验证边界

清理旧策划修订写入入口并记录认证契约冲突 ... 7d6fff577d

删除无生产调用的用户修订写入函数，保留持久记录兼容与门禁
调整原有门禁测试的历史记录夹具，保持行为断言
将 OAuth 契约冲突与关闭条件记录到 AGC 主方案
警告清单仅保留 AuthBridge 暂缓状态及主方案链接

记录五条 ts-rs 提示的保留决定 ... dd7c445c8a

明确四个未知字段属性提示对应的类型
记录类型核查结论及暂不引入依赖补丁的决定

lhk229 added 1 commit 2026-09-24 02:09:52 +08:00

Merge branch 'master' into opt/compile-warning cb60632c99

lhk229 commented 2026-09-24 12:35:55 +08:00

Author

Member

Copy Link

Copy Source

审查范围：HEAD cb60632c 相对 origin/master...HEAD（merge-base 87e52860，149 个文件，约 1.8k 增 / 7.0k 删）。审查期间 origin/master 已推进到 dcb4f2a1。

结论：行为层面未发现高危回归。Direct/Codex、Runtime 生命周期 / 恢复、Runner、编辑器适配、构建脚本和移动端检查脚本中的删除项，抽查后均能对应为 master 上已无生产调用方的死代码、test-only 入口或同步收窄的 cfg 边界；现役调用链没有发现断点。合入前还有 1 个必须项和 3 个建议项。

必须处理：先同步最新 master

当前分支尚未包含 origin/master 最新提交；本地执行 git merge-tree --write-tree HEAD origin/master 会在 docs/project-memory/shared-memory/pitfalls.md 产生内容冲突。原因是 master 的 e80279c17 在文首新增 2026-09-24 读秒粒度条目，而本分支也在文首新增“策划历史条目边界”说明。合并 / 变基时请同时保留这两段内容，然后重新跑 CI。

建议处理

1. docs/project-memory/shared-memory/pitfalls.md:5303 仍以现行口吻描述旧的“完整原文 NFKC + 独立确认句式”重生成授权规则，以及“Direct app-server 可写边界只限 game/、不得网络 / 命令”的旧沙箱边界；这与本分支已更新的 decision-log / 主实施计划口径相冲突。建议给该历史条目加退役限定，或改为指向当前 MCP 能力边界。
2. 本次删除了策划 V1/V2 清理计划 / 里程碑，但 apps/ai-game-creator-shell/tests/appSurface/harness.ts:378、:467 仍定义并导出 planningResponseStream、createPlanGddStateView；后者还引用已不存在定义的 PlanGddStateViewV1（导出见 :1528、:1543）。当前没有测试消费且 tsconfig 不覆盖该测试文件，所以不影响运行；但原验收项包含“旧 PlanGdd 类型和旧前端测试契约不再存在”，建议删除这两个无消费工厂，或在现役文档中说明保留理由。
3. 本机验证结果：默认 cargo check 通过，仅剩文档中明确暂缓的 3 条 AuthBridge Rust warning，另有 5 条已知 ts-rs 提示；cargo check --tests 也通过，但仍报告 383 条 warning（主要是 test profile 的 dead_code / unused import）。如果本 PR 只承诺默认构建 profile，建议在 PR 描述中明确 test targets 是后续范围；如果目标是更广义的“编译 warning 清零”，则还未闭环。

已验证：

• PR 当前 head 的 9 个 CI status 均为 success。
• cargo check、cargo check --tests 均通过。
• npm run check:encoding 通过（5055 个文件）。
• git diff --check origin/master...HEAD 通过。
• 对被删除的 Direct 审计 / 计时、旧 Home / sidecar、旧文本授权、旧策划修订写入、runner client 包装等符号做了调用方检索，未发现现役残留引用。

审查范围：HEAD `cb60632c` 相对 `origin/master...HEAD`（merge-base `87e52860`，149 个文件，约 1.8k 增 / 7.0k 删）。审查期间 `origin/master` 已推进到 `dcb4f2a1`。 结论：行为层面未发现高危回归。Direct/Codex、Runtime 生命周期 / 恢复、Runner、编辑器适配、构建脚本和移动端检查脚本中的删除项，抽查后均能对应为 master 上已无生产调用方的死代码、test-only 入口或同步收窄的 cfg 边界；现役调用链没有发现断点。合入前还有 1 个必须项和 3 个建议项。 **必须处理：先同步最新 master** 当前分支尚未包含 `origin/master` 最新提交；本地执行 `git merge-tree --write-tree HEAD origin/master` 会在 `docs/project-memory/shared-memory/pitfalls.md` 产生内容冲突。原因是 master 的 `e80279c17` 在文首新增 2026-09-24 读秒粒度条目，而本分支也在文首新增“策划历史条目边界”说明。合并 / 变基时请同时保留这两段内容，然后重新跑 CI。 **建议处理** 1. `docs/project-memory/shared-memory/pitfalls.md:5303` 仍以现行口吻描述旧的“完整原文 NFKC + 独立确认句式”重生成授权规则，以及“Direct app-server 可写边界只限 `game/`、不得网络 / 命令”的旧沙箱边界；这与本分支已更新的 decision-log / 主实施计划口径相冲突。建议给该历史条目加退役限定，或改为指向当前 MCP 能力边界。 2. 本次删除了策划 V1/V2 清理计划 / 里程碑，但 `apps/ai-game-creator-shell/tests/appSurface/harness.ts:378`、`:467` 仍定义并导出 `planningResponseStream`、`createPlanGddStateView`；后者还引用已不存在定义的 `PlanGddStateViewV1`（导出见 `:1528`、`:1543`）。当前没有测试消费且 tsconfig 不覆盖该测试文件，所以不影响运行；但原验收项包含“旧 PlanGdd 类型和旧前端测试契约不再存在”，建议删除这两个无消费工厂，或在现役文档中说明保留理由。 3. 本机验证结果：默认 `cargo check` 通过，仅剩文档中明确暂缓的 3 条 AuthBridge Rust warning，另有 5 条已知 ts-rs 提示；`cargo check --tests` 也通过，但仍报告 383 条 warning（主要是 test profile 的 dead_code / unused import）。如果本 PR 只承诺默认构建 profile，建议在 PR 描述中明确 test targets 是后续范围；如果目标是更广义的“编译 warning 清零”，则还未闭环。 已验证： - PR 当前 head 的 9 个 CI status 均为 success。 - `cargo check`、`cargo check --tests` 均通过。 - `npm run check:encoding` 通过（5055 个文件）。 - `git diff --check origin/master...HEAD` 通过。 - 对被删除的 Direct 审计 / 计时、旧 Home / sidecar、旧文本授权、旧策划修订写入、runner client 包装等符号做了调用方检索，未发现现役残留引用。

lhk229 added 4 commits 2026-09-24 12:55:03 +08:00

清理旧策划前端测试工厂残留 ... 6284c3d3fa

删除无消费者的 planningResponseStream、createPlanGddStateView 工厂及导出
移除失效的 PlanGddStateViewV1 类型引用
同步现役文档中的旧策划测试退役边界

修正 Direct 重生成授权与执行边界的过期记录 ... cfcefbad89

更新美术重生成的活动回合与稳定请求校验口径
撤销旧沙箱边界描述并关联现行宿主执行许可合同
保留有效的计费幂等与恢复要求

合并最新 master 到编译警告清理分支 ... 8fea34a27d

同步 origin/master 的对话过程卡计时与样式修复
合并踩坑记录冲突并保留策划退役说明和现行执行边界

清理编译警告过程文档并归并有效结论 ... bbff94b6cc

删除编译警告清理过程清单及文档索引引用
将保留决定、待优化项和验证范围归入开发运维文档
保留 OAuth 未决契约入口并补充 npm 混用版本排障结论

lhk229 marked the pull request as ready for review 2026-09-24 12:59:52 +08:00

lhk229 merged commit ec3a187dd7 into master 2026-09-24 13:01:32 +08:00

lhk229 deleted branch opt/compile-warning 2026-09-24 13:01:32 +08:00

lhk229 self-assigned this 2026-09-24 13:28:08 +08:00

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

Clear labels

Compat/Breaking

Breaking change that won't be backward compatible

Kind/Bug

Something is not working

Kind/Documentation

Documentation changes

Kind/Enhancement

Improve existing functionality

Kind/Feature

New functionality

Kind/Security

This is security issue

Kind/Testing

Issue or pull request related to testing

Priority

Critical

1

The priority is critical

Priority

High

2

The priority is high

Priority

Low

4

The priority is low

Priority

Medium

3

The priority is medium

Reviewed

Confirmed

1

Issue has been confirmed

Reviewed

Duplicate

2

This issue or pull request already exists

Reviewed

Invalid

3

Invalid issue

Reviewed

Won't Fix

3

This issue won't be fixed

Status

Abandoned

3

Somebody has started to work on this but abandoned work

Status

Blocked

1

Something is blocking this issue or pull request

Status

Need More Info

2

Feedback is required to reproduce issue or to continue work

No Label

Milestone

No items

No Milestone

Projects

Clear projects

No project

Assignees

Clear assignees

HuangJiaJia JenkenB k88936 (王德宇) kdletters (段舒康) laura2008 (王乐) lhk229 (孔令弘) Loer (农洛儿) loong suzmii (董羽秦) taobaisu (叶东旭) wuxiangwanzi (王子民) YixuanLiu ziyu

No Assignees lhk229

2 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: GenarrativeAI/Genarrative#503