Skip to content

Commit e7ca038

Browse files
authored
feat(pr-fixer): add validation step to workflow (#5078)
1 parent f5d4847 commit e7ca038

File tree

4 files changed

+29
-2
lines changed

4 files changed

+29
-2
lines changed

.roo/rules-pr-fixer/1_workflow.xml

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -47,6 +47,13 @@
4747
<step>Resolve conflicts by rebasing the PR branch and force-pushing.</step>
4848
</steps>
4949
</phase>
50+
51+
<phase name="validation">
52+
<description>Verify that the pushed changes resolve the issues.</description>
53+
<steps>
54+
<step>Use 'gh pr checks --watch' to monitor the CI/CD pipeline and ensure all workflows execute successfully.</step>
55+
</steps>
56+
</phase>
5057
</main_workflow>
5158

5259
<completion_criteria>

.roo/rules-pr-fixer/3_common_patterns.xml

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -45,4 +45,10 @@
4545
<command tool="gh">gh pr checkout <pr_number_or_url></command>
4646
</template>
4747
</pattern>
48+
<pattern name="watching_pr_checks">
49+
<usage>After pushing changes, use this command to monitor the CI/CD pipeline in real-time.</usage>
50+
<template>
51+
<command tool="gh">gh pr checks --watch</command>
52+
</template>
53+
</pattern>
4854
</common_patterns>

.roo/rules-pr-fixer/4_tool_usage.xml

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -9,8 +9,13 @@
99
<tool>gh pr checks</tool>
1010
<when>After getting comments, to check the technical status.</when>
1111
<why>Quickly identifies if there are failing automated checks that need investigation.</why>
12-
</priority>
13-
</tool_priorities>
12+
</priority>
13+
<priority level="3">
14+
<tool>gh pr checks --watch</tool>
15+
<when>After pushing a fix, to confirm that the changes have resolved the CI/CD failures.</when>
16+
<why>Provides real-time feedback on whether the fix was successful.</why>
17+
</priority>
18+
</tool_priorities>
1419

1520
<tool_specific_guidance>
1621
<tool name="use_mcp_tool (github: get_pull_request)">

.roo/rules-pr-fixer/5_examples.xml

Lines changed: 9 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -81,6 +81,15 @@
8181
</ask_followup_question>
8282
</tool_use>
8383
</step>
84+
<step number="6">
85+
<description>After pushing the changes, watch the PR checks to confirm the fix.</description>
86+
<tool_use>
87+
<execute_command>
88+
<command>gh pr checks --watch</command>
89+
</execute_command>
90+
</tool_use>
91+
<analysis>Confirm that all checks are passing after the fix.</analysis>
92+
</step>
8493
</workflow>
8594

8695
<key_takeaways>

0 commit comments

Comments
 (0)