Extract CLI Java components into new submodule

* Create new sub-module asciidoctorj-cli: migrated 'org.asciidoctor.jruby.cli'
classes and related tests to it.
* Created CliOptions in AsciidoctorUtils from values in AsciidoctorCliOptions
to avoid circular dependency.
* Updated README with new sub-module information.
* Fix jar names in command in README.
* Minor refactors and fixes.

Fixes asciidoctor#246

CHANGELOG.adoc

@@ -26,6 +26,7 @@ Improvement::
 * Expose `sectnum` property in Section interface (#1121)
 * Replace use of deprecated 'numbered' attribute by 'sectnums' (#1123) (@abelsromero)
 * Expose `source` and `source_lines` use of deprecated 'numbered' in Document interface (#1145) (@abelsromero)
+* Create `asciidoctorj-cli` module to prevent unnecessary dependencies to asciidoctorj jar consumers (#1149)
 
 Bug Fixes::
 

README.adoc

@@ -284,8 +284,13 @@ asciidoctorj::
   Also bundles optional RubyGems needed at runtime, such as coderay, tilt, haml and slim.
   Produces the asciidoctorj jar.
 
+asciidoctorj-cli::
+  `asciidoctorj` command Java components.
+  Produces the asciidoctorj-cli.
+
 asciidoctorj-distribution::
-  Produces the distribution zip that provides the standalone `asciidoctorj` command.
+  Produces the distribution zip that provides the standalone `asciidoctorj` command including Java components,
+  launch scripts and required libraries.
 
 asciidoctorj-arquillian-extension::
   Bundles an Arquillian extension that allows to inject an Asciidoctor instance or other instances commonly used by Asciidoctor tests into a test case.
@@ -310,6 +315,8 @@ asciidoctorj-arquillian-extension/
   build.gradle
 asciidoctorj-api/
   build.gradle
+asciidoctorj-cli/
+  build.gradle
 asciidoctorj-core/
   build.gradle
 asciidoctorj-distribution/

asciidoctorj-cli/build.gradle

@@ -0,0 +1,25 @@
+apply plugin: 'java-library'
+apply plugin: 'biz.aQute.bnd.builder'
+
+dependencies {
+  api project(':asciidoctorj-api')
+  api project(':asciidoctorj')
+  implementation "com.beust:jcommander:$jcommanderVersion"
+
+  testImplementation project(':asciidoctorj-arquillian-extension')
+  testImplementation "org.jsoup:jsoup:$jsoupVersion"
+}
+
+project.ext.publicationName = "mavenAsciidoctorJ"
+
+apply from: rootProject.file('gradle/publish.gradle')
+apply from: rootProject.file('gradle/signing.gradle')
+
+jar {
+  bnd(
+    ('Bundle-Name'): 'asciidoctorj-cli',
+    ('Bundle-SymbolicName'): 'org.asciidoctor.asciidoctorj-cli',
+    ('Import-Package'): 'com.beust.jcommander;resolution:=optional, *'
+  )
+  metaInf { from "$buildDir/version-info/" }
+}

asciidoctorj-cli/gradle.properties

@@ -0,0 +1,2 @@
+properName=AsciidoctorJ CLI
+description=AsciidoctorJ Command Line Interface

asciidoctorj-core/src/main/java/org/asciidoctor/jruby/cli/AsciidoctorCliOptions.java → asciidoctorj-cli/src/main/java/org/asciidoctor/jruby/cli/AsciidoctorCliOptions.java

@@ -116,7 +116,9 @@ public boolean isQuiet() {
         return quiet;
     }
 
-    public Severity getFailureLevel() { return failureLevel; }
+    public Severity getFailureLevel() {
+        return failureLevel;
+    }
 
     public boolean isRequire() {
         return this.require != null && this.require.size() > 0;

asciidoctorj-core/src/main/java/org/asciidoctor/jruby/cli/DocTypeEnum.java → asciidoctorj-cli/src/main/java/org/asciidoctor/jruby/cli/DocTypeEnum.java

@@ -2,6 +2,6 @@
 
 public enum DocTypeEnum {
 
-    article, book, inline;
+    article, book, inline
 
 }

asciidoctorj-core/src/main/java/org/asciidoctor/jruby/cli/MaxSeverityLogHandler.java → asciidoctorj-cli/src/main/java/org/asciidoctor/jruby/cli/MaxSeverityLogHandler.java

@@ -5,6 +5,7 @@
 import org.asciidoctor.log.Severity;
 
 public class MaxSeverityLogHandler implements LogHandler {
+
     private Severity maxSeverity = Severity.DEBUG;
 
     @Override

asciidoctorj-core/src/main/java/org/asciidoctor/jruby/cli/SafeModeConverter.java → asciidoctorj-cli/src/main/java/org/asciidoctor/jruby/cli/SafeModeConverter.java

@@ -1,8 +1,7 @@
 package org.asciidoctor.jruby.cli;
 
-import org.asciidoctor.SafeMode;
-
 import com.beust.jcommander.IStringConverter;
+import org.asciidoctor.SafeMode;
 
 public class SafeModeConverter implements IStringConverter<SafeMode> {
 

asciidoctorj-core/src/test/java/org/asciidoctor/jruby/cli/WhenAsciidoctorAPICallIsCalled.java → asciidoctorj-cli/src/test/java/org/asciidoctor/jruby/cli/WhenAsciidoctorAPICallIsCalled.java

@@ -1,7 +1,10 @@
 package org.asciidoctor.jruby.cli;
 
 import com.beust.jcommander.JCommander;
-import org.asciidoctor.*;
+import org.asciidoctor.Attributes;
+import org.asciidoctor.Options;
+import org.asciidoctor.OptionsBuilder;
+import org.asciidoctor.SafeMode;
 import org.asciidoctor.jruby.internal.AsciidoctorUtils;
 import org.junit.Test;
 
@@ -50,7 +53,6 @@ public void api_parameters_should_be_transformed_to_cli_command() {
         assertThat(asciidoctorCliOptions.getAttributes(), hasEntry("myAtribute", "myValue"));
         assertThat(asciidoctorCliOptions.getAttributes(), hasKey("sectnums"));
         assertThat(asciidoctorCliOptions.getAttributes(), hasKey("copycss!"));
-
     }
 
 }