Proposal: L1SenderV2 adapterParams maintenance — review @DavidAJohnston · exec Smart Agents · credit @rcondron

[EnergyHound]

Proposal for review: L1SenderV2 LayerZero adapterParams maintenance

Reviewer: @DavidAJohnston
Execution (when approved): Smart Agents via owner multisig
Diagnosis / payload: @rcondron
Author of this write-up: EnergyHound (docs only)

Decision requested from @DavidAJohnston: ACK / NACK (or ACK with preferred dest gas) on the owner config below.
This issue documents the maintenance item. Smart Agents execute after review; no contract upgrade.

---

Summary

What	Owner call: L1SenderV2.setLayerZeroConfig — set non-empty type-1 adapterParams
Why	Endpoint V1 default send library is SendUln301 (v4). Empty options now revert; ULNv2 used to inject 200k silently
Impact if not done	Ethereum MOR claim() paths that mint via LZ keep reverting
Funds	No loss — full L1 revert; only gas spent
Upgrade?	No
User self-fix?	No — params are contract state

---

Symptom

Since ~2026-08-05, L1 MOR reward claims that cross to Arbitrum via LayerZero fail.

Revert	0x6592671c…0000 = LZ_ULN_InvalidWorkerOptions(0)
DepositPool	0x47176B2Af9885dC6C4575d4eFd63895f7Aaa4790
Examples	0x8f42b692…c201 · 0x246674a2…86aa

Call path:
DepositPool.claim → DistributorV2 → L1SenderV2.sendMintMessage → Endpoint V1 send() → SendUln301 → revert.

---

Root cause

1. L1SenderV2 always passes layerZeroConfig.adapterParams into ILayerZeroEndpoint.send().
2. Live adapterParams are empty (length 0).
3. App never set its own send version → follows Endpoint default send version 4 (SendUln301).
4. UltraLightNodeV2 accepted empty params and defaulted dest gas to 200k type-1.
5. SendUln301 does not: UlnOptions.decode reverts if options.length < 2.

Check	Value
L1SenderV2	0x2Efd4430489e1a05A89c2f51811aC661B7E5FF84 (layerZeroConfig / #F3 — surface @rcondron flagged)
Owner	0x1FE04BC15Cf2c5A2d41a0b3a96725596676eBa1E (Safe; threshold 5)
Live routing	gateway Endpoint V1 · receiver L2MessageReceiver Arb · chainId 110 · zro 0x0 — leave unchanged
Last success (ULNv2)	0x2f73519b…8314 (block 25678531)

Quote check: estimateFees with empty options → same revert; with type-1 34-byte 200k → ~0.000048 ETH.

---

Proposed maintenance (@rcondron)

Keep all live routing fields. Write type-1 adapter params only:

// onlyOwner — L1SenderV2
setLayerZeroConfig({
  gateway:           0x66A71Dcef29A0fFBDBE3c6a460a3B5BC225Cd675, // Endpoint V1 (ETH)
  receiver:          0xd4a8ECcBe696295e68572A98b1aa70Aa9277d427, // L2MessageReceiver (Arb)
  receiverChainId:   110,
  zroPaymentAddress: address(0),
  adapterParams:     /* type-1, exactly 34 bytes, 200_000 gas */
});

adapterParams (recommended: 200k = historical ULNv2 default):

0x00010000000000000000000000000000000000000000000000000000000000030d40

ethers.utils.solidityPack(['uint16', 'uint256'], [1, 200000])

Detail
Encoding rule	Type-1 must be exactly 34 bytes or ULN301 reverts InvalidLegacyType1Option
Fees	~0.000048 ETH at 200k; frontend typically 0.0005–0.002 ETH (excess refunded)
Optional	250000 → …03d090 (safer dest gas; ~0.000001 ETH more). Can re-run maintenance later; only affects new messages
Scope	All sendMintMessage paths (standard / referrer / batch claims)
Not an alternative without upgrade	Pin old send library — L1SenderV2 has no setSendVersion

Calldata (200k) for Safe / simulation:

To:    0x2Efd4430489e1a05A89c2f51811aC661B7E5FF84
Value: 0
Data:
0x73ff24fd000000000000000000000000000000000000000000000000000000000000002000000000000000000000000066a71dcef29a0ffbdbe3c6a460a3b5bc225cd675000000000000000000000000d4a8eccbe696295e68572a98b1aa70aa9277d427000000000000000000000000000000000000000000000000000000000000006e000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000000a0000000000000000000000000000000000000000000000000000000000000002200010000000000000000000000000000000000000000000000000000000000030d40000000000000000000000000000000000000000000000000000000000000

---

LayerZero references

V1 adapter parameters (type-1 packing, 200k example)	https://docs.layerzero.network/v1/developers/evm/evm-guides/advanced/relayer-adapter-parameters
Error codes (InvalidWorkerOptions, legacy type-1)	https://docs.layerzero.network/v2/developers/evm/troubleshooting/error-messages
L1SenderV2 config (read)	https://etherscan.io/address/0x2Efd4430489e1a05A89c2f51811aC661B7E5FF84#readProxyContract#F3

---

Checklist

@DavidAJohnston — review

• Agree root cause (empty adapterParams under SendUln301)
• Agree payload: keep live gateway/receiver/110/zro; type-1 200k (or state 250k)
• ACK / NACK in a comment

Smart Agents — after ACK

• Simulate owner tx (Safe / Tenderly)
• Execute setLayerZeroConfig from owner multisig
• Re-read layerZeroConfig() — adapterParams length 34
• Canary claim → confirm MOR mint on Arbitrum
• Optional: scan other Endpoint-V1 OApps for empty adapterParams

---

Decision line

Propose: ACK type-1 200k as specified by @rcondron; Smart Agents execute owner maintenance.

@DavidAJohnston — please reply ACK, ACK + prefer 250k, or NACK + reason.

[EnergyHound]

Ping: Smart Agents

This issue is documentation only for the L1SenderV2 adapterParams owner maintenance (diagnosis/proposal credit @rcondron).

Execution is on Smart Agents when you pick it up — not an action item for other reviewers from this thread. @DavidAJohnston for visibility.

Payload, L0 refs, and checklist are in the issue body above.

[EnergyHound]

Review request — @DavidAJohnston

Finished proposal is in the issue body (symptom, root cause, @rcondron payload, L0 refs, Safe calldata, checklists).

Role
Review / ACK	@DavidAJohnston
Execute after ACK	Smart Agents (owner multisig)
Credit	@rcondron

Please reply ACK, ACK + prefer 250k, or NACK + reason. Docs only from this thread until then.

[EnergyHound]

Update — EXECUTED by protocol owner multisig

On-chain state now matches the maintenance payload (safer gas variant).

Safe nonce	217
Executed	2026-08-11 ~03:54 UTC
Tx	https://etherscan.io/tx/0xacdb78b2c20e10280a164ee17df6af994d910441cbcdd9eff5fceecf57769205
Safe tx	0x7950f774d39a609d2c34cbcb1ac2dcda848ef2984eb849b7e3bd8848bac6b1aa
adapterParams	type-1, exactly 34 bytes, gas 250_000 (…03d090) — not empty
Routing fields	gateway / receiver / chainId 110 / zro unchanged

Live check: layerZeroConfig() returns non-empty type-1 params; estimateFees with on-chain options succeeds (~0.000049 ETH). Empty options still revert (as expected under SendUln301).

Note: Ryan’s write-up used 200k as the historical ULNv2 default; the multisig shipped the 250k optional safer dest-gas value. Same fix class.

Claims path should be unblocked for new mints. Canary claim + Arbitrum delivery still recommended as post-exec hygiene.