Skip to content

fix: stage rust binaries in .claude/bin#3027

Merged
rysweet merged 2 commits intomainfrom
issue-2961-bin-staging
Mar 10, 2026
Merged

fix: stage rust binaries in .claude/bin#3027
rysweet merged 2 commits intomainfrom
issue-2961-bin-staging

Conversation

@rysweet
Copy link
Owner

@rysweet rysweet commented Mar 10, 2026

Summary

  • stage Rust amplihack and amplihack-hooks binaries into .claude/bin during wheel builds when available
  • update staged-hook discovery to prefer ~/.amplihack/.claude/bin/amplihack-hooks while preserving legacy search paths
  • add regression coverage for .claude/bin staging, wheel contents, and hook binary search priority

Validation

  • /home/azureuser/src/amploxy/.venv/bin/pytest tests/test_wheel_packaging.py tests/test_rust_hook_engine.py tests/integration/test_cli_unified_staging.py -q

Result: 51 passed

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@github-actions
Copy link
Contributor

🤖 Auto-fixed version bump

The version in pyproject.toml has been automatically bumped to the next patch version.

If you need a minor or major version bump instead, please update pyproject.toml manually and push the change.

@rysweet rysweet merged commit 8f7b99d into main Mar 10, 2026
1 check passed
@rysweet rysweet deleted the issue-2961-bin-staging branch March 10, 2026 16:50
@github-actions
Copy link
Contributor

Repo Guardian - Passed

All 8 changed files are durable, reusable content that belongs in the repository:

File Assessment
.claude/bin/.gitkeep Standard git placeholder for empty directory
build_hooks.py Durable build hook — parameterized Rust binary staging logic
pyproject.toml Version bump (0.6.20 → 0.6.21)
src/amplihack/__init__.py Source code — adds bin to ESSENTIAL_DIRS
src/amplihack/settings.py Source code — refactored binary search path candidates
tests/integration/test_cli_unified_staging.py Integration test for .claude/bin staging
tests/test_rust_hook_engine.py Unit tests for Rust hook binary lookup
tests/test_wheel_packaging.py Wheel packaging tests for staged Rust binaries

No ephemeral content, temporary scripts, or point-in-time documents detected.

Generated by Repo Guardian for issue #3027 ·

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