Random port on integrated devnet

README.md

@@ -343,8 +343,9 @@ More detailed documentation can be found [here](#account).
 
     const { res: currBalance } = await account.call(contract, "get_balance");
     const amount = BigInt(10);
-    // Passing max_fee is currently optional
-    await account.invoke(contract, "increase_balance", { amount }, { maxFee: BigInt("123") });
+
+    // Read more about max fee specification under # Funds and fees
+    await account.invoke(contract, "increase_balance", { amount }, { maxFee: BigInt("1000000000") });
 
     const { res: newBalance } = await account.call(contract, "get_balance");
     expect(newBalance).to.deep.equal(currBalance + amount);
@@ -610,7 +611,7 @@ npm install --save-dev influenceth__cairo_math_64x61@npm:@influenceth/cairo-math
 
 ```typescript
 paths: {
-    cairoPaths: ["./node_modules"]
+    cairoPaths: ["./node_modules"];
 }
 ```
 
@@ -690,11 +691,11 @@ await contract.invoke("increase_balance", { amount: 1 }, { wallet });
 
 Recompilation is performed when contracts are updated or when artifacts are missing. A file will be created with the name `cairo-files-cache.json` to handle caching. Recompilation is handled before the following [CLI commands](#cli-commands) are executed.
 
-- `npx hardhat starknet-deploy`
-- `npx hardhat starknet-invoke`
-- `npx hardhat starknet-call`
-- `npx hardhat run`
-- `npx hardhat test`
+-   `npx hardhat starknet-deploy`
+-   `npx hardhat starknet-invoke`
+-   `npx hardhat starknet-call`
+-   `npx hardhat run`
+-   `npx hardhat test`
 
 This feature is turned off by default and is specified in the `hardhat.config.ts` file.
 
@@ -784,6 +785,13 @@ Once your account has funds, you can specify a max fee greater than zero:
 await account.invoke(contract, "foo", { arg1: ... }, { maxFee: BigInt(...) });
 ```
 
+If you don't specify a `maxFee`, one will be calculated for you by applying an overhead of 50% to the result of fee estimation. You can also customize the overhead by providing a value for `overhead`:
+
+```typescript
+// maxFee will be 40% of estimated fee; if overhead not provided, a default value is used.
+await account.invoke(contract, "foo", { arg1: ... }, { overhead: 0.4 );
+```
+
 ### Multicalls
 
 You can also use the Account object to perform multi{calls, invokes, fee estimations}.

scripts/test.sh

@@ -9,7 +9,7 @@ CONFIG_FILE_NAME="hardhat.config.ts"
 
 # setup example repo
 rm -rf starknet-hardhat-example
-git clone -b plugin --single-branch [email protected]:Shard-Labs/starknet-hardhat-example.git
+git clone -b release-0.7.0 --single-branch [email protected]:Shard-Labs/starknet-hardhat-example.git
 cd starknet-hardhat-example
 git log -n 1
 npm install

src/account.ts

@@ -66,6 +66,23 @@ export abstract class Account {
         calldata?: StringMap,
         options?: InvokeOptions
     ): Promise<InvokeResponse> {
+        if (options?.maxFee && options?.overhead) {
+            const msg = "Both maxFee and overhead cannot be specified";
+            throw new StarknetPluginError(msg);
+        }
+
+        if (options?.maxFee === undefined || options?.maxFee === null) {
+            let overhead =
+                options?.overhead === undefined || options?.overhead === null
+                    ? 0.5
+                    : options?.overhead;
+            overhead = Math.round((1 + overhead) * 100);
+            const maxFee = await this.estimateFee(toContract, functionName, calldata, options);
+            options = {
+                ...options,
+                maxFee: (maxFee.amount * BigInt(overhead)) / BigInt(100)
+            };
+        }
         return (
             await this.interact(InteractChoice.INVOKE, toContract, functionName, calldata, options)
         ).toString();

src/external-server/docker-devnet.ts

@@ -18,6 +18,7 @@ export class DockerDevnet extends DockerServer {
     }
 
     protected async getContainerArgs(): Promise<string[]> {
-        return this.devnetArgs || [];
+        const containerArgs = this.devnetArgs || [];
+        return [...containerArgs, "--port", this.port];
     }
 }

src/external-server/external-server.ts

@@ -13,7 +13,7 @@ function sleep(amountMillis: number): Promise<void> {
     });
 }
 
-function isFreePort(port: number): Promise<boolean> {
+export async function isFreePort(port: number): Promise<boolean> {
     return new Promise((accept, reject) => {
         const sock = net.createConnection(port);
         sock.once("connect", () => {

src/task-actions.ts

@@ -26,7 +26,7 @@ import { getWalletUtil } from "./extend-utils";
 import { createIntegratedDevnet } from "./external-server";
 import { Recompiler } from "./recompiler";
 import { version } from "../package.json";
-import { getFreePort } from "./external-server/external-server";
+import { getFreePort, isFreePort } from "./external-server/external-server";
 
 function checkSourceExists(sourcePath: string): void {
     if (!fs.existsSync(sourcePath)) {
@@ -554,9 +554,14 @@ async function runWithDevnet(hre: HardhatRuntimeEnvironment, fn: () => Promise<u
         return;
     }
 
-    const port = await getFreePort();
-    const networkUrl = `http://127.0.0.1:${port}`;
-    hre.config.networks.integratedDevnet.url = networkUrl;
+    const { hostname, port } = new URL(hre.config.starknet.networkUrl);
+    const isPortFree = await isFreePort(parseInt(port));
+    if (!isPortFree) {
+        const newPort = await getFreePort();
+        const networkUrl = `http://${hostname}:${newPort}`;
+        hre.config.networks.integratedDevnet.url = networkUrl;
+        hre.config.starknet.networkUrl = networkUrl;
+    }
     const devnet = await createIntegratedDevnet(hre);
 
     await devnet.start();

src/types.ts

@@ -306,6 +306,7 @@ export interface InvokeOptions {
     wallet?: Wallet;
     nonce?: Numeric;
     maxFee?: Numeric;
+    overhead?: number;
 }
 
 export interface CallOptions {