test(appkit): add regression test for non-signal cache callers

Verify that direct getOrExecute callers (like telemetry-example-plugin)
that pass a plain async function without accepting the shared signal
continue to work after the in-flight deduplication refactor. Also update
the MockCacheManager signature to match the new real implementation.

Signed-off-by: Pawel Kosiec <pawel.kosiec@databricks.com>

Author: pkosiec

packages/appkit/src/cache/tests/cache-manager.test.ts

@@ -378,6 +378,24 @@ describe("CacheManager", () => {
       expect(r2).toBe("result-2");
       expect(fn).toHaveBeenCalledTimes(2);
     });
+
+    test("should work when fn ignores signal parameter (non-signal caller regression)", async () => {
+      const cache = await CacheManager.getInstance({
+        storage: createMockStorage(),
+      });
+      // Simulates direct callers like telemetry-example-plugin that pass
+      // a plain async function without accepting the shared signal.
+      const fn = vi.fn().mockResolvedValue("no-signal-result");
+
+      const result = await cache.getOrExecute(["key"], fn, "user1", {
+        ttl: 60,
+      });
+
+      expect(result).toBe("no-signal-result");
+      expect(fn).toHaveBeenCalledTimes(1);
+      // The cache passes the shared AbortSignal even if the fn ignores it
+      expect(fn).toHaveBeenCalledWith(expect.any(AbortSignal));
+    });
   });
 
   describe("disabled cache", () => {

packages/appkit/src/plugin/tests/cache.test.ts

@@ -20,9 +20,9 @@ class MockCacheManager {
 
   async getOrExecute<T>(
     key: (string | number | object)[],
-    fn: () => Promise<T>,
+    fn: (sharedSignal?: AbortSignal) => Promise<T>,
     userKey: string,
-    options?: { ttl?: number },
+    options?: { ttl?: number; callerSignal?: AbortSignal },
   ): Promise<T> {
     const cacheKey = this.generateKey(key, userKey);
     const cached = await this.get<T>(cacheKey);