chore(main): release 0.3.1

CHANGELOG.md

@@ -1,5 +1,12 @@
 # Changelog
 
+## [0.3.1](https://github.com/sbrow/envr/compare/v0.3.0...v0.3.1) (2026-06-17)
+
+
+### Performance Improvements
+
+* Replaced `fd` with custom internals. ([2ef733f](https://github.com/sbrow/envr/commit/2ef733fe58594b0a0b6e3ef85142b74af445ccb8))
+
 ## [0.3.0](https://github.com/sbrow/envr/compare/v0.2.1...v0.3.0) (2026-06-16)
 
 Version 0.3.0 represents a significant departure (and improvement) for envr.

README.md

@@ -33,6 +33,7 @@ repositories.
 - An SSH key pair (for encryption/decryption)
 - The following binaries:
    - [fd](https://github.com/sharkdp/fd)
+   - [git](https://git-scm.com)
 
 ## Installation
 

TEST_PLAN.md

@@ -35,7 +35,13 @@ Stdout will be captured by redirecting `os.stdout` to a pipe.
 
 ## Hard to test (interactive / external deps)
 
+### `cmd_deps` (cmd_deps.odin)
+- Needs `git` and/or `fd` in PATH
+- Test TTY and non-TTY paths
+- Skip if dependencies not available (with `#assert` like TODO 28 suggests)
+
 ### `cmd_scan` (cmd_scan.odin)
+- Needs `fd` installed
 - Test with fixture git repo containing `.env` files
 - Test `find_unbacked` integration (already partially tested in `cmd_check_test.odin`)
 - Non-TTY JSON output path
@@ -60,4 +66,5 @@ Stdout will be captured by redirecting `os.stdout` to a pipe.
 
 - DB integration tests should use in-memory SQLite (`:memory:`) where possible.
 - Temp dir fixtures should follow the pattern in `scan_test.odin`.
+- External dependency tests (`fd`, `git`) should use `#assert` to ensure the dependency is present rather than silently skipping (TODO 28).
 - Tests that manipulate the `HOME` env var must use a mutex to prevent races with parallel test execution.

TODOS.md

@@ -26,6 +26,8 @@
 
 15. Add a text filter to the multi_select.
 
+16. Create backup / fallback fd.
+
 17. Add tests for untested commands.
 
 18. 2 scan tests silently skip when fd isn't installed, tests pass without actually testing anything. These should use #assert to be sure that fd is in path.
@@ -38,8 +40,6 @@
 
 23. procedures should be ordered by use, main at the top, then in the order they are called from main.
 
-24. [x] Remove git dependency.
-
 ## Double-check AI output
 
 - [ ] cli.odin
@@ -78,11 +78,3 @@
 - [ ] ssh_test.odin
 - [ ] table.odin
 - [ ] table_test.odin
-- [ ] findr/findr_test.odin
-- [ ] findr/gitignore.odin
-- [ ] findr/gitignore_test.odin
-- [ ] findr/glob.odin
-- [ ] findr/glob_test.odin
-- [ ] findr/repos.odin
-- [ ] findr/test_env.odin
-- [ ] findr/walker.odin

WINDOWS.md

@@ -30,6 +30,10 @@ The application relies on external tools that need to be installed separately on
    - Install via: `winget install sharkdp.fd` or `choco install fd`
    - Alternative: `scoop install fd`
 
+2. **git** - Version control system
+   - Install via: `winget install Git.Git` or download from git-scm.com
+   - Usually already available on most development machines
+
 ## Minor Compatibility Notes
 
 ### File Permissions
@@ -61,6 +65,7 @@ if editor == "" {
 1. Install required dependencies:
    ```powershell
    winget install sharkdp.fd
+   winget install Git.Git
    ```
 
 2. Fix the path handling bug in `app/env_file.go:209`

cli.odin

@@ -43,6 +43,13 @@ key somewhere, otherwise your data could be lost forever.`,
 	{"list", "envr list", "View your tracked files", "", {}},
 	{"remove", "envr remove <path>", "Remove a .env file from your database", "", {}},
 	{"check", "envr check [path]", "Check if files are backed up", "", {}},
+	{
+		"deps",
+		"envr deps",
+		"Check for missing binaries",
+		"envr relies on external binaries for certain functionality.\n\nThe check command reports on which binaries are available and which are not.",
+		{},
+	},
 	{"version", "envr version", "Show envr's version", "", {}},
 	{"edit-config", "envr edit-config", "Edit your config with your default editor", "", {}},
 	{

cmd_deps.odin

@@ -0,0 +1,27 @@
+package main
+
+import "core:fmt"
+import "core:os"
+import "core:terminal"
+
+// TODO: Improve table rendering
+cmd_deps :: proc(cmd: ^Command) {
+	feats := check_features()
+
+	headers := []string{"Feature", "Status"}
+	rows: [dynamic][]string
+
+	if .Git in feats {
+		append(&rows, []string{"Git", "\u2713 Available"})
+	} else {
+		append(&rows, []string{"Git", "\u2717 Missing"})
+	}
+
+	if terminal.is_terminal(os.stdout) {
+		render_table(cmd.out, headers, rows[:])
+	} else {
+		render_json_rows(cmd.out, headers, rows[:])
+		fmt.wprint(cmd.out, "\n", flush = false)
+	}
+}
+

db.odin

@@ -2,12 +2,12 @@ package main
 
 import "core:crypto/hash"
 import "core:encoding/hex"
-import "core:encoding/ini"
 import "core:encoding/json"
 import "core:fmt"
 import "core:os"
 import "core:path/filepath"
 import "core:strings"
+import "core:time"
 
 import "sqlite"
 
@@ -51,6 +51,14 @@ delete_envfile :: proc(f: ^EnvFile) {
 	delete(f.contents)
 }
 
+make_temp_path :: proc() -> string {
+	ts := time.time_to_unix(time.now())
+	b: strings.Builder
+	strings.builder_init(&b)
+	defer strings.builder_destroy(&b)
+	fmt.sbprintf(&b, "/tmp/envr-%d-%d.db", os.get_pid(), ts)
+	return strings.to_string(b)
+}
 
 db_open :: proc(cfg_path: string) -> (Db, bool) {
 	cfg, ok := load_config(cfg_path)
@@ -228,24 +236,59 @@ db_restore_from_encrypted :: proc(db: ^rawptr, cfg: Config) -> bool {
 get_git_remotes :: proc(dir: string) -> [dynamic]string {
 	remotes: [dynamic]string
 	remote_set: map[string]bool
-	defer delete(remote_set)
 
-	config_path, _ := filepath.join({dir, ".git", "config"}, context.temp_allocator)
+	b: strings.Builder
-	m, _, ok := ini.load_map_from_path(config_path, context.allocator)
+	strings.builder_init(&b)
-	if !ok {
+	defer strings.builder_destroy(&b)
+	fmt.sbprintf(&b, "%s-git-remotes", make_temp_path())
+	tmp_path := strings.to_string(b)
+	tmp_file, tmp_err := os.open(tmp_path, os.O_CREATE | os.O_WRONLY | os.O_TRUNC)
+	if tmp_err != nil {
 		return remotes
 	}
-	defer ini.delete_map(m)
 
-	for section_name, section in m {
+	args := []string{"git", "remote", "-v"}
-		if strings.has_prefix(section_name, "remote ") {
+	desc := os.Process_Desc {
-			if url, ok := section["url"]; ok {
+		command     = args,
-				remote_set[url] = true
+		stdout      = tmp_file,
-			}
+		stderr      = nil,
+		working_dir = dir,
+	}
+
+	p, start_err := os.process_start(desc)
+	os.close(tmp_file)
+	if start_err != nil {
+		os.remove(tmp_path)
+		return remotes
+	}
+
+	state, wait_err := os.process_wait(p)
+	if wait_err != nil || state.exit_code != 0 {
+		os.remove(tmp_path)
+		return remotes
+	}
+
+	data, read_err := os.read_entire_file_from_path(tmp_path, context.allocator)
+	defer delete(data)
+	os.remove(tmp_path)
+	if read_err != nil {
+		return remotes
+	}
+
+	lines := strings.split(string(data), "\n")
+
+	for &line in lines {
+		line = strings.trim_space(line)
+		if len(line) == 0 {
+			continue
+		}
+		parts := strings.fields(line)
+		if len(parts) >= 2 {
+			remote_set[parts[1]] = true
 		}
 	}
 
-	for remote in remote_set {
+	for remote, _ in remote_set {
 		cloned, _ := strings.clone(remote)
 		append(&remotes, cloned)
 	}
@@ -473,6 +516,12 @@ update_dir :: proc(f: ^EnvFile, new_dir: string) {
 }
 
 find_moved_dirs :: proc(d: ^Db, f: ^EnvFile) -> ([dynamic]string, bool) {
+	feats := check_features()
+	if .Git not_in feats {
+		fmt.println("Error: git is required for moved dir detection")
+		return {}, false
+	}
+
 	roots, roots_ok := find_git_roots(d.cfg)
 	if !roots_ok {
 		return {}, false

db_test.odin

@@ -319,85 +319,11 @@ test_shares_remote_both_empty :: proc(t: ^testing.T) {
 	testing.expect(t, !shares_remote(&f, remotes), "both empty should not share")
 }
 
-delete_remotes :: proc(remotes: [dynamic]string) {
-	for &r in remotes {
-		delete(r)
-	}
-	delete(remotes)
-}
-
 @(test)
-test_get_git_remotes_single :: proc(t: ^testing.T) {
+test_make_temp_path_format :: proc(t: ^testing.T) {
-	base := fmt.tprintf("/tmp/envr-test-remotes-%d", os.get_pid())
+	p := make_temp_path()
-	os.mkdir_all(base)
+	testing.expect(t, strings.has_suffix(p, ".db"), "should end with .db")
-	defer os.remove_all(base)
+	testing.expect(t, strings.contains(p, fmt.tprintf("%d", os.get_pid())), "should contain PID")
-
-	git_dir := fmt.tprintf("%s/.git", base)
-	os.mkdir_all(git_dir)
-
-	config_content := "[core]\n\trepositoryformatversion = 0\n[remote \"origin\"]\n\turl = git@github.com:user/repo.git\n\tfetch = +refs/heads/*:refs/remotes/origin/*\n"
-	config_path := fmt.tprintf("%s/config", git_dir)
-	err := os.write_entire_file(config_path, transmute([]u8)config_content)
-	testing.expect(t, err == nil, "should write .git/config")
-
-	remotes := get_git_remotes(base)
-	defer delete_remotes(remotes)
-
-	testing.expect(t, len(remotes) == 1, "should find 1 remote")
-	if len(remotes) != 1 do return
-	testing.expect_value(t, remotes[0], "git@github.com:user/repo.git")
-}
-
-@(test)
-test_get_git_remotes_multiple :: proc(t: ^testing.T) {
-	base := fmt.tprintf("/tmp/envr-test-remotes-multi-%d", os.get_pid())
-	os.mkdir_all(base)
-	defer os.remove_all(base)
-
-	git_dir := fmt.tprintf("%s/.git", base)
-	os.mkdir_all(git_dir)
-
-	config_content := "[remote \"origin\"]\n\turl = git@github.com:user/repo.git\n[remote \"upstream\"]\n\turl = https://gitlab.com/upstream/repo.git\n"
-	config_path := fmt.tprintf("%s/config", git_dir)
-	err := os.write_entire_file(config_path, transmute([]u8)config_content)
-	testing.expect(t, err == nil, "should write .git/config")
-
-	remotes := get_git_remotes(base)
-	defer delete_remotes(remotes)
-
-	testing.expect(t, len(remotes) == 2, "should find 2 remotes")
-}
-
-@(test)
-test_get_git_remotes_no_config :: proc(t: ^testing.T) {
-	base := fmt.tprintf("/tmp/envr-test-remotes-none-%d", os.get_pid())
-	os.mkdir_all(base)
-	defer os.remove_all(base)
-
-	remotes := get_git_remotes(base)
-	defer delete_remotes(remotes)
-
-	testing.expect(t, len(remotes) == 0, "should return empty when no .git/config")
-}
-
-@(test)
-test_get_git_remotes_no_remotes :: proc(t: ^testing.T) {
-	base := fmt.tprintf("/tmp/envr-test-remotes-empty-%d", os.get_pid())
-	os.mkdir_all(base)
-	defer os.remove_all(base)
-
-	git_dir := fmt.tprintf("%s/.git", base)
-	os.mkdir_all(git_dir)
-
-	config_content := "[core]\n\trepositoryformatversion = 0\n\tbare = false\n"
-	config_path := fmt.tprintf("%s/config", git_dir)
-	err := os.write_entire_file(config_path, transmute([]u8)config_content)
-	testing.expect(t, err == nil, "should write .git/config")
-
-	remotes := get_git_remotes(base)
-	defer delete_remotes(remotes)
-
-	testing.expect(t, len(remotes) == 0, "should return empty when no remote sections")
 }
 
 @(test)

docs/cli/envr.md

@@ -45,6 +45,7 @@ at before, restore your backup with:
 
 * [envr backup](envr_backup.md)	 - Import a .env file into envr
 * [envr check](envr_check.md)	 - check if files in the current directory are backed up
+* [envr deps](envr_deps.md)	 - Check for missing binaries
 * [envr edit-config](envr_edit-config.md)	 - Edit your config with your default editor
 * [envr init](envr_init.md)	 - Set up envr
 * [envr list](envr_list.md)	 - View your tracked files

docs/cli/envr_deps.md

@@ -0,0 +1,24 @@
+## envr deps
+
+Check for missing binaries
+
+### Synopsis
+
+envr relies on external binaries for certain functionality.
+
+The check command reports on which binaries are available and which are not.
+
+```
+envr deps [flags]
+```
+
+### Options
+
+```
+  -h, --help   help for deps
+```
+
+### SEE ALSO
+
+* [envr](envr.md)	 - Manage your .env files.
+

features.odin

@@ -0,0 +1,47 @@
+package main
+
+import "base:runtime"
+import "core:mem"
+import "core:os"
+import "core:strings"
+
+Feature :: enum {
+	Git,
+}
+
+AvailableFeatures :: bit_set[Feature]
+
+check_features :: proc() -> AvailableFeatures {
+	feats: AvailableFeatures
+
+	s: mem.Scratch
+	mem.scratch_init(&s, 4 * mem.DEFAULT_PAGE_SIZE)
+	defer mem.scratch_destroy(&s)
+
+	context.temp_allocator = mem.scratch_allocator(&s)
+
+	path_env := os.get_env("PATH", context.temp_allocator)
+	paths := strings.split(path_env, ":", context.temp_allocator)
+
+	if find_binary(paths, "git") != "" {
+		feats += {.Git}
+	}
+
+	return feats
+}
+
+find_binary :: proc(
+	paths: []string,
+	name: string,
+	allocator: runtime.Allocator = context.temp_allocator,
+) -> string {
+	for p in paths {
+		candidate := strings.join({strings.trim_right(p, "/"), name}, "/", allocator)
+		_, err := os.stat(candidate, allocator)
+		if err == nil {
+			return candidate
+		}
+	}
+	return ""
+}
+

features_test.odin

@@ -0,0 +1,34 @@
+package main
+
+import "core:os"
+import "core:strings"
+import "core:testing"
+
+@(test)
+test_find_binary_exists :: proc(t: ^testing.T) {
+	path := os.get_env("PATH", context.temp_allocator)
+	paths := strings.split(path, ":", context.temp_allocator)
+
+	result := find_binary(paths, "sh")
+	testing.expect(t, result != "", "sh should be found on PATH")
+}
+
+@(test)
+test_find_binary_not_exists :: proc(t: ^testing.T) {
+	old_path := os.get_env("PATH", context.temp_allocator)
+	defer {
+		if old_path != "" {
+			os.set_env("PATH", old_path)
+		}
+	}
+
+	os.set_env("PATH", "/tmp/envr-nope")
+
+	path := os.get_env("PATH", context.temp_allocator)
+	paths := strings.split(path, ":", context.temp_allocator)
+
+
+	result := find_binary(paths, "no_such_binary_xyz")
+	testing.expect(t, result == "", "nonexistent binary should not be found")
+}
+

main.odin

@@ -18,6 +18,8 @@ main :: proc() {
 		cmd_init(&cmd)
 	case "version":
 		cmd_version(&cmd)
+	case "deps":
+		cmd_deps(&cmd)
 	case "list":
 		cmd_list(&cmd)
 	case "backup", "add":

version.txt

@@ -1 +1 @@
-0.3.0
+0.3.1