Skip to content

add updated configurations for new chains#18

Merged
CluEleSsUK merged 42 commits intomasterfrom
feat/add-new-chains
Jul 2, 2025
Merged

add updated configurations for new chains#18
CluEleSsUK merged 42 commits intomasterfrom
feat/add-new-chains

Conversation

@CluEleSsUK
Copy link
Copy Markdown
Contributor

@CluEleSsUK CluEleSsUK commented May 28, 2025

All checks now pass.

The DCIPHER_PUBLIC_KEY has been switched over to the new one which is used on all chains that support user billing.

The integration tests for the chains that do not support user billing yet, have been commented out temporarily.

Function calculateRequestPriceNative has been added to help users estimate how much their request will cost based on the callbackGasLimit and the current chain gas price (used by the RPC provider by default when not specified). This function is also used in the requestRandomness logic.

async calculateRequestPriceNative(callbackGasLimit: bigint): Promise<bigint> {
      const requestPrice = await this.contract.calculateRequestPriceNative(callbackGasLimit)
      return requestPrice;
  }

calculateRequestPriceNative returns a better estimate compared to calling RandomnessSender.estimateRequestPriceNative(uint32 _callbackGasLimit, uint256 _requestGasPriceWei)) and passing in a hardcoded network gas price which could be lower than the current network gas price and will give the user a lower estimate compared to calculateRequestPriceNative and in some cases could lead to a fee too low revert message from BlocklockSender. @CluEleSsUK

@najienka
Copy link
Copy Markdown
Contributor

najienka commented Jun 16, 2025

@CluEleSsUK All the tests should be good now.
Filecoin calibration testnet is still running the JS agent and will be switched over soon.
Filecoin tests are commented out.

@CluEleSsUK
Copy link
Copy Markdown
Contributor Author

gonna merge despite failures and some test skipping.

  • skipped sei because its testnet seems fucked
  • furnace is down rn
  • pol had no funds left but just added some

@CluEleSsUK CluEleSsUK merged commit 4580109 into master Jul 2, 2025
0 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants