Skip to content

Comments

fix: stop mutating OPENAI env during model setup#27

Merged
fmueller merged 1 commit intomainfrom
fix/phase-a-llm-env-isolation
Feb 23, 2026
Merged

fix: stop mutating OPENAI env during model setup#27
fmueller merged 1 commit intomainfrom
fix/phase-a-llm-env-isolation

Conversation

@fmueller
Copy link
Owner

Summary

  • remove OpenAISettings.configure_environment() so model setup no longer mutates process-wide OPENAI_* variables
  • update idea/brief/post-edit agent creation paths to rely on explicit provider settings only
  • replace the environment-mutation unit test with a regression test asserting generate_brief leaves OPENAI_* untouched

Validation

  • uv run ruff check
  • uv run mypy
  • uv run pytest

@fmueller fmueller merged commit 32a9874 into main Feb 23, 2026
3 checks passed
@fmueller fmueller deleted the fix/phase-a-llm-env-isolation branch February 23, 2026 19:22
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