Skip to content

refactor(importers): route home lookup through paths#209

Merged
upsetbit merged 1 commit into
masterfrom
codex/issue-80-userhome-paths
Jun 6, 2026
Merged

refactor(importers): route home lookup through paths#209
upsetbit merged 1 commit into
masterfrom
codex/issue-80-userhome-paths

Conversation

@upsetbit
Copy link
Copy Markdown
Contributor

@upsetbit upsetbit commented Jun 6, 2026

Closes #80.

Summary

  • add paths.UserHome() as the single wrapper around os.UserHomeDir
  • route importer DefaultRoots() through paths.UserHome()
  • route schedule home paths through paths.UserHome() and cover path fallback behavior

Local validation

  • go test ./internal/paths ./internal/importers/... ./internal/cli/schedule
  • devbox run -- just lint
  • devbox run -- just test-race

View with Codesmith Autofix with Codesmith
Need help on this PR? Tag /codesmith with what you need. Autofix is disabled.

@upsetbit upsetbit merged commit 174c80e into master Jun 6, 2026
5 checks passed
@upsetbit upsetbit deleted the codex/issue-80-userhome-paths branch June 6, 2026 07:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Route importer home-dir lookups through internal/paths

1 participant