fix(models): auto-sync only on api commands (#277)

Similar fix was recently made in the programs API in #276

Author: wpbonelli

modflow_devtools/models/__init__.py

@@ -1300,12 +1300,9 @@ def path(self) -> Path:
 
 def _try_best_effort_sync():
     """
-    Attempt to sync registries on first import.
+    Attempt to sync registries (best-effort, fails silently).
 
-    This is a best-effort operation - if it fails (network issues,
-    misconfiguration, etc.), we silently continue. The user will get
-    a clear error message if they try to use the registry without
-    having synced successfully.
+    Called by consumer commands before accessing model registries.
     """
     global _SYNC_ATTEMPTED
 
@@ -1323,10 +1320,6 @@ def _try_best_effort_sync():
         pass
 
 
-# Try to sync on first import (unless disabled)
-if not os.environ.get("MODFLOW_DEVTOOLS_NO_AUTO_SYNC"):
-    _try_best_effort_sync()
-
 # Lazy initialization of default registry
 _default_registry_cache = None
 

modflow_devtools/models/__main__.py

@@ -8,11 +8,13 @@
 """
 
 import argparse
+import os
 import sys
 
 from . import (
     _DEFAULT_CACHE,
     ModelSourceConfig,
+    _try_best_effort_sync,
 )
 
 
@@ -78,6 +80,10 @@ def cmd_sync(args):
 
 def cmd_info(args):
     """Info command handler."""
+    # Attempt auto-sync before showing info (unless disabled)
+    if not os.environ.get("MODFLOW_DEVTOOLS_NO_AUTO_SYNC"):
+        _try_best_effort_sync()
+
     config = ModelSourceConfig.load()
     status = config.status
 
@@ -96,6 +102,10 @@ def cmd_info(args):
 
 def cmd_list(args):
     """List command handler."""
+    # Attempt auto-sync before listing (unless disabled)
+    if not os.environ.get("MODFLOW_DEVTOOLS_NO_AUTO_SYNC"):
+        _try_best_effort_sync()
+
     cached = _DEFAULT_CACHE.list()
 
     if not cached: