refactor: move unnecessary in-function imports to module level

Move write_validation_jsonl import in upload.py and
load_validation_jsonl import in cmd_validate.py to module level.
These had no circular import justification (unlike the validate_bids
import in files/bids.py which must stay lazy).

Co-Authored-By: Claude Code 2.1.81 / Claude Opus 4.6 <noreply@anthropic.com>

Author: yarikoptic

dandi/cli/cmd_validate.py

@@ -16,7 +16,11 @@
 from .formatter import JSONFormatter, JSONLinesFormatter, YAMLFormatter
 from ..utils import pluralize
 from ..validate._core import validate as validate_
-from ..validate._io import validation_companion_path, write_validation_jsonl
+from ..validate._io import (
+    load_validation_jsonl,
+    validation_companion_path,
+    write_validation_jsonl,
+)
 from ..validate._types import Severity, ValidationResult
 
 lgr = logging.getLogger(__name__)
@@ -264,8 +268,6 @@ def validate(
         raise click.UsageError("--load and positional paths are mutually exclusive.")
 
     if load:
-        from ..validate._io import load_validation_jsonl
-
         results = load_validation_jsonl(load)
     else:
         results = _collect_results(paths, schema, devel_debug, allow_any_path)

dandi/upload.py

@@ -49,6 +49,7 @@
 from .support import pyout as pyouts
 from .support.pyout import naturalsize
 from .utils import ensure_datetime, path_is_subpath, pluralize
+from .validate._io import write_validation_jsonl
 from .validate._types import Severity
 
 
@@ -302,8 +303,6 @@ def process_path(dfile: DandiFile) -> Iterator[dict]:
                     yield {"status": "pre-validating"}
                     validation_statuses = dfile.get_validation_errors()
                     if validation_log_path is not None and validation_statuses:
-                        from .validate._io import write_validation_jsonl
-
                         write_validation_jsonl(
                             validation_statuses, validation_log_path, append=True
                         )