Skip to content

Fix RI reform exports to match standard pattern#7328

Open
DTrim99 wants to merge 1 commit intoPolicyEngine:mainfrom
DTrim99:fix-ri-reform-exports
Open

Fix RI reform exports to match standard pattern#7328
DTrim99 wants to merge 1 commit intoPolicyEngine:mainfrom
DTrim99:fix-ri-reform-exports

Conversation

@DTrim99
Copy link
Collaborator

@DTrim99 DTrim99 commented Feb 6, 2026

Summary

  • Fixes RI high earner tax and social security exemption reform exports to match standard pattern used by RI CTC
  • Changes __init__.py files to only export factory function, not pre-built reform

Test plan

  • RI high earner tax tests pass
  • RI social security exemption tests pass

Closes #7327

🤖 Generated with Claude Code

Closes PolicyEngine#7327

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@DTrim99
Copy link
Collaborator Author

DTrim99 commented Feb 6, 2026

The issue was that the __init__.py files were exporting both the factory function and pre-built reform, which differs from the working pattern used by RI CTC.

Before (broken):

from policyengine_us.reforms.states.ri.high_earner_tax.ri_high_earner_tax_reform import (
    create_ri_high_earner_tax_reform,
    ri_high_earner_tax,
)

After (fixed):

from .ri_high_earner_tax_reform import create_ri_high_earner_tax_reform

This matches the pattern in policyengine_us/reforms/states/ri/ctc/__init__.py which works correctly.

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.

Fix RI reform exports to match standard pattern

1 participant