Repository navigation
Conversation
01d6ed3 to
8b229be
Compare
|
Rebased this PR onto current Verification:
Head: |
8b229be to
038cf41
Compare
|
Rebased this onto current Verification on head
|
Signed-off-by: Emre K <110906681+kocaemre@users.noreply.github.com>
038cf41 to
d975760
Compare
|
Rebased onto Verification: I also checked #561: it carries the explicit workspace network flag only. This PR preserves the full resolved default permission set (including reviewer, approval policy, sandbox, and writable roots) across new/resumed/forked sessions, while explicit non-default modes remain unchanged. I did not modify #561. |
Summary
additionalDirectoriesto workspace-write roots.Closes #477
Test Plan
npm cinpm run test -- src/__tests__/CodexACPAgent/CodexAcpClient.test.ts -t 'preserves resolved prompt permissions|prompt permissions'— 2 passed, 120 skipped.npm run test -- src/__tests__/CodexACPAgent/CodexAcpClient.test.ts— 122 passed.npm run typechecknpm run buildgit diff --check upstream/main..HEADRebased onto
mainat202e66e; conflicts resolved while preserving upstream skipped-MCP-server metadata and app-server-loss handling. #561 overlaps the explicit network flag but not the broader resolved default permission set here; older #368 overlaps roots/network and merits maintainer scope review.