Skip to content

refactor(trezor): migrate auth options and state models to freezed#90

Merged
takenagain merged 2 commits intofeat/trezor-supportfrom
codex/review-and-migrate-models-to-freezed
Jun 23, 2025
Merged

refactor(trezor): migrate auth options and state models to freezed#90
takenagain merged 2 commits intofeat/trezor-supportfrom
codex/review-and-migrate-models-to-freezed

Conversation

@takenagain
Copy link
Copy Markdown
Contributor

@takenagain takenagain commented Jun 23, 2025

Summary

  • switch several data classes to freezed
  • generate new freezed files
  • exclude freezed outputs from indexes
  • update index files
  • add build dependencies for local auth

Testing

  • dart run build_runner build -d (per package)
  • dart run index_generator (per package)
  • dart format -o none .
  • flutter analyze (fails: 3865 issues)
  • flutter build web (fails to compile)

https://chatgpt.com/codex/tasks/task_e_685526214dec8331acbb038e611a1bb2

Summary by CodeRabbit

  • Refactor
    • Improved internal state management for authentication and device initialization, enhancing immutability and consistency.
  • Chores
    • Updated project dependencies and configuration files to support improved code generation and analysis.
  • New Features
    • Expanded public API by exporting additional types for broader accessibility.

Copilot AI review requested due to automatic review settings June 23, 2025 05:12
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Jun 23, 2025

Walkthrough

This update refactors authentication and Trezor initialization state classes to use the Freezed package for immutable data classes, adds generated Freezed files, updates dependencies and analyzer settings, adjusts index generator configurations to exclude Freezed files, and expands the types library export list.

Changes

File(s) Change Summary
.../lib/src/auth/auth_state.dart
.../lib/src/trezor/trezor_initialization_state.dart
Refactored state classes to use @freezed annotation and factory constructors for immutability and codegen.
.../lib/src/auth/auth_state.freezed.dart
.../lib/src/trezor/trezor_initialization_state.freezed.dart
Added Freezed-generated files implementing immutable data classes, value equality, and copyWith methods.
.../pubspec.yaml Added freezed_annotation to dependencies, freezed and build_runner to dev_dependencies.
.../index_generator.yaml (rpc_methods, sdk) Updated config to exclude **.freezed.dart files from index generation.
.../analysis_options.yaml Suppressed invalid_annotation_target analyzer error.
.../lib/src/types.dart Added export for seed_node/seed_node.dart.

Sequence Diagram(s)

sequenceDiagram
    participant UI
    participant AuthState (Freezed)
    participant TrezorState (Freezed)
    participant User

    UI->>AuthState (Freezed): Create/Update AuthenticationState
    AuthState (Freezed)-->>UI: Immutable state instance

    UI->>TrezorState (Freezed): Create/Update TrezorInitializationState
    TrezorState (Freezed)-->>UI: Immutable state instance

    AuthState (Freezed)->>User: Reference user data (optional)
    TrezorState (Freezed)->>UI: Reference device info (optional)
Loading

Suggested labels

enhancement

Suggested reviewers

  • CharlVS

Poem

In fields of code where Freezed winds blow,
Old boilerplate now melts like snow.
State classes trimmed, equality neat,
With copyWith magic, updates repeat.
A hop, a skip, dependencies grow—
The garden of types begins to show!
🐇✨


📜 Recent review details

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between f6b839c and ba5e09c.

📒 Files selected for processing (9)
  • packages/komodo_defi_local_auth/lib/src/auth/auth_state.dart (1 hunks)
  • packages/komodo_defi_local_auth/lib/src/auth/auth_state.freezed.dart (1 hunks)
  • packages/komodo_defi_local_auth/lib/src/trezor/trezor_initialization_state.dart (3 hunks)
  • packages/komodo_defi_local_auth/lib/src/trezor/trezor_initialization_state.freezed.dart (1 hunks)
  • packages/komodo_defi_local_auth/pubspec.yaml (1 hunks)
  • packages/komodo_defi_rpc_methods/index_generator.yaml (1 hunks)
  • packages/komodo_defi_sdk/index_generator.yaml (1 hunks)
  • packages/komodo_defi_types/analysis_options.yaml (1 hunks)
  • packages/komodo_defi_types/lib/src/types.dart (1 hunks)
🧰 Additional context used
🪛 GitHub Actions: Deploy Playground to Firebase Hosting on PR
packages/komodo_defi_local_auth/lib/src/auth/auth_state.dart

[error] 1-1: AssetNotFoundException: Missing generated file 'komodo_defi_types|lib/src/transactions/fee_info.freezed.dart' required by freezed code generation.

🔇 Additional comments (14)
packages/komodo_defi_rpc_methods/index_generator.yaml (1)

8-8: Exclude .freezed.dart files from index generation
Pattern addition aligns with existing .g.dart exclusion to keep generated boilerplate out of index files.

packages/komodo_defi_local_auth/pubspec.yaml (2)

29-29: Add freezed_annotation to dependencies
Enables use of @freezed annotation in the package for immutable data classes.


36-37: Add build_runner and freezed to dev_dependencies
Supports code generation for freezed models; versions are consistent with freezed_annotation.

packages/komodo_defi_types/analysis_options.yaml (1)

4-4: Suppress invalid_annotation_target analyzer warning
Prevents spurious errors when using code generation annotations (e.g., @freezed).

packages/komodo_defi_sdk/index_generator.yaml (1)

8-8: Exclude .freezed.dart files from index generation
Keeps generated .freezed.dart files out of the public index, mirroring other packages.

packages/komodo_defi_types/lib/src/types.dart (1)

48-48: Export seed_node/seed_node.dart in types.dart
Exposes the new SeedNode module in the public API surface; placement respects alphabetical grouping.

packages/komodo_defi_local_auth/lib/src/auth/auth_state.dart (3)

1-4: Excellent refactoring to freezed!

The migration from manual Equatable implementation to freezed is well-executed. The import change and part directive are correctly placed.


7-15: Clean freezed class definition.

The @freezed annotation and factory constructor properly replace the manual boilerplate while maintaining all original fields and their types.


17-22: Factory constructors preserved correctly.

The named factory constructors completed and error maintain the existing public API, ensuring backward compatibility.

packages/komodo_defi_local_auth/lib/src/trezor/trezor_initialization_state.freezed.dart (1)

1-155: Generated freezed file looks correct.

This auto-generated file follows standard freezed patterns with proper equality, copyWith methods, and immutability. No manual modifications needed.

packages/komodo_defi_local_auth/lib/src/trezor/trezor_initialization_state.dart (3)

1-5: Proper freezed setup.

Import changes and part directive are correctly implemented for the freezed migration.


8-18: Well-structured freezed class with private constructor.

The @freezed annotation, factory constructor, and private constructor TrezorInitializationState._() are properly set up to support the custom factory methods below.


20-154: Factory methods and business logic preserved.

All the complex factory methods (fromStatusResponse, fromInProgressDescription, fromUserActionRequired) and the toAuthenticationState() conversion method are correctly maintained, ensuring no functionality is lost in the refactoring.

packages/komodo_defi_local_auth/lib/src/auth/auth_state.freezed.dart (1)

1-155: Standard freezed generation looks correct.

This auto-generated file provides all the expected freezed functionality including immutability, value equality, and copyWith methods. No manual changes required.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share
🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

  • Review comments: Directly reply to a review comment made by CodeRabbit. Example:
    • I pushed a fix in commit <commit_id>, please review it.
    • Explain this complex logic.
    • Open a follow-up GitHub issue for this discussion.
  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
    • @coderabbitai explain this code block.
    • @coderabbitai modularize this function.
  • PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
    • @coderabbitai read src/utils.ts and explain its main purpose.
    • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
    • @coderabbitai help me debug CodeRabbit configuration file.

Support

Need help? Create a ticket on our support page for assistance with any issues or questions.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (Invoked using PR comments)

  • @coderabbitai pause to pause the reviews on a PR.
  • @coderabbitai resume to resume the paused reviews.
  • @coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
  • @coderabbitai full review to do a full review from scratch and review all the files again.
  • @coderabbitai summary to regenerate the summary of the PR.
  • @coderabbitai generate docstrings to generate docstrings for this PR.
  • @coderabbitai generate sequence diagram to generate a sequence diagram of the changes in this PR.
  • @coderabbitai resolve resolve all the CodeRabbit review comments.
  • @coderabbitai configuration to show the current CodeRabbit configuration for the repository.
  • @coderabbitai help to get help.

Other keywords and placeholders

  • Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
  • Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
  • Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (.coderabbit.yaml)

  • You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
  • Please see the configuration documentation for more information.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

@takenagain takenagain marked this pull request as draft June 23, 2025 05:12

This comment was marked as outdated.

@takenagain takenagain changed the title Migrate models to freezed refactor(trezor): migrate auth options and state models to freezed Jun 23, 2025
@takenagain takenagain self-assigned this Jun 23, 2025
@takenagain
Copy link
Copy Markdown
Contributor Author

@coderabbitai review

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Jun 23, 2025

✅ Actions performed

Review triggered.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@takenagain takenagain requested a review from Copilot June 23, 2025 11:09
Copy link
Copy Markdown
Contributor

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 refactors several authentication-related state models to use Freezed, updates indexing to exclude generated Freezed files, and adds necessary build dependencies.

  • Migrated TrezorInitializationState and AuthenticationState classes to Freezed
  • Excluded *.freezed.dart files in index generator configs
  • Added freezed, freezed_annotation, and build_runner dependencies for code generation

Reviewed Changes

Copilot reviewed 9 out of 9 changed files in this pull request and generated no comments.

Show a summary per file
File Description
packages/komodo_defi_types/lib/src/types.dart Export new seed_node model
packages/komodo_defi_types/analysis_options.yaml Ignored invalid_annotation_target lint
packages/komodo_defi_sdk/index_generator.yaml Added *.freezed.dart to exclude list
packages/komodo_defi_rpc_methods/index_generator.yaml Added *.freezed.dart to exclude list
packages/komodo_defi_local_auth/pubspec.yaml Added Freezed codegen dependencies
packages/komodo_defi_local_auth/lib/src/trezor/trezor_initialization_state.freezed.dart Generated Freezed code
packages/komodo_defi_local_auth/lib/src/trezor/trezor_initialization_state.dart Refactored Trezor initialization state to Freezed
packages/komodo_defi_local_auth/lib/src/auth/auth_state.freezed.dart Generated Freezed code
packages/komodo_defi_local_auth/lib/src/auth/auth_state.dart Refactored authentication state to Freezed
Comments suppressed due to low confidence (6)

packages/komodo_defi_types/analysis_options.yaml:4

  • Ignoring invalid_annotation_target globally may suppress legitimate errors in hand-written code; consider scoping this ignore to generated files only.
    invalid_annotation_target: ignore

packages/komodo_defi_sdk/index_generator.yaml:8

  • [nitpick] Quoting style in the exclude list is inconsistent; standardize on single or double quotes across all index_generator configs.
    - "**.freezed.dart"

packages/komodo_defi_rpc_methods/index_generator.yaml:8

  • [nitpick] Quoting style in the exclude list is inconsistent; standardize on single or double quotes across all index_generator configs.
    - '**.freezed.dart'

packages/komodo_defi_local_auth/pubspec.yaml:29

  • [nitpick] Verify whether freezed_annotation is needed at runtime; if it's only for code generation, consider moving it to dev_dependencies.
  freezed_annotation: ^3.0.0

packages/komodo_defi_local_auth/lib/src/auth/auth_state.dart:17

  • [nitpick] Add a documentation comment for the completed factory constructor to describe its behavior, matching the style of other documented members.
  factory AuthenticationState.completed(KdfUser user) =>

packages/komodo_defi_types/lib/src/types.dart:48

  • [nitpick] Align the indentation of the new export with the existing ones for consistency and readability.
export 'seed_node/seed_node.dart';

@takenagain takenagain marked this pull request as ready for review June 23, 2025 12:43
@takenagain takenagain merged commit 75228d7 into feat/trezor-support Jun 23, 2025
3 of 4 checks passed
@takenagain takenagain deleted the codex/review-and-migrate-models-to-freezed branch June 23, 2025 12:47
CharlVS pushed a commit that referenced this pull request Jun 24, 2025
* fix(trezor,activation): add PrivateKeyPolicy to AuthOptions and beyond

* feat(rpc): add trezor init task RPCs

* feat(sdk): trezor manager for init and user input requests

* feat(rpc): add task-based address generation RPCs

* feat(rpc): add trezor pubkey strategy with task-based address creation

* WIP: trezor login option in SDK example

* refactor(example): trezor auth mixin and remove previous trezor ui code

* fix(trezor): device info parsing and example auth state transitions

* refactor(trezor): add auth listener & implement coderabbitai suggestions

* fix(example): auth restoration and listener order

* fix(trezor): workaround for trezor-specifc scan issue

funded addresses not being scanned for fresh trezor logins (maybe not believed to be a "fresh" wallet)?

* fix(rpc): get new address task status response parsing

* refactor(trezor): move authentication and initialization logic to SDK (#81)

This is a concept solution that moves the Trezor initialization and authentication flow from the example BLoC to the SDK KomodoDefiAuth class & package.

Two issues that make integrating the trezor login flow with the current SDK difficult:

- Trezor login is a multi-step process that uses task-based RPCs, so Stream-based versions of the signin and register methods are required.
- Async user inputs (i.e. passphrase or pin) need to be sent to trezor as part of login process.

Solution implemented in this PR:

- Add stream-based signInStream and registerStream functions to KomodoDefiAuth with a new Authentication state/status class.
  - Alternative (breaking): modify the existing functions to return Stream
- Add input methods for passphrase and pin to KomodoDefiAuth
  - Alternative 1: Ask user for passphrase before starting trezor sign-in and pass it through the existing wallet password field.
  - Alternative 2: SDK consumers use device-specific managers (e.g. sdk.trezor.sendPassphrase).

Commits:

* Use TrezorWalletManager in example bloc

* refactor(trezor,auth): move trezor auth into komodo_defi_local_auth

* refactor: implement coderabbitai suggestions

* refactor(trezor): migrate auth options and state models to freezed (#90)

* migrate models to freezed

* fix(types): revert changes to seed_node and auth_options
CharlVS added a commit that referenced this pull request Aug 5, 2025
commit 0303d39
Author: CharlVS <77973576+CharlVS@users.noreply.github.com>
Date:   Fri Aug 1 15:52:14 2025 +0200

    chore: switch KDF to `dev`

    Switch KDF to `dev` to allow usage of unreleased changes (particularly related to fees/priorities)

commit 1fae456
Author: Francois <takenagain@users.noreply.github.com>
Date:   Fri Aug 1 17:46:34 2025 +0200

    fix(market-data-price): try fetch current price from komodo price repository first before cex repository (#167)

    * fix(market-data-manager): default to KomodoPriceRepository

    Binance CexRepository supports a limited number of symbols and no longer lists KMD price history after the delisting

    * refactor(market-data-manager): deduplicate and clarify intent

    * refactor(review): remove zero check and add log statements

    also simplify the priceIfKnown function

commit f8064f2
Author: CharlVS <77973576+CharlVS@users.noreply.github.com>
Date:   Fri Aug 1 15:50:09 2025 +0200

    fix(types): comprehensive bip39 validation

commit 60e5f2e
Author: Charl (Nitride) <77973576+CharlVS@users.noreply.github.com>
Date:   Wed Jul 30 21:18:06 2025 +0200

    feat: offline private key export (#160)

    * feat(hd): integrate hd privkey export

    * chore: roll KDF

    * chore: enable tendermint hd support

    * refactor(rpc): Misc rpc class formats/fixes

    * feat: private keys + fixed Tendermint

    * refactor(sdk): implement co-ordinated activation manager

    * fix: invalid path error

    Possible fix for paths containing special characters on Windows.

    * fix: remove unnecessary activation logging

    * chore(pr): apply formatting and fixes to PR

    Apply Dart formatting and fixes to all files modified in this PR.

    * feat(example): integrate private key export to example app

    * chore: roll KDF

    * chore: roll KDF

    * feat: fee estimation

    * fix(build): make re-build failure stand out

    * chore: roll KDF

    Roll KDF for optimisation for key export time.

    * feat(tx): further progress on fee/priority integration

    * chore: roll KDF fro `v2.6.1`

    TODO: Change from CI builds to GitHub release when artefacts are available.

    * fix(api): disable fee/priority features

    Disable fee/priority features because they are not included in the pending KDF release. They should be included in the following release (likely `2.7.0`)

    * fix: Temporarily disable fee/priority features

    Temporarily disable fee/priority features until the feature is available in the API (KDF)

    * feat: add memo support protocol value

    * fix: resolve incorrect `unban_pubkeys` parsing

    * fix: resolve regression in legacy withdrawal manager

    * feat(security): Include failed activations in privkey export

    * docs(build): expand asset build failure explination

    * fix: tweak private key type json to match class properties

    * fix(RPC): fix new address generation types

    * chore: bump version tags

    * fix: apply Claude PR review suggestions

commit 834b83f
Merge: 1526a23 a41f81f
Author: Charl (Nitride) <77973576+CharlVS@users.noreply.github.com>
Date:   Wed Jul 30 20:12:37 2025 +0200

    Merge pull request #165 from KomodoPlatform/migrate/dragon_logs_flutter

    feat: migrate dragon_logs_flutter to monorepo packages

commit a41f81f
Merge: 1526a23 a6a9a6e
Author: CharlVS <77973576+CharlVS@users.noreply.github.com>
Date:   Wed Jul 30 18:19:07 2025 +0200

    Merge commit 'a6a9a6ef4a308f95be1fafbb018449f198d8b4db' as 'packages/dragon_logs'

commit a6a9a6e
Author: CharlVS <77973576+CharlVS@users.noreply.github.com>
Date:   Wed Jul 30 18:19:07 2025 +0200

    Squashed 'packages/dragon_logs/' content from commit 399ef1d1

    git-subtree-dir: packages/dragon_logs
    git-subtree-split: 399ef1d17034029155b63b6bc1d9ab0a5a0fbb04

commit 1526a23
Merge: dc093db acda1d9
Author: CharlVS <77973576+CharlVS@users.noreply.github.com>
Date:   Mon Jul 28 21:10:37 2025 +0200

    Merge branch 'codex/implement-pubkey-unbanning-feature' into dev

commit acda1d9
Author: CharlVS <77973576+CharlVS@users.noreply.github.com>
Date:   Mon Jul 28 21:07:58 2025 +0200

    fix(ui): resolve stale asset balance widget

commit dc093db
Author: Charl (Nitride) <77973576+CharlVS@users.noreply.github.com>
Date:   Mon Jul 28 20:18:02 2025 +0200

    feat(pubkeys): add unbanning support (#161)

    * feat(pubkeys): add unbanning support

    * Update packages/komodo_defi_rpc_methods/lib/src/rpc_methods/wallet/unban_pubkeys.dart

    Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

    * fix(types): export missing RPC types

    ---------

    Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

commit 7d7e888
Author: CharlVS <77973576+CharlVS@users.noreply.github.com>
Date:   Mon Jul 28 20:14:54 2025 +0200

    fix(types): export missing RPC types

commit 9621aa2
Author: Charl (Nitride) <77973576+CharlVS@users.noreply.github.com>
Date:   Mon Jul 28 20:00:48 2025 +0200

    Update packages/komodo_defi_rpc_methods/lib/src/rpc_methods/wallet/unban_pubkeys.dart

    Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

commit 4a3d9c8
Author: Charl (Nitride) <77973576+CharlVS@users.noreply.github.com>
Date:   Mon Jul 28 18:33:47 2025 +0200

    feat(pubkeys): add unbanning support

commit b18af0b
Author: Charl (Nitride) <77973576+CharlVS@users.noreply.github.com>
Date:   Sun Jul 27 16:59:11 2025 +0200

    feat(ui): add hero support for coin icons (#159)

    * feat(ui): add hero support for asset icons

    * fix(ui): Hero Tag Ignored in AssetLogo Placeholder

    * fix: Move Hero widget outside Opacity widget for proper animation continuity

    - Fix AssetLogo.placeholder to pass heroTag parameter to _AssetLogoPlaceholder
    - Move Hero widget outside Opacity widget in AssetIcon to prevent animation interference
    - Ensures Hero animations work correctly for both asset icons and placeholder logos
    - Maintains visual continuity by placing Hero widgets as outermost components

commit c2d985a
Author: Charl (Nitride) <77973576+CharlVS@users.noreply.github.com>
Date:   Fri Jul 25 15:20:29 2025 +0200

    feat(fees): integrate fee management (#152)

    * feat(fees): add fee management support

    * docs(sdk): Document withdrawal manager

    Document withdrawal manager and implement PR review comments.

    * docs(sdk): Add documentation for fee manager

commit e1a63e2
Author: Charl (Nitride) <77973576+CharlVS@users.noreply.github.com>
Date:   Fri Jul 25 11:15:26 2025 +0200

    feat(activation): disable tx history when using external strategy (#151)

    * feat(activation): disable kdf tx history when using external strategy

    * docs: clarify mm2 history helper (#154)

commit 74d8eff
Author: Francois <takenagain@users.noreply.github.com>
Date:   Thu Jul 24 18:30:54 2025 +0200

    Revert "fix(activation): track activation status to avoid duplicate activation requests (#80)" (#153)

    This reverts commit abd3de0.

commit 7eefbd8
Author: dragonhound <35845239+smk762@users.noreply.github.com>
Date:   Mon Jul 21 20:24:39 2025 +0800

    adds `max_connected: 1` to utxo activation (#148)

commit b8ff967
Author: Charl (Nitride) <77973576+CharlVS@users.noreply.github.com>
Date:   Mon Jul 21 14:24:16 2025 +0200

    Revert "feat(rpc): support max_connected on activation (#149)" (#150)

    This reverts commit eac2b8f.

commit eac2b8f
Author: Charl (Nitride) <77973576+CharlVS@users.noreply.github.com>
Date:   Mon Jul 21 13:11:54 2025 +0200

    feat(rpc): support max_connected on activation (#149)

    * feat(rpc): support max_connected on activation

    * fix: limit max connections to 1

commit bb57e87
Author: CharlVS <77973576+CharlVS@users.noreply.github.com>
Date:   Sun Jul 20 00:15:49 2025 +0200

    fix(build): update config format

commit 6534d47
Author: Charl (Nitride) <77973576+CharlVS@users.noreply.github.com>
Date:   Sun Jul 20 00:04:09 2025 +0200

    feat(wallet_build_transformer): add flexible CDN support (#144)

commit abd3de0
Author: Francois <takenagain@users.noreply.github.com>
Date:   Thu Jul 17 18:54:01 2025 +0200

    fix(activation): track activation status to avoid duplicate activation requests (#80)

    * Track activation streams to avoid duplicate requests

    * fix activation stream propagation

    * fix(activation): provide historical states to late stream subscribers

    * fix(activation): use asset lookup in parent grouping

    using the asset itself as a placeholder led to assertion errors being thrown if the child asset is activated individually.

commit 9351c22
Author: Francois <takenagain@users.noreply.github.com>
Date:   Wed Jul 16 19:55:00 2025 +0200

    feat(ui): allow customizing SourceAddressField header (#135)

    * feat(ui): make source address field customizable

    * refactor: remove content override from SourceAddressField

commit a521a80
Author: Francois <takenagain@users.noreply.github.com>
Date:   Wed Jul 16 14:37:28 2025 +0200

    fix(withdraw): revert temporary IBC channel type changes (#136)

    #63

commit a44acf3
Author: CharlVS <77973576+CharlVS@users.noreply.github.com>
Date:   Mon Jul 14 16:14:45 2025 +0200

    fix(ui): make Divided button min width

commit aa51985
Author: Charl (Nitride) <77973576+CharlVS@users.noreply.github.com>
Date:   Mon Jul 7 17:15:04 2025 +0200

    feat: protect SDK after disposal (#116)

    * feat: prevent SDK reuse after disposal

    * refactor(sdk): promote dispose helper (#127)

commit 9cf9e37
Author: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Date:   Mon Jul 7 15:08:29 2025 +0200

    chore: roll KDF version to commit fbab8ec for branch main (#124)

    * chore: roll KDF version to commit fbab8ec for branch main

    * chore: update binaries

    * chore: roll coins assets

    ---------

    Co-authored-by: GitHub Actions <github-actions[bot]@users.noreply.github.com>
    Co-authored-by: CharlVS <77973576+CharlVS@users.noreply.github.com>

commit ce68c7a
Author: Charl (Nitride) <77973576+CharlVS@users.noreply.github.com>
Date:   Sat Jul 5 19:24:33 2025 +0200

    feat(seed): update seed node format (#87)

    * feat(seed): update seed node format with netid

    * Update packages/komodo_defi_types/lib/src/seed_node/seed_node.dart

    Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

    * feat(core): use constant netid in seed node filtering (#111)

    * fix: add missing imports/exports

    ---------

    Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

commit d7b6ede
Author: Francois <takenagain@users.noreply.github.com>
Date:   Sat Jul 5 18:37:15 2025 +0200

    feat(pubkey): add streamed new address API with Trezor confirmations (#123)

    * feat(pubkey): add streamed new address API with Trezor confirmations (#117)

    * Implement streamed new address state with confirmations

    * refactor: remove unused imports and make polling interval configurable

    * refactor: move confirm address details to types (#120)

    * refactor: move confirm address details to types

    * refactor: fix warnings

    * feat(example): use streamed address creation (#119)

    * Use streamed address generation

    * refactor: add cancel action and use normal stream rather than broadcast

    * chore: add json conversion support to `NewAddressState`

    * fix(asset-filter): exclude BCH and yield address creation error states

    * refactor: add task description parser and try cancel task if user cancel

commit d6d83c6
Author: Charl (Nitride) <77973576+CharlVS@users.noreply.github.com>
Date:   Fri Jul 4 18:30:50 2025 +0200

    fix(local_auth): ensure kdf running before wallet deletion (#118)

commit cda8498
Author: Francois <takenagain@users.noreply.github.com>
Date:   Thu Jul 3 20:44:32 2025 +0200

    feat(ui): add helper constructors for AssetLogo from legacy ticker and AssetId (#109)

    * feat(ui): add AssetLogo constructors

    * fix: correct protocol overlay for child assets (#115)

    * fix(example): use parent protocol for child assets in trailing icon

    * feat(ui): add placeholder asset logo

    used in legacy maker and taker forms in KW

    * feat(ui): add asset exists helper functions

    used in KW fiat onramp form

    * refactor: clarify doc comments and rename constructors

    * fix(assets): tendermint token parent parsing logic

    - place tendermint before tendermint token
    - replace subclass check with canBeParentOf

    * refactor(asset-logo): use theme `disabledColor` in place of  Opacity

    ---------

    Co-authored-by: Charl (Nitride) <77973576+CharlVS@users.noreply.github.com>

commit 3dc37d0
Author: CharlVS <77973576+CharlVS@users.noreply.github.com>
Date:   Thu Jul 3 19:16:58 2025 +0200

    add(ui): enhance TrendPercentageText

    Enhance `TrendPercentageText` with expanded use cases and automatic animations.

commit c5c30d4
Author: CharlVS <77973576+CharlVS@users.noreply.github.com>
Date:   Wed Jul 2 18:45:02 2025 +0200

    fix: resolve bug with dispose logic

    TODO(@CharlVS): Consider separating the KDF-related concerns of dispose into a separate method named terminate(). This helps differentiate between KDF resource clean-up and UI resource clean-up

commit 9d63c56
Author: CharlVS <77973576+CharlVS@users.noreply.github.com>
Date:   Wed Jul 2 15:34:14 2025 +0200

    add(auth): auth stream method with initial value

commit 81ba643
Author: CharlVS <77973576+CharlVS@users.noreply.github.com>
Date:   Wed Jul 2 11:09:33 2025 +0200

    fix: stop KDF when disposed

commit c8f27a7
Author: CharlVS <77973576+CharlVS@users.noreply.github.com>
Date:   Wed Jul 2 10:33:27 2025 +0200

    fix: SIA support

    Disable SIA as it is not yet supported. The SIA activation strategy will be merged soon, but this depends on a specific branch of KDF and will likely still be very long before it is available in a public release of KDF.

commit a02c4a2
Author: Francois <takenagain@users.noreply.github.com>
Date:   Wed Jul 2 09:48:09 2025 +0200

    fix(trezor): improve trezor error handling (#110)

    * chore(deps): bump logging version to 1.3.0 as in komodo-wallet

    * fix(trezor): sign out of existing wallet if initialisation fails

    fixes the following scenarios
    - trezor device not found
    - trezor device not selected

    and other edge cases that may cause trezor init or auth to fail, ensuring that the user already exists error is not thrown

    * feat(trezor): add priv_key_policy to tendermint, erc20, and zhtlc params

    * fix(trezor): yield error states rather than exceptions in streams

    * feat(activation,rpc): task-based eth activation

    * feat(asset-manager): asset filter strategy based on current wallet (#107)

    * feat(asset-manager): asset filter strategy based on current wallet

    * featasset-manager): limit trezor asset filter to utxo and qtum only

    * refactor: add error response check and evm subclasses set

    coderabbitai suggestions

    * feat: add name-based filter strategy (#108)

    * refactor: update authOptions references to use walletId

    * refactor(asset-filter): rename `name` to `strategyId`

    * ci(example): re-enable dry build run

commit 8a1daea
Author: Francois <takenagain@users.noreply.github.com>
Date:   Wed Jul 2 09:45:18 2025 +0200

    feat(ui): adjust error display layout for narrow screens (#114)

    * chore(deps): bump logging version to 1.3.0 as in komodo-wallet

    * fix(trezor): sign out of existing wallet if initialisation fails

    fixes the following scenarios
    - trezor device not found
    - trezor device not selected

    and other edge cases that may cause trezor init or auth to fail, ensuring that the user already exists error is not thrown

    * feat(trezor): add priv_key_policy to tendermint, erc20, and zhtlc params

    * fix(trezor): yield error states rather than exceptions in streams

    * feat(activation,rpc): task-based eth activation

    * feat(asset-manager): asset filter strategy based on current wallet (#107)

    * feat(asset-manager): asset filter strategy based on current wallet

    * featasset-manager): limit trezor asset filter to utxo and qtum only

    * refactor: add error response check and evm subclasses set

    coderabbitai suggestions

    * feat: add name-based filter strategy (#108)

    * refactor: update authOptions references to use walletId

    * refactor(asset-filter): rename `name` to `strategyId`

    * refactor: extract show details button

    * Refactor error display components

    * refactor: convert toggle to stateless widget

    * feat(ui): add toggle for icon and expose breakpoint

    * fix(ui): button interactivity no-op when showDetailsOverride is true

    Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>

    * refactor: remove unnecessary spread operator

    ---------

    Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>

commit 5f7da9e
Author: CharlVS <77973576+CharlVS@users.noreply.github.com>
Date:   Mon Jun 30 15:34:29 2025 +0200

    ci: skip melos init

    Disabled melos generation due to circular dependency issues caused by komodo_defi_rpc_methods and komodo_defi_types packages. This will be revisited in the.

    The app should already have the necessary generated files committed to the repository. If this is not the case, we have bigger issues.

commit f6f5820
Author: Charl (Nitride) <77973576+CharlVS@users.noreply.github.com>
Date:   Mon Jun 30 15:25:59 2025 +0200

    add: Wallet deletion example and fix global error parsing flaw (#99)

    * feat(auth): add wallet deletion

    * feat(example): add wallet deletion

    * add: wallet deletion for Trezor wallets

    * chore: re-generate melos

    * refactor: fix flaw in response error parsing

    Fix flaw in response error parsing where the custom error parser function would never be called because of the inheritence structure.

    * feat(example): move delete wallet action (#104)

commit fafc707
Author: Charl (Nitride) <77973576+CharlVS@users.noreply.github.com>
Date:   Mon Jun 30 14:57:00 2025 +0200

    feat(asset): add message signing support flag (#105)

commit 3701f49
Author: CharlVS <77973576+CharlVS@users.noreply.github.com>
Date:   Mon Jun 30 14:41:23 2025 +0200

    fix: kdf hashes

commit a2face7
Author: Charl (Nitride) <77973576+CharlVS@users.noreply.github.com>
Date:   Mon Jun 30 14:16:56 2025 +0200

    chore(kdf): update build config to latest staging (#103)

commit 41fb498
Author: Francois <takenagain@users.noreply.github.com>
Date:   Mon Jun 30 11:45:18 2025 +0200

    chore(deps): bump komodo-defi-framework to a25aea614 (#102)

    latest staging with hotfix for trezor coin activation

commit 7dfe68b
Author: Charl (Nitride) <77973576+CharlVS@users.noreply.github.com>
Date:   Fri Jun 27 14:58:42 2025 +0200

    chore: roll KDF (#100)

commit e5d022b
Author: Francois <takenagain@users.noreply.github.com>
Date:   Thu Jun 26 16:41:13 2025 +0200

    fix(pubkey-strategy): use new PrivateKeyPolicy constructors for checks (#97)

commit 24d09fd
Author: Francois <takenagain@users.noreply.github.com>
Date:   Wed Jun 25 16:08:18 2025 +0200

    fix(activation): eth PrivateKeyPolicy enum breaking changes (#96)

    * fix(rpc,activation): breaking EVM private key policy enum change

    * fix(activation-params): use pascal case for to and from json

    * test(example): add asset activation integration test

    * test(example): run in zone to catch microtask and uncaught exceptions

    * fix(auth): merge issues and update privkeypolicy comparisons

    * docs: expand doc comments and remove runzonedguarded

commit 2a50691
Author: Francois <takenagain@users.noreply.github.com>
Date:   Tue Jun 24 18:52:35 2025 +0200

    feat(sdk): add trezor support via RPC and SDK wrappers (#77)

    * fix(trezor,activation): add PrivateKeyPolicy to AuthOptions and beyond

    * feat(rpc): add trezor init task RPCs

    * feat(sdk): trezor manager for init and user input requests

    * feat(rpc): add task-based address generation RPCs

    * feat(rpc): add trezor pubkey strategy with task-based address creation

    * WIP: trezor login option in SDK example

    * refactor(example): trezor auth mixin and remove previous trezor ui code

    * fix(trezor): device info parsing and example auth state transitions

    * refactor(trezor): add auth listener & implement coderabbitai suggestions

    * fix(example): auth restoration and listener order

    * fix(trezor): workaround for trezor-specifc scan issue

    funded addresses not being scanned for fresh trezor logins (maybe not believed to be a "fresh" wallet)?

    * fix(rpc): get new address task status response parsing

    * refactor(trezor): move authentication and initialization logic to SDK (#81)

    This is a concept solution that moves the Trezor initialization and authentication flow from the example BLoC to the SDK KomodoDefiAuth class & package.

    Two issues that make integrating the trezor login flow with the current SDK difficult:

    - Trezor login is a multi-step process that uses task-based RPCs, so Stream-based versions of the signin and register methods are required.
    - Async user inputs (i.e. passphrase or pin) need to be sent to trezor as part of login process.

    Solution implemented in this PR:

    - Add stream-based signInStream and registerStream functions to KomodoDefiAuth with a new Authentication state/status class.
      - Alternative (breaking): modify the existing functions to return Stream
    - Add input methods for passphrase and pin to KomodoDefiAuth
      - Alternative 1: Ask user for passphrase before starting trezor sign-in and pass it through the existing wallet password field.
      - Alternative 2: SDK consumers use device-specific managers (e.g. sdk.trezor.sendPassphrase).

    Commits:

    * Use TrezorWalletManager in example bloc

    * refactor(trezor,auth): move trezor auth into komodo_defi_local_auth

    * refactor: implement coderabbitai suggestions

    * refactor(trezor): migrate auth options and state models to freezed (#90)

    * migrate models to freezed

    * fix(types): revert changes to seed_node and auth_options

commit b863ccb
Author: Charl (Nitride) <77973576+CharlVS@users.noreply.github.com>
Date:   Tue Jun 24 17:40:41 2025 +0200

    fix(auth): allow custom seeds for legacy wallets (#95)

commit 3e9f73c
Author: CharlVS <77973576+CharlVS@users.noreply.github.com>
Date:   Mon Jun 23 15:32:19 2025 +0200

    chore: re-generate melos configs

commit 1a4ec66
Author: CharlVS <77973576+CharlVS@users.noreply.github.com>
Date:   Mon Jun 23 15:31:36 2025 +0200

    fix: remove obsolete coins transformer

    Remove ETH coins config transformer which was temporarily required because of a breaking ETH change in KDF.

commit 58acbc2
Author: CharlVS <77973576+CharlVS@users.noreply.github.com>
Date:   Mon Jun 23 15:21:36 2025 +0200

    bug: fix missing pubkey equality operators

commit d05c13b
Author: CharlVS <77973576+CharlVS@users.noreply.github.com>
Date:   Mon Jun 23 15:19:50 2025 +0200

    chore: update KDF to release preview version

commit fdeb713
Author: dragonhound <35845239+smk762@users.noreply.github.com>
Date:   Wed Jun 18 20:38:33 2025 +0800

    removes BCH from _filteredCoins (#83)

commit a36357e
Author: Francois <takenagain@users.noreply.github.com>
Date:   Wed Jun 18 14:38:12 2025 +0200

    feat(ui): add AssetLogo widget (#78)

    * Use AssetLogo in example and tweak imports

    * style: match the styling with the existing komodo-wallet implementation

commit 8990c9c
Author: DeckerSU <support@decker.su>
Date:   Wed Jun 18 14:37:22 2025 +0200

    fix KDF not found warnings in _tryRenameFile of FetchDefiApiStep (#76)

    If the KDF binaries were already renamed from `mm2` to `kdf`—i.e., if the target folder already contained `kdf.exe` or `libkdf.a` after the last successful rename from `*mm2*`—the warning `KDF not found at` would still appear, which was a bit confusing. Now, if the folder already contains the renamed binary, the warning no longer appears.

commit 55176e1
Author: CharlVS <77973576+CharlVS@users.noreply.github.com>
Date:   Mon Jun 16 01:02:04 2025 +0200

    fix: revert ETH coins config migration transformer

commit 7d438bd
Author: Francois <takenagain@users.noreply.github.com>
Date:   Sun Jun 15 17:31:26 2025 +0200

    fix(kdf): enable p2p in noAuth mode (#86)

    * fix(kdf): enable p2p in noAuth mode

    * refactor(kdf): use SeedNodeService to get seednodes for noAuth

commit 4038b86
Author: CharlVS <77973576+CharlVS@users.noreply.github.com>
Date:   Sun Jun 15 02:15:36 2025 +0200

    chore: refresh KDF checksums

commit aa8148b
Author: Charl (Nitride) <77973576+CharlVS@users.noreply.github.com>
Date:   Sun Jun 15 00:17:09 2025 +0200

    feat: add configurable seed node system with remote fetching (#85)

    * feat: add custom seed nodes

    - Add custom seed nodes to accommodate breaking KDF changes.
    - Upgrade KDF for latest seed node changes.

    * feat: add configurable seed node system with remote fetching

    Implement a comprehensive seed node configuration system that fetches
    seed nodes from the Komodo Platform repository with automatic fallback
    to hardcoded defaults.

    ### New Features:
    - Add SeedNode and SeedNodeContact types for type-safe configuration
    - Implement SeedNodeUpdater for fetching remote seed nodes
    - Add SeedNodeService with automatic fallback mechanism
    - Add SeedNodeValidator with enhanced P2P configuration validation
    - Export seed node service from komodo_defi_framework

    ### Configuration Updates:
    - Update build_config.json to include seed-nodes.json mapping
    - Add seed_nodes.json to gitignore for dynamic assets
    - Change API branch from 'dev' to 'alt-main'
    - Update all platform binary checksums and commit hashes

    ### Framework Integration:
    - Add automatic seed node fetching to KdfStartupConfig
    - Integrate SeedNodeService into auth service for wallet creation
    - Fix trading protocol parameter name (trading_proto_v2 -> use_trading_proto_v2)
    - Improve boolean validation in SeedNodeValidator

    ### Documentation:
    - Add comprehensive SEED_NODES_GUIDE.md with usage examples
    - Include example code for seed node functionality
    - Add complete test coverage for new types and services

    ### Dependencies:
    - Add komodo_coin_updates dependency to komodo_defi_framework
    - Export seed node types from komodo_defi_types
    - Update pubspec.lock files across packages

    The seed node system ensures applications always have access to current
    seed nodes while maintaining backwards compatibility through fallback
    mechanisms. This addresses the upcoming requirement in v2.5.0-beta
    where seed nodes will be mandatory unless P2P is disabled.

    Breaking changes: None - all changes are additive with sensible defaults.

    * chore: PR cleanup

commit 2768355
Author: Charl (Nitride) <77973576+CharlVS@users.noreply.github.com>
Date:   Tue Jun 10 16:34:01 2025 +0200

    chore(android): downgrade Java compatibility from 21 to 17 (#79)

    Co-authored-by: DeckerSU <deckersu@protonmail.com>

commit f63bebb
Author: Francois <takenagain@users.noreply.github.com>
Date:   Fri Jun 6 13:46:17 2025 +0200

    fix(trezor,activation): add PrivateKeyPolicy to AuthOptions (#75)

    * fix(trezor,activation): add PrivateKeyPolicy to AuthOptions and beyond

    * refactor(types): replace `.from` with `.of`

    of has type annotations

    Co-authored-by: Charl (Nitride) <77973576+CharlVS@users.noreply.github.com>

    ---------

    Co-authored-by: Charl (Nitride) <77973576+CharlVS@users.noreply.github.com>

commit 040c040
Author: dragonhound <35845239+smk762@users.noreply.github.com>
Date:   Fri Jun 6 19:20:28 2025 +0800

    Update/ethscan v2 (#71)

    * update etherscan-proxy to v2

    * add distinct token url paths
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants