Skip to content

Conversation

@nazreen
Copy link
Contributor

@nazreen nazreen commented Nov 7, 2025

Problem

Running the wire task might result in

error:   [OApp] Failed to check send libraries for [CbUo5icVrxb1Gq9UhZKs7zQnzZxcx3PCWfKitwta4eiy @ SOLANA_V2_MAINNET] → [0x8e06d6df9880b641662D8e12a35c09c05c1e1CDe (BBOApp) @ BOUNCEBIT_V2_MAINNET]: Error: Failed to check the default send library for Solana program @ [76y77prsiCMvXMjuoZ5VRrhG5qYBrUMYTE5WgHqgjEn6 @ SOLANA_V2_MAINNET] for CbUo5icVrxb1Gq9UhZKs7zQnzZxcx3PCWfKitwta4eiy for BOUNCEBIT_V2_MAINNET: Error: Unable to find defaultSendLibraryConfig/sendLibraryConfig account at 8o69tuDaboy2Vu6NuLJx1joh8Ri4UMSUGEfxdvM2frB7/C53akiGwMxaaUk1a13UNLi8Aqeqyx4RmW7sUBRmagmyB

Diagnosis:

  • the sendLibraryConfig account was not initialized despite having run init-config prior to running the wire task

Background

For a Solana OApp wiring to work, there are two types of PDAs that are needed: Send/Receive Library PDAs (owned by Endpoint program) and ULN Config PDAs (owned by the ULN program)

#1645 fixed an issue that would appear during wiring where init-config would falsely report that all PDAs were initialized (bug introduced by a change upstream) when in fact they were not, resulting in an error when running the wire command. This PR however applied the fix only for oft-solana, but not for oapp-solana.

The fix involves updating sendConfigIsInitialized to also check send/library config accounts. We need the same for oapp-solana.

Changes

Instead of checking only for ULN config accounts, we should also check for send/receive library config accounts.

@nazreen nazreen marked this pull request as ready for review November 7, 2025 16:40
@nazreen nazreen changed the title fix (examples/oapp-solana): update message library PDAs and config PDAs checks DEVREL-916 fix (examples/oapp-solana): update message library PDAs and config PDAs checks Nov 11, 2025
@DanL0
Copy link
Contributor

DanL0 commented Nov 14, 2025

Testing:

LAYERZERO_EXAMPLES_REPOSITORY_REF=fix-oapp-solana-init-config LZ_ENABLE_SOLANA_OAPP_EXAMPLE=1 npx create-lz-oapp@latest

DanL0
DanL0 previously approved these changes Nov 14, 2025
Copy link
Contributor

@DanL0 DanL0 left a comment

Choose a reason for hiding this comment

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

@github-actions
Copy link
Contributor

🚨 E2E Tests Failed

The E2E tests failed during CI. These tests validate real blockchain interactions and may fail due to:

This is non-blocking and does not prevent merging. Check the action logs above for detailed failure information.

@github-actions
Copy link
Contributor

🚨 E2E Tests Failed

The E2E tests failed during CI. These tests validate real blockchain interactions and may fail due to:

This is non-blocking and does not prevent merging. Check the action logs above for detailed failure information.

@github-actions
Copy link
Contributor

github-actions bot commented Dec 4, 2025

🚨 E2E Tests Failed

The E2E tests failed during CI. These tests validate real blockchain interactions and may fail due to:

This is non-blocking and does not prevent merging. Check the action logs above for detailed failure information.

@github-actions
Copy link
Contributor

github-actions bot commented Dec 4, 2025

🚨 E2E Tests Failed

The E2E tests failed during CI. These tests validate real blockchain interactions and may fail due to:

This is non-blocking and does not prevent merging. Check the action logs above for detailed failure information.

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