RDKEMW-7398 : [AppManager] isInstalled Error Handling (#1551) #1571

madanagopalt · 2025-09-15T16:51:58Z

Reason for change: Fixing isInstalled Error Handling issue.
Test Procedure: verified
Risks: Low
Priority: P2
Signed-off-by: Sourav Dutta [email protected]

Signed-off-by: Sourav Dutta [email protected]

CLAassistant · 2025-09-15T19:25:27Z

Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
7 out of 21 committers have signed the CLA.

✅ ssitar583
✅ yuvaramachandran-gurusamy
✅ egalla204
✅ plaksh002c-public
✅ dkumar798
✅ klu339
✅ AravindanNC
❌ Adrian Muzyka
❌ satlead
❌ rosemarybennyy
❌ nhanasi
❌ pkumbh631
❌ rdkcm-rdke
❌ rekhap2kandhavelan
❌ satya200
❌ anand-ky
❌ sborushevsky
❌ preeja33
❌ arun-madhavan-013
❌ divyang-public
❌ tharun-kumarv

Adrian Muzyka seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
_{You have signed the CLA already but the status is still pending? Let us recheck it.}

Copilot

Pull Request Overview

This PR addresses error handling improvements in the AppManager's isInstalled functionality (RDKEMW-7398) and includes version bumps across multiple middleware components.

Key Changes:

Updated middleware layer package version from 8.3.3.0 to 8.3.4.0
Added sceneset package as a conditional runtime dependency
Updated source revisions (SRCREV) for 30+ middleware components including iarmmgrs, aamp, hdmicec, rfc, and others
Version bumps for numerous packages including devicesettings (1.0.18→1.0.22), rbus (2.6.0→2.9.0), aamp (2.8.0→2.9.2)
Added three new GitHub workflow files for PR validation, FOSSID scanning, and CLA verification

Reviewed Changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
recipes-core/packagegroups/packagegroup-middleware-layer.bb	Bumped package version and added conditional sceneset dependency
conf/include/generic-srcrev.inc	Updated source revisions for 30+ middleware components and added new package entries
conf/include/generic-pkgrev.inc	Updated package versions across middleware stack and consolidated related packages
.github/workflows/validate_pr_desc.yml	Added new workflow to validate PR title and description format
.github/workflows/fossid_integration_stateless_diffscan_target_repo.yml	Added FOSSID security scanning workflow
.github/workflows/cla.yml	Added CLA signature verification workflow
.github/workflows/auto_pr_creation_target_repo_caller.yml	Extended branch pattern to include versioned support branches

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

Copilot

Pull Request Overview

Copilot reviewed 7 out of 7 changed files in this pull request and generated no new comments.

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

Copilot

Pull Request Overview

Copilot reviewed 8 out of 8 changed files in this pull request and generated 4 comments.

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

Copilot · 2025-10-29T19:43:12Z

recipes-core/packagegroups/packagegroup-middleware-layer.bb


 inherit packagegroup volatile-bind-gen

 # For interim development and package depolyment to test should be using pre release tags


Corrected spelling of 'depolyment' to 'deployment'.

Suggested change

# For interim development and package depolyment to test should be using pre release tags

# For interim development and package deployment to test should be using pre release tags

Copilot · 2025-10-29T19:43:12Z

recipes-core/packagegroups/packagegroup-middleware-layer.bb

-PV = "8.3.3.0"
+PV = "8.4.1.0"

 # PRs are prefered to be be incremented during development stages for any updates in corresponding


Corrected spelling of 'prefered' to 'preferred' and removed duplicate word 'be'.

Suggested change

# PRs are prefered to be be incremented during development stages for any updates in corresponding

# PRs are preferred to be incremented during development stages for any updates in corresponding

Copilot · 2025-10-29T19:43:12Z

conf/include/generic-pkgrev.inc

-
-PV:pn-iarm-query-powerstate = "1.0.6"
+# All of these iarm-query-powerstate, iarm-set-powerstate, iarm-event-sender,
+# key-simulator and mfr-utils use sys_mon_tools repo.


[nitpick] Consider adding a reference to the sys_mon_tools repository URL or location in this comment to make it easier for developers to locate the shared source.

Suggested change

# key-simulator and mfr-utils use sys_mon_tools repo.

# key-simulator and mfr-utils use sys_mon_tools repo (see: https://github.com/example-org/sys_mon_tools).

Copilot · 2025-10-29T19:43:13Z

.github/workflows/validate_pr_desc.yml

+        
+        echo ""
+        echo "=== Validation Summary ==="
+        echo "PR Title: $([ "$TITLE_VALID" = true ] && echo "PASSED" || echo " FAILED")"


Inconsistent spacing in validation summary: 'PASSED' and ' FAILED' have different leading spaces. The space before 'FAILED' should be removed for consistent output formatting.

Suggested change

echo "PR Title: $([ "$TITLE_VALID" = true ] && echo "PASSED" || echo " FAILED")"

echo "PR Title: $([ "$TITLE_VALID" = true ] && echo "PASSED" || echo "FAILED")"

Copilot

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

Change-Id: I7e3bb29e11db4f44ccacec25e2c06c3284dabfea

…ic-support into topic/RDKEMW-9458 Change-Id: Ic73851602e0a3fb3899ffa71de81d53afa4ca502

Rebase and merge

* Update generic-pkgrev.inc * Update generic-srcrev.inc * Update generic-pkgrev.inc --------- Co-authored-by: Vismal S Kumar <[email protected]>

RDKEMW-9667: Update generic-pkgrev.inc

Copilot

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

* Update generic-srcrev.inc * Update generic-pkgrev.inc * RDKEMW-9558: Cleanup ripple calls in lifecyclemanager --------- Co-authored-by: Anand Kandasamy <[email protected]>

RDKEMW-9530: download manager service updates

Copilot

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

RDKEMW-9849: Move memcapture to middleware layer

…s-cpc

RDKEMW-10715: Add RDKAT-ATSPI2 unit tests and bug fixes

Reason for change: Update Firebolt C++ to a version that is JSON-RPC compliant and no longer depends on Thunder. Set initial versions of Firebolt C++ recipes. Risks: None Change-Id: I724a08afd7ebfb5bee1e4b5c82bbd256495858ec Signed-off-by: Tomasz Blaszczak <[email protected]> Co-authored-by: Anand Kandasamy <[email protected]>

Copilot

Pull request overview

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

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

Copilot · 2025-12-19T16:15:35Z

conf/include/generic-pkgrev.inc

+PV:pn-systimemgrinetrface = "1.4.0"
 PR:pn-systimemgrinetrface = "r0"
 PACKAGE_ARCH:pn-systimemgrinetrface = "${MIDDLEWARE_ARCH}"


The package name "systimemgrinetrface" appears to be a misspelling of "systimemgrinterface". The word "inetrface" should be "interface".

Suggested change

PV:pn-systimemgrinetrface = "1.4.0"

PR:pn-systimemgrinetrface = "r0"

PACKAGE_ARCH:pn-systimemgrinetrface = "${MIDDLEWARE_ARCH}"

PV:pn-systimemgrinterface = "1.4.0"

PR:pn-systimemgrinterface = "r0"

PACKAGE_ARCH:pn-systimemgrinterface = "${MIDDLEWARE_ARCH}"

Co-authored-by: ikavas409_comcast <[email protected]> Co-authored-by: madanagopalt <[email protected]> Co-authored-by: Anand Kandasamy <[email protected]>

* Update generic-srcrev.inc * Update generic-pkgrev.inc * Update generic-srcrev.inc * Update generic-pkgrev.inc * RDKEMW-10493: copilot fix for static analysis issues --------- Co-authored-by: agampa263 <[email protected]> Co-authored-by: Anand Kandasamy <[email protected]>

Copilot

Pull request overview

Copilot reviewed 8 out of 9 changed files in this pull request and generated 3 comments.

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

Copilot · 2025-12-19T16:34:52Z

conf/include/generic-pkgrev.inc

+PV:pn-systimemgrinetrface = "1.4.0"
 PR:pn-systimemgrinetrface = "r0"
 PACKAGE_ARCH:pn-systimemgrinetrface = "${MIDDLEWARE_ARCH}"


There's a typo in the comment. The word "inetrface" should be "interface".

Suggested change

PV:pn-systimemgrinetrface = "1.4.0"

PR:pn-systimemgrinetrface = "r0"

PACKAGE_ARCH:pn-systimemgrinetrface = "${MIDDLEWARE_ARCH}"

PV:pn-systimemgrinterface = "1.4.0"

PR:pn-systimemgrinterface = "r0"

PACKAGE_ARCH:pn-systimemgrinterface = "${MIDDLEWARE_ARCH}"

Copilot · 2025-12-19T16:34:53Z

conf/include/generic-srcrev.inc

+SRCREV:pn-rialto-gstreamer = "d91ab82133a6071eb1bdc360e59a390e9704925c"
+SRCREV:pn-rialto = "61f6aa83b2dd5abe40dae6a1b64d6c153f2a0eaa"
+SRCREV:pn-rialto-ocdm = "4f8e4556754cde3498d960d47fd0827f997ea43b"
 SRCREV:pn-xmidt-agent = "a42eef93f5f129da6a25f1ce5386b2ee"


The SRCREV for xmidt-agent appears to be incomplete. Git commit hashes should be 40 characters (full SHA-1) or at least 7-8 characters for short hashes. This value "a42eef93f5f129da6a25f1ce5386b2ee" is only 32 characters, which is unusual. Please verify this is the correct commit hash format for this package.

Suggested change

SRCREV:pn-xmidt-agent = "a42eef93f5f129da6a25f1ce5386b2ee"

SRCREV:pn-xmidt-agent = "a42eef93f5f129da6a25f1ce5386b2ee00000000"

Copilot · 2025-12-19T16:34:53Z

.github/workflows/validate_pr_title.yml

+    - name: Validate PR Title
+      env:
+        PR_TITLE: ${{ github.event.pull_request.title }}
+        PR_BODY: ${{ github.event.pull_request.body }}


The environment variable PR_BODY is defined but never used in the script. Consider removing it to avoid confusion, or if it's intended for future use, add a comment explaining its purpose.

Suggested change

PR_BODY: ${{ github.event.pull_request.body }}

Reason for change: move recipe information closer to developer’s repos Test Procedure: see Jira ticket Risks: Low Priority: P1 Signed-off-by: Mahmoud El-haron <[email protected]> Co-authored-by: Anand Kandasamy <[email protected]>

…ic-support into RDKEMW_11604

Version: Major Change-Id: I73f5a40e4c699035a6abb679666c301c0203a1e3

RDKEMW-11604: Migrate Develop branch from entservices-infra/rdkservcies-cpc

Copilot

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

RDKEMW-11621: Rebase with develop

RDKEMW-11621: Backmerge Release branch into Develop for MW 8.4.4.0 Release

github-actions · 2026-01-09T01:51:56Z

Thank you for your submission, we really appreciate it. Like many open-source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution. You can sign the CLA by just posting a Pull Request Comment same as the below format.

I have read the CLA Document and I hereby sign the CLA

17 out of 18 committers have signed the CLA.
✅ (satlead)[https://github.com/satlead]
✅ (Vismalskumar0)[https://github.com/Vismalskumar0]
✅ (madanagopalt)[https://github.com/madanagopalt]
✅ (nhanasi)[https://github.com/nhanasi]
✅ (anand-ky)[https://github.com/anand-ky]
✅ (rdkcm-rdke)[https://github.com/rdkcm-rdke]
✅ (gurpreet319)[https://github.com/gurpreet319]
✅ (yuvaramachandran-gurusamy)[https://github.com/yuvaramachandran-gurusamy]
✅ (ks734)[https://github.com/ks734]
✅ (vjain008)[https://github.com/vjain008]
✅ (divyang-public)[https://github.com/divyang-public]
✅ (muthushiamsankar)[https://github.com/muthushiamsankar]
✅ (AravindanNC)[https://github.com/AravindanNC]
✅ (sindhu-krishnan)[https://github.com/sindhu-krishnan]
✅ (satya200)[https://github.com/satya200]
✅ (egalla204)[https://github.com/egalla204]
✅ (sborushevsky)[https://github.com/sborushevsky]
❌ @rekhap2kandhavelan
_{You can retrigger this bot by commenting recheck in this Pull Request.}_{Posted by the CLA Assistant Lite bot.}

RDKEMW-11560: Update middleware package group

Copilot

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

madanagopalt requested review from a team as code owners September 15, 2025 16:51

Copilot AI review requested due to automatic review settings October 28, 2025 14:28

Copilot AI reviewed Oct 28, 2025

View reviewed changes

Copilot AI review requested due to automatic review settings October 28, 2025 16:14

Copilot AI reviewed Oct 28, 2025

View reviewed changes

Copilot AI review requested due to automatic review settings October 29, 2025 19:41

Copilot AI reviewed Oct 29, 2025

View reviewed changes

Copilot AI review requested due to automatic review settings October 30, 2025 19:24

Copilot AI reviewed Oct 30, 2025

View reviewed changes

satlead and others added 8 commits October 31, 2025 03:23

RDKEMW-9458: Handle Resolutions for Context

ce3a01d

Change-Id: I7e3bb29e11db4f44ccacec25e2c06c3284dabfea

Merge branch 'develop' of github.com:rdkcentral/meta-middleware-gener…

b225fee

…ic-support into topic/RDKEMW-9458 Change-Id: Ic73851602e0a3fb3899ffa71de81d53afa4ca502

Merge pull request #2062 from rdkcentral/develop

b2a0f3e

Rebase and merge

RDKEMW-9530: download manager service updates

60983d8

Update generic-srcrev.inc

ba29959

RDKE-900: Update the tr69hostif latest release (#2049)

94a7ace

* Update generic-pkgrev.inc * Update generic-srcrev.inc * Update generic-pkgrev.inc --------- Co-authored-by: Vismal S Kumar <[email protected]>

Merge branch 'develop' into topic/RDKEMW-9667-tag

0b4c0b5

Merge pull request #2047 from rdkcentral/topic/RDKEMW-9667-tag

4239236

RDKEMW-9667: Update generic-pkgrev.inc

Copilot AI review requested due to automatic review settings October 31, 2025 12:30

Copilot AI reviewed Oct 31, 2025

View reviewed changes

madanagopalt and others added 5 commits October 31, 2025 08:57

RDKEMW-9558: Cleanup ripple calls from lifecyclemanager (#2024)

7da76ad

* Update generic-srcrev.inc * Update generic-pkgrev.inc * RDKEMW-9558: Cleanup ripple calls in lifecyclemanager --------- Co-authored-by: Anand Kandasamy <[email protected]>

Merge branch 'develop' into topic/RDKEMW-9530_1

0a3c01c

Merge branch 'develop' into feature/RDKEMW-9849

22b8a65

Merge branch 'develop' into feature/RDKEMW-9242

cbf8335

Merge pull request #2025 from rdkcentral/topic/RDKEMW-9530_1

8185196

RDKEMW-9530: download manager service updates

Copilot AI review requested due to automatic review settings October 31, 2025 15:25

Copilot AI reviewed Oct 31, 2025

View reviewed changes

nhanasi added 2 commits October 31, 2025 11:46

Merge branch 'develop' into feature/RDKEMW-9849

9b298bf

Merge pull request #2033 from rdkcentral/feature/RDKEMW-9849

1e460b2

RDKEMW-9849: Move memcapture to middleware layer

filipe-norte-red and others added 7 commits December 18, 2025 14:19

RDKEMW-10715: Add RDKAT-ATSPI2 unit tests and bug fixes

75a5184

RDKEMWM-11604:igrate Develop branch from entservices-infra/rdkservcie…

78664e3

…s-cpc

Fix typo

17f1f59

Update SRCREV

ec3a13d

Update SRCREV

0b9f1ad

Merge pull request #2496 from rdkcentral/feature/RDKEMW-10715

89eab77

RDKEMW-10715: Add RDKAT-ATSPI2 unit tests and bug fixes

Copilot AI review requested due to automatic review settings December 19, 2025 16:11

Copilot started reviewing on behalf of anand-ky December 19, 2025 16:12 View session

Copilot AI reviewed Dec 19, 2025

View reviewed changes

IshvarKR and others added 2 commits December 19, 2025 11:26

RDKEMW-2803: [AI2.0][PackageManager ] Add L1 tests (#2504)

4391214

Co-authored-by: ikavas409_comcast <[email protected]> Co-authored-by: madanagopalt <[email protected]> Co-authored-by: Anand Kandasamy <[email protected]>

Copilot AI review requested due to automatic review settings December 19, 2025 16:31

Copilot started reviewing on behalf of anand-ky December 19, 2025 16:32 View session

Copilot AI reviewed Dec 19, 2025

View reviewed changes

melhar098 and others added 6 commits December 19, 2025 14:56

Updating the changelog and revision in Recipe file for 1.14.0

e333635

Merge branch 'develop' of github.com:rdkcentral/meta-middleware-gener…

ed602fc

…ic-support into RDKEMW_11604

RDKEMW-11560: Update middleware package group

307214b

Version: Major Change-Id: I73f5a40e4c699035a6abb679666c301c0203a1e3

Merge branch 'develop' into release/1.14.0

fdde922

Merge pull request #2511 from rdkcentral/RDKEMW_11604

23f72f6

RDKEMW-11604: Migrate Develop branch from entservices-infra/rdkservcies-cpc

Copilot AI review requested due to automatic review settings January 5, 2026 15:46

Copilot AI reviewed Jan 5, 2026

View reviewed changes

divyang-public and others added 2 commits January 5, 2026 17:53

Merge pull request #2529 from rdkcentral/develop

9843965

RDKEMW-11621: Rebase with develop

Merge pull request #2526 from rdkcentral/release/1.14.0

23d1bc2

RDKEMW-11621: Backmerge Release branch into Develop for MW 8.4.4.0 Release

madanagopalt and others added 2 commits January 9, 2026 19:38

Merge branch 'develop' into topic/RDKEMW-11551_1

8674771

Merge pull request #2521 from rdkcentral/topic/RDKEMW-11551_1

646e9d5

RDKEMW-11560: Update middleware package group

Copilot AI review requested due to automatic review settings January 9, 2026 18:33

Copilot AI reviewed Jan 9, 2026

View reviewed changes


		inherit packagegroup volatile-bind-gen

		# For interim development and package depolyment to test should be using pre release tags

	# PRs are prefered to be be incremented during development stages for any updates in corresponding
	# PRs are preferred to be incremented during development stages for any updates in corresponding

	# key-simulator and mfr-utils use sys_mon_tools repo.
	# key-simulator and mfr-utils use sys_mon_tools repo (see: https://github.com/example-org/sys_mon_tools).

	echo "PR Title: $([ "$TITLE_VALID" = true ] && echo "PASSED" \|\| echo " FAILED")"
	echo "PR Title: $([ "$TITLE_VALID" = true ] && echo "PASSED" \|\| echo "FAILED")"

	SRCREV:pn-xmidt-agent = "a42eef93f5f129da6a25f1ce5386b2ee"
	SRCREV:pn-xmidt-agent = "a42eef93f5f129da6a25f1ce5386b2ee00000000"

RDKEMW-7398 : [AppManager] isInstalled Error Handling (#1551) #1571

Are you sure you want to change the base?

RDKEMW-7398 : [AppManager] isInstalled Error Handling (#1551) #1571

Uh oh!

Conversation

madanagopalt commented Sep 15, 2025

Uh oh!

CLAassistant commented Sep 15, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Reviewed Changes

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull Request Overview

Uh oh!

Copilot AI Oct 29, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Oct 29, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Oct 29, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Oct 29, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Dec 19, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Dec 19, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 19, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI Dec 19, 2025

Choose a reason for hiding this comment

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

github-actions bot commented Jan 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

CLAassistant commented Sep 15, 2025 •

edited

Loading

github-actions bot commented Jan 9, 2026 •

edited

Loading