Fix Daily SPDD cache-memory persistence guidance to avoid denied shell writes#31247
Merged
Merged
Conversation
…anner Co-authored-by: pelikhan <4175913+pelikhan@users.noreply.github.com>
Copilot
AI
changed the title
Investigate Daily SPDD cache-memory write denial
Fix Daily SPDD cache-memory persistence guidance to avoid denied shell writes
May 9, 2026
Copilot created this pull request from a session on behalf of
pelikhan
May 9, 2026 17:00
View session
Contributor
There was a problem hiding this comment.
Pull request overview
Updates the Daily SPDD Spec Planner workflow prompt to prevent cache-memory persistence from using shell-based writes that are blocked by the workflow’s restricted bash allowlist.
Changes:
- Adds explicit guidance to persist the rotation state via a tool-based write (instead of shell redirection) to avoid permission denials.
Show a summary per file
| File | Description |
|---|---|
| .github/workflows/daily-spdd-spec-planner.md | Tightens rotation-state persistence guidance to avoid blocked shell writes to cache-memory. |
Copilot's findings
Tip
Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
- Files reviewed: 1/1 changed files
- Comments generated: 1
| - Process up to 5 files per run | ||
| - Continue from next file on the next run | ||
| - If cache is missing, initialize from the start of the sorted file list | ||
| - Persist rotation state using the `write` tool at that exact path (do not use shell write commands for cache updates) |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Bug Fix
What was the bug?
In run
25605587337, the agent could read cache-memory but failed to persist rotation state because it attempted shell-based writes under/tmp/gh-aw/cache-memory/....Those write paths were blocked by the workflow’s restricted shell allowlist, producing
Permission denied and could not request permission from user.How did you fix it?
daily-spdd-spec-planner.mdto require state persistence via thewritetool at the exact rotation file path.Testing
This PR is scoped to investigation + workflow guidance correction; runtime confirmation should come from subsequent workflow executions using the updated prompt.