Skip to content

fix(namegen): retry transient upstream failures in provisioning#247

Merged
aa-parky merged 1 commit intomainfrom
codex/phase5-namegen-retry-hardening
Mar 21, 2026
Merged

fix(namegen): retry transient upstream failures in provisioning#247
aa-parky merged 1 commit intomainfrom
codex/phase5-namegen-retry-hardening

Conversation

@aa-parky
Copy link
Copy Markdown
Collaborator

Summary

  • add bounded retry semantics to character provisioning namegen lookup for transient failures
  • retry request exceptions and retryable HTTP statuses (408/429/5xx)
  • keep non-retryable HTTP and payload validation failures fail-fast
  • expand unit tests to verify retry behavior and no-retry behavior explicitly

Testing

  • PYENV_VERSION=pms pytest -q tests/test_api/test_admin_create_character.py
  • PRE_COMMIT_HOME=/tmp/pre-commit-cache PYENV_VERSION=pms pre-commit run --files src/mud_server/services/character_provisioning.py tests/test_api/test_admin_create_character.py

@aa-parky aa-parky merged commit 670ebe9 into main Mar 21, 2026
13 checks passed
@aa-parky aa-parky deleted the codex/phase5-namegen-retry-hardening branch March 21, 2026 00: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