feat(firefox): network control and emulation on Firefox

.github/workflows/ci.yml

@@ -252,7 +252,7 @@ jobs:
         run: cargo test --profile ci --manifest-path cli/Cargo.toml e2e -- --ignored --test-threads=1 --skip e2e_firefox
 
       - name: Run Firefox e2e tests
-        run: cargo test --profile ci --manifest-path cli/Cargo.toml e2e_firefox -- --ignored --test-threads=1 --skip e2e_firefox_route_mock --skip e2e_firefox_allowed_domains --skip e2e_firefox_download --skip e2e_firefox_emulation
+        run: cargo test --profile ci --manifest-path cli/Cargo.toml e2e_firefox -- --ignored --test-threads=1
 
   windows-integration:
     name: Windows Integration Test

cli/src/connection.rs

@@ -592,16 +592,26 @@ fn apply_daemon_env(cmd: &mut Command, session: &str, opts: &DaemonOptions) {
     }
     if let Some(p) = opts.proxy {
         cmd.env("AGENT_BROWSER_PROXY", p);
+        match opts.proxy_username {
+            Some(proxy_username) => {
+                cmd.env("AGENT_BROWSER_PROXY_USERNAME", proxy_username);
+            }
+            None => {
+                cmd.env_remove("AGENT_BROWSER_PROXY_USERNAME");
+            }
+        }
+        match opts.proxy_password {
+            Some(proxy_password) => {
+                cmd.env("AGENT_BROWSER_PROXY_PASSWORD", proxy_password);
+            }
+            None => {
+                cmd.env_remove("AGENT_BROWSER_PROXY_PASSWORD");
+            }
+        }
     }
     if let Some(pb) = opts.proxy_bypass {
         cmd.env("AGENT_BROWSER_PROXY_BYPASS", pb);
     }
-    if let Some(pu) = opts.proxy_username {
-        cmd.env("AGENT_BROWSER_PROXY_USERNAME", pu);
-    }
-    if let Some(pp) = opts.proxy_password {
-        cmd.env("AGENT_BROWSER_PROXY_PASSWORD", pp);
-    }
     if opts.ignore_https_errors {
         cmd.env("AGENT_BROWSER_IGNORE_HTTPS_ERRORS", "1");
     }
@@ -1783,6 +1793,27 @@ mod tests {
         );
     }
 
+    #[test]
+    fn daemon_proxy_credentials_are_scoped_to_the_effective_proxy() {
+        let mut opts = test_daemon_options(None, false, None);
+        opts.proxy = Some("http://new-proxy:8080");
+        let mut cmd = Command::new("agent-browser");
+        apply_daemon_env(&mut cmd, "session", &opts);
+        let envs: std::collections::HashMap<_, _> = cmd.get_envs().collect();
+        assert_eq!(
+            envs.get(std::ffi::OsStr::new("AGENT_BROWSER_PROXY")),
+            Some(&Some(std::ffi::OsStr::new("http://new-proxy:8080")))
+        );
+        assert_eq!(
+            envs.get(std::ffi::OsStr::new("AGENT_BROWSER_PROXY_USERNAME")),
+            Some(&None)
+        );
+        assert_eq!(
+            envs.get(std::ffi::OsStr::new("AGENT_BROWSER_PROXY_PASSWORD")),
+            Some(&None)
+        );
+    }
+
     #[test]
     fn test_spawn_race_loser_does_not_overwrite_winner_config() {
         let guard = EnvGuard::new(&["AGENT_BROWSER_SOCKET_DIR", "AGENT_BROWSER_NAMESPACE"]);

cli/src/mcp.rs

@@ -1287,14 +1287,14 @@ fn parity_tools() -> Vec<Value> {
         tool(
             TOOL_SET_VIEWPORT,
             "Set viewport",
-            "Set viewport size. The optional scale must be finite.",
+            "Set viewport size. The optional scale must be finite. Firefox does not support mobile mode.",
             json!({ "width": int_schema(), "height": int_schema(), "scale": number_schema() }),
             &["width", "height"],
         ),
         tool(
             TOOL_SET_DEVICE,
             "Set device",
-            "Emulate a device by name.",
+            "Emulate a device by name. Firefox does not support set device; use set viewport plus the --user-agent launch flag or AGENT_BROWSER_USER_AGENT instead of set device.",
             json!({ "device": { "type": "string" } }),
             &["device"],
         ),
@@ -1329,7 +1329,7 @@ fn parity_tools() -> Vec<Value> {
         tool(
             TOOL_SET_MEDIA,
             "Set media",
-            "Set media emulation.",
+            "Set media emulation. Firefox does not support reduced motion.",
             json!({ "colorScheme": { "type": "string", "enum": ["dark", "light", "no-preference"] }, "reducedMotion": { "type": "string", "enum": ["reduce", "no-preference"] } }),
             &[],
         ),
@@ -2168,7 +2168,7 @@ fn shared_property(key: &str) -> Value {
         "allowedDomains" => json!({
             "type": "array",
             "items": { "type": "string" },
-            "description": "Restrict browser and read traffic to these domain patterns."
+            "description": "Restrict browser and read traffic to these domain patterns, including non-HTTP document navigations. Chromium sessions also disable RTCPeerConnection while this is active."
         }),
         "capture" => json!({
             "type": "array",