wordmove 6.0.0.alpha.5 → 6.0.0.alpha.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1ea52bea7c72c65b2b9a55a425d54bfd586b2c8c71e998517d489f4f55fd1616
-  data.tar.gz: 90430c49a99da9c6ded3519a103066b26c13ee6771d89114ed85b46dad39860f
+  metadata.gz: 1e1645bb92c5ec1ef2f74eb99eb0ad62eb42648af628104c23c6286e3bf1505b
+  data.tar.gz: 5b44c223013a491f2fbca11da41d063597363f8a28b07cf14f556d48b79c4aac
 SHA512:
-  metadata.gz: 95dd8ab8d22c5f8ec4faa15deb9b95792ecce5e2826d43266cbfdebac551e3f1ee6d646a86b2248ae5a16696849dc6e403ccd1a01d3cbadaa629ae0d62148ea4
-  data.tar.gz: e2413aaf068bd8993a1d4701504eddcdd3259b713f1bd1de92e1b10d7523234862c5e398e367277b8ddc80fb4ab00c717575ad6ac50cb6cb01fdfd6f53e4136f
+  metadata.gz: 1039454681f0c345713c3cb3777429197deac2a50f1f2bddf781fe2145e2ba099365a5600edad49272d680ddaa1b8f4be3a82d7a0f9f9929dbf3eb38c28f5999
+  data.tar.gz: 4215958770c983f0d156162e943eaefe06d55c7e4a9cc1d085e7ec87e7130cca7a43daf9881602350b01c65a5f576f38d1c85b12522bbd6964be84beadf5f59b

data/.rubocop.yml

@@ -1,5 +1,5 @@
 AllCops:
-  TargetRubyVersion: 2.7.1
+  TargetRubyVersion: 3.1.0
   DisplayCopNames: true
   DisplayStyleGuide: true
   NewCops: enable

data/.ruby-version

@@ -1 +1 @@
-2.7.1
+3.1.0

data/lib/wordmove/actions/adapt_local_db.rb

@@ -9,7 +9,7 @@ module Wordmove
     class AdaptLocalDb
       extend ::LightService::Action
       include Wordmove::Actions::Helpers
-      include Wordmove::Wpcli
+      include Wordmove::WpcliHelpers
 
       expects :local_options,
               :remote_options,
@@ -38,63 +38,104 @@ module Wordmove
 
         next context if simulate?(cli_options: context.cli_options)
 
-        result = Wordmove::Actions::RunLocalCommand.execute(
-          cli_options: context.cli_options,
-          logger: context.logger,
-          command: mysql_dump_command(
-            env_db_options: context.local_options[:database],
-            save_to_path: context.db_paths.local.path
-          )
-        )
-        context.fail_and_return!(result.message) if result.failure?
+        context.logger.task_step true, dump_command(context)
+        begin
+          system(dump_command(context), exception: true)
+        rescue RuntimeError, SystemExit => e
+          context.fail_and_return!("Local command status reports an error: #{e.message}")
+        end
 
         if context.cli_options[:no_adapt]
           context.logger.warn 'Skipping DB adapt'
         else
-          result = Wordmove::Actions::RunLocalCommand.execute(
-            cli_options: context.cli_options,
-            logger: context.logger,
-            command: wpcli_search_replace_command(context, :vhost)
-          )
-          context.fail_and_return!(result.message) if result.failure?
-
-          result = Wordmove::Actions::RunLocalCommand.execute(
-            cli_options: context.cli_options,
-            logger: context.logger,
-            command: wpcli_search_replace_command(context, :wordpress_path)
-          )
-          context.fail_and_return!(result.message) if result.failure?
-        end
+          %i[vhost wordpress_path].each do |key|
+            command = search_replace_command(context, key)
+            context.logger.task_step true, command
 
-        result = Wordmove::Actions::RunLocalCommand.execute(
-          cli_options: context.cli_options,
+            begin
+              system(command, exception: true)
+            rescue RuntimeError, SystemExit => e
+              context.fail_and_return!("Local command status reports an error: #{e.message}")
+            end
+          end
+        end
 
-          logger: context.logger,
-          command: mysql_dump_command(
-            env_db_options: context.local_options[:database],
-            save_to_path: context.db_paths.local.adapted_path
-          )
-        )
-        context.fail_and_return!(result.message) if result.failure?
+        context.logger.task_step true, dump_adapted_command(context)
+        begin
+          system(dump_adapted_command(context), exception: true)
+        rescue RuntimeError, SystemExit => e
+          context.fail_and_return!("Local command status reports an error: #{e.message}")
+        end
 
         if context.photocopier.is_a? Photocopier::SSH
-          result = Wordmove::Actions::RunLocalCommand.execute(
-            cli_options: context.cli_options,
-            logger: context.logger,
-            command: compress_command(file_path: context.db_paths.local.adapted_path)
-          )
-          context.fail_and_return!(result.message) if result.failure?
+          context.logger.task_step true, compress_command(context)
+          begin
+            system(compress_command(context), exception: true)
+          rescue RuntimeError, SystemExit => e
+            context.fail_and_return!("Local command status reports an error: #{e.message}")
+          end
+        end
+
+        context.logger.task_step true, import_original_db_command(context)
+        begin
+          system(import_original_db_command(context), exception: true)
+        rescue RuntimeError, SystemExit => e
+          context.fail_and_return!("Local command status reports an error: #{e.message}")
+        end
+      end
+
+      def self.dump_command(context)
+        "wp db export #{context.db_paths.local.path} --allow-root --quiet " \
+          "--path=#{wpcli_config_path(context)}"
+      end
+
+      def self.dump_adapted_command(context)
+        "wp db export #{context.db_paths.local.adapted_path} --allow-root --quiet " \
+          "--path=#{wpcli_config_path(context)}"
+      end
+
+      def self.import_original_db_command(context)
+        "wp db import #{context.db_paths.local.path} --allow-root --quiet " \
+          "--path=#{wpcli_config_path(context)}"
+      end
+
+      def self.compress_command(context)
+        command = ['nice']
+        command << '-n'
+        command << '0'
+        command << 'gzip'
+        command << '-9'
+        command << '-f'
+        command << "\"#{context.db_paths.local.adapted_path}\""
+        command.join(' ')
+      end
+
+      # Compose and returns the search-replace command. It's intended to be
+      # used from a +LightService::Action+
+      #
+      # @param context [LightService::Context] The context of an action
+      # @param config_key [:vhost, :wordpress_path] Determines what will be replaced in DB
+      # @return [String]
+      # @!scope class
+      def self.search_replace_command(context, config_key)
+        unless %i[vhost wordpress_path].include?(config_key)
+          raise ArgumentError, "Unexpected `config_key` #{config_key}.:vhost" \
+                               'or :wordpress_path expected'
         end
 
-        result = Wordmove::Actions::RunLocalCommand.execute(
-          cli_options: context.cli_options,
-          logger: context.logger,
-          command: mysql_import_command(
-            dump_path: context.db_paths.local.path,
-            env_db_options: context.local_options[:database]
-          )
-        )
-        context.fail_and_return!(result.message) if result.failure?
+        [
+          'wp search-replace',
+          "--path=#{wpcli_config_path(context)}",
+          '"\A' + context.dig(:local_options, config_key) + '\Z"', # rubocop:disable Style/StringConcatenation
+          '"' + context.dig(:remote_options, config_key) + '"', # rubocop:disable Style/StringConcatenation
+          '--regex-delimiter="|"',
+          '--regex',
+          '--precise',
+          '--quiet',
+          '--skip-columns=guid',
+          '--all-tables',
+          '--allow-root'
+        ].join(' ')
       end
     end
   end

data/lib/wordmove/actions/adapt_remote_db.rb

@@ -14,7 +14,7 @@ module Wordmove
     class AdaptRemoteDb
       extend ::LightService::Action
       include Wordmove::Actions::Helpers
-      include Wordmove::Wpcli
+      include Wordmove::WpcliHelpers
 
       expects :local_options,
               :cli_options,
@@ -39,11 +39,12 @@ module Wordmove
         next context if simulate?(cli_options: context.cli_options)
 
         if File.exist?(context.db_paths.local.gzipped_path)
-          Wordmove::Actions::RunLocalCommand.execute(
-            cli_options: context.cli_options,
-            logger: context.logger,
-            command: uncompress_command(file_path: context.db_paths.local.gzipped_path)
-          )
+          context.logger.task_step true, uncompress_command(context)
+          begin
+            system(uncompress_command(context), exception: true)
+          rescue RuntimeError, SystemExit => e
+            context.fail_and_return!("Local command status reports an error: #{e.message}")
+          end
         end
 
         unless File.exist?(context.db_paths.local.path)
@@ -52,37 +53,76 @@ module Wordmove
           )
         end
 
-        result = Wordmove::Actions::RunLocalCommand.execute(
-          cli_options: context.cli_options,
-          logger: context.logger,
-          command: mysql_import_command(
-            dump_path: context.db_paths.local.path,
-            env_db_options: context.local_options[:database]
-          )
-        )
-        context.fail_and_return!(result.message) if result.failure?
+        context.logger.task_step true, import_db_command(context)
+        begin
+          system(import_db_command(context), exception: true)
+        rescue RuntimeError, SystemExit => e
+          context.fail_and_return!("Local command status reports an error: #{e.message}")
+        end
 
         if context.cli_options[:no_adapt]
           context.logger.warn 'Skipping DB adapt'
           next context
         end
 
-        result = Wordmove::Actions::RunLocalCommand.execute(
-          cli_options: context.cli_options,
-          logger: context.logger,
-          command: wpcli_search_replace_command(context, :vhost)
-        )
-        context.fail_and_return!(result.message) if result.failure?
-
-        result = Wordmove::Actions::RunLocalCommand.execute(
-          cli_options: context.cli_options,
-          logger: context.logger,
-          command: wpcli_search_replace_command(context, :wordpress_path)
-        )
-        context.fail_and_return!(result.message) if result.failure?
+        %i[vhost wordpress_path].each do |key|
+          command = search_replace_command(context, key)
+          context.logger.task_step true, command
+          begin
+            system(command, exception: true)
+          rescue RuntimeError, SystemExit => e
+            context.fail_and_return!("Local command status reports an error: #{e.message}")
+          end
+        end
 
         context.logger.success 'Local DB adapted'
       end
+
+      # Construct the command to deflate a compressed file as a string.
+      #
+      # @param file_path [String] The path where the file to be deflated is located
+      # @return [String] the command
+      # @!scope class
+      def self.uncompress_command(context)
+        command = ['gzip']
+        command << '-d'
+        command << '-f'
+        command << "\"#{context.db_paths.local.gzipped_path}\""
+        command.join(' ')
+      end
+
+      def self.import_db_command(context)
+        "wp db import #{context.db_paths.local.path} --allow-root --quiet " \
+          "--path=#{wpcli_config_path(context)}"
+      end
+
+      # Compose and returns the search-replace command. It's intended to be
+      # used from a +LightService::Action+
+      #
+      # @param context [LightService::Context] The context of an action
+      # @param config_key [:vhost, :wordpress_path] Determines what will be replaced in DB
+      # @return [String]
+      # @!scope class
+      def self.search_replace_command(context, config_key)
+        unless %i[vhost wordpress_path].include?(config_key)
+          raise ArgumentError, "Unexpected `config_key` #{config_key}.:vhost" \
+                               'or :wordpress_path expected'
+        end
+
+        [
+          'wp search-replace',
+          "--path=#{wpcli_config_path(context)}",
+          '"\A' + context.dig(:remote_options, config_key) + '\Z"', # rubocop:disable Style/StringConcatenation
+          '"' + context.dig(:local_options, config_key) + '"', # rubocop:disable Style/StringConcatenation
+          '--regex-delimiter="|"',
+          '--regex',
+          '--precise',
+          '--quiet',
+          '--skip-columns=guid',
+          '--all-tables',
+          '--allow-root'
+        ].join(' ')
+      end
     end
   end
 end

data/lib/wordmove/actions/backup_local_db.rb

@@ -30,27 +30,41 @@ module Wordmove
           next context
         end
 
-        result = Wordmove::Actions::RunLocalCommand.execute(
-          cli_options: context.cli_options,
-          logger: context.logger,
-          command: mysql_dump_command(
-            env_db_options: context.local_options[:database],
-            save_to_path: context.db_paths.backup.local.path
-          )
-        )
-        context.fail_and_return!(result.message) if result.failure?
+        context.logger.task_step true, dump_command(context)
 
-        result = Wordmove::Actions::RunLocalCommand.execute(
-          cli_options: context.cli_options,
-          logger: context.logger,
-          command: compress_command(file_path: context.db_paths.backup.local.path)
-        )
-        context.fail_and_return!(result.message) if result.failure?
+        begin
+          system(dump_command(context), exception: true)
+        rescue RuntimeError, SystemExit => e
+          context.fail_and_return!("Local command status reports an error: #{e.message}")
+        end
+
+        context.logger.task_step true, compress_command(context)
+
+        begin
+          system(compress_command(context), exception: true)
+        rescue RuntimeError, SystemExit => e
+          context.fail_and_return!("Local command status reports an error: #{e.message}")
+        end
 
         context.logger.success(
           "Backup saved at #{context.db_paths.backup.local.gzipped_path}"
         )
       end
+
+      def self.dump_command(context)
+        "wp db export #{context.db_paths.backup.local.path} --allow-root --quiet"
+      end
+
+      def self.compress_command(context)
+        command = ['nice']
+        command << '-n'
+        command << '0'
+        command << 'gzip'
+        command << '-9'
+        command << '-f'
+        command << "\"#{context.db_paths.backup.local.path}\""
+        command.join(' ')
+      end
     end
   end
 end

data/lib/wordmove/actions/setup_context_for_db.rb

@@ -52,12 +52,10 @@ module Wordmove
           remote_options: context.remote_options
         ).url('import.php')
         DbPathsConfig.ftp.local.generated_dump_script_path = generate_dump_script(
-          remote_db_options: context.remote_options[:database],
-          token: token
+          remote_db_options: context.remote_options[:database], token:
         )
         DbPathsConfig.ftp.local.generated_import_script_path = generate_import_script(
-          remote_db_options: context.remote_options[:database],
-          token: token
+          remote_db_options: context.remote_options[:database], token:
         )
         DbPathsConfig.ftp.local.temp_path = local_wp_content_dir(
           local_options: context.local_options

data/lib/wordmove/assets/wordmove_schema_local.yml

@@ -3,6 +3,7 @@ mapping:
   vhost:
     pattern: /^https?:\/\//
   wordpress_path:
+    required: true
   database:
     type: map
     required: true
@@ -15,9 +16,6 @@ mapping:
         required: true
       host:
         required: true
-      mysqldump_options:
-      port:
-      charset:
   paths:
     type: map
     mapping:

data/lib/wordmove/cli.rb

@@ -32,23 +32,16 @@ module Wordmove
           exit 1
         end
 
-        def initial_context(cli_options)
-          cli_options.deep_symbolize_keys!
-          movefile = Wordmove::Movefile.new(cli_options)
-
-          [cli_options, movefile]
-        end
-
-        def movefile_from(**cli_options)
+        def movefile_from(cli_options)
           ensure_wordpress_options_presence!(cli_options)
-          Wordmove::Movefile.new(cli_options)
+          Wordmove::Movefile.new(cli_options, nil, true)
         rescue MovefileNotFound => e
           Logger.new($stdout).error(e.message)
           exit 1
         end
 
-        def call_organizer_with(klass:, movefile:, **cli_options)
-          result = klass.call(cli_options: cli_options, movefile: movefile)
+        def call_organizer_with(klass:, movefile:, cli_options:)
+          result = klass.call(cli_options, movefile)
 
           exit 0 if result.success?
 
@@ -114,22 +107,22 @@ module Wordmove
         private
 
         def call_pull_organizer_with(**cli_options)
-          movefile = movefile_from(**cli_options)
+          movefile = movefile_from(cli_options)
 
           if movefile.options.dig(movefile.environment, :ssh)
             call_organizer_with(
               klass: Wordmove::Organizers::Ssh::Pull,
-              movefile: movefile,
-              **cli_options
+              movefile:, cli_options:
             )
           elsif movefile.options.dig(movefile.environment, :ftp)
             call_organizer_with(
               klass: Wordmove::Organizers::Ftp::Pull,
-              movefile: movefile,
-              **cli_options
+              movefile:, cli_options:
             )
           else
-            raise NoAdapterFound, 'No valid adapter found.'
+            raise NoAdapterFound, 'No valid adapter found. It seems like your movefile.yml lacks ' \
+                                  'an ssh or ftp section for the current environment. ' \
+                                  'Run `wordmove doctor` for more info'
           end
         rescue NoAdapterFound => e
           Logger.new($stdout).error(e.message)
@@ -149,22 +142,22 @@ module Wordmove
         private
 
         def call_push_organizer_with(**cli_options)
-          movefile = movefile_from(**cli_options)
+          movefile = movefile_from(cli_options)
 
           if movefile.options.dig(movefile.environment, :ssh)
             call_organizer_with(
               klass: Wordmove::Organizers::Ssh::Push,
-              movefile: movefile,
-              **cli_options
+              movefile:, cli_options:
             )
           elsif movefile.options.dig(movefile.environment, :ftp)
             call_organizer_with(
               klass: Wordmove::Organizers::Ftp::Push,
-              movefile: movefile,
-              **cli_options
+              movefile:, cli_options:
             )
           else
-            raise NoAdapterFound, 'No valid adapter found.'
+            raise NoAdapterFound, 'No valid adapter found. It seems like your movefile.yml lacks ' \
+                                  'an ssh or ftp section for the current environment. ' \
+                                  'Run `wordmove doctor` for more info'
           end
         rescue NoAdapterFound => e
           Logger.new($stdout).error(e.message)

data/lib/wordmove/doctor/movefile.rb

@@ -5,7 +5,7 @@ module Wordmove
       attr_reader :movefile, :contents, :root_keys
 
       def initialize(cli_options = {}, dir = '.')
-        @movefile = Wordmove::Movefile.new(cli_options, dir)
+        @movefile = Wordmove::Movefile.new(cli_options, dir, false)
 
         begin
           @contents = movefile.options

data/lib/wordmove/doctor/mysql.rb

@@ -7,7 +7,7 @@ module Wordmove
         @logger = Logger.new($stdout).tap { |l| l.level = Logger::INFO }
         begin
           @config = Wordmove::Movefile
-                    .new({ config: movefile_name }, movefile_dir)
+                    .new({ config: movefile_name }, movefile_dir, false)
                     .options[:local][:database]
         rescue Psych::SyntaxError => e
           logger.error e.message

data/lib/wordmove/environments_list.rb

@@ -16,8 +16,8 @@ module Wordmove
     end
 
     def print
-      contents = parse_movefile(movefile: movefile)
-      generate_vhost_list(contents: contents)
+      contents = parse_movefile(movefile:)
+      generate_vhost_list(contents:)
       output
     end
 
@@ -55,7 +55,7 @@ module Wordmove
     #
     def generate_vhost_list(contents:)
       # select object which has 'vhost' only
-      vhosts = select_vhost(contents: contents)
+      vhosts = select_vhost(contents:)
       vhosts.each do |list|
         if list[:env] == :local
           @local_vhost << list

data/lib/wordmove/generators/movefile.rb

@@ -1,16 +1,15 @@
 module Wordmove
   module Generators
     class Movefile
-      include MovefileAdapter
-
       def self.generate
-        new.copy_movefile
+        copy_movefile
       end
 
-      def copy_movefile
+      def self.copy_movefile
+        wordpress_path = File.expand_path(Dir.pwd)
         content = ERB.new(File.read(File.join(__dir__, 'movefile.yml'))).result(binding)
-        files = Dry::Files.new
 
+        files = Dry::Files.new
         files.write('movefile.yml', content)
       end
     end

data/lib/wordmove/generators/movefile.yml

@@ -2,14 +2,15 @@ global:
   sql_adapter: wpcli
 
 local:
-  vhost: http://vhost.local
   wordpress_path: <%= wordpress_path %> # use an absolute path here
 
-  database:
-    name: <%= database.name %>
-    user: <%= database.user %>
-    password: "<%= database.password %>" # could be blank, so always use quotes around
-    host: <%= database.host %>
+  # paths: # you can customize wordpress internal paths
+  #   wp_content: wp-content
+  #   uploads: wp-content/uploads
+  #   plugins: wp-content/plugins
+  #   mu_plugins: wp-content/mu-plugins
+  #   themes: wp-content/themes
+  #   languages: wp-content/languages
 
 production:
   vhost: http://example.com
@@ -40,6 +41,7 @@ production:
     - 'wp-config.php'
     - 'wp-content/*.sql.gz'
     - '*.orig'
+    - 'wp-cli.yml'
 
   # paths: # you can customize wordpress internal paths
   #   wp_content: wp-content

data/lib/wordmove/movefile.rb

@@ -56,8 +56,6 @@ module Wordmove
     private
 
     def fetch(verbose = true) # rubocop:disable Style/OptionalBooleanParameter
-      load_dotenv
-
       entries = if config_file_name.nil?
                   Dir["#{File.join(start_dir, '{M,m}ovefile')}{,.yml,.yaml}"]
                 else
@@ -75,18 +73,40 @@ module Wordmove
       end
 
       found = entries.first
+
       logger.task("Using Movefile: #{found}") if verbose == true
-      YAML.safe_load(ERB.new(File.read(found)).result, [], [], true).deep_symbolize_keys!
+      load_dotenv(verbose)
+
+      options = YAML.safe_load(ERB.new(File.read(found)).result, symbolize_names: true)
+
+      merge_local_options_from_wpcli(options)
+    end
+
+    def merge_local_options_from_wpcli(options)
+      config_path = options.dig(:local, :wordpress_path)
+
+      options.merge(
+        local: {
+          database: {
+            password: Wordmove::WpcliHelpers.get_config('DB_PASSWORD', config_path:),
+            host: Wordmove::WpcliHelpers.get_config('DB_HOST', config_path:),
+            name: Wordmove::WpcliHelpers.get_config('DB_NAME', config_path:),
+            user: Wordmove::WpcliHelpers.get_config('DB_USER', config_path:)
+          },
+          vhost: Wordmove::WpcliHelpers.get_option('siteurl', config_path:),
+          wordpress_path: config_path
+        }
+      )
     end
 
-    def load_dotenv
+    def load_dotenv(verbose)
       env_files = Dir[File.join(start_dir, '.env')]
 
       found_env = env_files.first
 
       return false unless found_env.present?
 
-      logger.info("Using .env file: #{found_env}")
+      logger.info("Using .env file: #{found_env}") if verbose
       Dotenv.load(found_env)
     end
 

data/lib/wordmove/organizers/ftp/pull.rb

@@ -6,21 +6,23 @@ module Wordmove
         include Wordmove::Actions::Helpers
         include Wordmove::Actions::Ftp::Helpers
 
-        def self.call(cli_options:, movefile:)
+        # Can't use keyword arguments since LightService still has some problems with modern
+        # ruby syntax: https://github.com/adomokos/light-service/pull/224
+        def self.call(cli_options, movefile)
           logger = Logger.new($stdout, movefile.secrets).tap { |l| l.level = Logger::DEBUG }
           remote_options = movefile.options[movefile.environment]
-          ftp_opts = ftp_options(remote_options: remote_options)
+          ftp_opts = ftp_options(remote_options:)
 
           LightService::Configuration.logger = ::Logger.new($stdout) if cli_options[:debug]
 
           with(
-            cli_options: cli_options,
+            cli_options:,
             global_options: movefile.options[:global],
             local_options: movefile.options[:local],
-            remote_options: remote_options,
-            movefile: movefile,
-            guardian: Wordmove::Guardian.new(cli_options: cli_options, action: :pull),
-            logger: logger,
+            remote_options:,
+            movefile:,
+            guardian: Wordmove::Guardian.new(cli_options:, action: :pull),
+            logger:,
             photocopier: Photocopier::FTP
                           .new(ftp_opts)
                           .tap { |c| c.logger = logger }

data/lib/wordmove/organizers/ftp/push.rb

@@ -6,21 +6,23 @@ module Wordmove
         include Wordmove::Actions::Helpers
         include Wordmove::Actions::Ftp::Helpers
 
-        def self.call(cli_options:, movefile:)
+        # Can't use keyword arguments since LightService still has some problems with modern
+        # ruby syntax: https://github.com/adomokos/light-service/pull/224
+        def self.call(cli_options, movefile)
           logger = Logger.new($stdout, movefile.secrets).tap { |l| l.level = Logger::DEBUG }
           remote_options = movefile.options[movefile.environment]
-          ftp_opts = ftp_options(remote_options: remote_options)
+          ftp_opts = ftp_options(remote_options:)
 
           LightService::Configuration.logger = ::Logger.new($stdout) if cli_options[:debug]
 
           with(
-            cli_options: cli_options,
+            cli_options:,
             global_options: movefile.options[:global],
             local_options: movefile.options[:local],
-            remote_options: remote_options,
-            movefile: movefile,
-            guardian: Wordmove::Guardian.new(cli_options: cli_options, action: :push),
-            logger: logger,
+            remote_options:,
+            movefile:,
+            guardian: Wordmove::Guardian.new(cli_options:, action: :push),
+            logger:,
             photocopier: Photocopier::FTP
                           .new(ftp_opts)
                           .tap { |c| c.logger = logger }

data/lib/wordmove/organizers/ssh/pull.rb

@@ -6,21 +6,23 @@ module Wordmove
         include Wordmove::Actions::Helpers
         include Wordmove::Actions::Ssh::Helpers
 
-        def self.call(cli_options:, movefile:)
+        # Can't use keyword arguments since LightService still has some problems with modern
+        # ruby syntax: https://github.com/adomokos/light-service/pull/224
+        def self.call(cli_options, movefile)
           logger = Logger.new($stdout, movefile.secrets).tap { |l| l.level = Logger::DEBUG }
           remote_options = movefile.options[movefile.environment]
-          ssh_opts = ssh_options(remote_options: remote_options, simulate: cli_options[:simulate])
+          ssh_opts = ssh_options(remote_options:, simulate: cli_options[:simulate])
 
           LightService::Configuration.logger = ::Logger.new($stdout) if cli_options[:debug]
 
           with(
-            cli_options: cli_options,
+            cli_options:,
             global_options: movefile.options[:global],
             local_options: movefile.options[:local],
-            remote_options: remote_options,
-            movefile: movefile,
-            guardian: Wordmove::Guardian.new(cli_options: cli_options, action: :pull),
-            logger: logger,
+            remote_options:,
+            movefile:,
+            guardian: Wordmove::Guardian.new(cli_options:, action: :pull),
+            logger:,
             photocopier: Photocopier::SSH
                           .new(ssh_opts)
                           .tap { |c| c.logger = logger }

data/lib/wordmove/organizers/ssh/push.rb

@@ -6,21 +6,23 @@ module Wordmove
         include Wordmove::Actions::Helpers
         include Wordmove::Actions::Ssh::Helpers
 
-        def self.call(cli_options:, movefile:)
+        # Can't use keyword arguments since LightService still has some problems with modern
+        # ruby syntax: https://github.com/adomokos/light-service/pull/224
+        def self.call(cli_options, movefile)
           logger = Logger.new($stdout, movefile.secrets).tap { |l| l.level = Logger::DEBUG }
           remote_options = movefile.options[movefile.environment]
-          ssh_opts = ssh_options(remote_options: remote_options, simulate: cli_options[:simulate])
+          ssh_opts = ssh_options(remote_options:, simulate: cli_options[:simulate])
 
           LightService::Configuration.logger = ::Logger.new($stdout) if cli_options[:debug]
 
           with(
-            cli_options: cli_options,
+            cli_options:,
             global_options: movefile.options[:global],
             local_options: movefile.options[:local],
-            remote_options: remote_options,
-            movefile: movefile,
-            guardian: Wordmove::Guardian.new(cli_options: cli_options, action: :push),
-            logger: logger,
+            remote_options:,
+            movefile:,
+            guardian: Wordmove::Guardian.new(cli_options:, action: :push),
+            logger:,
             photocopier: Photocopier::SSH
                           .new(ssh_opts)
                           .tap { |c| c.logger = logger }

data/lib/wordmove/version.rb

@@ -1,3 +1,3 @@
 module Wordmove
-  VERSION = '6.0.0.alpha.5'.freeze
+  VERSION = '6.0.0.alpha.6'.freeze
 end

data/lib/wordmove/wpcli.rb

@@ -1,7 +1,7 @@
 module Wordmove
   # This class is a sort of mini-wrapper around the wp-cli executable.
   # It's responsible to run or produce wp-cli commands.
-  module Wpcli
+  module WpcliHelpers
     extend ActiveSupport::Concern
 
     included do
@@ -17,42 +17,27 @@ module Wordmove
         system('which wp > /dev/null 2>&1')
       end
 
-      # Compose and returns the search-replace command. It's intended to be
-      # used from a +LightService::Action+
-      #
-      # @param context [LightService::Context] The context of an action
-      # @param config_key [:vhost, :wordpress_path] Determines what will be replaced in DB
-      # @return [String]
-      # @!scope class
-      def wpcli_search_replace_command(context, config_key)
-        unless %i[vhost wordpress_path].include?(config_key)
-          raise ArgumentError, "Unexpected `config_key` #{config_key}.:vhost" \
-                               'or :wordpress_path expected'
-        end
-
-        [
-          'wp search-replace',
-          "--path=#{wpcli_config_path(context)}",
-          '"\A' + context.dig(:remote_options, config_key) + '\Z"', # rubocop:disable Style/StringConcatenation
-          '"' + context.dig(:local_options, config_key) + '"', # rubocop:disable Style/StringConcatenation
-          '--regex-delimiter="|"',
-          '--regex',
-          '--precise',
-          '--quiet',
-          '--skip-columns=guid',
-          '--all-tables',
-          '--allow-root'
-        ].join(' ')
-      end
-
       # Returns the wordpress path from wp-cli (with precedence) or from movefile
       #
-      # It's intended to be used from a +LightService::Action+
+      # It's intended to be used from a +LightService::Action+, but it also supports
+      # to receive a path as argument. If the argument is not a LightService::Context
+      # then it will be treated as a path.
+      # The path passed as argument should be the wordpress installation path, but it's
+      # not strictly mandatory: the method will try to load a wpcli's YAML config
+      # from that path, so you can potentially use it with any path
       #
-      # @param context [LightService::Context] The context of an action
+      # @param context [LightService::Context|String] The context of an action or a path as string
       # @return [String]
       # @!scope class
-      def wpcli_config_path(context)
+      def wpcli_config_path(context_or_path)
+        context = if context_or_path.is_a? LightService::Context
+                    context_or_path
+                  else
+                    # We need to make it quack like a duck in order to be
+                    # backward compatible with previous code
+                    { local_options: { wordpress_path: context_or_path } }
+                  end
+
         load_from_yml(context) || load_from_wpcli || context.dig(:local_options, :wordpress_path)
       end
 
@@ -63,7 +48,8 @@ module Wordmove
       # @!scope class
       # @!visibility private
       def load_from_yml(context)
-        yml_path = File.join(context.dig(:local_options, :wordpress_path), 'wp-cli.yml')
+        config_path = context.dig(:local_options, :wordpress_path) || '.'
+        yml_path = File.join(config_path, 'wp-cli.yml')
 
         return unless File.exist?(yml_path)
 
@@ -87,5 +73,13 @@ module Wordmove
         nil
       end
     end
+
+    def self.get_option(option, config_path:)
+      `wp option get #{option} --allow-root --path=#{config_path}`.chomp
+    end
+
+    def self.get_config(config, config_path:)
+      `wp config get #{config} --allow-root --path=#{config_path}`.chomp
+    end
   end
 end

data/lib/wordmove.rb

@@ -17,6 +17,8 @@ require 'yaml'
 
 require 'photocopier'
 
+require 'wordmove/wpcli'
+
 require 'wordmove/cli'
 require 'wordmove/doctor'
 require 'wordmove/doctor/movefile'
@@ -32,9 +34,7 @@ require 'wordmove/movefile'
 require 'wordmove/wordpress_directory'
 require 'wordmove/version'
 require 'wordmove/environments_list'
-require 'wordmove/wpcli'
 
-require 'wordmove/generators/movefile_adapter'
 require 'wordmove/generators/movefile'
 
 require 'wordmove/db_paths_config'
@@ -42,8 +42,8 @@ require 'wordmove/db_paths_config'
 require 'wordmove/actions/helpers'
 require 'wordmove/actions/ssh/helpers'
 require 'wordmove/actions/ftp/helpers'
-Dir[File.join(__dir__, 'wordmove/actions/**/*.rb')].sort.each { |file| require file }
-Dir[File.join(__dir__, 'wordmove/organizers/**/*.rb')].sort.each { |file| require file }
+Dir[File.join(__dir__, 'wordmove/actions/**/*.rb')].each { |file| require file }
+Dir[File.join(__dir__, 'wordmove/organizers/**/*.rb')].each { |file| require file }
 
 module Wordmove
   # Interactors' namespce. Interactors are called "Actions", following the LightService convention.

data/wordmove.gemspec

@@ -36,12 +36,12 @@ Gem::Specification.new do |spec|
   spec.add_runtime_dependency 'dry-configurable', '~> 0.13.0'
   spec.add_runtime_dependency 'kwalify', '~> 0.7.2'
   spec.add_runtime_dependency 'light-service', '~> 0.17.0'
-  spec.add_runtime_dependency 'photocopier', '~> 1.4', '>= 1.4.0'
+  spec.add_runtime_dependency 'photocopier', '~> 1.4', '>= 1.4.1'
   # spec.add_runtime_dependency 'thor', '~> 0.20.3'
   spec.add_runtime_dependency 'dry-cli', '~> 0.7.0'
   spec.add_runtime_dependency 'dry-files', '~> 0.1.0'
 
-  spec.required_ruby_version = '>= 2.6.0' # rubocop:disable Gemspec/RequiredRubyVersion
+  spec.required_ruby_version = '>= 3.1.0'
 
   spec.add_development_dependency 'bundler', '~> 2.3.3'
   spec.add_development_dependency 'pry-byebug', '~> 3.1'

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: wordmove
 version: !ruby/object:Gem::Version
-  version: 6.0.0.alpha.5
+  version: 6.0.0.alpha.6
 platform: ruby
 authors:
 - Stefano Verna
@@ -12,7 +12,7 @@ authors:
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2021-12-30 00:00:00.000000000 Z
+date: 2022-01-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -107,7 +107,7 @@ dependencies:
         version: '1.4'
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.4.0
+        version: 1.4.1
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -117,7 +117,7 @@ dependencies:
         version: '1.4'
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.4.0
+        version: 1.4.1
 - !ruby/object:Gem::Dependency
   name: dry-cli
   requirement: !ruby/object:Gem::Requirement
@@ -375,7 +375,6 @@ files:
 - lib/wordmove/exceptions.rb
 - lib/wordmove/generators/movefile.rb
 - lib/wordmove/generators/movefile.yml
-- lib/wordmove/generators/movefile_adapter.rb
 - lib/wordmove/guardian.rb
 - lib/wordmove/hook.rb
 - lib/wordmove/logger.rb
@@ -410,14 +409,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.6.0
+      version: 3.1.0
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">"
     - !ruby/object:Gem::Version
       version: 1.3.1
 requirements: []
-rubygems_version: 3.1.2
+rubygems_version: 3.3.3
 signing_key:
 specification_version: 4
 summary: Wordmove, Capistrano for Wordpress

data/lib/wordmove/generators/movefile_adapter.rb

@@ -1,89 +0,0 @@
-module Wordmove
-  module Generators
-    module MovefileAdapter
-      def wordpress_path
-        File.expand_path(Dir.pwd)
-      end
-
-      def database
-        DBConfigReader.config
-      end
-    end
-
-    class DBConfigReader
-      def self.config
-        new.config
-      end
-
-      def config
-        Struct.new(
-          :name,
-          :user,
-          :password,
-          :host,
-          keyword_init: true
-        ).new(database_config)
-      end
-
-      def database_config
-        if wp_config_exists?
-          WordpressDBConfig.config
-        else
-          DefaultDBConfig.config
-        end
-      end
-
-      def wp_config_exists?
-        File.exist?(WordpressDirectory.default_path_for(:wp_config))
-      end
-    end
-
-    class DefaultDBConfig
-      def self.config
-        {
-          name: 'database_name',
-          user: 'user',
-          password: 'password',
-          host: '127.0.0.1'
-        }
-      end
-    end
-
-    class WordpressDBConfig
-      def self.config
-        new.config
-      end
-
-      def wp_config
-        @wp_config ||= File.read(
-          WordpressDirectory.default_path_for(:wp_config)
-        ).encode('utf-8', invalid: :replace)
-      end
-
-      def wp_definitions
-        {
-          name: 'DB_NAME',
-          user: 'DB_USER',
-          password: 'DB_PASSWORD',
-          host: 'DB_HOST'
-        }
-      end
-
-      def wp_definition_regex(definition)
-        /['"]#{definition}['"],\s*["'](?<value>.*)['"]/
-      end
-
-      def defaults
-        DefaultDBConfig.config.clone
-      end
-
-      def config
-        wp_definitions.each_with_object(defaults) do |(key, definition), result|
-          wp_config.match(wp_definition_regex(definition)) do |match|
-            result[key] = match[:value]
-          end
-        end
-      end
-    end
-  end
-end