Skip to content

Conversation

@lempira
Copy link
Collaborator

@lempira lempira commented Nov 24, 2025

Closes #296

Copilot AI review requested due to automatic review settings November 24, 2025 22:20
@lempira lempira requested a review from joe-p as a code owner November 24, 2025 22:20
Copy link
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 removes the algokit_utils and algokit_utils_ffi crates as part of issue #296. The changes involve deleting all test files and source code directories associated with these crates.

Key changes:

  • Removal of all test files for KMD, indexer, algod, clients, and applications functionality
  • Deletion of common test utilities including fixtures, helpers, and test account management
  • Removal of transaction-related source code including payment, asset operations, and key registration
  • Deletion of the main library module and application-related code

Reviewed changes

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

File Description
Multiple test files Removed test infrastructure for KMD, indexer, algod APIs, and application clients
Common test modules Deleted test fixtures, helpers, account management, and mnemonic utilities
Transaction modules Removed payment, asset freeze/transfer, key registration transaction implementations
Library source files Deleted main library module and application client/deployer/factory code

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

@joe-p
Copy link
Collaborator

joe-p commented Nov 25, 2025

There's probably some helpful stuff we eventually want pull from utils, like a utility function for getting default dispenser on localnet, but we can just pull that from the branch if/when needed

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.

Remove utils and utils-ffi

3 participants