Repository navigation
Fix Repo Assist memory migration - #391
Conversation
Move legacy-file cleanup before agent execution so custom validation remains read-only, remove all stale memory entries except notes.json, and restore the one-file limit. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Keep the six-change limit until legacy memory deletions have been persisted. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
|
Compiled and traced this change in The generated order is correct:
For Deedle this now works because its memory branch has already been manually cleaned to contain only However, this PR alone does not migrate a still-dirty remote memory branch. The agent job removes legacy files from its working copy, but I restored |
Summary
notes.json, including legacyrunid.jsonand non-JSON filesmax-file-countto oneThis fixes upgraded repositories whose existing
memory/repo-assistbranch contains files from older Repo Assist memory formats.Validation
gh aw compile workflows/repo-assist.md --approvegh aw compile --validate workflows/repo-assist.mdnotes.jsonremainedSecurity review
The compilation reported a changed memory-validation script and required explicit approval. I reviewed the change: it only removes the validator's filesystem mutations and improves its error message. No secrets, actions, or redirects were added or removed. The new migration step operates only within
/tmp/gh-aw/repo-memory/defaultand preservesnotes.json.