fix(web): multiline highlight ranges are broken / omitted in LightweightCodeHighlighter
メンテナーはふだん 1 日以内に返信
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 82/100
- issue の種類
- バグ
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- typescript
- 領域
- frontend
調査の方向性
LightweightCodeHighlighter.tsx から始め、各行について highlightRanges がどのようにフィルタリングおよびマッピングされているかを確認します。symbolDefinitionPreview.tsx、fileMatch.tsx、referenceList.tsx で関連する使用箇所を確認します。開始行が開始列から line.length までを使用し、中間行が 0 から line.length までをハイライトし、終了行が 0 から終了列までを使用すれば完了です。
索引モデルが issue の本文から書いたものです。
説明
In LightweightCodeHighlighter.tsx, highlight ranges spanning multiple lines are not properly mapped across lines.
Currently, the component filters and maps ranges using:
// @todo: we will need to handle the case where a range spans multiple lines.
const ranges = highlightRanges?.filter(range => {
return range.start.lineNumber === lineNumber || range.end.lineNumber === lineNumber;
}).map(range => ({
from: range.start.column - 1,
to: range.end.column - 1,
}));
Consequences
- Middle Lines Dropped: Any line between
range.start.lineNumberandrange.end.lineNumberfails the conditionrange.start.lineNumber === lineNumber || range.end.lineNumber === lineNumber, resulting in zero highlighting for intermediate lines. - Column Inversion on Start/End Lines:
- Start Line:
tois set torange.end.column - 1(column from a different line). Ifstart.column > end.column,from > tocreates an invalid range. - End Line:
fromis set torange.start.column - 1(start line's column), clipping or omitting matches.
- Start Line:
Impacted Areas
This affects all views using LightweightCodeHighlighter:
- Symbol Definition Previews (
symbolDefinitionPreview.tsx): Multi-line types, interfaces, or function definitions have broken highlights in hover popovers. - Search Result Chunks (
fileMatch.tsx): Multi-line regex matches from Zoekt render corrupted or blank intermediate highlights. - References List (
referenceList.tsx): Multi-line call signatures and JSX nodes.
Expected Behavior
Highlight ranges spanning multiple lines should:
- Start line: Highlight from
start.columntoline.length. - Intermediate lines: Highlight the entire line (
0toline.length). - End line: Highlight from
0toend.column.
- 主要言語
- TypeScript
- スター
- 3.9k
- フォーク
- 374
- 平均マージ
- 17時間 15分
- マージ済み PR(30日)
- 38
環境構築
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
sourcebot-dev/sourcebot のほかの issue
-
bug(web): Intermediate directory nodes in buildFileTree receive child leaf paths instead of directory paths対応中かも @riteshvish02 が 4 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
sourcebot-dev/sourcebot#1707 ·
メンテナーはふだん 1 日以内に返信
-
[bug] Search result hydration performs duplicate repository lookups within the same chunk対応中かも @dipeshbabu が 14 日前に担当しました。 オープンbug triage needed
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
sourcebot-dev/sourcebot#1681 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
Linux ctags build instructions in CONTRIBUTING.md are missing build dependencies対応中かも @The-AarushiSingh が 20 日前に担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
sourcebot-dev/sourcebot#1633 ·
メンテナーはふだん 1 日以内に返信
-
Generic Git URL configs keep percent-encoded repo names対応中かも @Tyagiquamar が 20 日前に担当しました。 オープンbug
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
sourcebot-dev/sourcebot#1384 · コメント 4 件 ·
メンテナーはふだん 1 日以内に返信
-
MCP ask_codebase rejects explicit languageModel: getLanguageModelKey includes displayName which the MCP schema doesn't expose対応中かも @pranav718 が 37 日前に担当しました。 オープンask_sb bug
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
sourcebot-dev/sourcebot#1137 · コメント 4 件 ·
メンテナーはふだん 1 日以内に返信
sourcebot-dev/sourcebot の issue をすべて見る
似ている issue
-
Bump Firebase JS SDK (12.19.0 → 13.0.0)対応中かも @SelaseKay が今日担当しました。 オープンNeeds Attention type: enhancement
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
invertase/react-native-firebase#9364 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 82/100
メンテナーはふだん 4 日以内に返信
-
e2e-failure ready-to-code
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
redhat-developer/rhdh-plugin-export-overlays#4261 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
[Bug] 官网文档的图片挂了オープンbug
難易度 2/5 1〜3時間 初心者へのやさしさ 66/100
メンテナーはふだん 1 日以内に返信
-
area:cli bug triage:in-progress
難易度 1/5 1〜3時間 初心者へのやさしさ 82/100
メンテナーはふだん 1 日以内に返信