fixes and GoLive

Author: KSemenenko

AGENTS.md

@@ -326,6 +326,7 @@ Repo-specific design rules:
 - Go Live local recording MUST capture the same composed program feed that the active live/record session publishes; recording a black frame or a different source than the current program feed is a regression.
 - Go Live local recording artifacts MUST contain both decodable video and decodable audio from the real program feed, and their saved resolution/quality must match the active source or chosen output profile instead of silently degrading to a lower-quality fallback.
 - Go Live recording status and runtime panels MUST show the real recording details the browser runtime knows, including the resolved output profile and live session/file telemetry when available; blank recording metadata during an active local recording is a regression.
+- Go Live default `Full` program layout MUST record and publish only the current active program camera; extra camera overlays may appear only when the operator explicitly chooses a multi-source layout such as split or picture-in-picture.
 - Go Live audio meters MUST show real browser audio activity for microphone, program, and recording paths; static placeholder bars or seeded fake levels in the Audio tab are regressions.
 - Go Live MUST have one active browser-side broadcast spine per setup: choose LiveKit or VDO.Ninja for the upstream transport, but do not run both as the same session's primary publish path at once.
 - Go Live remote fan-out targets such as YouTube, Twitch, and custom RTMP MUST hang off the chosen upstream broadcast spine or its relay/egress layer; the browser runtime must not pretend to publish the same session independently to every platform without a real transport path.
@@ -341,6 +342,7 @@ Repo-specific design rules:
 - User preferences persistence MUST sit behind a platform-agnostic user-settings abstraction, with browser storage implemented via local storage and room for other platform-specific implementations; theme, teleprompter layout preferences, camera/scene preferences, and similar saved settings belong there instead of ad-hoc feature stores.
 - Streaming destination/platform configuration MUST be user-defined and persisted in settings; Settings and Go Live must not ship hardcoded platform instances, seeded destination accounts, or fixed fake provider rows beyond real runtime capabilities.
 - Runtime screens must not keep inline seeded operational data, fake demo rows, or screen-local platform/source presets in page/component code; reusable labels and presets belong in shared contracts or catalogs, while rendered rows must come from persisted settings, workspace state, or live session state.
+- Go Live source rails must not render anonymous, unlabeled, or device-less camera cards; stale persisted scene sources without a real `deviceId` must be pruned, and rendered source cards should expose their real source/device identifiers for diagnostics.
 - Build quality gates must stay green under `-warnaserror`.
 - GitHub Pages is the expected CI publish target for the standalone WebAssembly app; publish automation must keep the app browser-only and Pages-compatible.
 - GitHub Actions MUST keep separate, clearly named workflows for pull-request validation and release automation; vague workflow names are forbidden.

src/PrompterOne.Shared/AppShell/Services/AppBootstrapper.cs

@@ -119,6 +119,16 @@ private static (MediaSceneState State, bool Changed) NormalizeMediaScene(MediaSc
     {
         var changed = false;
         var normalizedCameras = state.Cameras
+            .Where(camera =>
+            {
+                var isValid = IsValidCameraSource(camera);
+                if (!isValid)
+                {
+                    changed = true;
+                }
+
+                return isValid;
+            })
             .Select(camera =>
             {
                 var normalizedLabel = MediaDeviceLabelSanitizer.Sanitize(camera.Label);
@@ -177,4 +187,10 @@ state with
 
         return normalized;
     }
+
+    private static bool IsValidCameraSource(SceneCameraSource camera)
+    {
+        return !string.IsNullOrWhiteSpace(camera.SourceId)
+            && !string.IsNullOrWhiteSpace(camera.DeviceId);
+    }
 }

src/PrompterOne.Shared/Contracts/UiTestIds.cs

@@ -437,6 +437,8 @@ public static class GoLive
         public const string SourcesCard = "go-live-sources-card";
         public const string StartRecording = "go-live-start-recording";
         public const string StartStream = "go-live-start-stream";
+        public const string SourceDeviceIdAttribute = "data-device-id";
+        public const string SourceIdAttribute = "data-source-id";
         public const string SourcePickerEmpty = "go-live-source-picker-empty";
         public const string SwitchSelectedSource = "go-live-switch-selected-source";
         public const string StreamIncludeCamera = "go-live-stream-include-camera";

src/PrompterOne.Shared/GoLive/Components/GoLiveSourcesCard.razor

@@ -33,6 +33,7 @@
             @foreach (var source in Sources)
             {
                 <article class="@GetSourceCardCssClass(source)"
+                         @attributes="GetSourceDiagnostics(source)"
                          data-testid="@UiTestIds.GoLive.SourceCamera(source.SourceId)">
                     <button type="button"
                             class="gl-cam-main"
@@ -195,4 +196,13 @@
             _ => "gl-cam-card"
         };
     }
+
+    private static IReadOnlyDictionary<string, object> GetSourceDiagnostics(SceneCameraSource source)
+    {
+        return new Dictionary<string, object>(StringComparer.Ordinal)
+        {
+            [UiTestIds.GoLive.SourceIdAttribute] = source.SourceId,
+            [UiTestIds.GoLive.SourceDeviceIdAttribute] = source.DeviceId
+        };
+    }
 }

tests/PrompterOne.App.Tests/AppShell/AppBootstrapperMediaSceneTests.cs

@@ -55,6 +55,50 @@ public async Task AppBootstrapper_NormalizesPersistedMediaSceneLabels_FromBrowse
         Assert.Equal(SanitizedMicrophoneLabel, persistedState.AudioBus.Inputs.Single().Label);
     }
 
+    [Fact]
+    public async Task AppBootstrapper_PrunesPersistedCameraSources_WithoutDeviceIds()
+    {
+        var harness = TestHarnessFactory.Create(this);
+        var bootstrapper = Services.GetRequiredService<AppBootstrapper>();
+        var savedScene = new MediaSceneState(
+            Cameras:
+            [
+                new SceneCameraSource(
+                    SourceId: "invalid-source",
+                    DeviceId: string.Empty,
+                    Label: string.Empty,
+                    Transform: new MediaSourceTransform()),
+                new SceneCameraSource(
+                    SourceId: "cam-source",
+                    DeviceId: "cam-1",
+                    Label: SanitizedCameraLabel,
+                    Transform: new MediaSourceTransform())
+            ],
+            PrimaryMicrophoneId: "mic-1",
+            PrimaryMicrophoneLabel: SanitizedMicrophoneLabel,
+            AudioBus: new AudioBusState(
+            [
+                new AudioInputState(
+                    DeviceId: "mic-1",
+                    Label: SanitizedMicrophoneLabel)
+            ]));
+
+        harness.JsRuntime.SavedJsonValues[BuildSettingsStorageKey(BrowserAppSettingsKeys.SceneSettings)] =
+            JsonSerializer.Serialize(savedScene);
+
+        await bootstrapper.EnsureReadyAsync();
+
+        var restoredState = Services.GetRequiredService<IMediaSceneService>().State;
+        var restoredCamera = Assert.Single(restoredState.Cameras);
+        Assert.Equal("cam-source", restoredCamera.SourceId);
+        Assert.Equal("cam-1", restoredCamera.DeviceId);
+
+        var persistedState = harness.JsRuntime.GetSavedValue<MediaSceneState>(BrowserAppSettingsKeys.SceneSettings);
+        var persistedCamera = Assert.Single(persistedState.Cameras);
+        Assert.Equal("cam-source", persistedCamera.SourceId);
+        Assert.Equal("cam-1", persistedCamera.DeviceId);
+    }
+
     private static string BuildSettingsStorageKey(string key) =>
         string.Concat(BrowserStorageKeys.SettingsPrefix, key);
 }

tests/PrompterOne.App.Tests/GoLive/GoLivePageTests.cs

@@ -333,6 +333,47 @@ public void GoLivePage_SelectsSecondCameraForCanvas()
                 cut.FindByTestId(UiTestIds.GoLive.SelectedSourceLabel).TextContent.Trim()));
     }
 
+    [Fact]
+    public void GoLivePage_PrunesAnonymousPersistedSources_AndExposesCameraDiagnostics()
+    {
+        var sceneState = new MediaSceneState(
+            [
+                new SceneCameraSource(
+                    "invalid-source",
+                    string.Empty,
+                    string.Empty,
+                    new MediaSourceTransform(IncludeInOutput: true)),
+                new SceneCameraSource(
+                    AppTestData.Camera.FirstSourceId,
+                    AppTestData.Camera.FirstDeviceId,
+                    AppTestData.Camera.FrontCamera,
+                    new MediaSourceTransform(IncludeInOutput: true)),
+                new SceneCameraSource(
+                    AppTestData.Camera.SecondSourceId,
+                    AppTestData.Camera.SecondDeviceId,
+                    AppTestData.Camera.SideCamera,
+                    new MediaSourceTransform(IncludeInOutput: false))
+            ],
+            "mic-1",
+            AppTestData.Scripts.BroadcastMic,
+            new AudioBusState([new AudioInputState("mic-1", AppTestData.Scripts.BroadcastMic)]));
+
+        SeedSceneState(sceneState);
+
+        Services.GetRequiredService<NavigationManager>()
+            .NavigateTo(AppTestData.Routes.GoLiveDemo);
+
+        var cut = Render<GoLivePage>();
+
+        cut.WaitForAssertion(() =>
+        {
+            var sourceCards = cut.FindAll($"article[data-testid^='{UiTestIds.GoLive.SourceCamera(string.Empty)}']");
+            Assert.Equal(2, sourceCards.Count);
+            Assert.DoesNotContain(sourceCards, card => string.IsNullOrWhiteSpace(card.GetAttribute(UiTestIds.GoLive.SourceIdAttribute)));
+            Assert.DoesNotContain(sourceCards, card => string.IsNullOrWhiteSpace(card.GetAttribute(UiTestIds.GoLive.SourceDeviceIdAttribute)));
+        });
+    }
+
     [Fact]
     public void GoLivePage_RendersStudioParitySurfaceAndRemoteRoomFlow()
     {