Skip to content

Handle attributes above 30 — surface isBeyondChart instead of silently clamping #23

Description

@StreamDemon

P2 from the 2026-07-03 review (docs/reviews/2026-07-03-rules-layer-review.md, PR #19).

Attribute values above 30 silently clamp to the chart's 30-row (effectBonus, attributes.ts:40). The isBeyondChart() helper exists and is exported, but deriveSheet never consults it and the character schema sets no upper bound — so a P.S. 45 character derives a sheet identical to P.S. 30 with no indication the book's "Attributes Beyond Thirty" rules (printed pp.284–286: Augmented/Robot/Supernatural P.S., continued P.E. coma %, I.Q. skill bonus) should apply.

Fix: surface a per-attribute beyondChart flag on the derived sheet (or reject >30 until the Beyond-Thirty rules are modeled). Full modeling of the Beyond-Thirty tables can be its own follow-up when supernatural R.C.C.s arrive (#12).

Metadata

Metadata

Assignees

No one assigned

    Labels

    area:rulesRules engine (@riftforge/rules)

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions