Clarify -inf sentinel in CHIP income limit parameter descriptions - #9174
Conversation
States with -inf in these parameters do not operate a separate CHIP program for that category; their CHIP-funded coverage runs through Medicaid expansion (M-CHIP) or the FCEP option. Document this in the parameter descriptions to prevent the values being read as missing data. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
|
Closing — documentation nice-to-have, not needed. The -inf sentinel behavior is working as intended. |
Program ReviewSource Documents
Branch Status⚠ PR branch is 3031 commit(s) behind main. Consider rebasing before merging. Review was scoped to the PR's actual changes — staleness did not affect findings. Critical (Must Fix)None. The new description sentences were verified as factually accurate about both the code and the policy:
Should Address
Suggestions
Validation Summary
Review Severity: COMMENTNext StepsTo auto-fix issues: Automated review generated by /review-program (read-only). |
MaxGhenis
left a comment
There was a problem hiding this comment.
Merging under the cross-family dual-review protocol (Claude Fable 5 + GPT-5.6-sol, independent reviews, both merge-ok).
Verified: diff is descriptions-only plus a changelog fragment (unified=0 scope check); the -.inf sentinel semantics hold in all three eligibility formulas (state_has_chip = income_limit > 0 in is_chip_eligible_child.py, is_chip_eligible_standard_pregnant_person.py, is_chip_fcep_eligible_person.py); the referenced Medicaid/FCEP parameter trees resolve; the California example is consistent with the repo's existing values (child/pregnant −inf, Medicaid child 2.66, FCEP 3.22).
Summary
The CHIP income limit parameters (
chip/child,chip/pregnant,chip/fcep) use-.infas a sentinel meaning the state does not operate a separate CHIP program for that category — the eligibility formulas gate onincome_limit > 0. Nothing in the parameter files documented this, so the values read as missing data (an API partner recently flagged California's-.infas an unfilled placeholder).California, for example, covers children through CHIP-funded Medicaid expansion (M-CHIP) up to 266% FPL — captured in the Medicaid child income limit parameters — and CHIP-funded pregnancy coverage through the FCEP option at 322% FPL, so
-.infin the separate-CHIP files is correct per the CMS eligibility-levels table.This PR adds one sentence to each of the three parameter descriptions explaining the sentinel and pointing to where the coverage for those states actually lives. No values or logic change.
Changes
parameters/gov/hhs/chip/child/income_limit.yaml: document that -inf = no separate children's CHIP (M-CHIP states)parameters/gov/hhs/chip/pregnant/income_limit.yaml: document that -inf = no standard separate pregnant CHIP (coverage via Medicaid or FCEP)parameters/gov/hhs/chip/fcep/income_limit.yaml: document that -inf = state has not adopted the FCEP option🤖 Generated with Claude Code