Repository navigation
fix: Automated Codecov User reports incorrectly make complete diff review context partial (#724) - #729
Merged
Conversation
…iff-review-context-partial: Preserve complete bounded review conversations for #724
|
To preview the documentation for this pull request, visit the following URL:
|
vypbot
self-requested a review
October 26, 2026 15:55
Closed
1 task done
Contributor
Bugbot: review complete
|
Contributor
Copilot approval · Human review neededStatus: A protected workflow, policy, or trust-boundary file changed. Technical evidenceRevision: |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #729 +/- ##
===========================================
+ Coverage 94.24% 94.25% +0.01%
===========================================
Files 989 990 +1
Lines 32020 32046 +26
Branches 9399 9404 +5
===========================================
+ Hits 30176 30205 +29
Misses 702 702
+ Partials 1142 1139 -3
🚀 New features to boost your workflow:
|
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.
Preserves complete review discussion when it fits the existing 24,000-character section budget. A 4,000-character fragment boundary no longer clips a longer human comment or a User-account report and falsely makes the review partial.
The pure packing policy normalizes and sanitizes before splitting, preserves surrogate pairs and non-colliding untrusted framing, counts logical sources rather than fragments, and includes metadata, separators and omission notices in the final budget. Native automated-author filtering remains unchanged. Actual omissions, oversized sources, malformed Unicode and incomplete diff evidence cannot be presented as complete; approval rules remain in force.
Validation:
Remote verification at
50d084cdac077fa1a694a361d4527f1406003214: CI Check, Branch Sync, Commit, Pull Request and all hosted Linux/macOS/Windows platform jobs pass. Bugbot completes both diff partitions with complete input coverage and zero findings. Codecov reports 100.00% patch coverage and zero missed lines in both changed source files. The guarded Approval check is neutral by design (protected-path); it recommends a human decision and has not issued an automatic approval. Merged intodevelopas70ce428342202b0eb320868134dc453f20ee575aafter the maintainer explicitly confirmed this PR and its neutral protected-path recommendation. No administrative check bypass was used. Local replay is not a claim that PR #719 has already rerun successfully. This implements the published companion's generic lossless design; it does not add trusted-producer classification or public configuration.Closes #724