GitHub
10/21/2025, 10:26 PMTap.load_streams_from_catalog(), which should reconstruct list[Stream] from the input catalog.
This makes it clear whether the Tap is expected to perform discovery or is run with catalog.
By default, for compatibility, the new function calls Tap.load_streams(), so it doesn't introduce a breaking change.
---
This is rather trivial function but has some logic, so I could use some hint on what kind of testing would be expected.
## Summary by Sourcery
Add a new load_streams_from_catalog method to centralize catalog‐based stream reconstruction and update both the core Tap class and its template to use it without breaking existing behavior.
New Features:
• Introduce Tap.load_streams_from_catalog to reconstruct streams from an input catalog
• Update the streams property to delegate stream loading to the new method when a catalog is provided
• Embed a default load_streams_from_catalog stub in the tap template to yield streams by catalog entries
Enhancements:
• Maintain backward compatibility by having load_streams_from_catalog default to calling load_streams()
---
📚 Documentation preview 📚: https://meltano-sdk--3084.org.readthedocs.build/en/3084/
meltano/sdkGitHub
10/21/2025, 10:26 PM