Skip to content

Conversation

andreypfau
Copy link
Member

No description provided.

Signed-off-by: andreypfau <andreypfau@ton.org>
Signed-off-by: andreypfau <andreypfau@ton.org>
Signed-off-by: andreypfau <andreypfau@ton.org>
@andreypfau andreypfau requested a review from Copilot October 7, 2025 00:29
Copy link
Contributor

@Copilot 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 performs a comprehensive crypto refactoring by migrating from third-party crypto libraries to an internal org.ton.kotlin.crypto package. The changes consolidate cryptographic operations into a centralized crypto module and update all affected code to use the new interfaces.

Key changes:

  • Migration from third-party crypto dependencies to internal org.ton.kotlin.crypto package
  • Replacement of specific key types (PrivateKey, PublicKey) with generic crypto interfaces (Signer, SignatureVerifier)
  • Update of package names from org.ton.tl to org.ton.kotlin.tl throughout the TL serialization layer

Reviewed Changes

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

Show a summary per file
File Description
Multiple DhtValue*.kt files Updated crypto imports and replaced PrivateKey/PublicKey with Signer/SignatureVerifier
Multiple TL files Changed package from org.ton.tl to org.ton.kotlin.tl
Multiple LiteServer function files Updated TL package imports
build.gradle.kts files Updated Kotlin version and added source directory configurations
Various other files Updated crypto and TL package imports

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

andreypfau and others added 2 commits October 7, 2025 09:31
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@andreypfau andreypfau merged commit 1f790e0 into main Oct 7, 2025
@andreypfau andreypfau deleted the crypto-refactoring branch October 7, 2025 00:34
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