Skip to content

refactor(askui): delegate controller path to askui-agent-os package#246

Open
mlikasam-askui wants to merge 3 commits intomainfrom
integrate-agent-os-package
Open

refactor(askui): delegate controller path to askui-agent-os package#246
mlikasam-askui wants to merge 3 commits intomainfrom
integrate-agent-os-package

Conversation

@mlikasam-askui
Copy link
Contributor

@mlikasam-askui mlikasam-askui commented Mar 12, 2026

Summary

Delegate AskUI Remote Device Controller path resolution to the askui-agent-os package instead of implementing it in the sdk repo.

Changes

  • Dependency: Add askui-agent-os>=26.1.1 in pyproject.toml.
  • AskUiControllerSettings:
    • Remove component_registry_file, installation_directory, and controller_path_setting (and related env vars).
    • Implement controller_path as AgentOS.controller_path() from askui-agent-os.
    • Drop in-repo logic for registry file, installation directory, and platform-specific paths.
  • Tests: Remove unit tests for the old resolution behavior; keep and adjust tests for controller_args and settings that still exist.

Motivation

  • Single source of truth for controller discovery in askui-agent-os.
  • Less duplication and easier maintenance when resolution rules change.

- Add askui-agent-os>=26.2.1 dependency
- Replace custom controller path resolution (component registry, installation
  directory, ASKUI_CONTROLLER_PATH) with AgentOS.controller_path()
- Remove deprecated installation_directory and related settings
- Simplify AskUiControllerSettings and trim unit tests to match
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