Skip to content

Fix repo memory artifact path resolution in push_repo_memory.cjs#7922

Merged
pelikhan merged 2 commits intomainfrom
copilot/investigate-memory-artifact-issue
Dec 27, 2025
Merged

Fix repo memory artifact path resolution in push_repo_memory.cjs#7922
pelikhan merged 2 commits intomainfrom
copilot/investigate-memory-artifact-issue

Conversation

Copy link
Contributor

Copilot AI commented Dec 27, 2025

The push_repo_memory job was failing to find downloaded artifacts because it incorrectly assumed a nested directory structure that doesn't exist.

Problem

When GitHub Actions uploads an artifact from /tmp/gh-aw/repo-memory/default, it stores files at the artifact root. On download to the same path, the script incorrectly looked for files at:

// Incorrect: assumed nested structure
const sourceMemoryPath = path.join(artifactDir, "memory", memoryId);
// Expected: /tmp/gh-aw/repo-memory/default/memory/default/ ❌

This path doesn't exist. The artifact content is directly at artifactDir.

Changes

  • actions/setup/js/push_repo_memory.cjs: Changed line 41 to use artifactDir directly instead of constructing a nested path
// Correct: artifactDir IS the memory directory
const sourceMemoryPath = artifactDir;

This allows workflows using repo-memory (like daily-code-metrics) to successfully persist state to git branches.

Original prompt

Reference: https://github.com/githubnext/gh-aw/actions/runs/20541879130/job/59007544061#step:7:1

Investigate why the repo memory artifact was not found.

Custom agent used: debug-agentic-workflow
Debug and refine agentic workflows using gh-aw CLI tools - analyze logs, audit runs, and improve workflow performance


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot AI changed the title [WIP] Investigate missing repo memory artifact Fix repo memory artifact path resolution in push_repo_memory.cjs Dec 27, 2025
Copilot AI requested a review from pelikhan December 27, 2025 20:08
@pelikhan pelikhan marked this pull request as ready for review December 27, 2025 20:32
@pelikhan pelikhan merged commit 5ec99c6 into main Dec 27, 2025
4 checks passed
@pelikhan pelikhan deleted the copilot/investigate-memory-artifact-issue branch December 27, 2025 20:33
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.

2 participants