Commit Graph

109 Commits

Author SHA1 Message Date
derpy-duck 214736c7b8 Typescript / relayer engine / sdk related changes; now blocked on a SPY Port issue 2023-04-04 19:06:46 +00:00
derpy-duck 7e150947d5 Relay provider can set chains enabled and disabled 2023-04-04 16:26:32 +00:00
derpy-duck 8746431744 Remove outdated names 2023-04-04 14:21:53 +00:00
derpy-duck d639d81bd3 Move CoreRelayerStructs into its own interface file for organization 2023-04-04 14:15:30 +00:00
derpy-duck 2316dd03c5 Relay provider is specified in the DeliveryInstructionsContainer, not each DeliveryInstruction 2023-04-04 01:56:16 +00:00
derpy-duck 7d30716f8a
Forward wrapper (#129)
* Some code cleanup fixes

* WIP commit, feel free to revert

* Forward failures now cause full reverts

* Merge with interface changes

* forge fmt
2023-04-03 17:30:09 -05:00
chase-45 76553c3219 null delivery address check 2023-04-03 16:00:32 -04:00
chase-45 7b4d340a23 more redelivery code cleanup 2023-04-03 15:43:31 -04:00
chase-45 2667a2d977 added requesting sender to delivery container 2023-04-03 14:55:07 -04:00
chase-45 cf1fb8db66 removing unused fields and checks 2023-04-03 14:37:48 -04:00
derpy-duck 00e49bb8ae Redeliveries removed 2023-03-31 21:23:55 +00:00
derpy-duck 6a89f7a805 typo fix and sdk fix 2023-03-28 17:43:25 +00:00
derpy-duck 0798e1afb7 Update specs/comments 2023-03-28 17:43:25 +00:00
derpy-duck cdaef45b1b Remove messages from redelivery instruction 2023-03-28 17:43:25 +00:00
derpy-duck 6e0991e612 typescript and sdk hopefully build 2023-03-28 17:43:25 +00:00
derpy-duck b9bef84607 typescript and sdk hopefully build 2023-03-28 17:43:25 +00:00
derpy-duck 737130d28e New MessageInfo type with payload id when encoding 2023-03-28 17:43:25 +00:00
derpy-duck fa50d9d835 Permissionless version 1 implementation (#126)
* Interface changes

* Interface changes

* Contract changes

* mockrelayerintegration changes

* Contract changes (#124)

* Forge tests pass

* merge
2023-03-28 17:43:25 +00:00
derpy-duck cb83fbb3ef Interface changes 2023-03-28 17:43:25 +00:00
derpy-duck 9e282508f7
Improve forge test coverage + Seperate tests into individual tests (#119)
* fast tests

* forge fmt

* Remove parameter from forge test generic relayer

* WIP

* WIP

* Move Forge Mock Generic Relayer to seperate file

* Split revert redelivery tests into first few seperate tests

* redelivery test changes

* WIP

* tests pass

* Redelivery tests less D.R.Y and seperated into individual tests

* Delivery tests less D.R.Y and seperated into individual tests

* Resend checks in seperate tests

* Forward test coverage!

* 100% test coverage in CoreRelayer and CoreRelayerDelivery!

* remove test coverage files

* Governance tests

* governance stack

* forge fmt

* removed console.sol
2023-03-23 14:14:48 -05:00
Joe Howarth eff19d28aa
Continuously send messages to create load (#118)
* save

* Continuously send messages to create load

* add metrics
2023-03-14 10:45:03 -07:00
derpy-duck 92dbc20c8b
Forge tests faster (#117)
* fast tests

* forge fmt

* add --via-ir to build

* remove forge test-fast; just run forge test for the same effect

* remove outdated and unused folder
2023-03-10 10:41:12 -05:00
derpy-duck db355a2f1d
Revert if delivery address is 0, and add forge tests testing this (#115)
* Revert if delivery address is 0, and add forge tests testing this

* Test passes
2023-03-02 13:37:12 -05:00
derpy-duck 27cbbc7804 forge fmt' 2023-03-02 10:02:46 -05:00
derpy-duck 81cb32981a Nice CoreRelayerDelivery comments 2023-03-02 10:02:46 -05:00
derpy-duck dab942c980 WIP 2023-03-02 10:02:46 -05:00
derpy-duck c2117b6ef1 deliverSingle is nice 2023-03-02 10:02:46 -05:00
derpy-duck 6fee6ba1e4
Nice comments in the code (#111)
* resend doesn't take nonce as parameter

* Replace 'delivery failure' with 'receiver failure' in specs

* helper send method

* Fix interface to match not having nonce for resend

* Fix forge tests to not have nonce as a parameter

* fix redeliveryTest

* Split CoreRelayer into CoreRelayer and CoreRelayerDelivery

* fix build error

* forge fmt

* fix interface of getdefaultrelayprovider

* Nice comments for main functions in CoreRelayer.sol

* Nice comments for CoreRelayerMessages

* forge fmt

* merge

* Removes wormholeMessageFee from storage to avoid excessive gas costs.

---------

Co-authored-by: Sebastián Claudio Nale <sebinale@gmail.com>
2023-02-28 10:09:01 -05:00
Sebastián Nale a292dc253d Adds `AssetConversion` struct to avoid multiple reads or stores. 2023-02-27 21:20:51 -03:00
derpy-duck 7b384caadc
Remove TODO comment, correct 10^-6 (#108) 2023-02-27 18:49:58 -05:00
derpy-duck 7416880825
Miscellaneous fixes (#104)
* resend doesn't take nonce as parameter

* Replace 'delivery failure' with 'receiver failure' in specs

* helper send method

* Fix interface to match not having nonce for resend

* Fix forge tests to not have nonce as a parameter

* fix redeliveryTest

* fix build error

* forge fmt

* fix interface of getdefaultrelayprovider
2023-02-27 18:49:38 -05:00
scnale 24d682bca4
Mark assembly as `memory safe` (#107)
* Adds `memory-safe` tag to assembly sections in `BytesLib`.

This improves codegen in the compiler and reduces the chance of
untraceable internal codegen errors when dealing with stack variables
that can't be referenced with an instruction.

Also, it removes unused functions in `BytesLib` to reduce code surface.

* Renames functions and contracts in reentrancy negative test.

* Adds `unchecked` block for overflow check.
2023-02-27 15:59:00 -07:00
derpy-duck f7de3d649d
Code cleanup (#109)
* DRY: Remove duplicate structs

* Move encode/decode/convert-request-to-instruction related files to CoreRelayerMessages

* remove rolloverchain; now it is just the chain of the first request

* forge test fix to accomdoate for removing two getters

* DRY: Remove repeated error messages

* Remove miscellaneous comments

* DRY send and forward

* replace relayprovider interface with address to match the IWormholeRelayer interface

* forge fmt

* Remove byteslib from CoreRelayer

* Remove the encoding and decoding of the delivery request -> just store the struct itself!

* forge fmt

* Rewriting of checks in send/resend/forward

* test passes

* DRY - Emit only one event

* DRY

* forge fmt

* Fix typescript error

* using IWormholeRelayer

* Consistent naming

* call wormhole.messageFee() once

* Remove unnecessary line

* Compute the length once, not every iteration

* forge tests pass
2023-02-27 14:32:57 -05:00
Sebastián Claudio Nale 004e1b3059 `forge fmt` run. 2023-02-21 12:56:07 -03:00
Sebastián Claudio Nale 01bd5540a1 Reuse `wormhole()` and `wormhole.messageFee()` expressions. 2023-02-21 12:56:07 -03:00
Sebastián Claudio Nale 315f0909c7 Comment a couple of optimization opportunities. 2023-02-21 12:56:07 -03:00
Sebastián Claudio Nale a2bdeaffea Optimize `convertToEncodedRedeliveryByTxHashInstruction`.
This reduces code size and gas used.
2023-02-21 12:56:07 -03:00
Sebastián Claudio Nale a2bca1782c Optimize `appendDeliveryInstruction`.
This simplification allows the compiler to emit less code.
2023-02-21 12:56:07 -03:00
Sebastián Claudio Nale 958d964ad5 Avoid call if the value sent is zero. 2023-02-21 12:56:07 -03:00
Sebastián Claudio Nale a5427bb177 Make refund and reward addresses `payable`. 2023-02-21 12:56:07 -03:00
Sebastián Claudio Nale 9acc9fcbd6 Cleans up `validateRedeliverySingle`. 2023-02-21 12:56:07 -03:00
Sebastián Claudio Nale 1003717cdb Adds `receive()` function in `MockRelayerIntegration`. 2023-02-21 12:43:33 -03:00
Sebastián Claudio Nale 3f713d7efc Adds `MockWormhole` contract for unit tests.
Also adds a `WormholeSimulator` interface for signing and non-signing
mocks. The signing mock can be useful when writing a test for a
specific testnet fork with a real wormhole deployment. In the unit tests,
only the non signing mock is used.
2023-02-21 12:43:33 -03:00
derpy-duck 45021b4691 forge fmt 2023-02-21 10:43:13 -05:00
derpy-duck a7bda1a057 Nice specs for IWormholeRelayer 2023-02-21 10:43:13 -05:00
Sebastián Claudio Nale 1cae1ba324 Fixes relayer reward to take into account forward publish fee. 2023-02-17 19:57:28 -03:00
Sebastián Claudio Nale 0b10f9e2a5 Puts enum type in the `Delivery` event. 2023-02-17 19:54:26 -03:00
Sebastián Claudio Nale f7174f2084 Adds named parameters to `Delivery` events. 2023-02-17 19:54:26 -03:00
Sebastián Claudio Nale eced885877 `forge fmt` run. 2023-02-17 19:54:26 -03:00
Sebastián Claudio Nale a41490cc38 Uses `abi.encodeCall` to typecheck some encoded calls. 2023-02-17 19:54:26 -03:00