Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Deploy all contracts to ink #1016

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

0xDEnYO
Copy link
Contributor

@0xDEnYO 0xDEnYO commented Feb 20, 2025

Which Jira task belongs to this PR?

https://lifi.atlassian.net/browse/LF-12287

Why did I implement it this way?

Checklist before requesting a review

Checklist for reviewer (DO NOT DEPLOY and contracts BEFORE CHECKING THIS!!!)

  • I have checked that any arbitrary calls to external contracts are validated and or restricted
  • I have checked that any privileged calls (i.e. storage modifications) are validated and or restricted
  • I have ensured that any new contracts have had AT A MINIMUM 1 preliminary audit conducted on by <company/auditor>

Copy link
Contributor

coderabbitai bot commented Feb 20, 2025

Warning

Rate limit exceeded

@0xDEnYO has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 14 minutes and 9 seconds before requesting another review.

⌛ How to resolve this issue?

After the wait time has elapsed, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

We recommend that you space out your commits to avoid hitting the rate limit.

🚦 How do rate limits work?

CodeRabbit enforces hourly rate limits for each developer per organization.

Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout.

Please see our FAQ for further information.

📥 Commits

Reviewing files that changed from the base of the PR and between 131b6e9 and 2488b38.

📒 Files selected for processing (2)
  • deployments/ink.diamond.json (1 hunks)
  • foundry.toml (2 hunks)

Walkthrough

This pull request integrates support for the new INK network. It adds a new Ethereum node URI variable (ETH_NODE_URI_INK) in environment and workflow files, and updates several configuration files (JSON and TOML) to include chain IDs, RPC endpoints, addresses, and network parameters for INK. Deployment records and diamond architecture configurations are introduced for ink-related smart contracts. In addition, the pull request adjusts deployment scripts by updating facet versions and refining log extraction, while also upgrading the Solidity compiler and EVM versions.

Changes

Files Change Summary
.env.example
.github/.../diamondEmergencyPause.yml
Added the ETH_NODE_URI_INK environment variable for the INK network mainnet RPC in both the sample environment file and the GitHub workflow.
config/across.json
config/dexs.json
config/gaszip.json
config/global.json
config/networks.json
config/relay.json
config/stargate.json
config/tokenwrapper.json
Introduced new "ink" network entries with dedicated chain IDs, addresses, RPC URLs, and other parameters. Also, restored and modified related network configurations (e.g. "soneium" in dexs.json, updated relay sections).
deployments/_deployments_log_file.json
deployments/ink.diamond.json
deployments/ink.json
Added new deployment records and configuration files capturing ink-specific smart contract deployments and diamond facets, including deployment metadata across several versions.
foundry.toml Upgraded the Solidity compiler (from 0.8.17 to 0.8.28) and EVM version (from shanghai to cancun); added a new RPC endpoint and an etherscan entry for the INK network.
networks Appended the new ink entry to the list of available networks.
script/deploy/_targetState.json
script/deploy/healthCheck.ts
script/tasks/diamondUpdateFacet.sh
Modified deployment scripts: removed the "Receiver" and "StargateFacet" entries, updated facet versions in the ink section, and enhanced the extraction of JSON log output in the update facet script.

Possibly related PRs


Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?

❤️ Share
🪧 Tips

Chat

There are 3 ways to chat with CodeRabbit:

  • Review comments: Directly reply to a review comment made by CodeRabbit. Example:
    • I pushed a fix in commit <commit_id>, please review it.
    • Generate unit testing code for this file.
    • Open a follow-up GitHub issue for this discussion.
  • Files and specific lines of code (under the "Files changed" tab): Tag @coderabbitai in a new review comment at the desired location with your query. Examples:
    • @coderabbitai generate unit testing code for this file.
    • @coderabbitai modularize this function.
  • PR comments: Tag @coderabbitai in a new PR comment to ask questions about the PR branch. For the best results, please provide a very specific query, as very limited context is provided in this mode. Examples:
    • @coderabbitai gather interesting stats about this repository and render them as a table. Additionally, render a pie chart showing the language distribution in the codebase.
    • @coderabbitai read src/utils.ts and generate unit testing code.
    • @coderabbitai read the files in the src/scheduler package and generate a class diagram using mermaid and a README in the markdown format.
    • @coderabbitai help me debug CodeRabbit configuration file.

Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments.

CodeRabbit Commands (Invoked using PR comments)

  • @coderabbitai pause to pause the reviews on a PR.
  • @coderabbitai resume to resume the paused reviews.
  • @coderabbitai review to trigger an incremental review. This is useful when automatic reviews are disabled for the repository.
  • @coderabbitai full review to do a full review from scratch and review all the files again.
  • @coderabbitai summary to regenerate the summary of the PR.
  • @coderabbitai generate docstrings to generate docstrings for this PR. (Beta)
  • @coderabbitai resolve resolve all the CodeRabbit review comments.
  • @coderabbitai configuration to show the current CodeRabbit configuration for the repository.
  • @coderabbitai help to get help.

Other keywords and placeholders

  • Add @coderabbitai ignore anywhere in the PR description to prevent this PR from being reviewed.
  • Add @coderabbitai summary to generate the high-level summary at a specific location in the PR description.
  • Add @coderabbitai anywhere in the PR title to generate the title automatically.

CodeRabbit Configuration File (.coderabbit.yaml)

  • You can programmatically configure CodeRabbit by adding a .coderabbit.yaml file to the root of your repository.
  • Please see the configuration documentation for more information.
  • If your editor has YAML language server enabled, you can add the path at the top of this file to enable auto-completion and validation: # yaml-language-server: $schema=https://coderabbit.ai/integrations/schema.v2.json

Documentation and Community

  • Visit our Documentation for detailed information on how to use CodeRabbit.
  • Join our Discord Community to get help, request features, and share feedback.
  • Follow us on X/Twitter for updates and announcements.

@lifi-action-bot lifi-action-bot marked this pull request as draft February 20, 2025 04:58
@0xDEnYO 0xDEnYO marked this pull request as ready for review February 20, 2025 04:58
Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 0

🧹 Nitpick comments (2)
config/gaszip.json (1)

16-16: New INK Router Entry Added

The addition of the "ink" entry with the address "0x2a37D63EAdFe4b4682a3c28C1c2cD4F109Cc2762" is consistent with other network entries. Please verify that using the same router address as the "mainnet" entry is intentional for the ink network.

deployments/ink.diamond.json (1)

69-83: Periphery Configuration – Review Empty Entries

In the "Periphery" section, several components like "LiFuelFeeCollector", "Permit2Proxy", "Receiver", and "RelayerCelerIM" have empty string values. If these are intentionally left unassigned for future deployment or configuration, it might help to add a comment or a TODO note to clarify the intent.

📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 0485bfe and b4abb39.

📒 Files selected for processing (18)
  • .env.example (1 hunks)
  • .github/workflows/diamondEmergencyPause.yml (1 hunks)
  • config/across.json (1 hunks)
  • config/dexs.json (2 hunks)
  • config/gaszip.json (1 hunks)
  • config/global.json (1 hunks)
  • config/networks.json (1 hunks)
  • config/relay.json (2 hunks)
  • config/stargate.json (4 hunks)
  • config/tokenwrapper.json (1 hunks)
  • deployments/_deployments_log_file.json (25 hunks)
  • deployments/ink.diamond.json (1 hunks)
  • deployments/ink.json (1 hunks)
  • foundry.toml (3 hunks)
  • networks (1 hunks)
  • script/deploy/_targetState.json (2 hunks)
  • script/deploy/healthCheck.ts (0 hunks)
  • script/tasks/diamondUpdateFacet.sh (1 hunks)
💤 Files with no reviewable changes (1)
  • script/deploy/healthCheck.ts
✅ Files skipped from review due to trivial changes (1)
  • networks
🧰 Additional context used
🧠 Learnings (2)
.github/workflows/diamondEmergencyPause.yml (1)
Learnt from: 0xDEnYO
PR: lifinance/contracts#894
File: .github/workflows/diamondEmergencyPause.yml:82-82
Timestamp: 2024-12-05T03:42:41.402Z
Learning: The `ETH_NODE_URI_WORLDCHAIN` secret is correctly configured in the GitHub repository.
script/deploy/_targetState.json (1)
Learnt from: ezynda3
PR: lifinance/contracts#861
File: script/deploy/_targetState.json:1453-1483
Timestamp: 2024-11-21T08:24:53.059Z
Learning: In `script/deploy/_targetState.json`, for the `abstract` network configuration, `ReceiverStargateV2` is correctly set to version `1.0.1`.
⏰ Context from checks skipped due to timeout of 90000ms (2)
  • GitHub Check: run-unit-tests
  • GitHub Check: run-unit-tests
🔇 Additional comments (45)
deployments/ink.json (1)

1-27: New INK Facets Configuration File Looks Correct

This new JSON file correctly lists all the facet names mapped to their corresponding Ethereum addresses for the ink network deployment. The structure is valid and consistent with the diamond architecture pattern requirements.

config/global.json (1)

5-5: INK Factory Address Integration

The new key "ink": "0xeBbbaC35500713C4AD49929e1bE4225c7efF6510" is integrated properly into the create3Factory object. Ensure that this address has been audited and is correctly associated with the ink network deployment requirements.

config/tokenwrapper.json (1)

30-30: TokenWrapper Configuration for INK Network

The new entry for "ink": "0x4200000000000000000000000000000000000006" has been added correctly. Please confirm that this address matches the deployed token wrapper on the ink network and is consistent with other network configurations in the project.

deployments/ink.diamond.json (1)

3-68: Facet Configuration for Diamond Deployment

The "Facets" section correctly maps facet contract addresses to their names and version numbers. The addresses here match those provided in the other ink-related deployment files. Ensure that the version numbers align with the deployed contract implementations and that they have been verified through prior audits.

.env.example (1)

21-21: New INK network variable added.
The addition of ETH_NODE_URI_INK=https://rpc-gel.inkonchain.com #[pre-commit-checker: not a secret] is consistent with the existing RPC variable format. Please verify that the provided URI is correct and accessible in the deployment environments.

.github/workflows/diamondEmergencyPause.yml (1)

69-69: Integration of INK network in workflow.
The new environment variable ETH_NODE_URI_INK: ${{ secrets.ETH_NODE_URI_INK }} is added to the Pause Diamond job configuration. Ensure that the corresponding secret is properly configured in the GitHub repository.

config/relay.json (1)

66-70: New INK network configuration in relay.json.
The "ink" entry is introduced with chainId: 13371, along with the specified relayReceiver and relaySolver addresses. Please verify that these addresses and the chain ID accurately reflect the intended parameters for the INK network.

config/across.json (1)

104-109: New INK network entry added.
The configuration for the "ink" network (with chainId: 13371, acrossSpokePool, weth, and an empty tokensToApprove array) is correctly added. Please confirm that these parameters match the deployment requirements for the INK network.

foundry.toml (3)

1-4: Updated Solidity compiler and EVM version.
The update to solc_version = '0.8.28' and evm_version = 'cancun' in the [profile.default] section is noted. Ensure that these versions are compatible with the current contracts and tooling.


46-48: New RPC endpoint for INK network.
Adding ink = "${ETH_NODE_URI_INK}" under [rpc_endpoints] is correctly implemented. Please verify that the environment variable is populated and that the endpoint is accessible in all relevant environments.


97-97: Etherscan integration for INK network.
The etherscan entry for ink has been added:

ink = { key = "${INK_ETHERSCAN_API_KEY}", url = "https://explorer.inkonchain.com/api/", verifier = "blockscout", chain = "57073"}

Please confirm that the chain ID 57073 and the API endpoint match the INK network’s specifications.

script/tasks/diamondUpdateFacet.sh (1)

121-125: Enhanced JSON Log Extraction and User Feedback
The updated logic uses

CLEAN_RETURN_DATA=$(echo "$RAW_RETURN_DATA" | grep -o '{"logs.*}')

to robustly capture the JSON log output by removing any extraneous content both before and after. Additionally, the new echo statements (lines 123–125) provide clear feedback by printing the diamondCut calldata. This improves clarity during execution.

config/stargate.json (4)

24-24: New INK Endpoint in endpointV2 Section
A new entry for the "ink" network is added with the endpoint address

"ink": "0xca29f3A6f966Cb2fc0dE625F8f325c0C46dbE958"

Ensure this value is verified against the deployed contract details for the INK network.


65-65: New INK Token Messaging Entry
The "ink" entry is now present in the tokenMessaging section with the address

"ink": "0x45f1A95A4D3f3836523F5c83673c797f4d4d263B"

Confirm that this address correctly supports token messaging for INK.


235-235: New INK Composer Address
An entry for "ink" has been added in the composers section with a zero address:

"ink": "0x0000000000000000000000000000000000000000"

Please verify if this zero address is intentionally used as a placeholder or if a valid contract address should be configured.


284-284: New INK Router Entry
A new router entry for "ink" is introduced with the address:

"ink": "0x0000000000000000000000000000000000000000"

Ensure that this zero address is the appropriate value for the deployment scenario or if it needs to be updated with a concrete router contract address.

config/networks.json (1)

344-361: New INK Network Configuration Added
A comprehensive configuration block for the "ink" network has been added. The new details include:

  • "chainId": 57073
  • "rpcUrl": "https://rpc-gel.inkonchain.com"
  • Explorer URLs ("explorerUrl": "https://explorer.inkonchain.com" and "explorerApiUrl": "https://explorer.inkonchain.com/api/")
  • Multicall and safe transaction endpoints, along with "gasZipChainId": 392

These settings align with the objective of integrating support for the INK network. Please double-check that all endpoints and IDs are accurate with respect to the latest network specifications.

config/dexs.json (2)

472-477: New "ink" network category added.

The new "ink" network block is introduced with four addresses. Please double-check that these addresses match the official INK deployment documentation and that they are in the expected order.


833-838: Restoration of "soneium" network category confirmed.

The "soneium" category has been reintroduced with the identical four addresses as before. Verify that the duplication with the "ink" category is intentional and that downstream logic (if any) can distinguish between these two network entries.

script/deploy/_targetState.json (1)

631-633: "ink" network facet version updates verified.

In the "ink" production configuration, the facets—namely "AcrossFacetV3", "AcrossFacetPackedV3", and "ReceiverAcrossV3"—are uniformly set to "1.0.0", and "ReceiverStargateV2" is updated to "1.0.0". These updates align with the intended restructuring for deploying contracts to ink. Please ensure that any related deployment scripts and health checks are updated to reference these new versions.

Also applies to: 637-637

deployments/_deployments_log_file.json (25)

805-818: Ink Deployment Record for Production Version "1.0.0" (Context: Preceding DiamondLoupeFacet)
This new block correctly introduces an ink network deployment record with all required fields. Please double-check that the ADDRESS and TIMESTAMP values are accurate and conform to the expected deployment details.


1611-1625: Ink Deployment Record for Production Version "1.0.0" (Context: Under OwnershipFacet)
The ink entry is added consistently with the established JSON schema. Verify that the provided ADDRESS, OPTIMIZER_RUNS, and TIMESTAMP match your deployment records.


2416-2430: Ink Deployment Record for Production Version "1.0.0" (Context: Under an earlier contract segment)
The structure is correct and mirrors the other network entries. Ensure that the ADDRESS and other fields such as CONSTRUCTOR_ARGS and VERIFIED status are correct for this deployment.


3523-3537: Ink Deployment Record for Production Version "1.0.1" (Context: Prior to AccessManagerFacet)
The record is added with the expected format. Confirm that the version "1.0.1" is intended for this contract and that the associated deployment details are accurate.


4329-4343: Ink Deployment Record for Production Version "1.0.0" (Context: Under WithdrawFacet)
The new ink network record follows the consistent pattern. It is recommended to double-check the ADDRESS and TIMESTAMP to ensure they reflect the latest deployment information.


5121-5135: Ink Deployment Record for Production Version "1.0.0" (Context: Preceding PeripheryRegistryFacet)
The entry appears to be correctly structured. Please verify that all numeric values (e.g., OPTIMIZER_RUNS) are intentional strings and that the deployment metadata is correct.


5926-5941: Ink Deployment Record for Production Version "1.0.0" (Context: Preceding LiFiDiamond)
This ink entry is well-formed. Ensure that the address “0x6adB9BBCaAbf760f401bdA7730Aa4F6C1eF65456” and its associated metadata correctly reflect the current production deployment.


6706-6721: Ink Deployment Record for Production Version "1.0.0" with Non-Empty Constructor Args
The record includes a non-empty CONSTRUCTOR_ARGS field, which is a deviation from the other entries. Please verify that these constructor arguments are correct and intentional for this specific deployment.


8557-8572: Ink Deployment Record for Production Version "1.0.0" (Context: Preceding HopFacetOptimized)
The deployment entry is correctly added with all expected fields. Confirm that the provided values, especially the ADDRESS and TIMESTAMP, align with your deployment logs.


12089-12104: Ink Deployment Record for Production Version "1.1.0" (Context: Preceding Executor)
This entry is structured consistently with previous entries. Ensure that the CONSTRUCTOR_ARGS provided for version "1.1.0" correctly match the contract’s required initialization parameters.


13069-13084: Ink Deployment Record for Production Version "2.1.0" (Context: Under Executor)
The ink record for version "2.1.0" is added in the expected format. Please reconfirm that the ADDRESS and other parameters, including the deployment TIMESTAMP, are up to date.


13850-13865: Ink Deployment Record for Production Version "1.0.1" (Context: Under FeeCollector)
This new entry appears consistent with other network entries. As always, verify that the deployment details (ADDRESS, TIMESTAMP, etc.) are correct.


21676-21691: Ink Deployment Record for Production Version "1.2.0" (Context: Under Receiver)
The JSON segment follows the established schema. Ensure that the record’s meta fields such as OPTIMIZER_RUNS and TIMESTAMP accurately represent the production deployment for this version.


24161-24177: Ink Deployment Record for Production Version "1.1.0" (Context: Under LiFuelFeeCollector)
The entry is structured properly and includes detailed CONSTRUCTOR_ARGS. Please verify that these arguments are correct and that the overall deployment record is accurate.


24955-24971: Ink Deployment Record for Production Version "1.0.1" (Context: Under GasRebateDistributor)
The record adheres to the standard format. Double-check that the ADDRESS and TIMESTAMP are correct and that the deployment parameters have been set intentionally.


25443-25458: Ink Deployment Record for Production Version "1.0.1" (Context: Under ReceiverStargateV2)
This ink network entry is properly formatted. Verify that all data fields, especially the numeric parameters encoded as strings, align with the expected deployment values.


25779-25794: Ink Deployment Record for Production Version "1.1.0" (Context: Under LiFiDEXAggregator)
The provided details are comprehensive, including a lengthy CONSTRUCTOR_ARGS field. Please recheck that these parameters are complete and correct for this deployment version.


26413-26428: Ink Deployment Record for Production Version "1.6.0" (Context: Under EmergencyPauseFacet)
This entry introduces a higher version with an extensive set of CONSTRUCTOR_ARGS. It is recommended to verify these arguments carefully to ensure they match the intended initialization sequence.


27051-27066: Ink Deployment Record for Production Version "1.0.1" (Context: Under AcrossFacetV3)
The deployment record is correctly added with all necessary fields. Please confirm that the ADDRESS and other metadata match the expected contract deployment details.


27439-27454: Ink Deployment Record for Production Version "1.1.0" (Context: Under ReceiverAcrossV3)
The JSON entry is consistent with previous entries, including a detailed CONSTRUCTOR_ARGS field. Verify that these values correctly represent the deployed contract configuration.


27808-27823: Ink Deployment Record for Production Version "1.1.0" (Context: Under AcrossFacetPackedV3)
This block is well structured and maintains consistency with the overall deployment log format. Please double-check the deployment parameters for accuracy.


28166-28181: Ink Deployment Record for Production Version "1.2.0" (Context: Under Permit2Proxy)
The record features a lengthy CONSTRUCTOR_ARGS field that should be validated against the contract’s constructor requirements. Apart from that, the structure is correct.


28804-28819: Ink Deployment Record for Production Version "2.0.2" (Context: Under GasZipPeriphery)
This new ink entry is added following the established pattern. Please ensure the address and associated metadata are correct and reflect the actual deployment details.


29207-29225: Ink Deployment Record for Production Version "1.0.1" (Context: Under RelayFacet)
The ink network deployment record here is consistent with other entries. It is advisable to verify that the long hex string in CONSTRUCTOR_ARGS is precise and requires no further update.


29626-29641: Ink Deployment Record for Production Version "1.0.0" (Context: Under GetGasFacet)
The entry follows the JSON schema and the overall style of previous ink deployments. Ensure that the provided ADDRESS and CONSTRUCTOR_ARGS are accurate reflections of the deployed contract’s state.

Copy link
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 0

🧹 Nitpick comments (2)
script/deploy/_targetState.json (2)

632-634: Ensure Consistency for Ink Network Contract Versions

The entries for "AcrossFacetV3", "AcrossFacetPackedV3", and "ReceiverAcrossV3" have been updated to "1.0.0" in the ink network configuration. Please verify that these version updates are correct per the deployment specifications and that all downstream contracts or scripts consuming these values are updated accordingly.


638-638: Verify ReceiverStargateV2 Version in Ink

The "ReceiverStargateV2" entry in the ink network has been updated to "1.0.0". Confirm that this change is intentional and that all related components (such as deployment scripts and health checks) align with this updated version.

📜 Review details

Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between b4abb39 and 131b6e9.

📒 Files selected for processing (5)
  • config/permit2Proxy.json (1 hunks)
  • deployments/_deployments_log_file.json (26 hunks)
  • deployments/ink.json (1 hunks)
  • script/deploy/_targetState.json (2 hunks)
  • script/tasks/diamondUpdateFacet.sh (1 hunks)
🚧 Files skipped from review as they are similar to previous changes (2)
  • script/tasks/diamondUpdateFacet.sh
  • deployments/ink.json
🧰 Additional context used
🧠 Learnings (1)
script/deploy/_targetState.json (1)
Learnt from: ezynda3
PR: lifinance/contracts#861
File: script/deploy/_targetState.json:1453-1483
Timestamp: 2024-11-21T08:24:53.059Z
Learning: In `script/deploy/_targetState.json`, for the `abstract` network configuration, `ReceiverStargateV2` is correctly set to version `1.0.1`.
⏰ Context from checks skipped due to timeout of 90000ms (3)
  • GitHub Check: run-unit-tests
  • GitHub Check: Analyze (javascript-typescript)
  • GitHub Check: generate-tag
🔇 Additional comments (28)
config/permit2Proxy.json (1)

19-19: New INK Network Entry Verification

The addition of the "ink" key with the address "0x000000000022D473030F116dDEE9F6B43aC78BA3" aligns with how other networks are configured in this file. Please verify that this address is correct, reflecting the actual permit2 proxy contract deployed on the INK network—especially since the same address is used for several networks.

deployments/_deployments_log_file.json (26)

805-819: Ink Deployment Record for Production Version "1.0.0" Added
A new deployment record under the "ink" key is introduced with the production configuration for version "1.0.0". All the required fields (ADDRESS, OPTIMIZER_RUNS, TIMESTAMP, CONSTRUCTOR_ARGS, SALT, and VERIFIED) are present. Please verify that the provided contract address is correctly checksummed and that the timestamp reflects the accurate deployment time.


1611-1626: Ink Deployment Log for OwnershipFacet Added
The new "ink" deployment entry (production, version "1.0.0") has been integrated under what appears to be the OwnershipFacet configuration. The structure and field values are consistent. Ensure that the ADDRESS and other deployment details match the intended deployment target for this facet.


2416-2427: Ink Record Integration for Current Facet
An ink deployment record for production version "1.0.0" with the address "0x42e4Ef53e8d905A5D02CC12b8C60D27800F76121" is added. The JSON structure is consistent, so please double-check that the deployment details align with the expected outcomes for this facet.


3523-3537: Ink Record for AccessManagerFacet Added
A new ink record under production with version "1.0.1" has been introduced. The entry follows the standard format. It is advisable to confirm that the version change (from “1.0.0” in other places to “1.0.1” here) is intended and that the deployment parameters (e.g., optimizer runs, timestamp) are correct.


4329-4343: Ink Entry for WithdrawFacet Added
This addition brings an ink deployment record for production version "1.0.0" under WithdrawFacet. The fields provided appear in the correct format. A quick verification of the ADDRESS and associated metadata is recommended to ensure consistency with other deployments.


5121-5135: Ink Deployment Log Update for PeripheryRegistryFacet
The addition of an ink record for production version "1.0.0" is detected. The record’s structure is compliant with the rest of the file. No issues were found, but please verify that the recorded details are aligned with your deployment tracking standards.


5926-5940: Ink Deployment Details for LiFiDiamond Added
A new ink deployment record has been added for production version "1.0.0" within the LiFiDiamond configuration. All the data fields appear correctly formatted; however, ensure that the deployment metadata such as the timestamp and the contract address meet your audit requirements.


6706-6720: New Ink Configuration for LiFiDiamondImmutable
This hunk introduces an ink deployment record for production version "1.0.0" under what seems to be LiFiDiamondImmutable. Note that the "CONSTRUCTOR_ARGS" field is non-trivial here; please validate that the concatenated data represents the correct initialization parameters.


8557-8571: Ink Record Inserted Before HopFacetOptimized
The new ink record with production version "1.0.0" is positioned just before the "HopFacetOptimized" key. This addition appears consistent with the recurring structure. Kindly ensure that the deployment details and address are accurate, in line with the other facets.


12089-12103: Ink Deployment Configuration for Executor Facet
The entry adds an ink deployment record for production version "1.1.0" complete with non-empty constructor arguments. It would be beneficial to verify that the provided initialization parameters are correct and conform to the intended deployment configuration.


13069-13083: Ink Deployment Record in FeeCollector Context Added
A new ink record for production version "2.1.0" is introduced with detailed and concatenated constructor arguments. Please double-check the long hex string in "CONSTRUCTOR_ARGS" to ensure its correctness, as this is critical during audits.


13850-13864: Ink Deployment Record for Receiver Added
An ink record with production version "1.0.1" is appended under the Receiver configuration. All necessary fields are provided and formatted correctly. Ensure that the contract address and deployment timestamp are validated against your records.


21676-21690: Ink Record for LiFuelFeeCollector Added
The deployment record for production version "1.2.0" under LiFuelFeeCollector is now included. The deployment metadata appears consistent; please verify that the ADDRESS provided is correctly cross-checked with other deployment records.


24161-24175: Ink Deployment Log Update for GasRebateDistributor
A new ink record for production version "1.1.0" is added under the GasRebateDistributor section. The structure and field-values are uniform with the rest of the records. Consider validating the length and format of the "CONSTRUCTOR_ARGS", if applicable.


24955-24969: Ink Deployment Record for StargateFacetV2 Added
A new ink deployment entry for production version "1.0.1" is incorporated. Please ensure that the deployment details, especially the address and version tag, are correctly synced with your deployment strategy.


25443-25457: Ink Entry Integration for ReceiverStargateV2
An ink record with production version "1.0.1" is added here. The record maintains consistency with other entries. If applicable, verify that this facet’s deployment parameters match the intended configuration.


25779-25793: Ink Deployment Record for LiFiDEXAggregator Added
This hunk introduces a new ink record for production version "1.1.0" with multiple concatenated parameters in "CONSTRUCTOR_ARGS". It is important to confirm that these parameters correctly initialize the contract as intended.


26413-26427: Ink Record Addition for EmergencyPauseFacet
The ink deployment record for production version "1.6.0" has been added, featuring comprehensive constructor arguments. Please review these parameters carefully to ensure they accurately reflect the intended deployment configuration for emergency pause functionality.


27051-27065: Ink Deployment Log Update for AcrossFacetV3
A new entry for production version "1.0.1" is now part of the AcrossFacetV3 deployment logs. The structure is standard; still, a verification of the hex values in the constructor arguments is recommended to catch any potential misconfiguration.


27439-27453: Ink Record for ReceiverAcrossV3 Added
This addition appends an ink deployment record for production version "1.1.0" under ReceiverAcrossV3. All fields are coordinated with the overall schema. Please ensure that the configuration parameters and contract address are accurate according to your deployment audits.


27808-27822: Ink Deployment Record for AcrossFacetPackedV3 Added
A new ink record for production version "1.1.0" is introduced with concatenated parameters in the "CONSTRUCTOR_ARGS". It is advisable to validate these parameters to ensure they correctly initialize the packed facet deployment.


28166-28180: Ink Deployment Addition for Permit2Proxy Added
The ink record for production version "1.2.0" under the Permit2Proxy configuration is added correctly. All required fields are present and well-formed. A final verification of the constructor arguments and address is recommended for absolute certainty.


28402-28416: Ink Record Insertion for GasZipFacet Added
A new ink deployment record for production version "1.0.2" under GasZipFacet is included. The standard field set is maintained. Please confirm that the contract address and other deployment details are verified.


28818-28832: Ink Deployment Log Update for GasZipPeriphery Added
The addition comprises an ink record for production version "2.0.2". All fields, such as TIMESTAMP and OPTIMIZER_RUNS, appear consistent with other entries. Ensure that these details reflect the intended gas optimization deployment for the ink network.


29224-29238: Ink Deployment Record for RelayFacet Added
A new ink record for production version "1.0.1" under RelayFacet is now present. The "CONSTRUCTOR_ARGS" appears as a complex hex string; please double-check that it correctly represents the initialization parameters and does not omit any necessary data.


29640-29654: Ink Deployment Record for GetGasFacet Added
The record for production version "1.0.0" in the GetGasFacet section has been added successfully. The deployment details are complete and follow the established format. A final verification of the contract address and parameters against internal deployment records is recommended.

script/deploy/_targetState.json (1)

468-468: Confirm Change for SquidFacet in Fantom

In the fantom network section, the "SquidFacet" entry is now marked with an update to "1.0.0". Please confirm that this update is deliberate and that it does not conflict with expected versions in other network configurations.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants