recognize if tty

Author: fentas

.bin/argsh

@@ -17,7 +17,7 @@ source "${PATH_BASE}/libraries/docker.sh"
 ###
 test::docs() {
   :args "Run tests for the documentation" "${@}"
-  docker run --rm -it $(docker::user) node:21-alpine sh -c "
+  docker run --rm $(docker::user) node:21-alpine sh -c "
     cd /workspace/www
     [ -d node_modules ] || yarn --silent --frozen-lockfile
     yarn build
@@ -26,7 +26,7 @@ test::docs() {
 
 lint::docs() {
   :args "Run linters for the documentation" "${@}"
-  docker run --rm -it $(docker::user) node:21-alpine sh -c "
+  docker run --rm $(docker::user) node:21-alpine sh -c "
     cd /workspace/www
     [ -d node_modules ] || yarn --silent --frozen-lockfile
     yarn lint:content
@@ -38,7 +38,7 @@ lint::vale() {
   :args "Run vale for the documentation" "${@}"
   local alert_level="${vale?"need to specify alert level"}"
   binary::exists docker || exit 1
-  docker run --rm -it $(docker::user) jdkato/vale:latest sh -c "
+  docker run --rm $(docker::user) jdkato/vale:latest sh -c "
     cd /workspace/www/vale
     ./run-vale.sh docs content \"${alert_level}\"
   "
@@ -101,8 +101,10 @@ argsh::docker() {
 ### main
 ###
 argsh::main() {
+  local tty=""
+  [[ -t 1 ]] && tty="-it"
   # shellcheck disable=SC2046
-  docker run --rm -it $(docker::user) \
+  docker run --rm ${tty} $(docker::user) \
     -e "BATS_LOAD" \
     -e "ARGSH_SOURCE" \
     -e "GIT_COMMIT_SHA=$(git rev-parse HEAD || :)" \

libraries/docker.sh

@@ -28,9 +28,9 @@ docker::user() {
 
   echo "${user}:x:${uid}:${gid}::${home}:${shell}" > /tmp/docker_passwd
   echo "${user}:x:${gid}:" > /tmp/docker_group
-  
-  echo -v "${PATH_BASE:-.}:${home}"
-  echo -v /tmp/docker_passwd:/etc/passwd -v /tmp/docker_group:/etc/group
-  echo -u "${uid}:${gid}"
-  echo -w "${home}"
+  echo "-v /tmp/docker_passwd:/etc/passwd -v /tmp/docker_group:/etc/group"
+  echo "-u ${uid}:${gid}"
+
+  echo "-v ${PATH_BASE:-.}:${home}"
+  echo "-w ${home}"
 }

libraries/fmt.sh

@@ -14,10 +14,10 @@ set -euo pipefail
 #   cat file.txt | fmt::tty
 fmt::tty() {
   local str="${1:-"$(cat)"}"
-  command -v fmt &>/dev/null || {
+  if ! command -v fmt &>/dev/null || [[ ! -t 1 ]]; then
     echo "${str}"
     return 0
-  }
+  fi
 
   local cols
   cols="$(tput cols)"

libraries/main.sh

@@ -27,8 +27,10 @@ argsh::shebang() {
       echo "This script requires Docker to be installed"
       return 1
     } >&2
+    local tty=""
+    [[ -t 1 ]] && tty="-it"
     # shellcheck disable=SC2046
-    docker run --rm -it $(docker::user) \
+    docker run --rm ${tty} $(docker::user) \
       -e "BATS_LOAD" \
       -e "ARGSH_SOURCE" \
       -e "GIT_COMMIT_SHA=$(git rev-parse HEAD || :)" \