From 807b697beaec8d8dc1f8844eabd73eed180bb1a4 Mon Sep 17 00:00:00 2001 From: Amadeus Folego Date: Sat, 27 Dec 2025 10:10:51 -0300 Subject: [PATCH 1/3] fix(expert): print error when no transport argument provided --- apps/expert/lib/expert/application.ex | 5 +++++ 1 file changed, 5 insertions(+) diff --git a/apps/expert/lib/expert/application.ex b/apps/expert/lib/expert/application.ex index fd41bf96..99227de0 100644 --- a/apps/expert/lib/expert/application.ex +++ b/apps/expert/lib/expert/application.ex @@ -59,6 +59,11 @@ defmodule Expert.Application do [communication: {GenLSP.Communication.TCP, [port: opts[:port]]}] true -> + IO.puts( + :stderr, + "ERROR: A transport argument (--stdio|--port PORT) must be provided, expert won't initialize." + ) + IO.puts(help_text) System.halt(1) From 3e92a650b16707e4f560f60c0a13b5c324648ccf Mon Sep 17 00:00:00 2001 From: Amadeus Folego Date: Sat, 27 Dec 2025 10:21:41 -0300 Subject: [PATCH 2/3] use status code 2 to indicate invalid cli argument --- apps/expert/lib/expert/application.ex | 5 +++-- 1 file changed, 3 insertions(+), 2 deletions(-) diff --git a/apps/expert/lib/expert/application.ex b/apps/expert/lib/expert/application.ex index 99227de0..df537b33 100644 --- a/apps/expert/lib/expert/application.ex +++ b/apps/expert/lib/expert/application.ex @@ -61,12 +61,13 @@ defmodule Expert.Application do true -> IO.puts( :stderr, - "ERROR: A transport argument (--stdio|--port PORT) must be provided, expert won't initialize." + "ERROR: A transport argument (--stdio|--port ) must be provided, expert won't initialize." ) IO.puts(help_text) - System.halt(1) + # Status code 2 is often used for invalid CLI argument + System.halt(2) end ensure_epmd_module!() From d550fed9d161dadfbff393dda6862b971a71c76f Mon Sep 17 00:00:00 2001 From: Amadeus Folego Date: Sat, 27 Dec 2025 10:40:56 -0300 Subject: [PATCH 3/3] fatal instead of error --- apps/expert/lib/expert/application.ex | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/apps/expert/lib/expert/application.ex b/apps/expert/lib/expert/application.ex index df537b33..b1e0d711 100644 --- a/apps/expert/lib/expert/application.ex +++ b/apps/expert/lib/expert/application.ex @@ -61,7 +61,7 @@ defmodule Expert.Application do true -> IO.puts( :stderr, - "ERROR: A transport argument (--stdio|--port ) must be provided, expert won't initialize." + "FATAL: A transport argument (--stdio|--port ) must be provided, expert won't initialize." ) IO.puts(help_text)