Skip to content

feat: progressive comment updates#32

Merged
yoavf merged 3 commits intomainfrom
feat/progressive-comment-updates
Oct 12, 2025
Merged

feat: progressive comment updates#32
yoavf merged 3 commits intomainfrom
feat/progressive-comment-updates

Conversation

@yoavf
Copy link
Owner

@yoavf yoavf commented Oct 10, 2025

Proressively update the comment - from the moment the action starts taking screenshots until it ends

@yoavf yoavf force-pushed the feat/progressive-comment-updates branch from cab4e25 to af0716c Compare October 12, 2025 05:00
@yoavf yoavf requested a review from Copilot October 12, 2025 07:06
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR implements progressive comment updates for screenshot capture, allowing real-time progress feedback in PR comments instead of waiting for all screenshots to complete.

  • Splits comment posting into initial comment creation and progressive updates
  • Adds progress callback to screenshot capture for incremental uploads and comment updates
  • Refactors comment posting functions to support status tracking and real-time updates

Reviewed Changes

Copilot reviewed 3 out of 5 changed files in this pull request and generated 1 comment.

File Description
src/screenshot-capture.ts Adds onProgress callback option to CaptureOptions interface and invokes it after each screenshot
src/index.ts Implements progressive screenshot capture with initial comment posting and incremental updates
src/comment-poster.ts Refactors comment posting into separate initial and update functions with status tracking

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@yoavf yoavf marked this pull request as ready for review October 12, 2025 07:11
@yoavf yoavf merged commit 74e4d55 into main Oct 12, 2025
2 checks passed
@yoavf yoavf deleted the feat/progressive-comment-updates branch October 12, 2025 10:14
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.

2 participants