cozy-sharing 39.0.1 → 39.0.2

package/CHANGELOG.md

@@ -3,6 +3,12 @@
 All notable changes to this project will be documented in this file.
 See [Conventional Commits](https://conventionalcommits.org) for commit guidelines.
 
+## [39.0.2](https://github.com/cozy/cozy-libs/compare/cozy-sharing@39.0.1...cozy-sharing@39.0.2) (2026-10-06)
+
+### Bug Fixes
+
+- **cozy-sharing:** Avoid falling back to driveId for nested sharing ([eac16ad](https://github.com/cozy/cozy-libs/commit/eac16ad94e2eddab74ea0d9b249cb91a05e98ac1))
+
 ## [39.0.1](https://github.com/cozy/cozy-libs/compare/cozy-sharing@39.0.0...cozy-sharing@39.0.1) (2026-10-06)
 
 **Note:** Version bump only for package cozy-sharing

package/dist/SharingProvider.js

@@ -136,9 +136,8 @@ export var SharingProvider = /*#__PURE__*/function (_Component) {
           while (1) switch (_context2.prev = _context2.next) {
             case 0:
               document = _ref2.document, recipients = _ref2.recipients, readOnlyRecipients = _ref2.readOnlyRecipients, description = _ref2.description, openSharing = _ref2.openSharing, sharedDrive = _ref2.sharedDrive;
-              _this2$props2 = _this2.props, client = _this2$props2.client, doctype = _this2$props2.doctype; // Documents nested in a shared drive have no sharing of their own
-
-              sharing = getDocumentSharing(_this2.state, document.id) || (document.driveId ? _getSharingById(_this2.state, document.driveId) : null);
+              _this2$props2 = _this2.props, client = _this2$props2.client, doctype = _this2$props2.doctype;
+              sharing = getDocumentSharing(_this2.state, document.id);
 
               if (!sharing) {
                 _context2.next = 12;

package/dist/SharingProvider.spec.js

@@ -793,21 +793,18 @@ describe('shared drive recipient revocation', function () {
       }
     }, _callee14);
   })));
-  it('adds recipients to the existing shared drive when sharing a document nested inside it', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee15() {
+  it('creates a new sharing when sharing a document nested inside a shared drive without its own sharing', /*#__PURE__*/_asyncToGenerator( /*#__PURE__*/_regeneratorRuntime.mark(function _callee15() {
     var nestedDocument;
     return _regeneratorRuntime.wrap(function _callee15$(_context15) {
       while (1) switch (_context15.prev = _context15.next) {
         case 0:
-          sharingCol.addRecipients.mockResolvedValue({
-            data: SHARED_DRIVE
-          });
           nestedDocument = {
             _id: 'nested_folder_456',
             id: 'nested_folder_456',
             driveId: SHARED_DRIVE.id,
             path: '/Shared folder/Sub folder'
           };
-          _context15.next = 4;
+          _context15.next = 3;
           return provider.share({
             document: nestedDocument,
             recipients: [recipient],
@@ -817,15 +814,19 @@ describe('shared drive recipient revocation', function () {
             sharedDrive: true
           });
 
-        case 4:
-          expect(sharingCol.create).not.toHaveBeenCalled();
-          expect(sharingCol.addRecipients).toHaveBeenCalledWith({
-            document: SHARED_DRIVE,
+        case 3:
+          expect(sharingCol.create).toHaveBeenCalledWith({
+            document: nestedDocument,
             recipients: [recipient],
-            readOnlyRecipients: []
+            readOnlyRecipients: [],
+            description: 'Sub folder',
+            previewPath: '/preview',
+            openSharing: false,
+            sharedDrive: true
           });
+          expect(sharingCol.addRecipients).not.toHaveBeenCalled();
 
-        case 6:
+        case 5:
         case "end":
           return _context15.stop();
       }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "cozy-sharing",
-  "version": "39.0.1",
+  "version": "39.0.2",
   "description": "Provides sharing login for React applications.",
   "main": "dist/index.js",
   "author": "Cozy",
@@ -58,7 +58,7 @@
     "cozy-intent": "^2.31.1",
     "cozy-minilog": "^3.10.1",
     "cozy-ui": "^140.5.0",
-    "cozy-ui-plus": "^14.0.0",
+    "cozy-ui-plus": "^14.0.1",
     "jest": "30.3.0",
     "jest-environment-jsdom": "30.3.0",
     "react": "16.12.0",
@@ -84,5 +84,5 @@
   "sideEffects": [
     "*.css"
   ],
-  "gitHead": "10d189a754a9ad65733ac9d1921834da3782e77b"
+  "gitHead": "a9f046c1c02c5ffc85bf5c46da1acb41407dc584"
 }

package/src/SharingProvider.jsx

@@ -260,10 +260,7 @@ export class SharingProvider extends Component {
     sharedDrive
   }) => {
     const { client, doctype } = this.props
-    // Documents nested in a shared drive have no sharing of their own
-    const sharing =
-      getDocumentSharing(this.state, document.id) ||
-      (document.driveId ? getSharingById(this.state, document.driveId) : null)
+    const sharing = getDocumentSharing(this.state, document.id)
     if (sharing) {
       const sharingResult = await this.addRecipients({
         document: sharing,

package/src/SharingProvider.spec.jsx

@@ -572,8 +572,7 @@ describe('shared drive recipient revocation', () => {
     })
   })
 
-  it('adds recipients to the existing shared drive when sharing a document nested inside it', async () => {
-    sharingCol.addRecipients.mockResolvedValue({ data: SHARED_DRIVE })
+  it('creates a new sharing when sharing a document nested inside a shared drive without its own sharing', async () => {
     const nestedDocument = {
       _id: 'nested_folder_456',
       id: 'nested_folder_456',
@@ -590,12 +589,16 @@ describe('shared drive recipient revocation', () => {
       sharedDrive: true
     })
 
-    expect(sharingCol.create).not.toHaveBeenCalled()
-    expect(sharingCol.addRecipients).toHaveBeenCalledWith({
-      document: SHARED_DRIVE,
+    expect(sharingCol.create).toHaveBeenCalledWith({
+      document: nestedDocument,
       recipients: [recipient],
-      readOnlyRecipients: []
+      readOnlyRecipients: [],
+      description: 'Sub folder',
+      previewPath: '/preview',
+      openSharing: false,
+      sharedDrive: true
     })
+    expect(sharingCol.addRecipients).not.toHaveBeenCalled()
   })
 
   it('clears effective recipients when the sharing was deleted (404)', async () => {