Skip to content

chore(config): default all feature flags off in base-config#425

Merged
islandbitcoin merged 1 commit into
mainfrom
chore/base-config-flags-off
Jul 2, 2026
Merged

chore(config): default all feature flags off in base-config#425
islandbitcoin merged 1 commit into
mainfrom
chore/base-config-flags-off

Conversation

@islandbitcoin

Copy link
Copy Markdown
Contributor

Summary

  • set bridge.enabled: false in dev/config/base-config.yaml — with cashout.enabled and topup.enabled already false, the base config now ships with every feature flag off
  • environments must opt in explicitly via their overrides

Context

Flags-off baseline for the v0.6.0 launch posture: anything reading the base config (local dev, unit/integration jest setups, write-sdl) sees bridge/cashout/topup disabled unless explicitly enabled.

Note for TEST/prod: the deployed server loads only /var/yaml/custom.yaml rendered from galoy.config in the deployments repo values — the TEST flags are controlled there (companion PR in lnflash/deployments flips cashout.enabled: false and pins bridge/topup off explicitly for the TEST environment).

Test Plan

  • yarn test:unit — 105 suites / 788 passed with the flag off (bridge specs mock @config)
  • CI integration suite on this PR

🤖 Generated with Claude Code

Set bridge.enabled to false so the dev/base config ships with every
feature flag off (bridge, cashout, topup — the latter two were already
false). Environments that want a feature on must enable it explicitly
in their overrides. Note the deployed server loads only the rendered
custom.yaml (galoy.config in the deployments values) — this file is the
dev/local baseline, so this change also makes local dev match the
flags-off launch posture.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@islandbitcoin islandbitcoin merged commit 03885da into main Jul 2, 2026
14 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.

2 participants