Skip to content

Rebind memory consolidation workspace roots#32197

Merged
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/dc6e75aef1ffa84a2f66fec6ed3a6fedcf9aa672
Jul 10, 2026
Merged

Rebind memory consolidation workspace roots#32197
copyberry[bot] merged 1 commit into
mainfrom
copyberry/codex-internal-to-codex-oss/dc6e75aef1ffa84a2f66fec6ed3a6fedcf9aa672

Conversation

@copyberry

@copyberry copyberry Bot commented Jul 10, 2026

Copy link
Copy Markdown

Rebind memory consolidation workspace roots

Why

The memory consolidation agent changes its working directory to the memory
root, but applying its sandbox policy directly to the permissions object can
leave workspace roots inherited from the parent configuration.

What changed

Apply the consolidation sandbox policy through Config so its workspace roots
are synchronized with the memory root. Add a test covering the working
directory, workspace roots, and effective legacy sandbox policy.

## Why

The memory consolidation agent changes its working directory to the memory
root, but applying its sandbox policy directly to the permissions object can
leave workspace roots inherited from the parent configuration.

## What changed

Apply the consolidation sandbox policy through `Config` so its workspace roots
are synchronized with the memory root. Add a test covering the working
directory, workspace roots, and effective legacy sandbox policy.

GitOrigin-RevId: dc6e75aef1ffa84a2f66fec6ed3a6fedcf9aa672
@copyberry
copyberry Bot force-pushed the copyberry/codex-internal-to-codex-oss/dc6e75aef1ffa84a2f66fec6ed3a6fedcf9aa672 branch from eacf7a8 to 601b61a Compare July 10, 2026 15:46
@copyberry
copyberry Bot merged commit 601b61a into main Jul 10, 2026
14 of 32 checks passed
@copyberry
copyberry Bot deleted the copyberry/codex-internal-to-codex-oss/dc6e75aef1ffa84a2f66fec6ed3a6fedcf9aa672 branch July 10, 2026 15:47
@github-actions github-actions Bot locked and limited conversation to collaborators Jul 10, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants