Skip to content

test: cover Qoder environment variable parsing - #80

Merged
heimanba merged 1 commit into
mainfrom
codex/qoder-env-parser-regression
Aug 12, 2026
Merged

test: cover Qoder environment variable parsing#80
heimanba merged 1 commit into
mainfrom
codex/qoder-env-parser-regression

Conversation

@heimanba

Copy link
Copy Markdown
Contributor

What changed

  • add a representative agents.yaml fixture with Qoder Agent environment variables
  • verify the public YAML loader preserves the environment_variables map

Why

The Qoder environment-variable support from PR #77 depends on this field surviving YAML schema parsing. A direct regression test prevents future schema changes from silently stripping the values before planning and apply.

Impact

Test-only change. Runtime behavior is unchanged.

Validation

  • bun test packages/sdk/tests/unit/parser.test.ts
  • bun run verify:scoped
  • pre-push bun scripts/verify.ts push (full typecheck, architecture checks, tests, and lint)

Change-Id: Id4732993b917693bf0d6e7a58a0a1ab96b37515b
@heimanba
heimanba force-pushed the codex/qoder-env-parser-regression branch from ffacb41 to c4d1d30 Compare August 12, 2026 03:32
@heimanba
heimanba marked this pull request as ready for review August 12, 2026 03:36
@heimanba
heimanba merged commit 42e667b into main Aug 12, 2026
16 checks passed
@heimanba
heimanba deleted the codex/qoder-env-parser-regression branch August 12, 2026 03:36
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.

1 participant