cozy-sharing 37.0.0 → 37.0.2

package/CHANGELOG.md

@@ -3,6 +3,16 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [37.0.2](https://github.com/cozy/cozy-libs/compare/cozy-sharing@37.0.1...cozy-sharing@37.0.2) (2026-07-21)
+
+**Note:** Version bump only for package cozy-sharing
+
+## [37.0.1](https://github.com/cozy/cozy-libs/compare/cozy-sharing@37.0.0...cozy-sharing@37.0.1) (2026-07-17)
+
+### Bug Fixes
+
+- **cozy-sharing:** Restore federated recipient management ([7442f26](https://github.com/cozy/cozy-libs/commit/7442f26f6cdb757d62d656f19d237d7694335c0c))
+
 # [37.0.0](https://github.com/cozy/cozy-libs/compare/cozy-sharing@36.5.0...cozy-sharing@37.0.0) (2026-07-16)
 
 ### Features

package/dist/components/FederatedFolder/FederatedFolderModal.js

@@ -54,6 +54,7 @@ var FederatedFolderModalContent = function FederatedFolderModalContent(_ref) {
       t = _useI18n.t;
 
   var _useSharingContext = useSharingContext(),
+      canReshare = _useSharingContext.canReshare,
       share = _useSharingContext.share,
       getSharingById = _useSharingContext.getSharingById,
       getSharingLink = _useSharingContext.getSharingLink,
@@ -278,6 +279,7 @@ var FederatedFolderModalContent = function FederatedFolderModalContent(_ref) {
   var documentPermissions = documentId ? getDocumentPermissions(documentId) : [];
   var displayedLink = existingDocument !== null && existingDocument !== void 0 && existingDocument.driveId ? getFederatedShareLink(existingDocument) : getSharingLink(existingDocument === null || existingDocument === void 0 ? void 0 : existingDocument._id);
   var isCurrentUserOwner = documentId ? isOwner(documentId) : false;
+  var canManageSharing = isCurrentUserOwner || (documentId ? canReshare(documentId) : false);
 
   var handleRevokeSelf = /*#__PURE__*/function () {
     var _ref4 = _asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee3(document) {
@@ -389,6 +391,7 @@ var FederatedFolderModalContent = function FederatedFolderModalContent(_ref) {
       enableCreateContact: true
     }))), /*#__PURE__*/React.createElement(WhoHasAccess, {
       isOwner: isCurrentUserOwner,
+      canManageSharing: canManageSharing,
       isSharedDrive: true,
       recipients: isSending ? frozenRecipients : existingRecipients,
       document: isSending ? frozenDoc : existingDocument,

package/dist/components/FederatedFolder/FederatedFolderModal.spec.js

@@ -19,6 +19,7 @@ var mockShareByLink = jest.fn();
 var mockRevoke = jest.fn();
 var mockRevokeSelf = jest.fn();
 var mockIsOwner = jest.fn();
+var mockCanReshare = jest.fn();
 var mockGetSharingLink = jest.fn();
 var mockGetFederatedShareLink = jest.fn();
 var mockGetRecipients = jest.fn().mockReturnValue([]);
@@ -38,6 +39,7 @@ jest.mock('../../hooks/useSharingContext', function () {
         revoke: mockRevoke,
         revokeSelf: mockRevokeSelf,
         isOwner: mockIsOwner,
+        canReshare: mockCanReshare,
         getSharingLink: mockGetSharingLink,
         getFederatedShareLink: mockGetFederatedShareLink,
         getDocumentPermissions: mockGetDocumentPermissions,
@@ -133,6 +135,7 @@ describe('FederatedFolderModal', function () {
     mockRevoke.mockResolvedValue();
     mockRevokeSelf.mockResolvedValue();
     mockIsOwner.mockReturnValue(false);
+    mockCanReshare.mockReturnValue(false);
     mockGetDocumentPermissions.mockReturnValue([]);
     mockFetchSharedDriveSharingLinks.mockResolvedValue([]);
     mockGetSharingLink.mockReturnValue('https://example.com/share/abc123');
@@ -726,36 +729,73 @@ describe('FederatedFolderModal', function () {
         }
       }, _callee19);
     })));
-  });
-  describe('onClose callback', function () {
-    it('should call onClose when close button is clicked', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee20() {
-      var _setup17, findByRole, closeButton;
+    it('should allow a write recipient to revoke another recipient', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee20() {
+      var otherRecipient, _setup17, findByLabelText;
 
       return _regeneratorRuntime.wrap(function _callee20$(_context20) {
         while (1) switch (_context20.prev = _context20.next) {
           case 0:
-            _setup17 = setup(), findByRole = _setup17.findByRole;
-            _context20.next = 3;
+            otherRecipient = {
+              instance: 'https://other.mycozy.cloud',
+              memberIndex: 2,
+              sharingId: 'sharing-id',
+              type: 'two-way'
+            };
+            mockCanReshare.mockReturnValue(true);
+            mockGetRecipients.mockReturnValue([otherRecipient]);
+            _setup17 = setup(), findByLabelText = _setup17.findByLabelText;
+            _context20.t0 = fireEvent;
+            _context20.next = 7;
+            return findByLabelText('Remove from sharing');
+
+          case 7:
+            _context20.t1 = _context20.sent;
+
+            _context20.t0.click.call(_context20.t0, _context20.t1);
+
+            _context20.next = 11;
+            return waitFor(function () {
+              expect(mockCanReshare).toHaveBeenCalledWith(mockDocument._id);
+              expect(mockRevoke).toHaveBeenCalledWith(mockDocument, otherRecipient.sharingId, otherRecipient.memberIndex);
+              expect(mockRevokeSelf).not.toHaveBeenCalled();
+            });
+
+          case 11:
+          case "end":
+            return _context20.stop();
+        }
+      }, _callee20);
+    })));
+  });
+  describe('onClose callback', function () {
+    it('should call onClose when close button is clicked', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee21() {
+      var _setup18, findByRole, closeButton;
+
+      return _regeneratorRuntime.wrap(function _callee21$(_context21) {
+        while (1) switch (_context21.prev = _context21.next) {
+          case 0:
+            _setup18 = setup(), findByRole = _setup18.findByRole;
+            _context21.next = 3;
             return findByRole('button', {
               name: /close/i
             });
 
           case 3:
-            closeButton = _context20.sent;
+            closeButton = _context21.sent;
             fireEvent.click(closeButton);
             expect(mockOnClose).toHaveBeenCalled();
 
           case 6:
           case "end":
-            return _context20.stop();
+            return _context21.stop();
         }
-      }, _callee20);
+      }, _callee21);
     })));
-    it('should ask confirmation when close button is clicked with pending recipients', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee21() {
-      var _setup18, getByRole, findByRole, getByText, closeButton;
+    it('should ask confirmation when close button is clicked with pending recipients', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee22() {
+      var _setup19, getByRole, findByRole, getByText, closeButton;
 
-      return _regeneratorRuntime.wrap(function _callee21$(_context21) {
-        while (1) switch (_context21.prev = _context21.next) {
+      return _regeneratorRuntime.wrap(function _callee22$(_context22) {
+        while (1) switch (_context22.prev = _context22.next) {
           case 0:
             usePendingRecipients.mockReturnValue({
               pendingRecipients: [{
@@ -766,14 +806,14 @@ describe('FederatedFolderModal', function () {
               selectedOption: 'readWrite',
               setSelectedOption: jest.fn()
             });
-            _setup18 = setup(), getByRole = _setup18.getByRole, findByRole = _setup18.findByRole, getByText = _setup18.getByText;
-            _context21.next = 4;
+            _setup19 = setup(), getByRole = _setup19.getByRole, findByRole = _setup19.findByRole, getByText = _setup19.getByText;
+            _context22.next = 4;
             return findByRole('button', {
               name: /close/i
             });
 
           case 4:
-            closeButton = _context21.sent;
+            closeButton = _context22.sent;
             fireEvent.click(closeButton);
             expect(getByText("Discard the changes you haven't saved?")).toBeTruthy();
             expect(mockOnClose).not.toHaveBeenCalled();
@@ -784,15 +824,15 @@ describe('FederatedFolderModal', function () {
 
           case 10:
           case "end":
-            return _context21.stop();
+            return _context22.stop();
         }
-      }, _callee21);
+      }, _callee22);
     })));
-    it('should close after discard confirmation', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee22() {
-      var _setup19, getByRole, findByRole, getByText, closeButton;
+    it('should close after discard confirmation', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee23() {
+      var _setup20, getByRole, findByRole, getByText, closeButton;
 
-      return _regeneratorRuntime.wrap(function _callee22$(_context22) {
-        while (1) switch (_context22.prev = _context22.next) {
+      return _regeneratorRuntime.wrap(function _callee23$(_context23) {
+        while (1) switch (_context23.prev = _context23.next) {
           case 0:
             usePendingRecipients.mockReturnValue({
               pendingRecipients: [{
@@ -803,14 +843,14 @@ describe('FederatedFolderModal', function () {
               selectedOption: 'readWrite',
               setSelectedOption: jest.fn()
             });
-            _setup19 = setup(), getByRole = _setup19.getByRole, findByRole = _setup19.findByRole, getByText = _setup19.getByText;
-            _context22.next = 4;
+            _setup20 = setup(), getByRole = _setup20.getByRole, findByRole = _setup20.findByRole, getByText = _setup20.getByText;
+            _context23.next = 4;
             return findByRole('button', {
               name: /close/i
             });
 
           case 4:
-            closeButton = _context22.sent;
+            closeButton = _context23.sent;
             fireEvent.click(closeButton);
             expect(getByText("Discard the changes you haven't saved?")).toBeTruthy();
             fireEvent.click(getByRole('button', {
@@ -820,9 +860,9 @@ describe('FederatedFolderModal', function () {
 
           case 9:
           case "end":
-            return _context22.stop();
+            return _context23.stop();
         }
-      }, _callee22);
+      }, _callee23);
     })));
   });
 });

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cozy-sharing",
-  "version": "37.0.0",
+  "version": "37.0.2",
   "description": "Provides sharing login for React applications.",
   "main": "dist/index.js",
   "author": "Cozy",
@@ -59,7 +59,7 @@
     "cozy-intent": "^2.31.1",
     "cozy-minilog": "^3.10.1",
     "cozy-ui": "^140.5.0",
-    "cozy-ui-plus": "^12.0.2",
+    "cozy-ui-plus": "^12.1.0",
     "jest": "30.3.0",
     "jest-environment-jsdom": "30.3.0",
     "react": "16.12.0",
@@ -85,5 +85,5 @@
   "sideEffects": [
     "*.css"
   ],
-  "gitHead": "58606cf6a38bd7dbfcf6708135c2a9df4a31f284"
+  "gitHead": "22faa5b53511d7720c575564ec52f446743209e6"
 }

package/src/components/FederatedFolder/FederatedFolderModal.jsx

@@ -35,6 +35,7 @@ const FederatedFolderModalContent = ({
   const client = useClient()
   const { t } = useI18n()
   const {
+    canReshare,
     share,
     getSharingById,
     getSharingLink,
@@ -224,6 +225,8 @@ const FederatedFolderModalContent = ({
     ? getFederatedShareLink(existingDocument)
     : getSharingLink(existingDocument?._id)
   const isCurrentUserOwner = documentId ? isOwner(documentId) : false
+  const canManageSharing =
+    isCurrentUserOwner || (documentId ? canReshare(documentId) : false)
 
   const handleRevokeSelf = async document => {
     await revokeSelf(document)
@@ -323,6 +326,7 @@ const FederatedFolderModalContent = ({
           </div>
           <WhoHasAccess
             isOwner={isCurrentUserOwner}
+            canManageSharing={canManageSharing}
             isSharedDrive
             recipients={isSending ? frozenRecipients : existingRecipients}
             document={isSending ? frozenDoc : existingDocument}

package/src/components/FederatedFolder/FederatedFolderModal.spec.jsx

@@ -13,6 +13,7 @@ const mockShareByLink = jest.fn()
 const mockRevoke = jest.fn()
 const mockRevokeSelf = jest.fn()
 const mockIsOwner = jest.fn()
+const mockCanReshare = jest.fn()
 const mockGetSharingLink = jest.fn()
 const mockGetFederatedShareLink = jest.fn()
 const mockGetRecipients = jest.fn().mockReturnValue([])
@@ -31,6 +32,7 @@ jest.mock('../../hooks/useSharingContext', () => ({
     revoke: mockRevoke,
     revokeSelf: mockRevokeSelf,
     isOwner: mockIsOwner,
+    canReshare: mockCanReshare,
     getSharingLink: mockGetSharingLink,
     getFederatedShareLink: mockGetFederatedShareLink,
     getDocumentPermissions: mockGetDocumentPermissions,
@@ -109,6 +111,7 @@ describe('FederatedFolderModal', () => {
     mockRevoke.mockResolvedValue()
     mockRevokeSelf.mockResolvedValue()
     mockIsOwner.mockReturnValue(false)
+    mockCanReshare.mockReturnValue(false)
     mockGetDocumentPermissions.mockReturnValue([])
     mockFetchSharedDriveSharingLinks.mockResolvedValue([])
     mockGetSharingLink.mockReturnValue('https://example.com/share/abc123')
@@ -472,6 +475,31 @@ describe('FederatedFolderModal', () => {
         expect(onRevokeSuccess).toHaveBeenCalledWith(mockDocument)
       })
     })
+
+    it('should allow a write recipient to revoke another recipient', async () => {
+      const otherRecipient = {
+        instance: 'https://other.mycozy.cloud',
+        memberIndex: 2,
+        sharingId: 'sharing-id',
+        type: 'two-way'
+      }
+      mockCanReshare.mockReturnValue(true)
+      mockGetRecipients.mockReturnValue([otherRecipient])
+
+      const { findByLabelText } = setup()
+
+      fireEvent.click(await findByLabelText('Remove from sharing'))
+
+      await waitFor(() => {
+        expect(mockCanReshare).toHaveBeenCalledWith(mockDocument._id)
+        expect(mockRevoke).toHaveBeenCalledWith(
+          mockDocument,
+          otherRecipient.sharingId,
+          otherRecipient.memberIndex
+        )
+        expect(mockRevokeSelf).not.toHaveBeenCalled()
+      })
+    })
   })
 
   describe('onClose callback', () => {