Greptile Summary

This PR splits :reticulum and :data module tests out of kotlin-tests shard 0 (which was consistently ~12 min vs. ~9–10 min for other shards, causing intermittent 15-min timeout failures) into a new parallel module-tests job. The new job runs after lint and validate-wrapper — same as the other test jobs — and is correctly wired into the ci-passed gate via both its needs declaration and the shell failure-check condition.

• The previously raised concern about the missing Robolectric SDK cache has been fully addressed: Clean AIDL generated code, Clean JaCoCo execution data, and Cache Robolectric Android SDKs steps are all present and identical to the kotlin-tests job.
• The ci-passed needs array and failure-check condition both include module-tests, so the gate will block on failures from the new job.
• One minor gap: the success summary table in the CI Summary step was not updated to include a Module Tests row, even though the failure table was. This creates a cosmetic inconsistency in the step summary output.

Confidence Score: 4/5

• Safe to merge — structural CI refactor with correct gate wiring and all required setup steps present.
• The change is a well-structured CI reorganization: the new job mirrors the kotlin-tests setup exactly, the Robolectric cache concern from the prior review has been resolved, and the ci-passed gate correctly blocks on the new job. The only issue is a cosmetic omission in the success summary table, which has no impact on correctness or reliability.
• No files require special attention beyond the minor success-summary table omission in .github/workflows/ci.yml.

Important Files Changed

Flowchart

%%{init: {'theme': 'neutral'}}%%
flowchart TD
    A([validate-wrapper]) --> D
    B([lint]) --> D
    A --> E
    B --> E
    A --> F
    B --> F
    A --> G
    B --> G

    D["kotlin-tests\n(4 shards × :app tests)"]
    E["module-tests\n(:reticulum + :data tests)"]
    F["instrumented-tests"]
    G["other jobs\n(threading-audit, proguard, python-tests)"]

    D --> H
    E --> H
    F --> H
    G --> H

    H(["ci-passed\n✅ CI Passed — APK Ready"])

_{Last reviewed commit: c7af7ee}

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@greptileai