Add description param to comment()/propose() with label/desc split by 1024 char boundary#10
Merged
Merged
Conversation
added 4 commits
July 11, 2026 14:34
…undary - Add _split_label_desc() helper: split long text (>1024) at word boundary - comment(): add description param, apply split before payload - propose(): apply split for consistency with long labels - progress(): inherits automatically via comment() wrapper - Bump version to 2.35.2 Acceptance: - comment --label "short" → label=short, desc=None - comment --label <2000 char> → label≈1024 (word boundary), description=tail - comment --desc <2000> (no label) → label=desc[:1024], description=tail - comment --label X --desc Y → label=X, desc=Y (both as-is) - propose --label/--desc — same split logic
Replace test_comment_no_description_kwarg with test_comment_accepts_description_with_split: - Test that comment() accepts description parameter - Test that long text (>1024) is split at word boundary - Test that both short label and description are passed as-is - Test split behavior for long label and long description separately All 93 tests pass.
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
1 participant
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.
Changes
_split_label_desc()that splits text exceeding 1024 chars at word boundarycomment()now accepts optionaldescriptionparameterpropose()applies same split logic for consistencyprogress()inherits from comment() automaticallyAcceptance
comment --label short→ label=short, desc=Nonecomment --label <2000>→ label≈1024 (word boundary), description=tailcomment --desc <2000>→ label=desc[:1024], description=tailcomment --label X --desc Y→ label=X, desc=Y (as-is)propose --label/--desc— analogous splitCommits
31d1466, b8b3bc6