Repository navigation
fix: 除外した差分に本文の文字数を配分しない - #7
Merged
Merged
Conversation
除外ファイルの見出しと通知にも予算を割り当てていたため、ファイル数が多いと対象外の差分まで消えていた。 一覧と省略通知を保ち、本文と連結用の改行の予算は空でない本文だけに配分する。 実際のプロンプトで小さい本文の保持と大きい本文の公平な配分を検証し、全除外と除外1万件超も確認する。
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
概要
.git-sc-ignoreの対象が多いと、対象外の小さな差分までプロンプトから消えていました。除外した本文には文字数を配分せず、残りの本文で予算を分けます。ファイル一覧・変更行数・省略通知は保持します。原因と変更
diff_file_excerpts()が残す見出しと通知にも、render_diff_excerpts()が本文予算を割り当てていました。除外済みの本文を空にし、空でない本文数から配分と連結用の改行数を計算します。lockfile・binary の要約は従来どおりです。設計相談で確認した、除外済み本文だけを配分から外す案を採用しました。要約の扱いは変えず、修正範囲をこの不具合に限定しています。
検証
make ci: format・clippy（Apple Intelligence 有効／無効）・全テストastro-sight review/impact: API変更・未解決の呼び出し元なしCloses #5