fix: compact nil conditions before marshalling

[johnny-jiang-rewind]

feature_to_hash called .map(&:to_wire) on enabled_for/disabled_for
without stripping nils, so a condition array containing nil raised
undefined method 'to_wire' for nil. The unless …empty? guard only
catches [], not [nil].
Add .compact before each .map(&:to_wire) so stray nil conditions are
dropped instead of blowing up serialization. Regressed in 3.3.0 (#43);
surfaced in rewind-admin's feature-flag update path.

[diff-vader-bot]

🟢 Risk: LOW — no substantive findings

👀 Diff Vader · Needs human review

Council skipped — policy-routed (external contributor — every org requires a human reviewer for non-members).

External contributor — every org requires a human reviewer for PRs from non-members.

[Copilot]

Pull request overview

This PR hardens the JSON marshaller so feature condition lists containing stray nil entries no longer raise during serialization, addressing a regression where nil.to_wire could crash the update path.

Changes:

• Compacts enabled_for and disabled_for arrays before mapping conditions to their wire format.
• Prevents marshalling from blowing up when condition arrays contain nil.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[smandegar]

Don't forget to bump the version

[diff-vader-bot]

🟢 Risk: LOW — no substantive findings

👀 Diff Vader · Needs human review

Council skipped — policy-routed (external contributor — every org requires a human reviewer for non-members).

External contributor — every org requires a human reviewer for PRs from non-members.