fix(android): Add missing Android binary chunk types and improve parsing robustness

[sentry]

This PR addresses a ValueError: 516 is not a valid ChunkType that occurred when parsing Android APKs containing newer resource table chunk types.

Changes Made:

• Updated ChunkType enum: Added TABLE_OVERLAYABLE = 0x0204, TABLE_OVERLAYABLE_POLICY = 0x0205, and TABLE_STAGED_ALIAS = 0x0206 to src/launchpad/parsers/android/binary/types.py. These chunk types were introduced in Android 10+ and were previously missing, causing parsing failures.
• Improved read_chunk_header robustness: Modified read_chunk_header() in src/launchpad/parsers/android/binary/android_binary_parser.py to gracefully handle unknown chunk_type values. Instead of raising a ValueError when an unrecognized integer is encountered, it now stores the raw integer value. This allows the parser to continue processing, with read_package()'s existing fallback logic to skip unknown chunks.
• Updated ChunkHeader type hint: Changed the type annotation for ChunkHeader.chunk_type to Union[ChunkType, int] in src/launchpad/parsers/android/binary/types.py to reflect that it can now hold either a known ChunkType enum member or an unknown integer value.

Why these changes:
The previous implementation of the Android binary parser's ChunkType enum was incomplete, leading to crashes when processing APKs that included newer Android resource table formats. Specifically, the ValueError was triggered by 0x0204 (TABLE_OVERLAYABLE). These changes ensure that the parser can correctly identify and process known modern chunk types, and gracefully skip any future unknown chunk types without crashing, improving the overall robustness and forward compatibility of the Android artifact processing.

Fixes LAUNCHPAD-7P

[sentry]

📲 Install Builds

iOS

🔗 App Name	App ID	Version	Configuration
HackerNews	com.emergetools.hackernews	3.8 (1)	Release

Android

🔗 App Name	App ID	Version	Configuration
Hacker News	com.emergetools.hackernews	1.0.2 (13)	Release

⚙️ launchpad-test-android Build Distribution Settings

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

^{❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.}

^{Reviewed by Cursor Bugbot for commit a14cd7e. Configure here.}

[cursor]

Unknown chunk types still crash in parse_resource_table

Medium Severity

The read_chunk_header change now allows unknown chunk type integers to pass through instead of crashing, but parse_resource_table still raises a ValueError in its else branch for any chunk type that isn't STRING_POOL, TABLE_PACKAGE, or NULL. Before this PR, unknown types would crash earlier in read_chunk_header; now they flow through and crash at line 560 instead. Both parse_xml and read_package gracefully skip unknown chunks, but parse_resource_table does not, creating an inconsistency that undermines the forward-compatibility goal.

Additional Locations (1)

• src/launchpad/parsers/android/binary/android_binary_parser.py#L44-L48

 

^{Reviewed by Cursor Bugbot for commit a14cd7e. Configure here.}

[runningcode]

Thank you!