Skip to content

Conversation

@Vinsho
Copy link
Contributor

@Vinsho Vinsho commented Feb 3, 2026

No description provided.

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

Adds chain attribution for DeBank “apps” parsing and switches deposit token parsing to a typed token model so downstream consumers can reliably access token fields.

Changes:

  • Introduced DEBANK_APP_CHAIN_MAP and used it to populate a chain: Blockchain field on parsed DebankPrediction and DebankAppDeposit.
  • Replaced raw dict token handling for app deposits with a new typed DebankDepositToken model and updated parsing to use asset_token_list.
  • Updated app parser tests to assert chain and typed token access.

Reviewed changes

Copilot reviewed 5 out of 5 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
blockapi/v2/models.py Adds DebankDepositToken, types app deposit tokens, and adds chain to app-level parsed models.
blockapi/v2/api/debank_maps.py Adds an app-id → chain mapping used during app parsing.
blockapi/v2/api/debank.py Threads chain through app parsing into deposits and predictions; switches deposit token extraction to asset_token_list.
blockapi/test/v2/api/debank/test_debank_fetch.py Adds an integration test stub for fetching apps (but currently not asserting).
blockapi/test/v2/api/debank/test_debank_app_parser.py Extends parser tests to assert chain and typed token access.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@Vinsho Vinsho force-pushed the fix/debank-support branch from e0ae7ed to cd5799f Compare February 4, 2026 08:24
@Vinsho Vinsho merged commit 83ff70f into master Feb 5, 2026
5 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.

2 participants