Big Update!

 - Annotation processor works now, and now also makes sure it can access fields/properties
 - CommandScheduler is now a singleton! This allows for a lot of helpful features
 - CommandInterface allows easy access to CommandScheduler
 - New CommandOpMode in case you don't like using Robots
 - Fixes to SuperTelemetry
 - Robot no longer needs an OpMode as a constructor parameter
 - Motor gearRatio and distancePerOutputRev separated, allowing for setTargetAngle and goToAngle
 - New Motor.Kind for easy construction
 - Servo range set to 300 degrees by default because all the goBILDA servos have that range
 - command module now requires minSdkVersion of 24

README.md

@@ -27,7 +27,7 @@ repositories {
 }
 
 dependencies {
-  implementation "com.github.amarcolini.joos:$module:0.4.5-alpha"
+  implementation "com.github.amarcolini.joos:$module:0.4.6-alpha"
 }
 ```
 
@@ -52,6 +52,6 @@ java {
 
 dependencies {
     //Gradle will automatically retrieve the correct dependencies based on your operating system
-    implementation "com.github.amarcolini.joos:gui:0.4.5-alpha"
+    implementation "com.github.amarcolini.joos:gui:0.4.6-alpha"
 }
 ````

build.gradle

@@ -1,7 +1,7 @@
 buildscript {
     ext {
         kotlin_version = "1.6.21"
-        lib_version = "0.4.5-alpha"
+        lib_version = "0.4.6-alpha"
     }
 
     repositories {

command/annotation/build.gradle

@@ -1,7 +1,5 @@
 plugins {
     id "kotlin"
-    id "java"
-    id 'com.google.devtools.ksp' version '1.6.21-1.0.5'
     id 'maven-publish'
 }
 
@@ -17,6 +15,8 @@ compileKotlin {
     }
 }
 
+group = group + ".command"
+
 dependencies {
     implementation "org.jetbrains.kotlin:kotlin-stdlib:$kotlin_version"
     implementation "org.jetbrains.kotlin:kotlin-reflect:$kotlin_version"
@@ -25,9 +25,6 @@ dependencies {
 
     implementation "com.squareup:javapoet:1.13.0"
     implementation "com.squareup:kotlinpoet:1.11.0"
-
-    ksp 'com.google.auto.service:auto-service:1.0.1'
-    implementation 'com.google.auto.service:auto-service-annotations:1.0.1'
 }
 
 publishing {
@@ -52,7 +49,7 @@ publishing {
     repositories {
         maven {
             name = 'testRepo'
-            url = '../testRepo'
+            url = '../../testRepo'
         }
     }
 }

command/annotation/src/main/kotlin/com/amarcolini/joos/dashboard/javaProcessor.kt

@@ -1,15 +1,12 @@
 package com.amarcolini.joos.dashboard
 
-import com.google.auto.service.AutoService
 import com.squareup.javapoet.*
-import com.squareup.kotlinpoet.asTypeName
 import java.util.*
 import javax.annotation.processing.*
 import javax.lang.model.SourceVersion
 import javax.lang.model.element.*
 import kotlin.collections.ArrayList
 
-@AutoService(Processor::class)
 @SupportedSourceVersion(SourceVersion.RELEASE_8)
 class ConfigProcessor : AbstractProcessor() {
     private val results = ArrayList<Pair<String, Pair<String, String>>>()
@@ -64,9 +61,10 @@ class ConfigElementVisitor : ElementVisitor<List<Pair<String, Pair<String, Strin
         emptyList()
 
     override fun visitType(p0: TypeElement?, p1: TypeElement?): List<Pair<String, Pair<String, String>>> {
+        if (p0?.modifiers?.contains(Modifier.PUBLIC) != true) return emptyList()
         val configMembers = ArrayList<Pair<String, Pair<String, String>>>()
-        val name = getAltName(p0, p1) ?: p0?.simpleName.toString()
-        p0?.enclosedElements?.forEach { member ->
+        val name = getAltName(p0, p1) ?: p0.simpleName.toString()
+        p0.enclosedElements?.forEach { member ->
             if (member.kind == ElementKind.FIELD && member.modifiers.containsAll(
                     listOf(
                         Modifier.PUBLIC, Modifier.STATIC
@@ -98,6 +96,7 @@ class ConfigElementVisitor : ElementVisitor<List<Pair<String, Pair<String, Strin
                 }
             }
             val parent = possibleParent ?: return emptyList()
+            if (!parent.modifiers.contains(Modifier.PUBLIC)) return emptyList()
             return listOf(
                 (getAltName(p0, p1) ?: getAltName(parent, p1)
                 ?: parent.simpleName.toString()) to (parent.qualifiedName.toString() to p0.simpleName.toString())

command/annotation/src/main/kotlin/com/amarcolini/joos/dashboard/kotlinProcessor.kt

@@ -1,14 +1,12 @@
 package com.amarcolini.joos.dashboard
 
-import com.google.auto.service.AutoService
 import com.google.devtools.ksp.*
 import com.google.devtools.ksp.processing.*
 import com.google.devtools.ksp.symbol.*
 import com.google.devtools.ksp.visitor.KSEmptyVisitor
 import com.squareup.kotlinpoet.*
 import kotlin.reflect.KProperty0
 import kotlin.io.*
-import kotlin.reflect.KClass
 
 class ConfigSymbolProcessor(
     private val environment: SymbolProcessorEnvironment,
@@ -79,6 +77,7 @@ class ConfigSymbolProcessor(
     inner class ConfigVisitor : KSEmptyVisitor<Resolver, Unit>() {
         override fun visitClassDeclaration(classDeclaration: KSClassDeclaration, data: Resolver) {
             processedClasses += classDeclaration
+            if (!classDeclaration.modifiers.contains(Modifier.PUBLIC)) return
             when (classDeclaration.classKind) {
                 ClassKind.INTERFACE, ClassKind.ENUM_CLASS, ClassKind.ENUM_ENTRY, ClassKind.ANNOTATION_CLASS -> return
                 ClassKind.OBJECT -> {
@@ -112,6 +111,7 @@ class ConfigSymbolProcessor(
             if (closestClass?.classKind != ClassKind.OBJECT && !property.modifiers.contains(Modifier.JAVA_STATIC)) return
             val parentClass =
                 if (closestClass?.isCompanionObject == true) closestClass.parentDeclaration as KSClassDeclaration else closestClass
+            if (!property.isPublic()) return
             val name = property.qualifiedName ?: return
             val parentClassName = parentClass?.simpleName?.asString()?.ifEmpty { null }
             val parentClassAltName = parentClass?.let { getAltName(it)?.ifEmpty { null } }
@@ -133,7 +133,6 @@ class ConfigSymbolProcessor(
     }
 }
 
-@AutoService(SymbolProcessorProvider::class)
 class ConfigSymbolProcessorProvider : SymbolProcessorProvider {
     override fun create(environment: SymbolProcessorEnvironment): SymbolProcessor {
         return ConfigSymbolProcessor(environment)

command/annotation/src/main/resources/META-INF/services/com.google.devtools.ksp.processing.SymbolProcessorProvider

@@ -0,0 +1 @@
+com.amarcolini.joos.dashboard.ConfigSymbolProcessorProvider

command/annotation/src/main/resources/META-INF/services/javax.annotation.processing.Processor

@@ -0,0 +1 @@
+com.amarcolini.joos.dashboard.ConfigProcessor

command/build.gradle

@@ -54,7 +54,7 @@ android {
         test.java.srcDirs += 'src/test/kotlin'
     }
     defaultConfig {
-        minSdkVersion 23
+        minSdkVersion 24
     }
 }
 

command/src/main/kotlin/com/amarcolini/joos/command/AbstractComponent.kt

@@ -4,23 +4,12 @@ package com.amarcolini.joos.command
  * An abstract version of [Component] with more quality of life features.
  */
 abstract class AbstractComponent : Component {
-    /**
-     * The scheduler currently using this component.
-     */
-    var scheduler: CommandScheduler? = null
-        internal set(value) {
-            subcomponents.forEach {
-                if (it is AbstractComponent) it.scheduler = value
-            }
-            field = value
-        }
-
     /**
      * A list of subcomponents that this component may use. All subcomponents are
      * updated as well, as long as there is a call to `super.update()`.
      */
     @JvmField
-    protected val subcomponents: MutableList<Component> = ArrayList()
+    protected val subcomponents: MutableSet<Component> = HashSet()
 
     override fun update() {
         subcomponents.forEach { it.update() }