fix: make git path resolution lazy instead of import-time #161

Merged

fuzzy merged 1 commit from fix/125-git-import-side-effect into main

2026-07-06 08:20:04 +00:00

automation commented

2026-07-06 08:16:42 +00:00

Owner

Moves shutil.which("git") from module-level import into a lazy _resolve_git() function. The module can now be imported in environments without git; GitNotFoundError only raises when a git function is actually called.

Closes #125

Moves `shutil.which("git")` from module-level import into a lazy `_resolve_git()` function. The module can now be imported in environments without git; `GitNotFoundError` only raises when a git function is actually called. Closes #125

automation added 1 commit

2026-07-06 08:16:42 +00:00

fix(git): make git path resolution lazy instead of import-time

CI / lint-and-test (pull_request) Successful in 1m31s

Details

2977520ceb

Move shutil.which('git') from module level into lazy
_resolve_git() function. The module can now be imported safely
in environments without git; GitNotFoundError only raises
when a git function is actually called.

Closes #125

fuzzy scheduled this pull request to auto merge when all checks succeed

2026-07-06 08:17:10 +00:00

the.auditor reviewed

2026-07-06 08:17:37 +00:00

the.auditor left a comment

Audit Review — PR #161

Clean fix for #125. Module-level shutil.which("git") moved to lazy _resolve_git() with caching. _GIT_PATH initialized to None, resolved on first _run_git() call. No more import-time side effects.

No suggestions.

Summary

Result: COMMENT — no issues found.

## Audit Review — PR #161 Clean fix for #125. Module-level `shutil.which("git")` moved to lazy `_resolve_git()` with caching. `_GIT_PATH` initialized to `None`, resolved on first `_run_git()` call. No more import-time side effects. No suggestions. ## Summary **Result:** COMMENT — no issues found.

the.auditor approved these changes

2026-07-06 08:17:42 +00:00

the.auditor left a comment

APPROVED. Clean lazy resolution, closes #125. Merge when ready.