[Plex 3240] Stellar relayer: consolidate config into config package#150
Open
Krish-vemula wants to merge 4 commits into
Open
[Plex 3240] Stellar relayer: consolidate config into config package#150Krish-vemula wants to merge 4 commits into
Krish-vemula wants to merge 4 commits into
Conversation
Soroban Contract Test Coverage92.71% line coverage — 18033 / 19451 lines hit
Per-Contract Breakdown
Full file-level coverage report |
Integration Test Coverage (excl. Token Pool) |
Integration Test Coverage (Token Pool) |
…ts in SetDefaults
…ocs.toml drift test
|
Code coverage report:
WARNING: go tool cover failed for coverage_target.out |
Krish-vemula
marked this pull request as ready for review
July 16, 2026 17:41
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.
Summary
Stellar relayer configuration was split across two packages: chain config in config and TXM config in
relayer/txm/config.go. This created an awkward dependency (config imported txm just for theConfigtype), two places to look for Stellar config, and no single source of truth for operator-facing defaults.This PR consolidates all Stellar config into config, adds a docs.toml source of truth for defaults, and fixes a two-phase resolution gap where
NewDecodedTOMLConfigresolved chain config but deferred TXMResolve()totxm.New.New files
relayer/txm/config.go)init(),Defaults(),SetFrommethods//go:embed docs.toml+GenerateDocs()ptrtest helper (production copy moved to config)