[codex] Add Maine income tax surcharge#8012
[codex] Add Maine income tax surcharge#8012DTrim99 wants to merge 7 commits intoPolicyEngine:mainfrom
Conversation
|
Implementation note: this models Maine H.P. 1491 / L.D. 2212, Part DDDD by adding an Validation run locally:
|
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #8012 +/- ##
==========================================
Coverage 100.00% 100.00%
==========================================
Files 18 7 -11
Lines 353 95 -258
Branches 4 0 -4
==========================================
- Hits 353 95 -258
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
Follow-up added to this PR:
Additional validation run locally:
|
|
Follow-up update pushed in
Validation:
|
PR Review🔴 Critical (Must Fix)
🟡 Should Address
🟢 Suggestions
Validation Summary
Next StepsAddress the critical hard-coded year issue before merge. The remaining items are quality improvements. To auto-fix: `/fix-pr 8012`. |
|
Addressed the CC review feedback in
On the surcharge uprating comment: I verified Validation:
|
Program Review: PR #8012 -- Maine Income Tax Surcharge + Affordability Payments + Property Tax CreditSource Documents
Critical (Must Fix)
Should Address
Suggestions
PDF Audit Summary
Validation Summary
Review Severity: REQUEST_CHANGESSix critical issues must be fixed before merge:
Next StepsTo auto-fix issues: |
|
Please rebase from upstream/main — there are merge conflicts with the latest main. Once rebased and CI is green, this should be ready for review. |
Replace the surcharge in_effect parameter with pre-2026 zero-rate and infinite-threshold defaults so the formula can be fully vectorized and evaluated in any year without the threshold breakdown lookup failing. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
- Fix me_affordability_payment_eligible to check claimed_as_dependent_on_another_return (the prior head-and-dependent intersection is unreachable in the entity model). - Update all L.D. 2212 hrefs from the committee amendment (item=2) to the chaptered Public Law 650 (item=37), decrementing Part T page refs by 1 to match the chaptered pagination. - Zero out the affordability payment amount for 2026+ so it does not leak past the 2025-only statutory authorization. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
- Remove redundant documentation fields and default_value on the ME affordability payment subtraction variables. - Reword surcharge/amount/in_effect parameter descriptions to use the standard "Maine uses/limits/provides" phrasing. - Nest income_limit filing-status entries under values: for consistency with other breakdown parameters. - Add subsection-level citations (T-1(1)(A), T-1(1)(C)(2), T-1(3), T-1(4), DDDD-1(7), CCCC-1(2-D)) to the new references. - Add absolute_error_margin: 0.1 on all new test cases. - Add SURVIVING_SPOUSE and at-threshold boundary tests for affordability eligibility; add below-threshold, at-threshold, and zero-income tests for the surcharge. - Add a 65+ regression test confirming the property tax cap stays at $2,000 in 2026. - Add me_2026_integration.yaml covering surcharge + AGI subtraction in a single 2026 scenario. - Note the statutory estates/trusts extension (Sec. DDDD-2) as a code comment and add a YAML comment explaining the direct-payment modeling choice in refundable.yaml. Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
e87f3c3 to
b5beb6d
Compare
Fixes #8011.
Summary
Tests