From 8d62e70131d9fa265b7429e1c160ed157601c5e1 Mon Sep 17 00:00:00 2001 From: nicomiguelino Date: Thu, 9 Jul 2026 15:57:19 -0700 Subject: [PATCH] fix: check both dependencies and devDependencies for @screenly/edge-apps Only devDependencies was checked, so Edge Apps that declare @screenly/edge-apps under dependencies would skip the build step. Fixes #12 --- initialize/action.yml | 2 +- update/action.yml | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/initialize/action.yml b/initialize/action.yml index 0581c23..3511e3b 100644 --- a/initialize/action.yml +++ b/initialize/action.yml @@ -26,7 +26,7 @@ runs: HAS_BUILD_STEP=false if [ -f "package.json" ]; then - SCREENLY_DEP=$(jq -r '.devDependencies["@screenly/edge-apps"] // empty' package.json) + SCREENLY_DEP=$(jq -r '((.dependencies // {}) + (.devDependencies // {}))["@screenly/edge-apps"] // empty' package.json) if [ -n "$SCREENLY_DEP" ]; then HAS_BUILD_STEP=true fi diff --git a/update/action.yml b/update/action.yml index 0afbb04..bc35e88 100644 --- a/update/action.yml +++ b/update/action.yml @@ -24,7 +24,7 @@ runs: HAS_BUILD_STEP=false if [ -f "package.json" ]; then - SCREENLY_DEP=$(jq -r '.devDependencies["@screenly/edge-apps"] // empty' package.json) + SCREENLY_DEP=$(jq -r '((.dependencies // {}) + (.devDependencies // {}))["@screenly/edge-apps"] // empty' package.json) if [ -n "$SCREENLY_DEP" ]; then HAS_BUILD_STEP=true fi