fix(needs-review): handle null payload in visual helper toggle

[Shirly8]

Description

Fixes #6253 - Can't toggle off from the ad-hoc panel if you've toggled on in DetailsView

Changes

• Updated onToggleVisualHelper in needs-review-card-selection-action-creator.ts to accept null payloads
• When payload is null (called from action-creator.ts), only toggle visual helper state without trying to enable/disable visualization
• Added test case to verify null payload handling

Testing

• Added unit test to verify that when toggleVisualHelper is called with null, it only toggles the visual helper state and doesn't try to enable/disable the visualization
• Existing tests continue to pass

Root Cause

When toggling the Needs Review visualization ON in DetailsView, action-creator.ts calls toggleVisualHelper.invoke(null) to sync the visual helper state. However, onToggleVisualHelper was trying to access payload.enabled which doesn't exist when payload is null, causing the toggle to get stuck in the ad-hoc panel.

[Shirly8]

@microsoft-github-policy-service agree