starknet 11.0.2 → 11.0.3

package/CHANGELOG.md

@@ -1,3 +1,9 @@
+## [11.0.3](https://github.com/starknet-io/starknet.js/compare/v11.0.2...v11.0.3) (2026-10-01)
+
+### Bug Fixes
+
+- **calldata:** let named arguments carry built Cairo types ([b05b91f](https://github.com/starknet-io/starknet.js/commit/b05b91f59d7795afcee3f52dbcbf98aab9aeb55d))
+
 ## [11.0.2](https://github.com/starknet-io/starknet.js/compare/v11.0.1...v11.0.2) (2026-09-22)
 
 ### Bug Fixes

package/dist/index.d.ts

@@ -16470,7 +16470,13 @@ declare class CairoTypeOption {
      * Build an option, from a value a caller passed or from the felts of a response.
      *
      * A {@link CairoOption} says its own branch, so `variant` is not given with one — nor with the
-     * response iterator, which reads the branch off the wire. Anywhere else it is required.
+     * response iterator, which reads the branch off the wire, nor with a `CairoTypeOption`, which
+     * is copied as it is. Anywhere else it is required.
+     *
+     * For the same reason, a `CairoOption` that holds a `CairoTypeOption` is refused: once it is
+     * unwrapped, the variant comes with the `CairoTypeOption`. Accepted, it would lose one level:
+     * `Some(Some(1))` would be sent as `Some(None)`. Write a nested value with `CairoOption`
+     * only, or build one `CairoTypeOption` for the whole type.
      *
      * `subType` is what makes an option of options work. Handed a `CairoOption`, this constructor
      * unwraps it and calls itself with what was inside, which for a nested option is another
@@ -16481,11 +16487,11 @@ declare class CairoTypeOption {
      * @param {string} optionCairoType the abi type, `core::option::Option::<T>`
      * @param {AllowArray<CairoTypeStrategy>} parsingStrategy how to build the value
      * @param {CairoOptionVariant | number} [variant] which branch, when the content does not say.
-     * Must be omitted when `content` is the response iterator.
+     * Must be omitted when `content` is the response iterator or a `CairoTypeOption`.
      * @param {boolean} [subType=false] true when called from the unwrapping of a nested
      * `CairoOption`, which is the only caller that should set it
-     * @throws {Error} when the type is not an option, when the variant is missing, absurd, or
-     * contradicts the content
+     * @throws {Error} when the type is not an option, or when the variant is missing, absurd,
+     * contradicts the content, or is given with a `CairoTypeOption`
      * @example
      * ```typescript
      * const type = 'core::option::Option::<core::integer::u8>';

package/dist/index.global.js

@@ -8167,7 +8167,6 @@ ${indent}}` : "}";
      * ```
      */
     low;
-    // TODO should be u128
     /**
      * The top 128 bits of the value.
      * @example
@@ -8177,7 +8176,6 @@ ${indent}}` : "}";
      * ```
      */
     high;
-    // TODO should be u128
     /**
      * The abi type this class serializes.
      * @example
@@ -8416,7 +8414,6 @@ ${indent}}` : "}";
      * ```
      */
     limb0;
-    // TODO should be u128
     /**
      * Bits 128 to 255 of the value.
      * @example
@@ -8426,7 +8423,6 @@ ${indent}}` : "}";
      * ```
      */
     limb1;
-    // TODO should be u128
     /**
      * Bits 256 to 383 of the value.
      * @example
@@ -8436,7 +8432,6 @@ ${indent}}` : "}";
      * ```
      */
     limb2;
-    // TODO should be u128
     /**
      * Bits 384 to 511 of the value.
      * @example
@@ -8446,7 +8441,6 @@ ${indent}}` : "}";
      * ```
      */
     limb3;
-    // TODO should be u128
     /**
      * The abi type this class serializes.
      * @example
@@ -13701,7 +13695,13 @@ ${indent}}` : "}";
      * Build an option, from a value a caller passed or from the felts of a response.
      *
      * A {@link CairoOption} says its own branch, so `variant` is not given with one — nor with the
-     * response iterator, which reads the branch off the wire. Anywhere else it is required.
+     * response iterator, which reads the branch off the wire, nor with a `CairoTypeOption`, which
+     * is copied as it is. Anywhere else it is required.
+     *
+     * For the same reason, a `CairoOption` that holds a `CairoTypeOption` is refused: once it is
+     * unwrapped, the variant comes with the `CairoTypeOption`. Accepted, it would lose one level:
+     * `Some(Some(1))` would be sent as `Some(None)`. Write a nested value with `CairoOption`
+     * only, or build one `CairoTypeOption` for the whole type.
      *
      * `subType` is what makes an option of options work. Handed a `CairoOption`, this constructor
      * unwraps it and calls itself with what was inside, which for a nested option is another
@@ -13712,11 +13712,11 @@ ${indent}}` : "}";
      * @param {string} optionCairoType the abi type, `core::option::Option::<T>`
      * @param {AllowArray<CairoTypeStrategy>} parsingStrategy how to build the value
      * @param {CairoOptionVariant | number} [variant] which branch, when the content does not say.
-     * Must be omitted when `content` is the response iterator.
+     * Must be omitted when `content` is the response iterator or a `CairoTypeOption`.
      * @param {boolean} [subType=false] true when called from the unwrapping of a nested
      * `CairoOption`, which is the only caller that should set it
-     * @throws {Error} when the type is not an option, when the variant is missing, absurd, or
-     * contradicts the content
+     * @throws {Error} when the type is not an option, or when the variant is missing, absurd,
+     * contradicts the content, or is given with a `CairoTypeOption`
      * @example
      * ```typescript
      * const type = 'core::option::Option::<core::integer::u8>';
@@ -13761,6 +13761,10 @@ ${indent}}` : "}";
         return;
       }
       if (content instanceof _CairoTypeOption) {
+        assert(
+          isUndefined(variant),
+          'when "content" parameter is a CairoTypeOption, do not define "variant" parameter.'
+        );
         this.content = content.content;
         this.isVariantSome = content.isVariantSome;
         this.optionCairoType = content.optionCairoType;
@@ -16222,11 +16226,17 @@ ${indent}}` : "}";
       `Your object includes the property : ${key}, containing an Uint512 object without the 'limb0' to 'limb3' keys.`
     );
   }
+  function getVariantType(abiEnum, variantName) {
+    return abiEnum.variants?.find((abiVariant) => abiVariant.name === variantName)?.type ?? "";
+  }
   function orderPropsByAbi(unorderedObject, abiOfObject, structs, enums) {
     const orderInput = (unorderedItem, abiType) => {
       if (CairoFixedArray.isTypeFixedArray(abiType)) {
         return orderFixedArray(unorderedItem, abiType);
       }
+      if (isCairoType(unorderedItem)) {
+        return unorderedItem;
+      }
       if (isTypeArray(abiType)) {
         return orderArray(unorderedItem, abiType);
       }
@@ -16339,14 +16349,8 @@ ${indent}}` : "}";
     const orderEnum = (unorderedObject2, abiObject) => {
       if (isTypeResult(abiObject.name)) {
         const unorderedResult = unorderedObject2;
-        const resultOkType = abiObject.name.substring(
-          abiObject.name.indexOf("<") + 1,
-          abiObject.name.lastIndexOf(",")
-        );
-        const resultErrType = abiObject.name.substring(
-          abiObject.name.indexOf(",") + 1,
-          abiObject.name.lastIndexOf(">")
-        );
+        const resultOkType = getVariantType(abiObject, "Ok");
+        const resultErrType = getVariantType(abiObject, "Err");
         if (unorderedResult.isOk()) {
           return new CairoResult(
             CairoResultVariant.Ok,
@@ -16360,10 +16364,7 @@ ${indent}}` : "}";
       }
       if (isTypeOption(abiObject.name)) {
         const unorderedOption = unorderedObject2;
-        const resultSomeType = abiObject.name.substring(
-          abiObject.name.indexOf("<") + 1,
-          abiObject.name.lastIndexOf(">")
-        );
+        const resultSomeType = getVariantType(abiObject, "Some");
         if (unorderedOption.isSome()) {
           return new CairoOption(
             CairoOptionVariant.Some,
@@ -16378,10 +16379,7 @@ ${indent}}` : "}";
         if (isUndefined(variant[1])) {
           return variant;
         }
-        const variantType = abiObject.type.substring(
-          abiObject.type.lastIndexOf("<") + 1,
-          abiObject.type.lastIndexOf(">")
-        );
+        const variantType = getVariantType(abiObject, variant[0]);
         if (variantType === "()") {
           return variant;
         }