Skip to content

Refactor AlarmBridge controller access#165

Open
elahd wants to merge 2 commits intomasterfrom
jkjx63-codex/review-integration-structure-and-mitigate-load-order-issues
Open

Refactor AlarmBridge controller access#165
elahd wants to merge 2 commits intomasterfrom
jkjx63-codex/review-integration-structure-and-mitigate-load-order-issues

Conversation

@elahd
Copy link
Copy Markdown
Member

@elahd elahd commented Jun 4, 2025

Summary

  • lazily create controllers via get_or_create_controller
  • remove eager controller instantiation from AlarmBridge.__init__
  • fetch user controllers from the registry in AuthenticationController
  • update imports for ruff

Testing

  • ruff check --fix .
  • mypy pyalarmdotcomajax (fails: Returning Any, class cannot subclass errors)
  • pytest -q

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

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