[autofix.ci] apply automated fixes

Author: autofix-ci[bot]

crates/forge_api/src/forge_api.rs

@@ -7,9 +7,8 @@ use forge_app::dto::ToolsOverview;
 use forge_app::{
     AgentProviderResolver, AgentRegistry, AppConfigService, AuthService, CommandInfra,
     CommandLoaderService, ConversationService, DataGenerationApp, EnvironmentInfra,
-    FileDiscoveryService, ForgeApp, GitApp, GrpcInfra, McpConfigManager, McpService,
-    PolicyService, ProviderAuthService, ProviderService, Services, User, UserUsage, Walker,
-    WorkspaceService,
+    FileDiscoveryService, ForgeApp, GitApp, GrpcInfra, McpConfigManager, McpService, PolicyService,
+    ProviderAuthService, ProviderService, Services, User, UserUsage, Walker, WorkspaceService,
 };
 use forge_config::ForgeConfig;
 use forge_domain::{Agent, ConsoleWriter, *};

crates/forge_domain/src/mcp.rs

@@ -13,17 +13,7 @@ use serde::{Deserialize, Serialize};
 /// Which `.mcp.json` declared a server: the user-level file (global to the
 /// machine) or the project-local one.
 #[derive(
-    Debug,
-    Clone,
-    Copy,
-    PartialEq,
-    Eq,
-    PartialOrd,
-    Ord,
-    Hash,
-    Serialize,
-    Deserialize,
-    JsonSchema,
+    Debug, Clone, Copy, PartialEq, Eq, PartialOrd, Ord, Hash, Serialize, Deserialize, JsonSchema,
 )]
 #[serde(rename_all = "lowercase")]
 pub enum Scope {

crates/forge_domain/src/policies/engine.rs

@@ -250,11 +250,7 @@ mod tests {
         // A `scope: user` rule must not affect a Local-scope operation.
         let fixture_workflow = PolicyConfig::new().add_policy(Policy::Simple {
             permission: Permission::Allow,
-            rule: Rule::Mcp(McpRule {
-                mcp: "*".to_string(),
-                scope: Some(Scope::User),
-                dir: None,
-            }),
+            rule: Rule::Mcp(McpRule { mcp: "*".to_string(), scope: Some(Scope::User), dir: None }),
         });
         let fixture = PolicyEngine::new(&fixture_workflow);
         let operation = PermissionOperation::Mcp {

crates/forge_services/src/forge_services.rs

@@ -30,12 +30,8 @@ use crate::tool_services::{
     ForgeFsUndo, ForgeFsWrite, ForgeImageRead, ForgePlanCreate, ForgeShell, ForgeSkillFetch,
 };
 
-type McpService<F> = ForgeMcpService<
-    ForgeMcpManager<F>,
-    F,
-    <F as McpServerInfra>::Client,
-    ForgePolicyService<F>,
->;
+type McpService<F> =
+    ForgeMcpService<ForgeMcpManager<F>, F, <F as McpServerInfra>::Client, ForgePolicyService<F>>;
 type AuthService<F> = ForgeAuthService<F>;
 
 /// ForgeApp is the main application container that implements the App trait.

crates/forge_services/src/mcp/service.rs

@@ -220,8 +220,7 @@ where
                         authorized.insert(name.clone());
                     }
                     Ok(_) => {
-                        denied
-                            .push((name.clone(), "Connection denied by user policy".to_string()));
+                        denied.push((name.clone(), "Connection denied by user policy".to_string()));
                     }
                     Err(err) => {
                         denied.push((name.clone(), format!("Policy check failed: {err:?}")));
@@ -477,18 +476,14 @@ mod tests {
             Ok(PolicyDecision { allowed: true, path: None })
         }
 
-        async fn allow_operation(
-            &self,
-            _operation: &PermissionOperation,
-        ) -> anyhow::Result<()> {
+        async fn allow_operation(&self, _operation: &PermissionOperation) -> anyhow::Result<()> {
             Ok(())
         }
     }
 
     // ── Fixture ──────────────────────────────────────────────────────────────
 
-    fn fixture()
-    -> ForgeMcpService<MockMcpManager, MockInfra, MockMcpClient, AlwaysAllowPolicy> {
+    fn fixture() -> ForgeMcpService<MockMcpManager, MockInfra, MockMcpClient, AlwaysAllowPolicy> {
         ForgeMcpService::new(
             Arc::new(MockMcpManager),
             Arc::new(MockInfra),

crates/forge_services/src/policy.rs

@@ -164,10 +164,7 @@ where
         + UserInfra,
 {
     /// Unconditionally persist an allow policy for the given operation.
-    async fn allow_operation(
-        &self,
-        operation: &PermissionOperation,
-    ) -> anyhow::Result<()> {
+    async fn allow_operation(&self, operation: &PermissionOperation) -> anyhow::Result<()> {
         self.add_policy_for_operation(operation).await?;
         Ok(())
     }