Skip to content

[WIP] Align step names to Save (X) to cache pattern#23979

Closed
Copilot wants to merge 1 commit intomainfrom
copilot/step-name-alignment-cache-to-save
Closed

[WIP] Align step names to Save (X) to cache pattern#23979
Copilot wants to merge 1 commit intomainfrom
copilot/step-name-alignment-cache-to-save

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Apr 2, 2026

Thanks for asking me to work on this. I will get started on it and keep this PR's description up to date as I form a plan and make progress.


This section details on the original issue you should resolve

<issue_title>[step-names] Align "Cache (X)" step names with "Save (X) to cache" pattern</issue_title>
<issue_description>### Step Name Alignment Issues

Found in: multiple .lock.yml files

Summary

Several workflows use Cache <X> as a step name for cache-save operations. The established project pattern (as used for Save qmd models to cache, Save qmd index to cache, Save cache-memory to cache (default)) is Save <X> to cache. The Cache <X> naming is ambiguous — it doesn't clarify whether data is being saved or restored — and is inconsistent with the rest of the codebase.

Issues Identified

[High Priority] Inconsistent pattern: "Cache (X)" → "Save (X) to cache"

Affected step names and files:

Current Name Suggested Name Files Affected
Cache node-llama-cpp binaries Save node-llama-cpp binaries to cache daily-doc-healer, daily-doc-updater, dev, developer-docs-consolidator, dictation-prompt, glossary-maintainer, smoke-codex, technical-doc-writer, unbloat-docs, weekly-blog-post-writer (10 files)
Cache layout spec data Save layout spec data to cache layout-spec-maintainer (1 file)
Cache prompt clustering data Save prompt clustering data to cache prompt-clustering-analysis (1 file)
Cache downloaded data Save downloaded data daily-news (1 file — uses repo-memory, not actions/cache)
Cache cache-memory file share data Restore cache-memory file share data ai-moderator (1 file — this step restores data, not saves it)

Issue:
All of these steps either use actions/cache@... to save data to the GitHub Actions cache, or perform a custom save to repo-memory. The established naming pattern (Save <X> to cache) makes the save direction explicit. The ambiguous Cache <X> prefix is used correctly for save operations in qmd-related steps elsewhere, and these should match.

Note: Cache cache-memory file share data in ai-moderator.lock.yml is particularly confusing because it is actually a restore operation, not a save.

Established pattern references:

  • Save qmd models to cache
  • Save qmd index to cache
  • Save cache-memory to cache (default)

Agentic Task Description

To fix these step names:

  1. Update source workflows — Change step names in the .md source files (not the .lock.yml files directly):
    • Cache node-llama-cpp binariesSave node-llama-cpp binaries to cache (in all 10 workflows)
    • Cache layout spec dataSave layout spec data to cache (in layout-spec-maintainer.md)
    • Cache prompt clustering dataSave prompt clustering data to cache (in prompt-clustering-analysis.md)
    • Cache downloaded dataSave downloaded data (in daily-news.md)
    • Cache cache-memory file share dataRestore cache-memory file share data (in ai-moderator.md)
  2. Recompile — Run gh aw compile <workflow>.md for each changed source file
  3. Verify — Confirm updated .lock.yml files reflect the new names

Related Files

  • Compiled workflows: .github/workflows/<name>.lock.yml
  • Naming patterns: /tmp/gh-aw/cache-memory/step-name-alignment/patterns.json

Priority

This issue is High Priority — 14 files affected, and the ambiguous naming makes it harder to understand workflow structure at a glance.


AI generated by Step Name Alignment for daily maintenance

Generated by Step Name Alignment ·

  • expires on Apr 3, 2026, 7:47 PM UTC

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

Copilot AI requested a review from pelikhan April 2, 2026 01:09
@pelikhan pelikhan closed this Apr 2, 2026
@github-actions github-actions Bot deleted the copilot/step-name-alignment-cache-to-save branch April 13, 2026 02:59
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 "Cache (X)" step names with "Save (X) to cache" pattern

2 participants