Use __all__ instead of import by name

python/lsst/daf/butler/remote_butler/__init__.py

@@ -25,4 +25,4 @@
 # You should have received a copy of the GNU General Public License
 # along with this program.  If not, see <http://www.gnu.org/licenses/>.
 
-from ._remote_butler import RemoteButler
+from ._remote_butler import *

python/lsst/daf/butler/remote_butler/_remote_butler.py

@@ -25,6 +25,8 @@
 # You should have received a copy of the GNU General Public License
 # along with this program.  If not, see <http://www.gnu.org/licenses/>.
 
+__all__ = ("RemoteButler",)
+
 from collections.abc import Collection, Iterable, Sequence
 from contextlib import AbstractContextManager
 from typing import Any, TextIO

python/lsst/daf/butler/remote_butler/server/__init__.py

@@ -25,5 +25,5 @@
 # You should have received a copy of the GNU General Public License
 # along with this program.  If not, see <http://www.gnu.org/licenses/>.
 
-from ._factory import Factory
-from ._server import app, factory_dependency
+from ._factory import *
+from ._server import *

python/lsst/daf/butler/remote_butler/server/_factory.py

@@ -27,6 +27,8 @@
 
 from lsst.daf.butler import Butler
 
+__all__ = ("Factory",)
+
 
 class Factory:
     def __init__(self, *, butler: Butler):

python/lsst/daf/butler/remote_butler/server/_server.py

@@ -27,7 +27,7 @@
 
 from __future__ import annotations
 
-__all__ = ()
+__all__ = ("app", "factory_dependency")
 
 import logging
 from functools import cache