Skip to content

fix(resolver): ignore whitespace-only OPENSPACE_LLM overrides#77

Open
Gujiassh wants to merge 1 commit intoHKUDS:mainfrom
Gujiassh:fix/resolver-trim-whitespace-env
Open

fix(resolver): ignore whitespace-only OPENSPACE_LLM overrides#77
Gujiassh wants to merge 1 commit intoHKUDS:mainfrom
Gujiassh:fix/resolver-trim-whitespace-env

Conversation

@Gujiassh
Copy link
Copy Markdown

Summary

  • treat whitespace-only OPENSPACE_LLM env values as empty overrides
  • avoid skipping host config fallback when OPENSPACE_LLM_API_KEY/OPENSPACE_LLM_API_BASE are only spaces
  • normalize OPENSPACE_CONFIG_PATH by trimming whitespace and returning None when empty
  • add resolver regression tests for whitespace handling

Validation

  • python3 -m unittest tests/test_host_detection_resolver.py

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