Skip to content

Normalize non-release agentic_commands.yml compiler metadata to dev#31829

Merged
pelikhan merged 5 commits into
mainfrom
copilot/set-compiler-version-dev-header
May 13, 2026
Merged

Normalize non-release agentic_commands.yml compiler metadata to dev#31829
pelikhan merged 5 commits into
mainfrom
copilot/set-compiler-version-dev-header

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 13, 2026

Bug Fix

What was the bug?

agentic_commands.yml embedded the current non-release build SHA in its compiler_version header metadata. That made locally/regenerated workflow output vary by build and caused unnecessary merge conflicts.

How did you fix it?

  • Centralized workflow metadata

    • Change centralized slash-command workflow generation to emit compiler_version: "dev" for non-release builds.
    • Preserve the existing behavior for release builds so tagged releases still write the actual release version.
  • Output stability

    • Regenerate .github/workflows/agentic_commands.yml so the checked-in header now matches the normalized non-release behavior.
    • This keeps dev-generated output deterministic across local builds and CI runs that are not release builds.
  • Coverage

    • Add focused tests for both paths:
      • non-release build → compiler_version: "dev"
      • release build → compiler_version: "<release version>"

Example

# Before
# gh-aw-commands: {"payload_version":"v1","schema_version":"v1","compiler_version":"6fcd6e2",...}

# After
# gh-aw-commands: {"payload_version":"v1","schema_version":"v1","compiler_version":"dev",...}

Copilot AI changed the title Use dev compiler metadata for non-release centralized workflows Normalize non-release agentic_commands.yml compiler metadata to dev May 13, 2026
Copilot AI requested a review from pelikhan May 13, 2026 00:31
@pelikhan pelikhan marked this pull request as ready for review May 13, 2026 00:31
Copilot AI review requested due to automatic review settings May 13, 2026 00:31
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Fixes non-deterministic compiler_version metadata in the generated centralized slash-command workflow by normalizing non-release builds to emit compiler_version: "dev" while preserving real versions for release builds.

Changes:

  • Update centralized slash-command workflow metadata generation to use "dev" for non-release builds and the actual version for release builds.
  • Regenerate .github/workflows/agentic_commands.yml so the checked-in header reflects the normalized non-release behavior.
  • Add/adjust tests to cover both non-release ("dev") and release ("<version>") compiler metadata paths.
Show a summary per file
File Description
pkg/workflow/central_slash_command_workflow.go Normalize compiler_version to "dev" for non-release builds when emitting gh-aw-commands header metadata.
pkg/workflow/central_slash_command_workflow_test.go Update existing test expectations and add coverage for release vs non-release compiler metadata.
.github/workflows/agentic_commands.yml Regenerate workflow header metadata to use "compiler_version":"dev" for non-release output stability.

Copilot's findings

Tip

Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

  • Files reviewed: 3/3 changed files
  • Comments generated: 1

Comment thread pkg/workflow/central_slash_command_workflow.go Outdated
@pelikhan pelikhan merged commit 049971b into main May 13, 2026
17 of 18 checks passed
@pelikhan pelikhan deleted the copilot/set-compiler-version-dev-header branch May 13, 2026 00:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants