svc-selkies: honor SELKIES_MODE env var in run script

[DL6ER]

---

• I have read the contributing guideline and understand that I have made the correct modifications

---

Description:

Replace the hardcoded --mode="websockets" in the svc-selkies s6 run script with --mode="${SELKIES_MODE:-websockets}", in both the production exec and the DEV_MODE branch.

Benefits of this PR and context:

Selkies declares a mode setting with env_var: SELKIES_MODE (see src/selkies/settings.py):

{'name': 'mode', 'type': 'str', 'default': 'websockets',
 'env_var': 'SELKIES_MODE',
 'help': "Specify the mode: 'webrtc' or 'websockets'; defaults to websockets"}

But because the run script passes --mode="websockets" on the command line, the hardcoded value wins over the env var. Setting -e SELKIES_MODE=webrtc on the container has no effect; users have to either patch the run script at runtime or live-edit the compiled /run/service/svc-selkies/run.

After this change, the env var works as documented: the default behaviour stays websockets while -e SELKIES_MODE=webrtc actually selects WebRTC mode at startup. Combined with the dual-mode supervisor (separate PRs), this lets users choose the initial mode declaratively.

How Has This Been Tested?

docker run -d -e SELKIES_MODE=webrtc ... webtop:patched
docker exec <id> ps -o pid,cmd | grep selkies
> /lsiopy/bin/python3 /lsiopy/bin/selkies --addr=localhost --mode=webrtc

Without env var:

docker run -d ... webtop:patched
docker exec <id> ps -o pid,cmd | grep selkies
> /lsiopy/bin/python3 /lsiopy/bin/selkies --addr=localhost --mode=websockets

Default behaviour preserved.

Source / References:

• Setting definition: selkies-project/selkies src/selkies/settings.py, field mode with env_var: SELKIES_MODE.
• Affected file in this repo: root/etc/s6-overlay/s6-rc.d/svc-selkies/run.

[Copilot]

Pull request overview

This PR updates the svc-selkies s6 run script so Selkies’ startup mode can be configured via the documented SELKIES_MODE environment variable instead of being overridden by a hardcoded CLI flag.

Changes:

• Replace hardcoded --mode="websockets" with --mode="${SELKIES_MODE:-websockets}" in the DEV_MODE (nodemon) launch path.
• Replace hardcoded --mode="websockets" with --mode="${SELKIES_MODE:-websockets}" in the production exec launch path.

---

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

[LinuxServer-CI]

I am a bot, here are the test results for this PR:
https://ci-tests.linuxserver.io/lspipepr/selkies-base/debiantrixie-aa141909-pkg-aa141909-dev-bf75ba339e1336f269106f2443b9cd22d9fca37a-pr-152/index.html
https://ci-tests.linuxserver.io/lspipepr/selkies-base/debiantrixie-aa141909-pkg-aa141909-dev-bf75ba339e1336f269106f2443b9cd22d9fca37a-pr-152/shellcheck-result.xml

Tag	Passed
amd64-debiantrixie-aa141909-pkg-aa141909-dev-bf75ba339e1336f269106f2443b9cd22d9fca37a-pr-152	✅
arm64v8-debiantrixie-aa141909-pkg-aa141909-dev-bf75ba339e1336f269106f2443b9cd22d9fca37a-pr-152	✅

[thelamer]

See #154 (comment)

Thanks for the PRs, just giving you the logic why this might not be merged for a bit and this target may change depending on live feedback.

[LinuxServer-CI]

This pull request has been automatically marked as stale because it has not had recent activity. This might be due to missing feedback from OP. It will be closed if no further activity occurs. Thank you for your contributions.