fix(cli): validate URL before loading config

Author: SergoHUH

lib/aireview/cli.rb

@@ -42,8 +42,9 @@ def start
 
     def run_review(argv)
       options, mr_url = review_options_and_url(argv)
+      parser_result = MrParser.parse(mr_url)
       config = load_review_config(options)
-      context = load_review_context(mr_url, config, options)
+      context = load_review_context(parser_result, config, options)
 
       execute_review(config, context, options)
     end
@@ -72,8 +73,7 @@ def load_review_config(options)
       config
     end
 
-    def load_review_context(mr_url, config, options)
-      parser_result = MrParser.parse(mr_url)
+    def load_review_context(parser_result, config, options)
       gitlab_client = build_gitlab_client(config, parser_result)
       merge_request, changes = fetch_merge_request_data(gitlab_client, parser_result)
 

spec/cli_spec.rb

@@ -0,0 +1,23 @@
+# frozen_string_literal: true
+
+require 'stringio'
+require 'aireview'
+
+RSpec.describe Aireview::CLI do
+  describe '.start' do
+    it 'rejects an invalid URL before loading config' do
+      error_output = StringIO.new
+
+      expect(Aireview::Config).not_to receive(:load)
+
+      status = described_class.start(
+        ['review', 'gitlab.company.com/team/project/-/merge_requests/123'],
+        out: StringIO.new,
+        err: error_output
+      )
+
+      expect(status).to eq(1)
+      expect(error_output.string).to include('Merge request URL must include http:// or https://')
+    end
+  end
+end