@gala-chain/gswap-sdk 2.0.0-rc.4 → 2.0.0

package/dist/index.d.cts

@@ -2242,7 +2242,7 @@ export type OnramperMode = (typeof ONRAMPER_MODES)[number];
  * `networkPrefix` `gala_gala`).
  */
 export interface OnramperWidgetRequest {
-  /** The wallet that receives the crypto, sent as given (the backend wants an `0x` Ethereum address). */
+  /** The wallet that receives the crypto, sent as given (the backend accepts `0x…` or `eth|…` Ethereum addresses and `client|…` aliases, and canonicalizes them). */
   walletAddress: string;
   mode?: OnramperMode | undefined;
   defaultFiat?: string | undefined;
@@ -3101,10 +3101,12 @@ export declare class Positions {
    * and would never confirm a full close, whose whole point is that the position stops
    * existing).
    *
-   * The "before" state is captured lazily, on the confirmation closure's own first poll —
-   * not eagerly here in `submit()` — so confirming a write never adds an extra round-trip to
-   * the write itself; the cost of establishing the baseline is paid by `confirm()`, only if
-   * and when the caller actually asks to confirm.
+   * The "before" state is read here, before the write is sent. Liquidity writes execute
+   * synchronously, so by the time `confirm()` first polls the effect has already landed: a
+   * baseline taken on that first poll would already include it, and an add or partial remove
+   * would never be seen to change and would time out. Only a full close needs no baseline
+   * (it waits for the position to disappear). If the baseline read fails the write is still
+   * sent and the closure falls back to taking the baseline on its first poll.
    */
   private buildPositionConfirmation;
   private get;

package/dist/index.d.ts

@@ -2242,7 +2242,7 @@ export type OnramperMode = (typeof ONRAMPER_MODES)[number];
  * `networkPrefix` `gala_gala`).
  */
 export interface OnramperWidgetRequest {
-  /** The wallet that receives the crypto, sent as given (the backend wants an `0x` Ethereum address). */
+  /** The wallet that receives the crypto, sent as given (the backend accepts `0x…` or `eth|…` Ethereum addresses and `client|…` aliases, and canonicalizes them). */
   walletAddress: string;
   mode?: OnramperMode | undefined;
   defaultFiat?: string | undefined;
@@ -3101,10 +3101,12 @@ export declare class Positions {
    * and would never confirm a full close, whose whole point is that the position stops
    * existing).
    *
-   * The "before" state is captured lazily, on the confirmation closure's own first poll —
-   * not eagerly here in `submit()` — so confirming a write never adds an extra round-trip to
-   * the write itself; the cost of establishing the baseline is paid by `confirm()`, only if
-   * and when the caller actually asks to confirm.
+   * The "before" state is read here, before the write is sent. Liquidity writes execute
+   * synchronously, so by the time `confirm()` first polls the effect has already landed: a
+   * baseline taken on that first poll would already include it, and an add or partial remove
+   * would never be seen to change and would time out. Only a full close needs no baseline
+   * (it waits for the position to disappear). If the baseline read fails the write is still
+   * sent and the closure falls back to taking the baseline on its first poll.
    */
   private buildPositionConfirmation;
   private get;

package/dist/index.mjs

@@ -4950,7 +4950,7 @@ var Positions = class {
 		const signed = await signPreSend(this.signer, method, dto);
 		const submitOptions = this.walletAddress === void 0 ? {} : { walletAddress: this.walletAddress };
 		const owner = this.walletAddress;
-		const positionConfirmation = this.buildPositionConfirmation(confirmation, owner);
+		const positionConfirmation = await this.buildPositionConfirmation(confirmation, owner);
 		const options = {
 			...submitOptions,
 			...positionConfirmation === void 0 ? {} : { positionConfirmation }
@@ -4964,12 +4964,14 @@ var Positions = class {
 	* and would never confirm a full close, whose whole point is that the position stops
 	* existing).
 	*
-	* The "before" state is captured lazily, on the confirmation closure's own first poll —
-	* not eagerly here in `submit()` — so confirming a write never adds an extra round-trip to
-	* the write itself; the cost of establishing the baseline is paid by `confirm()`, only if
-	* and when the caller actually asks to confirm.
+	* The "before" state is read here, before the write is sent. Liquidity writes execute
+	* synchronously, so by the time `confirm()` first polls the effect has already landed: a
+	* baseline taken on that first poll would already include it, and an add or partial remove
+	* would never be seen to change and would time out. Only a full close needs no baseline
+	* (it waits for the position to disappear). If the baseline read fails the write is still
+	* sent and the closure falls back to taking the baseline on its first poll.
 	*/
-	buildPositionConfirmation(confirmation, owner) {
+	async buildPositionConfirmation(confirmation, owner) {
 		if (confirmation === void 0) return void 0;
 		if (confirmation.kind === "createPool") {
 			const { token0, token1, fee } = confirmation;
@@ -4986,17 +4988,26 @@ var Positions = class {
 		}
 		if (owner === void 0) return void 0;
 		const { kind, identity } = confirmation;
+		const toBaseline = (current) => ({
+			liquidity: new BigNumber((current === null || current === void 0 ? void 0 : current.liquidity) ?? 0),
+			fees: new BigNumber((current === null || current === void 0 ? void 0 : current.fees0) ?? 0).plus(new BigNumber((current === null || current === void 0 ? void 0 : current.fees1) ?? 0))
+		});
 		let baseline;
+		if (kind !== "removeFull") try {
+			baseline = toBaseline(await this.getPosition({
+				...identity,
+				owner
+			}));
+		} catch {
+			baseline = void 0;
+		}
 		return async (signal) => {
 			const current = await this.getPosition({
 				...identity,
 				owner
 			}, signal);
 			if (baseline === void 0) {
-				baseline = {
-					liquidity: new BigNumber((current === null || current === void 0 ? void 0 : current.liquidity) ?? 0),
-					fees: new BigNumber((current === null || current === void 0 ? void 0 : current.fees0) ?? 0).plus(new BigNumber((current === null || current === void 0 ? void 0 : current.fees1) ?? 0))
-				};
+				baseline = toBaseline(current);
 				return kind === "removeFull" && current === null ? {
 					confirmed: true,
 					position: null