fix: use get_app().slug for app_bot_login instead of get_user().login#1114

Merged

myakove merged 13 commits into

mainfrom

fix/issue-1113-app-bot-login

Jun 18, 2026

Commits on Jun 16, 2026

fix: use get_app().slug for app_bot_login instead of get_user().login
rnetser
committed
fix: wrap slug access inside github_api_call for retry protection
rnetser
committed
fix: use AppAuth JWT for get_app() instead of installation token
rnetser
committed
fix: let exceptions propagate from get_github_app_slug for retry
rnetser
committed
style: fix ruff formatting
rnetser
committed
perf: cache GitHub App slug at module level
rnetser
committed
refactor: extract _create_github_integration helper and harden slug cache
rnetser
committed
test: add tests for app_bot_login initialization via get_github_app_slug
rnetser
committed
test: assert get_user not called in slug path, fix mock param order
rnetser
committed
fix: move network I/O outside lock, scope slug cache by app-id
rnetser
committed
fix: use Config.get_value for github-app-id instead of root_data
rnetser
committed
fix: fail-fast on missing github-app-id, assert get_value in tests
rnetser
committed
refactor: pass pre-fetched github-app-id to avoid duplicate config lookup
rnetser
committed