PR feedback: tighten NLU class access

src/main/java/io/spokestack/spokestack/nlu/tensorflow/EncodedTokens.java

@@ -8,7 +8,7 @@
  * A simple data class that represents both the text of tokens produced from a
  * full string and identifiers associated with those tokens.
  */
-public class EncodedTokens {
+final class EncodedTokens {
 
     private final List<String> originalTokens;
     private final List<Integer> ids;
@@ -19,7 +19,7 @@ public class EncodedTokens {
      *
      * @param spaceSeparated The original string split on whitespace.
      */
-    public EncodedTokens(String[] spaceSeparated) {
+    EncodedTokens(String[] spaceSeparated) {
         this.ids = new ArrayList<>();
         this.originalTokens = Arrays.asList(spaceSeparated);
     }

src/main/java/io/spokestack/spokestack/nlu/tensorflow/TFNLUOutput.java

@@ -12,7 +12,7 @@
  * An internal class to contain the business logic for turning raw model outputs
  * into usable intents/slots.
  */
-class TFNLUOutput {
+final class TFNLUOutput {
 
     TFNLUOutput() {
     }

src/main/java/io/spokestack/spokestack/nlu/tensorflow/WordpieceTextEncoder.java

@@ -24,7 +24,7 @@
  * produce the same results on CJK input as other Wordpiece tokenizers.
  * </p>
  */
-public class WordpieceTextEncoder implements TextEncoder {
+final class WordpieceTextEncoder implements TextEncoder {
     private static final String UNKNOWN = "[UNK]";
     private static final String SUFFIX_MARKER = "##";
 
@@ -41,7 +41,7 @@ public class WordpieceTextEncoder implements TextEncoder {
      *                   resource file.
      * @param nluContext Context used to surface loading errors.
      */
-    public WordpieceTextEncoder(SpeechConfig config, NLUContext nluContext) {
+    WordpieceTextEncoder(SpeechConfig config, NLUContext nluContext) {
         this(config, nluContext, Thread::new);
     }
 
@@ -55,7 +55,7 @@ public WordpieceTextEncoder(SpeechConfig config, NLUContext nluContext) {
      * @param threadFactory Thread factory used for creating a resource loading
      *                      thread.
      */
-    public WordpieceTextEncoder(SpeechConfig config, NLUContext nluContext,
+    WordpieceTextEncoder(SpeechConfig config, NLUContext nluContext,
                                 ThreadFactory threadFactory) {
         String vocabFile = config.getString("wordpiece-vocab-path");
         this.context = nluContext;

src/main/java/io/spokestack/spokestack/nlu/tensorflow/parsers/DigitsParser.java

@@ -30,7 +30,7 @@
  * multi-digit requirements (telephone numbers, social security numbers, etc.)
  * </p>
  */
-public class DigitsParser implements SlotParser {
+public final class DigitsParser implements SlotParser {
     private static final Map<String, Integer> ENG_ZERO = new HashMap<>();
     private static final Map<String, Integer> ENG_MOD10 = new HashMap<>();
     private static final Map<String, Integer> ENG_MOD20 = new HashMap<>();

src/main/java/io/spokestack/spokestack/nlu/tensorflow/parsers/IdentityParser.java

@@ -7,7 +7,7 @@
 /**
  * A parser that does not alter string slot values recognized by the model.
  */
-public class IdentityParser implements SlotParser {
+public final class IdentityParser implements SlotParser {
 
     /**
      * Create a new identity parser.

src/main/java/io/spokestack/spokestack/nlu/tensorflow/parsers/IntegerParser.java

@@ -25,7 +25,7 @@
  * than this will result in an exception.
  * </p>
  */
-public class IntegerParser implements SlotParser {
+public final class IntegerParser implements SlotParser {
     private static final Map<String, Integer> WORD_TO_NUM = new HashMap<>();
     private static final Map<String, Integer> MULTIPLIERS = new HashMap<>();
     private static final Pattern DIGIT_SPLIT_RE = Pattern.compile("[-,()\\s]");

src/main/java/io/spokestack/spokestack/nlu/tensorflow/parsers/SelsetParser.java

@@ -8,7 +8,7 @@
 /**
  * A parser that resolves selset values to their canonical names.
  */
-public class SelsetParser implements SlotParser {
+public final class SelsetParser implements SlotParser {
 
     /**
      * Create a new selset parser.