review: dedupe config filtering and picker multiselect construction

Commit

Soph2mo ago

nonZeroProfiles/nonZeroAgentConfigs were identical filters over two config types — back both with a generic nonZeroNamed keyed on IsZero. BuildReviewPickerFields built the built-in and plugin-skill multiselects (and their empty-state notes) twice; extract skillMultiSelectField. Covered by the existing picker structure/preselection tests.

Co-Authored-By: Claude Fable 5 noreply@anthropic.com Entire-Checkpoint: 4bfedc11ca7f

Checkpoints

/goal simplify this repo, make sure to de-slop it continue until you are happy, make sure to backfill tests and validate any significant change. The output of this should be a PR with green CI, review

Claude CodeFable 5
View session
Checkpoint 1