@pushchain/core 5.1.13 → 5.1.15

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@pushchain/core",
-  "version": "5.1.13",
+  "version": "5.1.15",
   "homepage": "https://push.org",
   "keywords": [
     "pushchain",

package/src/lib/orchestrator/internals/execute-standard.js

@@ -53,6 +53,28 @@ function encodePayloadForOrigin(ctx, payload) {
     }
     return (0, signing_1.encodeUniversalPayload)(payload);
 }
+/**
+ * Lazy variant of `encodePayloadForOrigin`. The encoded bytes only ride to
+ * the chain on the fee-locking branch (consumed by `lockFee` → origin
+ * gateway → relayer → `DecodeUniversalPayloadSolana`). On the non-fee-
+ * locking branch the bytes are placed into `req` but `req` is never sent —
+ * `sendUniversalTx` ships the proto-struct UniversalPayload via
+ * `MsgExecutePayload` instead.
+ *
+ * Skipping the encode on the non-fee-locking branch sidesteps the SVM
+ * encoder's u64 ceiling for the R2 SVM outbound recursive seam (Slack
+ * 2026-04-23 regression), which sets `execute.value = multicallNativeValue`
+ * — a wei-scale UPC budget that routinely exceeds 2^64 on thin pSOL/WPC
+ * pools. The chain decoder is u64-only, so we can't widen the wire format
+ * without coordinated chain work; this skip is the SDK-only fix that
+ * unblocks Riyanshu's case without changing the wire format.
+ */
+function encodePayloadForOriginIfNeeded(ctx, payload, feeLockingRequired) {
+    if (!feeLockingRequired) {
+        return '0x';
+    }
+    return encodePayloadForOrigin(ctx, payload);
+}
 function executeStandardPayload(ctx, execute, eventBuffer, getResponseCallbacks) {
     return tslib_1.__awaiter(this, void 0, void 0, function* () {
         var _a, _b, _c;
@@ -167,7 +189,7 @@ function executeStandardPayload(ctx, execute, eventBuffer, getResponseCallbacks)
             }, helpers_1.bigintReplacer));
             req = (0, payload_builder_1.buildUniversalTxRequest)(ctx.universalSigner.account.address, {
                 recipient: viem_1.zeroAddress, token: viem_1.zeroAddress, amount: BigInt(0),
-                payload: encodePayloadForOrigin(ctx, universalPayloadForReq),
+                payload: encodePayloadForOriginIfNeeded(ctx, universalPayloadForReq, feeLockingRequired),
             });
         }
         else {
@@ -196,7 +218,7 @@ function executeStandardPayload(ctx, execute, eventBuffer, getResponseCallbacks)
                     }, helpers_1.bigintReplacer));
                     req = (0, payload_builder_1.buildUniversalTxRequest)(ctx.universalSigner.account.address, {
                         recipient: viem_1.zeroAddress, token: viem_1.zeroAddress, amount: BigInt(0),
-                        payload: encodePayloadForOrigin(ctx, universalPayloadOther),
+                        payload: encodePayloadForOriginIfNeeded(ctx, universalPayloadOther, feeLockingRequired),
                     });
                 }
             }
@@ -217,7 +239,7 @@ function executeStandardPayload(ctx, execute, eventBuffer, getResponseCallbacks)
                 }, helpers_1.bigintReplacer));
                 req = (0, payload_builder_1.buildUniversalTxRequest)(ctx.universalSigner.account.address, {
                     recipient: viem_1.zeroAddress, token: viem_1.zeroAddress, amount: BigInt(0),
-                    payload: encodePayloadForOrigin(ctx, universalPayloadSelf),
+                    payload: encodePayloadForOriginIfNeeded(ctx, universalPayloadSelf, feeLockingRequired),
                 });
             }
         }

package/src/lib/orchestrator/internals/outbound-sync.d.ts

@@ -15,8 +15,8 @@ export declare const OUTBOUND_POLL_INTERVAL_MS = 3000;
 export declare const OUTBOUND_MAX_TIMEOUT_MS = 180000;
 /** Maximum total timeout for the R3 inbound round-trip polling loop (ms). */
 export declare const INBOUND_MAX_TIMEOUT_MS = 300000;
-/** Initial wait before the first inbound poll (ms). Set to 0 — the outbound just completed, so the indexer may already have the child UTX. */
-export declare const INBOUND_INITIAL_WAIT_MS = 0;
+/** Initial wait before the first inbound poll (ms). Validator observation + Push commit + indexer ingestion realistically take 15-25 s on testnet (esp. SVM source), so polling immediately just burns RPC on guaranteed-empty results. */
+export declare const INBOUND_INITIAL_WAIT_MS = 15000;
 /**
  * Thrown by `waitForOutboundTx` when the polling loop exceeds the configured
  * timeout without observing an external tx hash. Callers can `instanceof`

package/src/lib/orchestrator/internals/outbound-sync.js

@@ -30,8 +30,8 @@ exports.OUTBOUND_POLL_INTERVAL_MS = 3000; // 3s
 exports.OUTBOUND_MAX_TIMEOUT_MS = 180000; // 180s (3 min)
 /** Maximum total timeout for the R3 inbound round-trip polling loop (ms). */
 exports.INBOUND_MAX_TIMEOUT_MS = 300000; // 300s (5 min) — covers R3 inbound latency on testnet
-/** Initial wait before the first inbound poll (ms). Set to 0 — the outbound just completed, so the indexer may already have the child UTX. */
-exports.INBOUND_INITIAL_WAIT_MS = 0;
+/** Initial wait before the first inbound poll (ms). Validator observation + Push commit + indexer ingestion realistically take 15-25 s on testnet (esp. SVM source), so polling immediately just burns RPC on guaranteed-empty results. */
+exports.INBOUND_INITIAL_WAIT_MS = 15000;
 // ============================================================================
 // Typed errors
 // ============================================================================

package/src/lib/orchestrator/internals/route-handlers.js

@@ -1235,13 +1235,17 @@ function executeCeaToPushSvm(ctx, params, sourceChain, executeFn) {
         response.chain = sourceChain;
         const chainInfo = chain_1.CHAIN_INFO[sourceChain];
         response.chainNamespace = `${chain_1.VM_NAMESPACE[chainInfo.vm]}:${chainInfo.chainId}`;
-        // R3 SVM inbound round-trip tracking: enable only when funds flow back
-        // (drainAmount > 0). See the EVM executeCeaToPush note — child-utxId
-        // correlation uses the universal-tx-detector's deterministic derivation.
-        // NOTE: the detector is EVM-only today; SVM source-chain logs can't be
-        // decoded yet, so this branch will fall back to the (broken) cosmos
-        // search. Disable if SVM R3 tests start timing out in CI.
-        response._expectsInboundRoundTrip = drainAmount > BigInt(0);
+        // R3 SVM inbound round-trip tracking is disabled.
+        //
+        // For Route 3 SVM drains (rescue_funds-style flow, txType=RESCUE_FUNDS),
+        // empirical check on donut showed the keeper does NOT create a child
+        // universalTxId on Push for the Solana-side `UniversalTx(fromCEA=true)`
+        // log — the parent utxId's `outboundStatus=OBSERVED` is the terminal state.
+        // Polling for a deterministically-derived child id therefore never
+        // resolves, and the cosmos `inbound_tx_hash` text-search fallback also
+        // returns 0 hits. Receipts now resolve as soon as the Solana outbound is
+        // observed instead of timing out at 300s.
+        response._expectsInboundRoundTrip = false;
         return response;
     });
 }

package/src/lib/price-fetch/price-fetch.js

@@ -54,11 +54,29 @@ class PriceFetch {
                         // eslint-disable-next-line @typescript-eslint/no-explicit-any
                     }, { commitment: 'confirmed' });
                     const program = new anchor_1.Program(feeLocker_json_1.default, provider);
-                    const result = yield program.methods['getSolPrice']()
-                        .accounts({
-                        priceUpdate: PRICE_ACCOUNT,
-                    })
-                        .view();
+                    let result;
+                    try {
+                        result = yield program.methods['getSolPrice']()
+                            .accounts({
+                            priceUpdate: PRICE_ACCOUNT,
+                        })
+                            .view();
+                    }
+                    catch (err) {
+                        const e = err;
+                        console.error('[price-fetch][SVM getSolPrice] FAILED');
+                        console.error('  rpc:', rpcUrls[0]);
+                        console.error('  locker:', lockerContract);
+                        console.error('  priceAccount:', PRICE_ACCOUNT.toBase58());
+                        console.error('  message:', e === null || e === void 0 ? void 0 : e.message);
+                        if (e === null || e === void 0 ? void 0 : e.logs)
+                            console.error('  logs:', e.logs);
+                        if (e === null || e === void 0 ? void 0 : e.simulationResponse)
+                            console.error('  simulationResponse:', JSON.stringify(e.simulationResponse, null, 2));
+                        if (e === null || e === void 0 ? void 0 : e.programErrorStack)
+                            console.error('  programErrorStack:', e.programErrorStack);
+                        throw err;
+                    }
                     if (!result || !result.price) {
                         throw new Error('Invalid price data returned');
                     }

package/src/lib/universal/account/account.d.ts

@@ -94,7 +94,7 @@ export declare function deriveSvmCeaPda(ueaAddress: `0x${string}`, targetChain:
  *
  * @param account - UniversalAccount with chain and address.
  * @param options.chain - Target external chain for CEA derivation. If omitted, returns UEA.
- * @param options.skipNetworkCheck - If true, skip deployment check and return deployed: false.
+ * @param options.skipNetworkCheck - If true, skip deployment check and return deployed: null.
  * @returns The derived executor account address and deployment status.
  */
 export declare function deriveExecutorAccount(account: UniversalAccount, options?: {

package/src/lib/universal/account/account.js

@@ -471,7 +471,7 @@ function deriveSvmCeaPda(ueaAddress, targetChain, skipNetworkCheck) {
  *
  * @param account - UniversalAccount with chain and address.
  * @param options.chain - Target external chain for CEA derivation. If omitted, returns UEA.
- * @param options.skipNetworkCheck - If true, skip deployment check and return deployed: false.
+ * @param options.skipNetworkCheck - If true, skip deployment check and return deployed: null.
  * @returns The derived executor account address and deployment status.
  */
 function deriveExecutorAccount(account, options) {

package/src/lib/universal-tx-detector/child-inbounds.js

@@ -135,6 +135,21 @@ function resolveChildInboundsFromLogs(pushClient, sourceChainCaip, externalTxHas
             }
             catch (err) {
                 diagnostics === null || diagnostics === void 0 ? void 0 : diagnostics.push(`getUniversalTxByIdV2(${utxIdHex}) failed: ${err instanceof Error ? err.message : String(err)}`);
+                // Cosmos lookup failed (commonly NotFound while the indexer is still
+                // catching up). The utxId is deterministically derived from the source
+                // log, so surface it as notFound rather than dropping the entry —
+                // otherwise callers like inbound-tracker repeatedly re-run the full
+                // detector pass instead of advancing to lightweight status polling.
+                out.push({
+                    universalTxId: utxIdHex,
+                    sourceLogIndex: log.logIndex,
+                    sourceEventName: log.eventName,
+                    status: 0,
+                    statusName: STATUS_NAMES[0],
+                    pcTxHashes: [],
+                    outboundHashes: [],
+                    notFound: true,
+                });
             }
         }
         return out;