Repository navigation
Conversation
github-actions Bot
added a commit
to citizenadam/desloppify
that referenced
this pull request
Oct 11, 2026
This branch has not been deployed
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.
Summary
A review that explicitly selects 18 dimensions and assesses all 18 is currently rejected because two other language dimensions are absent from the run. The same gate blocks recovery with
review --import-run.Restrict the missing-dimension gate and its error detail to the packet selection. The broader scorecard coverage notice remains informational.
Evidence
After: all four pass; normal replay preserves previously trusted unselected assessment values, dates, and metadata without
--allow-partialor launching review batches.b1f6010: 5901 passed / 70 failed / 4 skipped, versus 5897 / 70 / 4 on that exact clean base. All 70 failed IDs are identical; the sandbox cannot download required tree-sitter parser artifacts. This comparison is limited to that integration base, not a full-suite claim about current upstream.Merge Danger
Door: two-way
Blast Radius: imports
Trusted imports still fail when a selected dimension is unassessed. Full/default selections retain their existing fail-closed behavior. The change does not remove or replace unselected assessments.