Ensure Component is present in generated CN1 access registry, support local/release sources, and validate Javadoc snippets

[liannacasper]

Motivation

• Make the playground tooling reliably expose core com.codename1.ui.Component (and other key UI types) to the BeanShell CN1 access registry so loose scripts can reference Component without explicit imports.
• Allow generating the CN1 access registry from either local repository sources or release source jars and fail fast if expected classes are missing.
• Validate extracted Javadoc snippets before generating Javadoc to catch snippet issues early.

Description

• Updated generate-cn1-access-registry.sh to support CN1_ACCESS_USE_LOCAL_SOURCES=true to scan local repo sources, otherwise download and extract release source jars for the configured cn1.version, with improved error handling and clearer logging.
• Modified the registry generator (GenerateCN1AccessRegistry.java) to treat types missing from the runtime classpath as supported for registration so they can be emitted into the generated registry.
• Changed the smoke-test runner run-playground-smoke-tests.sh to always regenerate the registry from release sources before testing, verify the presence of com.codename1.ui.Component and a set of key UI classes in the generated GeneratedCN1Access.java, and run the PlaygroundSmokeHarness via Maven.
• Adjusted PlaygroundRunner to bind Display and UIManager to their live instances and to expose Component to the interpreter namespace, removing the previous fallback resolver methods.
• Replaced CSS-related smoke tests with a focused smoke test (smokeComponentTypeResolvesWithoutExplicitImport) that asserts Component resolution without an explicit import in loose scripts, and simplified imports in PlaygroundSmokeHarness.
• Added a run_with_timeout helper and a pre-Javadoc validation step in .github/scripts/build_javadocs.sh that runs validate-extracted-javadoc-snippets.sh with a 300s timeout when present.
• Documentation (scripts/cn1playground/tools/README.md) updated to explain the local vs release source behavior and the smoke-test verification steps.

Testing

• Ran the playground smoke tests via scripts/cn1playground/tools/run-playground-smoke-tests.sh, which regenerates the CN1 access registry, verifies required com.codename1.ui.* entries (including Component), and then executes com.codenameone.playground.PlaygroundSmokeHarness; the smoke harness completed successfully.
• The Maven test-compile + exec invocation used by the smoke test runner completed successfully during verification.

---

Codex Task

[github-actions]

✅ Continuous Quality Report

Test & Coverage

• ✅ Tests: 2348 total, 0 failed, 0 skipped
• 📊 Line coverage: 53.31% [HTML preview] [Download]
  • Lowest covered classes
    • com.codename1.ui.Display$EdtException – 0.00%
    • com.codename1.ui.plaf.CSSBorder$LinearGradient – 0.00%
    • com.codename1.util.EasyThread$InQueueRunnable – 0.00%
    • com.codename1.components.FloatingActionButton$ReleaseActionListener – 0.00%
    • com.codename1.io.Oauth2$ShowAuthenticationActionListener – 0.00%
    • com.codename1.io.Oauth2$RefreshTokenActionListener – 0.00%
    • com.codename1.util.EasyThread$RunAndWaitRunnable – 0.00%
    • com.codename1.components.ToastBar$FlushAnimationCallback – 0.00%
    • com.codename1.ui.plaf.CSSBorder$ColorStop – 0.00%
    • com.codename1.ui.ElevationComparator – 0.00%

Static Analysis

• SpotBugs [HTML preview] [Download]
  • ❌ ByteCodeTranslator: 97 findings (High: 2, Normal: 68, Low: 27)
    • Top findings
      • High: com/codename1/tools/translator/BytecodeMethod.java:559 – Method com.codename1.tools.translator.BytecodeMethod.findWritableFields(Set) seems to be useless (rule: UC_USELESS_VOID_METHOD)
      • High: com/codename1/tools/translator/bytecodes/TryCatch.java:75 – Write to static field com.codename1.tools.translator.bytecodes.TryCatch.hasTryCatch from instance method com.codename1.tools.translator.bytecodes.TryCatch.appendInstruction(StringBuilder, List) (rule: ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD)
      • Normal: com/codename1/tools/translator/ByteCodeClass.java:1791 – com.codename1.tools.translator.ByteCodeClass.getBaseClassObject() may expose internal representation by returning ByteCodeClass.baseClassObject (rule: EI_EXPOSE_REP)
      • …and 94 more
  • ❌ android: 413 findings (High: 24, Normal: 196, Low: 193)
    • Top findings
      • High: CodenameOne/src/com/codename1/impl/CodenameOneThread.java:48 – com.codename1.impl.CodenameOneThread.STACK_FRAME_SIZE isn't final but should be (rule: MS_SHOULD_BE_FINAL)
      • High: com/codename1/impl/android/AndroidImplementation.java:1085 – Write to static field com.codename1.impl.android.AndroidImplementation.context from instance method com.codename1.impl.android.AndroidImplementation.setContext(Context) (rule: ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD)
      • High: com/codename1/impl/android/AndroidImplementation.java:1286 – com.codename1.impl.android.AndroidImplementation.isDarkMode() has Boolean return type and returns explicit null (rule: NP_BOOLEAN_RETURN_NULL)
      • …and 410 more
  • ❌ codenameone-maven-plugin: 487 findings (High: 1, Normal: 315, Low: 171)
    • Top findings
      • High: com/codename1/maven/AbstractCN1Mojo.java:807 – com.codename1.maven.AbstractCN1Mojo.OS isn't final but should be (rule: MS_SHOULD_BE_FINAL)
      • Normal: com/codename1/builders/AndroidGradleBuilder.java:1018 – Dead store to googleServicesJson in com.codename1.builders.AndroidGradleBuilder.build(File, BuildRequest) (rule: DLS_DEAD_LOCAL_STORE)
      • Normal: com/codename1/builders/AndroidGradleBuilder.java:1043 – Possible null pointer dereference in com.codename1.builders.AndroidGradleBuilder.build(File, BuildRequest) due to return value of called method (rule: NP_NULL_ON_SOME_PATH_FROM_RETURN_VALUE)
      • …and 484 more
  • ❌ core-unittests: 1 findings (Normal: 1)
    • Top findings
      • Normal: CodenameOne/src/com/codename1/ui/util/MutableResource.java:239 – com.codename1.ui.util.MutableResource.saveL10N(DataOutputStream, Hashtable) makes inefficient use of keySet iterator instead of entrySet iterator (rule: WMI_WRONG_MAP_ITERATOR)
  • ❌ ios: 125 findings (High: 4, Normal: 75, Low: 46)
    • Top findings
      • High: com/codename1/impl/ios/IOSImplementation.java:1228 – com.codename1.impl.ios.IOSImplementation.isDarkMode() has Boolean return type and returns explicit null (rule: NP_BOOLEAN_RETURN_NULL)
      • High: com/codename1/impl/ios/IOSImplementation.java:128 – com.codename1.impl.ios.IOSImplementation.nativeInstance isn't final but should be (rule: MS_SHOULD_BE_FINAL)
      • High: com/codename1/impl/ios/IOSImplementation.java:214 – Write to static field com.codename1.impl.ios.IOSImplementation.instance from instance method com.codename1.impl.ios.IOSImplementation.init(Object) (rule: ST_WRITE_TO_STATIC_FROM_INSTANCE_METHOD)
      • …and 122 more
• ✅ PMD: 0 findings (no issues) [Report archive]
• ✅ Checkstyle: 0 findings (no issues) [Report archive]

Generated automatically by the PR CI workflow.