Skip to content

ci: migrate to moonrepo/setup-rust for unified toolchain management#87

Merged
bug-ops merged 6 commits intomainfrom
feature/ci-moonrepo-setup
Feb 2, 2026
Merged

ci: migrate to moonrepo/setup-rust for unified toolchain management#87
bug-ops merged 6 commits intomainfrom
feature/ci-moonrepo-setup

Conversation

@bug-ops
Copy link
Owner

@bug-ops bug-ops commented Feb 2, 2026

Summary

  • Consolidate fragmented CI toolchain setup into moonrepo/setup-rust
  • Keep sccache for compilation caching (moonrepo does not support it)
  • Remove redundant sccache-action from maturin jobs (maturin handles it internally)
  • Add sccache to release wheel builds

Changes

Component Before After
Toolchain dtolnay/rust-toolchain moonrepo/setup-rust
Cargo cache Swatinem/rust-cache moonrepo (built-in)
Binary install taiki-e/install-action moonrepo bins:
Compilation cache sccache-action sccache-action (unchanged)

Jobs updated

  • lint-rust-fmt - nightly via channel parameter
  • lint-rust-clippy - moonrepo + sccache
  • test-rust - moonrepo with bins: cargo-nextest
  • coverage-rust - moonrepo with bins: cargo-llvm-cov
  • msrv - moonrepo with dynamic channel
  • build-test-wheel-* - removed redundant sccache setup
  • build-wheels-cross - removed redundant sccache setup
  • build-wheels (release) - added sccache: true
  • build-mcp-binaries - moonrepo with bins: cross
  • publish-crates - moonrepo with cache: false

Test plan

  • CI passes on all platforms (ubuntu, macos, windows)
  • Release workflow builds wheels correctly
  • sccache hit rates remain consistent

- Add ConfigBuilder tests for all setter methods
- Add RuntimeConfig reload and diff tests
- Add helpers module edge case tests
- Add QueryGuard comprehensive tests
- Increase coverage for hdbconnect-mcp config subsystem
Replace fragmented toolchain setup with moonrepo/setup-rust:
- Consolidate dtolnay/rust-toolchain + Swatinem/rust-cache + taiki-e/install-action
- Keep mozilla-actions/sccache-action for compilation caching
- Remove redundant sccache setup from maturin-action jobs
- Add sccache to release wheel builds for faster rebuilds

Jobs updated: lint-rust-fmt, lint-rust-clippy, test-rust, coverage-rust,
msrv, build-mcp-binaries, publish-crates

Net reduction: 23 lines of workflow configuration
@github-actions github-actions bot added ci documentation Improvements or additions to documentation rust Pull requests that update rust code arrow size/XL labels Feb 2, 2026
@codspeed-hq
Copy link

codspeed-hq bot commented Feb 2, 2026

Merging this PR will not alter performance

✅ 137 untouched benchmarks


Comparing feature/ci-moonrepo-setup (d3349d5) with main (a893d2f)

Open in CodSpeed

@codecov-commenter
Copy link

codecov-commenter commented Feb 2, 2026

Codecov Report

❌ Patch coverage is 99.58960% with 3 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
crates/hdbconnect-mcp/src/security/query_guard.rs 97.72% 2 Missing ⚠️
crates/hdbconnect-mcp/src/config/runtime.rs 98.66% 1 Missing ⚠️

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #87      +/-   ##
==========================================
+ Coverage   84.09%   86.64%   +2.54%     
==========================================
  Files          63       63              
  Lines       12618    13349     +731     
==========================================
+ Hits        10611    11566     +955     
+ Misses       2007     1783     -224     
Flag Coverage Δ
hdbconnect-arrow 86.59% <99.58%> (+2.56%) ⬆️
hdbconnect-mcp 86.59% <99.58%> (+2.56%) ⬆️
python 100.00% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines Coverage Δ
...rates/hdbconnect-arrow/src/conversion/processor.rs 97.49% <ø> (ø)
crates/hdbconnect-arrow/src/test_utils.rs 100.00% <100.00%> (+31.05%) ⬆️
crates/hdbconnect-mcp/src/auth/config.rs 100.00% <100.00%> (+23.44%) ⬆️
crates/hdbconnect-mcp/src/config/builder.rs 100.00% <100.00%> (+9.58%) ⬆️
crates/hdbconnect-mcp/src/config/mod.rs 95.12% <100.00%> (+37.22%) ⬆️
crates/hdbconnect-mcp/src/helpers.rs 84.91% <100.00%> (+10.58%) ⬆️
crates/hdbconnect-mcp/src/observability/metrics.rs 80.79% <100.00%> (+67.75%) ⬆️
crates/hdbconnect-mcp/src/observability/mod.rs 92.64% <100.00%> (+7.79%) ⬆️
crates/hdbconnect-mcp/src/config/runtime.rs 92.47% <98.66%> (+7.78%) ⬆️
crates/hdbconnect-mcp/src/security/query_guard.rs 97.71% <97.72%> (+28.74%) ⬆️
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@bug-ops bug-ops merged commit 7d02d61 into main Feb 2, 2026
38 checks passed
@bug-ops bug-ops deleted the feature/ci-moonrepo-setup branch February 2, 2026 15:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

arrow ci documentation Improvements or additions to documentation rust Pull requests that update rust code size/XL

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants