fix(plugins): preserve setup.cliBackends owner matching in provider scan

RomneyDa · RomneyDa · commit 75ab995b382e · 2026-05-24T13:28:43.000-07:00
resolveOwningPluginIdsForProvider now also checks plugin.setup?.cliBackends.
The pre-PR no-registry fallback used resolvePluginContributionOwners which
includes both top-level cliBackends and setup.cliBackends; the PR's manifest
scan replacement was missing the setup case.
diff --git a/src/plugins/providers.test.ts b/src/plugins/providers.test.ts
@@ -527,6 +527,18 @@ describe("resolvePluginProviders", () => {
     expectOwningPluginIds("codex-cli");
   });
 
+  it("maps setup-only cli backend ids to owning plugin ids via manifests", () => {
+    setManifestPlugins([
+      createManifestProviderPlugin({
+        id: "setup-only-backend-owner",
+        providerIds: [],
+        setup: { cliBackends: ["setup-only-cli"] },
+      }),
+    ]);
+
+    expectOwningPluginIds("setup-only-cli", ["setup-only-backend-owner"]);
+  });
+
   it("reflects provider ownership manifest changes on the next lookup", () => {
     setManifestPlugins([
       createManifestProviderPlugin({
diff --git a/src/plugins/providers.ts b/src/plugins/providers.ts
@@ -527,6 +527,9 @@ export function resolveOwningPluginIdsForProvider(params: {
         ) ||
         plugin.cliBackends.some(
           (backendId) => normalizeProviderId(backendId) === normalizedProvider,
+        ) ||
+        (plugin.setup?.cliBackends ?? []).some(
+          (backendId) => normalizeProviderId(backendId) === normalizedProvider,
         ),
     )
     .map((plugin) => plugin.id);
