Use JavaAPI bytecode in VM integration tests

[shai-almog]

Motivation

• Replace fragile in-test generation of java.* sources with the real maintained JavaAPI output so VM/translator integration tests compile against a canonical API surface.
• Reduce duplicated, hand-written stub logic (including java.lang.invoke stubs) that can mask mismatches between tests and the JavaAPI project.
• Make cross-JDK/target compilation deterministic by compiling JavaAPI/src with the same JDK/target used by each test run and reusing the produced classes.

Description

• Added CompilerHelper.compileJavaAPI(Path, CompilerConfig) to compile ../JavaAPI/src into a temporary classes directory using the requested JDK/target and updated compilation behavior to use --patch-module on supported JDKs or fall back to source/target flags for older JDKs.
• Added CompilerHelper.copyDirectory(Path, Path) and updated integration tests to copy compiled JavaAPI classes into test classes directories so the translator receives real API bytecode.
• Updated integration tests CleanTargetIntegrationTest, LockIntegrationTest, ReadWriteLockIntegrationTest, and StampedLockIntegrationTest to stop emitting large java.* source stubs and instead compile/test against the JavaAPI classes, switching between -classpath (JDK9+) and -bootclasspath (pre-9) as appropriate and guarding unsupported JDK/target combinations.
• Updated LambdaIntegrationTest to select an appropriate compiler config, compile the JavaAPI with that compiler, remove ad-hoc java.lang.invoke stub generation, and use the compiled JavaAPI when compiling the test app.

Testing

• No automated tests were executed as part of this change (no CI or mvn run was requested).
• The changes are limited to test harness code and test sources and add guards to skip incompatible JDK/target permutations to avoid spurious failures during multi-JDK runs.
• Local assertions in unit tests remain; the PR does not claim runnable CI verification in this rollout.

---

Codex Task

[github-actions]

✅ Continuous Quality Report

Test & Coverage

• ✅ Tests: 2283 total, 0 failed, 0 skipped
• 📊 Line coverage: 52.79% [HTML preview] [Download]
  • Lowest covered classes
    • com.codename1.io.rest.RequestBuilder$FetchAsPropertyListActionListener – 0.00%
    • com.codename1.io.rest.RequestBuilder$FetchAsPropertiesActionListener – 0.00%
    • com.codename1.ui.Display$EdtException – 0.00%
    • com.codename1.ui.BrowserWindow$BrowserForm – 0.00%
    • com.codename1.ui.plaf.CSSBorder$LinearGradient – 0.00%
    • com.codename1.io.rest.RequestBuilder$GetAsStringAsyncImplActionListener – 0.00%
    • com.codename1.io.rest.RequestBuilder$GetAsJsonMapActionListener – 0.00%
    • com.codename1.payment.Purchase$SynchronizeReceiptsSyncRunnable – 0.00%
    • com.codename1.io.rest.RequestBuilder$GetAsBytesAsyncImplActionListener – 0.00%
    • com.codename1.io.rest.RequestBuilder$FetchAsJsonMapActionListener – 0.00%

Static Analysis

• ✅ SpotBugs: 0 findings (no issues) [Report archive]
• ❌ PMD: 478 findings (P2: 42, P3: 426, P4: 10) [HTML preview] [Download]
  • Top findings
    • P2: CodenameOne/src/com/codename1/capture/Capture.java:252 – Use of modifier volatile is not recommended. (rule: AvoidUsingVolatile)
    • P2: CodenameOne/src/com/codename1/compat/java/util/Objects.java:47 – The method name and parameter number are suspiciously close to equals(Object) (rule: SuspiciousEqualsMethodName)
    • P2: CodenameOne/src/com/codename1/components/StorageImageAsync.java:43 – Use of modifier volatile is not recommended. (rule: AvoidUsingVolatile)
    • P2: CodenameOne/src/com/codename1/components/StorageImageAsync.java:44 – Use of modifier volatile is not recommended. (rule: AvoidUsingVolatile)
    • P2: CodenameOne/src/com/codename1/io/BufferedInputStream.java:38 – Use of modifier volatile is not recommended. (rule: AvoidUsingVolatile)
    • …and 473 more
• ❌ Checkstyle: 221053 findings (Warning: 221053) [HTML preview] [Download]
  • Top findings
    • Warning: CodenameOne/src/com/codename1/db/ThreadSafeDatabase.java:30:1 – Extra separation in import group before 'java.io.IOException' (rule: CustomImportOrderCheck)
    • Warning: CodenameOne/src/com/codename1/db/ThreadSafeDatabase.java:32 – First sentence of Javadoc is missing an ending period. (rule: SummaryJavadocCheck)
    • Warning: CodenameOne/src/com/codename1/db/ThreadSafeDatabase.java:36 – Line is longer than 100 characters (found 143). (rule: LineLengthCheck)
    • Warning: CodenameOne/src/com/codename1/db/ThreadSafeDatabase.java:39:5 – 'member def modifier' has incorrect indentation level 4, expected level should be 2. (rule: IndentationCheck)
    • Warning: CodenameOne/src/com/codename1/db/ThreadSafeDatabase.java:40:5 – 'member def modifier' has incorrect indentation level 4, expected level should be 2. (rule: IndentationCheck)
    • …and 221048 more

Generated automatically by the PR CI workflow.

[shai-almog]

iOS screenshot updates

Compared 30 screenshots: 23 matched, 6 updated, 1 missing reference.

• BrowserComponent — updated screenshot. Screenshot differs (1206x2622 px, bit depth 8).

  
  Preview info: Preview provided by instrumentation.
  Full-resolution PNG saved as BrowserComponent.png in workflow artifacts.

• graphics-draw-arc — updated screenshot. Screenshot differs (1206x2622 px, bit depth 8).

  
  Preview info: JPEG preview quality 20; JPEG preview quality 20; downscaled to 603x1311.
  Full-resolution PNG saved as graphics-draw-arc.png in workflow artifacts.

• graphics-draw-gradient — updated screenshot. Screenshot differs (1206x2622 px, bit depth 8).

  
  Preview info: JPEG preview quality 10; JPEG preview quality 10; downscaled to 844x1835.
  Full-resolution PNG saved as graphics-draw-gradient.png in workflow artifacts.

• graphics-draw-round-rect — missing reference. Reference screenshot missing at /Users/runner/work/CodenameOne/CodenameOne/scripts/ios/screenshots/graphics-draw-round-rect.png.

  
  Preview info: JPEG preview quality 10; JPEG preview quality 10; downscaled to 844x1835.
  Full-resolution PNG saved as graphics-draw-round-rect.png in workflow artifacts.

• graphics-draw-string — updated screenshot. Screenshot differs (1206x2622 px, bit depth 8).

  
  Preview info: JPEG preview quality 10; JPEG preview quality 10; downscaled to 422x918.
  Full-resolution PNG saved as graphics-draw-string.png in workflow artifacts.

• graphics-draw-string-decorated — updated screenshot. Screenshot differs (1206x2622 px, bit depth 8).

  
  Preview info: JPEG preview quality 10; JPEG preview quality 10; downscaled to 603x1311.
  Full-resolution PNG saved as graphics-draw-string-decorated.png in workflow artifacts.

• kotlin — updated screenshot. Screenshot differs (1206x2622 px, bit depth 8).

  
  Preview info: Preview provided by instrumentation.
  Full-resolution PNG saved as kotlin.png in workflow artifacts.

Benchmark Results

• VM Translation Time: 362 seconds
• Compilation Time: 142 seconds

Detailed Performance Metrics

Metric	Duration
Build Time Statistics
Setup & Unzip	37539 ms
Extract Extensions	18 ms
Google Services Setup	2 ms
Scan Classes	1797 ms
Extract Libs	1242 ms
Inject Build Hints	71 ms
Generate Unit Tests	3 ms
Generate Stubs	1789 ms
Compile Stubs	3185 ms
Generate Icons	1686 ms
Prepare ParparVM	272 ms
ParparVM Execution	196693 ms
Post-VM Setup	216 ms
CocoaPods	4895 ms
Finalize	21 ms
Total Time	249430 msMaven Overhead : 113000 ms
CocoaPods Install (Script)	1000 ms
Simulator Boot (Run)	83000 ms
App Install	22000 ms
App Launch	16000 ms
Test Execution	165000 ms

[github-actions]

✅ ByteCodeTranslator Quality Report

Test & Coverage

• ✅ Tests: 135 total, 0 failed, 2 skipped

Benchmark Results

• Execution Time: 9938 ms

• Hotspots (Top 20 sampled methods):

  • 22.81% java.lang.String.indexOf (395 samples)
  • 17.90% com.codename1.tools.translator.Parser.isMethodUsed (310 samples)
  • 12.82% java.util.ArrayList.indexOf (222 samples)
  • 7.91% com.codename1.tools.translator.Parser.addToConstantPool (137 samples)
  • 4.85% java.lang.Object.hashCode (84 samples)
  • 4.04% com.codename1.tools.translator.ByteCodeClass.markDependent (70 samples)
  • 3.70% com.codename1.tools.translator.ByteCodeClass.calcUsedByNative (64 samples)
  • 3.06% java.lang.System.identityHashCode (53 samples)
  • 1.67% com.codename1.tools.translator.Parser.generateClassAndMethodIndexHeader (29 samples)
  • 1.62% com.codename1.tools.translator.BytecodeMethod.optimize (28 samples)
  • 1.44% com.codename1.tools.translator.BytecodeMethod.appendMethodC (25 samples)
  • 1.27% com.codename1.tools.translator.ByteCodeClass.fillVirtualMethodTable (22 samples)
  • 1.15% java.lang.StringBuilder.append (20 samples)
  • 0.92% java.io.FileOutputStream.writeBytes (16 samples)
  • 0.75% com.codename1.tools.translator.Parser.cullMethods (13 samples)
  • 0.75% com.codename1.tools.translator.BytecodeMethod.appendCMethodPrefix (13 samples)
  • 0.58% java.io.FileOutputStream.open0 (10 samples)
  • 0.52% com.codename1.tools.translator.Parser.cullClasses (9 samples)
  • 0.40% java.io.FileInputStream.open0 (7 samples)
  • 0.40% com.codename1.tools.translator.ByteCodeField.equals (7 samples)

• ⚠️ Coverage report not generated.

Static Analysis

• ✅ SpotBugs: no findings (report was not generated by the build).
• ⚠️ PMD report not generated.
• ⚠️ Checkstyle report not generated.

Generated automatically by the PR CI workflow.