Skip to content

fix(step-names): align Set up Go, Install UV, and Upload source inconsistencies#17386

Merged
pelikhan merged 2 commits intomainfrom
copilot/align-step-names-again
Feb 21, 2026
Merged

fix(step-names): align Set up Go, Install UV, and Upload source inconsistencies#17386
pelikhan merged 2 commits intomainfrom
copilot/align-step-names-again

Conversation

Copy link
Contributor

Copilot AI commented Feb 21, 2026

Three step name inconsistencies across workflow lock files: one capitalization outlier against ~15 conforming workflows, one incorrect tool casing with a non-standard parenthetical, and a divergence between two shared imports.

Changes

  • daily-syntax-error-quality.mdSet up GoSetup Go (sole outlier vs. project convention)
  • copilot-setup-steps.ymlInstall UV (Python package manager)Install uv (official lowercase name; no other Install * step uses a parenthetical descriptor)
  • shared/trending-charts-simple.mdUpload source and dataUpload source files and data (aligns with the identical step in shared/python-dataviz.md)

Recompiled all 8 affected lock files: daily-syntax-error-quality, daily-copilot-token-report, audit-workflows, daily-firewall-report, daily-performance-summary, portfolio-analyst, prompt-clustering-analysis, stale-repo-identifier.

Warning

Firewall rules blocked me from connecting to one or more addresses (expand for details)

I tried to connect to the following addresses, but was blocked by firewall rules:

  • https://api.github.com/repos/actions/setup-go/git/ref/tags/v5
    • Triggering command: /usr/bin/gh gh api /repos/actions/setup-go/git/ref/tags/v5 --jq .object.sha /home/REDACTED/work/gh-aw/gh-aw/.github/workflows/breaking-change-checker.md /home/REDACTED/work/gh-aw/gh-aw/.github/workflows/changeset.md /home/REDACTED/work/gh-aw/gh-aw/.github/workflows/chroma-issue-indexer.md /home/REDACTED/work/gh-aw/gh-aw/.github/workflows/ci-coach.md /home/REDACTED/work/gh-aw/gh-aw/.github/workflows/ci-doctor.md /home/REDACTED/work/gh-aw/gh-aw/.github/workflows/claude-code-user-docs-review.md /home/REDACTED/work/gh-aw/gh-aw/.github/workflows/cli-consistency-checker.md (http block)

If you need me to access, download, or install something from one of these locations, you can either:

Original prompt

This section details on the original issue you should resolve

<issue_title>[step-names] Align step names: fix "Set up Go" outlier, "Install UV" terminology, and shared upload inconsistency</issue_title>
<issue_description>## Step Name Alignment Issues

Found across: .github/workflows/daily-syntax-error-quality.lock.yml, .github/workflows/daily-copilot-token-report.lock.yml, and shared imports.

Summary

Three distinct naming inconsistencies were identified across the workflow lock files: one capitalization outlier for a GitHub action, one incorrect tool name capitalization with a non-standard parenthetical, and an inconsistency between two shared import components.


Issues Identified

1. [High Priority] Inconsistent Naming: "Set up Go" → "Setup Go"

Current step name:

  • daily-syntax-error-quality.lock.yml (source: .github/workflows/daily-syntax-error-quality.md, line 35): Set up Go

Issue:
Every other workflow in the repository uses Setup Go (one word) or Setup Go for CLI build when calling actions/setup-go. The daily-syntax-error-quality.md workflow is the only one using the two-word form Set up Go, which is an outlier against the established project convention.

Count: 1 outlier vs. ~15 workflows using Setup Go / Setup Go for CLI build

Suggested improvement:

  • Change Set up GoSetup Go

2. [High Priority] Terminology Mismatch: "Install UV (Python package manager)" → "Install uv"

Current step name:

  • daily-copilot-token-report.lock.yml (source: .github/workflows/daily-copilot-token-report.md): Install UV (Python package manager)

Issue:
Two problems in one step name:

  1. Wrong capitalization: The tool is officially named uv (lowercase) by Astral, not UV. Using uppercase UV is factually incorrect.
  2. Non-standard parenthetical: No other Install * step in the repository appends a parenthetical explanation. Install GitHub Copilot CLI, Install Claude Code CLI, Install awf binary, etc. all omit parenthetical descriptions.

Additionally, the only other uv step in the repository (mcp-inspector.lock.yml) uses Setup uv (lowercase), showing an inconsistency in both capitalization and verb choice.

Suggested improvement:

  • Change Install UV (Python package manager)Install uv
  • For the verb: since this step runs a curl install script (not an action), Install uv is the correct imperative verb. The mcp-inspector.lock.yml step uses astral-sh/setup-uv action, so Setup uv is appropriate there.

3. [Medium Priority] Inconsistent Shared Import Names: "Upload source and data" vs "Upload source files and data"

Current step names:

  • shared/trending-charts-simple.md (line 26): Upload source and data
  • shared/python-dataviz.md (line 68): Upload source files and data

Issue:
Two shared import files define a semantically identical step (uploading Python source files and data artifacts) with different names. Workflows that import both (e.g. stale-repo-identifier.md) end up with both names in the same compiled lock file. The pattern Upload source files and data is more descriptive and explicit — "files" clarifies what "source" refers to.

Affected workflows (importing trending-charts-simple.md): audit-workflows, daily-firewall-report, daily-performance-summary, portfolio-analyst, prompt-clustering-analysis, stale-repo-identifier

Suggested improvement:

  • Change Upload source and data (in shared/trending-charts-simple.md) → Upload source files and data

Agentic Task Description

To apply these improvements:

  1. daily-syntax-error-quality.md — Find name: Set up Go and change to name: Setup Go
  2. daily-copilot-token-report.md — Find name: Install UV (Python package manager) and change to name: Install uv
  3. shared/trending-charts-simple.md — Find name: Upload source and data and change to name: Upload source files and data
  4. Recompile each affected workflow with gh aw compile (workflow-name).md to regenerate the .lock.yml files
  5. Verify the compiled .lock.yml files reflect the corrected names

Related Files

  • Source workflows: .github/workflows/daily-syntax-error-quality.md, .github/workflows/daily-copilot-token-report.md
  • Shared import: .github/workflows/shared/trending-charts-simple.md
  • Compiled lock files: corresponding .lock.yml files
  • Project glossary: docs/src/content/docs/reference/glossary.md

Priority

This issue is High Priority for items 1 and 2 (factual outlier / wrong tool capitalization) and Medium Priority for item 3 (shared import divergence).


AI generated by Step Name Alignment for daily maintenance

Generated by Step Name Alignment

  • expires <!-- gh-aw-expires: 2026-02-22...

🔒 GitHub Advanced Security automatically protects Copilot coding agent pull requests. You can protect all pull requests by enabling Advanced Security for your repositories. Learn more about Advanced Security.

… data

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Update step names for consistency across workflows fix(step-names): align Set up Go, Install UV, and Upload source inconsistencies Feb 21, 2026
@pelikhan pelikhan marked this pull request as ready for review February 21, 2026 04:23
Copilot AI review requested due to automatic review settings February 21, 2026 04:23
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

Aligns several GitHub Actions step names across source workflow templates and their compiled .lock.yml outputs to match existing naming conventions and avoid duplicate/near-duplicate step labels.

Changes:

  • Renamed the Go setup step in daily-syntax-error-quality from “Set up Go” to “Setup Go” (source + lock).
  • Renamed the uv install step from “Install UV (Python package manager)” to “Install uv” (shared setup steps + dependent lock).
  • Standardized the artifact upload step name to “Upload source files and data” in the shared trending charts import and all affected compiled lock workflows.

Reviewed changes

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

Show a summary per file
File Description
.github/workflows/daily-syntax-error-quality.md Renames the Go setup step to “Setup Go”.
.github/workflows/daily-syntax-error-quality.lock.yml Recompiled output reflecting the updated “Setup Go” step name and metadata hash.
.github/workflows/copilot-setup-steps.yml Renames uv install step to “Install uv” for consistent tool casing and step naming style.
.github/workflows/daily-copilot-token-report.lock.yml Recompiled output reflecting “Install uv” from imported setup steps.
.github/workflows/shared/trending-charts-simple.md Renames upload step to “Upload source files and data” to match shared naming.
.github/workflows/audit-workflows.lock.yml Recompiled output reflecting the updated upload step name and metadata hash.
.github/workflows/daily-firewall-report.lock.yml Recompiled output reflecting the updated upload step name and metadata hash.
.github/workflows/daily-performance-summary.lock.yml Recompiled output reflecting the updated upload step name and metadata hash.
.github/workflows/portfolio-analyst.lock.yml Recompiled output reflecting the updated upload step name and metadata hash.
.github/workflows/prompt-clustering-analysis.lock.yml Recompiled output reflecting the updated upload step name and metadata hash.
.github/workflows/stale-repo-identifier.lock.yml Recompiled output reflecting the updated upload step name and metadata hash.

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

@pelikhan pelikhan merged commit ee7757b into main Feb 21, 2026
120 checks passed
@pelikhan pelikhan deleted the copilot/align-step-names-again branch February 21, 2026 04:29
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 step names: fix "Set up Go" outlier, "Install UV" terminology, and shared upload inconsistency

3 participants