forst-lang / forst / 24636953969

76%

Build Type

push

github

Committed by haveyaseen

Commit Message

fix(compiler): parse run/build argv without touching os.Args

`runMain` no longer assigns `os.Args` before `ParseArgs`, which raced with
coverage metadata under `go test -race`. Added `compiler.ParseArgsFrom(argv, log)`
and wired the CLI to pass the `argv` slice through. Arg parsing tests now use
explicit slices instead of global `os.Args`.

For `forst dev`, after `filepath.Abs` the root must `Stat` successfully and be
a directory so invalid paths (e.g. embedded NUL) exit before `ListenAndServe`
instead of hanging until the test timeout.

Example:
```go
// cmd/forst: after dev/lsp/dump/… branches
args := compiler.ParseArgsFrom(argv, log)
```

Coverage Stats

9 of 14 new or added lines in 2 files covered. (64.29%)

89 existing lines in 5 files now uncovered.

21050 of 27513 relevant lines covered (76.51%)

89.31 hits per line