ddmitry/adversarial-review
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

docs: document Plan Mode and resume sandbox limitations

Browse Source 
- Add Plan Mode /tmp write limitation to SKILL.md (Step 4) and README
- Document that `codex exec resume` inherits sandbox from original session
- Remove none/low reasoning effort options (minimum is now medium)
- Add .claude to .gitignore (plan files from testing)

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
This commit is contained in:
ddadmin
2026-04-06 22:50:26 +03:00
co-authored by Claude Opus 4.6
parent 0b8891e19e
commit 834a74c798
3 changed files with 18 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files
.gitignore
+2 -1
View File
@@ -1 +1,2 @@
.codex/ .codex
.claude
README.md
+11
View File
@@ -153,6 +153,17 @@ The skill uses XML-structured prompts inspired by adversarial review methodology
See [examples/review-output.md](examples/review-output.md) for a sample See [examples/review-output.md](examples/review-output.md) for a sample
adversarial review output. adversarial review output.
## Known limitations
- **Plan Mode and `/tmp` writes.** In Claude Code Plan Mode, writing review
prompts to `/tmp` may trigger a permission prompt or cause Plan Mode to exit.
This does not affect review correctness — the review mode is already determined,
and only the plan file and temp files are modified.
- **`resume` inherits sandbox.** The `codex exec resume` command does not accept
`-s` (sandbox) — sandbox is inherited from the original session. The first
`codex exec` call sets the sandbox to `read-only`, and all subsequent resume
rounds use the same setting.
## Roadmap ## Roadmap
- [ ] Gemini as alternative reviewer backend - [ ] Gemini as alternative reviewer backend
SKILL.md
+5 -1
View File
@@ -16,7 +16,7 @@ Sends current work for adversarial review through an external AI model (OpenAI C
- `/adversarial-review plan` — force plan review - `/adversarial-review plan` — force plan review
- `/adversarial-review code` — force code review - `/adversarial-review code` — force code review
- `/adversarial-review <file-path>` — review a specific file (argument contains `/` or `.`) - `/adversarial-review <file-path>` — review a specific file (argument contains `/` or `.`)
- Override reasoning: `/adversarial-review xhigh` or `/adversarial-review low` (one of: `none`, `low`, `medium`, `high`, `xhigh`) - Override reasoning: `/adversarial-review xhigh` or `/adversarial-review medium` (one of: `medium`, `high`, `xhigh`)
- Override model: `/adversarial-review model:gpt-5.3-codex` (argument with `model:` prefix) - Override model: `/adversarial-review model:gpt-5.3-codex` (argument with `model:` prefix)
## Instructions ## Instructions
@@ -295,6 +295,8 @@ Flags:
**Prompt delivery:** write the prompt to `/tmp/codex-prompt-${REVIEW_ID}.md` via **Write tool**, then pass via stdin redirection (`- < file`). This avoids shell quoting issues with long XML prompts. **Prompt delivery:** write the prompt to `/tmp/codex-prompt-${REVIEW_ID}.md` via **Write tool**, then pass via stdin redirection (`- < file`). This avoids shell quoting issues with long XML prompts.
**Plan Mode note:** Writing to `/tmp` via Write tool may trigger a permission prompt or exit Plan Mode. This is a known Claude Code limitation — Plan Mode restricts edits to the plan file only. If this happens, it does not affect review correctness: the review mode is already determined, and the skill only edits the plan file and `/tmp` temp files.
```bash ```bash
timeout 600 codex exec \ timeout 600 codex exec \
-m gpt-5.4 \ -m gpt-5.4 \
@@ -391,6 +393,8 @@ timeout 600 codex exec resume ${CODEX_SESSION_ID} \
Use `timeout: 620000` in Bash tool parameters. Use `timeout: 620000` in Bash tool parameters.
**Note:** `resume` does not accept `-s` (sandbox) — sandbox is inherited from the original session. Do not pass `-s` to resume. The `-m` (model) flag is accepted if you need to override the model.
stderr is redirected to temp file for diagnostics. On resume failure, check `/tmp/codex-stderr-${REVIEW_ID}.txt` for details. stderr is redirected to temp file for diagnostics. On resume failure, check `/tmp/codex-stderr-${REVIEW_ID}.txt` for details.
3. Check the result by exit code: 3. Check the result by exit code: