Skip to content

Commit e7241d0

Browse files
committed
fix: add resolution output handling for patch conflicts in Node.js workflow
1 parent 23ba5a2 commit e7241d0

1 file changed

Lines changed: 20 additions & 0 deletions

File tree

.github/workflows/patch-node.yml

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -129,6 +129,17 @@ jobs:
129129
echo "PATCH_STATUS=${PATCH_STATUS:-clean}" >> $GITHUB_ENV
130130
echo "HAS_UNRESOLVED=${HAS_UNRESOLVED:-false}" >> $GITHUB_ENV
131131
echo "CREATE_PR=true" >> $GITHUB_ENV
132+
133+
cd ..
134+
135+
# Store resolution output for PR body if it exists
136+
if [ -f "../node/resolution_output.txt" ]; then
137+
echo "RESOLUTION_OUTPUT<<EOF" >> $GITHUB_ENV
138+
cat ../node/resolution_output.txt >> $GITHUB_ENV
139+
echo "EOF" >> $GITHUB_ENV
140+
else
141+
echo "RESOLUTION_OUTPUT=" >> $GITHUB_ENV
142+
fi
132143
133144
- name: Create Pull Request
134145
if: env.CREATE_PR == 'true'
@@ -144,11 +155,20 @@ jobs:
144155
145156
The workflow automatically attempts to resolve patch conflicts using AI when the OpenAI API key is available.
146157
158+
### Patch Status: ${{ env.PATCH_STATUS }}
159+
160+
${{ env.PATCH_STATUS == 'conflicts' && format('**Conflicts resolved:** {0}/{1}', env.CONFLICTS_RESOLVED, env.TOTAL_CONFLICTS) || '' }}
161+
147162
### Steps to verify:
148163
1. Review the patch changes
149164
2. Test the patched Node.js build
150165
3. Validate functionality
151166
4. Merge if everything looks good
167+
168+
${{ env.RESOLUTION_OUTPUT && '### AI Resolution Details' || '' }}
169+
${{ env.RESOLUTION_OUTPUT && '```' || '' }}
170+
${{ env.RESOLUTION_OUTPUT }}
171+
${{ env.RESOLUTION_OUTPUT && '```' || '' }}
152172
branch: "nodejs-v${{ inputs.nodeVersion }}"
153173
base: "main"
154174
delete-branch: true

0 commit comments

Comments
 (0)