Skip to content

chore: add avm client to build process#111

Merged
cusma merged 3 commits intomainfrom
build/clients
Mar 2, 2026
Merged

chore: add avm client to build process#111
cusma merged 3 commits intomainfrom
build/clients

Conversation

@cusma
Copy link
Collaborator

@cusma cusma commented Mar 2, 2026

This PR addresses #93

@cusma cusma requested a review from Copilot March 2, 2026 09:42
@cusma cusma self-assigned this Mar 2, 2026
@cusma cusma added dependencies Pull requests that update a dependency file python Pull requests that update python code sdk labels Mar 2, 2026
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

Updates the smart contract build script to generate additional client bindings as part of the algokit project run build workflow, aligning the build artifacts with the repository’s generated SDK layout.

Changes:

  • Adds puyapy-clientgen invocation to generate Python AVM (on-chain) clients from ARC-56 app specs.
  • Adds logging around client generation steps.
  • Renames the generated AVM client file to a snake_case {contract}_avm_client.py naming convention.
Comments suppressed due to low confidence (1)

smart_contracts/main.py:172

  • The build script currently generates Python App clients and Python AVM clients, but it doesn't generate the TypeScript client described in issue #93. To fully address the issue, add a TypeScript client generation step (e.g., invoking algokit generate client for TypeScript) and decide where the TS output should be written as a build artifact.
        # Generate Python App Clients (off-chain)
        for file_name in app_spec_file_names:
            logger.info(f"Generating App Client (off-chain) for {file_name}")

            client_file = file_name
            print(file_name)
            generate_result = subprocess.run(
                [
                    "algokit",
                    "generate",
                    "client",
                    str(output_dir),
                    "--output",
                    str(_get_output_path(target_dir, deployment_extension)),
                ],

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

cusma and others added 2 commits March 2, 2026 10:54
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@cusma cusma requested a review from sofinico March 2, 2026 09:56
@cusma cusma merged commit 0d4ecaa into main Mar 2, 2026
4 checks passed
@cusma cusma deleted the build/clients branch March 2, 2026 17:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

dependencies Pull requests that update a dependency file python Pull requests that update python code sdk

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants