Skip to content

fix: install altimate-code from npm instead of GitHub Release binaries#5

Open
anandgupta42 wants to merge 1 commit intomainfrom
fix/npm-install-method
Open

fix: install altimate-code from npm instead of GitHub Release binaries#5
anandgupta42 wants to merge 1 commit intomainfrom
fix/npm-install-method

Conversation

@anandgupta42
Copy link
Contributor

Summary

  • Switch from downloading standalone binaries from GitHub Releases to npm install -g altimate-code
  • Removes ~50 lines of shell logic (arch detection, download, extract, cache)
  • Matches the officially documented install method

Why

The standalone binary approach doesn't include NAPI native dependencies (@altimateai/altimate-core). This caused v0.5.10 to silently degrade to regex-only mode in CI. The npm package correctly resolves all platform-specific dependencies.

Test plan

  • Verify npm install -g altimate-code works on ubuntu-latest runner
  • Verify altimate-code check --help is available after install
  • Verify analysis results match previous behavior

🤖 Generated with Claude Code

The previous approach downloaded standalone binaries from GitHub Releases,
which don't include NAPI native dependencies like `@altimateai/altimate-core`.
This caused v0.5.10 to silently fail (graceful degradation to regex-only mode).

Switch to `npm install -g altimate-code` which is the officially supported
install method. npm correctly resolves all dependencies including the
platform-specific NAPI binaries via the wrapper package.

Benefits:
- Matches the documented install method in the README
- Automatically gets `@altimateai/altimate-core` and platform binaries
- No more architecture/OS detection shell logic
- No more GitHub API calls for version resolution
- Simpler action.yml (removed ~40 lines of download/extract/cache logic)

Trade-off: npm install takes ~10s vs cached binary lookup (~0s on cache hit).
The cache step was removed since npm has its own caching and the install is
fast enough for CI.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
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