Follow the Cloudflare worker's new settings contract

.codex/skills/run-headless-simulator/SKILL.md

@@ -227,6 +227,4 @@ Noise suppression:
 - `none`
 - `hyper-voice`
 
-Cloudflare participants run with `none` only.
-
 Run `just dev --logging info headless --help` or `hyper-client-simulator headless --help` if a flag fails.

browser/tests/cloudflare_driver.rs

@@ -269,7 +269,7 @@ async fn cloudflare_runtime_delegates_hyper_core_authentication_to_the_worker()
                     "videoActivated": true,
                     "screenshareActivated": false,
                     "autoGainControl": true,
-                    "noiseSuppression": "none",
+                    "noiseSuppression": "hyper-voice",
                     "transportMode": "webrtc",
                     "videoConstraintPublishWebcam": "1080p",
                     "videoConstraintSubscribe": "none",
@@ -317,7 +317,7 @@ async fn cloudflare_runtime_delegates_hyper_core_authentication_to_the_worker()
     assert!(started.joined);
     assert_eq!(
         started.noise_suppression,
-        client_simulator_config::NoiseSuppression::Disabled
+        client_simulator_config::NoiseSuppression::HyperVoice
     );
 
     participant.close().await;
@@ -331,6 +331,17 @@ async fn cloudflare_runtime_delegates_hyper_core_authentication_to_the_worker()
     assert_eq!(requests[0].path, "/sessions");
     assert!(request_json(&requests[0])["displayName"].is_string());
     assert!(request_json(&requests[0]).get("hyperSessionCookie").is_none());
+    // A participant that leaves its settings to hyper.video leaves them out of the request.
+    let settings = &request_json(&requests[0])["settings"];
+    for key in [
+        "autoGainControl",
+        "blur",
+        "videoConstraintPublishWebcam",
+        "videoConstraintSubscribe",
+        "videoMaxConcurrentTracks",
+    ] {
+        assert!(settings.get(key).is_none(), "{key}");
+    }
     assert_eq!(requests[1].path, "/sessions/cf-runtime-core/commands");
     assert_eq!(request_json(&requests[1]), json!({ "type": "leave" }));
     assert_eq!(requests[2].path, "/sessions/cf-runtime-core/close");

cloudflare-worker-client/src/client.rs

@@ -247,8 +247,6 @@ mod tests {
         ParticipantSettings,
         ParticipantSettingsNoiseSuppression,
         ParticipantSettingsTransport,
-        ParticipantSettingsVideoConstraintPublishWebcam,
-        ParticipantSettingsVideoConstraintSubscribe,
         SessionCreateRequest,
         SessionCreateRequestDisplayName,
         SessionCreateRequestFrontendKind,
@@ -364,13 +362,13 @@ mod tests {
             session_timeout_ms: Some(600_000.0),
             settings: ParticipantSettings {
                 audio_enabled: true,
-                auto_gain_control: true,
-                blur: false,
+                auto_gain_control: None,
+                blur: None,
                 noise_suppression: ParticipantSettingsNoiseSuppression::None,
                 screenshare_enabled: false,
                 transport: ParticipantSettingsTransport::Webrtc,
-                video_constraint_publish_webcam: ParticipantSettingsVideoConstraintPublishWebcam::None,
-                video_constraint_subscribe: ParticipantSettingsVideoConstraintSubscribe::None,
+                video_constraint_publish_webcam: None,
+                video_constraint_subscribe: None,
                 video_max_concurrent_tracks: None,
                 video_enabled: true,
             },

docs/browser-driver.md

@@ -93,7 +93,6 @@ For parity with the current local backend, `start()` should apply those launch s
 
 - Hyper Core applies noise suppression, blur, camera resolution, and transport before join, then applies audio/video in the lobby, and applies initial screenshare after joining. See [`browser/src/participant/local/core.rs`](../browser/src/participant/local/core.rs) `:143-171`.
 - Hyper Lite currently only applies audio/video/screenshare and ignores advanced media settings. See [`browser/src/participant/local/lite.rs`](../browser/src/participant/local/lite.rs) `:90-106`, `:153-175`.
-- The Cloudflare backend applies noise suppression `none` only, because the worker accepts no other value of hyper.video's setting. It sends `none` in place of `hyper-voice`, with a warning, and refuses `SetNoiseSuppression` with it. See [`browser/src/participant/cloudflare/mod.rs`](../browser/src/participant/cloudflare/mod.rs).
 
 There is currently no runtime command for:
 

src/headless.rs

@@ -55,7 +55,7 @@ pub struct HeadlessArgs {
     #[clap(long = "auto-gain-control", value_name = "ON")]
     pub auto_gain_control: Option<OwnOrDefault<bool>>,
 
-    /// Noise suppression model: none or hyper-voice. Cloudflare participants always run with none.
+    /// Noise suppression model: none or hyper-voice.
     #[clap(long = "noise-suppression", value_name = "MODEL")]
     pub noise_suppression: Option<NoiseSuppression>,