Skip to content

Align compiler step name capitalization: "copilot" → "Copilot"#13843

Merged
pelikhan merged 2 commits intomainfrom
copilot/align-compiler-step-name
Feb 5, 2026
Merged

Align compiler step name capitalization: "copilot" → "Copilot"#13843
pelikhan merged 2 commits intomainfrom
copilot/align-compiler-step-name

Conversation

Copy link
Contributor

Copilot AI commented Feb 5, 2026

The compiler generated "Assign copilot to created issues" (lowercase) while the project glossary consistently uses "Copilot" (capitalized) for the GitHub Copilot product name.

Changes

  • Source code updates (3 files):

    • pkg/workflow/compiler_safe_outputs_job.go:239 - Step name generation
    • pkg/workflow/create_issue.go:125 - Step name generation
    • pkg/workflow/compile_outputs_issue_test.go:349 - Test expectation
  • Recompiled workflows (2 files):

    • .github/workflows/breaking-change-checker.lock.yml
    • .github/workflows/duplicate-code-detector.lock.yml

Before/After

// Before
steps = append(steps, "      - name: Assign copilot to created issues\n")

// After
steps = append(steps, "      - name: Assign Copilot to created issues\n")

This aligns with existing step names: "Install GitHub Copilot CLI", "Execute GitHub Copilot CLI", "Copy Copilot session state files", etc.

Original prompt

This section details on the original issue you should resolve

<issue_title>[step-names] Align compiler-generated step name: "copilot" → "Copilot"</issue_title>
<issue_description>## Step Name Alignment Issue

Found during automated step name analysis across all workflow lock files.

Summary

The compiler generates a step name with lowercase "copilot" which doesn't align with the project glossary that consistently uses "Copilot" (capitalized) or "GitHub Copilot CLI" for the full product name.

Issue Identified

Inconsistent capitalization: "copilot" → "Copilot"

Current step name:

  • "Assign copilot to created issues" (lowercase - incorrect)

Issue:
The project glossary and established patterns use "Copilot" (capitalized) when referring to GitHub Copilot. Other step names correctly use:

  • "Install GitHub Copilot CLI" ✓
  • "Execute GitHub Copilot CLI" ✓
  • "Copy Copilot session state files" ✓
  • "Fetch Copilot PR data" ✓
  • "Fetch Copilot session data" ✓
  • "Validate COPILOT_GITHUB_TOKEN secret" ✓

Suggested improvement:

  • "Assign copilot to created issues" → "Assign Copilot to created issues"

Glossary reference: The glossary consistently refers to "GitHub Copilot" or "Copilot" (capitalized), never lowercase "copilot".

Source Locations

This step name is generated by the compiler in two locations:

  1. pkg/workflow/compiler_safe_outputs_job.go:239

    steps = append(steps, "      - name: Assign copilot to created issues\n")
  2. pkg/workflow/create_issue.go:125

    steps = append(steps, "      - name: Assign copilot to created issues\n")

Impact

Workflows affected: 2 compiled workflows

  • .github/workflows/breaking-change-checker.lock.yml (line 1175)
  • .github/workflows/duplicate-code-detector.lock.yml (line 1059)

Priority: Medium - This is a consistency and branding issue. The functionality is correct, but the naming doesn't follow the project's terminology standards.

Agentic Task Description

To fix this issue:

  1. Update compiler code - Change both instances in the Go source files:

    • In pkg/workflow/compiler_safe_outputs_job.go:239
    • In pkg/workflow/create_issue.go:125
    • Change: "Assign copilot to created issues""Assign Copilot to created issues"
  2. Recompile affected workflows - Run compilation to regenerate the .lock.yml files:

    gh aw compile breaking-change-checker.md
    gh aw compile duplicate-code-detector.md
  3. Verify the fix - Confirm the step names are updated in the lock files:

    grep -n "Assign Copilot to created issues" .github/workflows/*.lock.yml
  4. Update tests if needed - Check if any tests reference the old step name:

    • pkg/workflow/compile_outputs_issue_test.go:349 references this step name in tests

Context: Overall Step Name Quality

This issue was found during a comprehensive analysis of all 213 unique step names across 143 workflow lock files. The good news:

Excellent consistency overall! All major patterns are consistent:

  • Setup (component) (21 instances)
  • Install (tool) (20 instances)
  • Download (artifact) (17 instances)
  • Upload (artifact) (18 instances)
  • Create (resource) (10 instances)

Correct imperative mood throughout - No "Installing", "Downloading", etc.

Proper MCP terminology - All instances correctly use "MCP" (capitalized)

Correct Safe Outputs/Safe Inputs usage - Properly using title case in step names

This is the only inconsistency found that doesn't align with the glossary.

Related Files

  • Compiler source: pkg/workflow/compiler_safe_outputs_job.go, pkg/workflow/create_issue.go
  • Test file: pkg/workflow/compile_outputs_issue_test.go
  • Affected workflows: breaking-change-checker.lock.yml, duplicate-code-detector.lock.yml
  • Project glossary: docs/src/content/docs/reference/glossary.md

AI generated by Step Name Alignment for daily maintenance

AI generated by Step Name Alignment

  • expires on Feb 6, 2026, 11:57 PM UTC

Comments on the Issue (you are @copilot in this section)


💡 You can make Copilot smarter by setting up custom instructions, customizing its development environment and configuring Model Context Protocol (MCP) servers. Learn more Copilot coding agent tips in the docs.

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix step name capitalization for Copilot Align compiler step name capitalization: "copilot" → "Copilot" Feb 5, 2026
Copilot AI requested a review from pelikhan February 5, 2026 02:49
@pelikhan pelikhan marked this pull request as ready for review February 5, 2026 02:50
Copilot AI review requested due to automatic review settings February 5, 2026 02:50
@pelikhan pelikhan merged commit 750c4d4 into main Feb 5, 2026
6 of 7 checks passed
@pelikhan pelikhan deleted the copilot/align-compiler-step-name branch February 5, 2026 02:50
Copy link
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

This pull request corrects the capitalization of "copilot" to "Copilot" in compiler-generated step names to align with the project's consistent naming conventions for the GitHub Copilot product name.

Changes:

  • Updated step name from "Assign copilot to created issues" to "Assign Copilot to created issues"
  • Fixed capitalization in both compiler source files and test expectations
  • Recompiled affected workflow lock files to reflect the change

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.

Show a summary per file
File Description
pkg/workflow/create_issue.go Updated step name generation to use "Copilot" (capitalized)
pkg/workflow/compiler_safe_outputs_job.go Updated step name generation to use "Copilot" (capitalized)
pkg/workflow/compile_outputs_issue_test.go Updated test expectation to match corrected step name
.github/workflows/breaking-change-checker.lock.yml Recompiled workflow with corrected step name
.github/workflows/duplicate-code-detector.lock.yml Recompiled workflow with corrected step name

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

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.

[step-names] Align compiler-generated step name: "copilot" → "Copilot"

2 participants

Comments