Skip to content

Bump chrondb to fix native library install on macOS aarch64#102

Open
tleish wants to merge 1 commit into
avelino:mainfrom
tleish:bump-chrondb-macos-aarch64-fix
Open

Bump chrondb to fix native library install on macOS aarch64#102
tleish wants to merge 1 commit into
avelino:mainfrom
tleish:bump-chrondb-macos-aarch64-fix

Conversation

@tleish

@tleish tleish commented Jul 17, 2026

Copy link
Copy Markdown

Summary

(Supersedes #101, which couldn't be reopened after a force-push — same branch, same fix, now with a minimal Cargo.lock diff limited to the chrondb entry.)

Test plan

  • cargo build --release --locked and cargo test --locked (full suite) pass
  • Cleared ~/.chrondb and ran mcp rubymine --health against a real MCP server config on macOS aarch64: native library downloads and installs cleanly, no SetupFailed warning, health check returns {"server":"rubymine","status":"ok"}
  • Confirmed audit logging comes up and mcp logs shows the call

The pinned chrondb build (0.2.3-dev.12298e3) fails to install its
native library on macOS aarch64: the release tarball ships files flat
at the archive root, but the installer's extraction logic assumed a
nested lib/+include/ layout, so it errored out with "Archive did not
contain expected directory structure" and audit logging never came
up (avelino/chrondb#145).

avelino/chrondb#146 fixes the extraction logic and is included in this
build.
Copilot AI review requested due to automatic review settings July 17, 2026 14:03

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR updates the chrondb dependency to pick up an upstream fix for macOS aarch64 native library auto-download/extraction failures, restoring ChronDB-backed functionality (notably audit logging) on that platform.

Changes:

  • Bump chrondb from 0.2.3-dev.12298e3 to 0.2.3-dev.ed9ada3 in Cargo.toml.
  • Update Cargo.lock to the new chrondb version and checksum.

Reviewed changes

Copilot reviewed 1 out of 2 changed files in this pull request and generated no comments.

File Description
Cargo.toml Pins chrondb to the newer dev build that includes the macOS aarch64 extraction fix.
Cargo.lock Aligns the resolved chrondb package entry (version/checksum) with the updated dependency pin.

tleish added a commit to tleish/mcp that referenced this pull request Jul 17, 2026
Temporary, fork-only formula pointing at the v0.5.2-chrondb-fix
release binaries (avelino#102), for devs to install the fix
before it's merged upstream.
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.

2 participants