Skip to content

chore(deps): bump @screenly/edge-apps to ^1.1.0#848

Open
nicomiguelino wants to merge 2 commits into
masterfrom
chore/bump-edge-apps-library
Open

chore(deps): bump @screenly/edge-apps to ^1.1.0#848
nicomiguelino wants to merge 2 commits into
masterfrom
chore/bump-edge-apps-library

Conversation

@nicomiguelino

@nicomiguelino nicomiguelino commented Jun 21, 2026

Copy link
Copy Markdown
Contributor

User description

Bumps @screenly/edge-apps from ^1.0.0 to ^1.1.0 across all affected Edge Apps and updates their lockfiles.


PR Type

Enhancement


Description

  • Bump shared Edge Apps toolkit

  • Update affected app dev dependency declarations

  • Keep weather app excluded


Diagram Walkthrough

flowchart LR
  A["@screenly/edge-apps ^1.0.0"]
  B["Affected Edge Apps"]
  C["@screenly/edge-apps ^1.1.0"]
  A -- "devDependency bump" --> B
  B -- "uses" --> C
Loading

File Walkthrough

Relevant files
Dependencies
12 files
package.json
Bump Edge Apps dev dependency                                                       
+1/-1     
package.json
Bump Edge Apps dev dependency                                                       
+1/-1     
package.json
Bump Edge Apps dev dependency                                                       
+1/-1     
package.json
Bump Edge Apps dev dependency                                                       
+1/-1     
package.json
Bump Edge Apps dev dependency                                                       
+1/-1     
package.json
Bump Edge Apps dev dependency                                                       
+1/-1     
package.json
Bump Edge Apps dev dependency                                                       
+1/-1     
package.json
Bump Edge Apps dev dependency                                                       
+1/-1     
package.json
Bump Edge Apps dev dependency                                                       
+1/-1     
package.json
Bump Edge Apps dev dependency                                                       
+1/-1     
package.json
Bump Edge Apps dev dependency                                                       
+1/-1     
package.json
Bump Edge Apps dev dependency                                                       
+1/-1     

- update package.json and bun.lock for all affected Edge Apps
@github-actions

github-actions Bot commented Jun 21, 2026

Copy link
Copy Markdown

PR Reviewer Guide 🔍

(Review updated until commit 98f49d2)

Here are some key observations to aid the review process:

⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ No major issues detected

@github-actions

Copy link
Copy Markdown

PR Code Suggestions ✨

Explore these optional code suggestions:

CategorySuggestion                                                                                                                                    Impact
Possible issue
Remove local dependency path

Replace the local absolute file: dependency for @screenly/edge-apps with a portable
package reference. The current path only exists on one developer machine and will
break installs in CI and for other contributors.

edge-apps/weather/package.json [25]

-"@screenly/edge-apps": "file:/Users/nicomiguelino/Code/GitHub/Screenly/edge-apps-library/screenly-edge-apps-1.1.1.tgz",
+"@screenly/edge-apps": "^1.1.0",
Suggestion importance[1-10]: 9

__

Why: The suggestion is correct: the file:/Users/... dependency for @screenly/edge-apps is non-portable and will likely break CI and other developers' installs. Replacing it with ^1.1.0 also aligns edge-apps/weather/package.json with the other updated packages in this PR.

High

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Edge Apps in edge-apps/ to use @screenly/edge-apps ^1.1.0 and refreshes the corresponding Bun lockfiles to reflect the new dependency graph.

Changes:

  • Bump @screenly/edge-apps from ^1.0.0 to ^1.1.0 across multiple Edge Apps.
  • Update each affected app’s bun.lock to the newly resolved dependency tree (including new transitive deps like @sentry/* and jsdom).

Reviewed changes

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

Show a summary per file
File Description
edge-apps/welcome-app/package.json Bumps @screenly/edge-apps to ^1.1.0.
edge-apps/welcome-app/bun.lock Updates lockfile for the bumped dependency.
edge-apps/weather/package.json Bumps dependency, but includes additional script/dependency changes (review comments added).
edge-apps/weather/bun.lock Updates lockfile, but currently contains a local-path resolution (review comments added).
edge-apps/simple-timer/package.json Bumps @screenly/edge-apps to ^1.1.0.
edge-apps/simple-timer/bun.lock Updates lockfile for the bumped dependency.
edge-apps/simple-table-app/package.json Bumps @screenly/edge-apps to ^1.1.0.
edge-apps/simple-table-app/bun.lock Updates lockfile for the bumped dependency.
edge-apps/simple-message-app/package.json Bumps @screenly/edge-apps to ^1.1.0.
edge-apps/simple-message-app/bun.lock Updates lockfile for the bumped dependency.
edge-apps/qr-code/package.json Bumps @screenly/edge-apps to ^1.1.0.
edge-apps/qr-code/bun.lock Updates lockfile for the bumped dependency.
edge-apps/outlook-calendar/package.json Bumps @screenly/edge-apps to ^1.1.0.
edge-apps/outlook-calendar/bun.lock Updates lockfile for the bumped dependency.
edge-apps/menu-board/package.json Bumps @screenly/edge-apps to ^1.1.0.
edge-apps/menu-board/bun.lock Updates lockfile for the bumped dependency.
edge-apps/grafana/package.json Bumps @screenly/edge-apps to ^1.1.0.
edge-apps/grafana/bun.lock Updates lockfile for the bumped dependency.
edge-apps/google-calendar/package.json Bumps @screenly/edge-apps to ^1.1.0.
edge-apps/google-calendar/bun.lock Updates lockfile for the bumped dependency.
edge-apps/cap-alerting/package.json Bumps @screenly/edge-apps to ^1.1.0.
edge-apps/cap-alerting/bun.lock Updates lockfile for the bumped dependency.
edge-apps/calendar/package.json Bumps @screenly/edge-apps to ^1.1.0.
edge-apps/calendar/bun.lock Updates lockfile for the bumped dependency.
edge-apps/asset-metadata/package.json Bumps @screenly/edge-apps to ^1.1.0.
edge-apps/asset-metadata/bun.lock Updates lockfile for the bumped dependency.

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

Comment thread edge-apps/weather/package.json Outdated
Comment on lines +16 to +17
"format": "edge-apps-scripts format src/ README.md index.html",
"format:check": "edge-apps-scripts format --check src/ README.md index.html",
- weather app uses a local dev tarball and should not be included in this PR
@nicomiguelino nicomiguelino marked this pull request as ready for review June 21, 2026 00:37
@github-actions

Copy link
Copy Markdown

Persistent review updated to latest commit 98f49d2

@github-actions

Copy link
Copy Markdown

PR Code Suggestions ✨

No code suggestions found for the PR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants