You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Found two source-level step name inconsistencies that drift from previously established conventions. Both live in source .md files (so they propagate through every workflow that imports or hosts them).
Summary
One glossary-capitalization mismatch: MCP server should be MCP Server per the project glossary.
1. [Medium Priority] Glossary capitalization: MCP server → MCP Server
Current step name:
.github/workflows/shared/mcp/mempalace.md:46 — Start MemPalace MCP server
The rendered name also appears in the compiled workflow that consumes this shared import (e.g. .github/workflows/daily-fact.lock.yml).
Issue:
The project glossary defines the term as MCP Server (capital S). Every other related step name already follows this casing, for example:
Generate Safe Outputs MCP Server Config
Determine automatic lockdown mode for GitHub MCP Server
Start MCP Scripts HTTP Server
Start Safe Outputs MCP HTTP Server
Start MCP Gateway / Stop MCP Gateway
Only the MemPalace shared step uses lowercase server, making it the odd one out.
Suggested improvement:
Start MemPalace MCP server → Start MemPalace MCP Server
Glossary reference:### MCP Server — defined with capital S.
2. [Medium Priority] Pattern regression: Cache <X> should be Save <X> to cache
Current step names:
.github/workflows/layout-spec-maintainer.md:26 — Cache layout spec data
.github/workflows/prompt-clustering-analysis.md:38 — Cache prompt clustering data
Both render unchanged into the corresponding .lock.yml files.
Issue:
Closed issue #23941 ("Align Cache (X) step names with Save (X) to cache pattern") standardized this verb pattern across the project. The cache-memory step family already follows it:
Save cache-memory to cache (default)
Save cache-memory to cache (focus-areas)
Save cache-memory to cache (repo-audits)
The two workflows above are the only remaining holdouts using the older Cache <X> form.
Suggested improvement:
Cache layout spec data → Save layout spec data to cache
Cache prompt clustering data → Save prompt clustering data to cache
Medium Priority — neither finding is user-facing or behavioral, but both diverge from explicit conventions the project has already standardized, so fixing them keeps step names predictable in logs and the Actions UI.
Step Name Alignment Issues
Found two source-level step name inconsistencies that drift from previously established conventions. Both live in source
.mdfiles (so they propagate through every workflow that imports or hosts them).Summary
MCP servershould beMCP Serverper the project glossary.Cache <X>step name even though the project standardized onSave <X> to cachein closed issue [step-names] Align "Cache (X)" step names with "Save (X) to cache" pattern #23941.Issues Identified
1. [Medium Priority] Glossary capitalization:
MCP server→MCP ServerCurrent step name:
.github/workflows/shared/mcp/mempalace.md:46—Start MemPalace MCP serverThe rendered name also appears in the compiled workflow that consumes this shared import (e.g.
.github/workflows/daily-fact.lock.yml).Issue:
The project glossary defines the term as MCP Server (capital
S). Every other related step name already follows this casing, for example:Generate Safe Outputs MCP Server ConfigDetermine automatic lockdown mode for GitHub MCP ServerStart MCP Scripts HTTP ServerStart Safe Outputs MCP HTTP ServerStart MCP Gateway/Stop MCP GatewayOnly the MemPalace shared step uses lowercase
server, making it the odd one out.Suggested improvement:
Start MemPalace MCP server→Start MemPalace MCP ServerGlossary reference:
### MCP Server— defined with capitalS.2. [Medium Priority] Pattern regression:
Cache <X>should beSave <X> to cacheCurrent step names:
.github/workflows/layout-spec-maintainer.md:26—Cache layout spec data.github/workflows/prompt-clustering-analysis.md:38—Cache prompt clustering dataBoth render unchanged into the corresponding
.lock.ymlfiles.Issue:
Closed issue #23941 ("Align
Cache (X)step names withSave (X) to cachepattern") standardized this verb pattern across the project. The cache-memory step family already follows it:Save cache-memory to cache (default)Save cache-memory to cache (focus-areas)Save cache-memory to cache (repo-audits)The two workflows above are the only remaining holdouts using the older
Cache <X>form.Suggested improvement:
Cache layout spec data→Save layout spec data to cacheCache prompt clustering data→Save prompt clustering data to cacheRelated closed issue: #23941.
Agentic Task Description
To align these step names:
.mdfiles (not the.lock.yml):.github/workflows/shared/mcp/mempalace.mdline 46 →MCP Server.github/workflows/layout-spec-maintainer.mdline 26 →Save layout spec data to cache.github/workflows/prompt-clustering-analysis.mdline 38 →Save prompt clustering data to cachegh aw compile <workflow>.md(and any workflow that importsshared/mcp/mempalace.md, e.g.daily-fact)..lock.ymlfiles show the new names.Related Files
.github/workflows/shared/mcp/mempalace.md.github/workflows/layout-spec-maintainer.md.github/workflows/prompt-clustering-analysis.md.github/workflows/daily-fact.lock.yml,.github/workflows/layout-spec-maintainer.lock.yml,.github/workflows/prompt-clustering-analysis.lock.ymldocs/src/content/docs/reference/glossary.mdPriority
Medium Priority — neither finding is user-facing or behavioral, but both diverge from explicit conventions the project has already standardized, so fixing them keeps step names predictable in logs and the Actions UI.