From 1cfb0b1133cef1dda647bb3eaa95ddc312bbb3cd Mon Sep 17 00:00:00 2001 From: JD Davis Date: Tue, 16 Jun 2026 14:09:52 -0500 Subject: [PATCH] test: add native install and wrap e2e workflows (#837) ## Description Adds native GitHub Actions smoke coverage for `headroom install` and `headroom wrap ... --prepare-only`, replacing the closed #257 with a focused branch based on current `main`. Closes #257 ## Type of Change - [ ] Bug fix (non-breaking change that fixes an issue) - [ ] New feature (non-breaking change that adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality to change) - [ ] Documentation update - [ ] Performance improvement - [x] Code refactoring (no functional changes) ## Changes Made - Adds native install smoke coverage. - Adds native wrap prepare-only smoke coverage. - Keeps the matrix aligned with `init-native-e2e.yml` on Linux and macOS while Windows native wheel builds are blocked upstream. - Merged current upstream/main cleanly; after update the branch has no remaining diff against upstream/main. ## Testing - [ ] Unit tests pass (`pytest`) - [x] Linting passes (`ruff check .`) - [ ] Type checking passes (`mypy headroom`) - [ ] New tests added for new functionality - [x] Manual testing performed ### Test Output ```text git diff --check uv run --frozen ruff check headroom tests scripts --output-format concise npx --yes --package=@commitlint/cli --package=@commitlint/config-conventional -- commitlint --from upstream/main --to HEAD --config .commitlintrc.json cargo check -p headroom-core All local common gates passed. Branch has no remaining diff against upstream/main after the merge update. ``` ## Real Behavior Proof - Environment: Windows 11, Python 3.13.3 via `uv`, Rust/Cargo 1.95.0, isolated worktree `C:\git\headroom-jd-prs-native-e2e-expansion`. - Exact command / steps: merged `upstream/main`, ran common local gates, and pushed `46f556ee`. - Observed result: diff check, Ruff, commitlint, and Rust check passed locally. - Not tested: no focused pytest was run because the branch has no remaining code/workflow diff against upstream/main after update; full GitHub CI is running on the new head. ## Review Readiness - [x] I have performed a self-review - [x] This PR is ready for human review ## Checklist - [x] My code follows the project's style guidelines - [x] I have performed a self-review of my code - [x] I have commented my code, particularly in hard-to-understand areas - [ ] I have made corresponding changes to the documentation - [x] My changes generate no new warnings - [ ] I have added tests that prove my fix is effective or that my feature works - [x] New and existing unit tests pass locally with my changes - [ ] I have updated the CHANGELOG.md if applicable ## Screenshots (if applicable) N/A ## Additional Notes This PR is now effectively redundant against current `main` after the merge update; maintainers can close or merge according to project preference once governance/CI is green.