[PR #2917] [MERGED] test(android): preserve malformed DEX prototype rejection #5491

New issue

Closed

opened 2026-10-31 01:07:04 +00:00 by bsakaguchi · 0 comments

bsakaguchi commented

2026-10-31 01:07:04 +00:00

Owner

📋 Pull Request Information

Original PR: https://github.com/morluto/rea/pull/2917
Author: @N0zoM1z0
Created: 10/13/2026
Status: ✅ Merged
Merged: 10/14/2026
Merged by: @N0zoM1z0

Base: main ← Head: fix/android-dex-descriptor-portable-pattern

📝 Commits (10+)

f5a4665 fix(android): advertise portable DEX descriptor patterns
500fd6d test(android): retain malformed prototype boundary regressions
6fd9925 Merge branch 'main' into fix/android-dex-descriptor-portable-pattern
51b1f10 chore: update Android regression branch from main
c3e3628 Merge branch 'main' into fix/android-dex-descriptor-portable-pattern
b396e40 chore: sync verification branch with latest main
08a3132 chore: sync Android regression branch with latest main
46dce02 chore: sync upstream Hopper test corrections
4b404cc chore: sync upstream Hopper test formatting
13813bc chore: sync current CI and provider verification baseline

📊 Changes

1 file changed (+40 additions, -22 deletions)

View changed files

📝 tests/boundary/android/JadxAnalysis.test.ts (+40 -22)

📄 Description

Strengthen malformed DEX prototype rejection with parameter and return object descriptors containing each ASCII parenthesis, requiring the tagged JADX output error at the exact descriptor path. The AOSP DEX SimpleNameChar grammar excludes these characters. The production regex portability fix was merged independently in #2910; this PR changes only the Android boundary regression test.

Validation on latest captured main e049ef70f7: complete npm run check and all 13 Android boundary cases pass. Earlier real JADX 0.7.1/JDK21 ApiDemos public CLI/MCP verification passed; this test-only change adds no provider analysis work. Upstream PowerPC installation and CI-report fixtures have been repaired on main and are included in the integration.

Every own commit and the prepared final squash message retain Kaoru0822's canonical co-author trailer.

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/morluto/rea/pull/2917 **Author:** [@N0zoM1z0](https://github.com/N0zoM1z0) **Created:** 10/13/2026 **Status:** ✅ Merged **Merged:** 10/14/2026 **Merged by:** [@N0zoM1z0](https://github.com/N0zoM1z0) **Base:** `main` ← **Head:** `fix/android-dex-descriptor-portable-pattern` --- ### 📝 Commits (10+) - [`f5a4665`](https://github.com/morluto/rea/commit/f5a4665cce2e36b33cbf0055c44587913126a8aa) fix(android): advertise portable DEX descriptor patterns - [`500fd6d`](https://github.com/morluto/rea/commit/500fd6de88105b0d2e3edece59aca5d5f9f5f03f) test(android): retain malformed prototype boundary regressions - [`6fd9925`](https://github.com/morluto/rea/commit/6fd99255729bb73ffb99341bd38b3931da382ab7) Merge branch 'main' into fix/android-dex-descriptor-portable-pattern - [`51b1f10`](https://github.com/morluto/rea/commit/51b1f10da709d58773f8d6aec3aea7036b3bf39c) chore: update Android regression branch from main - [`c3e3628`](https://github.com/morluto/rea/commit/c3e362869c20a7c4a933baebae96d21edb61d0c3) Merge branch 'main' into fix/android-dex-descriptor-portable-pattern - [`b396e40`](https://github.com/morluto/rea/commit/b396e408daa0d0797e849eea0619353ffdfb2eaf) chore: sync verification branch with latest main - [`08a3132`](https://github.com/morluto/rea/commit/08a31321b554c599a30a526f0bd37db66b6619cb) chore: sync Android regression branch with latest main - [`46dce02`](https://github.com/morluto/rea/commit/46dce0286d5528eedf5b72629ebb162a0a772191) chore: sync upstream Hopper test corrections - [`4b404cc`](https://github.com/morluto/rea/commit/4b404cc0fa0e8415a9e03be0951ff79a2709f107) chore: sync upstream Hopper test formatting - [`13813bc`](https://github.com/morluto/rea/commit/13813bc524637c8af3035e486ea9fbc12a1372a7) chore: sync current CI and provider verification baseline ### 📊 Changes **1 file changed** (+40 additions, -22 deletions) <details> <summary>View changed files</summary> 📝 `tests/boundary/android/JadxAnalysis.test.ts` (+40 -22) </details> ### 📄 Description Strengthen malformed DEX prototype rejection with parameter and return object descriptors containing each ASCII parenthesis, requiring the tagged JADX output error at the exact descriptor path. The AOSP DEX SimpleNameChar grammar excludes these characters. The production regex portability fix was merged independently in #2910; this PR changes only the Android boundary regression test. Validation on latest captured main e049ef70f71aeb9965e9d75d4d4a84ea02eaee11: complete npm run check and all 13 Android boundary cases pass. Earlier real JADX 0.7.1/JDK21 ApiDemos public CLI/MCP verification passed; this test-only change adds no provider analysis work. Upstream PowerPC installation and CI-report fixtures have been repaired on main and are included in the integration. Every own commit and the prepared final squash message retain Kaoru0822's canonical co-author trailer. --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>

bsakaguchi

2026-10-31 01:07:04 +00:00