Skip to content

[codex] Add configurable LLM settings - #166

Draft
HUAN2022A wants to merge 1 commit into
TencentCloudADP:mainfrom
HUAN2022A:codex/add-llm-config
Draft

[codex] Add configurable LLM settings#166
HUAN2022A wants to merge 1 commit into
TencentCloudADP:mainfrom
HUAN2022A:codex/add-llm-config

Conversation

@HUAN2022A

Copy link
Copy Markdown

Summary

  • Fixes Inconsistent configuration files and useless LLM settings in FULLGUIDE #142 by adding an llm section to config/base_config.yaml and ConfigManager.
  • Uses YAML llm settings as runtime defaults while keeping .env variables as overrides for secrets and local changes.
  • Makes temperature and optional max_tokens configurable through YAML or LLM_TEMPERATURE / LLM_MAX_TOKENS.
  • Updates .env.example and FULLGUIDE.md to document the precedence.
  • Adds focused unittest coverage for config parsing, overrides, and OpenAI request parameters.

Why

FULLGUIDE documented llm overrides, but the code only read LLM_MODEL, LLM_BASE_URL, and LLM_API_KEY from .env. Temperature was hard-coded to 0.3, so runtime config could not affect LLM calls.

Validation

  • .venv\Scripts\python.exe -m unittest tests.test_llm_config
  • .venv\Scripts\python.exe -m py_compile config\config_loader.py config\init.py utils\call_llm_api.py tests\test_llm_config.py
  • Manual smoke test with local .env: LLMCompletionCall returned OK

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.

Inconsistent configuration files and useless LLM settings in FULLGUIDE

1 participant