Standardize command loading and registration

Author: bougyman

Gemfile.lock

@@ -1,16 +1,16 @@
 PATH
   remote: .
   specs:
-    linear-cli (0.3.9)
+    linear-cli (0.3.10)
       base64 (~> 0.2)
       dry-cli (~> 1.0)
       dry-cli-completion (~> 1.0)
       gqli (~> 1.2)
       httpx (~> 1.2)
       semantic_logger (~> 4.0)
-      sequel
-      sqlite3
-      tty-markdown
+      sequel (~> 5.0)
+      sqlite3 (~> 1.7)
+      tty-markdown (~> 0.7)
 
 GEM
   remote: https://rubygems.org/

lib/linear/commands/issue.rb

@@ -4,18 +4,20 @@ module Rubyists
   module Linear
     # The Cli module is defined in cli.rb and is the top-level namespace for all CLI commands
     module CLI
-      # This ALIASES hash will return the key as the value if the key is not found,
-      # otherwise it will return the value of the existing key
-      ALIASES = Hash.new { |h, k| h[k] = k }.merge(
-        'list' => 'ls'
-      )
+      module Issue
+        # This ALIASES hash will return the key as the value if the key is not found,
+        # otherwise it will return the value of the existing key
+        ALIASES = Hash.new { |h, k| h[k] = k }.merge(
+          'list' => 'ls'
+        )
+      end
 
       Pathname.new(__FILE__).dirname.join('issue').glob('*.rb').each do |file|
         require file.expand_path
         register 'issue', aliases: %w[i] do |issue|
           basename = File.basename(file, '.rb')
           # The filename is expected to define a class of the same name, but capitalized
-          issue.register ALIASES[basename], Issue.const_get(basename.capitalize)
+          issue.register Issue::ALIASES[basename], Issue.const_get(basename.capitalize)
         end
       end
     end

lib/linear/commands/team.rb

@@ -1,5 +1,25 @@
 # frozen_string_literal: true
 
-Pathname.new(__FILE__).dirname.join('team').glob('*.rb').each do |file|
-  require file.expand_path
+module Rubyists
+  module Linear
+    # The Cli module is defined in cli.rb and is the top-level namespace for all CLI commands
+    module CLI
+      module Team
+        # This ALIASES hash will return the key as the value if the key is not found,
+        # otherwise it will return the value of the existing key
+        ALIASES = Hash.new { |h, k| h[k] = k }.merge(
+          'list' => 'ls'
+        )
+      end
+
+      Pathname.new(__FILE__).dirname.join('team').glob('*.rb').each do |file|
+        require file.expand_path
+        register 'team', aliases: %w[t teams] do |team|
+          basename = File.basename(file, '.rb')
+          # The filename is expected to define a class of the same name, but capitalized
+          team.register Team::ALIASES[basename], Team.const_get(basename.capitalize)
+        end
+      end
+    end
+  end
 end

lib/linear/commands/team/list.rb

@@ -31,10 +31,6 @@ def teams_for(options)
           prepend Rubyists::Linear::CLI::Caller
         end
       end
-
-      register 'team', aliases: %w[t] do |team|
-        team.register 'ls', Team::List
-      end
     end
   end
 end