Add deploy script for HighloadWalletV3 contract · Pull Request #20 · TLMSLLC/ton-6950ea4e

Deleted user (ghost) · 2026-01-25T04:19:26Z

This script estimates the deploy amount for a HighloadWalletV3 contract and computes its address. It includes safety checks for deployment and allows for dynamic fee estimation.

Copilot

Pull request overview

This PR adds a deployment script for the HighloadWalletV3 contract that computes the contract address and estimates deployment costs. The script includes safety mechanisms to prevent accidental deployments and supports dynamic fee estimation with fallback options.

Changes:

Adds a TypeScript deployment script with configurable environment variables for contract deployment
Implements fee estimation logic with TonClient integration and fallback values
Includes safety checks requiring explicit CONFIRM_DEPLOY flag before any network operations

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

scripts/deploy.ts

Copilot · 2026-01-25T04:22:11Z

scripts/deploy.ts

+    // Fallback to default conservative amount
+    console.log('Using fallback deploy value:', DEPLOY_FALLBACK, 'TON');
+    const fallback = toNano(DEPLOY_FALLBACK);
+    const withMargin = BigInt(Math.ceil(Number(fallback) * DEPLOY_SAFE_MARGIN));


Converting bigint to number for multiplication can cause precision loss for large values. Since fallback is already a bigint from toNano(), consider using BigInt arithmetic throughout: const withMargin = (fallback * BigInt(Math.floor(DEPLOY_SAFE_MARGIN * 1000))) / 1000n to maintain precision.

Copilot · 2026-01-25T04:22:11Z

scripts/deploy.ts

+
+    // TODO: replace this placeholder with deriving the actual publicKey from your secret key
+    // For now we use a zeroed publicKey buffer as placeholder to compute address deterministically
+    const publicKeyPlaceholder = Buffer.alloc(32, 0);


Using a zeroed placeholder public key will result in computing an incorrect contract address. The address depends on the actual public key that will be used for deployment. This means the computed address will not match the actual deployed contract address if a different public key is used later, which could lead to funds being sent to the wrong address or deployment failures.

scripts/deploy.ts

Copilot · 2026-01-25T04:22:11Z

scripts/deploy.ts

+const DEPLOY_FALLBACK = process.env.DEPLOY_FALLBACK || '0.1';
+const CONFIRM_DEPLOY = process.env.CONFIRM_DEPLOY === 'true'; // must be explicitly set to "true" to send
+
+async function estimateDeployAmount(init: any): Promise<bigint> {


The function accepts 'any' type for the init parameter, which bypasses type safety. Consider defining a proper interface or type for the init parameter, such as: { code: Cell, data: Cell } to ensure type safety and better IDE support.

scripts/deploy.ts

fix for @ton/ton scripts Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

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

Copilot · 2026-01-25T04:37:04Z

@LucasL160 I've opened a new pull request, #24, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot · 2026-01-25T04:37:21Z

@LucasL160 I've opened a new pull request, #25, to work on those changes. Once the pull request is ready, I'll request review from you.

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

Deleted user (ghost) · 2026-01-25T04:38:21Z

Copilot open a new pull request to apply changes based on the comments in this thread

Copilot · 2026-01-25T04:38:29Z

@LucasL160 I've opened a new pull request, #26, to work on those changes. Once the pull request is ready, I'll request review from you.

Copilot

Pull request overview

Copilot reviewed 1 out of 1 changed files in this pull request and generated 6 comments.

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

Copilot · 2026-01-25T04:39:38Z

scripts/deploy.ts

+        console.warn('TonClient found but no known estimate method present. Falling back.');
+    } catch (e) {
+        console.warn(
+            'Could not use TonClient to estimate fees (skipped). Error:',


Precision loss when converting BigInt to Number. When converting fallback (a BigInt) to Number for multiplication, very large values could lose precision. The safer approach is to perform the multiplication in BigInt arithmetic directly:

const withMargin = (fallback * BigInt(Math.floor(DEPLOY_SAFE_MARGIN * 100))) / 100n;

This preserves BigInt precision throughout the calculation.

Copilot · 2026-01-25T04:39:38Z

scripts/deploy.ts

+
+const SUBWALLET_ID = Number(process.env.SUBWALLET_ID || HIGHLOAD_WALLET_V3_DEFAULT_SUBWALLET_ID);
+const TIMEOUT = Number(process.env.TIMEOUT || 3600);
+const DEPLOY_VALUE_ENV = process.env.DEPLOY_VALUE || '';


The 'any' type for the 'init' parameter reduces type safety. Consider defining a proper interface or using a type from '@ton/core' library, such as:

async function estimateDeployAmount(init: { code: Cell; data: Cell }): Promise<bigint>

This provides better type checking and IDE support.

Copilot · 2026-01-25T04:39:38Z

scripts/deploy.ts

+        // dynamic require to avoid TypeScript type errors if library evolves
+        // eslint-disable-next-line @typescript-eslint/no-var-requires
+        const { TonClient } = require('ton');
+        const client = new TonClient({ endpoint: RPC_URL });
+


Using require() for dynamic import breaks the ESM module pattern used in the rest of the file. Consider using dynamic import() instead for consistency:

const { TonClient } = await import('ton');

This maintains ESM consistency and is more compatible with modern TypeScript/Node.js practices.

Suggested change

// dynamic require to avoid TypeScript type errors if library evolves

// eslint-disable-next-line @typescript-eslint/no-var-requires

const { TonClient } = require('ton');

const client = new TonClient({ endpoint: RPC_URL });

// dynamic import to avoid TypeScript type errors if library evolves

const { TonClient } = await import('ton');

const client = new TonClient({ endpoint: RPC_URL });

Copilot · 2026-01-25T04:39:38Z

scripts/deploy.ts

+const SECRET_KEY_BASE64 = process.env.SECRET_KEY_BASE64 || '';
+const WORKCHAIN = Number(process.env.WORKCHAIN || 0);
+
+// Default subwallet id for HighloadWalletV3 as per the HighloadWalletV3 specification.


No validation for TIMEOUT value. The Number() conversion could result in NaN if the environment variable is not a valid number, which would cause issues when creating the config cell. Consider adding validation:

const timeout = Number(process.env.TIMEOUT || 3600); if (isNaN(timeout)) { throw new Error('TIMEOUT must be a valid number'); } const TIMEOUT = timeout;

Copilot · 2026-01-25T04:39:39Z

scripts/deploy.ts

+        subwalletId: SUBWALLET_ID,
+        timeout: TIMEOUT,
+    });
+


Precision loss when converting BigInt to Number for display. For very large values (near the Number.MAX_SAFE_INTEGER limit), this conversion could lose precision. Consider using a string-based division or a dedicated BigInt formatting function for accurate display.

Copilot · 2026-01-25T04:39:39Z

scripts/deploy.ts

+
+// Default subwallet id for HighloadWalletV3 as per the HighloadWalletV3 specification.
+// Can be overridden via the SUBWALLET_ID environment variable if needed.
+const HIGHLOAD_WALLET_V3_DEFAULT_SUBWALLET_ID = 0x10ad;


No validation for DEPLOY_SAFE_MARGIN value. If the environment variable contains a non-numeric value or negative number, Number() will return NaN or a negative value, which could cause unexpected behavior in calculations. Consider adding validation:

const margin = Number(process.env.DEPLOY_SAFE_MARGIN || '1.5'); if (isNaN(margin) || margin < 0) { throw new Error('DEPLOY_SAFE_MARGIN must be a non-negative number'); } const DEPLOY_SAFE_MARGIN = margin;

Add deploy script for HighloadWalletV3 contract

48d0c19

This script estimates the deploy amount for a HighloadWalletV3 contract and computes its address. It includes safety checks for deployment and allows for dynamic fee estimation.

Copilot AI review requested due to automatic review settings January 25, 2026 04:19

Copilot started reviewing on behalf of ghost January 25, 2026 04:19 View session

Copilot AI reviewed Jan 25, 2026

View reviewed changes

Update scripts/deploy.ts

35fc0fa

fix for @ton/ton scripts Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>

Copilot AI review requested due to automatic review settings January 25, 2026 04:36

Copilot started reviewing on behalf of ghost January 25, 2026 04:36 View session

Update scripts/deploy.ts

2289c8f

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

This was referenced Jan 25, 2026

[WIP] WIP: Address feedback on deploy script for HighloadWalletV3 contract #24

Draft

[WIP] WIP Address feedback on deploy script for HighloadWalletV3 contract #25

Open

Update scripts/deploy.ts

f8011ad

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

Copilot AI mentioned this pull request Jan 25, 2026

[WIP] Add deploy script for HighloadWalletV3 contract #26

Merged

Copilot AI reviewed Jan 25, 2026

View reviewed changes

ghost merged commit 69b4fde into master Jan 25, 2026
5 checks passed

This pull request was closed.

Conversation

Deleted user (ghost) commented Jan 25, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Copilot AI Jan 25, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 25, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Copilot AI Jan 25, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Copilot AI commented Jan 25, 2026

Uh oh!

Copilot AI commented Jan 25, 2026

Uh oh!

Deleted user (ghost) commented Jan 25, 2026

Uh oh!

Copilot AI commented Jan 25, 2026

Uh oh!

Copilot AI left a comment

Choose a reason for hiding this comment

Pull request overview

Uh oh!

Copilot AI Jan 25, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 25, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 25, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 25, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 25, 2026

Choose a reason for hiding this comment

Uh oh!

Copilot AI Jan 25, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants