feat(pr-fixer): add validation step to workflow (#5078)

This commit is contained in:
Daniel 2025-06-24 14:00:35 -05:00 • committed by GitHub
parent f5d4847dce
commit e7ca038cb5
No known key found for this signature in database
GPG key ID: B5690EEEBB952194
4 changed files with 29 additions and 2 deletions

View file

@ -47,6 +47,13 @@
<step>Resolve conflicts by rebasing the PR branch and force-pushing.</step>
</steps>
</phase>
<phase name="validation">
<description>Verify that the pushed changes resolve the issues.</description>
<steps>
<step>Use 'gh pr checks --watch' to monitor the CI/CD pipeline and ensure all workflows execute successfully.</step>
</steps>
</phase>
</main_workflow>
<completion_criteria>

View file

@ -45,4 +45,10 @@
<command tool="gh">gh pr checkout <pr_number_or_url></command>
</template>
</pattern>
<pattern name="watching_pr_checks">
<usage>After pushing changes, use this command to monitor the CI/CD pipeline in real-time.</usage>
<template>
<command tool="gh">gh pr checks --watch</command>
</template>
</pattern>
</common_patterns>

View file

@ -9,8 +9,13 @@
<tool>gh pr checks</tool>
<when>After getting comments, to check the technical status.</when>
<why>Quickly identifies if there are failing automated checks that need investigation.</why>
</priority>
</tool_priorities>
</priority>
<priority level="3">
<tool>gh pr checks --watch</tool>
<when>After pushing a fix, to confirm that the changes have resolved the CI/CD failures.</when>
<why>Provides real-time feedback on whether the fix was successful.</why>
</priority>
</tool_priorities>
<tool_specific_guidance>
<tool name="use_mcp_tool (github: get_pull_request)">

View file

@ -81,6 +81,15 @@
</ask_followup_question>
</tool_use>
</step>
<step number="6">
<description>After pushing the changes, watch the PR checks to confirm the fix.</description>
<tool_use>
<execute_command>
<command>gh pr checks --watch</command>
</execute_command>
</tool_use>
<analysis>Confirm that all checks are passing after the fix.</analysis>
</step>
</workflow>
<key_takeaways>