fix: remove if-wrappers from operator error flags

[toddbaert]

The if wrappers were added in #356 (my bad), but json-logic's if coerces its condition to a boolean; null and false are both falsy, so the if always selects the "false" variant for both cases. This makes it impossible to verify that operators return null (not false) on error, which is the entire point of the @operator-errors scenarios.

By using the operator directly as the top-level targeting expression, a null return propagates to the evaluator, which falls back to defaultVariant with reason DEFAULT; a false return would resolve to the "false" variant with reason TARGETING_MATCH. The distinction is preserved.

---

I tested this locally and with this and the expected fixes, I was able to get the test suite fully updated in in-process mode! No more inconsistencies!

[gemini-code-assist]

Code Review

This pull request refactors flag targeting configurations by removing redundant if wrappers and updating variant names across testkit-flags.json and edge-case-flags.json. However, multiple review comments point out that for boolean-returning operators—specifically sem_ver, starts_with, and ends_with—renaming variants to match will break the evaluator's ability to resolve results. It is recommended to retain the true and false variants to ensure proper targeting resolution and to distinguish valid matches from error states.

[gemini-code-assist]

For boolean-returning operators like sem_ver, the variants should remain "true" and "false" to correctly handle valid results and preserve the distinction from null errors, as intended in the PR description.

Suggested change

	"variants": { "match": "match", "fallback": "fallback" },
	"variants": { "true": "true", "false": "false", "fallback": "fallback" },

[toddbaert]

These are edge cases. We WANT to test for null returns.

[gemini-code-assist]

The ends_with operator returns a boolean. The variants should be "true" and "false" to match the operator's output when it doesn't error.

Suggested change

	"variants": { "match": "match", "fallback": "fallback" },
	"variants": { "true": "true", "false": "false", "fallback": "fallback" },

[gemini-code-assist]

The starts_with operator returns a boolean. The variants should be "true" and "false".

        "true": "true",
        "false": "false",
        "fallback": "fallback"

[gemini-code-assist]

The starts_with operator returns a boolean. The variants should be "true" and "false".

        "true": "true",
        "false": "false",
        "fallback": "fallback"

[gemini-code-assist]

The ends_with operator returns a boolean. The variants should be "true" and "false".

        "true": "true",
        "false": "false",
        "fallback": "fallback"

[toddbaert]

#368 (comment)

[gemini-code-assist]

The sem_ver operator returns a boolean. The variants should be "true" and "false".

        "true": "true",
        "false": "false",
        "fallback": "fallback"