Improve commit line break unwrapping for variable-length list markers #8417
+210
−37
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.
The
unwrapCommitMessageBodyfunction assumed fixed 2-space continuation indent for all list items. This broke unwrapping for numbered lists with multi-digit markers (10.,100.) and didn't match GitHub's lenient continuation rules.Changes
1.= 3,10.= 4, etc.)Example
Before (broken):
→ Lines not joined (treated as separate due to hardcoded +2 indent check)
After (fixed):
Test coverage added
Warning
Firewall rules blocked me from connecting to one or more addresses (expand for details)
I tried to connect to the following addresses, but was blocked by firewall rules:
update.code.visualstudio.com/opt/hostedtoolcache/node/20.20.0/x64/bin/node /opt/hostedtoolcache/node/20.20.0/x64/bin/node ./out/src/test/runTests.js(dns block)If you need me to access, download, or install something from one of these locations, you can either:
Original prompt
💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.