From 53dea1d9c7f1aaf032c04cfc2083bf4aaa385d79 Mon Sep 17 00:00:00 2001
From: Peter Steinberger <steipete@gmail.com>
Date: Sat, 11 Apr 2026 12:53:15 +0100
Subject: [PATCH] test: narrow web provider artifact invariants

---
 ...ublic-artifacts.explicit-fast-path.test.ts | 12 ++++++----
 .../web-provider-public-artifacts.test.ts     | 24 -------------------
 2 files changed, 7 insertions(+), 29 deletions(-)

diff --git a/src/plugins/web-provider-public-artifacts.explicit-fast-path.test.ts b/src/plugins/web-provider-public-artifacts.explicit-fast-path.test.ts
index 8dcf902bf2e..2e02de9c405 100644
--- a/src/plugins/web-provider-public-artifacts.explicit-fast-path.test.ts
+++ b/src/plugins/web-provider-public-artifacts.explicit-fast-path.test.ts
@@ -15,9 +15,9 @@ vi.mock("./manifest-registry.js", async (importOriginal) => {
 });
 
 import {
-  resolveBundledExplicitWebFetchProvidersFromPublicArtifacts,
-  resolveBundledExplicitWebSearchProvidersFromPublicArtifacts,
-} from "./web-provider-public-artifacts.explicit.js";
+  resolveBundledWebFetchProvidersFromPublicArtifacts,
+  resolveBundledWebSearchProvidersFromPublicArtifacts,
+} from "./web-provider-public-artifacts.js";
 
 describe("web provider public artifacts explicit fast path", () => {
   beforeEach(() => {
@@ -25,7 +25,8 @@ describe("web provider public artifacts explicit fast path", () => {
   });
 
   it("resolves bundled web search providers by explicit plugin id without manifest scans", () => {
-    const provider = resolveBundledExplicitWebSearchProvidersFromPublicArtifacts({
+    const provider = resolveBundledWebSearchProvidersFromPublicArtifacts({
+      bundledAllowlistCompat: true,
       onlyPluginIds: ["brave"],
     })?.[0];
 
@@ -35,7 +36,8 @@ describe("web provider public artifacts explicit fast path", () => {
   });
 
   it("resolves bundled web fetch providers by explicit plugin id without manifest scans", () => {
-    const provider = resolveBundledExplicitWebFetchProvidersFromPublicArtifacts({
+    const provider = resolveBundledWebFetchProvidersFromPublicArtifacts({
+      bundledAllowlistCompat: true,
       onlyPluginIds: ["firecrawl"],
     })?.[0];
 
diff --git a/src/plugins/web-provider-public-artifacts.test.ts b/src/plugins/web-provider-public-artifacts.test.ts
index a266a60842f..fb04f8329bc 100644
--- a/src/plugins/web-provider-public-artifacts.test.ts
+++ b/src/plugins/web-provider-public-artifacts.test.ts
@@ -4,10 +4,6 @@ import {
   hasBundledWebFetchProviderPublicArtifact,
   hasBundledWebSearchProviderPublicArtifact,
 } from "./web-provider-public-artifacts.explicit.js";
-import {
-  resolveBundledWebFetchProvidersFromPublicArtifacts,
-  resolveBundledWebSearchProvidersFromPublicArtifacts,
-} from "./web-provider-public-artifacts.js";
 
 describe("web provider public artifacts", () => {
   it("has a public artifact for every bundled web search provider declared in manifests", () => {
@@ -33,24 +29,4 @@ describe("web provider public artifacts", () => {
       expect(hasBundledWebFetchProviderPublicArtifact(pluginId)).toBe(true);
     }
   });
-
-  it("loads a lightweight bundled web search artifact smoke", () => {
-    const provider = resolveBundledWebSearchProvidersFromPublicArtifacts({
-      bundledAllowlistCompat: true,
-      onlyPluginIds: ["brave"],
-    })?.[0];
-
-    expect(provider?.pluginId).toBe("brave");
-    expect(provider?.createTool({ config: {} as never })).toBeNull();
-  });
-
-  it("prefers lightweight bundled web fetch contract artifacts", () => {
-    const provider = resolveBundledWebFetchProvidersFromPublicArtifacts({
-      bundledAllowlistCompat: true,
-      onlyPluginIds: ["firecrawl"],
-    })?.[0];
-
-    expect(provider?.pluginId).toBe("firecrawl");
-    expect(provider?.createTool({ config: {} as never })).toBeNull();
-  });
 });