Skip to content

Allow unlimited PR diff comment sections - #178

Merged
firewalker06 merged 1 commit into
mainfrom
fizzy-229-unlimited-section-comments
Sep 23, 2026
Merged

firewalker06 merged 1 commit into
mainfrom
fizzy-229-unlimited-section-comments

Conversation

@firewalker06

Copy link
Copy Markdown
Owner

Summary

  • remove the five-section limit in the PR diff composer and server request validation
  • retain per-section line/comment validation and duplicate-range protection
  • show an unbounded section count and cover six-section submission

Closes #229

Verification

  • bundle exec ruby test/remote_server_test.rb
  • bundle exec ruby test/remote_ui_asset_snapshot_test.rb
  • bin/test (fails in existing ManagedAgentTest.assert_poll_stops_stale_unstructured_output: expected stopped process to preserve its SIGTERM exit status)

Research

GitHub's review-comment API models comments per diff range and documents pagination (up to 100 per page), not a five-comment submission cap. Tycho only sends these validated draft sections to an agent; it does not post them to GitHub directly.

@firewalker06
firewalker06 merged commit 1035b40 into main Sep 23, 2026
2 checks passed
@firewalker06
firewalker06 deleted the fizzy-229-unlimited-section-comments branch September 23, 2026 15:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant