Port vncli from vecnode: pure-Rust researcher CLI - #1
Merged
Conversation
… distribute_cli.bat Ports the vn CLI/TUI, Docker-managed app runtime, git/system/network tooling, and Ollama+MCP integration into this public repo, with app-identity strings rebranded from vecnode to vncli (vecnode kept only as author attribution). GitHub download scripts now require an explicit username/org instead of defaulting to a specific account. Adds distribute_cli.bat to package a relocatable, prebuilt copy of the CLI, and a public README with badges. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
…hema cargo fmt --check was failing on rustfmt drift inherited from the source repo. cargo-deny (via EmbarkStudios/cargo-deny-action@v2, unpinned) now resolves a newer cargo-deny release whose config schema dropped vulnerability/unsound/notice as advisories fields, db-path/db-urls under [licenses], and the deny/copyleft/allow-osi-fsf-free/default fields there too - licenses are now purely an allow-list. Updated deny.toml accordingly. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
The custom db-urls pointed at a raw db.toml.gz snapshot format current cargo-deny doesn't fetch that way, causing a metadata-parse crash in CI. Falling back to cargo-deny's built-in default (git clone of rustsec/advisory-db) instead. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
- vn crate was missing a license field; add license = "MIT" to match the workspace license. - Bump anyhow 1.0.102 -> 1.0.103 (fixes RUSTSEC-2026-0190, an Error::downcast_mut unsoundness) and crossbeam-epoch 0.9.18 -> 0.9.20 (fixes RUSTSEC-2026-0204, an invalid pointer deref in fmt::Pointer); both are transitive deps pulled to exactly their patched versions. - openssl is banned but still reachable transitively via ollama-rs's own reqwest dependency (it doesn't disable reqwest's default-tls feature, and Cargo's feature unification pulls that into our single resolved reqwest even though our direct dependency is rustls-only and is the only TLS path our own code uses). Allow it only when reached through native-tls so an unrelated new openssl dependency would still be caught. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
…lama-rs webpki-roots ships CA cert data under CDLA-Permissive-2.0 (data license, not code) - add it to the allow-list. ollama-rs's published Cargo.toml omits a license field even though its repo's LICENSE.md is MIT; add a clarify entry (hash is a placeholder, to be corrected from CI's report). Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
option-ext (a small dirs-sys dependency) is MPL-2.0, which unlike GPL/AGPL only applies copyleft at the file level and is broadly considered safe to depend on without copyleft obligations on our own code. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
…e input rustsec/audit-check@v2's actual inputs are token/ignore/working-directory (no file input - that was carried over from the older actions-rs/audit-check action this config was likely modeled on). The unrecognized file: input was silently ignored, so it defaulted to repo root and couldn't find Cargo.lock under cli/. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Remote memory exhaustion via unbounded out-of-order stream reassembly, patched at >=0.11.15. quinn-proto is a transitive dep (rmcp's transport-streamable-http-server -> quinn stack). paste (unmaintained) and lru (unsound) advisories remain but are informational-only per cargo-audit's default config and don't fail the check. Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
vnCLI/TUI, Docker-managed research-app runtime, git/system/network tooling, and Ollama+MCP integration from the privatevecnodesource repo into this publicvnclirepo.VNCLI_REPO_ROOT, container/image names) from vecnode to vncli;vecnodeis kept only as author attribution (Cargo.toml, LICENSE, README).download_all_repos.*/download_all_orgs.*now require an explicit username/org instead of defaulting to a specific account.distribute_cli.bat, which builds a release binary and packages a relocatable, prebuilt copy of the CLI to the Desktop (verified end-to-end).README.mdwith install/feature docs and language/license badges.ubuntu-latest+windows-latestmatrix: fmt, clippy, build, test, cargo-deny, cargo-audit).Test plan
cargo build --manifest-path cli/Cargo.toml -p vncargo test --manifest-path cli/Cargo.toml -p vn(7 passed)cargo clippy --manifest-path cli/Cargo.toml -p vn --all-targets -- -D warningsdistribute_cli.bat, launched the packagedvn.exe --version/--help, confirmed output, cleaned up test artifact🤖 Generated with Claude Code