rm functions - #524
Merged
Merged
Conversation
Contributor
|
RensR
marked this pull request as ready for review
August 14, 2026 11:38
product-security-plaid-production
Bot
requested review from
MStreet3,
elatoskinas,
erikburt,
kalverra and
timothyF95
August 14, 2026 11:38
pavel-raykov
approved these changes
Aug 14, 2026
Contributor
There was a problem hiding this comment.
Pull request overview
This PR removes the Chainlink Functions Solidity code and related build/test plumbing from this repository, based on the note that Functions is now handled as part of CRE, and updates VRF tests to use VRF-local test helpers instead of Functions test fixtures.
Changes:
- Removed the
contracts/src/v0.8/functionsSolidity sources and the associated Foundry profile, scripts, snapshots, and CI matrix entries. - Updated VRF test imports to use
contracts/src/v0.8/vrf/testhelpers/MockLinkToken.soland fixed that helper’sIERC677Receiverimport path. - Removed geth wrapper generation entrypoints for Functions.
Reviewed changes
Copilot reviewed 72 out of 72 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| gethwrappers/go_generate.go | Stops running wrapper generation for the removed Functions package. |
| gethwrappers/functions/go_generate.go | Removes Functions wrapper generation directives. |
| contracts/src/v0.8/vrf/testhelpers/MockLinkToken.sol | Fixes IERC677Receiver import path to align with VRF-local usage. |
| contracts/src/v0.8/vrf/test/VRFV2PlusWrapper.t.sol | Switches MockLinkToken import from Functions fixtures to VRF testhelpers. |
| contracts/src/v0.8/vrf/test/VRFV2PlusWrapper_Optimism.t.sol | Switches MockLinkToken import from Functions fixtures to VRF testhelpers. |
| contracts/src/v0.8/vrf/test/VRFV2PlusWrapper_Migration.t.sol | Switches MockLinkToken import from Functions fixtures to VRF testhelpers. |
| contracts/src/v0.8/vrf/test/VRFV2PlusWrapper_Arbitrum.t.sol | Switches MockLinkToken import from Functions fixtures to VRF testhelpers. |
| contracts/src/v0.8/vrf/test/VRFV2PlusSubscriptionAPI.t.sol | Switches MockLinkToken import from Functions fixtures to VRF testhelpers. |
| contracts/src/v0.8/vrf/test/VRFV2Plus.t.sol | Switches MockLinkToken import from Functions fixtures to VRF testhelpers. |
| contracts/src/v0.8/vrf/test/VRFCoordinatorV2Plus_Migration.t.sol | Switches MockLinkToken import from Functions fixtures to VRF testhelpers. |
| contracts/src/v0.8/vrf/test/VRFCoordinatorV2Mock.t.sol | Switches MockLinkToken import from Functions fixtures to VRF testhelpers. |
| contracts/src/v0.8/vrf/test/VRFCoordinatorV2_5Mock.t.sol | Switches MockLinkToken import from Functions fixtures to VRF testhelpers. |
| contracts/src/v0.8/vrf/test/VRFCoordinatorV2_5_Optimism.t.sol | Switches MockLinkToken import from Functions fixtures to VRF testhelpers. |
| contracts/src/v0.8/vrf/test/VRFCoordinatorV2_5_Arbitrum.t.sol | Switches MockLinkToken import from Functions fixtures to VRF testhelpers. |
| contracts/src/v0.8/vrf/test/FixtureVRFCoordinatorV2_5.t.sol | Switches MockLinkToken import from Functions fixtures to VRF testhelpers. |
| contracts/src/v0.8/functions/v1_3_1/Routable.sol | Removes Functions v1.3.1 source (migrated out of this repo). |
| contracts/src/v0.8/functions/v1_3_1/ocr/OCR2Base.sol | Removes Functions v1.3.1 source (migrated out of this repo). |
| contracts/src/v0.8/functions/v1_3_1/ocr/OCR2Abstract.sol | Removes Functions v1.3.1 source (migrated out of this repo). |
| contracts/src/v0.8/functions/v1_3_1/mocks/FunctionsV1EventsMock.sol | Removes Functions v1.3.1 source (migrated out of this repo). |
| contracts/src/v0.8/functions/v1_3_1/libraries/FunctionsResponse.sol | Removes Functions v1.3.1 source (migrated out of this repo). |
| contracts/src/v0.8/functions/v1_3_1/libraries/FunctionsRequest.sol | Removes Functions v1.3.1 source (migrated out of this repo). |
| contracts/src/v0.8/functions/v1_3_1/libraries/ChainSpecificUtil.sol | Removes Functions v1.3.1 source (migrated out of this repo). |
| contracts/src/v0.8/functions/v1_3_1/interfaces/IOwnableFunctionsRouter.sol | Removes Functions v1.3.1 source (migrated out of this repo). |
| contracts/src/v0.8/functions/v1_3_1/interfaces/IFunctionsSubscriptions.sol | Removes Functions v1.3.1 source (migrated out of this repo). |
| contracts/src/v0.8/functions/v1_3_1/interfaces/IFunctionsRouter.sol | Removes Functions v1.3.1 source (migrated out of this repo). |
| contracts/src/v0.8/functions/v1_3_1/interfaces/IFunctionsCoordinator.sol | Removes Functions v1.3.1 source (migrated out of this repo). |
| contracts/src/v0.8/functions/v1_3_1/interfaces/IFunctionsClient.sol | Removes Functions v1.3.1 source (migrated out of this repo). |
| contracts/src/v0.8/functions/v1_3_1/interfaces/IFunctionsBilling.sol | Removes Functions v1.3.1 source (migrated out of this repo). |
| contracts/src/v0.8/functions/v1_3_1/FunctionsSubscriptions.sol | Removes Functions v1.3.1 source (migrated out of this repo). |
| contracts/src/v0.8/functions/v1_3_1/FunctionsRouter.sol | Removes Functions v1.3.1 source (migrated out of this repo). |
| contracts/src/v0.8/functions/v1_3_1/FunctionsCoordinator.sol | Removes Functions v1.3.1 source (migrated out of this repo). |
| contracts/src/v0.8/functions/v1_3_1/FunctionsClient.sol | Removes Functions v1.3.1 source (migrated out of this repo). |
| contracts/src/v0.8/functions/v1_3_1/FunctionsBilling.sol | Removes Functions v1.3.1 source (migrated out of this repo). |
| contracts/src/v0.8/functions/v1_3_1/example/FunctionsClientExample.sol | Removes Functions v1.3.1 source (migrated out of this repo). |
| contracts/src/v0.8/functions/v1_3_1/accessControl/TermsOfServiceAllowList.sol | Removes Functions v1.3.1 source (migrated out of this repo). |
| contracts/src/v0.8/functions/v1_3_1/accessControl/interfaces/ITermsOfServiceAllowList.sol | Removes Functions v1.3.1 source (migrated out of this repo). |
| contracts/src/v0.8/functions/v1_3_0_zksync/ZKSyncFunctionsRouter.sol | Removes zkSync-specific Functions router source (migrated out of this repo). |
| contracts/src/v0.8/functions/tests/v1_3_1/ZKSyncFunctionsRouter.t.sol | Removes Functions tests (migrated out of this repo). |
| contracts/src/v0.8/functions/tests/v1_3_1/testhelpers/ZKSyncFunctionsRouterHarness.sol | Removes Functions tests (migrated out of this repo). |
| contracts/src/v0.8/functions/tests/v1_3_1/testhelpers/FunctionsTestHelper.sol | Removes Functions tests (migrated out of this repo). |
| contracts/src/v0.8/functions/tests/v1_3_1/testhelpers/FunctionsSubscriptionsHarness.sol | Removes Functions tests (migrated out of this repo). |
| contracts/src/v0.8/functions/tests/v1_3_1/testhelpers/FunctionsRouterHarness.sol | Removes Functions tests (migrated out of this repo). |
| contracts/src/v0.8/functions/tests/v1_3_1/testhelpers/FunctionsLoadTestClient.sol | Removes Functions tests (migrated out of this repo). |
| contracts/src/v0.8/functions/tests/v1_3_1/testhelpers/FunctionsCoordinatorTestHelper.sol | Removes Functions tests (migrated out of this repo). |
| contracts/src/v0.8/functions/tests/v1_3_1/testhelpers/FunctionsCoordinatorHarness.sol | Removes Functions tests (migrated out of this repo). |
| contracts/src/v0.8/functions/tests/v1_3_1/testhelpers/FunctionsClientWithEmptyCallback.sol | Removes Functions tests (migrated out of this repo). |
| contracts/src/v0.8/functions/tests/v1_3_1/testhelpers/FunctionsClientUpgradeHelper.sol | Removes Functions tests (migrated out of this repo). |
| contracts/src/v0.8/functions/tests/v1_3_1/testhelpers/FunctionsClientTestHelper.sol | Removes Functions tests (migrated out of this repo). |
| contracts/src/v0.8/functions/tests/v1_3_1/testhelpers/FunctionsClientHarness.sol | Removes Functions tests (migrated out of this repo). |
| contracts/src/v0.8/functions/tests/v1_3_1/README.md | Removes Functions test documentation (migrated out of this repo). |
| contracts/src/v0.8/functions/tests/v1_3_1/OCR2.t.sol | Removes Functions tests (migrated out of this repo). |
| contracts/src/v0.8/functions/tests/v1_3_1/Gas.t.sol | Removes Functions tests (migrated out of this repo). |
| contracts/src/v0.8/functions/tests/v1_3_1/FunctionsRequest.t.sol | Removes Functions tests (migrated out of this repo). |
| contracts/src/v0.8/functions/tests/v1_3_1/FunctionsCoordinator.t.sol | Removes Functions tests (migrated out of this repo). |
| contracts/src/v0.8/functions/tests/v1_3_1/FunctionsClient.t.sol | Removes Functions tests (migrated out of this repo). |
| contracts/src/v0.8/functions/tests/v1_3_1/FunctionsBilling.t.sol | Removes Functions tests (migrated out of this repo). |
| contracts/src/v0.8/functions/tests/v1_3_1/ChainSpecificUtil.t.sol | Removes Functions tests (migrated out of this repo). |
| contracts/src/v0.8/functions/tests/v1_3_1/BaseTest.t.sol | Removes Functions tests (migrated out of this repo). |
| contracts/scripts/native_solc_compile_all_functions | Removes Functions native solc compile helper script. |
| contracts/scripts/native_solc_compile_all | Removes functions from the list of compiled products. |
| contracts/GNUmakefile | Removes functions from Foundry product list. |
| contracts/gas-snapshots/functions.gas-snapshot | Removes Functions gas snapshot baseline. |
| contracts/foundry.toml | Removes the Functions Foundry profile. |
| contracts/.solhintignore-test | Stops ignoring the (now removed) Functions tree in test solhint ignores. |
| contracts/.solhintignore | Stops ignoring Functions paths (now removed). |
| .github/workflows/solidity-foundry.yml | Removes Functions from the Foundry CI matrix and path filters. |
| .github/workflows/solidity-foundry-artifacts.yml | Removes Functions from artifacts workflow inputs / coverage gating. |
| .github/CODEOWNERS | Removes Functions-specific ownership entries. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
RensR
enabled auto-merge (squash)
August 14, 2026 11:43
ibrajer
approved these changes
Aug 14, 2026
jmank88
approved these changes
Aug 14, 2026
RayXpub
approved these changes
Aug 14, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Functions is now rolled into CRE