Skip to content

feat: add typed SMS configuration flow#1383

Open
Zetazzz wants to merge 4 commits into
mainfrom
feat/send-sms-verification-code-flow
Open

feat: add typed SMS configuration flow#1383
Zetazzz wants to merge 4 commits into
mainfrom
feat/send-sms-verification-code-flow

Conversation

@Zetazzz

@Zetazzz Zetazzz commented Jul 17, 2026

Copy link
Copy Markdown
Contributor

Summary

  • add extensible string-based SMS provider configuration to @constructive-io/graphql-types
  • parse SMS_* and DEVSMS_* environment variables in @constructive-io/graphql-env using shared 12factor-env coercion
  • preserve the existing defaults → config → environment → runtime override precedence
  • add a typed SEND_SMS_DRY_RUN switch for non-delivery validation

Validation

  • pnpm --dir graphql/env test -- __tests__/merge.test.ts --runInBand (9/9 passed)
  • pnpm --dir graphql/env build
  • pnpm --dir graphql/types build
  • pnpm --dir packages/12factor-env exec jest --runInBand (29/29 passed)
  • git diff --check
  • post-rebase full CI (build, unit, PG, integration, and AI matrices passed)

Cross-repository flow

This is the Constructive configuration layer for the new SMS verification-code flow.

@Zetazzz
Zetazzz force-pushed the feat/send-sms-verification-code-flow branch from 54e8cb6 to 57b49de Compare July 23, 2026 03:16
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