Skip to content

chore: integrate mr boxington for rust builds - #1

Merged
nicholasadamou merged 1 commit into
mainfrom
chore/integrate-mr-boxington
Aug 29, 2026
Merged

chore: integrate mr boxington for rust builds#1
nicholasadamou merged 1 commit into
mainfrom
chore/integrate-mr-boxington

Conversation

@nicholasadamou

@nicholasadamou nicholasadamou commented Aug 29, 2026

Copy link
Copy Markdown
Member

Why this PR is necessary

  • Rust CI recompiles shared crates across jobs without a rustc action cache.
  • Target-dir caches (Swatinem) are coarser and do not share work the way mbx does.

Summary

  • Switch CI to jdx/mr-boxington-action and mbx for build/test/clippy/run
  • Keep cargo fmt, cargo install, and cargo deny as plain cargo
  • Flox/docs updated for local mbx (dotagent: docs/hooks only)

Test steps

  • CI green on this PR
  • Confirm rust-cache steps are gone and mbx precedes compile commands after toolchain install

View with [code]smith Autofix with [code]smith
Need help on this PR? Tag @codesmith-bot with what you need. Autofix is disabled.

Summary by CodeRabbit

  • Documentation

    • Updated quickstart and development guidance to use the mbx command wrapper for running, building, testing, and linting.
    • Clarified that formatting and installation continue to use standard Cargo commands.
  • Chores

    • Updated automated checks, release builds, and pre-commit validation to use the standardized mbx workflow.
    • Improved shared build-cache handling across development and CI environments.

Replace Swatinem/rust-cache with mbx action caching; use mbx for
compile steps in CI and document local flox/dev usage.
@nicholasadamou nicholasadamou self-assigned this Aug 29, 2026
@nicholasadamou
nicholasadamou merged commit e5acac8 into main Aug 29, 2026
6 checks passed
@coderabbitai

coderabbitai Bot commented Aug 29, 2026

Copy link
Copy Markdown

Review Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: a5303e7e-1523-44cf-940c-66415a32c7a4

📥 Commits

Reviewing files that changed from the base of the PR and between 3ef381b and 90ca901.

📒 Files selected for processing (5)
  • .githooks/pre-commit
  • .github/workflows/ci.yml
  • .github/workflows/release.yml
  • AGENTS.md
  • README.md

📝 Walkthrough

Walkthrough

The pull request replaces supported Cargo test, lint, build, and run commands with mbx, adds the Mr. Boxington action to CI and release workflows, and documents the updated developer workflow. Formatting and installation continue to use Cargo.

Changes

Mr. Boxington command adoption

Layer / File(s) Summary
Local commands and developer guidance
.githooks/pre-commit, AGENTS.md, README.md
Pre-commit checks and documented test, lint, build, check, and run commands now use mbx. cargo fmt and cargo install remain unchanged.
CI workflow integration
.github/workflows/ci.yml
The CI workflow adds read-only repository contents permission, replaces Rust cache steps with jdx/mr-boxington-action@v1, and runs test, lint, and build commands through mbx.
Release workflow integration
.github/workflows/release.yml
The release workflow replaces Rust cache steps with jdx/mr-boxington-action@v1 and runs test, lint, build, and release binary commands through mbx. The formatting check remains unchanged.

Estimated code review effort: 2 (Simple) | ~10 minutes

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch chore/integrate-mr-boxington

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant