Skip to content

Add support for org.rdk.AppPackageManager callsign#46

Merged
astolcenburg merged 1 commit into
rdkcentral:mainfrom
astolcenburg:main
Jun 23, 2026
Merged

Add support for org.rdk.AppPackageManager callsign#46
astolcenburg merged 1 commit into
rdkcentral:mainfrom
astolcenburg:main

Conversation

@astolcenburg

Copy link
Copy Markdown
Contributor

Ref: #RDKEAPPRT-862

Copilot AI review requested due to automatic review settings June 23, 2026 11:16

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Adds support for multiple possible Thunder package manager plugin callsigns by detecting which plugin is active at runtime, enabling compatibility with devices exposing org.rdk.AppPackageManager.

Changes:

  • Replace single PACKAGE_MANAGER_CALLSIGN with an ordered list PACKAGE_MANAGER_CALLSIGNS in config.
  • Update RemotePackageManager to detect the active package manager callsign via Controller.status@<callsign>.
  • Centralize Thunder calls into makeRequest() for packageState and install.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
bolt/src/RemotePackageManager.cjs Adds callsign detection/caching and routes package-manager Thunder requests through the active callsign.
bolt/src/config.cjs Replaces single package-manager callsign constant with a prioritized list of supported callsigns.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread bolt/src/RemotePackageManager.cjs
@astolcenburg astolcenburg requested a review from ansu-mathew June 23, 2026 11:22
@astolcenburg astolcenburg merged commit ab6f1c4 into rdkcentral:main Jun 23, 2026
4 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Jun 23, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants