@osdk/client 2.5.2 → 2.6.0-beta.10

package/build/browser/observable/internal/object/ObjectCacheKeyRegistry.js

@@ -0,0 +1,126 @@
+/*
+ * Copyright 2025 Palantir Technologies, Inc. All rights reserved.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+import { extractRdpFieldNames } from "../utils/rdpFieldOperations.js";
+/**
+ * Registry that tracks relationships between object cache keys with different RDP configurations.
+ * This ensures we propagate updates across all "variants" of the same object.
+ */
+export class ObjectCacheKeyRegistry {
+  /**
+   * Map from base key (apiName:primaryKey) to all related cache key variants and metadata
+   */
+  baseToVariants = new Map();
+
+  /**
+   * Metadata for each cache key (apiName, primaryKey, rdpConfig)
+   */
+  keyMetadata = new WeakMap();
+
+  /**
+   * Register a cache key variant for an object
+   */
+  register(cacheKey, apiName, primaryKey, rdpConfig) {
+    const baseKey = this.makeBaseKey(apiName, primaryKey);
+    const primaryKeyStr = String(primaryKey);
+    this.keyMetadata.set(cacheKey, {
+      apiName,
+      primaryKey: primaryKeyStr,
+      rdpConfig,
+      rdpFieldSet: rdpConfig ? extractRdpFieldNames(rdpConfig) : undefined
+    });
+    let entry = this.baseToVariants.get(baseKey);
+    if (!entry) {
+      entry = {
+        variants: new Set(),
+        apiName,
+        primaryKey: primaryKeyStr
+      };
+      this.baseToVariants.set(baseKey, entry);
+    }
+    entry.variants.add(cacheKey);
+  }
+
+  /**
+   * Get all variant cache keys for a specific object
+   */
+  getVariants(apiName, primaryKey) {
+    const baseKey = this.makeBaseKey(apiName, primaryKey);
+    const entry = this.baseToVariants.get(baseKey);
+    return new Set(entry?.variants ?? []);
+  }
+
+  /**
+   * Unregister a cache key when it's being cleaned up
+   */
+  unregister(cacheKey) {
+    const metadata = this.keyMetadata.get(cacheKey);
+    if (!metadata) return;
+    const baseKey = this.makeBaseKey(metadata.apiName, metadata.primaryKey);
+    const entry = this.baseToVariants.get(baseKey);
+    if (entry) {
+      entry.variants.delete(cacheKey);
+      if (entry.variants.size === 0) {
+        this.baseToVariants.delete(baseKey);
+      }
+    }
+    this.keyMetadata.delete(cacheKey);
+  }
+
+  /**
+   * Get metadata for a cache key
+   */
+  getMetadata(cacheKey) {
+    return this.keyMetadata.get(cacheKey);
+  }
+
+  /**
+   * Get the count of variants for a specific object
+   */
+  getVariantCount(apiName, primaryKey) {
+    const baseKey = this.makeBaseKey(apiName, primaryKey);
+    return this.baseToVariants.get(baseKey)?.variants.size ?? 0;
+  }
+
+  /**
+   * Generate a base key from apiName and primaryKey
+   */
+  makeBaseKey(apiName, primaryKey) {
+    return `${apiName}:${primaryKey}`;
+  }
+
+  /**
+   * Check if a cache key has RDP configuration
+   */
+  hasRdpConfig(cacheKey) {
+    return this.keyMetadata.get(cacheKey)?.rdpConfig != null;
+  }
+
+  /**
+   * Get the RDP configuration for a cache key
+   */
+  getRdpConfig(cacheKey) {
+    return this.keyMetadata.get(cacheKey)?.rdpConfig;
+  }
+
+  /**
+   * Get the cached RDP field set for a cache key
+   */
+  getRdpFieldSet(cacheKey) {
+    return this.keyMetadata.get(cacheKey)?.rdpFieldSet ?? new Set();
+  }
+}
+//# sourceMappingURL=ObjectCacheKeyRegistry.js.map

package/build/browser/observable/internal/object/ObjectCacheKeyRegistry.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"ObjectCacheKeyRegistry.js","names":["extractRdpFieldNames","ObjectCacheKeyRegistry","baseToVariants","Map","keyMetadata","WeakMap","register","cacheKey","apiName","primaryKey","rdpConfig","baseKey","makeBaseKey","primaryKeyStr","String","set","rdpFieldSet","undefined","entry","get","variants","Set","add","getVariants","unregister","metadata","delete","size","getMetadata","getVariantCount","hasRdpConfig","getRdpConfig","getRdpFieldSet"],"sources":["ObjectCacheKeyRegistry.ts"],"sourcesContent":["/*\n * Copyright 2025 Palantir Technologies, Inc. All rights reserved.\n *\n * Licensed under the Apache License, Version 2.0 (the \"License\");\n * you may not use this file except in compliance with the License.\n * You may obtain a copy of the License at\n *\n *     http://www.apache.org/licenses/LICENSE-2.0\n *\n * Unless required by applicable law or agreed to in writing, software\n * distributed under the License is distributed on an \"AS IS\" BASIS,\n * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.\n * See the License for the specific language governing permissions and\n * limitations under the License.\n */\n\nimport type { Canonical } from \"../Canonical.js\";\nimport type { Rdp } from \"../RdpCanonicalizer.js\";\nimport { extractRdpFieldNames } from \"../utils/rdpFieldOperations.js\";\nimport type { ObjectCacheKey } from \"./ObjectCacheKey.js\";\n\ninterface CacheKeyMetadata {\n  apiName: string;\n  primaryKey: string;\n  rdpConfig?: Canonical<Rdp>;\n  rdpFieldSet?: ReadonlySet<string>;\n}\n\ninterface BaseKeyEntry {\n  variants: Set<ObjectCacheKey>;\n  apiName: string;\n  primaryKey: string;\n}\n\n/**\n * Registry that tracks relationships between object cache keys with different RDP configurations.\n * This ensures we propagate updates across all \"variants\" of the same object.\n */\nexport class ObjectCacheKeyRegistry {\n  /**\n   * Map from base key (apiName:primaryKey) to all related cache key variants and metadata\n   */\n  private baseToVariants = new Map<string, BaseKeyEntry>();\n\n  /**\n   * Metadata for each cache key (apiName, primaryKey, rdpConfig)\n   */\n  private keyMetadata = new WeakMap<ObjectCacheKey, CacheKeyMetadata>();\n\n  /**\n   * Register a cache key variant for an object\n   */\n  register(\n    cacheKey: ObjectCacheKey,\n    apiName: string,\n    primaryKey: string | number | boolean,\n    rdpConfig?: Canonical<Rdp>,\n  ): void {\n    const baseKey = this.makeBaseKey(apiName, primaryKey);\n    const primaryKeyStr = String(primaryKey);\n\n    this.keyMetadata.set(cacheKey, {\n      apiName,\n      primaryKey: primaryKeyStr,\n      rdpConfig,\n      rdpFieldSet: rdpConfig ? extractRdpFieldNames(rdpConfig) : undefined,\n    });\n\n    let entry = this.baseToVariants.get(baseKey);\n    if (!entry) {\n      entry = {\n        variants: new Set(),\n        apiName,\n        primaryKey: primaryKeyStr,\n      };\n      this.baseToVariants.set(baseKey, entry);\n    }\n    entry.variants.add(cacheKey);\n  }\n\n  /**\n   * Get all variant cache keys for a specific object\n   */\n  getVariants(\n    apiName: string,\n    primaryKey: string | number | boolean,\n  ): Set<ObjectCacheKey> {\n    const baseKey = this.makeBaseKey(apiName, primaryKey);\n    const entry = this.baseToVariants.get(baseKey);\n    return new Set(entry?.variants ?? []);\n  }\n\n  /**\n   * Unregister a cache key when it's being cleaned up\n   */\n  unregister(cacheKey: ObjectCacheKey): void {\n    const metadata = this.keyMetadata.get(cacheKey);\n    if (!metadata) return;\n\n    const baseKey = this.makeBaseKey(metadata.apiName, metadata.primaryKey);\n    const entry = this.baseToVariants.get(baseKey);\n\n    if (entry) {\n      entry.variants.delete(cacheKey);\n      if (entry.variants.size === 0) {\n        this.baseToVariants.delete(baseKey);\n      }\n    }\n\n    this.keyMetadata.delete(cacheKey);\n  }\n\n  /**\n   * Get metadata for a cache key\n   */\n  getMetadata(cacheKey: ObjectCacheKey): CacheKeyMetadata | undefined {\n    return this.keyMetadata.get(cacheKey);\n  }\n\n  /**\n   * Get the count of variants for a specific object\n   */\n  getVariantCount(\n    apiName: string,\n    primaryKey: string | number | boolean,\n  ): number {\n    const baseKey = this.makeBaseKey(apiName, primaryKey);\n    return this.baseToVariants.get(baseKey)?.variants.size ?? 0;\n  }\n\n  /**\n   * Generate a base key from apiName and primaryKey\n   */\n  private makeBaseKey(\n    apiName: string,\n    primaryKey: string | number | boolean,\n  ): string {\n    return `${apiName}:${primaryKey}`;\n  }\n\n  /**\n   * Check if a cache key has RDP configuration\n   */\n  hasRdpConfig(cacheKey: ObjectCacheKey): boolean {\n    return this.keyMetadata.get(cacheKey)?.rdpConfig != null;\n  }\n\n  /**\n   * Get the RDP configuration for a cache key\n   */\n  getRdpConfig(cacheKey: ObjectCacheKey): Canonical<Rdp> | undefined {\n    return this.keyMetadata.get(cacheKey)?.rdpConfig;\n  }\n\n  /**\n   * Get the cached RDP field set for a cache key\n   */\n  getRdpFieldSet(cacheKey: ObjectCacheKey): ReadonlySet<string> {\n    return this.keyMetadata.get(cacheKey)?.rdpFieldSet ?? new Set();\n  }\n}\n"],"mappings":"AAAA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;;AAIA,SAASA,oBAAoB,QAAQ,gCAAgC;AAgBrE;AACA;AACA;AACA;AACA,OAAO,MAAMC,sBAAsB,CAAC;EAClC;AACF;AACA;EACUC,cAAc,GAAG,IAAIC,GAAG,CAAuB,CAAC;;EAExD;AACF;AACA;EACUC,WAAW,GAAG,IAAIC,OAAO,CAAmC,CAAC;;EAErE;AACF;AACA;EACEC,QAAQA,CACNC,QAAwB,EACxBC,OAAe,EACfC,UAAqC,EACrCC,SAA0B,EACpB;IACN,MAAMC,OAAO,GAAG,IAAI,CAACC,WAAW,CAACJ,OAAO,EAAEC,UAAU,CAAC;IACrD,MAAMI,aAAa,GAAGC,MAAM,CAACL,UAAU,CAAC;IAExC,IAAI,CAACL,WAAW,CAACW,GAAG,CAACR,QAAQ,EAAE;MAC7BC,OAAO;MACPC,UAAU,EAAEI,aAAa;MACzBH,SAAS;MACTM,WAAW,EAAEN,SAAS,GAAGV,oBAAoB,CAACU,SAAS,CAAC,GAAGO;IAC7D,CAAC,CAAC;IAEF,IAAIC,KAAK,GAAG,IAAI,CAAChB,cAAc,CAACiB,GAAG,CAACR,OAAO,CAAC;IAC5C,IAAI,CAACO,KAAK,EAAE;MACVA,KAAK,GAAG;QACNE,QAAQ,EAAE,IAAIC,GAAG,CAAC,CAAC;QACnBb,OAAO;QACPC,UAAU,EAAEI;MACd,CAAC;MACD,IAAI,CAACX,cAAc,CAACa,GAAG,CAACJ,OAAO,EAAEO,KAAK,CAAC;IACzC;IACAA,KAAK,CAACE,QAAQ,CAACE,GAAG,CAACf,QAAQ,CAAC;EAC9B;;EAEA;AACF;AACA;EACEgB,WAAWA,CACTf,OAAe,EACfC,UAAqC,EAChB;IACrB,MAAME,OAAO,GAAG,IAAI,CAACC,WAAW,CAACJ,OAAO,EAAEC,UAAU,CAAC;IACrD,MAAMS,KAAK,GAAG,IAAI,CAAChB,cAAc,CAACiB,GAAG,CAACR,OAAO,CAAC;IAC9C,OAAO,IAAIU,GAAG,CAACH,KAAK,EAAEE,QAAQ,IAAI,EAAE,CAAC;EACvC;;EAEA;AACF;AACA;EACEI,UAAUA,CAACjB,QAAwB,EAAQ;IACzC,MAAMkB,QAAQ,GAAG,IAAI,CAACrB,WAAW,CAACe,GAAG,CAACZ,QAAQ,CAAC;IAC/C,IAAI,CAACkB,QAAQ,EAAE;IAEf,MAAMd,OAAO,GAAG,IAAI,CAACC,WAAW,CAACa,QAAQ,CAACjB,OAAO,EAAEiB,QAAQ,CAAChB,UAAU,CAAC;IACvE,MAAMS,KAAK,GAAG,IAAI,CAAChB,cAAc,CAACiB,GAAG,CAACR,OAAO,CAAC;IAE9C,IAAIO,KAAK,EAAE;MACTA,KAAK,CAACE,QAAQ,CAACM,MAAM,CAACnB,QAAQ,CAAC;MAC/B,IAAIW,KAAK,CAACE,QAAQ,CAACO,IAAI,KAAK,CAAC,EAAE;QAC7B,IAAI,CAACzB,cAAc,CAACwB,MAAM,CAACf,OAAO,CAAC;MACrC;IACF;IAEA,IAAI,CAACP,WAAW,CAACsB,MAAM,CAACnB,QAAQ,CAAC;EACnC;;EAEA;AACF;AACA;EACEqB,WAAWA,CAACrB,QAAwB,EAAgC;IAClE,OAAO,IAAI,CAACH,WAAW,CAACe,GAAG,CAACZ,QAAQ,CAAC;EACvC;;EAEA;AACF;AACA;EACEsB,eAAeA,CACbrB,OAAe,EACfC,UAAqC,EAC7B;IACR,MAAME,OAAO,GAAG,IAAI,CAACC,WAAW,CAACJ,OAAO,EAAEC,UAAU,CAAC;IACrD,OAAO,IAAI,CAACP,cAAc,CAACiB,GAAG,CAACR,OAAO,CAAC,EAAES,QAAQ,CAACO,IAAI,IAAI,CAAC;EAC7D;;EAEA;AACF;AACA;EACUf,WAAWA,CACjBJ,OAAe,EACfC,UAAqC,EAC7B;IACR,OAAO,GAAGD,OAAO,IAAIC,UAAU,EAAE;EACnC;;EAEA;AACF;AACA;EACEqB,YAAYA,CAACvB,QAAwB,EAAW;IAC9C,OAAO,IAAI,CAACH,WAAW,CAACe,GAAG,CAACZ,QAAQ,CAAC,EAAEG,SAAS,IAAI,IAAI;EAC1D;;EAEA;AACF;AACA;EACEqB,YAAYA,CAACxB,QAAwB,EAA8B;IACjE,OAAO,IAAI,CAACH,WAAW,CAACe,GAAG,CAACZ,QAAQ,CAAC,EAAEG,SAAS;EAClD;;EAEA;AACF;AACA;EACEsB,cAAcA,CAACzB,QAAwB,EAAuB;IAC5D,OAAO,IAAI,CAACH,WAAW,CAACe,GAAG,CAACZ,QAAQ,CAAC,EAAES,WAAW,IAAI,IAAIK,GAAG,CAAC,CAAC;EACjE;AACF","ignoreList":[]}

package/build/browser/observable/internal/object/ObjectCacheKeyRegistry.test.js

@@ -0,0 +1,173 @@
+/*
+ * Copyright 2025 Palantir Technologies, Inc. All rights reserved.
+ *
+ * Licensed under the Apache License, Version 2.0 (the "License");
+ * you may not use this file except in compliance with the License.
+ * You may obtain a copy of the License at
+ *
+ *     http://www.apache.org/licenses/LICENSE-2.0
+ *
+ * Unless required by applicable law or agreed to in writing, software
+ * distributed under the License is distributed on an "AS IS" BASIS,
+ * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+ * See the License for the specific language governing permissions and
+ * limitations under the License.
+ */
+
+import { describe, expect, it } from "vitest";
+import { ObjectCacheKeyRegistry } from "./ObjectCacheKeyRegistry.js";
+/**
+ * Helper function to create mock RDP objects for testing.
+ * Creates an RDP config with the specified field names, where each field
+ * is a mock Creator function.
+ */
+function createMockRdp(...fields) {
+  const rdp = {};
+  for (const field of fields) {
+    rdp[field] = () => ({});
+  }
+  return rdp;
+}
+
+/**
+ * Helper function to create a properly typed mock ObjectCacheKey for testing
+ */
+function createMockObjectCacheKey(apiName, pk, rdpConfig) {
+  return {
+    type: "object",
+    otherKeys: [apiName, pk, rdpConfig],
+    __cacheKey: {
+      value: {},
+      query: {},
+      args: [apiName, pk, rdpConfig]
+    }
+  };
+}
+describe("ObjectCacheKeyRegistry", () => {
+  it("registers and retrieves variant cache keys", () => {
+    const registry = new ObjectCacheKeyRegistry();
+    const mockRdpAddress = createMockRdp("address", "mockType");
+    const mockRdpPhone = createMockRdp("phone", "mockType");
+    const key1 = createMockObjectCacheKey("Employee", "emp1", undefined);
+    const key2 = createMockObjectCacheKey("Employee", "emp1", mockRdpAddress);
+    const key3 = createMockObjectCacheKey("Employee", "emp1", mockRdpPhone);
+    const unrelatedKey = createMockObjectCacheKey("Employee", "emp2", undefined);
+    registry.register(key1, "Employee", "emp1", undefined);
+    registry.register(key2, "Employee", "emp1", mockRdpAddress);
+    registry.register(key3, "Employee", "emp1", mockRdpPhone);
+    registry.register(unrelatedKey, "Employee", "emp2", undefined);
+    const variants = registry.getVariants("Employee", "emp1");
+    expect(variants.size).toBe(3);
+    expect(variants.has(key1)).toBe(true);
+    expect(variants.has(key2)).toBe(true);
+    expect(variants.has(key3)).toBe(true);
+    expect(variants.has(unrelatedKey)).toBe(false);
+
+    // Unrelated key should have its own set
+    const unrelated = registry.getVariants("Employee", "emp2");
+    expect(unrelated.size).toBe(1);
+    expect(unrelated.has(unrelatedKey)).toBe(true);
+  });
+  it("handles un-registration correctly", () => {
+    const registry = new ObjectCacheKeyRegistry();
+    const mockRdp = createMockRdp("address", "mockType");
+    const key1 = createMockObjectCacheKey("Employee", "emp1", undefined);
+    const key2 = createMockObjectCacheKey("Employee", "emp1", mockRdp);
+    registry.register(key1, "Employee", "emp1", undefined);
+    registry.register(key2, "Employee", "emp1", mockRdp);
+
+    // Both keys should be related
+    expect(registry.getVariants("Employee", "emp1").size).toBe(2);
+    registry.unregister(key1);
+
+    // key2 should now be alone
+    const related = registry.getVariants("Employee", "emp1");
+    expect(related.size).toBe(1);
+    expect(related.has(key2)).toBe(true);
+    expect(related.has(key1)).toBe(false);
+  });
+  it("getVariants returns all variants for an object", () => {
+    const registry = new ObjectCacheKeyRegistry();
+    const key1 = createMockObjectCacheKey("Employee", "emp1", undefined);
+    const key2 = createMockObjectCacheKey("Employee", "emp1", createMockRdp("address"));
+    registry.register(key1, "Employee", "emp1", undefined);
+    registry.register(key2, "Employee", "emp1", createMockRdp("address"));
+    const variants = registry.getVariants("Employee", "emp1");
+    expect(variants.size).toBe(2);
+    expect(variants.has(key1)).toBe(true);
+    expect(variants.has(key2)).toBe(true);
+
+    // Non-existent object should return empty set
+    const empty = registry.getVariants("Employee", "emp999");
+    expect(empty.size).toBe(0);
+  });
+  it("correctly identifies RDP configuration", () => {
+    const registry = new ObjectCacheKeyRegistry();
+    const keyWithoutRdp = createMockObjectCacheKey("Employee", "emp1", undefined);
+    const keyWithRdp = createMockObjectCacheKey("Employee", "emp1", createMockRdp("address"));
+    registry.register(keyWithoutRdp, "Employee", "emp1", undefined);
+    registry.register(keyWithRdp, "Employee", "emp1", createMockRdp("address"));
+    expect(registry.hasRdpConfig(keyWithoutRdp)).toBe(false);
+    expect(registry.hasRdpConfig(keyWithRdp)).toBe(true);
+    expect(registry.getRdpConfig(keyWithoutRdp)).toBeUndefined();
+    const rdpConfig = registry.getRdpConfig(keyWithRdp);
+    expect(rdpConfig).toBeDefined();
+    expect(Object.keys(rdpConfig)).toEqual(["address"]);
+    expect(typeof rdpConfig.address).toBe("function");
+  });
+  it("getVariantCount returns correct count of variants", () => {
+    const registry = new ObjectCacheKeyRegistry();
+    const key1 = createMockObjectCacheKey("Employee", "emp1", undefined);
+    const key2 = createMockObjectCacheKey("Employee", "emp1", createMockRdp("address"));
+    const key3 = createMockObjectCacheKey("Employee", "emp1", createMockRdp("phone"));
+    expect(registry.getVariantCount("Employee", "emp1")).toBe(0);
+    registry.register(key1, "Employee", "emp1", undefined);
+    expect(registry.getVariantCount("Employee", "emp1")).toBe(1);
+    registry.register(key2, "Employee", "emp1", createMockRdp("address"));
+    expect(registry.getVariantCount("Employee", "emp1")).toBe(2);
+    registry.register(key3, "Employee", "emp1", createMockRdp("phone"));
+    expect(registry.getVariantCount("Employee", "emp1")).toBe(3);
+    registry.unregister(key1);
+    expect(registry.getVariantCount("Employee", "emp1")).toBe(2);
+    expect(registry.getVariantCount("Employee", "emp999")).toBe(0);
+  });
+  it("handles registering same key multiple times", () => {
+    const registry = new ObjectCacheKeyRegistry();
+    const key1 = createMockObjectCacheKey("Employee", "emp1", undefined);
+    registry.register(key1, "Employee", "emp1", undefined);
+    registry.register(key1, "Employee", "emp1", undefined);
+    registry.register(key1, "Employee", "emp1", undefined);
+
+    // Should still have only one variant despite multiple registrations
+    expect(registry.getVariantCount("Employee", "emp1")).toBe(1);
+    registry.unregister(key1);
+    expect(registry.getVariantCount("Employee", "emp1")).toBe(0);
+  });
+  it("handles unregistering non-existent keys gracefully", () => {
+    const registry = new ObjectCacheKeyRegistry();
+    const key1 = createMockObjectCacheKey("Employee", "emp1", undefined);
+    const key2 = createMockObjectCacheKey("Employee", "emp2", undefined);
+    registry.register(key1, "Employee", "emp1", undefined);
+
+    // Unregistering non-existent key should not throw
+    registry.unregister(key2);
+
+    // key1 should still be registered
+    expect(registry.getVariantCount("Employee", "emp1")).toBe(1);
+  });
+  it("properly cleans up memory when all variants are unregistered", () => {
+    const registry = new ObjectCacheKeyRegistry();
+    const key1 = createMockObjectCacheKey("Employee", "emp1", undefined);
+    const key2 = createMockObjectCacheKey("Employee", "emp1", createMockRdp("address"));
+    registry.register(key1, "Employee", "emp1", undefined);
+    registry.register(key2, "Employee", "emp1", createMockRdp("address"));
+    expect(registry.getVariantCount("Employee", "emp1")).toBe(2);
+    registry.unregister(key1);
+    registry.unregister(key2);
+
+    // Should return 0 as the entire entry should be cleaned up
+    expect(registry.getVariantCount("Employee", "emp1")).toBe(0);
+    expect(registry.getVariants("Employee", "emp1").size).toBe(0);
+  });
+});
+//# sourceMappingURL=ObjectCacheKeyRegistry.test.js.map

package/build/browser/observable/internal/object/ObjectCacheKeyRegistry.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"ObjectCacheKeyRegistry.test.js","names":["describe","expect","it","ObjectCacheKeyRegistry","createMockRdp","fields","rdp","field","createMockObjectCacheKey","apiName","pk","rdpConfig","type","otherKeys","__cacheKey","value","query","args","registry","mockRdpAddress","mockRdpPhone","key1","undefined","key2","key3","unrelatedKey","register","variants","getVariants","size","toBe","has","unrelated","mockRdp","unregister","related","empty","keyWithoutRdp","keyWithRdp","hasRdpConfig","getRdpConfig","toBeUndefined","toBeDefined","Object","keys","toEqual","address","getVariantCount"],"sources":["ObjectCacheKeyRegistry.test.ts"],"sourcesContent":["/*\n * Copyright 2025 Palantir Technologies, Inc. All rights reserved.\n *\n * Licensed under the Apache License, Version 2.0 (the \"License\");\n * you may not use this file except in compliance with the License.\n * You may obtain a copy of the License at\n *\n *     http://www.apache.org/licenses/LICENSE-2.0\n *\n * Unless required by applicable law or agreed to in writing, software\n * distributed under the License is distributed on an \"AS IS\" BASIS,\n * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.\n * See the License for the specific language governing permissions and\n * limitations under the License.\n */\n\nimport type {\n  DerivedProperty,\n  ObjectOrInterfaceDefinition,\n  SimplePropertyDef,\n} from \"@osdk/api\";\nimport { describe, expect, it } from \"vitest\";\nimport type { ObjectHolder } from \"../../../object/convertWireToOsdkObjects/ObjectHolder.js\";\nimport type { Canonical } from \"../Canonical.js\";\nimport type { Rdp } from \"../RdpCanonicalizer.js\";\nimport type { ObjectCacheKey } from \"./ObjectCacheKey.js\";\nimport { ObjectCacheKeyRegistry } from \"./ObjectCacheKeyRegistry.js\";\nimport type { ObjectQuery } from \"./ObjectQuery.js\";\n\n/**\n * Helper function to create mock RDP objects for testing.\n * Creates an RDP config with the specified field names, where each field\n * is a mock Creator function.\n */\nfunction createMockRdp(...fields: string[]): Canonical<Rdp> {\n  const rdp: Rdp = {};\n  for (const field of fields) {\n    rdp[field] = () => ({} as DerivedProperty.Definition<\n      SimplePropertyDef,\n      ObjectOrInterfaceDefinition\n    >);\n  }\n  return rdp as Canonical<Rdp>;\n}\n\n/**\n * Helper function to create a properly typed mock ObjectCacheKey for testing\n */\nfunction createMockObjectCacheKey(\n  apiName: string,\n  pk: string,\n  rdpConfig?: Canonical<Rdp>,\n): ObjectCacheKey {\n  return {\n    type: \"object\" as const,\n    otherKeys: [apiName, pk, rdpConfig],\n    __cacheKey: {\n      value: {} as ObjectHolder,\n      query: {} as ObjectQuery,\n      args: [apiName, pk, rdpConfig],\n    },\n  };\n}\n\ndescribe(\"ObjectCacheKeyRegistry\", () => {\n  it(\"registers and retrieves variant cache keys\", () => {\n    const registry = new ObjectCacheKeyRegistry();\n\n    const mockRdpAddress = createMockRdp(\"address\", \"mockType\");\n    const mockRdpPhone = createMockRdp(\"phone\", \"mockType\");\n\n    const key1 = createMockObjectCacheKey(\"Employee\", \"emp1\", undefined);\n    const key2 = createMockObjectCacheKey(\"Employee\", \"emp1\", mockRdpAddress);\n    const key3 = createMockObjectCacheKey(\"Employee\", \"emp1\", mockRdpPhone);\n    const unrelatedKey = createMockObjectCacheKey(\n      \"Employee\",\n      \"emp2\",\n      undefined,\n    );\n\n    registry.register(key1, \"Employee\", \"emp1\", undefined);\n    registry.register(key2, \"Employee\", \"emp1\", mockRdpAddress);\n    registry.register(key3, \"Employee\", \"emp1\", mockRdpPhone);\n    registry.register(unrelatedKey, \"Employee\", \"emp2\", undefined);\n\n    const variants = registry.getVariants(\"Employee\", \"emp1\");\n    expect(variants.size).toBe(3);\n    expect(variants.has(key1)).toBe(true);\n    expect(variants.has(key2)).toBe(true);\n    expect(variants.has(key3)).toBe(true);\n    expect(variants.has(unrelatedKey)).toBe(false);\n\n    // Unrelated key should have its own set\n    const unrelated = registry.getVariants(\"Employee\", \"emp2\");\n    expect(unrelated.size).toBe(1);\n    expect(unrelated.has(unrelatedKey)).toBe(true);\n  });\n\n  it(\"handles un-registration correctly\", () => {\n    const registry = new ObjectCacheKeyRegistry();\n\n    const mockRdp = createMockRdp(\"address\", \"mockType\");\n    const key1 = createMockObjectCacheKey(\"Employee\", \"emp1\", undefined);\n    const key2 = createMockObjectCacheKey(\"Employee\", \"emp1\", mockRdp);\n\n    registry.register(key1, \"Employee\", \"emp1\", undefined);\n    registry.register(key2, \"Employee\", \"emp1\", mockRdp);\n\n    // Both keys should be related\n    expect(registry.getVariants(\"Employee\", \"emp1\").size).toBe(2);\n\n    registry.unregister(key1);\n\n    // key2 should now be alone\n    const related = registry.getVariants(\"Employee\", \"emp1\");\n    expect(related.size).toBe(1);\n    expect(related.has(key2)).toBe(true);\n    expect(related.has(key1)).toBe(false);\n  });\n\n  it(\"getVariants returns all variants for an object\", () => {\n    const registry = new ObjectCacheKeyRegistry();\n\n    const key1 = createMockObjectCacheKey(\"Employee\", \"emp1\", undefined);\n    const key2 = createMockObjectCacheKey(\n      \"Employee\",\n      \"emp1\",\n      createMockRdp(\"address\"),\n    );\n\n    registry.register(key1, \"Employee\", \"emp1\", undefined);\n    registry.register(\n      key2,\n      \"Employee\",\n      \"emp1\",\n      createMockRdp(\"address\"),\n    );\n\n    const variants = registry.getVariants(\"Employee\", \"emp1\");\n    expect(variants.size).toBe(2);\n    expect(variants.has(key1)).toBe(true);\n    expect(variants.has(key2)).toBe(true);\n\n    // Non-existent object should return empty set\n    const empty = registry.getVariants(\"Employee\", \"emp999\");\n    expect(empty.size).toBe(0);\n  });\n\n  it(\"correctly identifies RDP configuration\", () => {\n    const registry = new ObjectCacheKeyRegistry();\n\n    const keyWithoutRdp = createMockObjectCacheKey(\n      \"Employee\",\n      \"emp1\",\n      undefined,\n    );\n    const keyWithRdp = createMockObjectCacheKey(\n      \"Employee\",\n      \"emp1\",\n      createMockRdp(\"address\"),\n    );\n\n    registry.register(keyWithoutRdp, \"Employee\", \"emp1\", undefined);\n    registry.register(\n      keyWithRdp,\n      \"Employee\",\n      \"emp1\",\n      createMockRdp(\"address\"),\n    );\n\n    expect(registry.hasRdpConfig(keyWithoutRdp)).toBe(false);\n    expect(registry.hasRdpConfig(keyWithRdp)).toBe(true);\n\n    expect(registry.getRdpConfig(keyWithoutRdp)).toBeUndefined();\n\n    const rdpConfig = registry.getRdpConfig(keyWithRdp);\n    expect(rdpConfig).toBeDefined();\n    expect(Object.keys(rdpConfig!)).toEqual([\"address\"]);\n    expect(typeof rdpConfig!.address).toBe(\"function\");\n  });\n\n  it(\"getVariantCount returns correct count of variants\", () => {\n    const registry = new ObjectCacheKeyRegistry();\n\n    const key1 = createMockObjectCacheKey(\"Employee\", \"emp1\", undefined);\n    const key2 = createMockObjectCacheKey(\n      \"Employee\",\n      \"emp1\",\n      createMockRdp(\"address\"),\n    );\n    const key3 = createMockObjectCacheKey(\n      \"Employee\",\n      \"emp1\",\n      createMockRdp(\"phone\"),\n    );\n\n    expect(registry.getVariantCount(\"Employee\", \"emp1\")).toBe(0);\n\n    registry.register(key1, \"Employee\", \"emp1\", undefined);\n    expect(registry.getVariantCount(\"Employee\", \"emp1\")).toBe(1);\n\n    registry.register(\n      key2,\n      \"Employee\",\n      \"emp1\",\n      createMockRdp(\"address\"),\n    );\n    expect(registry.getVariantCount(\"Employee\", \"emp1\")).toBe(2);\n\n    registry.register(\n      key3,\n      \"Employee\",\n      \"emp1\",\n      createMockRdp(\"phone\"),\n    );\n    expect(registry.getVariantCount(\"Employee\", \"emp1\")).toBe(3);\n\n    registry.unregister(key1);\n    expect(registry.getVariantCount(\"Employee\", \"emp1\")).toBe(2);\n\n    expect(registry.getVariantCount(\"Employee\", \"emp999\")).toBe(0);\n  });\n\n  it(\"handles registering same key multiple times\", () => {\n    const registry = new ObjectCacheKeyRegistry();\n\n    const key1 = createMockObjectCacheKey(\"Employee\", \"emp1\", undefined);\n\n    registry.register(key1, \"Employee\", \"emp1\", undefined);\n    registry.register(key1, \"Employee\", \"emp1\", undefined);\n    registry.register(key1, \"Employee\", \"emp1\", undefined);\n\n    // Should still have only one variant despite multiple registrations\n    expect(registry.getVariantCount(\"Employee\", \"emp1\")).toBe(1);\n\n    registry.unregister(key1);\n    expect(registry.getVariantCount(\"Employee\", \"emp1\")).toBe(0);\n  });\n\n  it(\"handles unregistering non-existent keys gracefully\", () => {\n    const registry = new ObjectCacheKeyRegistry();\n\n    const key1 = createMockObjectCacheKey(\"Employee\", \"emp1\", undefined);\n    const key2 = createMockObjectCacheKey(\"Employee\", \"emp2\", undefined);\n\n    registry.register(key1, \"Employee\", \"emp1\", undefined);\n\n    // Unregistering non-existent key should not throw\n    registry.unregister(key2);\n\n    // key1 should still be registered\n    expect(registry.getVariantCount(\"Employee\", \"emp1\")).toBe(1);\n  });\n\n  it(\"properly cleans up memory when all variants are unregistered\", () => {\n    const registry = new ObjectCacheKeyRegistry();\n\n    const key1 = createMockObjectCacheKey(\"Employee\", \"emp1\", undefined);\n    const key2 = createMockObjectCacheKey(\n      \"Employee\",\n      \"emp1\",\n      createMockRdp(\"address\"),\n    );\n\n    registry.register(key1, \"Employee\", \"emp1\", undefined);\n    registry.register(\n      key2,\n      \"Employee\",\n      \"emp1\",\n      createMockRdp(\"address\"),\n    );\n\n    expect(registry.getVariantCount(\"Employee\", \"emp1\")).toBe(2);\n\n    registry.unregister(key1);\n    registry.unregister(key2);\n\n    // Should return 0 as the entire entry should be cleaned up\n    expect(registry.getVariantCount(\"Employee\", \"emp1\")).toBe(0);\n    expect(registry.getVariants(\"Employee\", \"emp1\").size).toBe(0);\n  });\n});\n"],"mappings":"AAAA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;;AAOA,SAASA,QAAQ,EAAEC,MAAM,EAAEC,EAAE,QAAQ,QAAQ;AAK7C,SAASC,sBAAsB,QAAQ,6BAA6B;AAGpE;AACA;AACA;AACA;AACA;AACA,SAASC,aAAaA,CAAC,GAAGC,MAAgB,EAAkB;EAC1D,MAAMC,GAAQ,GAAG,CAAC,CAAC;EACnB,KAAK,MAAMC,KAAK,IAAIF,MAAM,EAAE;IAC1BC,GAAG,CAACC,KAAK,CAAC,GAAG,OAAO,CAAC,CAAC,CAGpB;EACJ;EACA,OAAOD,GAAG;AACZ;;AAEA;AACA;AACA;AACA,SAASE,wBAAwBA,CAC/BC,OAAe,EACfC,EAAU,EACVC,SAA0B,EACV;EAChB,OAAO;IACLC,IAAI,EAAE,QAAiB;IACvBC,SAAS,EAAE,CAACJ,OAAO,EAAEC,EAAE,EAAEC,SAAS,CAAC;IACnCG,UAAU,EAAE;MACVC,KAAK,EAAE,CAAC,CAAiB;MACzBC,KAAK,EAAE,CAAC,CAAgB;MACxBC,IAAI,EAAE,CAACR,OAAO,EAAEC,EAAE,EAAEC,SAAS;IAC/B;EACF,CAAC;AACH;AAEAX,QAAQ,CAAC,wBAAwB,EAAE,MAAM;EACvCE,EAAE,CAAC,4CAA4C,EAAE,MAAM;IACrD,MAAMgB,QAAQ,GAAG,IAAIf,sBAAsB,CAAC,CAAC;IAE7C,MAAMgB,cAAc,GAAGf,aAAa,CAAC,SAAS,EAAE,UAAU,CAAC;IAC3D,MAAMgB,YAAY,GAAGhB,aAAa,CAAC,OAAO,EAAE,UAAU,CAAC;IAEvD,MAAMiB,IAAI,GAAGb,wBAAwB,CAAC,UAAU,EAAE,MAAM,EAAEc,SAAS,CAAC;IACpE,MAAMC,IAAI,GAAGf,wBAAwB,CAAC,UAAU,EAAE,MAAM,EAAEW,cAAc,CAAC;IACzE,MAAMK,IAAI,GAAGhB,wBAAwB,CAAC,UAAU,EAAE,MAAM,EAAEY,YAAY,CAAC;IACvE,MAAMK,YAAY,GAAGjB,wBAAwB,CAC3C,UAAU,EACV,MAAM,EACNc,SACF,CAAC;IAEDJ,QAAQ,CAACQ,QAAQ,CAACL,IAAI,EAAE,UAAU,EAAE,MAAM,EAAEC,SAAS,CAAC;IACtDJ,QAAQ,CAACQ,QAAQ,CAACH,IAAI,EAAE,UAAU,EAAE,MAAM,EAAEJ,cAAc,CAAC;IAC3DD,QAAQ,CAACQ,QAAQ,CAACF,IAAI,EAAE,UAAU,EAAE,MAAM,EAAEJ,YAAY,CAAC;IACzDF,QAAQ,CAACQ,QAAQ,CAACD,YAAY,EAAE,UAAU,EAAE,MAAM,EAAEH,SAAS,CAAC;IAE9D,MAAMK,QAAQ,GAAGT,QAAQ,CAACU,WAAW,CAAC,UAAU,EAAE,MAAM,CAAC;IACzD3B,MAAM,CAAC0B,QAAQ,CAACE,IAAI,CAAC,CAACC,IAAI,CAAC,CAAC,CAAC;IAC7B7B,MAAM,CAAC0B,QAAQ,CAACI,GAAG,CAACV,IAAI,CAAC,CAAC,CAACS,IAAI,CAAC,IAAI,CAAC;IACrC7B,MAAM,CAAC0B,QAAQ,CAACI,GAAG,CAACR,IAAI,CAAC,CAAC,CAACO,IAAI,CAAC,IAAI,CAAC;IACrC7B,MAAM,CAAC0B,QAAQ,CAACI,GAAG,CAACP,IAAI,CAAC,CAAC,CAACM,IAAI,CAAC,IAAI,CAAC;IACrC7B,MAAM,CAAC0B,QAAQ,CAACI,GAAG,CAACN,YAAY,CAAC,CAAC,CAACK,IAAI,CAAC,KAAK,CAAC;;IAE9C;IACA,MAAME,SAAS,GAAGd,QAAQ,CAACU,WAAW,CAAC,UAAU,EAAE,MAAM,CAAC;IAC1D3B,MAAM,CAAC+B,SAAS,CAACH,IAAI,CAAC,CAACC,IAAI,CAAC,CAAC,CAAC;IAC9B7B,MAAM,CAAC+B,SAAS,CAACD,GAAG,CAACN,YAAY,CAAC,CAAC,CAACK,IAAI,CAAC,IAAI,CAAC;EAChD,CAAC,CAAC;EAEF5B,EAAE,CAAC,mCAAmC,EAAE,MAAM;IAC5C,MAAMgB,QAAQ,GAAG,IAAIf,sBAAsB,CAAC,CAAC;IAE7C,MAAM8B,OAAO,GAAG7B,aAAa,CAAC,SAAS,EAAE,UAAU,CAAC;IACpD,MAAMiB,IAAI,GAAGb,wBAAwB,CAAC,UAAU,EAAE,MAAM,EAAEc,SAAS,CAAC;IACpE,MAAMC,IAAI,GAAGf,wBAAwB,CAAC,UAAU,EAAE,MAAM,EAAEyB,OAAO,CAAC;IAElEf,QAAQ,CAACQ,QAAQ,CAACL,IAAI,EAAE,UAAU,EAAE,MAAM,EAAEC,SAAS,CAAC;IACtDJ,QAAQ,CAACQ,QAAQ,CAACH,IAAI,EAAE,UAAU,EAAE,MAAM,EAAEU,OAAO,CAAC;;IAEpD;IACAhC,MAAM,CAACiB,QAAQ,CAACU,WAAW,CAAC,UAAU,EAAE,MAAM,CAAC,CAACC,IAAI,CAAC,CAACC,IAAI,CAAC,CAAC,CAAC;IAE7DZ,QAAQ,CAACgB,UAAU,CAACb,IAAI,CAAC;;IAEzB;IACA,MAAMc,OAAO,GAAGjB,QAAQ,CAACU,WAAW,CAAC,UAAU,EAAE,MAAM,CAAC;IACxD3B,MAAM,CAACkC,OAAO,CAACN,IAAI,CAAC,CAACC,IAAI,CAAC,CAAC,CAAC;IAC5B7B,MAAM,CAACkC,OAAO,CAACJ,GAAG,CAACR,IAAI,CAAC,CAAC,CAACO,IAAI,CAAC,IAAI,CAAC;IACpC7B,MAAM,CAACkC,OAAO,CAACJ,GAAG,CAACV,IAAI,CAAC,CAAC,CAACS,IAAI,CAAC,KAAK,CAAC;EACvC,CAAC,CAAC;EAEF5B,EAAE,CAAC,gDAAgD,EAAE,MAAM;IACzD,MAAMgB,QAAQ,GAAG,IAAIf,sBAAsB,CAAC,CAAC;IAE7C,MAAMkB,IAAI,GAAGb,wBAAwB,CAAC,UAAU,EAAE,MAAM,EAAEc,SAAS,CAAC;IACpE,MAAMC,IAAI,GAAGf,wBAAwB,CACnC,UAAU,EACV,MAAM,EACNJ,aAAa,CAAC,SAAS,CACzB,CAAC;IAEDc,QAAQ,CAACQ,QAAQ,CAACL,IAAI,EAAE,UAAU,EAAE,MAAM,EAAEC,SAAS,CAAC;IACtDJ,QAAQ,CAACQ,QAAQ,CACfH,IAAI,EACJ,UAAU,EACV,MAAM,EACNnB,aAAa,CAAC,SAAS,CACzB,CAAC;IAED,MAAMuB,QAAQ,GAAGT,QAAQ,CAACU,WAAW,CAAC,UAAU,EAAE,MAAM,CAAC;IACzD3B,MAAM,CAAC0B,QAAQ,CAACE,IAAI,CAAC,CAACC,IAAI,CAAC,CAAC,CAAC;IAC7B7B,MAAM,CAAC0B,QAAQ,CAACI,GAAG,CAACV,IAAI,CAAC,CAAC,CAACS,IAAI,CAAC,IAAI,CAAC;IACrC7B,MAAM,CAAC0B,QAAQ,CAACI,GAAG,CAACR,IAAI,CAAC,CAAC,CAACO,IAAI,CAAC,IAAI,CAAC;;IAErC;IACA,MAAMM,KAAK,GAAGlB,QAAQ,CAACU,WAAW,CAAC,UAAU,EAAE,QAAQ,CAAC;IACxD3B,MAAM,CAACmC,KAAK,CAACP,IAAI,CAAC,CAACC,IAAI,CAAC,CAAC,CAAC;EAC5B,CAAC,CAAC;EAEF5B,EAAE,CAAC,wCAAwC,EAAE,MAAM;IACjD,MAAMgB,QAAQ,GAAG,IAAIf,sBAAsB,CAAC,CAAC;IAE7C,MAAMkC,aAAa,GAAG7B,wBAAwB,CAC5C,UAAU,EACV,MAAM,EACNc,SACF,CAAC;IACD,MAAMgB,UAAU,GAAG9B,wBAAwB,CACzC,UAAU,EACV,MAAM,EACNJ,aAAa,CAAC,SAAS,CACzB,CAAC;IAEDc,QAAQ,CAACQ,QAAQ,CAACW,aAAa,EAAE,UAAU,EAAE,MAAM,EAAEf,SAAS,CAAC;IAC/DJ,QAAQ,CAACQ,QAAQ,CACfY,UAAU,EACV,UAAU,EACV,MAAM,EACNlC,aAAa,CAAC,SAAS,CACzB,CAAC;IAEDH,MAAM,CAACiB,QAAQ,CAACqB,YAAY,CAACF,aAAa,CAAC,CAAC,CAACP,IAAI,CAAC,KAAK,CAAC;IACxD7B,MAAM,CAACiB,QAAQ,CAACqB,YAAY,CAACD,UAAU,CAAC,CAAC,CAACR,IAAI,CAAC,IAAI,CAAC;IAEpD7B,MAAM,CAACiB,QAAQ,CAACsB,YAAY,CAACH,aAAa,CAAC,CAAC,CAACI,aAAa,CAAC,CAAC;IAE5D,MAAM9B,SAAS,GAAGO,QAAQ,CAACsB,YAAY,CAACF,UAAU,CAAC;IACnDrC,MAAM,CAACU,SAAS,CAAC,CAAC+B,WAAW,CAAC,CAAC;IAC/BzC,MAAM,CAAC0C,MAAM,CAACC,IAAI,CAACjC,SAAU,CAAC,CAAC,CAACkC,OAAO,CAAC,CAAC,SAAS,CAAC,CAAC;IACpD5C,MAAM,CAAC,OAAOU,SAAS,CAAEmC,OAAO,CAAC,CAAChB,IAAI,CAAC,UAAU,CAAC;EACpD,CAAC,CAAC;EAEF5B,EAAE,CAAC,mDAAmD,EAAE,MAAM;IAC5D,MAAMgB,QAAQ,GAAG,IAAIf,sBAAsB,CAAC,CAAC;IAE7C,MAAMkB,IAAI,GAAGb,wBAAwB,CAAC,UAAU,EAAE,MAAM,EAAEc,SAAS,CAAC;IACpE,MAAMC,IAAI,GAAGf,wBAAwB,CACnC,UAAU,EACV,MAAM,EACNJ,aAAa,CAAC,SAAS,CACzB,CAAC;IACD,MAAMoB,IAAI,GAAGhB,wBAAwB,CACnC,UAAU,EACV,MAAM,EACNJ,aAAa,CAAC,OAAO,CACvB,CAAC;IAEDH,MAAM,CAACiB,QAAQ,CAAC6B,eAAe,CAAC,UAAU,EAAE,MAAM,CAAC,CAAC,CAACjB,IAAI,CAAC,CAAC,CAAC;IAE5DZ,QAAQ,CAACQ,QAAQ,CAACL,IAAI,EAAE,UAAU,EAAE,MAAM,EAAEC,SAAS,CAAC;IACtDrB,MAAM,CAACiB,QAAQ,CAAC6B,eAAe,CAAC,UAAU,EAAE,MAAM,CAAC,CAAC,CAACjB,IAAI,CAAC,CAAC,CAAC;IAE5DZ,QAAQ,CAACQ,QAAQ,CACfH,IAAI,EACJ,UAAU,EACV,MAAM,EACNnB,aAAa,CAAC,SAAS,CACzB,CAAC;IACDH,MAAM,CAACiB,QAAQ,CAAC6B,eAAe,CAAC,UAAU,EAAE,MAAM,CAAC,CAAC,CAACjB,IAAI,CAAC,CAAC,CAAC;IAE5DZ,QAAQ,CAACQ,QAAQ,CACfF,IAAI,EACJ,UAAU,EACV,MAAM,EACNpB,aAAa,CAAC,OAAO,CACvB,CAAC;IACDH,MAAM,CAACiB,QAAQ,CAAC6B,eAAe,CAAC,UAAU,EAAE,MAAM,CAAC,CAAC,CAACjB,IAAI,CAAC,CAAC,CAAC;IAE5DZ,QAAQ,CAACgB,UAAU,CAACb,IAAI,CAAC;IACzBpB,MAAM,CAACiB,QAAQ,CAAC6B,eAAe,CAAC,UAAU,EAAE,MAAM,CAAC,CAAC,CAACjB,IAAI,CAAC,CAAC,CAAC;IAE5D7B,MAAM,CAACiB,QAAQ,CAAC6B,eAAe,CAAC,UAAU,EAAE,QAAQ,CAAC,CAAC,CAACjB,IAAI,CAAC,CAAC,CAAC;EAChE,CAAC,CAAC;EAEF5B,EAAE,CAAC,6CAA6C,EAAE,MAAM;IACtD,MAAMgB,QAAQ,GAAG,IAAIf,sBAAsB,CAAC,CAAC;IAE7C,MAAMkB,IAAI,GAAGb,wBAAwB,CAAC,UAAU,EAAE,MAAM,EAAEc,SAAS,CAAC;IAEpEJ,QAAQ,CAACQ,QAAQ,CAACL,IAAI,EAAE,UAAU,EAAE,MAAM,EAAEC,SAAS,CAAC;IACtDJ,QAAQ,CAACQ,QAAQ,CAACL,IAAI,EAAE,UAAU,EAAE,MAAM,EAAEC,SAAS,CAAC;IACtDJ,QAAQ,CAACQ,QAAQ,CAACL,IAAI,EAAE,UAAU,EAAE,MAAM,EAAEC,SAAS,CAAC;;IAEtD;IACArB,MAAM,CAACiB,QAAQ,CAAC6B,eAAe,CAAC,UAAU,EAAE,MAAM,CAAC,CAAC,CAACjB,IAAI,CAAC,CAAC,CAAC;IAE5DZ,QAAQ,CAACgB,UAAU,CAACb,IAAI,CAAC;IACzBpB,MAAM,CAACiB,QAAQ,CAAC6B,eAAe,CAAC,UAAU,EAAE,MAAM,CAAC,CAAC,CAACjB,IAAI,CAAC,CAAC,CAAC;EAC9D,CAAC,CAAC;EAEF5B,EAAE,CAAC,oDAAoD,EAAE,MAAM;IAC7D,MAAMgB,QAAQ,GAAG,IAAIf,sBAAsB,CAAC,CAAC;IAE7C,MAAMkB,IAAI,GAAGb,wBAAwB,CAAC,UAAU,EAAE,MAAM,EAAEc,SAAS,CAAC;IACpE,MAAMC,IAAI,GAAGf,wBAAwB,CAAC,UAAU,EAAE,MAAM,EAAEc,SAAS,CAAC;IAEpEJ,QAAQ,CAACQ,QAAQ,CAACL,IAAI,EAAE,UAAU,EAAE,MAAM,EAAEC,SAAS,CAAC;;IAEtD;IACAJ,QAAQ,CAACgB,UAAU,CAACX,IAAI,CAAC;;IAEzB;IACAtB,MAAM,CAACiB,QAAQ,CAAC6B,eAAe,CAAC,UAAU,EAAE,MAAM,CAAC,CAAC,CAACjB,IAAI,CAAC,CAAC,CAAC;EAC9D,CAAC,CAAC;EAEF5B,EAAE,CAAC,8DAA8D,EAAE,MAAM;IACvE,MAAMgB,QAAQ,GAAG,IAAIf,sBAAsB,CAAC,CAAC;IAE7C,MAAMkB,IAAI,GAAGb,wBAAwB,CAAC,UAAU,EAAE,MAAM,EAAEc,SAAS,CAAC;IACpE,MAAMC,IAAI,GAAGf,wBAAwB,CACnC,UAAU,EACV,MAAM,EACNJ,aAAa,CAAC,SAAS,CACzB,CAAC;IAEDc,QAAQ,CAACQ,QAAQ,CAACL,IAAI,EAAE,UAAU,EAAE,MAAM,EAAEC,SAAS,CAAC;IACtDJ,QAAQ,CAACQ,QAAQ,CACfH,IAAI,EACJ,UAAU,EACV,MAAM,EACNnB,aAAa,CAAC,SAAS,CACzB,CAAC;IAEDH,MAAM,CAACiB,QAAQ,CAAC6B,eAAe,CAAC,UAAU,EAAE,MAAM,CAAC,CAAC,CAACjB,IAAI,CAAC,CAAC,CAAC;IAE5DZ,QAAQ,CAACgB,UAAU,CAACb,IAAI,CAAC;IACzBH,QAAQ,CAACgB,UAAU,CAACX,IAAI,CAAC;;IAEzB;IACAtB,MAAM,CAACiB,QAAQ,CAAC6B,eAAe,CAAC,UAAU,EAAE,MAAM,CAAC,CAAC,CAACjB,IAAI,CAAC,CAAC,CAAC;IAC5D7B,MAAM,CAACiB,QAAQ,CAACU,WAAW,CAAC,UAAU,EAAE,MAAM,CAAC,CAACC,IAAI,CAAC,CAACC,IAAI,CAAC,CAAC,CAAC;EAC/D,CAAC,CAAC;AACJ,CAAC,CAAC","ignoreList":[]}

package/build/browser/observable/internal/object/ObjectQuery.js

@@ -14,12 +14,12 @@
  * limitations under the License.
  */
 
-import deepEqual from "fast-deep-equal";
 import { BehaviorSubject, connectable, map } from "rxjs";
 import { additionalContext } from "../../../Client.js";
 import { getBulkObjectLoader } from "../BulkObjectLoader.js";
 import { Query } from "../Query.js";
 import { tombstone } from "../tombstone.js";
+import { RDP_CONFIG_IDX } from "./ObjectCacheKey.js";
 export class ObjectQuery extends Query {
   #apiName;
   #pk;
@@ -64,65 +64,17 @@ export class ObjectQuery extends Query {
     });
   }
   writeToStore(data, status, batch) {
-    const entry = batch.read(this.cacheKey);
-    if (entry && deepEqual(data, entry.value)) {
-      // Check if both data AND status are the same
-      if (entry.status === status) {
-        if (process.env.NODE_ENV !== "production") {
-          this.logger?.child({
-            methodName: "writeToStore"
-          }).debug(`Object was deep equal and status unchanged (${status}), skipping update`);
-        }
-        // Return the existing entry without writing to avoid unnecessary notifications
-        return entry;
-      }
-      if (process.env.NODE_ENV !== "production") {
-        this.logger?.child({
-          methodName: "writeToStore"
-        }).debug(`Object was deep equal, just setting status (old status: ${entry.status}, new status: ${status})`);
-      }
-      // must do a "full write" here so that the lastUpdated is updated but we
-      // don't want to retrigger anyone's memoization on the value!
-      return batch.write(this.cacheKey, entry.value, status);
-    }
-    if (process.env.NODE_ENV !== "production") {
-      this.logger?.child({
-        methodName: "writeToStore"
-      }).debug(JSON.stringify({
-        status
-      }), data);
-    }
-    const ret = batch.write(this.cacheKey, data, status);
-    batch.changes.registerObject(this.cacheKey, data, /* isNew */!entry);
-    return ret;
+    batch.read(this.cacheKey);
+    const rdpConfig = this.cacheKey.otherKeys[RDP_CONFIG_IDX];
+    this.store.objectCacheKeyRegistry.register(this.cacheKey, this.#apiName, this.#pk, rdpConfig);
+    this.store.objects.propagateWrite(this.cacheKey, data, status, batch);
+    return batch.read(this.cacheKey);
   }
   deleteFromStore(status, batch) {
-    const entry = batch.read(this.cacheKey);
-    if (entry && deepEqual(tombstone, entry.value)) {
-      if (process.env.NODE_ENV !== "production") {
-        this.logger?.child({
-          methodName: "deleteFromStore"
-        }).debug(`Object was deep equal, just setting status`);
-      }
-      // must do a "full write" here so that the lastUpdated is updated but we
-      // don't want to retrigger anyone's memoization on the value!
-      return batch.write(this.cacheKey, entry.value, status);
-    }
-    if (process.env.NODE_ENV !== "production") {
-      this.logger?.child({
-        methodName: "deleteFromStore"
-      }).debug(JSON.stringify({
-        status
-      }));
-    }
-
-    // if there is no entry then there is nothing to do
-    if (!entry || !entry.value) {
-      return;
-    }
-    const ret = batch.delete(this.cacheKey, status);
-    batch.changes.deleteObject(this.cacheKey);
-    return ret;
+    const rdpConfig = this.cacheKey.otherKeys[RDP_CONFIG_IDX];
+    this.store.objectCacheKeyRegistry.register(this.cacheKey, this.#apiName, this.#pk, rdpConfig);
+    this.store.objects.propagateWrite(this.cacheKey, tombstone, status, batch);
+    return batch.read(this.cacheKey);
   }
   invalidateObjectType = (objectType, changes) => {
     if (this.#apiName === objectType) {

package/build/browser/observable/internal/object/ObjectQuery.js.map

@@ -1 +1 @@
-{"version":3,"file":"ObjectQuery.js","names":["deepEqual","BehaviorSubject","connectable","map","additionalContext","getBulkObjectLoader","Query","tombstone","ObjectQuery","apiName","pk","constructor","store","subject","type","cacheKey","opts","process","env","NODE_ENV","client","logger","child","msgPrefix","otherKeys","x","JSON","stringify","join","undefined","_createConnectable","pipe","status","object","value","lastUpdated","isOptimistic","connector","_fetchAndStore","methodName","debug","obj","fetch","batch","writeToStore","data","entry","read","write","ret","changes","registerObject","deleteFromStore","delete","deleteObject","invalidateObjectType","objectType","modified","add","revalidate","Promise","resolve"],"sources":["ObjectQuery.ts"],"sourcesContent":["/*\n * Copyright 2025 Palantir Technologies, Inc. All rights reserved.\n *\n * Licensed under the Apache License, Version 2.0 (the \"License\");\n * you may not use this file except in compliance with the License.\n * You may obtain a copy of the License at\n *\n *     http://www.apache.org/licenses/LICENSE-2.0\n *\n * Unless required by applicable law or agreed to in writing, software\n * distributed under the License is distributed on an \"AS IS\" BASIS,\n * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.\n * See the License for the specific language governing permissions and\n * limitations under the License.\n */\n\nimport type { ObjectTypeDefinition, PrimaryKeyType } from \"@osdk/api\";\nimport deepEqual from \"fast-deep-equal\";\nimport type { Connectable, Observable, Subject } from \"rxjs\";\nimport { BehaviorSubject, connectable, map } from \"rxjs\";\nimport { additionalContext } from \"../../../Client.js\";\nimport type { ObjectHolder } from \"../../../object/convertWireToOsdkObjects/ObjectHolder.js\";\nimport type { ObjectPayload } from \"../../ObjectPayload.js\";\nimport type {\n  CommonObserveOptions,\n  Status,\n} from \"../../ObservableClient/common.js\";\nimport type { BatchContext } from \"../BatchContext.js\";\nimport { getBulkObjectLoader } from \"../BulkObjectLoader.js\";\nimport type { Changes } from \"../Changes.js\";\nimport type { Entry } from \"../Layer.js\";\nimport { Query } from \"../Query.js\";\nimport type { Store } from \"../Store.js\";\nimport type { SubjectPayload } from \"../SubjectPayload.js\";\nimport { tombstone } from \"../tombstone.js\";\nimport type { ObjectCacheKey } from \"./ObjectCacheKey.js\";\n\nexport class ObjectQuery extends Query<\n  ObjectCacheKey,\n  ObjectPayload,\n  CommonObserveOptions\n> {\n  #apiName: string;\n  #pk: string | number | boolean;\n\n  constructor(\n    store: Store,\n    subject: Subject<SubjectPayload<ObjectCacheKey>>,\n    type: string,\n    pk: PrimaryKeyType<ObjectTypeDefinition>,\n    cacheKey: ObjectCacheKey,\n    opts: CommonObserveOptions,\n  ) {\n    super(\n      store,\n      subject,\n      opts,\n      cacheKey,\n      process.env.NODE_ENV !== \"production\"\n        ? (\n          store.client[additionalContext].logger?.child({}, {\n            msgPrefix: `ObjectQuery<${\n              cacheKey.otherKeys.map(x => JSON.stringify(x)).join(\", \")\n            }>`,\n          })\n        )\n        : undefined,\n    );\n    this.#apiName = type;\n    this.#pk = pk;\n  }\n\n  protected _createConnectable(\n    subject: Observable<SubjectPayload<ObjectCacheKey>>,\n  ): Connectable<ObjectPayload> {\n    return connectable<ObjectPayload>(\n      subject.pipe(\n        map((x) => {\n          return {\n            status: x.status,\n            object: x.value,\n            lastUpdated: x.lastUpdated,\n            isOptimistic: x.isOptimistic,\n          };\n        }),\n      ),\n      {\n        connector: () =>\n          new BehaviorSubject<ObjectPayload>({\n            status: \"init\",\n            object: undefined,\n            lastUpdated: 0,\n            isOptimistic: false,\n          }),\n      },\n    );\n  }\n\n  async _fetchAndStore(): Promise<void> {\n    if (process.env.NODE_ENV !== \"production\") {\n      this.logger?.child({ methodName: \"_fetchAndStore\" }).debug(\n        \"calling _fetchAndStore\",\n      );\n    }\n\n    // TODO: In the future, implement tracking of network requests to ensure\n    // we're not making unnecessary network calls. This would need dedicated\n    // tests separate from subscription notification tests.\n\n    const obj = await getBulkObjectLoader(this.store.client)\n      .fetch(this.#apiName, this.#pk);\n\n    this.store.batch({}, (batch) => {\n      this.writeToStore(obj, \"loaded\", batch);\n    });\n  }\n\n  writeToStore(\n    data: ObjectHolder,\n    status: Status,\n    batch: BatchContext,\n  ): Entry<ObjectCacheKey> {\n    const entry = batch.read(this.cacheKey);\n\n    if (entry && deepEqual(data, entry.value)) {\n      // Check if both data AND status are the same\n      if (entry.status === status) {\n        if (process.env.NODE_ENV !== \"production\") {\n          this.logger?.child({ methodName: \"writeToStore\" }).debug(\n            `Object was deep equal and status unchanged (${status}), skipping update`,\n          );\n        }\n        // Return the existing entry without writing to avoid unnecessary notifications\n        return entry;\n      }\n\n      if (process.env.NODE_ENV !== \"production\") {\n        this.logger?.child({ methodName: \"writeToStore\" }).debug(\n          `Object was deep equal, just setting status (old status: ${entry.status}, new status: ${status})`,\n        );\n      }\n      // must do a \"full write\" here so that the lastUpdated is updated but we\n      // don't want to retrigger anyone's memoization on the value!\n      return batch.write(this.cacheKey, entry.value, status);\n    }\n\n    if (process.env.NODE_ENV !== \"production\") {\n      this.logger?.child({ methodName: \"writeToStore\" }).debug(\n        JSON.stringify({ status }),\n        data,\n      );\n    }\n    const ret = batch.write(this.cacheKey, data, status);\n    batch.changes.registerObject(this.cacheKey, data, /* isNew */ !entry);\n\n    return ret;\n  }\n\n  deleteFromStore(\n    status: Status,\n    batch: BatchContext,\n  ): Entry<ObjectCacheKey> | undefined {\n    const entry = batch.read(this.cacheKey);\n\n    if (entry && deepEqual(tombstone, entry.value)) {\n      if (process.env.NODE_ENV !== \"production\") {\n        this.logger?.child({ methodName: \"deleteFromStore\" }).debug(\n          `Object was deep equal, just setting status`,\n        );\n      }\n      // must do a \"full write\" here so that the lastUpdated is updated but we\n      // don't want to retrigger anyone's memoization on the value!\n      return batch.write(this.cacheKey, entry.value, status);\n    }\n\n    if (process.env.NODE_ENV !== \"production\") {\n      this.logger?.child({ methodName: \"deleteFromStore\" }).debug(\n        JSON.stringify({ status }),\n      );\n    }\n\n    // if there is no entry then there is nothing to do\n    if (!entry || !entry.value) {\n      return;\n    }\n\n    const ret = batch.delete(this.cacheKey, status);\n    batch.changes.deleteObject(this.cacheKey);\n\n    return ret;\n  }\n\n  invalidateObjectType = (\n    objectType: string,\n    changes: Changes | undefined,\n  ): Promise<void> => {\n    if (this.#apiName === objectType) {\n      changes?.modified.add(this.cacheKey);\n      return this.revalidate(true);\n    }\n    return Promise.resolve();\n  };\n}\n"],"mappings":"AAAA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;;AAGA,OAAOA,SAAS,MAAM,iBAAiB;AAEvC,SAASC,eAAe,EAAEC,WAAW,EAAEC,GAAG,QAAQ,MAAM;AACxD,SAASC,iBAAiB,QAAQ,oBAAoB;AAQtD,SAASC,mBAAmB,QAAQ,wBAAwB;AAG5D,SAASC,KAAK,QAAQ,aAAa;AAGnC,SAASC,SAAS,QAAQ,iBAAiB;AAG3C,OAAO,MAAMC,WAAW,SAASF,KAAK,CAIpC;EACA,CAACG,OAAO;EACR,CAACC,EAAE;EAEHC,WAAWA,CACTC,KAAY,EACZC,OAAgD,EAChDC,IAAY,EACZJ,EAAwC,EACxCK,QAAwB,EACxBC,IAA0B,EAC1B;IACA,KAAK,CACHJ,KAAK,EACLC,OAAO,EACPG,IAAI,EACJD,QAAQ,EACRE,OAAO,CAACC,GAAG,CAACC,QAAQ,KAAK,YAAY,GAEjCP,KAAK,CAACQ,MAAM,CAAChB,iBAAiB,CAAC,CAACiB,MAAM,EAAEC,KAAK,CAAC,CAAC,CAAC,EAAE;MAChDC,SAAS,EAAE,eACTR,QAAQ,CAACS,SAAS,CAACrB,GAAG,CAACsB,CAAC,IAAIC,IAAI,CAACC,SAAS,CAACF,CAAC,CAAC,CAAC,CAACG,IAAI,CAAC,IAAI,CAAC;IAE7D,CAAC,CAAC,GAEFC,SACN,CAAC;IACD,IAAI,CAAC,CAACpB,OAAO,GAAGK,IAAI;IACpB,IAAI,CAAC,CAACJ,EAAE,GAAGA,EAAE;EACf;EAEUoB,kBAAkBA,CAC1BjB,OAAmD,EACvB;IAC5B,OAAOX,WAAW,CAChBW,OAAO,CAACkB,IAAI,CACV5B,GAAG,CAAEsB,CAAC,IAAK;MACT,OAAO;QACLO,MAAM,EAAEP,CAAC,CAACO,MAAM;QAChBC,MAAM,EAAER,CAAC,CAACS,KAAK;QACfC,WAAW,EAAEV,CAAC,CAACU,WAAW;QAC1BC,YAAY,EAAEX,CAAC,CAACW;MAClB,CAAC;IACH,CAAC,CACH,CAAC,EACD;MACEC,SAAS,EAAEA,CAAA,KACT,IAAIpC,eAAe,CAAgB;QACjC+B,MAAM,EAAE,MAAM;QACdC,MAAM,EAAEJ,SAAS;QACjBM,WAAW,EAAE,CAAC;QACdC,YAAY,EAAE;MAChB,CAAC;IACL,CACF,CAAC;EACH;EAEA,MAAME,cAAcA,CAAA,EAAkB;IACpC,IAAIrB,OAAO,CAACC,GAAG,CAACC,QAAQ,KAAK,YAAY,EAAE;MACzC,IAAI,CAACE,MAAM,EAAEC,KAAK,CAAC;QAAEiB,UAAU,EAAE;MAAiB,CAAC,CAAC,CAACC,KAAK,CACxD,wBACF,CAAC;IACH;;IAEA;IACA;IACA;;IAEA,MAAMC,GAAG,GAAG,MAAMpC,mBAAmB,CAAC,IAAI,CAACO,KAAK,CAACQ,MAAM,CAAC,CACrDsB,KAAK,CAAC,IAAI,CAAC,CAACjC,OAAO,EAAE,IAAI,CAAC,CAACC,EAAE,CAAC;IAEjC,IAAI,CAACE,KAAK,CAAC+B,KAAK,CAAC,CAAC,CAAC,EAAGA,KAAK,IAAK;MAC9B,IAAI,CAACC,YAAY,CAACH,GAAG,EAAE,QAAQ,EAAEE,KAAK,CAAC;IACzC,CAAC,CAAC;EACJ;EAEAC,YAAYA,CACVC,IAAkB,EAClBb,MAAc,EACdW,KAAmB,EACI;IACvB,MAAMG,KAAK,GAAGH,KAAK,CAACI,IAAI,CAAC,IAAI,CAAChC,QAAQ,CAAC;IAEvC,IAAI+B,KAAK,IAAI9C,SAAS,CAAC6C,IAAI,EAAEC,KAAK,CAACZ,KAAK,CAAC,EAAE;MACzC;MACA,IAAIY,KAAK,CAACd,MAAM,KAAKA,MAAM,EAAE;QAC3B,IAAIf,OAAO,CAACC,GAAG,CAACC,QAAQ,KAAK,YAAY,EAAE;UACzC,IAAI,CAACE,MAAM,EAAEC,KAAK,CAAC;YAAEiB,UAAU,EAAE;UAAe,CAAC,CAAC,CAACC,KAAK,CACtD,+CAA+CR,MAAM,oBACvD,CAAC;QACH;QACA;QACA,OAAOc,KAAK;MACd;MAEA,IAAI7B,OAAO,CAACC,GAAG,CAACC,QAAQ,KAAK,YAAY,EAAE;QACzC,IAAI,CAACE,MAAM,EAAEC,KAAK,CAAC;UAAEiB,UAAU,EAAE;QAAe,CAAC,CAAC,CAACC,KAAK,CACtD,2DAA2DM,KAAK,CAACd,MAAM,iBAAiBA,MAAM,GAChG,CAAC;MACH;MACA;MACA;MACA,OAAOW,KAAK,CAACK,KAAK,CAAC,IAAI,CAACjC,QAAQ,EAAE+B,KAAK,CAACZ,KAAK,EAAEF,MAAM,CAAC;IACxD;IAEA,IAAIf,OAAO,CAACC,GAAG,CAACC,QAAQ,KAAK,YAAY,EAAE;MACzC,IAAI,CAACE,MAAM,EAAEC,KAAK,CAAC;QAAEiB,UAAU,EAAE;MAAe,CAAC,CAAC,CAACC,KAAK,CACtDd,IAAI,CAACC,SAAS,CAAC;QAAEK;MAAO,CAAC,CAAC,EAC1Ba,IACF,CAAC;IACH;IACA,MAAMI,GAAG,GAAGN,KAAK,CAACK,KAAK,CAAC,IAAI,CAACjC,QAAQ,EAAE8B,IAAI,EAAEb,MAAM,CAAC;IACpDW,KAAK,CAACO,OAAO,CAACC,cAAc,CAAC,IAAI,CAACpC,QAAQ,EAAE8B,IAAI,EAAE,WAAY,CAACC,KAAK,CAAC;IAErE,OAAOG,GAAG;EACZ;EAEAG,eAAeA,CACbpB,MAAc,EACdW,KAAmB,EACgB;IACnC,MAAMG,KAAK,GAAGH,KAAK,CAACI,IAAI,CAAC,IAAI,CAAChC,QAAQ,CAAC;IAEvC,IAAI+B,KAAK,IAAI9C,SAAS,CAACO,SAAS,EAAEuC,KAAK,CAACZ,KAAK,CAAC,EAAE;MAC9C,IAAIjB,OAAO,CAACC,GAAG,CAACC,QAAQ,KAAK,YAAY,EAAE;QACzC,IAAI,CAACE,MAAM,EAAEC,KAAK,CAAC;UAAEiB,UAAU,EAAE;QAAkB,CAAC,CAAC,CAACC,KAAK,CACzD,4CACF,CAAC;MACH;MACA;MACA;MACA,OAAOG,KAAK,CAACK,KAAK,CAAC,IAAI,CAACjC,QAAQ,EAAE+B,KAAK,CAACZ,KAAK,EAAEF,MAAM,CAAC;IACxD;IAEA,IAAIf,OAAO,CAACC,GAAG,CAACC,QAAQ,KAAK,YAAY,EAAE;MACzC,IAAI,CAACE,MAAM,EAAEC,KAAK,CAAC;QAAEiB,UAAU,EAAE;MAAkB,CAAC,CAAC,CAACC,KAAK,CACzDd,IAAI,CAACC,SAAS,CAAC;QAAEK;MAAO,CAAC,CAC3B,CAAC;IACH;;IAEA;IACA,IAAI,CAACc,KAAK,IAAI,CAACA,KAAK,CAACZ,KAAK,EAAE;MAC1B;IACF;IAEA,MAAMe,GAAG,GAAGN,KAAK,CAACU,MAAM,CAAC,IAAI,CAACtC,QAAQ,EAAEiB,MAAM,CAAC;IAC/CW,KAAK,CAACO,OAAO,CAACI,YAAY,CAAC,IAAI,CAACvC,QAAQ,CAAC;IAEzC,OAAOkC,GAAG;EACZ;EAEAM,oBAAoB,GAAGA,CACrBC,UAAkB,EAClBN,OAA4B,KACV;IAClB,IAAI,IAAI,CAAC,CAACzC,OAAO,KAAK+C,UAAU,EAAE;MAChCN,OAAO,EAAEO,QAAQ,CAACC,GAAG,CAAC,IAAI,CAAC3C,QAAQ,CAAC;MACpC,OAAO,IAAI,CAAC4C,UAAU,CAAC,IAAI,CAAC;IAC9B;IACA,OAAOC,OAAO,CAACC,OAAO,CAAC,CAAC;EAC1B,CAAC;AACH","ignoreList":[]}
+{"version":3,"file":"ObjectQuery.js","names":["BehaviorSubject","connectable","map","additionalContext","getBulkObjectLoader","Query","tombstone","RDP_CONFIG_IDX","ObjectQuery","apiName","pk","constructor","store","subject","type","cacheKey","opts","process","env","NODE_ENV","client","logger","child","msgPrefix","otherKeys","x","JSON","stringify","join","undefined","_createConnectable","pipe","status","object","value","lastUpdated","isOptimistic","connector","_fetchAndStore","methodName","debug","obj","fetch","batch","writeToStore","data","read","rdpConfig","objectCacheKeyRegistry","register","objects","propagateWrite","deleteFromStore","invalidateObjectType","objectType","changes","modified","add","revalidate","Promise","resolve"],"sources":["ObjectQuery.ts"],"sourcesContent":["/*\n * Copyright 2025 Palantir Technologies, Inc. All rights reserved.\n *\n * Licensed under the Apache License, Version 2.0 (the \"License\");\n * you may not use this file except in compliance with the License.\n * You may obtain a copy of the License at\n *\n *     http://www.apache.org/licenses/LICENSE-2.0\n *\n * Unless required by applicable law or agreed to in writing, software\n * distributed under the License is distributed on an \"AS IS\" BASIS,\n * WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.\n * See the License for the specific language governing permissions and\n * limitations under the License.\n */\n\nimport type { ObjectTypeDefinition, PrimaryKeyType } from \"@osdk/api\";\nimport type { Connectable, Observable, Subject } from \"rxjs\";\nimport { BehaviorSubject, connectable, map } from \"rxjs\";\nimport { additionalContext } from \"../../../Client.js\";\nimport type { ObjectHolder } from \"../../../object/convertWireToOsdkObjects/ObjectHolder.js\";\nimport type { ObjectPayload } from \"../../ObjectPayload.js\";\nimport type {\n  CommonObserveOptions,\n  Status,\n} from \"../../ObservableClient/common.js\";\nimport type { BatchContext } from \"../BatchContext.js\";\nimport { getBulkObjectLoader } from \"../BulkObjectLoader.js\";\nimport type { Changes } from \"../Changes.js\";\nimport type { Entry } from \"../Layer.js\";\nimport { Query } from \"../Query.js\";\nimport type { Store } from \"../Store.js\";\nimport type { SubjectPayload } from \"../SubjectPayload.js\";\nimport { tombstone } from \"../tombstone.js\";\nimport { type ObjectCacheKey, RDP_CONFIG_IDX } from \"./ObjectCacheKey.js\";\n\nexport class ObjectQuery extends Query<\n  ObjectCacheKey,\n  ObjectPayload,\n  CommonObserveOptions\n> {\n  #apiName: string;\n  #pk: string | number | boolean;\n\n  constructor(\n    store: Store,\n    subject: Subject<SubjectPayload<ObjectCacheKey>>,\n    type: string,\n    pk: PrimaryKeyType<ObjectTypeDefinition>,\n    cacheKey: ObjectCacheKey,\n    opts: CommonObserveOptions,\n  ) {\n    super(\n      store,\n      subject,\n      opts,\n      cacheKey,\n      process.env.NODE_ENV !== \"production\"\n        ? (\n          store.client[additionalContext].logger?.child({}, {\n            msgPrefix: `ObjectQuery<${\n              cacheKey.otherKeys.map(x => JSON.stringify(x)).join(\", \")\n            }>`,\n          })\n        )\n        : undefined,\n    );\n    this.#apiName = type;\n    this.#pk = pk;\n  }\n\n  protected _createConnectable(\n    subject: Observable<SubjectPayload<ObjectCacheKey>>,\n  ): Connectable<ObjectPayload> {\n    return connectable<ObjectPayload>(\n      subject.pipe(\n        map((x) => {\n          return {\n            status: x.status,\n            object: x.value,\n            lastUpdated: x.lastUpdated,\n            isOptimistic: x.isOptimistic,\n          };\n        }),\n      ),\n      {\n        connector: () =>\n          new BehaviorSubject<ObjectPayload>({\n            status: \"init\",\n            object: undefined,\n            lastUpdated: 0,\n            isOptimistic: false,\n          }),\n      },\n    );\n  }\n\n  async _fetchAndStore(): Promise<void> {\n    if (process.env.NODE_ENV !== \"production\") {\n      this.logger?.child({ methodName: \"_fetchAndStore\" }).debug(\n        \"calling _fetchAndStore\",\n      );\n    }\n\n    // TODO: In the future, implement tracking of network requests to ensure\n    // we're not making unnecessary network calls. This would need dedicated\n    // tests separate from subscription notification tests.\n\n    const obj = await getBulkObjectLoader(this.store.client)\n      .fetch(this.#apiName, this.#pk);\n\n    this.store.batch({}, (batch) => {\n      this.writeToStore(obj, \"loaded\", batch);\n    });\n  }\n\n  writeToStore(\n    data: ObjectHolder,\n    status: Status,\n    batch: BatchContext,\n  ): Entry<ObjectCacheKey> {\n    const entry = batch.read(this.cacheKey);\n    const rdpConfig = this.cacheKey.otherKeys[RDP_CONFIG_IDX];\n\n    this.store.objectCacheKeyRegistry.register(\n      this.cacheKey,\n      this.#apiName,\n      this.#pk,\n      rdpConfig,\n    );\n\n    this.store.objects.propagateWrite(this.cacheKey, data, status, batch);\n\n    return batch.read(this.cacheKey)!;\n  }\n\n  deleteFromStore(\n    status: Status,\n    batch: BatchContext,\n  ): Entry<ObjectCacheKey> | undefined {\n    const rdpConfig = this.cacheKey.otherKeys[RDP_CONFIG_IDX];\n\n    this.store.objectCacheKeyRegistry.register(\n      this.cacheKey,\n      this.#apiName,\n      this.#pk,\n      rdpConfig,\n    );\n\n    this.store.objects.propagateWrite(\n      this.cacheKey,\n      tombstone,\n      status,\n      batch,\n    );\n\n    return batch.read(this.cacheKey);\n  }\n\n  invalidateObjectType = (\n    objectType: string,\n    changes: Changes | undefined,\n  ): Promise<void> => {\n    if (this.#apiName === objectType) {\n      changes?.modified.add(this.cacheKey);\n      return this.revalidate(true);\n    }\n    return Promise.resolve();\n  };\n}\n"],"mappings":"AAAA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;AACA;;AAIA,SAASA,eAAe,EAAEC,WAAW,EAAEC,GAAG,QAAQ,MAAM;AACxD,SAASC,iBAAiB,QAAQ,oBAAoB;AAQtD,SAASC,mBAAmB,QAAQ,wBAAwB;AAG5D,SAASC,KAAK,QAAQ,aAAa;AAGnC,SAASC,SAAS,QAAQ,iBAAiB;AAC3C,SAA8BC,cAAc,QAAQ,qBAAqB;AAEzE,OAAO,MAAMC,WAAW,SAASH,KAAK,CAIpC;EACA,CAACI,OAAO;EACR,CAACC,EAAE;EAEHC,WAAWA,CACTC,KAAY,EACZC,OAAgD,EAChDC,IAAY,EACZJ,EAAwC,EACxCK,QAAwB,EACxBC,IAA0B,EAC1B;IACA,KAAK,CACHJ,KAAK,EACLC,OAAO,EACPG,IAAI,EACJD,QAAQ,EACRE,OAAO,CAACC,GAAG,CAACC,QAAQ,KAAK,YAAY,GAEjCP,KAAK,CAACQ,MAAM,CAACjB,iBAAiB,CAAC,CAACkB,MAAM,EAAEC,KAAK,CAAC,CAAC,CAAC,EAAE;MAChDC,SAAS,EAAE,eACTR,QAAQ,CAACS,SAAS,CAACtB,GAAG,CAACuB,CAAC,IAAIC,IAAI,CAACC,SAAS,CAACF,CAAC,CAAC,CAAC,CAACG,IAAI,CAAC,IAAI,CAAC;IAE7D,CAAC,CAAC,GAEFC,SACN,CAAC;IACD,IAAI,CAAC,CAACpB,OAAO,GAAGK,IAAI;IACpB,IAAI,CAAC,CAACJ,EAAE,GAAGA,EAAE;EACf;EAEUoB,kBAAkBA,CAC1BjB,OAAmD,EACvB;IAC5B,OAAOZ,WAAW,CAChBY,OAAO,CAACkB,IAAI,CACV7B,GAAG,CAAEuB,CAAC,IAAK;MACT,OAAO;QACLO,MAAM,EAAEP,CAAC,CAACO,MAAM;QAChBC,MAAM,EAAER,CAAC,CAACS,KAAK;QACfC,WAAW,EAAEV,CAAC,CAACU,WAAW;QAC1BC,YAAY,EAAEX,CAAC,CAACW;MAClB,CAAC;IACH,CAAC,CACH,CAAC,EACD;MACEC,SAAS,EAAEA,CAAA,KACT,IAAIrC,eAAe,CAAgB;QACjCgC,MAAM,EAAE,MAAM;QACdC,MAAM,EAAEJ,SAAS;QACjBM,WAAW,EAAE,CAAC;QACdC,YAAY,EAAE;MAChB,CAAC;IACL,CACF,CAAC;EACH;EAEA,MAAME,cAAcA,CAAA,EAAkB;IACpC,IAAIrB,OAAO,CAACC,GAAG,CAACC,QAAQ,KAAK,YAAY,EAAE;MACzC,IAAI,CAACE,MAAM,EAAEC,KAAK,CAAC;QAAEiB,UAAU,EAAE;MAAiB,CAAC,CAAC,CAACC,KAAK,CACxD,wBACF,CAAC;IACH;;IAEA;IACA;IACA;;IAEA,MAAMC,GAAG,GAAG,MAAMrC,mBAAmB,CAAC,IAAI,CAACQ,KAAK,CAACQ,MAAM,CAAC,CACrDsB,KAAK,CAAC,IAAI,CAAC,CAACjC,OAAO,EAAE,IAAI,CAAC,CAACC,EAAE,CAAC;IAEjC,IAAI,CAACE,KAAK,CAAC+B,KAAK,CAAC,CAAC,CAAC,EAAGA,KAAK,IAAK;MAC9B,IAAI,CAACC,YAAY,CAACH,GAAG,EAAE,QAAQ,EAAEE,KAAK,CAAC;IACzC,CAAC,CAAC;EACJ;EAEAC,YAAYA,CACVC,IAAkB,EAClBb,MAAc,EACdW,KAAmB,EACI;IACTA,KAAK,CAACG,IAAI,CAAC,IAAI,CAAC/B,QAAQ,CAAC;IACvC,MAAMgC,SAAS,GAAG,IAAI,CAAChC,QAAQ,CAACS,SAAS,CAACjB,cAAc,CAAC;IAEzD,IAAI,CAACK,KAAK,CAACoC,sBAAsB,CAACC,QAAQ,CACxC,IAAI,CAAClC,QAAQ,EACb,IAAI,CAAC,CAACN,OAAO,EACb,IAAI,CAAC,CAACC,EAAE,EACRqC,SACF,CAAC;IAED,IAAI,CAACnC,KAAK,CAACsC,OAAO,CAACC,cAAc,CAAC,IAAI,CAACpC,QAAQ,EAAE8B,IAAI,EAAEb,MAAM,EAAEW,KAAK,CAAC;IAErE,OAAOA,KAAK,CAACG,IAAI,CAAC,IAAI,CAAC/B,QAAQ,CAAC;EAClC;EAEAqC,eAAeA,CACbpB,MAAc,EACdW,KAAmB,EACgB;IACnC,MAAMI,SAAS,GAAG,IAAI,CAAChC,QAAQ,CAACS,SAAS,CAACjB,cAAc,CAAC;IAEzD,IAAI,CAACK,KAAK,CAACoC,sBAAsB,CAACC,QAAQ,CACxC,IAAI,CAAClC,QAAQ,EACb,IAAI,CAAC,CAACN,OAAO,EACb,IAAI,CAAC,CAACC,EAAE,EACRqC,SACF,CAAC;IAED,IAAI,CAACnC,KAAK,CAACsC,OAAO,CAACC,cAAc,CAC/B,IAAI,CAACpC,QAAQ,EACbT,SAAS,EACT0B,MAAM,EACNW,KACF,CAAC;IAED,OAAOA,KAAK,CAACG,IAAI,CAAC,IAAI,CAAC/B,QAAQ,CAAC;EAClC;EAEAsC,oBAAoB,GAAGA,CACrBC,UAAkB,EAClBC,OAA4B,KACV;IAClB,IAAI,IAAI,CAAC,CAAC9C,OAAO,KAAK6C,UAAU,EAAE;MAChCC,OAAO,EAAEC,QAAQ,CAACC,GAAG,CAAC,IAAI,CAAC1C,QAAQ,CAAC;MACpC,OAAO,IAAI,CAAC2C,UAAU,CAAC,IAAI,CAAC;IAC9B;IACA,OAAOC,OAAO,CAACC,OAAO,CAAC,CAAC;EAC1B,CAAC;AACH","ignoreList":[]}

package/build/browser/observable/internal/object/ObjectsHelper.js

@@ -14,18 +14,21 @@
  * limitations under the License.
  */
 
+import deepEqual from "fast-deep-equal";
 import { AbstractHelper } from "../AbstractHelper.js";
+import { tombstone } from "../tombstone.js";
+import { mergeObjectFields } from "../utils/rdpFieldOperations.js";
 import { ObjectQuery } from "./ObjectQuery.js";
 export class ObjectsHelper extends AbstractHelper {
   observe(options, subFn) {
     return super.observe(options, subFn);
   }
-  getQuery(options) {
+  getQuery(options, rdpConfig) {
     const apiName = typeof options.apiName === "string" ? options.apiName : options.apiName.apiName;
     const {
       pk
     } = options;
-    const objectCacheKey = this.cacheKeys.get("object", apiName, pk);
+    const objectCacheKey = this.cacheKeys.get("object", apiName, pk, rdpConfig ?? undefined);
     return this.store.queries.get(objectCacheKey, () => new ObjectQuery(this.store, this.store.subjects.get(objectCacheKey), apiName, pk, objectCacheKey, {
       dedupeInterval: 0
     }));
@@ -33,16 +36,73 @@ export class ObjectsHelper extends AbstractHelper {
 
   /**
    * Internal helper method for writing objects to the store and returning their
-   * object keys
+   * object keys. For list queries with RDPs, the rdpConfig is included in the
+   * cache key to ensure proper data isolation.
    * @internal
    */
-  storeOsdkInstances(values, batch) {
+  storeOsdkInstances(values, batch, rdpConfig) {
     // update the cache for any object that has changed
     // and save the mapped values to return
     return values.map(v => this.getQuery({
       apiName: v.$apiName,
       pk: v.$primaryKey
-    }).writeToStore(v, "loaded", batch).cacheKey);
+    }, rdpConfig).writeToStore(v, "loaded", batch).cacheKey);
+  }
+
+  /**
+   * Write an object to cache and propagate to all related cache keys
+   * @internal
+   */
+  propagateWrite(sourceCacheKey, value, status, batch) {
+    const existing = batch.read(sourceCacheKey);
+    const dataChanged = !existing || existing.value === undefined || value === tombstone || !deepEqual(existing.value, value);
+    const statusChanged = !existing || existing.status !== status;
+    if (!dataChanged && !statusChanged) {
+      return;
+    }
+    const valueToWrite = !dataChanged && existing ? existing.value : value;
+    batch.write(sourceCacheKey, valueToWrite, status);
+    if (value !== tombstone) {
+      batch.changes.registerObject(sourceCacheKey, value, !existing);
+    }
+    const metadata = this.store.objectCacheKeyRegistry.getMetadata(sourceCacheKey);
+    const relatedKeys = metadata ? this.store.objectCacheKeyRegistry.getVariants(metadata.apiName, metadata.primaryKey) : new Set([sourceCacheKey]);
+    for (const targetKey of relatedKeys) {
+      if (targetKey === sourceCacheKey || !this.isKeyActive(targetKey)) {
+        continue;
+      }
+      if (value === tombstone) {
+        batch.write(targetKey, tombstone, status);
+        continue;
+      }
+      const targetCurrentValue = batch.read(targetKey)?.value;
+      const merged = this.mergeForTarget(value, targetCurrentValue && this.isObjectHolder(targetCurrentValue) ? targetCurrentValue : undefined, sourceCacheKey, targetKey);
+      batch.write(targetKey, merged, status);
+    }
+  }
+
+  /**
+   * Check if a cache key is actively observed
+   */
+  isKeyActive(key) {
+    const subject = this.store.subjects.peek(key);
+    return subject?.observed === true;
+  }
+
+  /**
+   * Type guard to check if a value is an ObjectHolder
+   */
+  isObjectHolder(value) {
+    return value != null && typeof value === "object" && "$apiName" in value && "$primaryKey" in value;
+  }
+
+  /**
+   * Merge object data for a specific target cache key, preserving RDP fields
+   */
+  mergeForTarget(sourceValue, targetCurrentValue, sourceCacheKey, targetCacheKey) {
+    const sourceRdpFields = this.store.objectCacheKeyRegistry.getRdpFieldSet(sourceCacheKey);
+    const targetRdpFields = this.store.objectCacheKeyRegistry.getRdpFieldSet(targetCacheKey);
+    return mergeObjectFields(sourceValue, sourceRdpFields, targetRdpFields, targetCurrentValue);
   }
 }
 //# sourceMappingURL=ObjectsHelper.js.map