Skip to content

Commit d5e7ac2

Browse files
committed
feat: enhance PR validation and monitoring instructions for real-time checks
1 parent ecad9c1 commit d5e7ac2

File tree

4 files changed

+18
-18
lines changed

4 files changed

+18
-18
lines changed

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

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -52,11 +52,10 @@
5252
</phase>
5353

5454
<phase name="validation">
55-
<description>Verify that the pushed changes resolve the issues using non-interactive commands.</description>
55+
<description>Verify that the pushed changes resolve the issues.</description>
5656
<steps>
57-
<step>Use 'gh pr checks --json' to get current check status without interactive monitoring.</step>
58-
<step>Check specific workflow runs with 'gh run list --pr' to verify CI/CD pipeline status.</step>
59-
<step>Use polling approach: wait briefly, then re-check status until all checks complete.</step>
57+
<step>Use 'gh pr checks --watch' to monitor check status in real-time until all checks complete.</step>
58+
<step>If needed, check specific workflow runs with 'gh run list --pr' for detailed CI/CD pipeline status.</step>
6059
<step>Verify that all translation updates (if any) have been completed and committed.</step>
6160
<step>Confirm PR is ready for review by checking mergeable state with 'gh pr view --json'.</step>
6261
</steps>

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

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -66,11 +66,12 @@
6666
</template>
6767
</pattern>
6868

69-
<pattern name="non_interactive_checks">
70-
<usage>Check PR status without interactive monitoring.</usage>
69+
<pattern name="real_time_monitoring">
70+
<usage>Monitor PR checks in real-time as they run.</usage>
7171
<template>
72-
<command tool="gh">gh pr checks <pr_number> --json state,conclusion,name,detailsUrl</command>
73-
<comment>Get specific check details without interactive watch</comment>
72+
<command tool="gh">gh pr checks <pr_number> --watch</command>
73+
<comment>Continuously monitor check status with automatic updates</comment>
74+
<alternative>For one-time status check: gh pr checks <pr_number> --json state,conclusion,name,detailsUrl</alternative>
7475
<command tool="gh">gh run list --pr <pr_number> --json databaseId,status,conclusion</command>
7576
</template>
7677
</pattern>

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

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -75,10 +75,10 @@
7575

7676
<tool name="gh pr checks">
7777
<best_practices>
78-
<practice>Use --json flag to get structured output instead of interactive watch.</practice>
79-
<practice>Format: 'gh pr checks <pr_number> --json state,conclusion,name'</practice>
80-
<practice>Poll periodically instead of using --watch for automated workflows.</practice>
81-
<practice>Use 'gh run list --pr <pr_number>' to get detailed workflow status.</practice>
78+
<practice>Use --watch flag to monitor checks in real-time: 'gh pr checks <pr_number> --watch'</practice>
79+
<practice>For one-time status checks, use --json flag: 'gh pr checks <pr_number> --json state,conclusion,name'</practice>
80+
<practice>The --watch flag automatically updates the display as check statuses change.</practice>
81+
<practice>Use 'gh run list --pr <pr_number>' to get detailed workflow status if needed.</practice>
8282
</best_practices>
8383
</tool>
8484

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

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -82,13 +82,13 @@
8282
</tool_use>
8383
</step>
8484
<step number="6">
85-
<description>After pushing the changes, check PR status using non-interactive commands.</description>
85+
<description>After pushing the changes, monitor PR checks in real-time.</description>
8686
<tool_use>
8787
<execute_command>
88-
<command>gh pr checks 4365 --json state,conclusion,name</command>
88+
<command>gh pr checks 4365 --watch</command>
8989
</execute_command>
9090
</tool_use>
91-
<analysis>Check status without interactive monitoring. Poll periodically if needed until all checks complete.</analysis>
91+
<analysis>Monitor checks continuously until all complete. The --watch flag provides real-time updates as check statuses change.</analysis>
9292
</step>
9393
</workflow>
9494

@@ -178,13 +178,13 @@ Ensure consistency across all language files and maintain the same context and t
178178
</step>
179179

180180
<step number="6">
181-
<description>Verify CI status without interactive monitoring.</description>
181+
<description>Monitor CI status in real-time.</description>
182182
<tool_use>
183183
<execute_command>
184-
<command>gh pr checks 1234 --json state,conclusion,name</command>
184+
<command>gh pr checks 1234 --watch</command>
185185
</execute_command>
186186
</tool_use>
187-
<analysis>Check that all tests pass and translation completeness is maintained using structured output.</analysis>
187+
<analysis>Watch CI checks continuously until all tests pass. The --watch flag provides automatic updates as check statuses change.</analysis>
188188
</step>
189189
</workflow>
190190

0 commit comments

Comments
 (0)