Skip to content

fix: extract CLI wrapper, remove AGENT_EVAL_MODE from .env#800

Merged
christso merged 1 commit intomainfrom
fix/move-eval-mode-out-of-env
Mar 28, 2026
Merged

fix: extract CLI wrapper, remove AGENT_EVAL_MODE from .env#800
christso merged 1 commit intomainfrom
fix/move-eval-mode-out-of-env

Conversation

@christso
Copy link
Copy Markdown
Collaborator

@christso christso commented Mar 28, 2026

Summary

  • Removed AGENT_EVAL_MODE from .env.example — workflow preferences don't belong with secrets
  • Created scripts/agentv_cli.py as a shared deterministic CLI resolver (env var → .env → PATH)
  • Deduplicated _find_agentv() / _find_env_key() from run_tests.py, run_code_graders.py, bench.py
  • Simplified SKILL.md — eval mode comes from user instruction, default agent

Closes #799

Test plan

  • uv run agentv_cli.py eval --help resolves CLI correctly with/without AGENTV_CLI set
  • uv run run_tests.py still works (imports from shared module)
  • uv run run_code_graders.py still works
  • uv run bench.py still works

🤖 Generated with Claude Code

…EVAL_MODE from .env

AGENT_EVAL_MODE was a workflow preference stored alongside API keys in
.env — wrong place for non-secret config. Eval mode is now determined
by user instruction, defaulting to agent mode.

Also deduplicated _find_agentv() / _find_env_key() from three scripts
into a shared agentv_cli.py module that handles CLI resolution
deterministically.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying agentv with  Cloudflare Pages  Cloudflare Pages

Latest commit: a2e2e8c
Status: ✅  Deploy successful!
Preview URL: https://afa322ba.agentv.pages.dev
Branch Preview URL: https://fix-move-eval-mode-out-of-en.agentv.pages.dev

View logs

@christso christso merged commit bb343f0 into main Mar 28, 2026
2 checks passed
@christso christso deleted the fix/move-eval-mode-out-of-env branch March 28, 2026 03:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Move AGENT_EVAL_MODE out of .env, extract shared CLI wrapper

1 participant