shell: fixed -h,--help flag (#976)

src/cli/shell.rs

@@ -13,7 +13,7 @@ use crate::toolset::{ToolSource, ToolsetBuilder};
 ///
 /// Only works in a session where rtx is already activated.
 #[derive(Debug, clap::Args)]
-#[clap(verbatim_doc_comment, after_long_help = AFTER_LONG_HELP)]
+#[clap(verbatim_doc_comment, alias = "s", after_long_help = AFTER_LONG_HELP)]
 pub struct Shell {
     /// Tool(s) to use
     #[clap(value_name="TOOL@VERSION", value_parser = ToolArgParser)]

src/shell/bash.rs

@@ -28,13 +28,15 @@ impl Shell for Bash {
               shift
 
               case "$command" in
-              deactivate|shell)
-                eval "$(command rtx "$command" "$@")"
-                ;;
-              *)
-                command rtx "$command" "$@"
+              deactivate|s|shell)
+                # if argv doesn't contains -h,--help
+                if [[ ! " $@ " =~ " --help " ]] && [[ ! " $@ " =~ " -h " ]]; then
+                  eval "$(command rtx "$command" "$@")"
+                  return $?
+                fi
                 ;;
               esac
+              command rtx "$command" "$@"
             }}
 
             _rtx_hook() {{

src/shell/fish.rs

@@ -32,9 +32,21 @@ impl Shell for Fish {
               set command $argv[1]
               set -e argv[1]
 
+              if contains -- --help $argv
+                command rtx "$command" $argv
+                return $status
+              end
+
               switch "$command"
-              case deactivate shell
-                source (command rtx "$command" $argv|psub)
+              case deactivate s shell
+                # if help is requested, don't eval
+                if contains -- -h $argv
+                  command rtx "$command" $argv
+                else if contains -- --help $argv
+                  command rtx "$command" $argv
+                else
+                  source (command rtx "$command" $argv |psub)
+                end
               case '*'
                 command rtx "$command" $argv
               end

src/shell/snapshots/rtx__shell__bash__tests__hook_init.snap

@@ -15,13 +15,15 @@ rtx() {
   shift
 
   case "$command" in
-  deactivate|shell)
-    eval "$(command rtx "$command" "$@")"
-    ;;
-  *)
-    command rtx "$command" "$@"
+  deactivate|s|shell)
+    # if argv doesn't contains -h,--help
+    if [[ ! " $@ " =~ " --help " ]] && [[ ! " $@ " =~ " -h " ]]; then
+      eval "$(command rtx "$command" "$@")"
+      return $?
+    fi
     ;;
   esac
+  command rtx "$command" "$@"
 }
 
 _rtx_hook() {

src/shell/snapshots/rtx__shell__bash__tests__hook_init_nix.snap

@@ -14,13 +14,15 @@ rtx() {
   shift
 
   case "$command" in
-  deactivate|shell)
-    eval "$(command rtx "$command" "$@")"
-    ;;
-  *)
-    command rtx "$command" "$@"
+  deactivate|s|shell)
+    # if argv doesn't contains -h,--help
+    if [[ ! " $@ " =~ " --help " ]] && [[ ! " $@ " =~ " -h " ]]; then
+      eval "$(command rtx "$command" "$@")"
+      return $?
+    fi
     ;;
   esac
+  command rtx "$command" "$@"
 }
 
 _rtx_hook() {

src/shell/snapshots/rtx__shell__fish__tests__hook_init.snap

@@ -14,9 +14,21 @@ function rtx
   set command $argv[1]
   set -e argv[1]
 
+  if contains -- --help $argv
+    command rtx "$command" $argv
+    return $status
+  end
+
   switch "$command"
-  case deactivate shell
-    source (command rtx "$command" $argv|psub)
+  case deactivate s shell
+    # if help is requested, don't eval
+    if contains -- -h $argv
+      command rtx "$command" $argv
+    else if contains -- --help $argv
+      command rtx "$command" $argv
+    else
+      source (command rtx "$command" $argv |psub)
+    end
   case '*'
     command rtx "$command" $argv
   end

src/shell/snapshots/rtx__shell__fish__tests__hook_init_nix.snap

@@ -13,9 +13,21 @@ function rtx
   set command $argv[1]
   set -e argv[1]
 
+  if contains -- --help $argv
+    command rtx "$command" $argv
+    return $status
+  end
+
   switch "$command"
-  case deactivate shell
-    source (command rtx "$command" $argv|psub)
+  case deactivate s shell
+    # if help is requested, don't eval
+    if contains -- -h $argv
+      command rtx "$command" $argv
+    else if contains -- --help $argv
+      command rtx "$command" $argv
+    else
+      source (command rtx "$command" $argv |psub)
+    end
   case '*'
     command rtx "$command" $argv
   end

src/shell/snapshots/rtx__shell__zsh__tests__hook_init.snap

@@ -15,13 +15,15 @@ rtx() {
   shift
 
   case "$command" in
-  deactivate|shell)
-    eval "$(command rtx "$command" "$@")"
-    ;;
-  *)
-    command rtx "$command" "$@"
+  deactivate|s|shell)
+    # if argv doesn't contains -h,--help
+    if [[ ! " $@ " =~ " --help " ]] && [[ ! " $@ " =~ " -h " ]]; then
+      eval "$(command rtx "$command" "$@")"
+      return $?
+    fi
     ;;
   esac
+  command rtx "$command" "$@"
 }
 
 _rtx_hook() {

src/shell/snapshots/rtx__shell__zsh__tests__hook_init_nix.snap

@@ -14,13 +14,15 @@ rtx() {
   shift
 
   case "$command" in
-  deactivate|shell)
-    eval "$(command rtx "$command" "$@")"
-    ;;
-  *)
-    command rtx "$command" "$@"
+  deactivate|s|shell)
+    # if argv doesn't contains -h,--help
+    if [[ ! " $@ " =~ " --help " ]] && [[ ! " $@ " =~ " -h " ]]; then
+      eval "$(command rtx "$command" "$@")"
+      return $?
+    fi
     ;;
   esac
+  command rtx "$command" "$@"
 }
 
 _rtx_hook() {

src/shell/zsh.rs

@@ -32,13 +32,15 @@ impl Shell for Zsh {
               shift
 
               case "$command" in
-              deactivate|shell)
-                eval "$(command rtx "$command" "$@")"
-                ;;
-              *)
-                command rtx "$command" "$@"
+              deactivate|s|shell)
+                # if argv doesn't contains -h,--help
+                if [[ ! " $@ " =~ " --help " ]] && [[ ! " $@ " =~ " -h " ]]; then
+                  eval "$(command rtx "$command" "$@")"
+                  return $?
+                fi
                 ;;
               esac
+              command rtx "$command" "$@"
             }}
 
             _rtx_hook() {{