Skip to content

fix: honor req_llm load_dotenv when starting llm_db#536

Merged
mikehostetler merged 3 commits intomainfrom
fix/issue-527-dotenv-llmdb
Mar 23, 2026
Merged

fix: honor req_llm load_dotenv when starting llm_db#536
mikehostetler merged 3 commits intomainfrom
fix/issue-527-dotenv-llmdb

Conversation

@mikehostetler
Copy link
Contributor

Summary

  • make req_llm own llm_db startup so it can set the default dotenv behavior before llm_db starts
  • propagate config :req_llm, load_dotenv: false to llm_db unless :llm_db has its own explicit load_dotenv config
  • add regressions for the issue [Bug]: Environment variables clobbered [redux] #527 startup path, existing env preservation, and explicit :llm_db override behavior

Testing

  • mix format --check-formatted
  • MIX_ENV=test mix compile --warnings-as-errors
  • MIX_ENV=test mix test test/req_llm/application_test.exs
  • MIX_ENV=test mix test

Closes #527

@mikehostetler mikehostetler merged commit adaf1fe into main Mar 23, 2026
6 checks passed
@mikehostetler mikehostetler deleted the fix/issue-527-dotenv-llmdb branch March 23, 2026 13:24
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.

[Bug]: Environment variables clobbered [redux]

1 participant