fix: pin altimate-core + force re-fetch to prevent stale native binaries#475
Conversation
There was a problem hiding this comment.
Claude Code Review
This repository is configured for manual code reviews. Comment @claude review to trigger a review.
Tip: disable this comment in your organization's Code Review settings.
|
Caution Review failedThe pull request is closed. ℹ️ Recent review info⚙️ Run configurationConfiguration used: Repository UI Review profile: CHILL Plan: Pro Run ID: ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (1)
Disabled knowledge base sources:
📝 WalkthroughWalkthroughA dependency in Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Poem
🚥 Pre-merge checks | ✅ 1 | ❌ 2❌ Failed checks (2 warnings)
✅ Passed checks (1 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
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. Comment |
…e binaries Change `^0.2.5` → `0.2.5` so npm/pnpm/bun won't skip re-downloading when a newer version is published. With the caret range, package managers could serve a cached `0.2.5` binary even after `0.2.6` ships, causing missing export errors at runtime. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
e0672f7 to
b641dbf
Compare
Summary
Two fixes to prevent stale
@altimateai/altimate-corenative binaries during auto-upgrade:^0.2.5→0.2.5— prevents npm from satisfying the range with a cached older version when a newer one is published--force: forces npm/pnpm/bun to re-download all dependencies including platform binaries, working around npm/cli#4828Why both?
--forceactually mattersContext
Companion fix to altimate-core-internal#113 which adds runtime export validation as a safety net.
Fixes #469
Test plan
npm install -g --forceverified with dry-run — correctly re-fetches all platform binariesbun install -g --forceverified — forces latest versions from registrypnpm install --forceconfirmed valid via docs