justinmclean opened a new pull request, #339:
URL: https://github.com/apache/airflow-steward/pull/339
## Summary
Convert tools/agent-isolation/ to a uv Python project by adding
pyproject.toml (hatchling build, pytest dev dependency) and a test suite under
tests/ that exercises the three core shell scripts:
- test_claude_iso.py: verifies env-var stripping (AWS_*, GH_TOKEN,
ANTHROPIC_API_KEY, etc. are absent after exec env -i), passthrough of
whitelisted vars (HOME, PATH, USER, SHELL, LANG), CLAUDE_ISO_ALLOW explicit
injection, and the missing-claude-exits-127 path.
- test_sandbox_add_project_root.py: verifies file creation, allowRead /
allowWrite content, idempotency, merge with existing content, dry-run
(no-write, preview output), non-git-dir graceful exit, and missing-jq graceful
exit.
- test_sandbox_bypass_warn.py: verifies exit-0 for normal payloads, exit-1 +
banner for dangerouslyDisableSandbox:true (with command and description shown),
and the grep pattern robustness (whitespace variants, false value, unrelated
boolean field).
Generated-by: Claude (Opus 4.7)
## Type of change
<!-- Tick all that apply. -->
- [ ] Skill change (`.claude/skills/<name>/`) — eval fixtures updated below
- [ ] Tool / bridge contract (`tools/<system>/*.md`)
- [ ] Python package (`tools/*/` with `pyproject.toml`)
- [ ] Groovy reference impl
- [ ] Cross-cutting (RFC, AGENTS.md, sandbox, privacy-LLM)
- [ ] Documentation (`docs/`, `README.md`, `CONTRIBUTING.md`)
- [ ] Project template (`projects/_template/`)
- [ ] CI / dev loop (`prek`, workflows, validators)
- [X] Other: more tests
## Test plan
- [ ] `prek run --all-files` passes
- [X] For Python packages touched: `uv run pytest` / `ruff check` / `mypy`
passes
- [ ] For Groovy bridges touched: command-line invocation tested end-to-end
- [ ] For skill changes: eval suite passes for the affected skill
(`PYTHONPATH=tools/skill-evals/src python3 -m skill_evals.runner
tools/skill-evals/evals/<skill>/`)
- [ ] For skill *behaviour* changes: a new or updated eval fixture is
included in this PR
(a regression test for the bug fixed / the behaviour added — see
CONTRIBUTING.md)
- [ ] Other:
--
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: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
