Skip to content

feat(twilio): add SMS integration MVP and dev-only settings gating#60

Draft
DJJones66 wants to merge 3 commits intomainfrom
feature/twilio-mvp-consolidated
Draft

feat(twilio): add SMS integration MVP and dev-only settings gating#60
DJJones66 wants to merge 3 commits intomainfrom
feature/twilio-mvp-consolidated

Conversation

@DJJones66
Copy link
Copy Markdown
Member

Summary

This PR adds the BrainDrive Twilio SMS MVP and related local/dev integration work so SMS can be tested as a Gateway client without exposing the feature broadly yet.

What’s included

  • Add Twilio SMS settings, webhook handling, test-send flow, and Gateway integration.
  • Add dev-only visibility for the SMS (Twilio) settings tab.
  • Keep the Twilio webhook publicly reachable through the Gateway auth/pre-handler path.
  • Add /twilio proxy support in the Vite dev server for local/dev testing.
  • Update dev compose config so dev installs report install_mode=dev.
  • Add named-tunnel-friendly allowed host support for dev web access.

Why

This keeps SMS testing available for developers and testers while avoiding exposure in local, quickstart, and production installs until the workflow is more fully validated.

Testing

  • Verified targeted Twilio/settings tests pass.
  • Verified config tests pass.
  • Verified client typecheck passes.
  • Verified TypeScript build passes.
  • Verified dev stack now reports install_mode=dev via /api/config.

Notes for testers

  • SMS (Twilio) appears only when running a dev install.
  • Dev stack should be started with the dev compose flow so BRAINDRIVE_INSTALL_MODE=dev is present.
  • Twilio webhook and named tunnel testing should use the generated BrainDrive webhook URL exactly.

Risks / Follow-up

  • Twilio and tunnel setup are environment-sensitive, so end-to-end testing still depends on correct external configuration.
  • The feature is intentionally gated to dev installs for now.
  • Additional polish or broader rollout controls may still be needed before merging to production-facing flows.

- Show SMS (Twilio) settings tab only when install_mode=dev.

- Add dev install-mode support in shared runtime/client config parsing.

- Fix compose.dev to set BRAINDRIVE_INSTALL_MODE=dev.

- Keep Twilio webhook reachable in gateway pre-handler and dev proxy routing.

- Add Vite allowed host config for named tunnel testing.
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