Fix snapshot reference resolution order#182
Conversation
There was a problem hiding this comment.
Pull request overview
This PR adjusts snapshot reference handling in the configuration settings loader so snapshot-referenced settings are resolved and processed inline, preserving their intended position in the overall selector/setting precedence order.
Changes:
- Replace deferred snapshot reference collection + post-processing with inline snapshot resolution inside
processSettings. - Introduce
allowSnapshotRefto prevent nested snapshot reference resolution andsnapshotClientto lazily initialize/reuse an App Configuration client for snapshot reads. - Remove the now-unneeded
resolveSnapshotReferenceshelper and update in-code documentation to reflect the new behavior.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
@linglingye001 does it mean a user can disable snapshot reference resolution? Do we support the similar switch in other providers? |
No, |
Fix #180