Skip to content

Comments

fix(iterate-pr): Reorder workflow to catch late review bot feedback#58

Merged
dcramer merged 1 commit intomainfrom
fix/iterate-pr-feedback-timing
Feb 19, 2026
Merged

fix(iterate-pr): Reorder workflow to catch late review bot feedback#58
dcramer merged 1 commit intomainfrom
fix/iterate-pr-feedback-timing

Conversation

@dcramer
Copy link
Member

@dcramer dcramer commented Feb 19, 2026

The iterate-pr skill had a race condition: after CI completes, review bot
feedback (Sentry, Warden, Bugbot, etc.) may not yet be visible. The
workflow checked CI first, then feedback once — but bots often post
seconds after CI checks turn green, causing the skill to declare success
before seeing actionable feedback.

iterate-pr changes:

  • Gather and address existing review feedback before checking CI, so
    comments are handled early
  • Add a post-CI delayed re-check: sleep 10 then re-fetch feedback to
    catch late-arriving review bot comments
  • Update exit conditions to require the post-CI re-check to be clean

agents-md change:

  • Add /skill-creator rule to Writing Rules: always use it when creating
    or updating skills

Gather and address existing review feedback before checking CI status,
and add a post-CI delayed re-check (sleep 10s + re-fetch) to catch
review bot comments that arrive after CI checks turn green.

Also add a skill-creator rule to agents-md writing guidelines.

Co-Authored-By: Claude <noreply@anthropic.com>
@dcramer dcramer marked this pull request as ready for review February 19, 2026 20:06
@dcramer dcramer merged commit 5bf1b98 into main Feb 19, 2026
9 checks passed
@dcramer dcramer deleted the fix/iterate-pr-feedback-timing branch February 19, 2026 20:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant