fix: handle EOF#255
Merged
Merged
Conversation
…st-clarification PR comments - When implement makes no code changes, Claude writes the PR comment body to .comment_body (no-change rationale or request for clarification). Workflow posts it on the PR instead of claiming 'commit(s) pushed'. - Track changes_pushed in implement-verify loop; use conditional PR comment (Claude's text or generic fallback when no .comment_body). - Implement prompt: optional path to write .comment_body for (a) no-change scenarios or (b) when more information is needed. - ensureMetaFiles: skip defaults when .comment_body exists. Reset .comment_body in git so it is never committed. - .gitignore and AGENTS.md: document .comment_body and Request more info vs comment body from implement. Made-with: Cursor
…/pr_body to avoid EOF in content GitHub Actions fails with 'Matching delimiter not found EOF' when .pr_title or .pr_body contains a line that is exactly 'EOF'. Use PR_TITLE_DELIM and PR_BODY_DELIM instead. Made-with: Cursor
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.
This pull request enhances the "issue-auto-implement" GitHub Action to support scenarios where no code changes are made in response to a review or issue. It introduces a
.comment_bodyfile for Claude to communicate no-change rationales or requests for clarification, updates documentation and workflow logic to handle this new flow, and ensures that PR comments accurately reflect whether code changes were made.Support for no-change/clarification scenarios:
.comment_bodyfile when no code changes are needed (e.g., reviewer feedback is a question, current approach is preferred, or clarification is required). The workflow now posts this comment on the PR instead of creating a commit or PR when appropriate. (.github/actions/issue-auto-implement/.gitignore,.github/actions/issue-auto-implement/assess/src/implement.ts, [1] [2] [3] [4] [5]Workflow and output logic updates:
changes_pushedoutput and posting the appropriate PR comment based on whether code changes were made. (.github/actions/issue-auto-implement/action.yml, [1] [2]Documentation improvements:
AGENTS.mdto explain the difference between requesting more information and providing a comment body from implement, and clarified the implement step flow for both code change and no-change cases. (.github/actions/issue-auto-implement/AGENTS.md, [1] [2]