@oxyhq/core 3.8.1 → 3.9.0

package/dist/cjs/mixins/OxyServices.workspaces.js

@@ -26,6 +26,9 @@ function OxyServicesWorkspacesMixin(Base) {
         async createWorkspace(data) {
             try {
                 const res = await this.makeRequest('POST', '/workspaces', data, { cache: false });
+                // Bust the cached workspace list so the new workspace appears on the
+                // next `getWorkspaces()` read within the TTL window.
+                this.clearCacheEntry('GET:/workspaces');
                 return res.workspace;
             }
             catch (error) {
@@ -53,6 +56,9 @@ function OxyServicesWorkspacesMixin(Base) {
         async updateWorkspace(workspaceId, data) {
             try {
                 const res = await this.makeRequest('PATCH', `/workspaces/${encodeURIComponent(workspaceId)}`, data, { cache: false });
+                // Bust the cached detail and list — both surface workspace fields.
+                this.clearCacheEntry(`GET:/workspaces/${encodeURIComponent(workspaceId)}`);
+                this.clearCacheEntry('GET:/workspaces');
                 return res.workspace;
             }
             catch (error) {
@@ -65,7 +71,12 @@ function OxyServicesWorkspacesMixin(Base) {
          */
         async deleteWorkspace(workspaceId) {
             try {
-                return await this.makeRequest('DELETE', `/workspaces/${encodeURIComponent(workspaceId)}`, undefined, { cache: false });
+                const result = await this.makeRequest('DELETE', `/workspaces/${encodeURIComponent(workspaceId)}`, undefined, { cache: false });
+                // Bust every cached representation of the deleted workspace.
+                this.clearCacheEntry(`GET:/workspaces/${encodeURIComponent(workspaceId)}`);
+                this.clearCacheEntry(`GET:/workspaces/${encodeURIComponent(workspaceId)}/members`);
+                this.clearCacheEntry('GET:/workspaces');
+                return result;
             }
             catch (error) {
                 throw this.handleError(error);
@@ -94,6 +105,7 @@ function OxyServicesWorkspacesMixin(Base) {
         async inviteWorkspaceMember(workspaceId, data) {
             try {
                 const res = await this.makeRequest('POST', `/workspaces/${encodeURIComponent(workspaceId)}/members`, data, { cache: false });
+                this._invalidateWorkspaceMembership(workspaceId);
                 return res.member;
             }
             catch (error) {
@@ -109,6 +121,7 @@ function OxyServicesWorkspacesMixin(Base) {
         async updateWorkspaceMember(workspaceId, memberId, data) {
             try {
                 const res = await this.makeRequest('PATCH', `/workspaces/${encodeURIComponent(workspaceId)}/members/${encodeURIComponent(memberId)}`, data, { cache: false });
+                this._invalidateWorkspaceMembership(workspaceId);
                 return res.member;
             }
             catch (error) {
@@ -122,7 +135,9 @@ function OxyServicesWorkspacesMixin(Base) {
          */
         async removeWorkspaceMember(workspaceId, memberId) {
             try {
-                return await this.makeRequest('DELETE', `/workspaces/${encodeURIComponent(workspaceId)}/members/${encodeURIComponent(memberId)}`, undefined, { cache: false });
+                const result = await this.makeRequest('DELETE', `/workspaces/${encodeURIComponent(workspaceId)}/members/${encodeURIComponent(memberId)}`, undefined, { cache: false });
+                this._invalidateWorkspaceMembership(workspaceId);
+                return result;
             }
             catch (error) {
                 throw this.handleError(error);
@@ -136,11 +151,31 @@ function OxyServicesWorkspacesMixin(Base) {
          */
         async transferWorkspaceOwnership(workspaceId, data) {
             try {
-                return await this.makeRequest('POST', `/workspaces/${encodeURIComponent(workspaceId)}/transfer-ownership`, data, { cache: false });
+                const result = await this.makeRequest('POST', `/workspaces/${encodeURIComponent(workspaceId)}/transfer-ownership`, data, { cache: false });
+                // Ownership change alters roles in the member list AND the detail, and
+                // can change which workspaces the caller "owns" in the list view.
+                this._invalidateWorkspaceMembership(workspaceId);
+                this.clearCacheEntry('GET:/workspaces');
+                return result;
             }
             catch (error) {
                 throw this.handleError(error);
             }
         }
+        /**
+         * Bust the cached member list and detail for a workspace after a membership
+         * mutation. The member list (`getWorkspaceMembers`) and the detail
+         * (`getWorkspace`, which can embed member counts) both go stale when the
+         * member set or a member's role changes.
+         *
+         * Internal helper (leading underscore); not part of the supported public
+         * surface. Public rather than `private` because mixins compose into an
+         * exported anonymous class, where TypeScript cannot represent a private
+         * member in the emitted declaration file (TS4094).
+         */
+        _invalidateWorkspaceMembership(workspaceId) {
+            this.clearCacheEntry(`GET:/workspaces/${encodeURIComponent(workspaceId)}/members`);
+            this.clearCacheEntry(`GET:/workspaces/${encodeURIComponent(workspaceId)}`);
+        }
     };
 }

package/dist/cjs/utils/cache.js

@@ -72,11 +72,18 @@ class TTLCache {
      * Set a value in cache
      * @param key Cache key
      * @param data Data to cache
-     * @param ttl Optional TTL override (uses default if not provided)
+     * @param ttl Optional TTL override (uses default if not provided). An
+     *   explicit `0` or negative value is honored as "already expired" — the
+     *   entry is stored with `expiresAt <= now`, so the next `get`/`has` treats
+     *   it as a miss — rather than silently falling back to the default TTL.
      */
     set(key, data, ttl) {
         const now = Date.now();
-        const expiresAt = now + (ttl || this.defaultTTL);
+        // Distinguish "no override provided" (undefined → use default) from an
+        // explicit `0`/negative (do not cache). `ttl || this.defaultTTL` collapsed
+        // both into the default, making `cacheTTL:0` impossible to honor.
+        const effectiveTTL = ttl === undefined ? this.defaultTTL : ttl;
+        const expiresAt = now + effectiveTTL;
         this.cache.set(key, { data, timestamp: now, expiresAt });
     }
     /**