Conversation
…ctoring (#312) * feat: add polish foundation support in sdk * fix: export typed rpc error models * test(auth): cover delete wallet general error mapping * test(auth): cover generalized wallet rpc errors * fix: harden KDF error parsing against malformed responses - Wrap KdfErrorRegistry.tryParse parser invocation in try/catch to gracefully fall back to GeneralErrorResponse on malformed error_data - Make _intFromJson and _doubleFromJson null-safe (return 0 instead of throwing ArgumentError) - Make Mm2Duration.fromJson handle null input (return Duration.zero) - Apply same fixes to the generator script so regenerated code stays consistent - Refine doc comments on delete_wallet and change_mnemonic_password * fix: harden stream controller lifecycle and refactor fee guard - Add isCancelled flag with canEmit/tearDownResources helpers to orderbook and swap-status stream controllers to prevent race conditions where events could be emitted after cancellation - Extract _throwIfFeeEstimationDisabled helper in FeeManager to remove duplicated guard logic across ETH, UTXO and Tendermint paths - Update bundled coins repo commit hash
* feat(sdk): add high-level balance and tx manager interfaces * fix(sdk): address pr review cancellation and ordering bugs
* feat(sdk): add TRON and TRC20 support * chore(sdk): remove generated whitespace * fix(sdk): align config parsing and KDF compatibility
* refactor(withdrawals): enforce preview-then-execute workflow to prevent duplicate signing - Add executeWithdrawal() method to broadcast pre-signed transactions - Deprecate withdraw() method in favor of two-step preview-execute flow - Update example app to use new executeWithdrawal() method - Prevents transactions from being signed multiple times - Ensures users always review transaction details before broadcast Breaking Changes: - withdraw() is now deprecated (still functional but not recommended) - Recommended flow: previewWithdrawal() → executeWithdrawal() * Update packages/komodo_defi_sdk/lib/src/withdrawals/withdrawal_manager.dart Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * use latest KDF commit in `sia-rc-cleanup` * migrate SIA updates from old branch * fix wallet build errors * fix sia activation param * chore: update generated code chore: update generated code by running `melos run prepare` (requires Dart `melos` tool to be installed) * fix: fix broken sia RPC method test fix broken sia RPC method test. NB! Only fixing static analysis errors - the functionality of the test itself remains unchanged * clean task loop exit * implement SIA withdrawals * exclude SIA coins from pk export * fix(sia): explicitly remove support for tx streaming for sia * test(sia): add test case to confirm that tx history streaming is not supported for sia * chore(deps): bump kdf to 85d354d * update kdf commit * updates kdf commit --------- Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> Co-authored-by: smk762 <smk@komodoplatform.com> Co-authored-by: Francois <takenagain@users.noreply.github.com>
The squash merge of the SIA branch into dev left a duplicate executeWithdrawal method in LegacyWithdrawalManager. Remove the second copy which also inconsistently used WithdrawalException directly instead of the _mapError helper.
Add a dedicated metadata mutex to serialise read-modify-write cycles on user metadata, preventing concurrent coin activations from overwriting each other's state. Introduces updateActiveUserMetadataKey for atomic single-key transforms and migrates setOrRemoveActiveUserKeyValue to use it. Also updates bundled coins config to master branch (aceacfb).
…builds (#329) Replace T.toString() parsing for List/Map handling with compile-time type checks so dart2js and dart2wasm (including minification) behave correctly. Add unit tests for value/valueOrNull list and map paths.
Update bundled_coins_repo_commit to 32a0cdc2e9ab5927a571cb1c85fc4be080567e70.
The streaming manager previously fell through to enable_* calls after a short 2-second timeout even when no first byte had been received from the SSE / SharedWorker connection. KDF responds with UnknownClient when the client is not yet registered, causing all three retry attempts to fail and leaving orderbook (and other) streams permanently broken for the session. Changes: - Increase the first-byte wait to 30s per attempt (3 attempts with reconnect between each) so we never call enable_* before KDF has actually registered the SSE client. - Abort with an exception instead of proceeding without readiness; the fallback RPC + polling path in OrderbookBloc handles this gracefully. - On UnknownClient retries, re-run the full first-byte readiness gate instead of a fixed 3s sleep. - Always reset the first-byte Completer on disconnect(), even if it was never completed, so subsequent sessions don't block on a stale future.
- dragon_charts_flutter@0.1.1-dev.4 - dragon_logs@2.0.1 - komodo_cex_market_data@0.1.0 - komodo_coin_updates@2.0.0 - komodo_coins@0.3.2 - komodo_defi_framework@0.4.0 - komodo_defi_local_auth@0.4.0 - komodo_defi_rpc_methods@0.4.0 - komodo_defi_sdk@0.5.0 - komodo_defi_types@0.4.0 - komodo_ui@0.3.1 - komodo_wallet_build_transformer@0.4.1 - komodo_wallet_cli@0.5.0
…on (#335) Add a secondary cache for current (non-historical) prices so priceIfKnown does not return null during periodic live-cache clears. Document streaming tech debt and cover behaviour with unit tests.
|
You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard. |
CharlVS
changed the title
Contributor
|
Visit the preview URL for this PR (updated for commit dbc1472): https://komodo-playground--pr337-dev-yphndsuf.web.app (expires Wed, 08 Apr 2026 17:18:25 GMT) 🔥 via Firebase Hosting GitHub Action 🌎 Sign: 2bfedd77fdea45b25ba7c784416e81f177aa5c47 |
CharlVS
changed the title
* fix(komodo_defi_types): support TRON explorer URLs * chore(sdk): commit pending SDK updates
* feat(komodo_defi_sdk): add Tronscan transaction history strategy Implement TronscanTransactionStrategy for fetching transaction history from the Tronscan HTTP API for TRX native and TRC-20 token assets. - Support mainnet (apilist.tronscanapi.com) and Nile testnet endpoints - Fetch TRX transfers via /api/transfer, TRC-20 via /api/token_trc20/transfers - Two-layer pagination: server-side batching with client-side aggregation - Retry on 429/503 with Retry-After header and exponential backoff with jitter - Optional TRON-PRO-API-KEY header for improved rate limits - Merge duplicate TransactionInfo across multiple pubkey addresses - Wire through TransactionHistoryStrategyFactory, Manager, and bootstrap - Add KomodoDefiSdkConfig.tronProApiKey configuration field - Update tests for TRX/TRC-20 asset selection, ordering, and retry behavior * feat(transaction-history): migrate TRON history to TRONGrid
Prevent CoinGecko support checks from poisoning repository selection and add TRX fallback IDs so price history can resolve. Deduplicate shared asset activation so already-active assets stop failing into polling fallbacks.
- Add recovery watchers in BalanceManager when activation fails at startup so callers do not need to re-subscribe after races - Refactor balance polling into reusable fetchLatestBalance helper - Improve FeeInfo types and freezed codegen in komodo_defi_types - Add fee_info_display widget in komodo_ui - Update build_config, dev_builds_artefact_downloader, and update_api_config CLI - Add balance_manager_recovery_test and fee_info_test
#342) NotSufficientBalance is shared across RPC namespaces; KdfErrorRegistry previously returned null for all ambiguous types, causing withdraw status errors to fall back to GenericErrorResponse. - Thread rpcMethodHint from ApiClient.post and BaseRequest.parseResponse - Extend GeneralErrorResponse.toTypedException with optional hint - When hint matches withdraw task methods, parse withdraw-specific balance errors
- enforce single-terminal SIA activation progress semantics\n- add first-terminal-wins guard in activation manager\n- validate and reject SIA withdraw source selection (from)\n- route SIA preview/execute/one-shot withdraw through legacy flow\n- remove SIA-specific withdraw RPC surface and tests; use generic withdraw parsing\n- add SIA activation, withdrawal, and tx-history strategy coverage BREAKING CHANGE: removed withdrawSia(), SiaWithdrawRequest, and SiaWithdrawResponse. Migrate to generic withdraw()/WithdrawStatusResponse + WithdrawResult.
* fix(ui): avoid duplicate icon precache requests * fix(ui): retry CDN icon loads after transient failures * fix(ui): harden icon precache fallback and manifest caching
…ache (#346) * fix(ui): avoid duplicate icon precache requests * fix(ui): retry CDN icon loads after transient failures * fix(ui): harden icon precache fallback and manifest caching * fix(market-data): add failure cooldown to CoinGecko coin list fetching When getCoinList() failed, _cachedCoinList stayed null and every subsequent price request re-triggered the API call via supports(), creating a request storm that exhausted rate limits. Add a 5-minute cooldown after failures to prevent retry spam, matching the existing RepositoryFallbackMixin backoff duration. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> --------- Co-authored-by: Claude Opus 4.6 <noreply@anthropic.com>
- Bump api_commit_hash to d56a7bc5028f10b9c1bb8370bb2ee16444796a23 (branch: main) - Update all platform checksums from v3.0.0-beta release - Add libkdf-macos-universal2-* pattern to macOS matching_pattern - Switch coins_repo_branch from feat/add-tron-coins to master
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Release Overview
This PR is the release candidate merge from
devintomainfor the current SDK release line.This cycle is anchored by the SDK/runtime work that landed in
#262, then layered with new protocol support, higher-level SDK APIs, KDF/framework upgrades, and a broad round of web/startup/auth hardening. The result is a materially stronger SDK stack with new asset coverage, better streaming behavior, and cleaner package boundaries for the next public release.Major Changes
RPC, caching, and streaming foundations
#262across the core SDK stack, minimizing RPC usage and adding comprehensive caching and streaming support.enable_*calls on a real SSE first-byte event so stream registration no longer races KDF startup (#332).#335).num/intcompatibility in minified and mixed-runtime paths (#336).New protocol and asset support
#316).#320).#319,#321,#225).#254).SDK and platform hardening
#312).#314).#313,#315).6133and invalid configs (#318).#328).Framework, tooling, and build pipeline
3.0.0-betapreview and latest bundled coins roll (#317,#331).z.cashsources and normalize build-source inputs (#301).Published Packages
Breaking package releases
komodo_cex_market_datav0.1.0komodo_coin_updatesv2.0.0komodo_defi_frameworkv0.4.0komodo_defi_local_authv0.4.0komodo_defi_rpc_methodsv0.4.0komodo_defi_sdkv0.5.0komodo_defi_typesv0.4.0komodo_wallet_cliv0.5.0Other package releases
dragon_charts_flutterv0.1.1-dev.4dragon_logsv2.0.1komodo_coinsv0.3.2komodo_uiv0.3.1komodo_wallet_build_transformerv0.4.1Changelog Highlights
Core SDK stack
komodo_defi_sdk v0.5.0#262.komodo_defi_framework v0.4.03.0.0-betapreview.#262.komodo_defi_rpc_methods v0.4.0#262.komodo_defi_types v0.4.0#262.komodo_defi_local_auth v0.4.0#262.Coins and market data
komodo_coin_updates v2.0.0#262.komodo_cex_market_data v0.1.0#262.komodo_coins v0.3.2UI and build tooling
komodo_ui v0.3.1komodo_wallet_build_transformer v0.4.1assets/transformer_invoker.txtand aligns source inputs.dragon_logs v2.0.1dragon_charts_flutter v0.1.1-dev.4Changelog
CHANGELOG.mdcontains the full package-by-package release notes for2026-03-23and should be treated as the authoritative release record for this PR.Validation
flutter analyze packages --no-pubis not green in this checkout because package tests reference missing dev-only dependencies such asmocktail, and several packages include externalvery_good_analysisfiles that are not present here.0 errors,28 warnings, and60 infosusing:flutter analyze packages/dragon_charts_flutter/lib packages/dragon_logs/lib packages/komodo_cex_market_data/lib packages/komodo_coin_updates/lib packages/komodo_coins/lib packages/komodo_defi_framework/lib packages/komodo_defi_local_auth/lib packages/komodo_defi_rpc_methods/lib packages/komodo_defi_sdk/lib packages/komodo_defi_types/lib packages/komodo_ui/lib packages/komodo_wallet_build_transformer/lib --no-pubConsumer Coordination
b5626fdc1d507172a8b37e79d6120423343ff6af, which includes this release line through#334.devhead also includes#335and#336, so those two fixes are present here but are not yet rolled into the app submodule.Review Focus
#262across the SDK stack.Diff Snapshot
23commits ahead ofmain184files changed37,725insertions /2,032deletionsFull package-by-package detail is in
CHANGELOG.md.