Skip to content

fix config and stream timeout validation#4095

Open
hamb1y wants to merge 1 commit into
HKUDS:mainfrom
hamb1y:codex/config-env-validation-4065-4067
Open

fix config and stream timeout validation#4095
hamb1y wants to merge 1 commit into
HKUDS:mainfrom
hamb1y:codex/config-env-validation-4065-4067

Conversation

@hamb1y
Copy link
Copy Markdown
Contributor

@hamb1y hamb1y commented May 29, 2026

Scope

Config and environment validation

Issues Fixed

Key Files

  • nanobot/config/loader.py
  • nanobot/utils/helpers.py
  • nanobot/providers/anthropic_provider.py
  • nanobot/providers/bedrock_provider.py
  • nanobot/providers/openai_codex_provider.py
  • nanobot/providers/openai_compat_provider.py
  • tests/config/test_env_interpolation.py
  • tests/providers/test_custom_provider.py

Validation

  • uv run pytest tests/config/test_env_interpolation.py::test_load_config_invalid_json_fails_closed tests/providers/test_custom_provider.py::test_invalid_stream_idle_timeout_falls_back_to_default tests/providers/test_anthropic_stream_idle.py tests/providers/test_openai_codex_provider.py::test_codex_request_honors_stream_idle_timeout_env -q

@hamb1y hamb1y marked this pull request as ready for review May 29, 2026 17:41
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: invalid config silently falls back to defaults Bug: invalid NANOBOT_STREAM_IDLE_TIMEOUT_S crashes streaming setup

1 participant