Skip to content

fix: clamp inverted visible range in ListView to prevent RangeError (fixes #333230) - #333236

Merged
Connor Peet (connor4312) merged 6 commits into
microsoft:mainfrom
vscodebot-pr:fix/listview-invalid-array-length-333230-aw-33202685801
Aug 31, 2026
Merged

fix: clamp inverted visible range in ListView to prevent RangeError (fixes #333230)#333236
Connor Peet (connor4312) merged 6 commits into
microsoft:mainfrom
vscodebot-pr:fix/listview-invalid-array-length-333230-aw-33202685801

Merge branch 'main' into fix/listview-invalid-array-length-333230-aw-…

7429373
Select commit
Loading
Failed to load commit list.
VS Code Engineering / VS Code PR Check succeeded Aug 31, 2026 in 0s

Approved by 3/3 collaborators

PR classified as community.

⚠️ Head commit is NOT signed by a trusted team member, so the full 3-approval requirement applies. (Clean GitHub merge 7429373 of base into PR branch, but PR-branch parent dcfcbcb is not signed by a trusted team member: Clean GitHub merge dcfcbcb of base into PR branch, but PR-branch parent cb7a02b is not signed by a trusted team member: Head commit cb7a02b is signed by GitHub, not by a trusted team member. GitHub adds its own "Verified" signature to commits it creates on your behalf — e.g. edits made in the github.com web editor, suggestions applied from a review, or commits made via Codespaces or the GitHub API. That signature only proves GitHub made the commit, not that a trusted team member authored it, so it does not earn the signing credit. To get the credit, create the commit locally, and sign it with a trusted key.)

Tip: to earn the credit (−1 approval), the head commit must be signed by you with one of your own keys listed in security/trusted-signing-keys.csv. A "Verified" badge that GitHub adds to commits it creates on your behalf — web-editor edits, applied suggestions, Codespaces, API commits — does not count.

✅ Collaborator approvals: 3/3 (connor4312, roblourens, dmitrivMS).

✅ No newly added runnable binary files.

✅ No protected engineering system files changed.

Automation rules

errors-fix automation rules passed.