Require ack for IbcReceiveResponse construction

contracts/ibc-reflect-send/src/ibc.rs

@@ -101,9 +101,7 @@ pub fn ibc_packet_receive(
     _env: Env,
     _packet: IbcPacketReceiveMsg,
 ) -> Result<IbcReceiveResponse, Never> {
-    Ok(IbcReceiveResponse::new()
-        .set_ack(b"{}")
-        .add_attribute("action", "ibc_packet_ack"))
+    Ok(IbcReceiveResponse::new(b"{}").add_attribute("action", "ibc_packet_ack"))
 }
 
 #[entry_point]

contracts/ibc-reflect/src/contract.rs

@@ -255,8 +255,7 @@ pub fn ibc_packet_receive(
         // we try to capture all app-level errors and convert them into
         // acknowledgement packets that contain an error code.
         let acknowledgement = encode_ibc_error(format!("invalid packet: {e}"));
-        Ok(IbcReceiveResponse::new()
-            .set_ack(acknowledgement)
+        Ok(IbcReceiveResponse::new(acknowledgement)
             .add_event(Event::new("ibc").add_attribute("packet", "receive")))
     })
 }
@@ -269,9 +268,7 @@ fn receive_who_am_i(deps: DepsMut, caller: String) -> StdResult<IbcReceiveRespon
     };
     let acknowledgement = to_json_binary(&AcknowledgementMsg::Ok(response))?;
     // and we are golden
-    Ok(IbcReceiveResponse::new()
-        .set_ack(acknowledgement)
-        .add_attribute("action", "receive_who_am_i"))
+    Ok(IbcReceiveResponse::new(acknowledgement).add_attribute("action", "receive_who_am_i"))
 }
 
 // processes PacketMsg::Balances variant
@@ -284,9 +281,7 @@ fn receive_balances(deps: DepsMut, caller: String) -> StdResult<IbcReceiveRespon
     };
     let acknowledgement = to_json_binary(&AcknowledgementMsg::Ok(response))?;
     // and we are golden
-    Ok(IbcReceiveResponse::new()
-        .set_ack(acknowledgement)
-        .add_attribute("action", "receive_balances"))
+    Ok(IbcReceiveResponse::new(acknowledgement).add_attribute("action", "receive_balances"))
 }
 
 // processes PacketMsg::Dispatch variant
@@ -307,8 +302,7 @@ fn receive_dispatch(
     // we wrap it in a submessage to properly report errors
     let msg = SubMsg::reply_on_error(wasm_msg, RECEIVE_DISPATCH_ID);
 
-    Ok(IbcReceiveResponse::new()
-        .set_ack(acknowledgement)
+    Ok(IbcReceiveResponse::new(acknowledgement)
         .add_submessage(msg)
         .add_attribute("action", "receive_dispatch"))
 }
@@ -324,8 +318,7 @@ fn execute_error(text: String) -> StdResult<IbcReceiveResponse> {
 fn execute_return_msgs(msgs: Vec<CosmosMsg>) -> StdResult<IbcReceiveResponse> {
     let acknowledgement = to_json_binary(&AcknowledgementMsg::<ReturnMsgsResponse>::Ok(()))?;
 
-    Ok(IbcReceiveResponse::new()
-        .set_ack(acknowledgement)
+    Ok(IbcReceiveResponse::new(acknowledgement)
         .add_messages(msgs)
         .add_attribute("action", "receive_dispatch"))
 }

packages/std/src/ibc.rs

@@ -636,8 +636,14 @@ impl<T> Default for IbcReceiveResponse<T> {
 }
 
 impl<T> IbcReceiveResponse<T> {
-    pub fn new() -> Self {
-        Self::default()
+    /// Create a new response with the given acknowledgement.
+    pub fn new(ack: impl Into<Binary>) -> Self {
+        Self {
+            acknowledgement: ack.into(),
+            messages: vec![],
+            attributes: vec![],
+            events: vec![],
+        }
     }
 
     /// Set the acknowledgement for this response.
@@ -649,7 +655,7 @@ impl<T> IbcReceiveResponse<T> {
     ///
     /// fn make_response_with_ack() -> IbcReceiveResponse {
     ///     let ack = StdAck::success(b"\x01"); // 0x01 is a FungibleTokenPacketSuccess from ICS-20.
-    ///     IbcReceiveResponse::new().set_ack(ack)
+    ///     IbcReceiveResponse::new(ack)
     /// }
     /// ```
     pub fn set_ack(mut self, ack: impl Into<Binary>) -> Self {
@@ -695,14 +701,14 @@ impl<T> IbcReceiveResponse<T> {
     /// ## Examples
     ///
     /// ```
-    /// use cosmwasm_std::{attr, IbcReceiveResponse};
+    /// use cosmwasm_std::{attr, IbcReceiveResponse, StdAck};
     ///
     /// let attrs = vec![
     ///     ("action", "reaction"),
     ///     ("answer", "42"),
     ///     ("another", "attribute"),
     /// ];
-    /// let res: IbcReceiveResponse = IbcReceiveResponse::new().add_attributes(attrs.clone());
+    /// let res: IbcReceiveResponse = IbcReceiveResponse::new(StdAck::success(b"\x01")).add_attributes(attrs.clone());
     /// assert_eq!(res.attributes, attrs);
     /// ```
     pub fn add_attributes<A: Into<Attribute>>(
@@ -718,10 +724,10 @@ impl<T> IbcReceiveResponse<T> {
     /// ## Examples
     ///
     /// ```
-    /// use cosmwasm_std::{CosmosMsg, IbcReceiveResponse};
+    /// use cosmwasm_std::{CosmosMsg, IbcReceiveResponse, StdAck};
     ///
     /// fn make_response_with_msgs(msgs: Vec<CosmosMsg>) -> IbcReceiveResponse {
-    ///     IbcReceiveResponse::new().add_messages(msgs)
+    ///     IbcReceiveResponse::new(StdAck::success(b"\x01")).add_messages(msgs)
     /// }
     /// ```
     pub fn add_messages<M: Into<CosmosMsg<T>>>(self, msgs: impl IntoIterator<Item = M>) -> Self {
@@ -733,10 +739,10 @@ impl<T> IbcReceiveResponse<T> {
     /// ## Examples
     ///
     /// ```
-    /// use cosmwasm_std::{SubMsg, IbcReceiveResponse};
+    /// use cosmwasm_std::{SubMsg, StdAck, IbcReceiveResponse};
     ///
     /// fn make_response_with_submsgs(msgs: Vec<SubMsg>) -> IbcReceiveResponse {
-    ///     IbcReceiveResponse::new().add_submessages(msgs)
+    ///     IbcReceiveResponse::new(StdAck::success(b"\x01")).add_submessages(msgs)
     /// }
     /// ```
     pub fn add_submessages(mut self, msgs: impl IntoIterator<Item = SubMsg<T>>) -> Self {

packages/std/src/stdack.rs

@@ -87,8 +87,8 @@ impl StdAck {
     ///
     /// let ack = StdAck::success(b"\x01"); // 0x01 is a FungibleTokenPacketSuccess from ICS-20.
     ///
-    /// let res: IbcReceiveResponse = IbcReceiveResponse::new().set_ack(ack.to_binary());
-    /// let res: IbcReceiveResponse = IbcReceiveResponse::new().set_ack(ack); // Does the same but consumes the instance
+    /// let res: IbcReceiveResponse = IbcReceiveResponse::new(ack.to_binary());
+    /// let res: IbcReceiveResponse = IbcReceiveResponse::new(ack); // Does the same but consumes the instance
     /// ```
     pub fn to_binary(&self) -> Binary {
         // We need a non-failing StdAck -> Binary conversion to allow using StdAck in