Skip to content

Conversation

@joemonem
Copy link
Contributor

@joemonem joemonem commented Aug 14, 2025

Motivation

HackMD file

Implementation

  • Preserve hops in AMP Context
  • Extract original sender from previous hops
  • The proxy ADO itself just forwards messages, its authorization function is where the bulk of the logic resides

Testing

ibc test

Version Changes

  • kernel: 1.2.5-b.1 -> 1.2.5-b.2
  • proxy: 0.1.0

Checklist

  • Versions bumped correctly and documented
  • Changelog entry added or label applied

@github-actions github-actions bot added the packages/std Includes changes to the Standard crate label Aug 14, 2025
@coderabbitai
Copy link
Contributor

coderabbitai bot commented Aug 14, 2025

Important

Review skipped

Auto reviews are disabled on this repository.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing Touches
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch joe/proxy-ado

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

#[andr_exec]
#[cw_serde]
#[cfg_attr(not(target_arch = "wasm32"), derive(cw_orch::ExecuteFns))]
pub enum ExecuteMsg {
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can convert execution into Vec of Execution enum that has Instantiate, Execute and Migrate so user can batch multiple transactions.
Along with all these messages we would need a flag for each item to fail or continue on error.

@joemonem joemonem requested a review from SlayerAnsh September 3, 2025 08:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

packages/std Includes changes to the Standard crate

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants