Skip to content

评审模型解析静默失败:js-yaml 依赖损坏致路由表为空(GLM-5.1 解析为 null,0.0s 失败) #21

Closed

Description

@Nicholas023

现象:合并 #9-#13 后首次全仓扫描(2026-10-20 03:40,任务 20261020-034006-od7c)0.0s 失败:「评审模型解析失败:未找到可用端点」。此前同配置(GLM-5.1)5 次审查全部成功(02:00-02:20)。

根因(已核):主仓 node_modules/js-yaml 为残损 junction(指向全局 @deepseek-ai/dsh 包内 js-yaml 空目录,package.json 缺失),解析链 loadProfileLlmConfig -> createRequire(import.meta.url)('js-yaml') 失败 -> try/catch 静默返回 null -> 候选路由表为空 -> resolveProvider('GLM-5.1') 返回 null -> 失败文案误导为「端点未配置」。serve 经 junction 加载仓库代码,与本地复现同一条解析链。

排除项(已核):新旧 model-provider 逻辑无行为差异;本地 8090 最小端到端测试通过(codecheck scan 2 文件完成评审 1m33s);serve env 无有效 DSH_HOME;GLM 内网端点连通性另查(与本次失败无关)。

影响:评审模型候选只剩 session;显式另选模型(GLM-5.1 等)不可用;失败信息掩盖真实原因(依赖缺失)。

待修方向(建议):① 仓库 node_modules/js-yaml 改为真实安装(现场已修复坏 junction,本 issue 关注代码层健壮性);② loadProfileLlmConfig 在 js-yaml 加载失败时显式记录原因并给出可诊断信息,不再静默返回 null。

Activity

  1. self-assigned this
    on Oct 19, 2026
  2. added a commit that references this issue on Oct 20, 2026
  3. Nicholas023 commented on Oct 20, 2026

    @Nicholas023
    OwnerAuthor

    修复已合并:PR #29(整合方案,main@9e74866,fix(#21) ad19737)。已核:诊断输出 + return null 契约不变(e2e-switch 端点解析验证)。

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

bugSomething isn't working

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions