fix: incorrect ID usage in IBC connection and packet handlers (#474)

* Initial plan

* Fix ConnOpenAck typo: use conn_id_on_a instead of conn_id_on_b

Co-authored-by: rnbguy <3789449+rnbguy@users.noreply.github.com>

* fix: use correct port/channel IDs in ack_path verification

Co-authored-by: rnbguy <3789449+rnbguy@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: rnbguy <3789449+rnbguy@users.noreply.github.com>

Author: Copilot

cairo-contracts/packages/core/src/channel/components/handler.cairo

@@ -1053,8 +1053,8 @@ pub mod ChannelHandlerComponent {
 
             let paths = ack_path(
                 connection_end.counterparty.prefix,
-                msg.packet.port_id_on_a,
-                msg.packet.chan_id_on_a,
+                msg.packet.port_id_on_b,
+                msg.packet.chan_id_on_b,
                 msg.packet.seq_on_a,
             );
 

cairo-contracts/packages/core/src/connection/components/handler.cairo

@@ -251,7 +251,7 @@ pub mod ConnectionHandlerComponent {
             let expected_conn_end_on_b = ConnectionEndTrait::try_open(
                 conn_end_on_a.counterparty.client_id.clone(),
                 conn_end_on_a.client_id.clone(),
-                msg.conn_id_on_b.clone(),
+                msg.conn_id_on_a.clone(),
                 conn_end_on_a.counterparty.prefix.clone(),
                 conn_end_on_a.delay_period,
             );