jesus/prvng_core
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

refactor(setup/mod): selective imports + drop dead logging import (ADR-025 L3)

Browse source 
setup/mod.nu had 4 star-imports. Resolution per target:

Converted to selective:
  config/accessor.nu      -> [config-get]                   1 symbol
  utils.nu (re-export)    -> [create_versions_file ...]     4 symbols
  config.nu (re-export)   -> [env_file_providers ...]       2 symbols

Dropped:
  utils/logging.nu        -> 0 used symbols in this file    DEAD

Also promoted the accessor import to absolute path
(lib_provisioning/config/accessor/core.nu) per ADR-025 rule.

Validation:
  nu --ide-check 50 setup/mod.nu -> 0 errors

Refs: ADR-025, .coder/benchmarks/phase2-transitivity.md Layer 3
This commit is contained in:
Jesús Pérez 2026-04-17 08:29:31 +01:00
parent 61b81b1b67
commit b551554519
Signed by: jesus
GPG key ID: 9F243E355E0BC939
1 changed files with 6 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

11
nulib/lib_provisioning/setup/mod.nu
View file

@ -2,12 +2,13 @@
# Orchestrates all setup subcommands with helper functions for configuration management
# Follows Nushell guidelines: explicit types, single purpose, no try-catch
use ../config/accessor.nu *
use ../utils/logging.nu *
# Selective imports (ADR-025 Phase 3 Layer 3).
# utils/logging.nu star-import was dead (no symbols used in this file) — removed.
use lib_provisioning/config/accessor/core.nu [config-get]
# Re-export existing utilities and config helpers
export use utils.nu *
export use config.nu *
# Re-export existing utilities and config helpers (selective)
export use utils.nu [create_versions_file providers_install setup_config_path tools_install]
export use config.nu [env_file_providers install_config]
# Note: wizard.nu is imported by callers directly - avoid circular import with mod.nu
# ============================================================================