[GR-49359] Use java.io.Console#isTerminal() on 22+ for isatty() checks

Author: eregon

src/launcher/java/org/truffleruby/launcher/RubyLauncher.java

@@ -29,6 +29,7 @@
 import org.graalvm.polyglot.PolyglotException;
 import org.graalvm.polyglot.Source;
 import org.graalvm.polyglot.Value;
+import org.truffleruby.shared.IsTTYHelper;
 import org.truffleruby.shared.Metrics;
 import org.truffleruby.shared.TruffleRuby;
 import org.truffleruby.shared.options.OptionsCatalog;
@@ -230,7 +231,7 @@ protected boolean parseCommonOption(String defaultOptionPrefix, Map<String, Stri
     @Override
     protected boolean runLauncherAction() {
         String pager;
-        if (helpOptionUsed && System.console() != null && !(pager = getPagerFromEnv()).isEmpty()) {
+        if (helpOptionUsed && IsTTYHelper.isTTY() && !(pager = getPagerFromEnv()).isEmpty()) {
             try {
                 Process process = new ProcessBuilder(pager.split(" "))
                         .redirectOutput(Redirect.INHERIT) // set the output of the pager to the terminal and not a pipe
@@ -264,7 +265,7 @@ private int runRubyMain(Context.Builder contextBuilder, CommandLineOptions confi
                     break;
                 case IRB:
                     config.executionAction = ExecutionAction.PATH;
-                    if (System.console() != null) {
+                    if (IsTTYHelper.isTTY()) {
                         getError().println(
                                 "[ruby] WARNING: truffleruby starts IRB when stdin is a TTY instead of reading from stdin, use '-' to read from stdin");
                         config.executionAction = ExecutionAction.PATH;

src/main/java/org/truffleruby/stdlib/readline/ConsoleHolder.java

@@ -62,6 +62,7 @@
 
 import com.oracle.truffle.api.CompilerDirectives;
 import com.oracle.truffle.api.CompilerDirectives.TruffleBoundary;
+import org.truffleruby.shared.IsTTYHelper;
 
 public final class ConsoleHolder {
 
@@ -116,8 +117,7 @@ private ConsoleHolder(
         this.in = new IoStream(context, language, inFd, inIo);
         this.out = new IoStream(context, language, outFd, outIo);
 
-        boolean isTTY = System.console() != null;
-        boolean system = isTTY && inFd == 0 && outFd == 1;
+        boolean system = IsTTYHelper.isTTY() && inFd == 0 && outFd == 1;
 
         final Terminal terminal;
         try {

src/shared/java/org/truffleruby/shared/IsTTYHelper.java

@@ -0,0 +1,44 @@
+/*
+ * Copyright (c) 2023 Oracle and/or its affiliates. All rights reserved. This
+ * code is released under a tri EPL/GPL/LGPL license. You can use it,
+ * redistribute it and/or modify it under the terms of the:
+ *
+ * Eclipse Public License version 2.0, or
+ * GNU General Public License version 2, or
+ * GNU Lesser General Public License version 2.1.
+ */
+package org.truffleruby.shared;
+
+import java.io.Console;
+import java.lang.reflect.InvocationTargetException;
+import java.lang.reflect.Method;
+
+public final class IsTTYHelper {
+
+    private static final Method IS_TERMINAL_METHOD = getIsTerminalMethod();
+
+    private static Method getIsTerminalMethod() {
+        try {
+            return Console.class.getMethod("isTerminal");
+        } catch (NoSuchMethodException e) {
+            return null;
+        }
+    }
+
+    public static boolean isTTY() {
+        Console console = System.console();
+        if (console == null) {
+            return false;
+        }
+        if (IS_TERMINAL_METHOD != null) {
+            try {
+                return (boolean) IS_TERMINAL_METHOD.invoke(console);
+            } catch (IllegalAccessException | InvocationTargetException e) {
+                throw new Error(e);
+            }
+        } else {
+            return true;
+        }
+    }
+
+}