Skip to content

Improve:优化 SQL 语义诊断范围#2125

Merged
t8y2 merged 1 commit into
t8y2:mainfrom
zipg:codex/optimize-sql-diagnostics-viewport
Jun 29, 2026
Merged

Improve:优化 SQL 语义诊断范围#2125
t8y2 merged 1 commit into
t8y2:mainfrom
zipg:codex/optimize-sql-diagnostics-viewport

Conversation

@zipg

@zipg zipg commented Jun 29, 2026

Copy link
Copy Markdown
Contributor

变更说明

  • SQL 语义诊断只分析当前可见区域对应的完整可执行语句,避免大 SQL 编辑器反复全文解析
  • 滚动停止后刷新当前可见语句诊断,并保留其它区域已有诊断结果
  • 诊断片段返回的位置会映射回完整编辑器文档

验证

  • ./node_modules/.bin/vitest run packages/app-tests/sqlSemanticDiagnostics.test.ts apps/desktop/src/lib/tests/sqlStatementRanges.spec.ts
  • ./node_modules/.bin/vue-tsc --noEmit --project apps/desktop/tsconfig.json
  • ./node_modules/.bin/oxlint apps/desktop/src/components/editor/QueryEditor.vue apps/desktop/src/lib/sqlSemanticDiagnostics.ts packages/app-tests/sqlSemanticDiagnostics.test.ts && ./node_modules/.bin/oxfmt --check apps/desktop/src/components/editor/QueryEditor.vue apps/desktop/src/lib/sqlSemanticDiagnostics.ts packages/app-tests/sqlSemanticDiagnostics.test.ts

@zipg zipg changed the title 优化 SQL 语义诊断范围 Improve:优化 SQL 语义诊断范围 Jun 29, 2026
@t8y2 t8y2 merged commit 79298f4 into t8y2:main Jun 29, 2026
8 checks passed
@t8y2

t8y2 commented Jun 29, 2026

Copy link
Copy Markdown
Owner

Thanks for the contribution! Merged in 79298f4, will be released in the next version.

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.

2 participants