You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@magpie.apache.org by "potiuk (via GitHub)" <gi...@apache.org> on 2026/09/25 17:25:44 UTC

[PR] docs(sandbox): backticks break the gh exclusion; version check skips alias [magpie]

potiuk opened a new pull request, #1384:
URL: https://github.com/apache/magpie/pull/1384

   ## Summary
   
   - `docs/setup/sandbox-troubleshooting.md` (gh `OSStatus -26276` / `HTTP 401` entry): a backtick anywhere in the command keeps `gh` inside the sandbox, even escaped or single-quoted. Added the table row, a sentence on why (the match treats it as a command substitution), and a Fix bullet: pass Markdown titles and bodies with `--body-file` / `--input`, never inline.
   - `setup-isolated-setup-verify` check 5 and `setup-isolated-setup-install` now read the runtime version with `command claude --version`, so the doc-suggested `alias claude=claude-iso` doesn't launch the isolation wrapper just to print a version.
   - `docs/mode-economics.md`: regenerated skill token counts (hook-required).
   
   ## Type of change
   
   - [x] Skill change — wording only, no behaviour change; no eval fixture change
   - [x] Documentation (`docs/`)
   
   ## Test plan
   
   - [x] `prek run --files` on the changed files passes (after `skill-token-count --write`)
   - [x] Measured on macOS, Claude Code 2.1.280, with `"gh *"` in `excludedCommands`:
     - `gh search issues --repo apache/magpie "claude version"` → ran outside the sandbox
     - same with a newline in the argument → outside
     - same with ``"\`claude --version\`"`` → **sandboxed** (`open ~/.config/gh/config.yml: operation not permitted`)
     - same with ``'`claude --version`'`` → **sandboxed**
   - [ ] Eval suite for `setup-isolated-setup-verify` not re-run (wording-only change).
   
   ## Linked issues
   
   Refs #1289
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@magpie.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


Re: [PR] docs(sandbox): backticks break the gh exclusion; version check skips alias [magpie]

Posted by "potiuk (via GitHub)" <gi...@apache.org>.
potiuk merged PR #1384:
URL: https://github.com/apache/magpie/pull/1384


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@magpie.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org