← All bounties
S · 50 pts simplex-bot Open

#17 · Add `isScoreboardConfigured()` guard and graceful fallback

+90 pts on merge Claim on GitLab
Submit a merge request that closes this issue to earn points on merge — bounties stack on the standard +40 merge_pr award. New to the repo? See the Contributing guide, and Local Development & SSO to run the app locally (cp .dev.vars.example .dev.vars && pnpm dev, after rebasing on main) and fully test your change before opening the MR.

Description

Context

Per the bot-utils convention, every external service client should have an is*Configured() guard. SimpleX bot needs one for scoreboard so forks without a scoreboard get a clean "This server doesn't have points enabled" instead of a stack trace.

Acceptance criteria

  • [ ] isScoreboardConfigured() returns true only if SCOREBOARD_URL and BOT_API_KEY are both set
  • [ ] All scoreboard-touching commands use featureDisabled('points') when guard returns false
  • [ ] Doc string follows the pattern in packages/shared-utils/src/bot-utils/

Technical hints

  • Mirror an existing guard in packages/shared-utils/src/bot-utils/ (e.g. career-board-client.ts's isCareerBoardConfigured())
  • Export from packages/shared-utils/src/bot-utils/index.ts so both bots share it

--- *Submit a merge request that says Closes #<this issue iid> in the description. On merge to main, the GitLab webhook will auto-award your bounty + the standard merge_pr (40 pts) to your scoreboard account. Email on commit must match your Authentik email.*

Relevant files

No relevant files linked yet.

Hints

No hints yet.

Activity

No points have been awarded for this bounty yet.

Comments

No comments yet. Be the first to weigh in.

↑ Top