fix(gradle): Use application variant instead of varian output to hook to correct package task for modules cleanup

[krystofwoldrich]

📢 Type of change

• Bugfix
• New feature
• Enhancement
• Refactoring

📜 Description

The variant output name includes architecture but the package task has only the application variant in the name.

demo-x86_64-release vs. demoRelease

💡 Motivation and Context

closes: #3157

💚 How did you test it?

📝 Checklist

• I reviewed submitted code
• I added tests to verify changes
• No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled
• All tests passing
• No breaking changes

🔮 Next steps

[github-actions]

Android (legacy) Performance metrics 🚀

	Plain	With Sentry	Diff
Startup time	371.90 ms	382.24 ms	10.35 ms
Size	17.73 MiB	19.83 MiB	2.09 MiB

Baseline results on branch: main

Startup times

Revision	Plain	With Sentry	Diff
d197b5c+dirty	338.94 ms	354.87 ms	15.93 ms
e73f4ed+dirty	332.96 ms	354.33 ms	21.37 ms
34aba08	328.10 ms	342.84 ms	14.74 ms
8900e1a+dirty	430.68 ms	456.13 ms	25.44 ms
3853f43	329.68 ms	346.32 ms	16.64 ms
0db0c72	372.12 ms	386.00 ms	13.88 ms
9a3ca65+dirty	326.93 ms	330.14 ms	3.21 ms
52a8031+dirty	311.55 ms	321.37 ms	9.82 ms
15c80ab+dirty	336.27 ms	350.58 ms	14.31 ms
70caa60+dirty	299.00 ms	321.02 ms	22.02 ms

App size

Revision	Plain	With Sentry	Diff
d197b5c+dirty	17.73 MiB	20.04 MiB	2.31 MiB
e73f4ed+dirty	17.73 MiB	20.04 MiB	2.31 MiB
34aba08	17.73 MiB	19.80 MiB	2.07 MiB
8900e1a+dirty	17.73 MiB	19.75 MiB	2.01 MiB
3853f43	17.73 MiB	19.81 MiB	2.08 MiB
0db0c72	17.73 MiB	19.75 MiB	2.02 MiB
9a3ca65+dirty	17.73 MiB	20.04 MiB	2.31 MiB
52a8031+dirty	17.73 MiB	20.04 MiB	2.31 MiB
15c80ab+dirty	17.73 MiB	20.04 MiB	2.31 MiB
70caa60+dirty	17.73 MiB	19.75 MiB	2.01 MiB

[github-actions]

iOS (legacy) Performance metrics 🚀

	Plain	With Sentry	Diff
Startup time	1284.57 ms	1286.12 ms	1.55 ms
Size	2.36 MiB	2.86 MiB	512.99 KiB

Baseline results on branch: main

Startup times

Revision	Plain	With Sentry	Diff
d197b5c+dirty	1217.61 ms	1242.66 ms	25.05 ms
e73f4ed+dirty	1243.27 ms	1244.52 ms	1.25 ms
e2b64fe+dirty	1232.22 ms	1255.20 ms	22.98 ms
8900e1a+dirty	1210.27 ms	1218.66 ms	8.39 ms
9a3ca65+dirty	1247.06 ms	1274.58 ms	27.52 ms
34aba08+dirty	1276.78 ms	1308.52 ms	31.74 ms
52a8031+dirty	1280.88 ms	1289.78 ms	8.90 ms
15c80ab+dirty	1223.74 ms	1228.96 ms	5.22 ms
70caa60+dirty	1218.27 ms	1230.30 ms	12.03 ms
86d6d2c+dirty	1267.55 ms	1286.21 ms	18.66 ms

App size

Revision	Plain	With Sentry	Diff
d197b5c+dirty	2.36 MiB	2.82 MiB	462.86 KiB
e73f4ed+dirty	2.36 MiB	2.82 MiB	469.44 KiB
e2b64fe+dirty	2.36 MiB	2.85 MiB	495.80 KiB
8900e1a+dirty	2.36 MiB	2.83 MiB	479.25 KiB
9a3ca65+dirty	2.36 MiB	2.82 MiB	462.89 KiB
34aba08+dirty	2.36 MiB	2.85 MiB	495.32 KiB
52a8031+dirty	2.36 MiB	2.82 MiB	469.44 KiB
15c80ab+dirty	2.36 MiB	2.83 MiB	474.49 KiB
70caa60+dirty	2.36 MiB	2.83 MiB	479.27 KiB
86d6d2c+dirty	2.36 MiB	2.82 MiB	462.82 KiB

[github-actions]

Android (new) Performance metrics 🚀

	Plain	With Sentry	Diff
Startup time	281.39 ms	323.90 ms	42.51 ms
Size	7.15 MiB	8.10 MiB	972.54 KiB

Baseline results on branch: main

Startup times

Revision	Plain	With Sentry	Diff
d197b5c+dirty	258.75 ms	313.61 ms	54.86 ms
e73f4ed+dirty	262.98 ms	311.02 ms	48.04 ms
e2b64fe+dirty	258.82 ms	304.26 ms	45.44 ms
8900e1a+dirty	371.40 ms	377.70 ms	6.31 ms
9a3ca65+dirty	344.96 ms	358.92 ms	13.96 ms
34aba08+dirty	331.79 ms	376.69 ms	44.91 ms
52a8031+dirty	330.72 ms	358.76 ms	28.03 ms
15c80ab+dirty	276.38 ms	327.54 ms	51.17 ms
70caa60+dirty	308.83 ms	393.06 ms	84.23 ms
86d6d2c+dirty	267.21 ms	325.24 ms	58.04 ms

App size

Revision	Plain	With Sentry	Diff
d197b5c+dirty	7.15 MiB	8.09 MiB	962.72 KiB
e73f4ed+dirty	7.15 MiB	8.09 MiB	965.94 KiB
e2b64fe+dirty	7.15 MiB	8.07 MiB	947.16 KiB
8900e1a+dirty	7.15 MiB	8.03 MiB	901.79 KiB
9a3ca65+dirty	7.15 MiB	8.09 MiB	962.83 KiB
34aba08+dirty	7.15 MiB	8.07 MiB	946.13 KiB
52a8031+dirty	7.15 MiB	8.09 MiB	965.95 KiB
15c80ab+dirty	7.15 MiB	8.09 MiB	966.13 KiB
70caa60+dirty	7.15 MiB	8.03 MiB	901.79 KiB
86d6d2c+dirty	7.15 MiB	8.09 MiB	962.69 KiB

[marandaneto]

Have you tested this using normal apk generation and bundle generation, with splitting per architecture enabled or not?
We need to ensure this does not break any other way to generate your apks or bundles.

[krystofwoldrich]

@marandaneto Yes, I've tested bundle and apks with arch splitting and without.

For the splitting, I used this setting.

    splits {
        abi {
            reset()
            enable true
            universalApk true  // If true, also generate a universal APK
            include "armeabi-v7a", "x86", "arm64-v8a", "x86_64"
        }
    }

[liamjones]

@krystofwoldrich Thanks for the fix! I think the changelog is still wrong in main post merge? This wasn't released in 5.8.1, right?