Skip to content

Release: develop -> main#104

Merged
TaprootFreak merged 2 commits into
mainfrom
develop
May 2, 2026
Merged

Release: develop -> main#104
TaprootFreak merged 2 commits into
mainfrom
develop

Conversation

@github-actions
Copy link
Copy Markdown

Automatic Release PR

This PR was automatically created after changes were pushed to develop.

Commits: 1 new commit(s)

Checklist

  • Review all changes
  • Verify CI passes
  • Approve and merge when ready for production

* Accept /start as a subscribe alias

Telegram's first-contact button on a fresh chat sends /start, which
the bot previously fell through silently. Mapping it to the same
handler as /subscribe gives users one-click onboarding without losing
the explicit /subscribe command for users that already know it.

* Align subscribe/unsubscribe replies with monitoring bots

Match the explicit-feedback style used by the new /subscribe self-
service in the monitoring/rangekeeper/lds bots:

- /subscribe when already subscribed: 'You are already subscribed.'
  (was: silent return)
- /subscribe when new: 'You are now subscribed. Use /unsubscribe to stop.'
  (was: 'You are now subscribed.')
- /unsubscribe when not subscribed: 'You are not subscribed.'
  (was: silent return)
- /unsubscribe when subscribed: unchanged

Plus add /start to the telegramHandles list so HelpMessage advertises
it consistently after the alias commit.
The api container runs as the `node` user but the .api persistence
directory (telegram.groups.json, twitter.token.json) was never
explicitly created in the image — when bind-mounted as a Docker
named volume the runtime path is initialised as root:root, and
writeBackupGroups() fails with EACCES on every /subscribe.

Pre-create /app/.api in the image with the correct ownership so
fresh volumes inherit it. Existing volumes still need a one-off
chown (handled separately via the deploy stack).
@TaprootFreak TaprootFreak merged commit adeefe9 into main May 2, 2026
2 of 3 checks 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.

1 participant