Compare code semantically across a force push in review bundles
メンテナーはふだん 1 日以内に返信
@Alan52254 がすでに取り組んでいます。
2026年11月6日 から。
評価
この issue はまだ評価されていません。
説明
Problem
After a force push, review-bundle.sh --since points a reviewer to a tree diff or a git range-diff. Both compare lines. When an author reworks a commit, the range-diff reads almost as long as the whole diff: 925 of 1200 lines on #586, 624 of 1284 on #584.
Current behavior
diffmodel.py pairs moved units by indentation, which workbench.md calls a guess at syntax and not a parse. since.range-diff.txt shows a reworked commit as removed and added lines, so a rename or a moved function reads as new code.
Desired outcome
A reviewer reading a rewritten pull request sees which functions or blocks changed in meaning, and which only moved or were renamed, without rereading the whole commit.
Scope
- A comparison that builds on
diffmodel.py's unit pairing. - The rewrites of #585, #584 and #586 as its cases.
- No code from or call to OmniDiff, whose license is incompatible with CodeTrial.
Out of scope
Replacing since.diff and since.range-diff.txt; they stay the fallback.
Related
#605 adds those two files for #587.
- 主要言語
- Rust
- スター
- 223
- フォーク
- 148
- 平均マージ
- 1日 10時間
- マージ済み PR（30日）
- 227
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートあり
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
sysprog21/codetrial のほかの issue
-
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
sysprog21/codetrial#604 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
Calculator % disagrees with // on negative operands対応中かも @YunJ-Chang が今日担当しました。 オープン
sysprog21/codetrial#602 · コメント 1 件 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
-
Start does nothing visible in the lobby when site data is blocked対応中かも @sidney-Hung が今日担当しました。 オープン
sysprog21/codetrial#600 · コメント 1 件 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
-
Add a Copy results action to the coding test panel対応中かも @seallllllllll が今日担当しました。 オープン
sysprog21/codetrial#598 · コメント 1 件 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
-
Review history drops older reports that still fit対応中かも @HellHBBD が今日担当しました。 オープン
sysprog21/codetrial#596 · コメント 2 件 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
sysprog21/codetrial の issue をすべて見る
似ている issue
-
bug: merge_insert does not detect concurrent inserts of the same FixedSizeBinary primary key対応中かも @lance-gatefixer が今日担当しました。 オープンbug
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
lance-format/lance#10811 ·
メンテナーはふだん 2 日以内に返信
-
provider testing
難易度 2/5 1〜3時間 初心者へのやさしさ 78/100
mishraprafful/multihull#234 ·
メンテナーはふだん 1 日以内に返信
-
comp-documentation
難易度 1/5 1時間未満 初心者へのやさしさ 84/100
ClickHouse/ClickHouse#129829 ·
メンテナーはふだん 1 日以内に返信
-
bug needs triage python
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
メンテナーはふだん 1 日以内に返信
-
state:needs triage
難易度 2/5 1〜3時間 初心者へのやさしさ 87/100
zed-industries/zed#66628 ·
メンテナーはふだん 1 日以内に返信