Skip to content

docs: clarify env var fallback for eval interpolation#928

Merged
christso merged 1 commit intomainfrom
fix/927-env-fallback-docs
Apr 4, 2026
Merged

docs: clarify env var fallback for eval interpolation#928
christso merged 1 commit intomainfrom
fix/927-env-fallback-docs

Conversation

@christso
Copy link
Copy Markdown
Collaborator

@christso christso commented Apr 4, 2026

Summary

  • clarify in docs that ${{ VAR_NAME }} resolution reads exported process environment variables directly
  • note that .env files are optional and only augment the environment when present
  • add an explicit regression test covering interpolation with no .env file

Closes #927.

Red

Issue #927 described .env as required in CI. That matched the docs wording before this PR:

  • targets/configuration.mdx said ${{ VARIABLE_NAME }} reads values "from your .env file"
  • quickstart.mdx instructed users to define target env names in .env

Green

Verified on current origin/main and this branch that no .env file is required for interpolation:

CI_TEMPLATE_PATH=/tmp/template bun -e "import { loadTestSuite } from './packages/core/src/evaluation/yaml-parser.ts'; const suite = await loadTestSuite(process.argv[1], process.cwd()); console.log(suite.tests[0].workspace?.template);" /tmp/env-path.eval.yaml

This resolved workspace.template to the exported path directly.

Verification

  • bun test packages/core/test/evaluation/interpolation-integration.test.ts
  • Manual repro: loadTestSuite() resolved ${{ CI_TEMPLATE_PATH }} with no .env present

@cloudflare-workers-and-pages
Copy link
Copy Markdown

Deploying agentv with  Cloudflare Pages  Cloudflare Pages

Latest commit: 54b0e17
Status: ✅  Deploy successful!
Preview URL: https://0bd95836.agentv.pages.dev
Branch Preview URL: https://fix-927-env-fallback-docs.agentv.pages.dev

View logs

@christso christso merged commit 8fe760c into main Apr 4, 2026
4 checks passed
@christso christso deleted the fix/927-env-fallback-docs branch April 4, 2026 01:02
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.

Support reading env vars as fallback when .env file is missing

1 participant