Fix Windows build (workspace tokio io-std) + add Windows release workflow#406
Closed
namesreallyblank wants to merge 2067 commits intoruvnet:mainfrom
Closed
Fix Windows build (workspace tokio io-std) + add Windows release workflow#406namesreallyblank wants to merge 2067 commits intoruvnet:mainfrom
namesreallyblank wants to merge 2067 commits intoruvnet:mainfrom
Conversation
Built from commit 72c2a8e Platforms updated: - linux-x64-gnu - linux-arm64-gnu - darwin-x64 - darwin-arm64 - win32-x64-msvc 🤖 Generated by GitHub Actions
Built from commit 881ba31 Platforms updated: - linux-x64-gnu - linux-arm64-gnu - darwin-x64 - darwin-arm64 - win32-x64-msvc 🤖 Generated by GitHub Actions
rvlite's writer_lease.rs used __errno_location (Linux libc) under a generic #[cfg(unix)] guard, causing link failures on macOS where the equivalent is __error. Split the extern and wrapper into separate #[cfg(target_os)] blocks matching the pattern already used in rvf-runtime/src/locking.rs. Closes ruvnet#174 Co-Authored-By: claude-flow <ruv@ruv.net>
fix(rvlite): macOS linking bug — platform-specific errno
Built from commit ba66880 Platforms updated: - linux-x64-gnu - linux-arm64-gnu - darwin-x64 - darwin-arm64 - win32-x64-msvc 🤖 Generated by GitHub Actions
The chat() method signature is (sessionId, content, options?) but the API handler was passing (sessionId, agentId, message), causing TS2559. Fix by creating a session from agentId first, then calling chat(sessionId, message). Co-Authored-By: claude-flow <ruv@ruv.net>
- Build real Linux 6.6.80 bzImage with tinyconfig+virtio+net (1.5 MB) - Create build-rvf.js that assembles self-contained RVF binary: KERNEL_SEG: gzip-compressed Linux 6.6.80 bzImage (MicroLinux, x86_64) WASM_SEG: Full RuvBot Node.js runtime bundle (2.2 MB) META_SEG: Package metadata with capabilities PROFILE_SEG: AI assistant domain profile WITNESS_SEG: Build provenance chain MANIFEST_SEG: Segment directory - Include kernel/bzImage and ruvbot.rvf in npm distribution - Add build:rvf script to package.json - Total RVF: 3.67 MB, bootable with Firecracker/QEMU/Cloud Hypervisor Co-Authored-By: claude-flow <ruv@ruv.net>
…a-binaries feat(ruvbot): self-contained RVF with real Linux 6.6 kernel + API server fix
Co-Authored-By: claude-flow <ruv@ruv.net>
Built from commit e34a47c Platforms updated: - linux-x64-gnu - linux-arm64-gnu - darwin-x64 - darwin-arm64 - win32-x64-msvc 🤖 Generated by GitHub Actions
Built from commit 5e776f3 Platforms updated: - linux-x64-gnu - linux-arm64-gnu - darwin-x64 - darwin-arm64 - win32-x64-msvc 🤖 Generated by GitHub Actions
Adds run-rvf.js that extracts the kernel from KERNEL_SEG, builds a minimal initramfs with a static init binary, and boots via QEMU. Rebuilds kernel with CONFIG_BLK_DEV_INITRD for initramfs support. Modes: --boot (QEMU), --runtime (Node.js), --inspect (manifest) Co-Authored-By: claude-flow <ruv@ruv.net>
Add comprehensive RVF section to README covering segment layout, build/run commands, boot output, supported hypervisors, and kernel config. Add run:rvf and inspect:rvf npm scripts. Co-Authored-By: claude-flow <ruv@ruv.net>
Built from commit 293b31a Platforms updated: - linux-x64-gnu - linux-arm64-gnu - darwin-x64 - darwin-arm64 - win32-x64-msvc 🤖 Generated by GitHub Actions
…iner 6-layer defense-in-depth in a single sealed RVF file: 1. TEE attestation (SGX, SEV-SNP, TDX, ARM CCA) with bound keys 2. Hardened Linux microkernel (16 security configs, REQUIRES_TEE) 3. eBPF packet filter (XDP) + syscall enforcer (Seccomp) 4. AIDefence WASM engine (injection, jailbreak, PII, behavioral) 5. Ed25519 signing + SHAKE-256 content hashes + Paranoid policy 6. 6-role RBAC + Coherence Gate authorization 20 capabilities verified, 10/10 AIDefence tests, 3/3 tamper rejections, 30-entry witness chain, 1000 threat signatures (512-dim), 3 tenant stores. Co-Authored-By: claude-flow <ruv@ruv.net>
Include the generated 2.1 MB .rvf binary artifact in repo alongside the v2.0 optimized example (22 capabilities, zero warnings, Paranoid policy, audited queries, COW branching, SSN/encoding detection). Co-Authored-By: claude-flow <ruv@ruv.net>
Copy the 2.1 MB sealed RVF artifact to examples/ for easier discovery. Co-Authored-By: claude-flow <ruv@ruv.net>
- Add security_hardened.rvf entry to RVF Cognitive Containers section - Add to examples table as top entry - Link ADR-042 alongside ADR-030 and ADR-031 - Update capabilities table from 20 to 22 (COW branching, audited queries, exfil detection) Co-Authored-By: claude-flow <ruv@ruv.net>
Upgrade from 22 to 30 capabilities exercising every major RVF API: - KernelBinding anti-tamper (manifest_root + policy_hash binding) - Dual WASM modules (Interpreter + Microkernel, self-bootstrapping) - DASHBOARD_SEG embedded security monitoring UI - Scalar quantization (int8, 4x compression) via rvf-quant - Binary quantization (1-bit, 32x compression) + Hamming distance - Filter deletion + compaction lifecycle - QEMU requirements check via rvf-launch - Freeze/seal permanent immutability - Additional kernel flags: VIRTIO_NET, VSOCK, INGEST_API - RvfOptions: signing=true, profile=3, m=32, ef_construction=400 Co-Authored-By: claude-flow <ruv@ruv.net>
feat(security): ADR-042 Security RVF — AIDefence + TEE hardened container
Built from commit 6b20327 Platforms updated: - linux-x64-gnu - linux-arm64-gnu - darwin-x64 - darwin-arm64 - win32-x64-msvc 🤖 Generated by GitHub Actions
…Learning Implement trait-based IntelligenceProvider extension point for external quality signals. Addresses PR ruvnet#190 proposal (renumbered from ADR-029 to avoid collision with existing ADR-029-rvf-canonical-format). - IntelligenceProvider trait with load_signals() and quality_weights() - FileSignalProvider built-in for JSON file-based signal exchange - IntelligenceLoader for multi-provider registration and aggregation - QualitySignal, QualityFactors, ProviderQualityWeights types - calibration_bias() on TaskComplexityAnalyzer for router feedback - 12 unit tests (all passing) Co-Authored-By: claude-flow <ruv@ruv.net>
…roviders feat(intelligence): ADR-043 External Intelligence Providers
Built from commit db8f83a Platforms updated: - linux-x64-gnu - linux-arm64-gnu - darwin-x64 - darwin-arm64 - win32-x64-msvc 🤖 Generated by GitHub Actions
TypeScript IntelligenceProvider, FileSignalProvider, and IntelligenceLoader matching the Rust ADR-043 implementation. Also fixes invalid category slug for ruvllm crate publish. Co-Authored-By: claude-flow <ruv@ruv.net>
Built from commit 4e37468 Platforms updated: - linux-x64-gnu - linux-arm64-gnu - darwin-x64 - darwin-arm64 - win32-x64-msvc 🤖 Generated by GitHub Actions
… validation, file size limits Security hardening for ADR-043 intelligence module: - Replace String outcome/verdict with Outcome and HumanVerdict enums (type safety) - Add MAX_SIGNAL_FILE_SIZE (10 MiB) and MAX_SIGNALS_PER_FILE (10,000) limits - BufReader streaming parse instead of read_to_string (prevent double allocation) - Validate quality_score range (finite, 0.0-1.0) on load - NaN protection in calibration_bias() - TypeScript: top-level imports, runtime validation, file size checks, score clamping - Bump workspace to 2.0.4, @ruvector/ruvllm to 2.5.1 - Published ruvllm@2.0.4 to crates.io, @ruvector/ruvllm@2.5.1 to npm Co-Authored-By: claude-flow <ruv@ruv.net>
Built from commit 62436a4 Platforms updated: - linux-x64-gnu - linux-arm64-gnu - darwin-x64 - darwin-arm64 - win32-x64-msvc 🤖 Generated by GitHub Actions
…nd (ruvnet#180) Fix SPARQL parser backtrack, executor memory leak, and add catch_unwind
Published ruvector-postgres@2.0.4 to crates.io with SPARQL parser backtrack fix, executor memory leak fix, and catch_unwind safety. Co-Authored-By: claude-flow <ruv@ruv.net>
Co-Authored-By: claude-flow <ruv@ruv.net>
Built from commit a4d891a Platforms updated: - linux-x64-gnu - linux-arm64-gnu - darwin-x64 - darwin-arm64 - win32-x64-msvc 🤖 Generated by GitHub Actions
Built from commit 4f70c24 Platforms updated: - linux-x64-gnu - linux-arm64-gnu - darwin-x64 - darwin-arm64 - win32-x64-msvc 🤖 Generated by GitHub Actions
- New README for domain expansion crate (cross-domain transfer learning) - All 20 AI OS table layer titles now link to their crate READMEs Co-Authored-By: claude-flow <ruv@ruv.net>
Built from commit c82f72f Platforms updated: - linux-x64-gnu - linux-arm64-gnu - darwin-x64 - darwin-arm64 - win32-x64-msvc 🤖 Generated by GitHub Actions
Built from commit 79ea21a Platforms updated: - linux-x64-gnu - linux-arm64-gnu - darwin-x64 - darwin-arm64 - win32-x64-msvc 🤖 Generated by GitHub Actions
Built from commit 926aa52 Platforms updated: - linux-x64-gnu - linux-arm64-gnu - darwin-x64 - darwin-arm64 - win32-x64-msvc 🤖 Generated by GitHub Actions
Built from commit 89c4f2d Platforms updated: - linux-x64-gnu - linux-arm64-gnu - darwin-x64 - darwin-arm64 - win32-x64-msvc 🤖 Generated by GitHub Actions
…infrastructure map Add genomics, trading, quantum, neuromorphic, graph intelligence, nervous systems, scientific OCR, knowledge graphs, GNN, and SONA as specialized domains with transfer value explanations. Add cross-domain transfer examples table, domain connectivity diagram, and underlying infrastructure layer map. Co-Authored-By: claude-flow <ruv@ruv.net>
The DRY refactor that extracted shared clustering code accidentally removed the HashMap import still needed by merge_scenes. https://claude.ai/code/session_01H1GkTK5z9ppVVQDQukjBsY
Built from commit cff4349 Platforms updated: - linux-x64-gnu - linux-arm64-gnu - darwin-x64 - darwin-arm64 - win32-x64-msvc 🤖 Generated by GitHub Actions
…ross-domain transfer Performance optimizations (net -134 lines): - BinaryHeap kNN: O(n log k) vs O(n log n) full sort in SpatialIndex - Zero-clone behavior tree tick via pointer-based borrow splitting - VecDeque percept buffer for O(1) front eviction - HashSet assigned_robots for O(1) membership checks - Shared clustering module eliminates 3 duplicate implementations Correctness fixes: - UntilFail decorator 10k iteration guard prevents infinite loops - OccupancyGrid bounds-checked get() returns Option<f32> - Pipeline position_history capped at 1000 entries - Skill learning gracefully handles empty demonstrations - Anomaly type gets Serialize/Deserialize derives Dead code removal: - Remove unused TrajectoryPoint struct - Remove unused tracing and rand dependencies Domain expansion integration (behind `domain-expansion` feature flag): - RoboticsDomain implements domain::Domain trait with 5 task categories: PointCloudClustering, ObstacleAvoidance, SceneGraphConstruction, SkillSequencing, SwarmFormation - 64-dim embedding space compatible with planning/orchestration/synthesis - Reference solutions, difficulty scaling, cross-domain transfer tests - Enables Meta Thompson Sampling transfer between robotics and existing domains (Rust synthesis, structured planning, tool orchestration) All 257 tests pass (231 unit + 25 integration + 1 doc-test). https://claude.ai/code/session_01H1GkTK5z9ppVVQDQukjBsY
- Remove unsafe pointer aliasing in BehaviorTree::tick(), use safe disjoint field borrowing instead (P0) - Fix usize underflow in score_scene_graph when expected_objects < 2 (P0) - Fix cluster ID overflow in reference_solution for PointCloudClustering (P0) - Fix NaN handling in MaxDistEntry::cmp — NaN treated as maximally distant so it gets evicted from kNN heap first (P1) - Clamp cosine_distance output to prevent negative values from floating-point rounding (P1) - Change search_radius to return Ok(Vec::new()) for empty index instead of Err(EmptyIndex) for correct semantics (P1) - Add debug_assert guards for empty slices in bounding_sphere and cluster_to_object (P1) - Remove dead PipelineConfig.spatial_search_k field (P2) - Use serde_json::from_value instead of to_string+from_str roundtrip in domain_expansion for better performance (P2) All 257 tests pass. https://claude.ai/code/session_01H1GkTK5z9ppVVQDQukjBsY
…usion, and benchmarks New modules for ruvector-robotics: - bridge/gaussian: GaussianSplat types, PointCloud→Gaussian conversion, vwm-viewer JSON export - planning: A* pathfinding on OccupancyGrid with octile heuristic, potential field velocity commands - mcp/executor: ToolExecutor dispatching ToolRequests to perception pipeline and spatial index - perception/sensor_fusion: multi-sensor cloud fusion with timestamp alignment and voxel downsampling Rewrites integration tests to use actual crate APIs instead of local reimplementations, eliminating ~280 lines of false-positive test code. Adds 15 benchmark groups covering all new modules (Gaussian conversion, A* planning, potential fields, sensor fusion, MCP execution). All 270+ tests pass including domain-expansion feature. https://claude.ai/code/session_01H1GkTK5z9ppVVQDQukjBsY
New `rvf` feature flag enables the `ruvf::RoboticsRvf` wrapper that bridges point clouds, scene graphs, trajectories, Gaussian splats, and obstacles into the RuVector Format (.rvf) for persistence and similarity search. RoboticsRvf supports: - pack_point_cloud (dim 3) - pack_scene_objects / pack_scene_graph (dim 9) - pack_trajectory (dim 3) - pack_gaussians (dim 7) — converts PointCloud→GaussianSplatCloud→RVF - pack_obstacles (dim 6) - query_nearest (kNN via HNSW index) - open/open_readonly/close lifecycle 9 unit tests covering create, ingest, query, reopen, dimension mismatch, and empty data rejection. Also fixes unused import warnings in integration tests. All 290 tests pass across default, domain-expansion, and rvf features. https://claude.ai/code/session_01H1GkTK5z9ppVVQDQukjBsY
…erception SpatialIndex: replace Vec<Vec<f32>> with flat Vec<f32> buffer for cache locality and zero per-point heap allocation; use squared Euclidean distance in kNN/radius search (defer sqrt to final k results); fuse cosine distance into single loop. Clustering: add union-by-rank to union-find preventing tree degeneration (O(α(n)) amortized); add #[inline] on hot helpers. A* planning: add closed set (HashSet) to avoid re-expanding nodes; reuse neighbor buffer to eliminate per-expansion Vec allocation; pre-allocate HashMap capacity; add #[inline] on helpers. Perception: defer sqrt in bounding_sphere (compare squared distances, one sqrt at end); defer sqrt in scene graph edge construction (filter on squared threshold); add #[inline] on dist_3d. Sensor fusion: pre-allocate merged vectors from total eligible cloud size. Anomaly detection: fuse distance + statistics into single pass using Welford's online algorithm (eliminates one full data pass). All 281 tests pass. https://claude.ai/code/session_01H1GkTK5z9ppVVQDQukjBsY
Implements ADR-057 with 7 modules (2,940 lines, 54 tests): - types: 4 new segment types (FederatedManifest 0x33, DiffPrivacyProof 0x34, RedactionLog 0x35, AggregateWeights 0x36) - pii_strip: 3-stage pipeline (detect, redact, attest) with 12 regex rules - diff_privacy: Gaussian/Laplace noise, RDP accountant, gradient clipping - federation: ExportBuilder + ImportMerger with version-aware conflict resolution - aggregate: FedAvg, FedProx, Byzantine-tolerant weighted averaging - policy: FederationPolicy for selective sharing with allow/deny lists - error: 15 typed error variants Also updates rvf-types with 4 new segment discriminants (0x33-0x36), workspace Cargo.toml, and root README (crate count, segment count, federated learning code example). Co-Authored-By: claude-flow <ruv@ruv.net>
feat: rvf-federation crate for federated transfer learning
Built from commit c550fff Platforms updated: - linux-x64-gnu - linux-arm64-gnu - darwin-x64 - darwin-arm64 - win32-x64-msvc 🤖 Generated by GitHub Actions
Clippy fixes (8 warnings → 0): - Replace 5 manual Default impls with #[derive(Default)] - Use .clamp() instead of .min().max() chain - Use .is_some_and() instead of .map_or(false, ...) - Add type alias for complex return type in scene_graph_to_adjacency P0 correctness fixes from code review: - Fix NaN panic: use unwrap_or(Ordering::Equal) in cognitive_core think() - Fix integer overflow: use checked_mul in OccupancyGrid::new - Fix potential unwrap: use map_or in domain_expansion score_avoidance Co-Authored-By: claude-flow <ruv@ruv.net>
Add repository, homepage, keywords, categories for crates.io listing. Pin optional dependency versions (ruvector-domain-expansion 2.0.4, rvf-runtime 0.2, rvf-types 0.2) required for cargo publish. Co-Authored-By: claude-flow <ruv@ruv.net>
…egration-VOZu2 Add ruvector-robotics: unified cognitive robotics platform
Built from commit 85df6b9 Platforms updated: - linux-x64-gnu - linux-arm64-gnu - darwin-x64 - darwin-arm64 - win32-x64-msvc 🤖 Generated by GitHub Actions
The ruvector-cli MCP transport (crates/ruvector-cli/src/mcp/transport.rs)
calls tokio::io::stdin() and tokio::io::stdout() at lines 30-31, but the
workspace tokio dependency in the root Cargo.toml omits the `io-std`
feature. Because ruvector-cli inherits tokio via `workspace = true`, the
binary cannot be built from a clean workspace:
cargo build --release -p ruvector-cli --bin ruvector-mcp
error[E0425]: cannot find function `stdin` in module `tokio::io`
error[E0425]: cannot find function `stdout` in module `tokio::io`
Adding `io-std` to the workspace tokio features resolves the regression
for every consumer at once (no per-crate overrides needed).
Verified: `cargo build --release -p ruvector-cli --bin ruvector-mcp`
now finishes cleanly on macOS aarch64 (release profile).
Adds .github/workflows/release-windows.yml building the ruvector-mcp.exe binary on windows-latest using the MSVC toolchain (x86_64-pc-windows-msvc). Triggers: - push of any `v*` tag (attaches the .exe to the GitHub Release) - workflow_dispatch (manual run; uploads as a 30-day artifact) Steps: 1. dtolnay/rust-toolchain@stable with x86_64-pc-windows-msvc target 2. actions/cache@v4 over ~/.cargo/registry, ~/.cargo/git, and target 3. cargo build --release --bin ruvector-mcp -p ruvector-cli 4. Rename to ruvector-mcp-x86_64-pc-windows-msvc.exe + SHA256 5. Upload as artifact (always) and attach to release (tags only) Why: the current workflow set has 27 entries but none builds a Windows binary for ruvector-mcp. Downstream consumers that ship the MCP server to Windows users currently have to cross-compile locally (which fails without the io-std feature fix in the preceding commit). With this workflow, tagging a release will publish a ready-to-download .exe. Depends on the io-std fix in the previous commit; without it the build step would fail on every platform, not just Windows.
Owner
|
Thanks for sending this — the diagnosis is correct and the intent is good, but the PR can't go in as-is. Detailed feedback below. State of this PRGitHub shows 7,313 files changed / +3,187,916 / −30,706, mergeable: CONFLICTING, 0 CI checks. The branch is forked from a base so old it predates ruvbot, ADR-042/043, RVF v3.0, and dozens of NAPI binary refreshes. The actual diff is dominated by upstream work neither of us touched. Before any merge discussion this needs a clean rebase onto current I reviewed your two intended commits in isolation: Commit
|
This was referenced Apr 29, 2026
Author
|
Closing in favor of split PRs per the maintainer review:
Both branches were cut fresh from current |
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
3 participants
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
Two small, focused fixes that together unblock building and shipping
ruvector-mcpon Windows (and incidentally repair builds on every other platform).1.
fix(cli): add io-std to workspace tokio features(commit fc51094)crates/ruvector-cli/src/mcp/transport.rscallstokio::io::stdin()andtokio::io::stdout()(lines 30-31), but the workspace tokio dep in the rootCargo.tomlomits theio-stdfeature. Sinceruvector-cliinherits tokio viaworkspace = true, the build fails everywhere from a clean checkout:Adding
io-stdto the workspace dep fixes every consumer at once (no per-crate overrides). Verified locally:cargo build --release -p ruvector-cli --bin ruvector-mcpnow finishes cleanly on macOS aarch64 (release profile, ~98s on M-series).2.
ci: add Windows release workflow for ruvector-mcp(commit a11e617)Adds
.github/workflows/release-windows.yml. The repo has 27 existing workflows but none produces a Windows binary forruvector-mcp. With this workflow:v*tag → buildsruvector-mcp.exeonwindows-latest(MSVC) and attaches it to the GitHub Release.workflow_dispatch→ uploads the .exe as a 30-day artifact for manual smoke tests.Uses
dtolnay/rust-toolchain@stablewithx86_64-pc-windows-msvc,actions/cache@v4over the cargo dirs, andsoftprops/action-gh-release@v2for the tag-triggered upload. Output asset isruvector-mcp-x86_64-pc-windows-msvc.exewith a SHA256 printed in the build log.Depends on commit 1 — without
io-stdthe workflow'scargo buildstep would fail.Motivation
This came out of a downstream Windows port effort (Clorch, an OSS orchestration project that consumes
ruvector-mcpas a memory backend). Without these two fixes, Windows users either have to cross-compile locally (which silently fails on the sameio-stdregression) or skip RuVector entirely. With them, a single tag push publishes a ready-to-download.exe.Test plan
cargo build --release -p ruvector-cli --bin ruvector-mcpon macOS aarch64 succeeds (verified before opening PR).Release Windows Binaryworkflow can be triggered viaworkflow_dispatchand produces the artifact.No runtime behaviour changes — the io-std feature flag only affects what symbols are compiled into tokio.