Skip to content

Commit 0a951da

Browse files
authored
Merge pull request #130 from github/copilot/fix-lock-file-sync-issue
Fix codegen-agentic-fix workflow lock file out of sync
2 parents c9396fc + 8df52aa commit 0a951da

4 files changed

Lines changed: 25 additions & 25 deletions

File tree

.github/copilot-instructions.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -222,6 +222,8 @@ Test method names are converted to lowercase snake_case for snapshot filenames t
222222
- **DO NOT** modify test snapshots in `target/copilot-sdk/test/snapshots/` - these come from reference implementation
223223
- **DO NOT** alter the Eclipse formatter configuration in `pom.xml` without team consensus
224224
- **DO NOT** remove or skip Checkstyle or Spotless checks
225+
- **YOU MUST ALWAYS** run `gh aw compile <workflow-name>` after editing any `.github/workflows/*.md` agentic workflow source file to regenerate the corresponding `.lock.yml`. The lock file contains a content hash of the frontmatter — any edit to the `.md` without recompiling will cause the workflow to fail at runtime with a "lock file out of sync" error.
226+
- **DO NOT** edit `.github/workflows/*.lock.yml` directly — these are auto-generated by `gh aw compile` from the `.md` source files.
225227

226228
### Security Guidelines
227229

.github/workflows/codegen-agentic-fix.lock.yml

Lines changed: 17 additions & 18 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

.github/workflows/codegen-agentic-fix.md

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -33,9 +33,8 @@ tools:
3333
toolsets: [context, repos]
3434

3535
safe-outputs:
36-
push_to_pull_request_branch:
37-
max: 3
38-
add_comment:
36+
push-to-pull-request-branch:
37+
add-comment:
3938
target: "*"
4039
max: 5
4140
noop:
@@ -204,7 +203,7 @@ For each attempt:
204203

205204
### Step 5: Push fixes
206205

207-
After `mvn verify` passes, commit all changes and use the `push_to_pull_request_branch` safe-output tool to push to PR #${{ inputs.pr_number }}:
206+
After `mvn verify` passes, commit all changes and use the `push-to-pull-request-branch` safe-output tool to push to PR #${{ inputs.pr_number }}:
208207

209208
```bash
210209
git add -A
@@ -213,13 +212,13 @@ git commit -m "Fix codegen and build failures after @github/copilot update
213212
Automated fix applied by codegen-agentic-fix workflow."
214213
```
215214

216-
Then call the `push_to_pull_request_branch` tool to push your commits to the PR branch.
215+
Then call the `push-to-pull-request-branch` tool to push your commits to the PR branch.
217216

218217
### Step 6: Failure handling
219218

220219
If all 3 attempts fail:
221220

222-
1. Call the `add_comment` tool on PR #${{ inputs.pr_number }} explaining:
221+
1. Call the `add-comment` tool on PR #${{ inputs.pr_number }} explaining:
223222
- What errors remain
224223
- What fixes were attempted
225224
- Whether the issue is in the code generator or handwritten code

.github/workflows/copilot-setup-steps.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,7 @@ jobs:
2626
- name: Install gh-aw extension
2727
uses: github/gh-aw/actions/setup-cli@ce1794953e0ec42adc41b6fca05e02ab49ee21c3 # v0.68.3
2828
with:
29-
version: v0.42.17
29+
version: v0.68.3
3030

3131
# Setup Node.js
3232
- uses: actions/setup-node@48b55a011bda9f5d6aeb4c2d9c7362e8dae4041e # v6

0 commit comments

Comments
 (0)