Skip to content

DO NOT MERGE — responder test sandbox#152

Closed
microsasa wants to merge 6 commits intomainfrom
test/responder-sandbox
Closed

DO NOT MERGE — responder test sandbox#152
microsasa wants to merge 6 commits intomainfrom
test/responder-sandbox

Conversation

@microsasa
Copy link
Owner

Test PR for debugging the review-responder agent. Contains intentionally imperfect code. DO NOT MERGE.

Sasa Junuzovic and others added 3 commits March 17, 2026 21:28
Exact match to the version that successfully addressed review
threads (run 23117978475, 2026-03-15), with only two changes:
1. Added if: label gate for aw PRs
2. Removed resolve-pull-request-review-thread safe-output and
   step 6e (orchestrator handles resolution)

No other changes. This commit should not be modified.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Replace pull_request_review trigger with workflow_dispatch + pr_number
input. No more loops from stray review events. Orchestrator or
manual dispatch controls when responder runs.

Removed aw label check and triggering review check — no longer
applicable with workflow_dispatch. Renumbered steps.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
If CI checks fail after addressing review comments, fix the
issues and re-run until they pass. Do not push broken code.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@microsasa microsasa added the aw Created by agentic workflow label Mar 18, 2026
Copilot AI review requested due to automatic review settings March 18, 2026 05:02
Copy link

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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copy link

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.

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Copy link
Owner Author

@microsasa microsasa left a comment

Choose a reason for hiding this comment

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

Review for responder testing

Sasa Junuzovic and others added 2 commits March 17, 2026 22:56
The reply-to-pull-request-review-comment and push-to-pull-request-branch
safe output handlers default to target 'triggering' which requires a
pull_request event context. Setting target to '*' lets the agent
specify the PR number in each message, enabling workflow_dispatch.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Intentionally imperfect code for testing the review-responder agent.
DO NOT MERGE this branch to main.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@github-actions
Copy link
Contributor

Commit pushed: aac05d4

Generated by Review Responder

Copilot AI review requested due to automatic review settings March 18, 2026 06:06
@microsasa microsasa closed this Mar 18, 2026
microsasa pushed a commit that referenced this pull request Mar 19, 2026
MCP tools intermittently return empty results for review data.
The agent sometimes falls back to REST API on its own (PR #152)
but not always (PR #158). Explicitly instruct it to use the
REST API endpoints for comments and reviews.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
microsasa pushed a commit that referenced this pull request Mar 19, 2026
MCP tools intermittently return empty results for review data.
The agent sometimes falls back to REST API on its own (PR #152)
but not always (PR #158). Explicitly instruct it to use the
REST API endpoints for comments and reviews.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

aw Created by agentic workflow review-response-attempted

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants