[Feature] 清理 lithe-core 与 lithe-agent-host 的 4 处无用导入和死代码，并为 Core 增加警告门禁

Aperta
  • Qualcosa non va?
#1,796 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 1 giorno

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
4/5
Tempo stimato
1-2 giorni
Idoneità per principianti
57/100
Tipo di issue
Refactoring
Chiarezza
Specificata chiaramente
Stato di attività
Attiva
Stack tecnologico
rust, typescript
Ambito
backend, ci-cd, testing

Direzione di ricerca

Start with the four Rust locations named in the issue: rust/lithe-core/src/network/mod.rs, rust/lithe-core/src/git/workspace_status.rs, rust/lithe-core/src/lsp/interface/plugin_function.rs, and rust/lithe-agent-host/src/lib.rs; compare the proposed cleanup with the existing tests and contract fixture at shared/contracts/plugins/v1/fixtures/backend/function-protocol.json. Then check how the Core workspace runs CI linting, using #1789 as context for the warning gate. Done when the unused code is removed, BACKEND_EXITED has an explicit mapping covered by the fixture test, and Core warnings fail the chosen CI check.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

area:other enhancement P2 platform:cross-platform refactor review: high
Problem to solve / 要解决的问题

在 preview 5a54c41aa 上用 macOS arm64、rustc 1.96.0 执行 cd rust && cargo test --workspace，lithe-core 和 lithe-agent-host 会报 4 条无用代码警告。逐项核对如下，依据是全仓库搜索，范围包括 Swift、Windows、契约 fixture 和脚本：

  1. 网络代理类型的 re-export
    • 位置：rust/lithe-core/src/network/mod.rs:11-14，pub(crate) use proxy::{…} 中的 NetworkProxyMode、NetworkProxyPolicy、NetworkProxyPolicyKind、NetworkProxyType。这条 re-export 由 #1571 引入。
    • 警告：rustc unused_imports。
    • 依据：crate 内只用到同一行的 proxy_policy 和 NetworkProxyPolicyRequest。全仓库搜索这 4 个名字，其他命中都不经过这条 re-export：Swift 里同名但无关的类型（macos/Sources/Lithe/Models/Settings/NetworkProxyPreferences.swift），以及 CoreCommand::NetworkProxyPolicy 这个枚举成员。
  2. 测试模块的多余导入
    • 位置：rust/lithe-core/src/git/workspace_status.rs:102，测试模块的 use super::*;。
    • 警告：rustc unused_imports。
    • 依据：测试只通过 crate::execute_json 调用命令，没有用到父模块的符号。
  3. BACKEND_EXITED 常量
    • 位置：rust/lithe-core/src/lsp/interface/plugin_function.rs:252，codes::BACKEND_EXITED（"plugin.backend.exited"），由 #1595 引入。
    • 警告：rustc dead_code。
    • 依据：Rust 内没有任何引用。这个码由宿主报给页面，macOS 在 macos/Sources/Lithe/Application/Features/PluginHostServiceDispatch.swift:66 另写了同一个字符串。契约 fixture shared/contracts/plugins/v1/fixtures/backend/function-protocol.json:713 要求：宿主把它报给后端时，要改写成 plugin.host.failed。host_failure（plugin_function.rs:1836-1862）现在是靠 _ => codes::HOST_FAILED 兜底满足这条要求，:1858 的注释提到了这种情况，但代码没有用这个常量。
  4. 测试辅助函数 resolve
    • 位置：rust/lithe-agent-host/src/lib.rs:296-299，#[cfg(test)] fn resolve。
    • 警告：rustc dead_code（lib test 构建）。
    • 依据：rust/lithe-agent-host/src/tests.rs:3278 定义了同名闭包 let resolve = |launch| resolve_with(launch, &codex_cli);，测试里的调用全部走这个闭包，lib 里的辅助函数没有调用方。

保留的代价：

  • Core 每次构建都带着这几条警告，新出现的警告容易被淹没，评审时也分不清哪些是本次改动引入的。
  • BACKEND_EXITED 留在 codes 里，看起来 Rust 会专门处理这个码，实际行为却来自兜底分支，读代码的人会误判它的处理路径。
  • Core crate 没有警告门禁：目前只有 scripts/remote/verify-remote-workspace.sh:44-45 对 lithe-remote 运行 clippy -D warnings，所以这类警告能合进 preview。

lithe-git-host 的 forward_slash_path 警告（rust/lithe-git-host/src/configuration.rs:52）已经由 #1789 跟踪，本单不重复处理。

Proposed solution / 建议方案
  1. re-export：从 network/mod.rs 的 re-export 中删掉未用的 4 个类型，保留 proxy_policy 和 NetworkProxyPolicyRequest。风险很低，只影响 crate 内可见性。
  2. 多余导入：删除 workspace_status.rs 测试模块中的 use super::*;。
  3. BACKEND_EXITED（推荐方案）：在 host_failure 里显式写一条 Some(codes::BACKEND_EXITED) => codes::HOST_FAILED，把 :1858 的注释移到这一行。这样 fixture :713 的行为由具名分支保证，常量也有了用处。
    • 不推荐直接删除常量：这是契约里登记的错误码，Rust 侧保留具名常量，可以防止以后被别的含义复用（错误码必须一码一义）。
    • 风险：无行为变化，现有 fixture 测试即可验证。
  4. 测试辅助函数：删除 lithe-agent-host/src/lib.rs:296-299 的 #[cfg(test)] fn resolve。
  5. 警告门禁：清理完成后，为 Core workspace 增加警告门禁，例如在 CI 中运行 cargo clippy --workspace --all-targets -- -D warnings，并与 #1789 的处理一致。stable 工具链升级会带来新的 lint（#1789 就是这种情况），门禁放在哪个工作流、是否固定工具链版本，由维护者决定。
Target platform / 目标平台

Cross-platform / 跨平台

Area / 模块

Other / 其他

Priority / 优先级

P2 / 后续考虑：目前不着急，可安排到后续版本

Alternatives considered / 已考虑的替代方案

用 #[allow(unused_imports)] 或 #[allow(dead_code)] 压掉警告。不推荐：无用代码会继续留着，也违背 develop-lithe 第九节的清理要求。

Additional context / 补充信息
  • 复现命令：cd rust && cargo test --workspace --no-fail-fast，在构建输出中搜索 ^warning。
  • 发现经过：2026-11-07 把 Core 拆分为 Lithe-IDEA/Lithe-core-rust 时，跑完整 workspace 测试发现。拆分出的文件与主仓库逐字节一致，所以主仓库 preview 上有相同的警告。
  • 相关：#1789（lithe-git-host、lithe-core-rules 的 clippy 告警）。
Checklist / 提交前检查
  • I searched existing issues for duplicates. / 我已搜索现有 Issue，确认没有重复建议。
Lingua principale
TypeScript
Stelle
1.8k
Fork
151
Merge medio
10h 29m
PR unite (30g)
407

Preparare l'ambiente

Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di 1lck/Lithe-IDEA

Tutte le issue di 1lck/Lithe-IDEA

Issue simili

Altre issue su TypeScript

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.