Skip to content

SHARD-2738: default evm data collection to true#93

Open
urnotsam wants to merge 1 commit intodevfrom
SHARD-2738
Open

SHARD-2738: default evm data collection to true#93
urnotsam wants to merge 1 commit intodevfrom
SHARD-2738

Conversation

@urnotsam
Copy link

@urnotsam urnotsam commented Aug 27, 2025

PR Type

enhancement


Description

  • Set EVM data collection options to true by default

  • Enable indexing of original transaction data by default

  • Enable contract info and token transfer decoding by default


Changes walkthrough 📝

Relevant files
Enhancement
index.ts
Default EVM data collection and decoding options enabled 

src/config/index.ts

  • Changed default config to enable indexOriginalTxData
  • Enabled decodeContractInfo and decodeTokenTransfer by default
  • Updated processData defaults for broader EVM data collection
  • +3/-3     

    Need help?
  • Type /help how to ... in the comments thread for any questions about PR-Agent usage.
  • Check out the documentation for more information.
  • @github-actions
    Copy link

    PR Reviewer Guide 🔍

    Here are some key observations to aid the review process:

    ⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
    🏅 Score: 95
    🧪 No relevant tests
    🔒 No security concerns identified
    ⚡ Recommended focus areas for review

    Default Behavior Change

    The default values for EVM data collection and decoding options have been changed to true. This alters the application's default behavior and may have performance or resource usage implications. Reviewers should ensure this change is intentional and that documentation and downstream consumers are updated accordingly.

    indexOriginalTxData: true,
    decodeContractInfo: true,
    decodeTokenTransfer: true,

    @github-actions
    Copy link

    PR Code Suggestions ✨

    No code suggestions found for the PR.

    @justin-shardeum justin-shardeum changed the title SAHRD-2738: default evm data collection to true SHARD-2738: default evm data collection to true Aug 27, 2025
    Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

    Projects

    None yet

    Development

    Successfully merging this pull request may close these issues.

    1 participant