opensumi / core

🚀 A framework helps you quickly build AI Native IDE products. https://preview.opensumi.com
https://opensumi.com
MIT License
2.81k stars 354 forks source link

fix: inline live diff zone range #3810

Closed Ricbet closed 3 weeks ago

Ricbet commented 3 weeks ago

Types

Background or solution

zone 表示用户选中的代码区域,我们会在这个区域范围内进行流式 diff 渲染

之前的处理方式是采用 monaco 的 decoration 来进行 rang 范围计算,但这种方式偶尔会有问题,会导致流式渲染的区域超过这个 zone 的范围,不仅 diff 的对比不正确,更会导致未选中的代码部分被覆写

现在重新调整了 zone 的计算方式,在每次要 handleEdits 时实时的根据 newFullRangeTextLines 内容长度重新计算 zone

Changelog

修复 inline live 模式下流式 diff 渲染时会出现溢出覆写的情况

railway-app[bot] commented 3 weeks ago

This PR was not deployed automatically as @Ricbet does not have access to the Railway project.

In order to get automatic PR deploys, please add @Ricbet to the project inside the project settings page.

codecov[bot] commented 3 weeks ago

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 55.09%. Comparing base (07a6f8a) to head (1c80241).

Files Patch % Lines
...rowser/widget/inline-diff/inline-diff-previewer.ts 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #3810 +/- ## ========================================== + Coverage 55.08% 55.09% +0.01% ========================================== Files 1550 1550 Lines 94559 94558 -1 Branches 19363 19371 +8 ========================================== + Hits 52085 52095 +10 + Misses 35277 35268 -9 + Partials 7197 7195 -2 ``` | [Flag](https://app.codecov.io/gh/opensumi/core/pull/3810/flags?src=pr&el=flags&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opensumi) | Coverage Δ | | |---|---|---| | [jsdom](https://app.codecov.io/gh/opensumi/core/pull/3810/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opensumi) | `50.53% <0.00%> (+0.01%)` | :arrow_up: | | [node](https://app.codecov.io/gh/opensumi/core/pull/3810/flags?src=pr&el=flag&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opensumi) | `15.56% <0.00%> (+<0.01%)` | :arrow_up: | Flags with carried forward coverage won't be shown. [Click here](https://docs.codecov.io/docs/carryforward-flags?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=opensumi#carryforward-flags-in-the-pull-request-comment) to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.