Upgrade to 6.4.1 leads to "error: indent/outdent mismatch: 1"

[dolfinus]

Hi.

I have a script with a function printing a help message using << HEREDOC syntax:

#!/usr/bin/env bash
# Use this script to test if a given URL is available

WAITFORIT_cmdname=${0##*/}

usage()
{
    cat << USAGE >&2
Usage:
    $WAITFORIT_cmdname -u url [-s] [-t timeout] [-- command args]
    -u URL | --url=URL       URL under test
    -s | --strict               Only execute subcommand if the test succeeds
    -q | --quiet                Don't output any status messages
    -t TIMEOUT | --timeout=TIMEOUT
                                Timeout in seconds, zero for no timeout
    -- COMMAND ARGS             Execute command with args after the test finishes
USAGE
    exit 1
}

# rest of script

After upgrading 6.2.1 -> 6.4.1 pre-commit started to show an error:

File ./docker/wait-for-curl.sh: error: indent/outdent mismatch: 1.
beautysh.formatter - ERROR: Indent/outdent mismatch: final tab level = 1

This is because this syntax is valid outside of function

cat << USAGE
   ...
USAGE

And this is valid inside of function (beautysh automatically adds an indent):

usage()
{
   cat << USAGE
   ...
USAGE
}

But this is not valid:

usage()
{
   cat << USAGE
   ...
   USAGE
}

./docker/wait-for-curl.sh: line 19: warning: here-document at line 8 delimited by end-of-file (wanted `EOF')
./docker/wait-for-curl.sh: line 20: syntax error: unexpected end of file from `{' command on line 7

Although it is expected by beautysh 6.4.1

[rdbeni0]

I have the same issue, and receiving error:

File ./script: error: indent/outdent mismatch: -2.
beautysh.formatter - ERROR: Indent/outdent mismatch: final tab level = -2

this file has big

cat <<EOF > /file/somewhere
blabla
blablabla
EOF

blocks and beautysh stopped working

[semseysandor]

I've the same error, but the cause is different. As I see, beautysh v6.3.0+ doesn't like quotes in the cases of a switch case:

$ beautysh - << EXAMPLE
case $1 in
  "foo") bar ;;
esac
EXAMPLE

File (stdin): error: indent/outdent mismatch: -1.
beautysh.formatter - ERROR: Indent/outdent mismatch: final tab level = -1

# Without quotes it's OK

$ beautysh - << EXAMPLE
case $1 in
  foo) bar ;;
esac
EXAMPLE

In the previous example the quotes are not needed, but if we want to match the empty string, it can't be avoided:

$ beautysh - << EXAMPLE
case $1 in
  "") bar ;;
esac
EXAMPLE

File (stdin): error: indent/outdent mismatch: -1.
beautysh.formatter - ERROR: Indent/outdent mismatch: final tab level = -1

# Single quotes also

$ beautysh - << EXAMPLE
case $1 in
  '') bar ;;
esac
EXAMPLE

File (stdin): error: indent/outdent mismatch: -1.
beautysh.formatter - ERROR: Indent/outdent mismatch: final tab level = -1

Note: as a workaround you can wrap problematic blocks with # @formatter:off, # @formatter:on tags.

[dolfinus]

With # @formatter:off beautysh for some reason formats this:

# @formatter:off
usage()
{
    cat << USAGE >&2
Usage:
    $WAITFORIT_cmdname -u url [-s] [-t timeout] [-- command args]
    -u URL | --url=URL       URL under test
    -s | --strict               Only execute subcommand if the test succeeds
    -q | --quiet                Don't output any status messages
    -t TIMEOUT | --timeout=TIMEOUT
                                Timeout in seconds, zero for no timeout
    -- COMMAND ARGS             Execute command with args after the test finishes
USAGE
    exit 1
}
# @formatter:on

to this:

# @formatter:off
usage()
{
    cat << USAGE >&2
Usage:
    $WAITFORIT_cmdname -u url [-s] [-t timeout] [-- command args]
    -u URL | --url=URL       URL under test
    -s | --strict               Only execute subcommand if the test succeeds
-q | --quiet                Don't output any status messages
    -t TIMEOUT | --timeout=TIMEOUT
                                Timeout in seconds, zero for no timeout
    -- COMMAND ARGS             Execute command with args after the test finishes
USAGE
    exit 1
}
# @formatter:on

[semseysandor]

@formatter tags work for my switch cases, maybe it's a separate issue, that they are not respected?