Enhance Non-Rowan Yield Reward Functionality with Optional Duration Feature

[snobbee]

Currently, the Sifchain codebase includes a feature for non-Rowan yield rewards, where users can add non-Rowan liquidity to a reward bucket. The system then automatically distributes the entire liquidity in the bucket to the corresponding liquidity pool. This is a valuable functionality, but it could be further enhanced for better flexibility and utility.

Feature Request:
I propose adding an optional duration parameter to this existing functionality. When this duration is set, any new liquidity added to the reward bucket should not be distributed immediately in its entirety. Instead, it would be more beneficial if the distribution of this new liquidity is spread out over the specified duration. This means the liquidity would be distributed in smaller, incremental portions throughout the duration period, rather than all at once.

Benefits:

• Controlled Distribution: This enhancement would allow for a more controlled and gradual distribution of liquidity, which could be beneficial for managing market impact and providing a more steady support to the liquidity pool.
• Flexibility for Users: Users who contribute liquidity can have more options in terms of how their assets are utilized over time.
• Enhanced Strategic Options: This feature could open up new strategic possibilities for liquidity management within the Sifchain ecosystem.

I believe this enhancement will add significant value to the Sifchain platform, offering greater control and flexibility to users in the management of non-Rowan liquidity rewards.

[snobbee]

1. 📝 Indexing

I'm indexing the files within your repository.

2. 🔎 Searching

I'm searching for relevant snippets in your repository.

I found the following snippets in your repository.

View Snippets

https://github.com/Sifchain/sifnode/blob/main/x/dispensation/claims.md#L0-L15

https://github.com/Sifchain/sifnode/blob/main/x/dispensation/claims.md#L15-L30

https://github.com/Sifchain/sifnode/blob/main/x/dispensation/claims.md#L30-L45

https://github.com/Sifchain/sifnode/blob/main/x/oracle/types/prophecy.go#L0-L32

https://github.com/Sifchain/sifnode/blob/main/x/oracle/types/prophecy.go#L0-L72

3. 🌐 Planning

I'm creating a plan for the discussion and coding steps.

I have created the following workflow as a basis for how the discussion and coding will be derived.

💡 Steps

Add Duration Parameter to AddLiquidityToRewardBucket Message

Action: modify
File: x/clp/types/message_add_liquidity_to_rewards_bucket.go
Lines: [1, 50]
Dependencies: []
Instructions: Add a new field 'Duration' of type 'time.Duration' to the 'MsgAddLiquidityToRewardBucket' struct. This field will represent the duration over which the added liquidity will be distributed. Update the 'NewMsgAddLiquidityToRewardBucket' function to accept this new parameter and assign it to the created message. Update the 'ValidateBasic' function to check that the duration is not negative.

Details

Update AddLiquidityToRewardBucket Handler

Action: modify
File: x/clp/keeper/msg_server_add_liquidity_to_rewards_bucket.go
Lines: [1, 50]
Dependencies: ['Step 1']
Instructions: Update the 'AddLiquidityToRewardBucket' function to handle the new duration parameter. Instead of distributing the liquidity immediately, schedule it to be distributed gradually over the specified duration. This will likely involve creating a new function to handle the distribution, which will be called at regular intervals until the duration has passed.

Details

Update CLI Command

Action: modify
File: x/clp/client/cli/tx_add_liquidity_to_rewards_bucket.go
Lines: [1, 50]
Dependencies: ['Step 1']
Instructions: Update the 'CmdAddLiquidityToRewardBucket' function to accept the new duration parameter from the command line. Pass this parameter when creating the 'MsgAddLiquidityToRewardBucket' message.

Details

Update REST API

Action: modify
File: x/clp/client/rest/tx.go
Lines: [1, 50]
Dependencies: ['Step 1']
Instructions: Update the 'registerTxRoutes' function to accept the new duration parameter from the HTTP request. Pass this parameter when creating the 'MsgAddLiquidityToRewardBucket' message.

Details

Update Unit Tests

Action: modify
File: x/clp/keeper/msg_server_add_liquidity_to_rewards_bucket_test.go
Lines: [1, 50]
Dependencies: ['Step 1', 'Step 2']
Instructions: Update the unit tests for the 'AddLiquidityToRewardBucket' message and handler to account for the new duration parameter. Ensure that the tests cover cases where the duration is zero (for immediate distribution), positive (for gradual distribution), and negative (which should be rejected).

I have created the following workflow as a basis for how the discussion and coding will be derived.

💡 Steps

Update ValidateBasic Function

Action: modify
File: x/clp/types/message_add_liquidity_to_rewards_bucket.go
Lines: [1, 50]
Dependencies: []
Instructions: In the 'ValidateBasic' function of the 'MsgAddLiquidityToRewardBucket' message, add a check to return an error if the 'Duration' field is negative. Import the 'errors' package from 'x/clp/types/errors.go' to use predefined error types.

Details

Implement Time-based Job Scheduler

Action: create
File: x/clp/keeper/scheduler.go
Lines: [1, 100]
Dependencies: ['Step 1']
Instructions: Create a new file 'scheduler.go' in the 'x/clp/keeper' directory. Implement a time-based job scheduler that will handle the incremental distribution of liquidity over the specified duration. This scheduler should be able to queue jobs and execute them at the correct intervals based on the block time.

Details

Update CLI for Duration Parameter

Action: modify
File: x/clp/client/cli/tx.go
Lines: [1, 200]
Dependencies: ['Step 2']
Instructions: Modify the CLI command for adding liquidity to a reward bucket to accept an optional 'duration' flag. Ensure that the flag is parsed correctly and passed to the message constructor. Use the 'flag' package to define the new flag.

Details

Update REST API for Duration Parameter

Action: modify
File: x/clp/client/rest/tx.go
Lines: [1, 150]
Dependencies: ['Step 3']
Instructions: Update the REST API endpoint for adding liquidity to a reward bucket to parse the optional 'duration' parameter from the request body. Validate the input and ensure it is correctly included in the message that is created and signed.

Details

Create Unit Tests for Duration Parameter

Action: create
File: x/clp/keeper/add_liquidity_to_rewards_bucket_test.go
Lines: [1, 200]
Dependencies: ['Step 4']
Instructions: Create comprehensive unit tests in 'add_liquidity_to_rewards_bucket_test.go' to cover the new duration parameter functionality. Test cases should include zero, negative, non-numeric, and out-of-range duration values. Use the 'testify' package for assertions.

[snobbee]

The workflow is well-structured but could use more detail in certain areas. For negative duration, the 'ValidateBasic' function should return an error. For the gradual distribution functionality, consider using a time-based job scheduler or a similar mechanism to distribute the liquidity over time. For the CLI command and REST API updates, include error handling for invalid inputs. For the unit tests, ensure coverage for edge cases such as zero, negative, and extremely large duration values, as well as non-numeric and out-of-range inputs for the CLI command and REST API.