Skip to content

Require Repo Assist memory write validation - #398

Merged
dsyme merged 1 commit into
mainfrom
fix/repo-assist-memory-write-check
Oct 10, 2026
Merged

dsyme merged 1 commit into
mainfrom
fix/repo-assist-memory-write-check

Conversation

@dsyme

@dsyme dsyme commented Oct 10, 2026

Copy link
Copy Markdown
Contributor

Summary

Require Repo Assist to parse notes.json locally and receive a successful push_repo_memory MCP result after every memory update, repairing and retrying on failure.

Failure addressed

A Deedle run wrote malformed JSON but did not invoke push_repo_memory; terminal validation then failed after the agent could no longer repair it.

Validation

  • gh aw compile workflows/repo-assist.md --strict --validate --no-emit
  • git diff --check

Security review

This changes agent instructions only. It introduces no secret, action, permission, redirect, network destination, or validator change.

Make JSON parsing and the recoverable push_repo_memory check mandatory after every memory update.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@dsyme
dsyme merged commit c37c599 into main Oct 10, 2026
@dsyme
dsyme deleted the fix/repo-assist-memory-write-check branch October 10, 2026 10:19
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.

1 participant