Merge pull request #159 from ElrondNetwork/better-interactions

Breaking changes (erdjs 10): improve contract interactions and interpretation of contract results

Author: andreibancioiu

.github/workflows/erdjs-publish-alpha-beta.yml

@@ -0,0 +1,29 @@
+name: Publish erdjs (alpha / beta)
+
+on:
+  workflow_dispatch:
+    inputs:
+      channel:
+        type: choice
+        description: NPM channel
+        options: 
+        - alpha
+        - beta
+
+jobs:
+  publish-npm:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+      - uses: actions/setup-node@v1
+        with:
+          node-version: 12
+          registry-url: https://registry.npmjs.org/
+      
+      - run: npm ci
+      - run: npm test
+      
+      - name: Publish to npmjs
+        env:
+          NODE_AUTH_TOKEN: ${{secrets.NPM_TOKEN}}
+        run: npm publish --access=public --tag=${{ github.event.inputs.channel }}

CHANGELOG.md

@@ -7,6 +7,36 @@ Check [Keep a Changelog](http://keepachangelog.com/) for recommendations on how
 ## Unreleased
  - TBD
 
+## [10.0.0]
+ - [Breaking changes: improve contract interactions and interpretation of contract results](https://github.com/ElrondNetwork/elrond-sdk-erdjs/pull/159)
+ 
+**Breaking changes**
+
+ - `ExecutionResultsBundle` and `QueryResponseBundle` have been removed, and replaced by `TypedOutcomeBundle` (and its untyped counterpart, `UntypedOutcomeBundle`).
+ - `SmartContractResults` has been changed to not use the concepts `immediate result` and `resulting calls` anymore. Instead, interpreting `SmartContractResults.items` is now the responsibility of the `ResultsParser` (on which the contract controllers depend).
+ - Redesigned `QueryResponse`, changed most of its public interface. Results interpretation is now the responsibility of the results parser, called by the smart contract controllers.
+ - `interpretQueryResponse()` and `interpretExecutionResults()` do not exist on the `Interaction` object anymore. Now, querying / executing an interaction against the controller will return the interpreted results.
+ - `TokenIdentifierValue` is constructed using a `string`, not a `buffer`. Its `valueOf()` is now a string, as well.
+ - The `Interaction` constructor does not receive the `interpretingFunction` parameter anymore.
+ - `Interaction.getInterpretingFunction()` and `Interaction.getExecutingFunction()` have been removed, replaced by `Interaction.getFunction()`.
+ - `DefaultInteractionRunner` has been removed, and replaced by **smart contract controllers**.
+ - `StrictChecker` has been renamed to `InteractionChecker`. It's public interface - the function `checkInteraction()` - has changed as well (it also requires the endpoint definition now, as a second parameter).
+ - The functions `getReceipt()`, `getSmartContractResults()` and `getLogs()` of `TransactionOnNetwork` have been removed. The underlying properties are now public.
+ - Renamed `OptionValue.newMissingType()` to `OptionValue.newMissingTyped()`
+ - Queries with a return code different than `Ok` do not automatically throw an exception anymore (`assertSuccess()` has to be called explicitly in order to throw).
+ 
+**Other changes**
+
+ - `SmartContract`, in addition to `methods`, now also has a `methodAuto` object that allows one to create interactions without explicitly specifying the types of the arguments. Automatic type inference (within erdjs' typesystem) is leveraged. The type inference system was implemented in the past, in the `nativeSerializer` component - PR https://github.com/ElrondNetwork/elrond-sdk-erdjs/pull/9 by @claudiu725.
+ - Added utility function `getFieldValue()` on `Struct` and `EnumValue`.
+ - Refactoring in the `networkProvider` package (under development, in order to merge the provider interfaces under a single one)
+ - Added utility function `Interaction.useThenIncrementNonceOf()`
+ - Fixed `nativeSerializer` to not depend on `SmartContract`, `ContractWrapper` and `TestWallet` anymore (gathered under an interface).
+ - Replaced the old `lottery-egld` with the new `lottery-esdt` in integration tests.
+ - Added missing tests for some components: `nativeSerializer`, `struct`, `enum`.
+ - Added utility function `OptionalValue.newMissing()`. Added "fake" covariance wrt. "null type parameter" (when value is missing) on `OptionalType`.
+ - Added utility functions (typed value factories): `createListOfAddresses`, `createListOfTokenIdentifiers`.
+
 ## [9.2.3]
  - [Fix log level in transaction watcher.](https://github.com/ElrondNetwork/elrond-sdk-erdjs/pull/160)
 

README.md

@@ -18,17 +18,17 @@ Elrond SDK for JavaScript and TypeScript (written in TypeScript).
 
 The most comprehensive usage examples are captured within the unit and the integration tests. Specifically, in the `*.spec.ts` files of the source code. For example:
 
- - [transaction.dev.net.spec.ts](https://github.com/ElrondNetwork/elrond-sdk-erdjs/tree/main/src/transaction.dev.net.spec.ts)
+ - [transaction.local.net.spec.ts](https://github.com/ElrondNetwork/elrond-sdk-erdjs/tree/main/src/transaction.local.net.spec.ts)
  - [address.spec.ts](https://github.com/ElrondNetwork/elrond-sdk-erdjs/tree/main/src/address.spec.ts)
  - [transactionPayloadBuilders.spec.ts](https://github.com/ElrondNetwork/elrond-sdk-erdjs/tree/main/src/smartcontracts/transactionPayloadBuilders.spec.ts)
  - [smartContract.spec.ts](https://github.com/ElrondNetwork/elrond-sdk-erdjs/tree/main/src/smartcontracts/smartContract.spec.ts)
- - [smartContract.dev.net.spec.ts](https://github.com/ElrondNetwork/elrond-sdk-erdjs/tree/main/src/smartcontracts/smartContract.dev.net.spec.ts)
+ - [smartContract.local.net.spec.ts](https://github.com/ElrondNetwork/elrond-sdk-erdjs/tree/main/src/smartcontracts/smartContract.local.net.spec.ts)
  - [query.spec.ts](https://github.com/ElrondNetwork/elrond-sdk-erdjs/tree/main/src/smartcontracts/query.spec.ts)
  - [query.main.net.spec.ts](https://github.com/ElrondNetwork/elrond-sdk-erdjs/tree/main/src/smartcontracts/query.main.net.spec.ts)
 
 For advanced smart contract interaction, using ABIs, please see the following test files:
 
- - [interaction.dev.net.spec.ts](https://github.com/ElrondNetwork/elrond-sdk-erdjs/tree/main/src/smartcontracts/interaction.dev.net.spec.ts) 
+ - [interaction.local.net.spec.ts](https://github.com/ElrondNetwork/elrond-sdk-erdjs/tree/main/src/smartcontracts/interaction.local.net.spec.ts) 
  - [abiRegistry.spec.ts](https://github.com/ElrondNetwork/elrond-sdk-erdjs/tree/main/src/smartcontracts/typesystem/abiRegistry.spec.ts)
  - [argSerializer.spec.ts](https://github.com/ElrondNetwork/elrond-sdk-erdjs/tree/main/src/smartcontracts/argSerializer.spec.ts) 
 

package-lock.json

@@ -1,6 +1,6 @@
 {
   "name": "@elrondnetwork/erdjs",
-  "version": "9.2.3",
+  "version": "10.0.0-beta.1",
   "description": "Smart Contracts interaction framework",
   "main": "out/index.js",
   "types": "out/index.d.js",

package.json

@@ -409,6 +409,33 @@ export class ErrTypingSystem extends Err {
   }
 }
 
+/**
+ * Signals a missing field on a struct.
+ */
+ export class ErrMissingFieldOnStruct extends Err {
+  public constructor(fieldName: string, structName: string) {
+    super(`field ${fieldName} does not exist on struct ${structName}`);
+  }
+}
+
+/**
+ * Signals a missing field on an enum.
+ */
+ export class ErrMissingFieldOnEnum extends Err {
+  public constructor(fieldName: string, enumName: string) {
+    super(`field ${fieldName} does not exist on enum ${enumName}`);
+  }
+}
+
+/**
+ * Signals an error when parsing the contract results.
+ */
+ export class ErrCannotParseContractResults extends Err {
+  public constructor(details: string) {
+    super(`cannot parse contract results: ${details}`);
+  }
+}
+
 /**
  * Signals a generic codec (encode / decode) error.
  */

src/errors.ts

@@ -5,7 +5,7 @@ import { Hash } from "../hash";
 import { IContractQueryResponse, IContractResultItem, IContractResults } from "./interface";
 import { GasLimit, GasPrice } from "../networkParams";
 import { Nonce } from "../nonce";
-import { ArgSerializer, EndpointDefinition, MaxUint64, ReturnCode, TypedValue } from "../smartcontracts";
+import { MaxUint64, ReturnCode } from "../smartcontracts";
 import { TransactionHash } from "../transaction";
 
 export class ContractResults implements IContractResults {
@@ -80,16 +80,6 @@ export class ContractResultItem implements IContractResultItem {
 
         return item;
     }
-
-    getOutputUntyped(): Buffer[] {
-        // TODO: Decide how to parse "data" (immediate results vs. other results).
-        throw new Error("Method not implemented.");
-    }
-
-    getOutputTyped(_endpointDefinition: EndpointDefinition): TypedValue[] {
-        // TODO: Decide how to parse "data" (immediate results vs. other results).
-        throw new Error("Method not implemented.");
-    }
 }
 
 export class ContractQueryResponse implements IContractQueryResponse {
@@ -110,14 +100,7 @@ export class ContractQueryResponse implements IContractQueryResponse {
         return response;
     }
 
-    getOutputUntyped(): Buffer[] {
-        let buffers = this.returnData.map((item) => Buffer.from(item || "", "base64"));
-        return buffers;
-    }
-
-    getOutputTyped(endpointDefinition: EndpointDefinition): TypedValue[] {
-        let buffers = this.getOutputUntyped();
-        let values = new ArgSerializer().buffersToValues(buffers, endpointDefinition!.output);
-        return values;
+    getReturnDataParts(): Buffer[] {
+        return this.returnData.map((item) => Buffer.from(item || ""));
     }
 }

src/networkProvider/contractResults.ts

@@ -9,7 +9,7 @@ import { NetworkStake } from "../networkStake";
 import { NetworkStatus } from "../networkStatus";
 import { Nonce } from "../nonce";
 import { Signature } from "../signature";
-import { EndpointDefinition, Query, ReturnCode, TypedValue } from "../smartcontracts";
+import { Query, ReturnCode } from "../smartcontracts";
 import { Stats } from "../stats";
 import { Transaction, TransactionHash, TransactionStatus } from "../transaction";
 import { TransactionLogs } from "../transactionLogs";
@@ -204,19 +204,15 @@ export interface IContractResultItem {
     gasPrice: GasPrice;
     callType: number;
     returnMessage: string;
-
-    getOutputUntyped(): Buffer[];
-    getOutputTyped(endpointDefinition: EndpointDefinition): TypedValue[];
 }
 
 export interface IContractQueryResponse {
     returnData: string[];
     returnCode: ReturnCode;
     returnMessage: string;
     gasUsed: GasLimit;
-
-    getOutputUntyped(): Buffer[];
-    getOutputTyped(endpointDefinition: EndpointDefinition): TypedValue[];
+    
+    getReturnDataParts(): Buffer[];
 }
 
 export interface IContractSimulation {

src/networkProvider/interface.ts

@@ -227,7 +227,7 @@ describe("test network providers on devnet: Proxy and API", function () {
         let proxyResponse = await proxyProvider.queryContract(query);
 
         assert.deepEqual(apiResponse, proxyResponse);
-        assert.deepEqual(apiResponse.getOutputUntyped(), proxyResponse.getOutputUntyped());
+        assert.deepEqual(apiResponse.getReturnDataParts(), proxyResponse.getReturnDataParts());
 
         // Query: increment counter
         query = new Query({
@@ -240,7 +240,7 @@ describe("test network providers on devnet: Proxy and API", function () {
         proxyResponse = await proxyProvider.queryContract(query);
 
         assert.deepEqual(apiResponse, proxyResponse);
-        assert.deepEqual(apiResponse.getOutputUntyped(), proxyResponse.getOutputUntyped());
+        assert.deepEqual(apiResponse.getReturnDataParts(), proxyResponse.getReturnDataParts());
 
         // Query: issue ESDT
         query = new Query({
@@ -260,6 +260,6 @@ describe("test network providers on devnet: Proxy and API", function () {
         proxyResponse = await proxyProvider.queryContract(query);
 
         assert.deepEqual(apiResponse, proxyResponse);
-        assert.deepEqual(apiResponse.getOutputUntyped(), proxyResponse.getOutputUntyped());
+        assert.deepEqual(apiResponse.getReturnDataParts(), proxyResponse.getReturnDataParts());
     });
 });

src/networkProvider/providers.dev.net.spec.ts

@@ -70,7 +70,7 @@ export class ProxyProvider implements IProvider {
             return this.doPostGeneric("vm-values/query", data, (response) =>
                 QueryResponse.fromHttpResponse(response.data || response.vmOutput)
             );
-        } catch (err) {
+        } catch (err: any) {
             throw errors.ErrContractQuery.increaseSpecificity(err);
         }
     }