[FEAT]: Replace confirmation request with explicit execution.

[seivan]

By submitting this issue, you agree to follow our Code of Conduct.

Describe the need

The confirmation and print for history with use of --shell-out (default in ghcs) is unnecessary if the confirmation can just be execution.

Version

gh-copilot version 1.0.3 (2024-05-08)

Code

    TMPFILE="$(mktemp -t gh-copilotXXXXXX)"
    cleanup() {
      rm -f "$TMPFILE"
      unset TMPFILE FIXED_CMD

    }
    TMPFILE="$(mktemp -t gh-copilotXXXXXX)"
    trap cleanup EXIT
    if GH_DEBUG="$GH_DEBUG" GH_HOST="$GH_HOST" gh copilot suggest -t "$TARGET" "$@" --shell-out "$TMPFILE"; then
      if [ -s "$TMPFILE" ]; then
        FIXED_CMD="$(cat $TMPFILE)"
        print -z "$FIXED_CMD"
      fi
    else
      return 1
    fi

By using print -z we inline the command in the prompt and the user can confirm by executing it via enter.
This also removes print -s for storing it in history, as executing it should be enough.
Also removes the echo for a new line.

Also not sure why, but shouldn't TARGET, TMPFILE and FIXED_CMD be locals so they are unset after?

[seivan]

Fixes: #83