cozy-sharing 38.1.5 → 38.1.7

package/CHANGELOG.md

@@ -3,6 +3,18 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [38.1.7](https://github.com/cozy/cozy-libs/compare/cozy-sharing@38.1.6...cozy-sharing@38.1.7) (2026-10-02)
+
+### Bug Fixes
+
+- **cozy-sharing:** Clear effective recipients when the sharing is gone ([24897a7](https://github.com/cozy/cozy-libs/commit/24897a7ed7381a2f9f6bd2c229622689756a23ad)), closes [linagora/twake-drive#4264](https://github.com/linagora/twake-drive/issues/4264)
+
+## [38.1.6](https://github.com/cozy/cozy-libs/compare/cozy-sharing@38.1.5...cozy-sharing@38.1.6) (2026-10-01)
+
+### Bug Fixes
+
+- **cozy-sharing:** Keep an unshared folder shareable by its owner ([d0bdac7](https://github.com/cozy/cozy-libs/commit/d0bdac725d3b854be789def6c67ae31ca8db59bd)), closes [linagora/twake-drive#4264](https://github.com/linagora/twake-drive/issues/4264)
+
 ## [38.1.5](https://github.com/cozy/cozy-libs/compare/cozy-sharing@38.1.4...cozy-sharing@38.1.5) (2026-09-30)
 
 **Note:** Version bump only for package cozy-sharing

package/dist/SharingProvider.js

@@ -618,21 +618,39 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
         return _regeneratorRuntime.wrap(function _callee11$(_context11) {
           while (1) switch (_context11.prev = _context11.next) {
             case 0:
-              _context11.next = 2;
+              _context11.prev = 0;
+              _context11.next = 3;
               return _this2.sharingCol.fetchEffectiveRecipients(fileId, {
                 driveId: driveId
               });
 
-            case 2:
+            case 3:
               resp = _context11.sent;
 
               _this2.dispatch(receiveEffectiveRecipients(fileId, mapEffectiveRecipients((resp === null || resp === void 0 ? void 0 : resp.data) || [])));
 
-            case 4:
+              _context11.next = 12;
+              break;
+
+            case 7:
+              _context11.prev = 7;
+              _context11.t0 = _context11["catch"](0);
+
+              if (!((_context11.t0 === null || _context11.t0 === void 0 ? void 0 : _context11.t0.status) !== 404)) {
+                _context11.next = 11;
+                break;
+              }
+
+              throw _context11.t0;
+
+            case 11:
+              _this2.dispatch(receiveEffectiveRecipients(fileId, []));
+
+            case 12:
             case "end":
               return _context11.stop();
           }
-        }, _callee11);
+        }, _callee11, null, [[0, 7]]);
       }));
 
       return function (_x19, _x20) {

package/dist/SharingProvider.spec.js

@@ -12,7 +12,7 @@ import React from 'react';
 import { createMockClient } from 'cozy-client';
 import { SharingProvider } from './SharingProvider';
 import SharingContext from './context';
-import reducer, { addSharingLink, getDocumentPermissions, getDocumentSharing, receiveSharings } from './state';
+import reducer, { addSharingLink, getDocumentPermissions, getDocumentSharing, getEffectiveRecipients, receiveEffectiveRecipients, receiveSharings } from './state';
 import AppLike from '../test/AppLike';
 
 var AppWrapper = function AppWrapper(_ref) {
@@ -831,6 +831,45 @@ describe('shared drive recipient revocation', function () {
       }
     }, _callee15);
   })));
+  it('clears effective recipients when the sharing was deleted (404)', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee16() {
+    return _regeneratorRuntime.wrap(function _callee16$(_context16) {
+      while (1) switch (_context16.prev = _context16.next) {
+        case 0:
+          provider.dispatch(receiveEffectiveRecipients(document.id, [{
+            sharingId: SHARED_DRIVE.id,
+            memberIndex: 1
+          }]));
+          sharingCol.fetchEffectiveRecipients.mockRejectedValue(Object.assign(new Error('deleted'), {
+            status: 404
+          }));
+          _context16.next = 4;
+          return expect(provider.fetchEffectiveRecipients(document.id, SHARED_DRIVE.id)).resolves.toBeUndefined();
+
+        case 4:
+          expect(getEffectiveRecipients(provider.state, document.id)).toEqual([]);
+
+        case 5:
+        case "end":
+          return _context16.stop();
+      }
+    }, _callee16);
+  })));
+  it('rethrows non-404 errors from fetching effective recipients', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee17() {
+    return _regeneratorRuntime.wrap(function _callee17$(_context17) {
+      while (1) switch (_context17.prev = _context17.next) {
+        case 0:
+          sharingCol.fetchEffectiveRecipients.mockRejectedValue(Object.assign(new Error('boom'), {
+            status: 500
+          }));
+          _context17.next = 3;
+          return expect(provider.fetchEffectiveRecipients(document.id, SHARED_DRIVE.id)).rejects.toThrow('boom');
+
+        case 3:
+        case "end":
+          return _context17.stop();
+      }
+    }, _callee17);
+  })));
 });
 describe('fetchSharedDriveSharingLinks', function () {
   var PERM_DRIVE_FILE = {
@@ -855,10 +894,10 @@ describe('fetchSharedDriveSharingLinks', function () {
     id: 'file_in_drive',
     driveId: 'drive_123'
   };
-  it('fetches shared drive sharing links by file id', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee16() {
+  it('fetches shared drive sharing links by file id', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee18() {
     var mockFindLinksByIds, mockClient, provider, result, permissions;
-    return _regeneratorRuntime.wrap(function _callee16$(_context16) {
-      while (1) switch (_context16.prev = _context16.next) {
+    return _regeneratorRuntime.wrap(function _callee18$(_context18) {
+      while (1) switch (_context18.prev = _context18.next) {
         case 0:
           mockFindLinksByIds = jest.fn().mockResolvedValue({
             data: [PERM_DRIVE_FILE]
@@ -868,11 +907,11 @@ describe('fetchSharedDriveSharingLinks', function () {
             findLinksByIds: mockFindLinksByIds
           });
           provider = setupProvider(mockClient);
-          _context16.next = 6;
+          _context18.next = 6;
           return provider.fetchSharedDriveSharingLinks(driveFile);
 
         case 6:
-          result = _context16.sent;
+          result = _context18.sent;
           expect(mockClient.collection).toHaveBeenCalledWith('io.cozy.permissions', {
             driveId: 'drive_123'
           });
@@ -885,14 +924,14 @@ describe('fetchSharedDriveSharingLinks', function () {
 
         case 14:
         case "end":
-          return _context16.stop();
+          return _context18.stop();
       }
-    }, _callee16);
+    }, _callee18);
   })));
-  it('returns an empty array and skips the API call for non-shared-drive documents', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee17() {
+  it('returns an empty array and skips the API call for non-shared-drive documents', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee19() {
     var mockClient, findLinksByIds, provider, result;
-    return _regeneratorRuntime.wrap(function _callee17$(_context17) {
-      while (1) switch (_context17.prev = _context17.next) {
+    return _regeneratorRuntime.wrap(function _callee19$(_context19) {
+      while (1) switch (_context19.prev = _context19.next) {
         case 0:
           mockClient = createMockClient({});
           findLinksByIds = jest.fn();
@@ -901,27 +940,27 @@ describe('fetchSharedDriveSharingLinks', function () {
           });
           provider = setupProvider(mockClient);
           provider.dispatch = jest.fn();
-          _context17.next = 7;
+          _context19.next = 7;
           return provider.fetchSharedDriveSharingLinks({
             _id: 'regular_file'
           });
 
         case 7:
-          result = _context17.sent;
+          result = _context19.sent;
           expect(result).toEqual([]);
           expect(findLinksByIds).not.toHaveBeenCalled();
           expect(provider.dispatch).not.toHaveBeenCalled();
 
         case 11:
         case "end":
-          return _context17.stop();
+          return _context19.stop();
       }
-    }, _callee17);
+    }, _callee19);
   })));
-  it('uses document.id as a fallback when _id is missing', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee18() {
+  it('uses document.id as a fallback when _id is missing', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee20() {
     var mockFindLinksByIds, mockClient, provider;
-    return _regeneratorRuntime.wrap(function _callee18$(_context18) {
-      while (1) switch (_context18.prev = _context18.next) {
+    return _regeneratorRuntime.wrap(function _callee20$(_context20) {
+      while (1) switch (_context20.prev = _context20.next) {
         case 0:
           mockFindLinksByIds = jest.fn().mockResolvedValue({
             data: [PERM_DRIVE_FILE]
@@ -931,7 +970,7 @@ describe('fetchSharedDriveSharingLinks', function () {
             findLinksByIds: mockFindLinksByIds
           });
           provider = setupProvider(mockClient);
-          _context18.next = 6;
+          _context20.next = 6;
           return provider.fetchSharedDriveSharingLinks({
             id: 'file_in_drive',
             driveId: 'drive_123'
@@ -942,14 +981,14 @@ describe('fetchSharedDriveSharingLinks', function () {
 
         case 7:
         case "end":
-          return _context18.stop();
+          return _context20.stop();
       }
-    }, _callee18);
+    }, _callee20);
   })));
-  it('does not dispatch when the response has no data', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee19() {
+  it('does not dispatch when the response has no data', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee21() {
     var mockFindLinksByIds, mockClient, provider, result;
-    return _regeneratorRuntime.wrap(function _callee19$(_context19) {
-      while (1) switch (_context19.prev = _context19.next) {
+    return _regeneratorRuntime.wrap(function _callee21$(_context21) {
+      while (1) switch (_context21.prev = _context21.next) {
         case 0:
           mockFindLinksByIds = jest.fn().mockResolvedValue({
             data: []
@@ -960,19 +999,19 @@ describe('fetchSharedDriveSharingLinks', function () {
           });
           provider = setupProvider(mockClient);
           provider.dispatch = jest.fn();
-          _context19.next = 7;
+          _context21.next = 7;
           return provider.fetchSharedDriveSharingLinks(driveFile);
 
         case 7:
-          result = _context19.sent;
+          result = _context21.sent;
           expect(result).toEqual([]);
           expect(provider.dispatch).not.toHaveBeenCalled();
 
         case 10:
         case "end":
-          return _context19.stop();
+          return _context21.stop();
       }
-    }, _callee19);
+    }, _callee21);
   })));
 });
 describe('shareByLink shared drive 409 recovery', function () {
@@ -998,10 +1037,10 @@ describe('shareByLink shared drive 409 recovery', function () {
     id: 'file_in_drive',
     driveId: 'drive_123'
   };
-  it('falls back to fetching existing links when create returns 409', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee20() {
+  it('falls back to fetching existing links when create returns 409', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee22() {
     var conflict, mockCreateSharingLink, mockFindLinksByIds, mockClient, provider, resp;
-    return _regeneratorRuntime.wrap(function _callee20$(_context20) {
-      while (1) switch (_context20.prev = _context20.next) {
+    return _regeneratorRuntime.wrap(function _callee22$(_context22) {
+      while (1) switch (_context22.prev = _context22.next) {
         case 0:
           conflict = Object.assign(new Error('Conflict'), {
             status: 409
@@ -1016,27 +1055,27 @@ describe('shareByLink shared drive 409 recovery', function () {
             findLinksByIds: mockFindLinksByIds
           });
           provider = setupProvider(mockClient);
-          _context20.next = 8;
+          _context22.next = 8;
           return provider.shareByLink(driveFile, {
             verbs: ['GET']
           });
 
         case 8:
-          resp = _context20.sent;
+          resp = _context22.sent;
           expect(mockCreateSharingLink).toHaveBeenCalled();
           expect(mockFindLinksByIds).toHaveBeenCalledWith(['file_in_drive']);
           expect(resp.data.id).toBe(PERM_DRIVE_FILE.id);
 
         case 12:
         case "end":
-          return _context20.stop();
+          return _context22.stop();
       }
-    }, _callee20);
+    }, _callee22);
   })));
-  it('does not duplicate existing links when 409 recovery refetches them', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee21() {
+  it('does not duplicate existing links when 409 recovery refetches them', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee23() {
     var conflict, mockCreateSharingLink, mockFindLinksByIds, mockClient, provider, resp, permissions;
-    return _regeneratorRuntime.wrap(function _callee21$(_context21) {
-      while (1) switch (_context21.prev = _context21.next) {
+    return _regeneratorRuntime.wrap(function _callee23$(_context23) {
+      while (1) switch (_context23.prev = _context23.next) {
         case 0:
           conflict = Object.assign(new Error('Conflict'), {
             status: 409
@@ -1051,13 +1090,13 @@ describe('shareByLink shared drive 409 recovery', function () {
             findLinksByIds: mockFindLinksByIds
           });
           provider = setupProvider(mockClient, {}, reducer(undefined, addSharingLink(PERM_DRIVE_FILE)));
-          _context21.next = 8;
+          _context23.next = 8;
           return provider.shareByLink(driveFile, {
             verbs: ['GET']
           });
 
         case 8:
-          resp = _context21.sent;
+          resp = _context23.sent;
           permissions = getDocumentPermissions(provider.state, 'file_in_drive');
           expect(resp.data.id).toBe(PERM_DRIVE_FILE.id);
           expect(provider.dispatch).not.toHaveBeenCalled();
@@ -1065,14 +1104,14 @@ describe('shareByLink shared drive 409 recovery', function () {
 
         case 13:
         case "end":
-          return _context21.stop();
+          return _context23.stop();
       }
-    }, _callee21);
+    }, _callee23);
   })));
-  it('rethrows the original error when 409 happens and no link is found', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee22() {
+  it('rethrows the original error when 409 happens and no link is found', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee24() {
     var conflict, mockCreateSharingLink, mockFindLinksByIds, mockClient, provider;
-    return _regeneratorRuntime.wrap(function _callee22$(_context22) {
-      while (1) switch (_context22.prev = _context22.next) {
+    return _regeneratorRuntime.wrap(function _callee24$(_context24) {
+      while (1) switch (_context24.prev = _context24.next) {
         case 0:
           conflict = Object.assign(new Error('Conflict'), {
             status: 409
@@ -1088,16 +1127,16 @@ describe('shareByLink shared drive 409 recovery', function () {
           });
           provider = setupProvider(mockClient);
           provider.dispatch = jest.fn();
-          _context22.next = 9;
+          _context24.next = 9;
           return expect(provider.shareByLink(driveFile, {
             verbs: ['GET']
           })).rejects.toBe(conflict);
 
         case 9:
         case "end":
-          return _context22.stop();
+          return _context24.stop();
       }
-    }, _callee22);
+    }, _callee24);
   })));
 });
 describe('updateSharingMemberType', function () {
@@ -1140,37 +1179,37 @@ describe('updateSharingMemberType', function () {
       })
     };
   });
-  it('should throw when sharing is not found', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee23() {
-    return _regeneratorRuntime.wrap(function _callee23$(_context23) {
-      while (1) switch (_context23.prev = _context23.next) {
+  it('should throw when sharing is not found', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee25() {
+    return _regeneratorRuntime.wrap(function _callee25$(_context25) {
+      while (1) switch (_context25.prev = _context25.next) {
         case 0:
-          _context23.next = 2;
+          _context25.next = 2;
           return expect(instance.updateSharingMemberType('unknown-id', 1, 'one-way')).rejects.toThrow('Sharing not found');
 
         case 2:
         case "end":
-          return _context23.stop();
+          return _context25.stop();
       }
-    }, _callee23);
+    }, _callee25);
   })));
-  it('should throw when member is not found', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee24() {
-    return _regeneratorRuntime.wrap(function _callee24$(_context24) {
-      while (1) switch (_context24.prev = _context24.next) {
+  it('should throw when member is not found', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee26() {
+    return _regeneratorRuntime.wrap(function _callee26$(_context26) {
+      while (1) switch (_context26.prev = _context26.next) {
         case 0:
-          _context24.next = 2;
+          _context26.next = 2;
           return expect(instance.updateSharingMemberType('sharing-123', 99, 'one-way')).rejects.toThrow('Member not found');
 
         case 2:
         case "end":
-          return _context24.stop();
+          return _context26.stop();
       }
-    }, _callee24);
+    }, _callee26);
   })));
-  it('should call setReadOnly when switching to one-way', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee25() {
-    return _regeneratorRuntime.wrap(function _callee25$(_context25) {
-      while (1) switch (_context25.prev = _context25.next) {
+  it('should call setReadOnly when switching to one-way', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee27() {
+    return _regeneratorRuntime.wrap(function _callee27$(_context27) {
+      while (1) switch (_context27.prev = _context27.next) {
         case 0:
-          _context25.next = 2;
+          _context27.next = 2;
           return instance.updateSharingMemberType('sharing-123', 1, 'one-way');
 
         case 2:
@@ -1179,15 +1218,15 @@ describe('updateSharingMemberType', function () {
 
         case 4:
         case "end":
-          return _context25.stop();
+          return _context27.stop();
       }
-    }, _callee25);
+    }, _callee27);
   })));
-  it('should call setReadWrite when switching to two-way', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee26() {
-    return _regeneratorRuntime.wrap(function _callee26$(_context26) {
-      while (1) switch (_context26.prev = _context26.next) {
+  it('should call setReadWrite when switching to two-way', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee28() {
+    return _regeneratorRuntime.wrap(function _callee28$(_context28) {
+      while (1) switch (_context28.prev = _context28.next) {
         case 0:
-          _context26.next = 2;
+          _context28.next = 2;
           return instance.updateSharingMemberType('sharing-123', 1, 'two-way');
 
         case 2:
@@ -1196,15 +1235,15 @@ describe('updateSharingMemberType', function () {
 
         case 4:
         case "end":
-          return _context26.stop();
+          return _context28.stop();
       }
-    }, _callee26);
+    }, _callee28);
   })));
-  it('should optimistically dispatch a state update when switching to one-way', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee27() {
-    return _regeneratorRuntime.wrap(function _callee27$(_context27) {
-      while (1) switch (_context27.prev = _context27.next) {
+  it('should optimistically dispatch a state update when switching to one-way', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee29() {
+    return _regeneratorRuntime.wrap(function _callee29$(_context29) {
+      while (1) switch (_context29.prev = _context29.next) {
         case 0:
-          _context27.next = 2;
+          _context29.next = 2;
           return instance.updateSharingMemberType('sharing-123', 1, 'one-way');
 
         case 2:
@@ -1221,15 +1260,15 @@ describe('updateSharingMemberType', function () {
 
         case 3:
         case "end":
-          return _context27.stop();
+          return _context29.stop();
       }
-    }, _callee27);
+    }, _callee29);
   })));
-  it('should optimistically dispatch a state update when switching to two-way', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee28() {
-    return _regeneratorRuntime.wrap(function _callee28$(_context28) {
-      while (1) switch (_context28.prev = _context28.next) {
+  it('should optimistically dispatch a state update when switching to two-way', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee30() {
+    return _regeneratorRuntime.wrap(function _callee30$(_context30) {
+      while (1) switch (_context30.prev = _context30.next) {
         case 0:
-          _context28.next = 2;
+          _context30.next = 2;
           return instance.updateSharingMemberType('sharing-123', 1, 'two-way');
 
         case 2:
@@ -1246,44 +1285,44 @@ describe('updateSharingMemberType', function () {
 
         case 3:
         case "end":
-          return _context28.stop();
+          return _context30.stop();
       }
-    }, _callee28);
+    }, _callee30);
   })));
-  it('should rethrow error if setReadOnly fails', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee29() {
-    return _regeneratorRuntime.wrap(function _callee29$(_context29) {
-      while (1) switch (_context29.prev = _context29.next) {
+  it('should rethrow error if setReadOnly fails', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee31() {
+    return _regeneratorRuntime.wrap(function _callee31$(_context31) {
+      while (1) switch (_context31.prev = _context31.next) {
         case 0:
           instance.sharingCol.setReadOnly.mockRejectedValue(new Error('Network error'));
-          _context29.next = 3;
+          _context31.next = 3;
           return expect(instance.updateSharingMemberType('sharing-123', 1, 'one-way')).rejects.toThrow('Network error');
 
         case 3:
         case "end":
-          return _context29.stop();
+          return _context31.stop();
       }
-    }, _callee29);
+    }, _callee31);
   })));
-  it('should rethrow error if setReadWrite fails', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee30() {
-    return _regeneratorRuntime.wrap(function _callee30$(_context30) {
-      while (1) switch (_context30.prev = _context30.next) {
+  it('should rethrow error if setReadWrite fails', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee32() {
+    return _regeneratorRuntime.wrap(function _callee32$(_context32) {
+      while (1) switch (_context32.prev = _context32.next) {
         case 0:
           instance.sharingCol.setReadWrite.mockRejectedValue(new Error('Network error'));
-          _context30.next = 3;
+          _context32.next = 3;
           return expect(instance.updateSharingMemberType('sharing-123', 1, 'two-way')).rejects.toThrow('Network error');
 
         case 3:
         case "end":
-          return _context30.stop();
+          return _context32.stop();
       }
-    }, _callee30);
+    }, _callee32);
   })));
-  it('should rollback optimistic state update if API call fails', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee31() {
-    return _regeneratorRuntime.wrap(function _callee31$(_context31) {
-      while (1) switch (_context31.prev = _context31.next) {
+  it('should rollback optimistic state update if API call fails', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee33() {
+    return _regeneratorRuntime.wrap(function _callee33$(_context33) {
+      while (1) switch (_context33.prev = _context33.next) {
         case 0:
           instance.sharingCol.setReadOnly.mockRejectedValue(new Error('Network error'));
-          _context31.next = 3;
+          _context33.next = 3;
           return expect(instance.updateSharingMemberType('sharing-123', 1, 'one-way')).rejects.toThrow('Network error');
 
         case 3:
@@ -1322,14 +1361,14 @@ describe('updateSharingMemberType', function () {
 
         case 7:
         case "end":
-          return _context31.stop();
+          return _context33.stop();
       }
-    }, _callee31);
+    }, _callee33);
   })));
-  it('should preserve realtime changes when rolling back an API failure', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee33() {
+  it('should preserve realtime changes when rolling back an API failure', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee35() {
     var realtimeSharing;
-    return _regeneratorRuntime.wrap(function _callee33$(_context33) {
-      while (1) switch (_context33.prev = _context33.next) {
+    return _regeneratorRuntime.wrap(function _callee35$(_context35) {
+      while (1) switch (_context35.prev = _context35.next) {
         case 0:
           realtimeSharing = _objectSpread(_objectSpread({}, mockSharing), {}, {
             attributes: _objectSpread(_objectSpread({}, mockSharing.attributes), {}, {
@@ -1341,9 +1380,9 @@ describe('updateSharingMemberType', function () {
               })]
             })
           });
-          instance.sharingCol.setReadOnly.mockImplementation( /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee32() {
-            return _regeneratorRuntime.wrap(function _callee32$(_context32) {
-              while (1) switch (_context32.prev = _context32.next) {
+          instance.sharingCol.setReadOnly.mockImplementation( /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee34() {
+            return _regeneratorRuntime.wrap(function _callee34$(_context34) {
+              while (1) switch (_context34.prev = _context34.next) {
                 case 0:
                   instance.state = _objectSpread(_objectSpread({}, instance.state), {}, {
                     sharings: instance.state.sharings.map(function (sharing) {
@@ -1354,11 +1393,11 @@ describe('updateSharingMemberType', function () {
 
                 case 2:
                 case "end":
-                  return _context32.stop();
+                  return _context34.stop();
               }
-            }, _callee32);
+            }, _callee34);
           })));
-          _context33.next = 4;
+          _context35.next = 4;
           return expect(instance.updateSharingMemberType('sharing-123', 1, 'one-way')).rejects.toThrow('Network error');
 
         case 4:
@@ -1387,9 +1426,9 @@ describe('updateSharingMemberType', function () {
 
         case 7:
         case "end":
-          return _context33.stop();
+          return _context35.stop();
       }
-    }, _callee33);
+    }, _callee35);
   })));
 }); // TODO Convert with react-testing-library
 // describe('hasWriteAccess', () => {

package/dist/components/FederatedFolder/FederatedFolderModal.js

@@ -123,8 +123,9 @@ var FederatedFolderModalContent = function FederatedFolderModalContent(_ref) {
   var displayedLink = existingDocument !== null && existingDocument !== void 0 && existingDocument.driveId ? getFederatedShareLink(existingDocument) : getSharingLink(existingDocument === null || existingDocument === void 0 ? void 0 : existingDocument._id);
   var isSharedDriveRoot = Boolean((existingDocument === null || existingDocument === void 0 ? void 0 : existingDocument.driveId) && sharedDriveRootIds.some(function (id) {
     return id === (existingDocument === null || existingDocument === void 0 ? void 0 : existingDocument._id) || id === (existingDocument === null || existingDocument === void 0 ? void 0 : existingDocument.id);
-  }));
-  var isInsideSharedDrive = Boolean((existingDocument === null || existingDocument === void 0 ? void 0 : existingDocument.driveId) && !isSharedDriveRoot);
+  })); // Ignore a stale driveId left after an owner revokes the last recipient, so their own folder isn't wrongly restricted to link-only (linagora/twake-drive#4264).
+
+  var isInsideSharedDrive = Boolean((existingDocument === null || existingDocument === void 0 ? void 0 : existingDocument.driveId) && !isSharedDriveRoot && (sharedDriveSharing || !isOwner(documentId)));
   var isCurrentUserOwner = existingDocument !== null && existingDocument !== void 0 && existingDocument.driveId ? Boolean(sharedDriveSharing === null || sharedDriveSharing === void 0 || (_sharedDriveSharing$a2 = sharedDriveSharing.attributes) === null || _sharedDriveSharing$a2 === void 0 ? void 0 : _sharedDriveSharing$a2.owner) : documentId ? isOwner(documentId) : false;
   var isMemberReadOnly = isInsideSharedDrive && !isCurrentUserOwner && !hasWriteAccess(documentId, existingDocument === null || existingDocument === void 0 ? void 0 : existingDocument.driveId);
   var canManageSharing = isCurrentUserOwner || (documentId ? canReshare(documentId) : false);