@caravan/psbt 1.0.1 → 1.1.0

package/dist/index.js

@@ -235,7 +235,10 @@ var PsbtV2 = class _PsbtV2 extends PsbtV2Maps {
    * Globals Getters/Setters
    */
   get PSBT_GLOBAL_XPUB() {
-    return getNonUniqueKeyTypeValues(this.globalMap, "01" /* PSBT_GLOBAL_XPUB */);
+    return getNonUniqueKeyTypeValues(
+      this.globalMap,
+      "01" /* PSBT_GLOBAL_XPUB */
+    );
   }
   get PSBT_GLOBAL_TX_VERSION() {
     const val = this.globalMap.get("02" /* PSBT_GLOBAL_TX_VERSION */);
@@ -409,16 +412,28 @@ var PsbtV2 = class _PsbtV2 extends PsbtV2Maps {
     );
   }
   get PSBT_IN_RIPEMD160() {
-    return getNonUniqueKeyTypeValues(this.inputMaps, "0a" /* PSBT_IN_RIPEMD160 */);
+    return getNonUniqueKeyTypeValues(
+      this.inputMaps,
+      "0a" /* PSBT_IN_RIPEMD160 */
+    );
   }
   get PSBT_IN_SHA256() {
-    return getNonUniqueKeyTypeValues(this.inputMaps, "0b" /* PSBT_IN_SHA256 */);
+    return getNonUniqueKeyTypeValues(
+      this.inputMaps,
+      "0b" /* PSBT_IN_SHA256 */
+    );
   }
   get PSBT_IN_HASH160() {
-    return getNonUniqueKeyTypeValues(this.inputMaps, "0c" /* PSBT_IN_HASH160 */);
+    return getNonUniqueKeyTypeValues(
+      this.inputMaps,
+      "0c" /* PSBT_IN_HASH160 */
+    );
   }
   get PSBT_IN_HASH256() {
-    return getNonUniqueKeyTypeValues(this.inputMaps, "0d" /* PSBT_IN_HASH256 */);
+    return getNonUniqueKeyTypeValues(
+      this.inputMaps,
+      "0d" /* PSBT_IN_HASH256 */
+    );
   }
   get PSBT_IN_PREVIOUS_TXID() {
     const indices = [];
@@ -570,6 +585,106 @@ var PsbtV2 = class _PsbtV2 extends PsbtV2Maps {
       "fc" /* PSBT_OUT_PROPRIETARY */
     );
   }
+  /**
+   * Operator Role Validation Getters
+   */
+  /**
+   * Returns true if the PsbtV2 is ready for an operator taking the Constructor
+   * role.
+   *
+   * This check assumes that the Creator used this class's constructor method to
+   * initialize the PsbtV2 without passing a psbt (constructor  defaults were
+   * set).
+   */
+  get isReadyForConstructor() {
+    if (this.PSBT_GLOBAL_FALLBACK_LOCKTIME === null) {
+      return false;
+    }
+    if (!this.isModifiable(["INPUTS" /* INPUTS */]) && !this.isModifiable(["OUTPUTS" /* OUTPUTS */])) {
+      return false;
+    }
+    return true;
+  }
+  /**
+   * Returns true if the PsbtV2 is ready for an operator taking the Updater
+   * role.
+   *
+   * Before signatures are added, but after an input is added, a PsbtV2 is
+   * likely to be ready for Constructor, ready for Updater, and ready for Signer
+   * simultaneously.
+   *
+   * According to BIP370, the Updater can modify the sequence number, but it is
+   * unclear if the Updater retains permissions provided in psbtv0 (BIP174). It
+   * is likely not the case that the Updater has the same permissions as
+   * previously because it seems to now be the realm of the Constructor to add
+   * inputs and outputs.
+   */
+  get isReadyForUpdater() {
+    if (this.PSBT_GLOBAL_INPUT_COUNT === 0) {
+      return false;
+    }
+    if (!this.isModifiable(["INPUTS" /* INPUTS */])) {
+      return false;
+    }
+    return true;
+  }
+  /**
+   * Returns true if the PsbtV2 is ready for an operator taking the Signer role.
+   */
+  get isReadyForSigner() {
+    if (this.PSBT_GLOBAL_INPUT_COUNT === 0) {
+      return false;
+    }
+    if (this.isReadyForTransactionExtractor) {
+      return false;
+    }
+    return true;
+  }
+  /**
+   * Returns true if the PsbtV2 is ready for an operator taking the Combiner
+   * role.
+   */
+  get isReadyForCombiner() {
+    return this.isReadyForConstructor || this.isReadyForUpdater || this.isReadyForSigner;
+  }
+  /**
+   * Unimplemented. Returns false.
+   */
+  get isReadyForInputFinalizer() {
+    console.warn(
+      "PsbtV2.isReadyForInputFinalizer has been called, however, this getter is unimplemented and shouldn't be used."
+    );
+    return false;
+  }
+  /**
+   * Returns true if the PsbtV2 is ready for an operator taking the Transaction
+   * Extractor role.
+   *
+   * If all the inputs have been finalized, then the psbt is ready for the
+   * Transaction Extractor. According to BIP 174, it's the responsibility of the
+   * Input Finalizer to add scriptSigs or scriptWitnesses and then remove other
+   * details besides the UTXO. This getter checks that the Input Finalizer has
+   * finished its job.
+   */
+  get isReadyForTransactionExtractor() {
+    for (let i = 0; i < this.PSBT_GLOBAL_INPUT_COUNT; i++) {
+      if (!this.PSBT_IN_FINAL_SCRIPTSIG[i] && !this.PSBT_IN_FINAL_SCRIPTWITNESS[i]) {
+        return false;
+      }
+      if (this.PSBT_IN_FINAL_SCRIPTSIG[i] && !this.PSBT_IN_NON_WITNESS_UTXO[i] || this.PSBT_IN_FINAL_SCRIPTWITNESS[i] && !this.PSBT_IN_WITNESS_UTXO[i]) {
+        return false;
+      }
+      if (
+        // Strings
+        this.PSBT_IN_REDEEM_SCRIPT[i] || this.PSBT_IN_WITNESS_SCRIPT[i] || this.PSBT_IN_POR_COMMITMENT[i] || this.PSBT_IN_TAP_KEY_SIG[i] || this.PSBT_IN_TAP_INTERNAL_KEY[i] || this.PSBT_IN_TAP_MERKLE_ROOT[i] || // Numbers
+        this.PSBT_IN_SIGHASH_TYPE[i] !== null || this.PSBT_IN_SEQUENCE[i] !== null || this.PSBT_IN_REQUIRED_TIME_LOCKTIME[i] !== null || this.PSBT_IN_REQUIRED_HEIGHT_LOCKTIME[i] !== null || // Arrays of non-unique keytype values
+        this.PSBT_IN_PARTIAL_SIG[i].filter((el) => el !== null).length > 0 || this.PSBT_IN_BIP32_DERIVATION[i].filter((el) => el !== null).length > 0 || this.PSBT_IN_RIPEMD160[i].filter((el) => el !== null).length > 0 || this.PSBT_IN_SHA256[i].filter((el) => el !== null).length > 0 || this.PSBT_IN_HASH160[i].filter((el) => el !== null).length > 0 || this.PSBT_IN_HASH256[i].filter((el) => el !== null).length > 0 || this.PSBT_IN_TAP_SCRIPT_SIG[i].filter((el) => el !== null).length > 0 || this.PSBT_IN_TAP_LEAF_SCRIPT[i].filter((el) => el !== null).length > 0 || this.PSBT_IN_TAP_BIP32_DERIVATION[i].filter((el) => el !== null).length > 0
+      ) {
+        return false;
+      }
+    }
+    return true;
+  }
   /**
    * Other Getters/Setters
    */
@@ -615,6 +730,10 @@ var PsbtV2 = class _PsbtV2 extends PsbtV2Maps {
     this.PSBT_GLOBAL_INPUT_COUNT = 0;
     this.PSBT_GLOBAL_OUTPUT_COUNT = 0;
     this.PSBT_GLOBAL_FALLBACK_LOCKTIME = 0;
+    this.PSBT_GLOBAL_TX_MODIFIABLE = [
+      "INPUTS" /* INPUTS */,
+      "OUTPUTS" /* OUTPUTS */
+    ];
   }
   /**
    * Checks initial construction of any valid PsbtV2. It is called when a psbt
@@ -672,6 +791,11 @@ var PsbtV2 = class _PsbtV2 extends PsbtV2Maps {
    * defined for PsbtV2.
    */
   dangerouslySetGlobalTxVersion1() {
+    if (!this.isReadyForConstructor) {
+      throw Error(
+        "The PsbtV2 is not ready for a Constructor. The PSBT_GLOBAL_TX_VERSION should not be forced to version 1."
+      );
+    }
     console.warn("Dangerously setting PsbtV2.PSBT_GLOBAL_TX_VERSION to 1!");
     const bw = new import_bufio3.BufferWriter();
     bw.writeI32(1);
@@ -700,6 +824,11 @@ var PsbtV2 = class _PsbtV2 extends PsbtV2Maps {
     witnessScript,
     bip32Derivation
   }) {
+    if (!this.isReadyForConstructor) {
+      throw Error(
+        "The PsbtV2 is not ready for a Constructor. Inputs cannot be added."
+      );
+    }
     if (!this.isModifiable(["INPUTS" /* INPUTS */])) {
       throw Error(
         "PsbtV2.PSBT_GLOBAL_TX_MODIFIABLE inputs cannot be modified."
@@ -753,6 +882,11 @@ var PsbtV2 = class _PsbtV2 extends PsbtV2Maps {
     witnessScript,
     bip32Derivation
   }) {
+    if (!this.isReadyForConstructor) {
+      throw Error(
+        "The PsbtV2 is not ready for a Constructor. Outputs cannot be added."
+      );
+    }
     if (!this.isModifiable(["OUTPUTS" /* OUTPUTS */])) {
       throw Error(
         "PsbtV2.PSBT_GLOBAL_TX_MODIFIABLE outputs cannot be modified."
@@ -792,6 +926,11 @@ var PsbtV2 = class _PsbtV2 extends PsbtV2Maps {
    * Removes an input-map from inputMaps.
    */
   deleteInput(index) {
+    if (!this.isReadyForConstructor) {
+      throw Error(
+        "The PsbtV2 is not ready for a Constructor. Inputs cannot be removed."
+      );
+    }
     if (!this.isModifiable(["INPUTS" /* INPUTS */])) {
       throw Error(
         "PsbtV2.PSBT_GLOBAL_TX_MODIFIABLE inputs cannot be modified."
@@ -805,6 +944,11 @@ var PsbtV2 = class _PsbtV2 extends PsbtV2Maps {
    * Removes an output-map from outputMaps.
    */
   deleteOutput(index) {
+    if (!this.isReadyForConstructor) {
+      throw Error(
+        "The PsbtV2 is not ready for a Constructor. Outputs cannot be removed."
+      );
+    }
     if (!this.isModifiable(["OUTPUTS" /* OUTPUTS */])) {
       throw Error(
         "PsbtV2.PSBT_GLOBAL_TX_MODIFIABLE outputs cannot be modified."
@@ -818,7 +962,7 @@ var PsbtV2 = class _PsbtV2 extends PsbtV2Maps {
     this.PSBT_GLOBAL_OUTPUT_COUNT = this.outputMaps.length;
   }
   /**
-   * Checks that provided flags are present in PSBT_GLOBAL_TX_MODIFIABLE.
+   * Checks that all provided flags are present in PSBT_GLOBAL_TX_MODIFIABLE.
    */
   isModifiable(flags) {
     for (const flag of flags) {
@@ -842,6 +986,11 @@ var PsbtV2 = class _PsbtV2 extends PsbtV2Maps {
    * https://github.com/bitcoin/bips/blob/master/bip-0174.mediawiki#signer
    */
   addPartialSig(inputIndex, pubkey, sig) {
+    if (!this.isReadyForSigner) {
+      throw Error(
+        "The PsbtV2 is not ready for a Signer. Partial sigs cannot be added."
+      );
+    }
     if (!this.inputMaps[inputIndex]) {
       throw Error(`PsbtV2 has no input at ${inputIndex}`);
     }
@@ -920,17 +1069,17 @@ var PsbtV2 = class _PsbtV2 extends PsbtV2Maps {
     this.PSBT_GLOBAL_TX_MODIFIABLE = modifiable;
   }
   /**
-   * Attempts to return a PsbtV2 by converting from a PsbtV0 string or Buffer
+   * Attempts to return a PsbtV2 by converting from a PsbtV0 string or Buffer.
+   *
+   * This method first starts with a fresh PsbtV2 having just been created. It
+   * then takes the PsbtV2 through its operator saga through the Signer role. In
+   * this sense validation for each operator role will be performed.
    */
   static FromV0(psbt, allowTxnVersion1 = false) {
     const psbtv0Buf = bufferize(psbt);
     const psbtv0 = import_bitcoinjs_lib2.Psbt.fromBuffer(psbtv0Buf);
     const psbtv0GlobalMap = psbtv0.data.globalMap;
     const psbtv2 = new _PsbtV2();
-    psbtv2.PSBT_GLOBAL_TX_MODIFIABLE = [
-      "INPUTS" /* INPUTS */,
-      "OUTPUTS" /* OUTPUTS */
-    ];
     const txVersion = psbtv0.data.getTransaction().readInt32LE(0);
     if (txVersion === 1 && allowTxnVersion1) {
       psbtv2.dangerouslySetGlobalTxVersion1();
@@ -944,7 +1093,7 @@ var PsbtV2 = class _PsbtV2 extends PsbtV2Maps {
         globalXpub.path
       );
     }
-    let txInputs = [];
+    const txInputs = [];
     for (const [index, txInput] of psbtv0.txInputs.entries()) {
       txInputs[index] = txInput;
     }
@@ -964,7 +1113,7 @@ var PsbtV2 = class _PsbtV2 extends PsbtV2Maps {
         bip32Derivation: input.bip32Derivation
       });
     }
-    let txOutputs = [];
+    const txOutputs = [];
     for (const [index, txOutput] of psbtv0.txOutputs.entries()) {
       txOutputs[index] = txOutput;
     }

package/dist/index.mjs

@@ -216,7 +216,10 @@ var PsbtV2 = class _PsbtV2 extends PsbtV2Maps {
    * Globals Getters/Setters
    */
   get PSBT_GLOBAL_XPUB() {
-    return getNonUniqueKeyTypeValues(this.globalMap, "01" /* PSBT_GLOBAL_XPUB */);
+    return getNonUniqueKeyTypeValues(
+      this.globalMap,
+      "01" /* PSBT_GLOBAL_XPUB */
+    );
   }
   get PSBT_GLOBAL_TX_VERSION() {
     const val = this.globalMap.get("02" /* PSBT_GLOBAL_TX_VERSION */);
@@ -390,16 +393,28 @@ var PsbtV2 = class _PsbtV2 extends PsbtV2Maps {
     );
   }
   get PSBT_IN_RIPEMD160() {
-    return getNonUniqueKeyTypeValues(this.inputMaps, "0a" /* PSBT_IN_RIPEMD160 */);
+    return getNonUniqueKeyTypeValues(
+      this.inputMaps,
+      "0a" /* PSBT_IN_RIPEMD160 */
+    );
   }
   get PSBT_IN_SHA256() {
-    return getNonUniqueKeyTypeValues(this.inputMaps, "0b" /* PSBT_IN_SHA256 */);
+    return getNonUniqueKeyTypeValues(
+      this.inputMaps,
+      "0b" /* PSBT_IN_SHA256 */
+    );
   }
   get PSBT_IN_HASH160() {
-    return getNonUniqueKeyTypeValues(this.inputMaps, "0c" /* PSBT_IN_HASH160 */);
+    return getNonUniqueKeyTypeValues(
+      this.inputMaps,
+      "0c" /* PSBT_IN_HASH160 */
+    );
   }
   get PSBT_IN_HASH256() {
-    return getNonUniqueKeyTypeValues(this.inputMaps, "0d" /* PSBT_IN_HASH256 */);
+    return getNonUniqueKeyTypeValues(
+      this.inputMaps,
+      "0d" /* PSBT_IN_HASH256 */
+    );
   }
   get PSBT_IN_PREVIOUS_TXID() {
     const indices = [];
@@ -551,6 +566,106 @@ var PsbtV2 = class _PsbtV2 extends PsbtV2Maps {
       "fc" /* PSBT_OUT_PROPRIETARY */
     );
   }
+  /**
+   * Operator Role Validation Getters
+   */
+  /**
+   * Returns true if the PsbtV2 is ready for an operator taking the Constructor
+   * role.
+   *
+   * This check assumes that the Creator used this class's constructor method to
+   * initialize the PsbtV2 without passing a psbt (constructor  defaults were
+   * set).
+   */
+  get isReadyForConstructor() {
+    if (this.PSBT_GLOBAL_FALLBACK_LOCKTIME === null) {
+      return false;
+    }
+    if (!this.isModifiable(["INPUTS" /* INPUTS */]) && !this.isModifiable(["OUTPUTS" /* OUTPUTS */])) {
+      return false;
+    }
+    return true;
+  }
+  /**
+   * Returns true if the PsbtV2 is ready for an operator taking the Updater
+   * role.
+   *
+   * Before signatures are added, but after an input is added, a PsbtV2 is
+   * likely to be ready for Constructor, ready for Updater, and ready for Signer
+   * simultaneously.
+   *
+   * According to BIP370, the Updater can modify the sequence number, but it is
+   * unclear if the Updater retains permissions provided in psbtv0 (BIP174). It
+   * is likely not the case that the Updater has the same permissions as
+   * previously because it seems to now be the realm of the Constructor to add
+   * inputs and outputs.
+   */
+  get isReadyForUpdater() {
+    if (this.PSBT_GLOBAL_INPUT_COUNT === 0) {
+      return false;
+    }
+    if (!this.isModifiable(["INPUTS" /* INPUTS */])) {
+      return false;
+    }
+    return true;
+  }
+  /**
+   * Returns true if the PsbtV2 is ready for an operator taking the Signer role.
+   */
+  get isReadyForSigner() {
+    if (this.PSBT_GLOBAL_INPUT_COUNT === 0) {
+      return false;
+    }
+    if (this.isReadyForTransactionExtractor) {
+      return false;
+    }
+    return true;
+  }
+  /**
+   * Returns true if the PsbtV2 is ready for an operator taking the Combiner
+   * role.
+   */
+  get isReadyForCombiner() {
+    return this.isReadyForConstructor || this.isReadyForUpdater || this.isReadyForSigner;
+  }
+  /**
+   * Unimplemented. Returns false.
+   */
+  get isReadyForInputFinalizer() {
+    console.warn(
+      "PsbtV2.isReadyForInputFinalizer has been called, however, this getter is unimplemented and shouldn't be used."
+    );
+    return false;
+  }
+  /**
+   * Returns true if the PsbtV2 is ready for an operator taking the Transaction
+   * Extractor role.
+   *
+   * If all the inputs have been finalized, then the psbt is ready for the
+   * Transaction Extractor. According to BIP 174, it's the responsibility of the
+   * Input Finalizer to add scriptSigs or scriptWitnesses and then remove other
+   * details besides the UTXO. This getter checks that the Input Finalizer has
+   * finished its job.
+   */
+  get isReadyForTransactionExtractor() {
+    for (let i = 0; i < this.PSBT_GLOBAL_INPUT_COUNT; i++) {
+      if (!this.PSBT_IN_FINAL_SCRIPTSIG[i] && !this.PSBT_IN_FINAL_SCRIPTWITNESS[i]) {
+        return false;
+      }
+      if (this.PSBT_IN_FINAL_SCRIPTSIG[i] && !this.PSBT_IN_NON_WITNESS_UTXO[i] || this.PSBT_IN_FINAL_SCRIPTWITNESS[i] && !this.PSBT_IN_WITNESS_UTXO[i]) {
+        return false;
+      }
+      if (
+        // Strings
+        this.PSBT_IN_REDEEM_SCRIPT[i] || this.PSBT_IN_WITNESS_SCRIPT[i] || this.PSBT_IN_POR_COMMITMENT[i] || this.PSBT_IN_TAP_KEY_SIG[i] || this.PSBT_IN_TAP_INTERNAL_KEY[i] || this.PSBT_IN_TAP_MERKLE_ROOT[i] || // Numbers
+        this.PSBT_IN_SIGHASH_TYPE[i] !== null || this.PSBT_IN_SEQUENCE[i] !== null || this.PSBT_IN_REQUIRED_TIME_LOCKTIME[i] !== null || this.PSBT_IN_REQUIRED_HEIGHT_LOCKTIME[i] !== null || // Arrays of non-unique keytype values
+        this.PSBT_IN_PARTIAL_SIG[i].filter((el) => el !== null).length > 0 || this.PSBT_IN_BIP32_DERIVATION[i].filter((el) => el !== null).length > 0 || this.PSBT_IN_RIPEMD160[i].filter((el) => el !== null).length > 0 || this.PSBT_IN_SHA256[i].filter((el) => el !== null).length > 0 || this.PSBT_IN_HASH160[i].filter((el) => el !== null).length > 0 || this.PSBT_IN_HASH256[i].filter((el) => el !== null).length > 0 || this.PSBT_IN_TAP_SCRIPT_SIG[i].filter((el) => el !== null).length > 0 || this.PSBT_IN_TAP_LEAF_SCRIPT[i].filter((el) => el !== null).length > 0 || this.PSBT_IN_TAP_BIP32_DERIVATION[i].filter((el) => el !== null).length > 0
+      ) {
+        return false;
+      }
+    }
+    return true;
+  }
   /**
    * Other Getters/Setters
    */
@@ -596,6 +711,10 @@ var PsbtV2 = class _PsbtV2 extends PsbtV2Maps {
     this.PSBT_GLOBAL_INPUT_COUNT = 0;
     this.PSBT_GLOBAL_OUTPUT_COUNT = 0;
     this.PSBT_GLOBAL_FALLBACK_LOCKTIME = 0;
+    this.PSBT_GLOBAL_TX_MODIFIABLE = [
+      "INPUTS" /* INPUTS */,
+      "OUTPUTS" /* OUTPUTS */
+    ];
   }
   /**
    * Checks initial construction of any valid PsbtV2. It is called when a psbt
@@ -653,6 +772,11 @@ var PsbtV2 = class _PsbtV2 extends PsbtV2Maps {
    * defined for PsbtV2.
    */
   dangerouslySetGlobalTxVersion1() {
+    if (!this.isReadyForConstructor) {
+      throw Error(
+        "The PsbtV2 is not ready for a Constructor. The PSBT_GLOBAL_TX_VERSION should not be forced to version 1."
+      );
+    }
     console.warn("Dangerously setting PsbtV2.PSBT_GLOBAL_TX_VERSION to 1!");
     const bw = new BufferWriter3();
     bw.writeI32(1);
@@ -681,6 +805,11 @@ var PsbtV2 = class _PsbtV2 extends PsbtV2Maps {
     witnessScript,
     bip32Derivation
   }) {
+    if (!this.isReadyForConstructor) {
+      throw Error(
+        "The PsbtV2 is not ready for a Constructor. Inputs cannot be added."
+      );
+    }
     if (!this.isModifiable(["INPUTS" /* INPUTS */])) {
       throw Error(
         "PsbtV2.PSBT_GLOBAL_TX_MODIFIABLE inputs cannot be modified."
@@ -734,6 +863,11 @@ var PsbtV2 = class _PsbtV2 extends PsbtV2Maps {
     witnessScript,
     bip32Derivation
   }) {
+    if (!this.isReadyForConstructor) {
+      throw Error(
+        "The PsbtV2 is not ready for a Constructor. Outputs cannot be added."
+      );
+    }
     if (!this.isModifiable(["OUTPUTS" /* OUTPUTS */])) {
       throw Error(
         "PsbtV2.PSBT_GLOBAL_TX_MODIFIABLE outputs cannot be modified."
@@ -773,6 +907,11 @@ var PsbtV2 = class _PsbtV2 extends PsbtV2Maps {
    * Removes an input-map from inputMaps.
    */
   deleteInput(index) {
+    if (!this.isReadyForConstructor) {
+      throw Error(
+        "The PsbtV2 is not ready for a Constructor. Inputs cannot be removed."
+      );
+    }
     if (!this.isModifiable(["INPUTS" /* INPUTS */])) {
       throw Error(
         "PsbtV2.PSBT_GLOBAL_TX_MODIFIABLE inputs cannot be modified."
@@ -786,6 +925,11 @@ var PsbtV2 = class _PsbtV2 extends PsbtV2Maps {
    * Removes an output-map from outputMaps.
    */
   deleteOutput(index) {
+    if (!this.isReadyForConstructor) {
+      throw Error(
+        "The PsbtV2 is not ready for a Constructor. Outputs cannot be removed."
+      );
+    }
     if (!this.isModifiable(["OUTPUTS" /* OUTPUTS */])) {
       throw Error(
         "PsbtV2.PSBT_GLOBAL_TX_MODIFIABLE outputs cannot be modified."
@@ -799,7 +943,7 @@ var PsbtV2 = class _PsbtV2 extends PsbtV2Maps {
     this.PSBT_GLOBAL_OUTPUT_COUNT = this.outputMaps.length;
   }
   /**
-   * Checks that provided flags are present in PSBT_GLOBAL_TX_MODIFIABLE.
+   * Checks that all provided flags are present in PSBT_GLOBAL_TX_MODIFIABLE.
    */
   isModifiable(flags) {
     for (const flag of flags) {
@@ -823,6 +967,11 @@ var PsbtV2 = class _PsbtV2 extends PsbtV2Maps {
    * https://github.com/bitcoin/bips/blob/master/bip-0174.mediawiki#signer
    */
   addPartialSig(inputIndex, pubkey, sig) {
+    if (!this.isReadyForSigner) {
+      throw Error(
+        "The PsbtV2 is not ready for a Signer. Partial sigs cannot be added."
+      );
+    }
     if (!this.inputMaps[inputIndex]) {
       throw Error(`PsbtV2 has no input at ${inputIndex}`);
     }
@@ -901,17 +1050,17 @@ var PsbtV2 = class _PsbtV2 extends PsbtV2Maps {
     this.PSBT_GLOBAL_TX_MODIFIABLE = modifiable;
   }
   /**
-   * Attempts to return a PsbtV2 by converting from a PsbtV0 string or Buffer
+   * Attempts to return a PsbtV2 by converting from a PsbtV0 string or Buffer.
+   *
+   * This method first starts with a fresh PsbtV2 having just been created. It
+   * then takes the PsbtV2 through its operator saga through the Signer role. In
+   * this sense validation for each operator role will be performed.
    */
   static FromV0(psbt, allowTxnVersion1 = false) {
     const psbtv0Buf = bufferize(psbt);
     const psbtv0 = Psbt2.fromBuffer(psbtv0Buf);
     const psbtv0GlobalMap = psbtv0.data.globalMap;
     const psbtv2 = new _PsbtV2();
-    psbtv2.PSBT_GLOBAL_TX_MODIFIABLE = [
-      "INPUTS" /* INPUTS */,
-      "OUTPUTS" /* OUTPUTS */
-    ];
     const txVersion = psbtv0.data.getTransaction().readInt32LE(0);
     if (txVersion === 1 && allowTxnVersion1) {
       psbtv2.dangerouslySetGlobalTxVersion1();
@@ -925,7 +1074,7 @@ var PsbtV2 = class _PsbtV2 extends PsbtV2Maps {
         globalXpub.path
       );
     }
-    let txInputs = [];
+    const txInputs = [];
     for (const [index, txInput] of psbtv0.txInputs.entries()) {
       txInputs[index] = txInput;
     }
@@ -945,7 +1094,7 @@ var PsbtV2 = class _PsbtV2 extends PsbtV2Maps {
         bip32Derivation: input.bip32Derivation
       });
     }
-    let txOutputs = [];
+    const txOutputs = [];
     for (const [index, txOutput] of psbtv0.txOutputs.entries()) {
       txOutputs[index] = txOutput;
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@caravan/psbt",
-  "version": "1.0.1",
+  "version": "1.1.0",
   "description": "typescript library for working with PSBTs",
   "main": "./dist/index.js",
   "types": "./dist/index.d.ts",