Skip to content

fix(config): accept documented compose host - #26

Merged
volschin merged 4 commits into
mainfrom
fix/env-boundary-cleanup
Jul 30, 2026
Merged

fix(config): accept documented compose host#26
volschin merged 4 commits into
mainfrom
fix/env-boundary-cleanup

Conversation

@volschin

Copy link
Copy Markdown
Owner

Summary

  • accept the documented Compose-only DGX_HOST_IP key as a validated IPv4/IPv6 compatibility field while preserving strict rejection of every other unknown setting
  • exclude the compatibility field from settings representations and serialization
  • align TODO.md with the stable named Qwen Base clone path and distinguish live-proven production behavior from diagnostic fixture assumptions
  • include the approved design and implementation plan

Verification

  • TDD red: both documented IPv4/IPv6 cases failed with extra_forbidden before the implementation
  • TDD green: valid IPv4/IPv6 accepted; invalid IP and unrelated unknown dotenv key rejected
  • real private repository .env loaded as IPv4Address; compatibility field absent from repr and model_dump
  • venv/bin/pytest -q — 310 passed
  • ruff check agent tests dgx/tts — passed
  • ruff format --check agent tests dgx/tts — 53 files already formatted
  • all three Compose configurations validated
  • git diff --check origin/main...HEAD — passed

Operational cleanup

The already-merged feat/shared-ai-traefik worktree plus its local and remote branches were removed after proving its head was clean and an ancestor of main. The unrelated locked Claude worktree was left untouched.

Live validation

No new runtime deployment is required for this local configuration-boundary fix. The existing Portainer TTS deployment was not changed.

@volschin
volschin merged commit 983511a into main Jul 30, 2026
5 checks passed
@volschin
volschin deleted the fix/env-boundary-cleanup branch July 30, 2026 08:32
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