fix(android): correct long term access directory picker response

[OldStager01]

PR: Rename 'status' key to 'bookmarkStatus' in Android directory picker response

Description

This PR standardizes the response format for directory picking on Android by renaming the status key to bookmarkStatus when requestLongTermAccess is set to true. This change ensures consistency across platforms and aligns with the expected type definitions.

Changes Made

• Modified processDirectoryPickerResult() method in RNDocumentPickerModule.kt to use bookmarkStatus instead of status in the response object when handling long-term directory access permissions
• Updated releaseLongTermAccess() method to also use bookmarkStatus instead of status in the response for consistency

Technical Details

The changes were made in two specific locations in packages/document-picker/android/src/main/java/com/reactnativedocumentpicker/RNDocumentPickerModule.kt:

1. In the processDirectoryPickerResult() method (~line 392-399):

   - map.putString("status", "success")
   + map.putString("bookmarkStatus", "success")

   - map.putString("status", "error")
   + map.putString("bookmarkStatus", "error")

2. In the releaseLongTermAccess() method (~line 352-358):

   - result.putString("status", "success")
   + result.putString("bookmarkStatus", "success")

   - result.putString("status", "error")
   + result.putString("bookmarkStatus", "error")

Example Usage

When using pickDirectory() with long-term access:

const response = await pickDirectory({ requestLongTermAccess: true });

Previous Response Format (Incorrect)

{
  "bookmark": "Y29udGVudDovL2NvbS5hbmRyb2lkLmV4dGVybmFsc3RvcmFnZS5kb2N1bWVudHMvdHJlZS9wcmltYXJ5JTNBRG93bmxvYWQlMkZRdWljayUyMFNoYXJl",
  "status": "success",
  "uri": "content://com.android.externalstorage.documents/tree/primary%3ADownload%2FQuick%20Share"
}

Fixed Response Format (Correct)

{
  "bookmark": "Y29udGVudDovL2NvbS5hbmRyb2lkLmV4dGVybmFsc3RvcmFnZS5kb2N1bWVudHMvdHJlZS9wcmltYXJ5JTNBRG93bmxvYWQlMkZRdWljayUyMFNoYXJl",
  "bookmarkStatus": "success",
  "uri": "content://com.android.externalstorage.documents/tree/primary%3ADownload%2FQuick%20Share"
}

Testing

• Verified the response now includes bookmarkStatus instead of status when requestLongTermAccess is true
• Tested both success and error cases to ensure all relevant fields are updated correctly

Impact

This change ensures consistent naming conventions, improving maintainability and cross-platform compatibility by aligning the Android implementation with the expected type definitions used across the library.

[changeset-bot]

🦋 Changeset detected

Latest commit: 6b030f8

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
@react-native-documents/picker	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[vonovak]

Thank you for the PR!

[OldStager01]

You're welcome! Glad to contribute. Let me know if there's anything else I can help with.

Thank you for the PR!