[jsscripting] Fix build and cleanup (#10654)

Related to #8516

Signed-off-by: Wouter Born <github@maindrain.net>

diff --git a/bom/openhab-addons/pom.xml b/bom/openhab-addons/pom.xml
index 3485b385073aa9769ab151608003c19d715ead9a..4735007ce2af3ea51951094df80a0cc87b59f652 100644 (file)
--- a/bom/openhab-addons/pom.xml
+++ b/bom/openhab-addons/pom.xml
@@ -23,17 +23,17 @@
     </dependency>
     <dependency>
       <groupId>org.openhab.addons.bundles</groupId>
-      <artifactId>org.openhab.automation.jythonscripting</artifactId>
+      <artifactId>org.openhab.automation.jsscripting</artifactId>
       <version>${project.version}</version>
     </dependency>
     <dependency>
       <groupId>org.openhab.addons.bundles</groupId>
-      <artifactId>org.openhab.automation.pidcontroller</artifactId>
+      <artifactId>org.openhab.automation.jythonscripting</artifactId>
       <version>${project.version}</version>
     </dependency>
     <dependency>
       <groupId>org.openhab.addons.bundles</groupId>
-      <artifactId>org.openhab.automation.jsscripting</artifactId>
+      <artifactId>org.openhab.automation.pidcontroller</artifactId>
       <version>${project.version}</version>
     </dependency>
     <dependency>

diff --git a/bundles/org.openhab.automation.jsscripting/src/main/java/org/openhab/automation/jsscripting/internal/DebuggingGraalScriptEngine.java b/bundles/org.openhab.automation.jsscripting/src/main/java/org/openhab/automation/jsscripting/internal/DebuggingGraalScriptEngine.java
index 0f21f455d16daceba5e7e479048961497fbe9fcf..956f4fbdf72869a10b34d92b68fd36df0fb339d0 100644 (file)
--- a/bundles/org.openhab.automation.jsscripting/src/main/java/org/openhab/automation/jsscripting/internal/DebuggingGraalScriptEngine.java
+++ b/bundles/org.openhab.automation.jsscripting/src/main/java/org/openhab/automation/jsscripting/internal/DebuggingGraalScriptEngine.java
@@ -17,7 +17,6 @@ import javax.script.Invocable;
 import javax.script.ScriptEngine;
 import javax.script.ScriptException;
 
-import org.eclipse.jdt.annotation.NonNullByDefault;
 import org.graalvm.polyglot.PolyglotException;
 import org.openhab.automation.jsscripting.internal.scriptengine.InvocationInterceptingScriptEngineWithInvocable;
 import org.slf4j.Logger;
@@ -28,11 +27,11 @@ import org.slf4j.LoggerFactory;
  *
  * @author Jonathan Gilbert - Initial contribution
  */
-@NonNullByDefault
 class DebuggingGraalScriptEngine<T extends ScriptEngine & Invocable>
         extends InvocationInterceptingScriptEngineWithInvocable<T> {
 
-    private static final Logger stackLogger = LoggerFactory.getLogger("org.openhab.automation.script.javascript.stack");
+    private static final Logger STACK_LOGGER = LoggerFactory
+            .getLogger("org.openhab.automation.script.javascript.stack");
 
     public DebuggingGraalScriptEngine(T delegate) {
         super(delegate);
@@ -42,7 +41,7 @@ class DebuggingGraalScriptEngine<T extends ScriptEngine & Invocable>
     public ScriptException afterThrowsInvocation(ScriptException se) {
         Throwable cause = se.getCause();
         if (cause instanceof PolyglotException) {
-            stackLogger.error("Failed to execute script:", cause);
+            STACK_LOGGER.error("Failed to execute script:", cause);
         }
         return se;
     }

diff --git a/bundles/org.openhab.automation.jsscripting/src/main/java/org/openhab/automation/jsscripting/internal/OpenhabGraalJSScriptEngine.java b/bundles/org.openhab.automation.jsscripting/src/main/java/org/openhab/automation/jsscripting/internal/OpenhabGraalJSScriptEngine.java
index 8541eb7db8505ce5ed774b8395e7fdf20256fa9c..7ba654eec7cfbab43945446bea73cb14973f3394 100644 (file)
--- a/bundles/org.openhab.automation.jsscripting/src/main/java/org/openhab/automation/jsscripting/internal/OpenhabGraalJSScriptEngine.java
+++ b/bundles/org.openhab.automation.jsscripting/src/main/java/org/openhab/automation/jsscripting/internal/OpenhabGraalJSScriptEngine.java
@@ -13,8 +13,7 @@
 
 package org.openhab.automation.jsscripting.internal;
 
-import static org.openhab.core.automation.module.script.ScriptEngineFactory.CONTEXT_KEY_ENGINE_IDENTIFIER;
-import static org.openhab.core.automation.module.script.ScriptEngineFactory.CONTEXT_KEY_EXTENSION_ACCESSOR;
+import static org.openhab.core.automation.module.script.ScriptEngineFactory.*;
 
 import java.io.File;
 import java.io.IOException;
@@ -48,17 +47,15 @@ import com.oracle.truffle.js.scriptengine.GraalJSScriptEngine;
  */
 public class OpenhabGraalJSScriptEngine extends InvocationInterceptingScriptEngineWithInvocable<GraalJSScriptEngine> {
 
-    private static final Logger logger = LoggerFactory.getLogger(OpenhabGraalJSScriptEngine.class);
+    private static final Logger LOGGER = LoggerFactory.getLogger(OpenhabGraalJSScriptEngine.class);
 
     private static final String REQUIRE_WRAPPER_NAME = "__wraprequire__";
     private static final String MODULE_DIR = String.join(File.separator, OpenHAB.getConfigFolder(), "automation", "lib",
             "javascript", "personal");
 
     // these fields start as null because they are populated on first use
-    @NonNullByDefault({})
-    private String engineIdentifier;
-    @NonNullByDefault({})
-    private Consumer<String> scriptDependencyListener;
+    private @NonNullByDefault({}) String engineIdentifier;
+    private @NonNullByDefault({}) Consumer<String> scriptDependencyListener;
 
     private boolean initialized = false;
 
@@ -115,7 +112,7 @@ public class OpenhabGraalJSScriptEngine extends InvocationInterceptingScriptEngi
         scriptDependencyListener = (Consumer<String>) ctx
                 .getAttribute("oh.dependency-listener"/* CONTEXT_KEY_DEPENDENCY_LISTENER */);
         if (scriptDependencyListener == null) {
-            logger.warn(
+            LOGGER.warn(
                     "Failed to retrieve script script dependency listener from engine bindings. Script dependency tracking will be disabled.");
         }
 

diff --git a/bundles/org.openhab.automation.jsscripting/src/main/java/org/openhab/automation/jsscripting/internal/ScriptExtensionModuleProvider.java b/bundles/org.openhab.automation.jsscripting/src/main/java/org/openhab/automation/jsscripting/internal/ScriptExtensionModuleProvider.java
index d2d14011426d9b1e7e6c6b51861ddb4ed06e2732..e5d77ff4213e79f6b0e2feb3e5ba1c8382cdef4e 100644 (file)
--- a/bundles/org.openhab.automation.jsscripting/src/main/java/org/openhab/automation/jsscripting/internal/ScriptExtensionModuleProvider.java
+++ b/bundles/org.openhab.automation.jsscripting/src/main/java/org/openhab/automation/jsscripting/internal/ScriptExtensionModuleProvider.java
@@ -59,7 +59,6 @@ public class ScriptExtensionModuleProvider {
     }
 
     private Optional<Value> runtimeModule(String name, String scriptIdentifier, Context ctx) {
-
         Map<String, Object> symbols;
 
         if (DEFAULT_MODULE_NAME.equals(name)) {

diff --git a/bundles/org.openhab.automation.jsscripting/src/main/java/org/openhab/automation/jsscripting/internal/fs/PrefixedSeekableByteChannel.java b/bundles/org.openhab.automation.jsscripting/src/main/java/org/openhab/automation/jsscripting/internal/fs/PrefixedSeekableByteChannel.java
index 2bd98ca8580425f565936b53456f6fb2b91208c0..d9d62f6ee3b93a7a391ad022a9fc72c961da0d7e 100644 (file)
--- a/bundles/org.openhab.automation.jsscripting/src/main/java/org/openhab/automation/jsscripting/internal/fs/PrefixedSeekableByteChannel.java
+++ b/bundles/org.openhab.automation.jsscripting/src/main/java/org/openhab/automation/jsscripting/internal/fs/PrefixedSeekableByteChannel.java
@@ -36,7 +36,6 @@ public class PrefixedSeekableByteChannel implements SeekableByteChannel {
 
     @Override
     public int read(ByteBuffer dst) throws IOException {
-
         int read = 0;
 
         if (position < prefix.length) {
@@ -63,7 +62,6 @@ public class PrefixedSeekableByteChannel implements SeekableByteChannel {
 
     @Override
     public SeekableByteChannel position(long newPosition) throws IOException {
-
         this.position = newPosition;
 
         if (newPosition > prefix.length) {

diff --git a/bundles/org.openhab.automation.jsscripting/src/main/java/org/openhab/automation/jsscripting/internal/scriptengine/InvocationInterceptingScriptEngineWithInvocable.java b/bundles/org.openhab.automation.jsscripting/src/main/java/org/openhab/automation/jsscripting/internal/scriptengine/InvocationInterceptingScriptEngineWithInvocable.java
index 082a98dde154adaec6ff33855a82737de3da9574..57640e8de7dc67f9627551a18c38465b4c8cfb38 100644 (file)
--- a/bundles/org.openhab.automation.jsscripting/src/main/java/org/openhab/automation/jsscripting/internal/scriptengine/InvocationInterceptingScriptEngineWithInvocable.java
+++ b/bundles/org.openhab.automation.jsscripting/src/main/java/org/openhab/automation/jsscripting/internal/scriptengine/InvocationInterceptingScriptEngineWithInvocable.java
@@ -21,8 +21,6 @@ import javax.script.ScriptContext;
 import javax.script.ScriptEngine;
 import javax.script.ScriptException;
 
-import org.eclipse.jdt.annotation.NonNullByDefault;
-
 /**
  * Delegate allowing AOP-style interception of calls, either before Invocation, or upon a {@link ScriptException}.
  * being thrown.
@@ -30,7 +28,6 @@ import org.eclipse.jdt.annotation.NonNullByDefault;
  * @param <T> The delegate class
  * @author Jonathan Gilbert - Initial contribution
  */
-@NonNullByDefault
 public abstract class InvocationInterceptingScriptEngineWithInvocable<T extends ScriptEngine & Invocable>
         extends DelegatingScriptEngineWithInvocable<T> {
 

diff --git a/bundles/pom.xml b/bundles/pom.xml
index 29fe4b9adc98fd9d4e1d7f629ebbfad29ec7d467..adb1ec2c78d8890d51c8dce6f9d5c3ed3df238dc 100644 (file)
--- a/bundles/pom.xml
+++ b/bundles/pom.xml
@@ -19,9 +19,9 @@
   <modules>
     <!-- automation -->
     <module>org.openhab.automation.groovyscripting</module>
+    <module>org.openhab.automation.jsscripting</module>
     <module>org.openhab.automation.jythonscripting</module>
     <module>org.openhab.automation.pidcontroller</module>
-    <module>org.openhab.automation.jsscripting</module>
     <!-- io -->
     <module>org.openhab.io.homekit</module>
     <module>org.openhab.io.hueemulation</module>