update breakage response type (#642)

appmgr/src/config/mod.rs

@@ -230,10 +230,7 @@ pub async fn set_dry(
         .configured()
         .put(&mut tx, &true)
         .await?;
-    Ok(BreakageRes {
-        patch: tx.abort().await?,
-        breakages,
-    })
+    Ok(BreakageRes(breakages))
 }
 
 pub async fn set_impl(

appmgr/src/control.rs

@@ -111,10 +111,7 @@ pub async fn stop_dry(
     let mut breakages = BTreeMap::new();
     stop_common(&mut tx, &id, &mut breakages).await?;
 
-    Ok(BreakageRes {
-        breakages,
-        patch: tx.abort().await?,
-    })
+    Ok(BreakageRes(breakages))
 }
 
 pub async fn stop_impl(ctx: RpcContext, id: PackageId) -> Result<WithRevision<()>, Error> {

appmgr/src/dependencies.rs

@@ -323,10 +323,7 @@ pub struct TaggedDependencyError {
 
 #[derive(Clone, Debug, Serialize, Deserialize)]
 #[serde(rename_all = "kebab-case")]
-pub struct BreakageRes {
-    pub patch: DiffPatch,
-    pub breakages: BTreeMap<PackageId, TaggedDependencyError>,
-}
+pub struct BreakageRes(pub BTreeMap<PackageId, TaggedDependencyError>);
 
 #[derive(Clone, Debug, Default, Deserialize, Serialize)]
 pub struct Dependencies(pub BTreeMap<PackageId, DepInfo>);

appmgr/src/install/mod.rs

@@ -142,10 +142,7 @@ pub async fn uninstall_dry(
     break_all_dependents_transitive(&mut tx, &id, DependencyError::NotInstalled, &mut breakages)
         .await?;
 
-    Ok(BreakageRes {
-        breakages,
-        patch: tx.abort().await?,
-    })
+    Ok(BreakageRes(breakages))
 }
 
 pub async fn uninstall_impl(ctx: RpcContext, id: PackageId) -> Result<WithRevision<()>, Error> {