Remove unused workspace entries and simplify Rust code (#61379)

## What changed

- Remove the `codex-v8-poc`, `sentry`, and `syntect` workspace dependency entries and the `codex-v8-poc` entry from the `cargo-machete` ignore list.
- Remove unnecessary clones from MCP configuration and web search serialization tests.
- Simplify the final protected-metadata check in `WritableRoot::is_path_writable` to return the negated predicate directly.

GitOrigin-RevId: 80c1dd3106fc24c0767696936c719bb1fbb47227

codex-rs/protocol/src/models.rs

@@ -4141,9 +4141,9 @@ mod tests {
             let parsed: ResponseItem = serde_json::from_str(json_literal)?;
             let expected = ResponseItem::WebSearchCall {
                 encrypted_content: None,
-                id: expected_id.clone(),
-                status: expected_status.clone(),
-                action: expected_action.clone(),
+                id: expected_id,
+                status: expected_status,
+                action: expected_action,
                 internal_chat_message_metadata_passthrough: None,
             };
             assert_eq!(parsed, expected);

codex-rs/protocol/src/protocol.rs

@@ -1353,11 +1353,7 @@ impl WritableRoot {
             }
         }
 
-        if self.path_contains_protected_metadata_name(path) {
-            return false;
-        }
-
-        true
+        !self.path_contains_protected_metadata_name(path)
     }
 
     fn path_contains_protected_metadata_name(&self, path: &Path) -> bool {