Move Importmap::Update to ImportmapUpdate (#17)

We don't want to touch the `Importmap` namespace.

Author: neilvcarvalho

exe/importmap-update

@@ -47,8 +47,8 @@ OptionParser.new do |opts|
 end.parse!
 
 begin
-  config = Importmap::Update::Config.load(options[:config_path])
-rescue Importmap::Update::Config::ConfigError => e
+  config = ImportmapUpdate::Config.load(options[:config_path])
+rescue ImportmapUpdate::Config::ConfigError => e
   warn "Config error: #{e.message}"
   exit 2
 end
@@ -66,24 +66,24 @@ if token.nil? || token.empty?
 end
 
 rails_root = ENV.fetch("RAILS_ROOT", ".")
-runner = Importmap::Update::Commands::ShellRunner.new(cwd: rails_root)
-gh = Importmap::Update::GitHubClient.new(repo:, token:)
-git = Importmap::Update::GitClient.new(
+runner = ImportmapUpdate::Commands::ShellRunner.new(cwd: rails_root)
+gh = ImportmapUpdate::GitHubClient.new(repo:, token:)
+git = ImportmapUpdate::GitClient.new(
   repo: Git.open(rails_root),
   author_name: ENV.fetch("IMPORTMAP_AUTHOR_NAME", "github-actions[bot]"),
   author_email: ENV.fetch("IMPORTMAP_AUTHOR_EMAIL", "github-actions[bot]@users.noreply.github.com")
 )
 
-outdated = Importmap::Update::Parsers::OutdatedParser.parse(ENV.fetch("IMPORTMAP_OUTDATED_OUTPUT"))
-vulnerabilities = Importmap::Update::Parsers::AuditParser.parse(ENV.fetch("IMPORTMAP_AUDIT_OUTPUT"))
-plan = Importmap::Update::Planner.new(
+outdated = ImportmapUpdate::Parsers::OutdatedParser.parse(ENV.fetch("IMPORTMAP_OUTDATED_OUTPUT"))
+vulnerabilities = ImportmapUpdate::Parsers::AuditParser.parse(ENV.fetch("IMPORTMAP_AUDIT_OUTPUT"))
+plan = ImportmapUpdate::Planner.new(
   outdated:, vulnerabilities:, config:
 ).call
 
 existing_prs = gh.list_open_prs(branch_prefix: config.branch_prefix)
-reconciled = Importmap::Update::Reconciler.new(plan:, existing_prs:).call
+reconciled = ImportmapUpdate::Reconciler.new(plan:, existing_prs:).call
 
-executor = Importmap::Update::Executor.new(
+executor = ImportmapUpdate::Executor.new(
   gh:, git:, runner:,
   base_branch: ENV.fetch("IMPORTMAP_BASE_BRANCH", "main"),
   commit_message_prefix: config.commit_message.prefix,

lib/commands.rb

@@ -3,61 +3,59 @@
 require "open3"
 require "json"
 
-module Importmap
-  module Update
-    # Abstracts execution of external commands (bin/importmap) so the rest
-    # of the codebase doesn't shell out directly. This is the seam tests hook
-    # into — production code runs commands for real, tests inject a
-    # FixtureRunner that replays pre-recorded (argv → stdout, exit) tuples.
-    #
-    # The interface deliberately mirrors what Open3.capture3 returns:
-    #
-    #   runner.run("bin/importmap", "outdated")
-    #     # => Result(stdout: "...", stderr: "...", success: true, exit: 0)
-    #
-    # Commands are passed as an argv array, not a shell string. That's both
-    # safer (no shell-injection surprises from package names) and easier to
-    # match against fixture keys.
-    module Commands
-      Result = Data.define(:stdout, :stderr, :exit_code) do
-        def success?
-          exit_code == 0
-        end
+module ImportmapUpdate
+  # Abstracts execution of external commands (bin/importmap) so the rest
+  # of the codebase doesn't shell out directly. This is the seam tests hook
+  # into — production code runs commands for real, tests inject a
+  # FixtureRunner that replays pre-recorded (argv → stdout, exit) tuples.
+  #
+  # The interface deliberately mirrors what Open3.capture3 returns:
+  #
+  #   runner.run("bin/importmap", "outdated")
+  #     # => Result(stdout: "...", stderr: "...", success: true, exit: 0)
+  #
+  # Commands are passed as an argv array, not a shell string. That's both
+  # safer (no shell-injection surprises from package names) and easier to
+  # match against fixture keys.
+  module Commands
+    Result = Data.define(:stdout, :stderr, :exit_code) do
+      def success?
+        exit_code == 0
       end
+    end
 
-      class CommandError < StandardError
-        attr_reader :argv, :result
+    class CommandError < StandardError
+      attr_reader :argv, :result
 
-        def initialize(argv, result)
-          @argv = argv
-          @result = result
-          super("`#{argv.join(" ")}` exited #{result.exit_code}: #{result.stderr.strip}")
-        end
+      def initialize(argv, result)
+        @argv = argv
+        @result = result
+        super("`#{argv.join(" ")}` exited #{result.exit_code}: #{result.stderr.strip}")
       end
+    end
 
-      # Production runner: actually executes the command.
-      class ShellRunner
-        # @param cwd [String, nil] working directory (defaults to current)
-        def initialize(cwd: nil)
-          @cwd = cwd
-        end
+    # Production runner: actually executes the command.
+    class ShellRunner
+      # @param cwd [String, nil] working directory (defaults to current)
+      def initialize(cwd: nil)
+        @cwd = cwd
+      end
 
-        def run(*argv)
-          opts = {}
-          opts[:chdir] = @cwd if @cwd
-          Bundler.with_unbundled_env do
-            stdout, stderr, status = Open3.capture3(*argv, opts)
-            Result.new(stdout:, stderr:, exit_code: status.exitstatus)
-          end
+      def run(*argv)
+        opts = {}
+        opts[:chdir] = @cwd if @cwd
+        Bundler.with_unbundled_env do
+          stdout, stderr, status = Open3.capture3(*argv, opts)
+          Result.new(stdout:, stderr:, exit_code: status.exitstatus)
         end
+      end
 
-        # Raises on non-zero exit. Use when you have no recovery strategy
-        # and just want to surface the error to the caller.
-        def run!(*argv)
-          result = run(*argv)
-          raise CommandError.new(argv, result) unless result.success?
-          result
-        end
+      # Raises on non-zero exit. Use when you have no recovery strategy
+      # and just want to surface the error to the caller.
+      def run!(*argv)
+        result = run(*argv)
+        raise CommandError.new(argv, result) unless result.success?
+        result
       end
     end
   end