fix: GET /admin/connector-gallery filters stale ids from enabledConnectorIds
Review finding (Minor 1): the stored enabledConnectorIds were returned
unfiltered, so a firm that had curated a since-removed registry entry
(e.g. canva/apollo, removed 13/08/2026) would see it echoed back. This
inflates the admin card's tick count - its "all visible" footer
compares visible.size === registry.length, so a phantom stale id makes
that comparison false while a real entry stays effectively hidden from
the count - and a curation stored as ONLY stale ids would let an
untick compute a payload the server canonicalises to [] ("all
visible"), inverting the admin's intent.
Filters the stored list against connectorRegistryIds() before
returning, mirroring the write-side tolerance already applied to
PATCH /admin/connector-gallery.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
| Repository | Sloth-ninja/JessicaOSS |
|---|---|
| Author | sloth-ninja <ezanahaddis@gmail.com> |
| Authored | |
| Parents | 05571808 |
| Stats | 2 files changed , +30 , -4 |
| Part of | Pilot-feedback fixes: firm UI, connector registry, and legacy writes |
Capture this commit into my fork
Download a Markdown prompt that tells Claude how to port this
exact commit into your working tree. Run it via
claude -p < capture-commit-eedbb795.md
from inside the repo you want the change in.