Skip to content

Fix step name hyphenation: cache-memory and repo-memory#10022

Merged
pelikhan merged 4 commits intomainfrom
copilot/fix-step-names-hyphenation
Jan 15, 2026
Merged

Fix step name hyphenation: cache-memory and repo-memory#10022
pelikhan merged 4 commits intomainfrom
copilot/fix-step-names-hyphenation

Conversation

Copy link
Contributor

Copilot AI commented Jan 15, 2026

Step names used "cache memory" and "repo memory" (non-hyphenated) inconsistent with the project glossary which requires hyphenation in technical contexts.

Changes

Source files (3):

  • pkg/workflow/prompts.go - "Append cache memory instructions" → "Append cache-memory instructions"
  • pkg/workflow/repo_memory_prompt.go - "Append repo memory instructions" → "Append repo-memory instructions"
  • pkg/workflow/cache.go - "Restore/Cache cache memory file share data" → "Restore/Cache cache-memory file share data"

Test files (4):

  • Updated assertions in prompts_test.go, cache_memory_*_test.go to match new step names

Compiled workflows (67):

  • Recompiled all .lock.yml files with updated step names

Example

# Before
- name: Append cache memory instructions to prompt
- name: Restore cache memory file share data

# After  
- name: Append cache-memory instructions to prompt
- name: Restore cache-memory file share data

Aligns with existing hyphenated patterns: "Create cache-memory directory", "Download cache-memory artifact", "Upload repo-memory artifact".

Original prompt

This section details on the original issue you should resolve

<issue_title>[step-names] Use hyphenated form for cache-memory and repo-memory in step names</issue_title>
<issue_description>## Step Name Alignment Issue

Priority: Medium
Category: Glossary terminology alignment

Summary

Several step names use "cache memory" and "repo memory" (non-hyphenated) when they should use "cache-memory" and "repo-memory" (hyphenated) according to the project glossary.

Issues Identified

Inconsistent Hyphenation: cache memory → cache-memory

Current step names:

  • "Append cache memory instructions to prompt"
  • "Restore cache memory file share data"
  • "Restore cache memory file share data (focus-areas)"

Issue:
The project glossary defines "Cache Memory" with the guidance: "Use hyphenated form in technical contexts" and provides examples like "cache-memory" and "Create cache-memory directory". These step names are in technical contexts but use the non-hyphenated form.

Suggested improvements:

  • "Append cache-memory instructions to prompt"
  • "Restore cache-memory file share data"
  • "Restore cache-memory file share data (focus-areas)"

Glossary reference: See Cache Memory


Inconsistent Hyphenation: repo memory → repo-memory

Current step names:

  • "Append repo memory instructions to prompt"

Issue:
Similar to cache-memory, the glossary indicates that "repo-memory" should use the hyphenated form in technical contexts, matching patterns seen in steps like "Clone repo-memory branch", "Download repo-memory artifact", etc.

Suggested improvement:

  • "Append repo-memory instructions to prompt"

Glossary reference: See Cache Memory (repo-memory follows the same pattern)


Context

This issue was discovered during the daily step name alignment check on 2026-01-15. The analysis scanned 124 workflows and reviewed 173 unique step names.

Good news: Previous issues identified on 2026-01-14 have been successfully resolved:

  • ✅ "Downloading container images" → "Download container images" (imperative form) - FIXED
  • ✅ "Set up" vs "Setup" inconsistency - FIXED (all now use "Setup")

Impact

Consistency: Most step names (16+ occurrences) correctly use the hyphenated form. The 4 non-hyphenated instances create inconsistency.

Examples of correct usage in other steps:

  • "Create cache-memory directory"
  • "Download cache-memory artifact (default)"
  • "Save cache-memory to cache (focus-areas)"
  • "Clone repo-memory branch (campaigns)"
  • "Upload repo-memory artifact (default)"

Agentic Task Description

To fix these step names:

  1. Locate source workflows - These step names are likely generated by the workflow compiler in shared action templates, not individual workflow .md files
  2. Update templates - Check these locations for the step name definitions:
    • actions/*/action.yml files that define reusable actions
    • Workflow compiler code that generates these standardized steps
    • Shell scripts that append instructions to prompts
  3. Apply hyphenation - Change "cache memory" → "cache-memory" and "repo memory" → "repo-memory" in step name strings
  4. Recompile workflows - Run gh aw compile to regenerate all .lock.yml files
  5. Verify consistency - Confirm all instances now use the hyphenated form

Related Files

Likely locations to check:

  • .github/actions/*/action.yml - Reusable GitHub Actions
  • compiler/ - Workflow compilation code
  • .github/workflows/*.md - Source workflow files (if they explicitly override step names)

Priority Justification

Medium Priority because:

  • ✅ Not misleading (meaning is clear)
  • ⚠️ Inconsistent with established glossary and existing patterns
  • ⚠️ Only 4 occurrences but affects multiple workflows
  • ✅ Easy to fix once the source template is located

🤖 AI generated by Step Name Alignment workflow for daily maintenance

AI generated by Step Name Alignment

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.

Copilot AI and others added 3 commits January 15, 2026 03:29
Co-authored-by: mnkiefer <8320933+mnkiefer@users.noreply.github.com>
Co-authored-by: mnkiefer <8320933+mnkiefer@users.noreply.github.com>
Co-authored-by: mnkiefer <8320933+mnkiefer@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix step names to use hyphenated form for cache-memory and repo-memory Fix step name hyphenation: cache-memory and repo-memory Jan 15, 2026
Copilot AI requested a review from mnkiefer January 15, 2026 03:48
@pelikhan pelikhan marked this pull request as ready for review January 15, 2026 04:42
@pelikhan pelikhan merged commit f34a7e1 into main Jan 15, 2026
@pelikhan pelikhan deleted the copilot/fix-step-names-hyphenation branch January 15, 2026 04:42
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] Use hyphenated form for cache-memory and repo-memory in step names

3 participants