Skip to content

Conversation

H00N24
Copy link
Collaborator

@H00N24 H00N24 commented Aug 28, 2025

🚀 Pull Request Overview

This PR adds a hash of Solc versions for the cache key generation and adds restore keys for backward compatibility. This change helps with refreshing the cache on Solc version changes.


📜 Tests Checklist

Before submitting this PR, ensure that all tests pass and meet the following conditions:

Category Status Compilation Comment Results Review
fail_to_start ❌ Failed with compilation error 🟢 Yes 🔴 No
violated_rules ✅ Passed, with violations found 🔴 No 🟢 Yes
verified_rules ✅ Passed without violations 🔴 No 🟢 Yes
solana_violated_rules ✅ Passed, with violations found 🔴 No 🟢 Yes

Please verify that your changes meet the above conditions for each category of tests.
If something doesn't match, investigate before submitting the PR.


@H00N24 H00N24 requested review from a team and Copilot August 28, 2025 09:28
Copy link

Certora Run Started (Fail to Start)

  • Group ID: ab007407-ec34-4499-9383-ecfbdbcddb0f
Config Status Link Log File
conf-start-error.conf Failed (1) - tests/evm/conf-start-error.conf-22473b9a6efe.log
conf-start-error.conf --method "counter()" Failed (1) - tests/evm/conf-start-error.conf-df011487dc52.log
conf-verified.conf Compiled - tests/evm/conf-verified.conf-c6b9b2a4fbda.log
conf-verified.conf --rule monotone --method "counter()" Compiled - tests/evm/conf-verified.conf-8c42189dceed.log
conf-violations.conf --method "counter()" Compiled - tests/evm/conf-violations.conf-47d7c895d17d.log
conf-violations.conf --rule invertible Compiled - tests/evm/conf-violations.conf-42c3c1e3df05.log

Certora Run Summary

  • Started 0 jobs
  • 2 jobs failed

Download Logs

Copy link
Contributor

@Copilot 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 enhances the Solidity compiler binary caching mechanism by incorporating Solc version hashes into the cache key generation. This ensures that the cache is properly invalidated when Solc versions change, while maintaining backward compatibility through restore keys.

  • Generates a hash-based cache key from the Solc versions input to ensure cache refreshes on version changes
  • Adds conditional caching logic that only applies when Solc versions are specified
  • Implements restore keys for backward compatibility with existing cache entries

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link

@certora-run certora-run bot left a comment

Choose a reason for hiding this comment

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

Verification Results

  • Group ID: 1e700aad-4f63-4817-b7bc-5e0ce2a268ef
Job Result VERIFIED Link
conf-verified.conf --rule monotone --method "counter()" 2 Link
conf-verified.conf --rule invertible 2 Link
conf-verified.conf --method "counter()" 3 Link
conf-verified.conf 3 Link

Copy link

@certora-run certora-run bot left a comment

Choose a reason for hiding this comment

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

Verification Results

  • Group ID: 34521df0-ec2c-4473-af65-87169afd864b
Job Result SANITY_FAILED VIOLATED Link
Default.conf 2 1 Link

Copy link

@certora-run certora-run bot left a comment

Choose a reason for hiding this comment

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

Verification Results

  • Group ID: 335e5c53-fe65-4386-a72b-b6c5460b759b
Job Result VERIFIED VIOLATED Link
conf-violations.conf --rule invertible 2 0 Link
conf-violations.conf 2 1 Link
conf-verified.conf --method "counter()" 3 0 Link
conf-verified.conf 3 0 Link

@H00N24 H00N24 merged commit 11979c6 into main Aug 28, 2025
6 of 8 checks passed
@H00N24 H00N24 deleted the feat/solc-cache-key branch August 28, 2025 10:28
Copy link

Certora Run Started (Fail to Start)

  • Group ID: 52d2a38e-3581-4dbc-9480-16ab40607220
Config Status Link Log File
conf-start-error.conf Failed (1) - tests/evm/conf-start-error.conf-d1ced5d90e42.log
conf-start-error.conf --method "counter()" Failed (1) - tests/evm/conf-start-error.conf-4f785c9bc13e.log
conf-verified.conf Compiled - tests/evm/conf-verified.conf-0441efd5fd2e.log
conf-verified.conf --rule monotone --method "counter()" Compiled - tests/evm/conf-verified.conf-2a547117b420.log
conf-violations.conf --method "counter()" Compiled - tests/evm/conf-violations.conf-22a41298178e.log
conf-violations.conf --rule invertible Compiled - tests/evm/conf-violations.conf-6054a73b4e90.log

Certora Run Summary

  • Started 0 jobs
  • 2 jobs failed

Download Logs

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.

2 participants