Skip to content

Issue 1.3 strategy parser#19

Merged
soomtochukwu merged 9 commits intoDXmakers:mainfrom
sudo-robi:issue-1.3-strategy-parser
Mar 30, 2026
Merged

Issue 1.3 strategy parser#19
soomtochukwu merged 9 commits intoDXmakers:mainfrom
sudo-robi:issue-1.3-strategy-parser

Conversation

@sudo-robi
Copy link
Copy Markdown
Contributor

closes #3

Description:
Implements a parser tool for the agent to convert conversational intent into a strict JSON payload for use by the frontend and smart contracts.

Features:

Converts input into a JSON object with the schema: monthlyContributionAmount, blendAllocationX, soroswapAllocationX, goldAllocationX.
Ensures allocation percentages always sum to exactly 100.
Provides fallbacks for malformed or missing input.
Includes unit tests for schema validation and normalization.
Acceptance Criteria:

Agent outputs valid JSON matching the schema.
Allocations always sum to 100.
Unit tests verify schema and fallback behavior.

@drips-wave
Copy link
Copy Markdown

drips-wave bot commented Mar 27, 2026

@sudo-robi Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

- Replace env.register_contract() with env.register() (SDK deprecation)
- Fix dev_dependencies → dev-dependencies in Cargo.toml
- Add test_initialize_gold_trustline
- Add test_deposit_and_withdraw
- Add test_lp_shares_accumulate_across_deposits
- Add test_zero_lp_percentage_no_shares
- Add test_50_50_split_precision_odd_amount
- Add test_multiple_users_isolated_lp_shares
- Add test_allocation_exceeds_100_percent (should_panic)
- Extract shared setup_env() test helper

All 8 tests pass with zero warnings.
@sudo-robi
Copy link
Copy Markdown
Contributor Author

@soomtochukwu all done

@soomtochukwu soomtochukwu merged commit 82b4874 into DXmakers:main Mar 30, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Issue 1.3: Strategy Parser

2 participants