From 7bf1ff968cd0ce0c3486d4507fd8d9ae31f6c5e4 Mon Sep 17 00:00:00 2001
From: Eli Mallon <eli@iame.li>
Date: Tue, 30 Jun 2026 13:37:12 -0700
Subject: [PATCH] live-vod: default livestream recording on for beta users

Most users with VOD access want their streams kept, so recording defaults on
(opposite of debugRecording): a missing settings record or unset flag records,
and only an explicit livestreamRecording=false opts out. Lookup/decode errors
still fail closed.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
---
 .../settings/privacy-category-settings.tsx           |  3 ++-
 js/app/src/screens/upload.tsx                        |  1 +
 pkg/director/s3_upload.go                            | 12 +++++++-----
 3 files changed, 10 insertions(+), 6 deletions(-)

diff --git a/js/app/components/settings/privacy-category-settings.tsx b/js/app/components/settings/privacy-category-settings.tsx
index c7943f04f..e548bd28a 100644
--- a/js/app/components/settings/privacy-category-settings.tsx
+++ b/js/app/components/settings/privacy-category-settings.tsx
@@ -24,7 +24,8 @@ export function PrivacyCategorySettings() {
     (state) => state.createServerSettingsRecord,
   );
   const debugRecordingOn = serverSettings?.debugRecording === true;
-  const livestreamRecordingOn = serverSettings?.livestreamRecording === true;
+  // Defaults on (unlike debugRecording): only an explicit `false` turns it off.
+  const livestreamRecordingOn = serverSettings?.livestreamRecording !== false;
   // The livestream-recording toggle is only meaningful for accounts in the VOD
   // beta — the node won't record anyone else regardless of this flag — so we
   // only surface it to them.
diff --git a/js/app/src/screens/upload.tsx b/js/app/src/screens/upload.tsx
index c7d56cdc8..23341cb4b 100644
--- a/js/app/src/screens/upload.tsx
+++ b/js/app/src/screens/upload.tsx
@@ -1776,6 +1776,7 @@ export function UploadDraftsScreen() {
                   <Button
                     size="sm"
                     variant="destructive"
+                    style={[{ width: "auto" }]}
                     onPress={() => handleDeleteDraft(draft.uri)}
                   >
                     Discard
diff --git a/pkg/director/s3_upload.go b/pkg/director/s3_upload.go
index fcb428583..6030fc972 100644
--- a/pkg/director/s3_upload.go
+++ b/pkg/director/s3_upload.go
@@ -25,9 +25,9 @@ const vodInviteFeature = "vod"
 //     upload gate (spxrpc.betaFeatureGranted): when no issuer is configured
 //     (self-hosted / dev) we fall back to cli.StreamIsAllowed, the same
 //     allowlist livestreaming itself uses.
-//   - The user has opted in via the livestreamRecording server setting. Not
-//     everyone in the beta wants their streams recorded, so it's off until
-//     explicitly enabled.
+//   - The user has not turned recording off via the livestreamRecording server
+//     setting. This defaults ON for beta users (most want their streams kept) —
+//     the opposite of debugRecording — so only an explicit `false` opts out.
 //
 // Evaluated once at stream start (like debugRecording in media.shouldRecord); a
 // mid-stream settings change takes effect on the streamer's next session. On
@@ -54,14 +54,16 @@ func (ss *StreamSession) shouldRecordLivestream(ctx context.Context, repoDID str
 		return false
 	}
 	if settings == nil {
-		return false
+		// No settings record at all: default on for beta users.
+		return true
 	}
 	spsettings, err := settings.ToStreamplaceServerSettings()
 	if err != nil {
 		log.Error(ctx, "live recording: failed to decode server settings", "error", err, "repoDID", repoDID)
 		return false
 	}
-	return spsettings.LivestreamRecording != nil && *spsettings.LivestreamRecording
+	// Default on: record unless the streamer explicitly set the flag to false.
+	return spsettings.LivestreamRecording == nil || *spsettings.LivestreamRecording
 }
 
 func (ss *StreamSession) maybeStartS3Upload(ctx context.Context, repoDID string) {
-- 
2.51.2