chore: update rhiza to v1.1.1 - #870
Merged
Merged
Conversation
This was referenced Jul 9, 2026
Contributor
There was a problem hiding this comment.
Pull request overview
This PR bumps the pinned Rhiza template from v1.0.2 to v1.1.1 and syncs the downstream repository with upstream template changes, including updates to Makefile bundles, GitHub reusable workflows, and the synced .rhiza/tests suite.
Changes:
- Update
.rhiza/template.yml/.rhiza/template.lockto Rhizav1.1.1and refresh the synced file set. - Replace
.rhiza/requirements/*and.rhiza/utils/*-based tooling with per-target provisioning viauv run --with …andrhiza-toolscommands. - Flatten the synced
.rhiza/testslayout (new top-level test modules; remove prior subdirectory-based suites).
Reviewed changes
Copilot reviewed 47 out of 52 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| ruff.toml | Removes .rhiza/utils-specific lint exceptions now that the utils scripts are removed. |
| docs/development/TESTS.md | Updates docs to reflect uv run --with …-provisioned test dependencies (no separate install). |
| .rhiza/utils/suppression_report.py | Removed (utility script no longer shipped in template). |
| .rhiza/utils/suppression_parse.py | Removed (utility script no longer shipped in template). |
| .rhiza/utils/suppression_audit.py | Removed (utility script no longer shipped in template). |
| .rhiza/utils/pip_audit_policy.py | Removed (replaced by rhiza-tools pip-audit usage). |
| .rhiza/tests/utils/test_suppression_audit.py | Removed (tests for removed .rhiza/utils suppression audit scripts). |
| .rhiza/tests/utils/test_pip_audit_policy.py | Removed (tests for removed .rhiza/utils pip-audit policy script). |
| .rhiza/tests/test_readme_validation.py | Adds README code-block validation tests in new flattened suite layout. |
| .rhiza/tests/test_pyproject.py | Adds pyproject structure/version validation tests in new flattened suite layout. |
| .rhiza/tests/test_git_repo_fixture.py | Adds self-tests for the shared git_repo fixture in new flattened suite layout. |
| .rhiza/tests/test_docstrings.py | Adds doctest discovery/execution test in new flattened suite layout. |
| .rhiza/tests/sync/conftest.py | Removed (suite flattened; prior sync category removed). |
| .rhiza/tests/structure/test_requirements.py | Removed (requirements folder removed from template). |
| .rhiza/tests/structure/test_project_layout.py | Removed (suite flattened; prior structure category removed). |
| .rhiza/tests/stress/README.md | Updates stress docs to reflect flattened test layout (removes integration tests link). |
| .rhiza/tests/shell/test_scripts.sh | Removed (shell-script test suite no longer shipped in template). |
| .rhiza/tests/README.md | Rewrites suite documentation for flattened layout and make rhiza-test usage. |
| .rhiza/tests/integration/test_virtual_env_unexport.py | Removed (suite flattened; prior integration category removed). |
| .rhiza/tests/integration/test_test_mk.py | Removed (suite flattened; prior integration category removed). |
| .rhiza/tests/integration/test_docs_targets.py | Removed (suite flattened; prior integration category removed). |
| .rhiza/tests/integration/test_book_targets.py | Removed (suite flattened; prior integration category removed). |
| .rhiza/tests/api/test_releasing_targets.py | Removed (suite flattened; prior api category removed). |
| .rhiza/tests/api/test_makefile_targets.py | Removed (suite flattened; prior api category removed). |
| .rhiza/tests/api/test_makefile_api.py | Removed (suite flattened; prior api category removed). |
| .rhiza/tests/api/test_make_variable_overrides.py | Removed (suite flattened; prior api category removed). |
| .rhiza/tests/api/test_github_targets.py | Removed (suite flattened; prior api category removed). |
| .rhiza/tests/api/conftest.py | Removed (suite flattened; prior api category removed). |
| .rhiza/template.yml | Bumps template ref to v1.1.1. |
| .rhiza/template.lock | Updates template lock sha/ref and synced file inventory for v1.1.1. |
| .rhiza/rhiza.mk | Updates rhiza-test to provision dependencies inline via uv run --with … and drops .rhiza/utils coverage gating. |
| .rhiza/requirements/tools.txt | Removed (template no longer installs dev tools from .rhiza/requirements). |
| .rhiza/requirements/tests.txt | Removed (template no longer installs test deps from .rhiza/requirements). |
| .rhiza/requirements/README.md | Removed (requirements folder removed). |
| .rhiza/requirements/marimo.txt | Removed (requirements folder removed). |
| .rhiza/requirements/docs.txt | Removed (requirements folder removed). |
| .rhiza/make.d/test.mk | Switches targets to uv run --with …; updates security/suppression tooling to rhiza-tools; updates paths for flattened .rhiza/tests. |
| .rhiza/make.d/quality.mk | Updates suppression-audit target to use rhiza-tools and ensures uv tooling is installed. |
| .rhiza/make.d/bootstrap.mk | Removes .rhiza/requirements/*.txt installs; relies on per-target uv run --with … provisioning. |
| .rhiza/make.d/book.mk | Provisions marimo inline via uv run --with marimo. |
| .github/workflows/rhiza_weekly.yml | Bumps reusable workflow ref to v1.1.1. |
| .github/workflows/rhiza_sync.yml | Bumps reusable workflow ref to v1.1.1. |
| .github/workflows/rhiza_scorecard.yml | Bumps reusable workflow ref to v1.1.1. |
| .github/workflows/rhiza_release.yml | Bumps action ref to v1.1.1. |
| .github/workflows/rhiza_paper.yml | Bumps reusable workflow ref to v1.1.1. |
| .github/workflows/rhiza_mutation.yml | Bumps reusable workflow ref to v1.1.1. |
| .github/workflows/rhiza_marimo.yml | Bumps reusable workflow ref to v1.1.1. |
| .github/workflows/rhiza_fuzzing.yml | Bumps reusable workflow ref to v1.1.1. |
| .github/workflows/rhiza_codeql.yml | Bumps reusable workflow ref to v1.1.1. |
| .github/workflows/rhiza_ci.yml | Bumps reusable workflow ref to v1.1.1. |
| .github/workflows/rhiza_book.yml | Bumps reusable workflow ref to v1.1.1. |
| .github/workflows/rhiza_benchmark.yml | Bumps reusable workflow ref to v1.1.1. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
|
||
| test-pyproject: install ## run pyproject.toml structure tests | ||
| @${UV_BIN} run pytest .rhiza/tests/structure/test_pyproject.py \ | ||
| @${UV_BIN} run --with pytest pytest .rhiza/tests/test_pyproject.py \ |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
reftov1.1.1in.rhiza/template.yml(wasv1.0.2)github-project(unchanged — matches thegithub.comorigin)make syncto apply upstream template changes; conflicts resolved taking upstream. The sync flattened.rhiza/tests/(removed theapi/,integration/,structure/,sync/,utils/subdirs into top-level test modules), removed the.rhiza/requirements/*and.rhiza/utils/*helper files, and refreshed the.github/workflows/*,.rhiza/make.d/*, andruff.toml. No.rejfiles or conflict markers remained in tracked files.docs/index.mdis a deliberately customized project landing page and was preserved (the upstream--8<-- "README.md"include hunk did not apply and was intentionally not forced)..rhiza/.rhiza-version(tool version,0.18.0) left untouched — decoupled from the template ref.Quality gates
make fmtmake typecheckmake docs-coveragemake deptrymake securitymake validate.rhiza/tests47 passedmake testScorecard
Scoped to locally-owned
src/,tests/, and project config (Rhiza-managed files excluded perCLAUDE.md).make fmtgreen, ruff cleanmake validate)data.py(808 LOC) and a few B(10) blocks sit at the edgePortfolio→Data), no cycles;data.pytrends god-module_-private modulesindex.md,MIGRATION.md,getting_started,cost_modelsOverall: 9.7 / 10.
Highest-leverage improvement: split the 808-line
src/jquantstats/data.pyand decompose its B(10) blocks (Data.from_prices,_apply_null_strategy) to bring the largest module and worst functions down a grade.Recommendations (below-10 subcategories)
data.py(808 LOC) and decompose its B(10) functions — Code complexity 9→10.data.pycoupling as a god-module — Overall architecture 9→10.🤖 Generated with Claude Code