Skip to content

Comments

fix: stop mutating OPENAI env during model setup#27

Open
fmueller wants to merge 1 commit intomainfrom
fix/phase-a-llm-env-isolation
Open

fix: stop mutating OPENAI env during model setup#27
fmueller wants to merge 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

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