feat(agent-isolation): expose harness-agnostic agent-iso entry point#764
Open
justinmclean wants to merge 2 commits into
Open
feat(agent-isolation): expose harness-agnostic agent-iso entry point#764justinmclean wants to merge 2 commits into
justinmclean wants to merge 2 commits into
Conversation
The env-isolation core of agent-iso.sh already worked for any CLI; this makes that capability explicit and documented: - Add `agent-iso()` shell function (sourced) and a sub-command mode `bash agent-iso.sh agent-iso <cli> [args]` (direct exec) so any harness CLI can be launched with the same credential-strip posture that claude-iso and opencode-iso provide. A symlink named `agent-iso` (no .sh extension) also works — first positional arg is the CLI name. - The Claude-specific --settings sandbox allowRead injection continues to be skipped for all non-claude agents (guard was already in place). - Change `**Harness:** Claude Code, OpenCode` → `**Harness:** agnostic` in tools/agent-isolation/README.md; the env-isolation layer is now available for Codex, Cursor, Gemini CLI, Aider, and any future CLI. - Update docs/vendor-neutrality.md harness table and harness index to reflect agent-isolation moving from portable to agnostic; the hook regenerates the score section automatically. - Update docs/adapters/add-a-harness.md Step 2 and Step 4 to document `agent-iso <cli>` as the ready-made layer-0 wrapper for runtimes without a built-in hook API. - Add tests/test_generic_iso.py covering both the direct-exec and sourced entry points: credential strip, no --settings injection, banner, missing-CLI exit-127, no-CLI-arg usage message, and multiple harness names (codex/cursor/gemini). All 72 agent-isolation tests pass; vendor-neutrality-score and skill-and-tool-validate both pass. Generated-by: Claude (claude-sonnet-4-6)
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
The env-isolation core of agent-iso.sh already worked for any CLI; this makes that capability explicit and documented:
agent-iso()shell function (sourced) and a sub-command modebash agent-iso.sh agent-iso <cli> [args](direct exec) so any harness CLI can be launched with the same credential-strip posture that claude-iso and opencode-iso provide. A symlink namedagent-iso(no .sh extension) also works — first positional arg is the CLI name.**Harness:** Claude Code, OpenCode→**Harness:** agnosticin tools/agent-isolation/README.md; the env-isolation layer is now available for Codex, Cursor, Gemini CLI, Aider, and any future CLI.agent-iso <cli>as the ready-made layer-0 wrapper for runtimes without a built-in hook API.All 72 agent-isolation tests pass; vendor-neutrality-score and skill-and-tool-validate both pass.
Generated-by: Claude (claude-sonnet-4-6)
Type of change
.claude/skills/<name>/) — eval fixtures updated belowtools/<system>/*.md)tools/*/withpyproject.toml)docs/,README.md,CONTRIBUTING.md)projects/_template/)prek, workflows, validators)Test plan
prek run --all-filespassesuv run pytest/ruff check/mypypasses(
PYTHONPATH=tools/skill-evals/src python3 -m skill_evals.runner tools/skill-evals/evals/<skill>/)(a regression test for the bug fixed / the behaviour added — see CONTRIBUTING.md)