From d360320300c7d8666fb89a16a61848147e9fa709 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Wed, 21 Jan 2026 16:06:36 +0000 Subject: [PATCH 1/2] Initial plan From 76215f643a4d553de2126ec2b971dd6d5a08aeea Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Wed, 21 Jan 2026 16:18:03 +0000 Subject: [PATCH 2/2] Update cli-version-checker workflow to require recompile after constant updates - Mark step 2 as **REQUIRED** with explicit instruction that recompile MUST be run - Update guidelines section to make recompile requirement mandatory - Clarifies that recompile is not optional and must always be run after updating constants Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com> --- .github/workflows/cli-version-checker.lock.yml | 8 ++++---- .github/workflows/cli-version-checker.md | 4 ++-- 2 files changed, 6 insertions(+), 6 deletions(-) diff --git a/.github/workflows/cli-version-checker.lock.yml b/.github/workflows/cli-version-checker.lock.yml index a0ea86e4aaf..0794fbe288c 100644 --- a/.github/workflows/cli-version-checker.lock.yml +++ b/.github/workflows/cli-version-checker.lock.yml @@ -807,7 +807,7 @@ jobs: ### Update Process 1. Edit `./pkg/constants/constants.go` with new version(s) - 2. Run `make recompile` to update workflows + 2. **REQUIRED**: Run `make recompile` to update workflows (MUST be run after any constant changes) 3. Verify changes with `git status` 4. **REQUIRED**: Create issue via safe-outputs with detailed analysis (do NOT skip this step) @@ -884,7 +884,7 @@ jobs: - Use commands like `copilot help ` or ` --help` - Compare help output between old and new versions (both main help and subcommand help) - **SAVE TO CACHE**: Store help outputs (main and all subcommands) and version check results in cache-memory - - Test with `make recompile` before creating PR + - **REQUIRED**: Always run `make recompile` after updating constants to regenerate workflow lock files - **DO NOT COMMIT** `*.lock.yml` or `pkg/workflow/js/*.js` files directly ## Common JSON Parsing Issues @@ -931,11 +931,11 @@ jobs: fi ``` + PROMPT_EOF + cat << 'PROMPT_EOF' >> "$GH_AW_PROMPT" **Best Practice**: Combine stderr filtering with jq extraction for most reliable results: ```bash npm view @github/copilot --json 2>/dev/null | jq -r '.version' - PROMPT_EOF - cat << 'PROMPT_EOF' >> "$GH_AW_PROMPT" ``` ## Error Handling diff --git a/.github/workflows/cli-version-checker.md b/.github/workflows/cli-version-checker.md index dd74e5c6f74..e5cd50c838b 100644 --- a/.github/workflows/cli-version-checker.md +++ b/.github/workflows/cli-version-checker.md @@ -164,7 +164,7 @@ For each CLI tool update: ### Update Process 1. Edit `./pkg/constants/constants.go` with new version(s) -2. Run `make recompile` to update workflows +2. **REQUIRED**: Run `make recompile` to update workflows (MUST be run after any constant changes) 3. Verify changes with `git status` 4. **REQUIRED**: Create issue via safe-outputs with detailed analysis (do NOT skip this step) @@ -241,7 +241,7 @@ Template structure: - Use commands like `copilot help ` or ` --help` - Compare help output between old and new versions (both main help and subcommand help) - **SAVE TO CACHE**: Store help outputs (main and all subcommands) and version check results in cache-memory -- Test with `make recompile` before creating PR +- **REQUIRED**: Always run `make recompile` after updating constants to regenerate workflow lock files - **DO NOT COMMIT** `*.lock.yml` or `pkg/workflow/js/*.js` files directly ## Common JSON Parsing Issues