@nauticana/sail 0.5.5 → 0.5.7

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@nauticana/sail",
-  "version": "0.5.5",
+  "version": "0.5.7",
   "description": "Metadata-driven Angular component library for CRUD admin frontends backed by the keel Go backend. Tables, forms, navigation, OTP/2FA/social auth, billing.",
   "main": "src/index.ts",
   "files": [

package/src/component/login/consent_gate.ts

@@ -92,17 +92,26 @@ export class ConsentGateComponent {
       }
       this.formValue.set(this.form.getRawValue());
       this.formValid.set(this.form.valid);
+      this.consentStateChange.emit(this.state());
     });
 
+    // Emit on every form change. We emit explicitly here rather than via a
+    // standalone `effect(() => emit(state()))` because output emission
+    // from inside an effect is unreliable in production AOT builds: the
+    // effect's scheduler runs in a microtask outside the user-event CD
+    // tick, and parent OnPush views can miss the propagated signal write.
+    // Emitting inside the RxJS subscription puts the emission on the
+    // synchronous click-handler path, where Angular's zone-aware CD
+    // picks up the parent's `consents` signal write deterministically.
     this.form.valueChanges.pipe(takeUntilDestroyed(this.destroyRef)).subscribe(() => {
       this.formValue.set(this.form.getRawValue());
       this.formValid.set(this.form.valid);
+      this.consentStateChange.emit(this.state());
     });
     this.form.statusChanges.pipe(takeUntilDestroyed(this.destroyRef)).subscribe(() => {
       this.formValid.set(this.form.valid);
+      this.consentStateChange.emit(this.state());
     });
-
-    effect(() => this.consentStateChange.emit(this.state()));
   }
 
   protected readonly ConsentType = ConsentType;

package/src/model/auth.ts

@@ -43,19 +43,26 @@ export interface SignupConsent {
 }
 
 // ── Phone / email OTP ──
-// `contactType` is a frontend-only convenience field used to drive UI
-// navigation (phone-confirm vs email-confirm). It is NOT sent to the
-// backend — keel detects phone-vs-email from the `contact` value itself.
-// `BaseAuthService.sendOtp` strips this field before posting, since keel
-// rejects unknown JSON fields with 400.
+// `contactType` selects the OTP delivery channel. As of keel v0.5.11
+// the backend honors this field directly: 'phone' (default — `contact`
+// is normalized to E.164 and OTP goes via SMS) or 'email' (`contact`
+// is lowercased + trimmed and OTP goes via the keel MailClient).
+// Older keel deployments ignore anything other than phone — pin the
+// consumer's go.mod when this matters.
 export type OtpContactType = 'phone' | 'email';
 export type OtpPurpose     = 'login' | 'register' | 'verify';
 
 export interface OtpRequest {
   contact:        string;
-  contactType?:   OtpContactType;  // frontend-only, stripped before POST
+  contactType?:   OtpContactType;  // 'phone' (default) | 'email'
   purpose?:       OtpPurpose;
   defaultRegion?: string;          // ISO country hint for phone normalization
+  // The OTHER contact the user supplied at signup, persisted on the
+  // new user_account row even though it isn't being OTP-verified at
+  // this step. When contactType='email', this is the user's phone;
+  // when contactType='phone', this is the user's email. Optional —
+  // omit for login flows (only registration creates the row).
+  secondaryContact?: string;
   policyType?:    string;
   policyVersion?: string;
   policyRegion?:  string;

package/src/service/auth.service.ts

@@ -263,8 +263,12 @@ export abstract class BaseAuthService {
    * numbers by comparing 200 vs 404.
    */
   sendOtp(req: OtpRequest): Observable<OtpResponse> {
-    const { contactType: _contactType, ...payload } = req;
-    return this.http.post<OtpResponse>(this.otpSendUrl, payload);
+    // Send the request as-is — `contactType` is now a real backend
+    // field (keel v0.5.11+) that selects the SMS-vs-email dispatch
+    // channel. The pre-v0.5.11 strip-before-post workaround was
+    // removed when email-OTP support landed in keel/handler/
+    // otp_handler.go.
+    return this.http.post<OtpResponse>(this.otpSendUrl, req);
   }
 
   resendOtp(otpToken: string, purpose?: OtpRequest['purpose']): Observable<{ status: string }> {