@bananapus/omnichain-deployers-v6 0.0.6 → 0.0.7

package/STYLE_GUIDE.md

@@ -371,6 +371,8 @@ jobs:
         uses: foundry-rs/foundry-toolchain@v1
       - name: Run tests
         run: forge test --fail-fast --summary --detailed --skip "*/script/**"
+        env:
+          RPC_ETHEREUM_MAINNET: ${{ secrets.RPC_ETHEREUM_MAINNET }}
       - name: Check contract sizes
         run: FOUNDRY_PROFILE=ci_sizes forge build --sizes --skip "*/test/**" --skip "*/script/**" --skip SphinxUtils
 ```
@@ -435,6 +437,15 @@ Run `forge fmt` before committing. The `[fmt]` config in `foundry.toml` enforces
 
 CI checks formatting via `forge fmt --check`.
 
+### CI Secrets
+
+| Secret | Purpose |
+|--------|--------|
+| `NPM_TOKEN` | npm publish access (used by `publish.yml`) |
+| `RPC_ETHEREUM_MAINNET` | Ethereum mainnet RPC URL for fork tests (used by `test.yml`) |
+
+Fork tests require `RPC_ETHEREUM_MAINNET` — they fail if it's missing.
+
 ### Branching
 
 - `main` is the primary branch

package/foundry.toml

@@ -6,9 +6,6 @@ optimizer_runs = 200
 libs = ["node_modules", "lib"]
 fs_permissions = [{ access = "read-write", path = "./"}]
 
-[profile.ci_sizes]
-optimizer_runs = 200
-
 [fuzz]
 runs = 4096
 

package/package.json

@@ -1,6 +1,6 @@
 {
     "name": "@bananapus/omnichain-deployers-v6",
-    "version": "0.0.6",
+    "version": "0.0.7",
     "license": "MIT",
     "repository": {
         "type": "git",
@@ -18,12 +18,12 @@
     },
     "dependencies": {
         "@bananapus/721-hook-v6": "^0.0.13",
-        "@bananapus/buyback-hook-v6": "^0.0.9",
-        "@bananapus/core-v6": "^0.0.13",
+        "@bananapus/buyback-hook-v6": "^0.0.10",
+        "@bananapus/core-v6": "^0.0.14",
         "@bananapus/ownable-v6": "^0.0.7",
         "@bananapus/permission-ids-v6": "^0.0.6",
         "@bananapus/suckers-v6": "^0.0.8",
-        "@openzeppelin/contracts": "^5.2.0",
+        "@openzeppelin/contracts": "^5.6.1",
         "@uniswap/v4-core": "^1.0.2"
     },
     "devDependencies": {

package/src/JBOmnichainDeployer.sol

@@ -209,6 +209,7 @@ contract JBOmnichainDeployer is
         bool usesTiered721Hook = address(tiered721Hook) != address(0);
         if (usesTiered721Hook) {
             JBPayHookSpecification[] memory tiered721HookSpecs;
+            // slither-disable-next-line unused-return
             (, tiered721HookSpecs) = IJBRulesetDataHook(address(tiered721Hook)).beforePayRecordedWith(context);
             // The 721 hook returns a single spec (itself) whose amount is the total split amount.
             if (tiered721HookSpecs.length > 0) {
@@ -386,6 +387,7 @@ contract JBOmnichainDeployer is
         });
 
         // Store the 721 hook separately from the custom data hook.
+        // slither-disable-next-line reentrancy-benign
         tiered721HookOf[projectId] = hook;
 
         // Launch the project, and sanity check the project ID.
@@ -469,6 +471,7 @@ contract JBOmnichainDeployer is
         });
 
         // Store the 721 hook separately from the custom data hook.
+        // slither-disable-next-line reentrancy-benign
         tiered721HookOf[projectId] = hook;
 
         // Transfer the hook's ownership to the project.
@@ -644,6 +647,7 @@ contract JBOmnichainDeployer is
         });
 
         // Store the 721 hook separately from the custom data hook.
+        // slither-disable-next-line reentrancy-benign
         tiered721HookOf[projectId] = hook;
 
         // Transfer the hook's ownership to the project.