Skip to content

Conversation

@glitch003
Copy link
Contributor

No description provided.

Copilot AI review requested due to automatic review settings December 19, 2025 21:06
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR adds the k256 precompile configuration to the litMainnet network in the Hardhat config. The change enables support for Arbitrum Stylus contracts on the litMainnet chain by configuring the required precompile contract addresses.

  • Adds stylusContractsForTests configuration block to the litMainnet network
  • Configures k256 contract address as 0x029bedeacaf6821ce9a6bd7c8ac73350f24a014f
  • Configures p256 contract address with fallback to environment variable

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines +148 to +150
p256:
process.env.LIT_STYLUS_P256_CONTRACT_ADDRESS ||
'0x0000000000000000000000000000000000000000',
Copy link

Copilot AI Dec 19, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The p256 contract address is set to the zero address (0x0000000000000000000000000000000000000000), which will cause failures in both tests and deployment scripts. The test validation checks that both stylusContractsForTests.p256 and stylusContractsForTests.k256 are truthy values, and the deploy script throws an error if either address is missing. A valid p256 contract address should be provided instead of using the zero address as a placeholder.

Suggested change
p256:
process.env.LIT_STYLUS_P256_CONTRACT_ADDRESS ||
'0x0000000000000000000000000000000000000000',
p256: process.env.LIT_STYLUS_P256_CONTRACT_ADDRESS,

Copilot uses AI. Check for mistakes.
@github-actions
Copy link

PASS [ 43.199s] (3/3) lit_node::test toxiproxy::perf_tests::load_with_no_latency
PASS [ 43.216s] (2/3) lit_node::test toxiproxy::perf_tests::load_with_50ms_latency_single_link
PASS [ 92.193s] (1/3) lit_node::test toxiproxy::perf_tests::load_with_50ms_latency_all_links

@DashKash54
Copy link

Required when re-deploying Naga-proto

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