Skip to content

Revert "v30 interop OZ audit, M-04"#1932

Merged
0xValera merged 1 commit into
kl/medium-interopfrom
revert-1811-vg/v30-oz-m04
Dec 16, 2025
Merged

Revert "v30 interop OZ audit, M-04"#1932
0xValera merged 1 commit into
kl/medium-interopfrom
revert-1811-vg/v30-oz-m04

Conversation

@0xValera

@0xValera 0xValera commented Dec 16, 2025

Copy link
Copy Markdown
Collaborator

Reverts #1811. It was merged in kl/medium-interop instead of kl/v30-audit-base by accident.

@0xValera
0xValera marked this pull request as ready for review December 16, 2025 14:38
@0xValera
0xValera merged commit 37897b7 into kl/medium-interop Dec 16, 2025
15 of 16 checks passed
@0xValera
0xValera deleted the revert-1811-vg/v30-oz-m04 branch December 16, 2025 14:39
@github-actions

Copy link
Copy Markdown
Contributor

Coverage after merging revert-1811-vg/v30-oz-m04 into kl/medium-interop will be

84.83%

Coverage Report
FileStmtsBranchesFuncsLinesUncovered Lines
contracts/bridge
   BridgeHelper.sol100%100%100%100%
   BridgedStandardERC20.sol96.25%100%92.31%97.01%231–232
   L1ERC20Bridge.sol97.78%100%100%97.30%261
   L1Nullifier.sol88.44%100%87.10%88.69%129–130, 182, 191, 432–433, 436, 462, 647, 649, 698, 710, 713, 715, 731, 745–746, 750–751
   UpgradeableBeaconDeployer.sol0%100%0%0%15–16, 18, 20–21
contracts/bridge/asset-router
   AssetRouterBase.sol85.90%100%70.59%90.16%128, 200, 310, 49, 58, 81
   L1AssetRouter.sol91.62%100%86.67%92.70%104, 319, 330, 412–413, 432, 590, 601, 615, 620
contracts/bridge/asset-tracker
   AssetTrackerBase.sol78.38%100%66.67%84%113, 142, 59, 84
   GWAssetTracker.sol80.92%100%77.14%81.50%101–103, 109, 113, 116–117, 124–125, 136–137, 219, 229, 236, 238, 240, 242–244, 260, 305, 309, 345, 472–473, 475, 481, 485, 489, 495–496, 500, 519–522, 609–610, 78, 97–99
   L1AssetTracker.sol97.89%100%100%97.52%226, 404, 61
   LegacySharedBridgeAddresses.sol0%100%0%0%23, 26, 29, 32, 36–41, 43
contracts/bridge/interfaces
   AssetHandlerModifiers.sol75%100%100%66.67%13
contracts/bridge/ntv
   L1NativeTokenVault.sol73.83%100%77.27%72.94%101–103, 150, 154–156, 179–185, 187, 256–257, 259, 270, 272, 278, 92–93
   NativeTokenVaultBase.sol82.08%100%66.67%85.88%136–139, 141–142, 144, 219, 310, 46, 48, 494, 496, 50, 52, 564, 601, 611–612, 616–617, 88, 93, 98
contracts/common
   MessageVerification.sol94.44%100%88.89%96.30%88
   ReentrancyGuard.sol95%100%100%94.12%79
contracts/common/l2-helpers
   L2ContractHelper.sol94.34%100%100%93.33%102, 108, 112
   SystemContractsCaller.sol52.50%100%60%51.43%44–45, 47, 49, 51, 53, 66, 69, 72, 75, 78, 83, 89, 91, 93, 96, 98
contracts/common/libraries
   DataEncoding.sol82.14%100%78.26%83.15%129–130, 137–138, 183, 197, 225, 233–234, 245, 259, 270, 288, 294, 50
   DynamicIncrementalMerkle.sol81.03%100%85.71%80.39%143–146, 148–151, 153–154
   DynamicIncrementalMerkleMemory.sol98.96%100%100%98.84%196
   FullMerkle.sol100%100%100%100%
   FullMerkleMemory.sol85.86%100%100%84.78%114, 131, 149, 163, 89–98
   Merkle.sol98.15%100%100%97.96%83
   MessageHashing.sol96%100%100%95.38%118, 135, 154
   SemVer.sol100%100%100%100%
   UncheckedMath.sol100%100%100%100%
   UnsafeBytes.sol100%100%100%100%
   ZKSyncOSBytecodeInfo.sol0%100%0%0%17, 22, 30, 33
contracts/common/libraries/TransientPrimitives
   TransientPrimitives.sol100%100%100%100%
contracts/core/bridgehub
   BridgehubBase.sol88.83%100%88.89%88.82%134, 141, 159, 189, 207, 261, 264, 283, 299, 330, 38, 40, 42, 490, 493, 552, 556, 559
   L1Bridgehub.sol90%100%91.67%89.77%207, 282, 286–287, 290, 300, 66–67, 90
   L2Bridgehub.sol47.83%100%40%50%104–105, 113, 115–116, 125, 130–131, 133–134, 46, 48–51, 77, 90–91
contracts/core/chain-asset-handler
   ChainAssetHandlerBase.sol79.12%100%75%80%129–130, 180, 183, 195–196, 250, 267, 44, 46, 48, 50, 89, 96–97
   L1ChainAssetHandler.sol92.68%100%88.89%93.75%68–69
   L2ChainAssetHandler.sol88.24%100%87.50%88.46%121, 70, 94
contracts/core/chain-registration
   ChainRegistrationSender.sol88.24%100%100%85.19%41, 85, 89, 95
contracts/core/ctm-deployment
   CTMDeploymentTracker.sol73.47%100%81.82%71.05%108, 112, 116, 141, 145–146, 34, 42, 67, 80–81
contracts/core/message-root
   L1MessageRoot.sol67.44%100%75%65.71%104, 108–109, 65–66, 72–73, 81, 85, 89, 91–92
   L2MessageRoot.sol60%100%44.44%64.52%39–40, 45–46, 50, 57, 62–63, 68–70
   MessageRootBase.sol82.30%100%76.19%83.70%113, 117, 123–125, 131–133, 196, 212, 285, 314, 35, 38, 40
contracts/governance
   AccessControlRestriction.sol100%100%100%100%
   ChainAdmin.sol97.87%100%100%97.30%39
   ChainAdminOwnable.sol44.83%100%40%45.83%28, 38–40, 46–48, 57, 66, 77–79, 81
   Governance.sol100%100%100%100%
   L2ProxyAdminDeployer.sol0%100%0%0%16–18, 20
   PermanentRestriction.sol88.80%100%100%87.39%104, 111–112, 200–201, 204–205, 208, 210–211, 240, 288, 310, 341
   ServerNotifier.sol93.10%100%100%90.91%55, 65
   TransitionaryOwner.sol0%100%0%0%16–17, 21–23
contracts/governance/restriction
   Restriction.sol60%100%50%66.67%21
   RestrictionValidator.sol100%100%100%100%
contracts/interop
   AttributesDecoder.sol100%100%100%100%
   InteropCenter.sol93.25%100%89.47%93.75%114, 202, 427, 501, 54–55, 59–60, 72
   InteropDataEncoding.sol100%100%100%100%
   InteropHandler.sol90.91%100%81.82%91.92%33–35, 377, 399, 404, 41–42
   L2MessageVerification.sol90%100%100%88.89%35
contracts/l2-upgrades
   L2ComplexUpgrader.sol0%100%0%0%23–25, 39, 44, 46, 56, 62–63, 70, 79–81, 84,

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.

1 participant