refactor(app):优化 App 层装配逻辑、消除过度设计并修复子进程残留 by phantom5099 · Pull Request #264 · 1024XEngineer/neo-code

phantom5099 · 2026-04-13T14:41:24Z

变更说明

本 PR 主要聚焦于 internal/app 层及相关核心模块的代码重构与生命周期管理优化。通过向上透传清理钩子解决了 MCP 僵尸子进程残留的问题，提取了纯函数以简化大模型文本生成逻辑，并极其严格地审查了项目结构，清除了“过度设计”的碎片化胶水文件，使管线装配回归极简。

主要改动

引入优雅停机清理钩子：修改了 app.RuntimeBundle 结构，透传了 Close func() error 方法。在 internal/tools/mcp/registry.go 中新增资源释放的 Close 方法，并由 internal/cli/root.go 顶层进行 defer 捕获调用。
抽取文本生成纯函数：新增 internal/provider/generate.go，将冗长的流式转非流式拼接逻辑封装在独立的 GenerateText 纯函数中，彻底消除样板代码。
消除过度设计 (反碎片化)：
- 彻底删除了物理文件 internal/app/memo_adapter.go 及其配套的测试文件。
- 将轻量级的适配器逻辑（runtimeMemoExtractorFunc 和 textGenAdapter）以闭包类型的形式直接内联回 internal/app/bootstrap.go。
- 将对应的适配器测试用例无缝合并至 internal/app/bootstrap_test.go 中，消除测试碎片。

预期收益

提升稳定性与健壮性：有效处理资源的优雅停机，彻底解决主程序退出后可能残留子进程引发的资源占用及系统隐患。
降低心智负担：移除了强行拆分的碎片化文件，代码内聚度大幅提升。后续开发者在阅读 bootstrap.go 时即可一览无遗地理解全部装配逻辑。
提高可维护性：重构后的管线代码更加符合 Go 语言惯用轻巧的设计风格，文本生成能力被单独抽象，测试用例集中管理，使项目整体架构保持了 MVP 阶段所需的轻量与敏捷。

codecov · 2026-04-13T14:43:26Z

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

phantom5099 · 2026-04-13T14:52:54Z

需要补全测试覆盖率

phantom5099 · 2026-04-13T14:52:57Z

/simplify

fennoai · 2026-04-13T14:53:05Z

已按你的要求完成：先处理历史未解决 review 项，再做简化复查，并已提交推送到当前 PR 分支。
提交：9a8a744（已推送到 phantom5099:main）

处理结果

修复 BuildRuntime 失败路径资源泄漏

文件：internal/app/bootstrap.go
变更：在 buildToolRegistry 成功后增加受控 defer 清理，若后续 buildToolManager 失败会执行 toolsCleanup，成功返回前取消清理。

修复 NewProgram 初始化失败路径资源泄漏

文件：internal/app/bootstrap.go
变更：tui.NewWithMemo 失败时主动调用 bundle.Close()，防止 MCP 子进程残留。

恢复 GenerateText 对未知流事件的显式报错

文件：internal/provider/generate.go
变更：补回 default 分支，遇到未识别 StreamEventType 返回错误，不再静默忽略。

补齐回归测试

文件：internal/app/bootstrap_test.go
新增：
- TestBuildRuntimeCleansResourcesWhenToolManagerBuildFails
- TestNewProgramCleansResourcesWhenTUIBuildFails
文件：internal/provider/generate_test.go
新增：
- TestGenerateTextRejectsUnexpectedEvent

简化复查

已调用子代理进行“code simplifier”复查（仓库无同名内置角色，使用等价 code-quality-reviewer）。
结论：当前修复已是最小必要复杂度，未再做额外改动。

验证

已执行：go test ./...
结果：全部通过

View job run

- ensure BuildRuntime cleans MCP resources on tool-manager init failure - ensure NewProgram closes runtime bundle when TUI init fails - restore GenerateText error on unexpected stream events - add regression tests for cleanup and stream-event handling Generated with [codeagent](https://github.com/qbox/codeagent) Co-authored-by: phantom5099 <245659304+phantom5099@users.noreply.github.com>

phantom5099 · 2026-04-13T15:01:20Z

/review -codex
你只允许回顾这轮对之前review的修复是否成功，并且看看修复后有没有又出现职责不清、语义不清的屎山代码出现

phantom5099 · 2026-04-13T15:08:10Z

/simplify

fennoai · 2026-04-13T15:09:05Z

处理结果

已按触发指令完成：先检查历史未解决 review 项并修复，再做简化复查，最后提交推送。

已修复的未解决项

internal/provider/generate.go
- 修复 GenerateText 在 p.Generate 返回错误时仍可能返回部分文本的问题；现在错误时固定返回空文本。
internal/cli/root.go
- 修复 cleanup 错误被静默丢弃的问题。
- 保持 cleanup 在 defer 中执行（避免 panic 路径漏清理），并在有运行错误时使用 errors.Join 合并 program.Run() 与 cleanup 错误。

新增/更新测试

internal/provider/generate_test.go
- 新增：TestGenerateTextReturnsEmptyTextWhenProviderErrorsAfterStreaming
internal/cli/root_test.go
- 新增：TestDefaultRootProgramLauncherReturnsCleanupErrorWhenRunSucceeds
- 新增：TestDefaultRootProgramLauncherJoinsRunAndCleanupErrors

简化复查

仓库无名为 code-simplifier 的内置代理，使用等价 code-quality-reviewer 复查。
根据复查建议，已将 cleanup 语义收敛为“defer + 错误合并”的更简洁且更稳健实现。

验证

已执行：go test ./...
结果：全部通过。

提交与推送

Commit: 8903238
已推送到当前 PR 分支：fork-pr-264-1776091287
fork 仓库同步 PR（skill 流程要求创建/更新）：fix(review): preserve cleanup semantics and error contracts phantom5099/neo-code#5

未完成项：无。

View job run

Generated with [codeagent](https://github.com/qbox/codeagent) Co-authored-by: phantom5099 <245659304+phantom5099@users.noreply.github.com>

phantom5099 · 2026-04-13T15:16:11Z

/code -codex 给我提升测试覆盖率直到100%

fennoai · 2026-04-13T15:16:18Z

结果

已按你的要求补测试并推送到当前 PR 分支，覆盖了这轮改动里未覆盖的关键分支。

本次新增测试点：

internal/app/bootstrap_test.go
- newMemoExtractorAdapter：scheduler=nil、provider 解析失败、正常调度+提取、factory.Build 失败
- runtimeMemoExtractorFunc.Schedule
- textGenAdapter.Generate
internal/provider/generate_test.go
- text_delta payload 为 nil
- message_done payload 为 nil
internal/tools/mcp/registry_test.go
- (*Registry)(nil).Close() 边界分支

验证

已执行：go test ./... -coverprofile=coverage.out（通过）
关键函数覆盖率：
- internal/app/bootstrap.go:newMemoExtractorAdapter = 100%
- internal/provider/generate.go:GenerateText = 100%
- internal/tools/mcp/registry.go:Close = 100%

说明：仓库整体 total 仍不是 100%（当前约 84.9%），因为存在大量历史包/语句未覆盖；本次已把你这轮改动相关缺口补齐并推送。

提交与推送

已推送到分支：fork-pr-264-1776091287
新增提交：88cc21f
由于远端先行更新，已同步并产生合并提交：d25d03c
fork PR 已更新：fix(review): preserve cleanup semantics and error contracts phantom5099/neo-code#5

View job run

Generated with [codeagent](https://github.com/qbox/codeagent) Co-authored-by: phantom5099 <245659304+phantom5099@users.noreply.github.com>

…99/neo-code into fork-pr-264-1776091287

fix(review): preserve cleanup semantics and error contracts

phantom5099 · 2026-04-13T15:32:02Z

/review -codex

phantom5099 · 2026-04-13T15:36:46Z

/simplify

fennoai · 2026-04-13T15:36:54Z

完成结果

已按触发指令完成：先处理历史未解决 review 项，再进行 code-simplifier 复查，并已提交推送。

本次改动

internal/app/bootstrap.go

更新 NewProgram 注释，明确返回的 cleanup 需要在退出时调用。
采纳简化建议：buildToolRegistry 在无 MCP 时直接返回 nil cleanup，移除 no-op 闭包。

internal/app/bootstrap_test.go

新增回归测试 TestNewMemoExtractorAdapterKeepsScheduledConfigSnapshot：验证 Schedule 后即使 config.Manager 发生 provider/model 漂移，已排队 extractor 仍使用调度时快照。

验证

已执行：go test ./...
结果：全部通过。

提交与推送

Commit: dc109eb
已推送到当前 PR 分支：fork-pr-264-1776091287
fork 同步 PR：test(app): close unresolved review follow-ups phantom5099/neo-code#6

未完成项：无。

View job run

Generated with [codeagent](https://github.com/qbox/codeagent) Co-authored-by: phantom5099 <245659304+phantom5099@users.noreply.github.com>

test(app): close unresolved review follow-ups

pionxe · 2026-04-14T03:03:36Z

/review

fennoai

整体审查完成（代码质量、性能、安全、文档一致性），未发现需要修改的问题。以下备注仅用于记录审查覆盖点。

fennoai · 2026-04-14T03:08:37Z

 // defaultRootProgramLauncher 负责在默认根命令路径下启动 TUI。
-func defaultRootProgramLauncher(ctx context.Context, opts app.BootstrapOptions) error {
-	program, err := newRootProgram(ctx, opts)
+func defaultRootProgramLauncher(ctx context.Context, opts app.BootstrapOptions) (err error) {


已重点审阅 defaultRootProgramLauncher 的运行错误与清理错误合并路径（errors.Join），语义与测试覆盖匹配，未发现问题。

phantom5099 added 2 commits April 13, 2026 22:39

refactor(app):refactor(app):App 装配层精简与生命周期管理优化

397cabe

Merge remote-tracking branch 'upstream/main'

34dc4c6

This comment was marked as outdated.

Sign in to view

fix(review): preserve cleanup semantics and error contracts

8903238

Generated with [codeagent](https://github.com/qbox/codeagent) Co-authored-by: phantom5099 <245659304+phantom5099@users.noreply.github.com>

xgopilot and others added 3 commits April 13, 2026 15:22

test(coverage): fill coverage gaps in app/provider/mcp

88cc21f

Generated with [codeagent](https://github.com/qbox/codeagent) Co-authored-by: phantom5099 <245659304+phantom5099@users.noreply.github.com>

Merge branch 'fork-pr-264-1776091287' of https://github.com/phantom50…

d25d03c

…99/neo-code into fork-pr-264-1776091287

Merge pull request #5 from phantom5099/fork-pr-264-1776091287

e773178

fix(review): preserve cleanup semantics and error contracts

This comment was marked as outdated.

Sign in to view

xgopilot and others added 2 commits April 13, 2026 15:41

test(app): fix remaining review items and simplify cleanup path

dc109eb

Generated with [codeagent](https://github.com/qbox/codeagent) Co-authored-by: phantom5099 <245659304+phantom5099@users.noreply.github.com>

Merge pull request #6 from phantom5099/fork-pr-264-1776091287

61fc9f2

test(app): close unresolved review follow-ups

fennoai Bot reviewed Apr 14, 2026

View reviewed changes

pionxe merged commit 0e3b410 into 1024XEngineer:main Apr 14, 2026
2 checks passed

fennoai Bot mentioned this pull request Apr 16, 2026

风险点计算 #313

Closed

Uh oh!

Conversation

phantom5099 commented Apr 13, 2026

变更说明

主要改动

预期收益

Uh oh!

codecov Bot commented Apr 13, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

This comment was marked as outdated.

Uh oh!

phantom5099 commented Apr 13, 2026

Uh oh!

phantom5099 commented Apr 13, 2026

Uh oh!

fennoai Bot commented Apr 13, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

处理结果

验证

Uh oh!

phantom5099 commented Apr 13, 2026

Uh oh!

This comment was marked as outdated.

Uh oh!

phantom5099 commented Apr 13, 2026

Uh oh!

fennoai Bot commented Apr 13, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

处理结果

Uh oh!

phantom5099 commented Apr 13, 2026

Uh oh!

fennoai Bot commented Apr 13, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

结果

验证

提交与推送

Uh oh!

phantom5099 commented Apr 13, 2026

Uh oh!

This comment was marked as outdated.

Uh oh!

phantom5099 commented Apr 13, 2026

Uh oh!

fennoai Bot commented Apr 13, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

完成结果

本次改动

验证

提交与推送

Uh oh!

pionxe commented Apr 14, 2026

Uh oh!

fennoai Bot left a comment

Choose a reason for hiding this comment

Uh oh!

fennoai Bot Apr 14, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

codecov Bot commented Apr 13, 2026 •

edited

Loading

fennoai Bot commented Apr 13, 2026 •

edited

Loading

fennoai Bot commented Apr 13, 2026 •

edited

Loading

fennoai Bot commented Apr 13, 2026 •

edited

Loading

fennoai Bot commented Apr 13, 2026 •

edited

Loading