Skip to content

Add skylite-ux install scripts - #1426

Merged
MickLesk merged 4 commits into
community-scripts:mainfrom
bzumhagen:main
Feb 8, 2026
Merged

Add skylite-ux install scripts#1426
MickLesk merged 4 commits into
community-scripts:mainfrom
bzumhagen:main

Conversation

@bzumhagen

@bzumhagen bzumhagen commented Feb 7, 2026

Copy link
Copy Markdown

Scripts which are clearly AI generated and not further revised by the Author of this PR (in terms of Coding Standards and Script Layout) may be closed without review.

✍️ Description

Adds new scripts for installing Skylite UX on proxmox

🔗 Related PR / Issue

Link: #

✅ Prerequisites (X in brackets)

  • Self-review completed – Code follows project standards.
  • Tested thoroughly – Changes work as expected.
  • No breaking changes – Existing functionality remains intact.
  • No security risks – No hardcoded secrets, unnecessary privilege escalations, or permission issues.

🛠️ Type of Change (X in brackets)

  • 🐞 Bug fix – Resolves an issue without breaking functionality.
  • New feature – Adds new, non-breaking functionality.
  • 💥 Breaking change – Alters existing functionality in a way that may require updates.
  • 🆕 New script – A fully functional and tested script or script set.
  • 🌍 Website update – Changes to website-related JSON files or metadata.
  • 🔧 Refactoring / Code Cleanup – Improves readability or maintainability without changing functionality.
  • 📝 Documentation update – Changes to README, AppName.md, CONTRIBUTING.md, or other docs.

🔍 Code & Security Review (X in brackets)

  • Follows Code_Audit.md & CONTRIBUTING.md guidelines
  • Uses correct script structure (AppName.sh, AppName-install.sh, AppName.json)
  • No hardcoded credentials

📋 Additional Information (optional)


📦 Application Requirements (for new scripts)

Required for 🆕 New script submissions.
Pull requests that do not meet these requirements may be closed without review.

  • The application is at least 6 months old
  • The application is actively maintained
  • The application has 600+ GitHub stars
  • Official release tarballs are published
  • I understand that not all scripts will be accepted due to various reasons and criteria by the community-scripts ORG

@bzumhagen
bzumhagen requested review from a team as code owners February 7, 2026 00:13
@greptile-apps

greptile-apps Bot commented Feb 7, 2026

Copy link
Copy Markdown
Contributor

Greptile Overview

Greptile Summary

Added installation scripts for Skylite-UX, a self-hosted family management application with calendar, todos, and shopping lists. The PR includes all three required files (ct/skylite-ux.sh, install/skylite-ux-install.sh, frontend/public/json/skylite-ux.json) following project conventions.

Key Implementation Details:

  • Uses PostgreSQL 16 with proper database setup via setup_postgresql_db function
  • Deploys from GitHub release 2026.2.2 using fetch_and_deploy_gh_release
  • Builds Nuxt.js application with Prisma ORM for database migrations
  • Runs as systemd service with automatic restart on failure
  • Requires 4GB RAM for build process (can be reduced after installation)

Standards Compliance:

  • All files follow required header format with copyright and licensing
  • Uses parameter expansion syntax for variable defaults in ct/skylite-ux.sh
  • Install script ends with proper cleanup sequence (motd_ssh, customize, cleanup_lxc)
  • JSON metadata includes all required schema fields with correct logo CDN format
  • Service creation uses systemctl enable -q --now pattern
  • Heredoc syntax follows standard convention (cat <<EOF>file)
  • Uses Debian 13 as base OS across all files

Confidence Score: 5/5

  • This PR is safe to merge with no issues found
  • All three files strictly adhere to project coding standards and custom rules. The implementation uses approved utility functions (setup_postgresql_db, fetch_and_deploy_gh_release), follows standardized patterns throughout, and includes proper error handling. No security concerns, code quality issues, or deviations from conventions detected.
  • No files require special attention

Important Files Changed

Filename Overview
ct/skylite-ux.sh Container script following all project standards with proper header, variable defaults, update function, and completion message
install/skylite-ux-install.sh Installation script correctly using utility functions, PostgreSQL setup, and proper cleanup sequence
frontend/public/json/skylite-ux.json JSON metadata file with complete schema, correct logo CDN, and helpful notes about build requirements

Comment thread frontend/public/json/skylite-ux.json Outdated
Comment thread install/skylite-ux-install.sh Outdated
Comment thread install/skylite-ux-install.sh Outdated
Comment thread install/skylite-ux-install.sh Outdated
Comment thread install/skylite-ux-install.sh Outdated
Comment thread install/skylite-ux-install.sh Outdated
Comment thread ct/skylite-ux.sh
Comment thread ct/skylite-ux.sh Outdated
Comment thread ct/skylite-ux.sh Outdated
Comment thread ct/skylite-ux.sh Outdated

@CrazyWolf13 CrazyWolf13 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

thanks, looks good to me, @bzumhagen can you check if it works with node 22 or 24?

@bzumhagen

Copy link
Copy Markdown
Author

thanks, looks good to me, @bzumhagen can you check if it works with node 22 or 24?

@CrazyWolf13 yes, I tested with Node 24 and it works fine, so I assume it works fine with Node 22 as well. Would you like me to upgrade the version being used in the installer?

@CrazyWolf13

Copy link
Copy Markdown
Member

@bzumhagen Perfect, if it works fine I guess we can bump to node 24, as 20 will be EOL in april.

@bzumhagen

Copy link
Copy Markdown
Author

K updated to 24

@MickLesk
MickLesk merged commit 46bed0c into community-scripts:main Feb 8, 2026
1 check passed
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.

3 participants