Skip to content

feat: close iOS/Android platform parity gaps (#368) - #372

Merged
plrthink merged 1 commit into
cursor/ios-zip-compatibility-b7edfrom
cursor/platform-parity-b7ed
Aug 15, 2026
Merged

feat: close iOS/Android platform parity gaps (#368)#372
plrthink merged 1 commit into
cursor/ios-zip-compatibility-b7edfrom
cursor/platform-parity-b7ed

Conversation

@plrthink

@plrthink plrthink commented Jul 25, 2026

Copy link
Copy Markdown
Collaborator

Summary

Implements #368 (P1): close iOS/Android platform parity gaps.

Rebased onto #371 after review-comment fixes. Tip version is 9.4.0.

Stack

Merge order: #369 (done) → #370#371this PR (tip)

Open in Web Open in Cursor 

@cursor

cursor Bot commented Aug 15, 2026

Copy link
Copy Markdown
Contributor

Bugbot is not enabled for your account, so this pull request was not reviewed.

Enable Bugbot in the Cursor dashboard to get automatic reviews on future PRs.

@devin-ai-integration devin-ai-integration Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Devin Review found 3 potential issues.

View 1 additional finding in Devin Review.

Open in Devin Review

Comment thread ios/RNZipArchive.mm
Comment thread ios/RNZipArchive.mm
Comment thread android/src/main/java/com/rnziparchive/RNZipArchiveModule.java Outdated
@cursor
cursor Bot force-pushed the cursor/platform-parity-b7ed branch 2 times, most recently from ba77abd to b289c31 Compare August 15, 2026 13:13
@cursor
cursor Bot changed the base branch from master to cursor/ios-zip-compatibility-b7ed August 15, 2026 13:13
Rebased onto #371 after #369 was squash-merged.

Co-authored-by: Perry <plrthink@gmail.com>
@cursor
cursor Bot force-pushed the cursor/ios-zip-compatibility-b7ed branch from b6fe8dd to 31fd8c0 Compare August 15, 2026 14:08
@cursor
cursor Bot force-pushed the cursor/platform-parity-b7ed branch from b289c31 to aec3135 Compare August 15, 2026 14:08
@plrthink
plrthink merged commit 8612546 into cursor/ios-zip-compatibility-b7ed Aug 15, 2026
plrthink added a commit that referenced this pull request Aug 16, 2026
* fix(ios): zip interoperability for server-side unzippers (#367) (#371)

* fix(ios): zip interoperability for server-side unzippers (#367)

Rebased onto #370 after #369 was squash-merged.

Co-authored-by: Perry <plrthink@gmail.com>

* docs: file-array zipWithPassword honors encryptionType on iOS

The interoperability change writes ZipCrypto vs AES based on
encryptionType for file arrays; drop the outdated README callout.

Co-authored-by: Perry <plrthink@gmail.com>

* docs: note iOS file-array zipWithPassword default is now ZipCrypto

Omitting encryptionType used to always write WinZip-AES for iOS file
arrays. Callers who need AES must pass AES-128 or AES-256.

Co-authored-by: Perry <plrthink@gmail.com>

---------

Co-authored-by: Cursor Agent <cursoragent@cursor.com>

* feat: close iOS/Android platform parity gaps (#368) (#372)

Rebased onto #371 after #369 was squash-merged.

Co-authored-by: Cursor Agent <cursoragent@cursor.com>

* fix: restore Maestro install retries and align unzip failure progress

Keep the hardened E2E Maestro install from master (retry + version
check). Emit 0% progress when iOS unzip/unzipAssets fails, matching
Android. Document the iOS file-array ZipCrypto default in MIGRATION.md.

Co-authored-by: Perry <plrthink@gmail.com>

* docs: point README migrating section at v9.2–v9.4 notes

Co-authored-by: Perry <plrthink@gmail.com>

---------

Co-authored-by: Cursor Agent <cursoragent@cursor.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants