Skip to content

Enforce partnership S-corp leaf exports - #45

Merged
MaxGhenis merged 1 commit into
mainfrom
codex/issue35-partnership-20260614
Jun 14, 2026
Merged

Enforce partnership S-corp leaf exports#45
MaxGhenis merged 1 commit into
mainfrom
codex/issue35-partnership-20260614

Conversation

@MaxGhenis

Copy link
Copy Markdown
Contributor

Closes #35.

Summary

  • declare PUF tax-detail outputs as the partnership_income and s_corp_income leaves plus partnership_self_employment_net_earnings instead of the partnership_s_corp_income aggregate
  • make the PolicyEngine-US adapter exclude partnership_s_corp_income from accepted input variables and reject it on export, even while the current PE-US wheel still reports it as an input
  • update engine-backed round-trip tests to prove the leaves persist and the aggregate is absent/blocked

Validation

  • env -u UV_FROZEN uv run --python 3.13 ruff format --check .
  • env -u UV_FROZEN uv run --python 3.13 ruff check .
  • git diff --check
  • env -u UV_FROZEN uv run --python 3.13 --all-packages --all-groups --all-extras python -m pytest packages/populace-frame/tests/test_policyengine_us_adapter.py packages/populace-build/tests/test_us_plan.py -q
  • env -u UV_FROZEN uv run --python 3.13 --all-packages --all-groups --all-extras python -m pytest packages/populace-frame/tests packages/populace-build/tests -q

Review

  • /cycle subagent review found no actionable findings; added the suggested explicit assertion that the reloaded H5 does not contain partnership_s_corp_income.

@MaxGhenis
MaxGhenis merged commit d71414f into main Jun 14, 2026
4 checks passed
@MaxGhenis
MaxGhenis deleted the codex/issue35-partnership-20260614 branch June 14, 2026 21:59
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.

Propagate partnership/S-corp income split (policyengine-us#8614) into the US build

1 participant