Skip to content

feat: Add Defi spring (SNF Style) rewards decoder and sanitizer#2

Open
akiraonstarknet wants to merge 1 commit intoForgeYields:mainfrom
trovesfi:main
Open

feat: Add Defi spring (SNF Style) rewards decoder and sanitizer#2
akiraonstarknet wants to merge 1 commit intoForgeYields:mainfrom
trovesfi:main

Conversation

@akiraonstarknet
Copy link

No description provided.

Copilot AI review requested due to automatic review settings September 18, 2025 17:25
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds support for Defi Spring (SNF Style) rewards claiming functionality by implementing a new decoder and sanitizer component. The changes introduce a new component that can handle reward claiming operations following the standard contract structure used by SNF for rewards.

  • Adds a new defi_spring_snf_style module with interface and implementation
  • Integrates the new component into the existing SimpleDecoderAndSanitizer contract
  • Provides a claim function interface for reward claiming with amount and proof parameters

Reviewed Changes

Copilot reviewed 4 out of 5 changed files in this pull request and generated 1 comment.

File Description
packages/vault_allocator/src/lib.cairo Adds module declaration for defi_spring_snf_style
packages/vault_allocator/src/decoders_and_sanitizers/simple_decoder_and_sanitizer.cairo Integrates the new component with imports, storage, events, and implementation
packages/vault_allocator/src/decoders_and_sanitizers/defi_spring_snf_style/interface.cairo Defines the interface for Defi Spring SNF Style decoder with claim function
packages/vault_allocator/src/decoders_and_sanitizers/defi_spring_snf_style/defi_spring_snf_style_decoder_and_sanitizer.cairo Implements the component with a placeholder claim function

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

0xSacha pushed a commit that referenced this pull request Dec 7, 2025
Fix hyperlane decoder and add to simple decoder
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.

1 participant