chore(*): code cleanup & output formatting

Author: devnote-dev

src/commands/build.cr

@@ -14,7 +14,7 @@ module Geode::Commands
 
       add_usage "build [-p|--pipe] [targets...]"
       add_argument "targets", description: "the targets to build", multiple: true
-      add_option 'p', "pipe", description: "pipes the build output"
+      add_option 'p', "pipe", description: "pipe the build output"
     end
 
     def run(arguments : Cling::Arguments, options : Cling::Options) : Nil

src/commands/format.cr

@@ -2,10 +2,11 @@ module Geode::Commands
   protected def self.format_command(command : Cling::Command) : String
     String.build do |str|
       if header = command.header
-        str << header << "\n\n"
+        str << header << '\n'
       else
         str << "Command".colorize.magenta << " • " << command.name << '\n'
       end
+      str << '\n'
 
       if description = command.description
         str << description
@@ -14,17 +15,16 @@ module Geode::Commands
 
       unless command.usage.empty?
         str << "Usage".colorize.magenta << '\n'
-        command.usage.each do |use|
-          str << "»  " << use << '\n'
-        end
+        command.usage.each { |use| str << "»  " << use << '\n' }
         str << '\n'
       end
 
       unless command.children.empty?
         str << "Commands".colorize.magenta << '\n'
         max_size = command.children.keys.map(&.size).max + 4
+
         command.children.each do |name, cmd|
-          str << "»  #{name}"
+          str << "»  " << name
           if summary = cmd.summary
             str << " " * (max_size - name.size)
             str << summary
@@ -36,16 +36,21 @@ module Geode::Commands
 
       unless command.arguments.empty?
         str << "Arguments".colorize.magenta << '\n'
+        max_size = 4 + command.arguments.keys.max_of &.size
+
         command.arguments.each do |name, argument|
-          str << "»  #{name}\t#{argument.description}"
+          str << "»  " << name
+          str << " " * (max_size - name.size)
+          str << argument.description
           str << " (required)" if argument.required?
           str << '\n'
         end
         str << '\n'
       end
 
       str << "Options".colorize.magenta << '\n'
-      max_size = command.options.map { |n, o| n.size + (o.short ? 2 : 0) + 2 }.max + 2
+      max_size = 2 + command.options.max_of { |n, o| 2 + n.size + (o.short ? 2 : 0) }
+
       command.options.each do |name, option|
         name_size = 2 + option.long.size + (option.short ? 2 : -2)
 

src/commands/list.cr

@@ -8,8 +8,6 @@ module Geode::Commands
         transitive dependencies will also be listed even if they are not directly
         required by your project.
         DESC
-
-      add_usage "list"
     end
 
     def run(arguments : Cling::Arguments, options : Cling::Options) : Nil

src/commands/version.cr

@@ -4,8 +4,6 @@ module Geode::Commands
       @name = "version"
       @summary = "gets the version information about Geode"
       @description = "Gets the version information about Geode."
-
-      add_usage "geode version"
     end
 
     def run(arguments : Cling::Arguments, options : Cling::Options) : Nil

src/commands/watch.cr

@@ -8,7 +8,7 @@ module Geode::Commands
       add_argument "target", description: "the name of the target"
       add_option 'c', "check-start", description: "check for the target executable at the start"
       add_option 'i', "interval", description: "the wait interval in seconds", type: :single, default: 0.5
-      add_option 'p', "pipe", description: "pipes the build output"
+      add_option 'p', "pipe", description: "pipe the build output"
       add_option 's', "skip-start", description: "skip building at the start"
     end
 

src/shard.cr

@@ -19,7 +19,6 @@ module Geode
 
   class Shard
     include YAML::Serializable
-    include YAML::Serializable::Unmapped
 
     NAME_REGEX = /\A[a-z][a-z0-9_-]+\z/