Skip to content

fix(core): contain lsp warmup defects - #30226

Merged
nexxeln merged 1 commit into
devfrom
nxl/contain-lsp-warmup-defects
Jun 1, 2026
Merged

fix(core): contain lsp warmup defects#30226
nexxeln merged 1 commit into
devfrom
nxl/contain-lsp-warmup-defects

Conversation

@nexxeln

@nexxeln nexxeln commented Jun 1, 2026

Copy link
Copy Markdown
Member

summary

  • treat read-triggered LSP warm-up as fully best effort
  • prevent detached LSP defects from escaping after a successful file read

@nexxeln
nexxeln merged commit d85f8cd into dev Jun 1, 2026
12 checks passed
@nexxeln
nexxeln deleted the nxl/contain-lsp-warmup-defects branch June 1, 2026 10:38
0xLLLLH pushed a commit to 0xLLLLH/opencode that referenced this pull request Jun 2, 2026
avion23 pushed a commit to avion23/opencode that referenced this pull request Jun 10, 2026
isaacfinnegan pushed a commit to isaacfinnegan/whispercode that referenced this pull request Jun 15, 2026
leduyphuc1702 pushed a commit to leduyphuc1702/opencode-workflow that referenced this pull request Jun 16, 2026
zackslash pushed a commit to zackslash/opencode that referenced this pull request Jun 28, 2026
SMARK2022 added a commit to SMARK2022/opencode that referenced this pull request Jul 9, 2026
- 默认启用 LSP:未配置 lsp 时视为 true(cfg.lsp === false 才禁用),
  降低使用门槛。LSP 按需启动(touchFile 时 spawn),不影响启动性能。
- 回移 PR anomalyco#30226:read.ts warm 从 Effect.ignore 改为 Effect.ignoreCause,
  隔离后台 LSP 预热 defect 避免逸出到 read scope。
- 增量诊断策略:编辑后只显示新引入的错误(reportDelta),不重复展示
  预存错误。借鉴 Copilot "new errors introduced by a file edit" 语义,
  避免模型误判编辑失败(issue anomalyco#9102)。
- MAX_PER_FILE 从 20 降至 5:5 条足以指示问题,减少 token 消耗。
- metadata.diagnostics 统一存储新错误数组(delta),diagnosticSummary
  存储摘要供 TUI 渲染紧凑状态行(✓ 0 new · N existing)。
- VSCode Bridge LSP Backend:新增 sdks/vscode/src/lsp.ts 暴露 VSCode 的
  LSP 能力(诊断/hover/定义/引用/符号),通过 bridge HTTP 端点传输。
  opencode 端 LSP Service 优先走 bridge,无 bridge 时回退到内置 LSP。
  对 agent 完全透明,有 bridge 时零安装全语言覆盖。
- TUI DiagnosticsDisplay 增强:有新错误显示详情,无新错误但有已存在
  错误显示紧凑摘要,无错误不显示。apply_patch 添加诊断渲染(此前缺失)。
MarsQiu007 referenced this pull request in MarsQiu007/openNovel Jul 31, 2026
banlanzs pushed a commit to banlanzs/opencode that referenced this pull request Aug 6, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant