As of v0.3.0 (c73dc6c).
Summary
Bundle of small documentation/hygiene fixes:
-
README NOVA_CONFIG example has no effect as written. README.md (Configuration section) shows:
NOVA_CONFIG=/path/to/config.toml
As a standalone line this assigns a shell variable that Nova never sees. It should either be exported or shown as a command prefix:
export NOVA_CONFIG=/path/to/config.toml
# or
NOVA_CONFIG=/path/to/config.toml nova check
docs/README.ja.md should be updated to match.
-
CONTRIBUTING.md uses cargo-fmt / cargo-clippy binary names. These hyphenated forms work inside the Nix dev shell but the conventional invocations are cargo fmt --check and cargo clippy --all-targets -- -D warnings (which is also what CI runs). Using the standard form avoids confusing contributors who are not using nix develop.
-
.gitignore contains both .envrc and !.envrc. The two lines (from the toptal direnv template) cancel each other out, and .envrc is intentionally committed. Both lines can be removed.
-
SegmentCache::evict_oldest relies on a small fixed capacity. The O(n) scan is fine for the current capacity of 128, but a short comment near the implementation would make that assumption explicit for future maintainers.
🤖 Generated with Claude Code — Claude Fable 5
As of v0.3.0 (c73dc6c).
Summary
Bundle of small documentation/hygiene fixes:
README
NOVA_CONFIGexample has no effect as written. README.md (Configuration section) shows:As a standalone line this assigns a shell variable that Nova never sees. It should either be exported or shown as a command prefix:
docs/README.ja.mdshould be updated to match.CONTRIBUTING.md uses
cargo-fmt/cargo-clippybinary names. These hyphenated forms work inside the Nix dev shell but the conventional invocations arecargo fmt --checkandcargo clippy --all-targets -- -D warnings(which is also what CI runs). Using the standard form avoids confusing contributors who are not usingnix develop..gitignorecontains both.envrcand!.envrc. The two lines (from the toptal direnv template) cancel each other out, and.envrcis intentionally committed. Both lines can be removed.SegmentCache::evict_oldestrelies on a small fixed capacity. The O(n) scan is fine for the current capacity of 128, but a short comment near the implementation would make that assumption explicit for future maintainers.🤖 Generated with Claude Code — Claude Fable 5