Improve explicit imports hygiene

[ChrisRackauckas-Claude]

Summary

• Eliminated all implicit imports by using selective using Package: name1, name2 syntax
• Added ExplicitImports.jl tests to CI to prevent regression
• All tests pass

Changes Made

Source Files Updated

• src/ModelOrderReduction.jl: Added explicit imports for DocStringExtensions, ModelingToolkit, LinearAlgebra, and Setfield symbols
• src/utils.jl: Added explicit import for SparseArrays.sparse
• src/precompile.jl: Added explicit imports for PrecompileTools macros

Test Infrastructure

• test/explicit_imports.jl: New test file checking for implicit and stale imports
• test/runtests.jl: Added ExplicitImports test suite
• test/Project.toml: Added ExplicitImports as a test dependency

Before

Module ModelOrderReduction is relying on the following implicit imports:
* `DocStringExtensions` (and 3 exported names)
* `LinearAlgebra` (and 5 exported names)  
* `ModelingToolkit` (and 13 exported names)
* `PrecompileTools` (and 3 exported names)
* `Setfield` (and 2 exported names)
* `SparseArrays` (and 1 exported name)
Total: 31 implicit imports

After

Module ModelOrderReduction is not relying on any implicit imports.
✓ No implicit imports found!
✓ No stale explicit imports found!

Benefits

• Prevents breakage: Explicit imports protect against breaking changes when dependencies modify their exports
• Improves clarity: Makes it clear which symbols come from which packages
• Catches issues early: ExplicitImports tests in CI will catch any future regressions
• Follows best practices: Aligns with Julia community standards for package development

cc @ChrisRackauckas

🤖 Generated with Claude Code

[chatgpt-codex-connector]

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

[chatgpt-codex-connector]

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.