Skip to content

Refactor character asset updater#4

Open
oyekamal wants to merge 1 commit intomainfrom
codex/refactor-update_character_asset_name.py
Open

Refactor character asset updater#4
oyekamal wants to merge 1 commit intomainfrom
codex/refactor-update_character_asset_name.py

Conversation

@oyekamal
Copy link
Copy Markdown
Member

Summary

  • resolve the mouth image configuration path using the module directory and add defensive error handling
  • replace direct printing with structured logging in the asset updater utility
  • add a regression test to ensure asset updates work from arbitrary working directories

Testing

  • pytest tests/test_update_character_asset_name.py -vv

https://chatgpt.com/codex/tasks/task_b_68ecc9228aa4832fb47a3ab0f0dd7ea1

@oyekamal oyekamal self-assigned this Oct 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant