Skip to content

fix(ci): keep the review checkout clean for Pullfrog's checkout_pr - #60

Merged
afonsojramos merged 1 commit into
mainfrom
fix/submission-review-clean-tree
Oct 9, 2026
Merged

afonsojramos merged 1 commit into
mainfrom
fix/submission-review-clean-tree

Conversation

@afonsojramos

Copy link
Copy Markdown
Member

Summary

The Submission review workflow writes the reviewer's prompt to submission-review-prompt.md in the repository root, leaving an untracked file. Pullfrog's checkout_pr tool refuses to switch branches over a dirty tree, so the agent had to delete the file before it could check out the pull request (it recorded this in its learnings). The prompt file is now added to .git/info/exclude, and the step asserts the tree is clean.

Validation

  • Reproduced the step in a fresh clone: with the prompt written, git status --porcelain is empty.
  • actionlint passes; zizmor reports nothing.
  • Re-running the review on an open submission after merge confirms checkout_pr works without the workaround.

@coderabbitai

coderabbitai Bot commented Oct 9, 2026

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 13 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

⚙️ Run configuration
  • Configuration used: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: 7a39801a-6708-4b4f-9846-e97d2e424958

📥 Commits

Reviewing files that changed from the base of the PR and between dd26c78 and 614c10f.


📒 Files selected for processing (1)
  • .github/workflows/submission-review.yml


  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@pullfrog pullfrog Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

✅ No new issues found.

Reviewed changes — reviewed the single CI fix keeping the review checkout clean for checkout_pr:

  • Prompt file excluded from git — Write the prompt step appends submission-review-prompt.md to .git/info/exclude so the generated file no longer leaves the tree dirty.
  • Clean-tree assertion — test -z "$(git status --porcelain)" fails the step loudly if anything else dirties the checkout, instead of surfacing later as a checkout_pr failure.

Pullfrog  | View workflow run | Using muse-spark-1.3-contributor (free via Pullfrog for OSS) | 𝕏

@afonsojramos
afonsojramos merged commit ce8ae4a into main Oct 9, 2026
7 checks passed
@afonsojramos
afonsojramos deleted the fix/submission-review-clean-tree branch October 9, 2026 23:20
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