Adding auto-fix buildifier and auto-fix rubocop

  * to bin/pre-commit file which is symlinked by bin/setup
    to .git/hooks/pre-commit.

If, after running the formatters it's determined that some files have
changed (using git status -s), it shows the diff between the two states
and aborts the commit so that you have a chance to add fixed files.

It does not automatically add anything to git because it would be
a very large and often invalid assumption.

Author: Konstantin Gredeskoul

BUILD.bazel

@@ -2,11 +2,11 @@ load("@com_github_bazelbuild_buildtools//buildifier:def.bzl", "buildifier")
 
 buildifier(
     name = "buildifier",
-    exclude_patterns = [".git/**"],
+    exclude_patterns = ["./.git/**/*"],
 )
 
 buildifier(
     name = "buildifier-check",
-    exclude_patterns = [".git/**"],
+    exclude_patterns = ["./.git/**/*"],
     mode = "check",
 )

bin/ci

@@ -14,16 +14,20 @@ export BAZEL_BUILD_OPTS="--curses=no --verbose_failures -j 15 --show_progress_ra
 export BAZEL_TEST_OPTS="--verbose_failures --test_output=streamed --test_verbose_timeout_warnings "
 export RUBY_VERSION=$(cat .ruby-version)
 
+export BashMatic__Expr="
+  source ${SHELL_INIT}
+  [[ -f ${HOME}/.bashmatic/init.sh ]] && source ${HOME}/.bashmatic/init.sh
+  set -e
+"
+
 [[ -n ${DEBUG} ]] && set -x
 
 trap exit 1 INT
 
 # Runs Rubocop Tests
 test::rubocop() {
   /usr/bin/env bash -c " 
-    source ${SHELL_INIT}
-    [[ -f ${HOME}/.bashmatic/init.sh ]] && source  ${HOME}/.bashmatic/init.sh
-    set -e
+    ${BashMatic__Expr}
     h2 'Running target: rubocop'
     bundle install --path=${BUNDLE_PATH}
     bundle exec rubocop -E -D .rubocop.yml --force-exclusion
@@ -33,22 +37,17 @@ test::rubocop() {
 # Runs Buildifier
 test::buildifier() {
   /usr/bin/env bash -c "
-    source ${SHELL_INIT}
-    [[ -f ${HOME}/.bashmatic/init.sh ]] && source  ${HOME}/.bashmatic/init.sh
+    ${BashMatic__Expr}
     h2 'Running target: buildifier'
-    set -e
     bazel run :buildifier-check
   "
 }
 
 # Builds and runs workspace inside examples/simple_script
 test::simple-script() {
   /usr/bin/env bash -c "
-    source ${SHELL_INIT}
-    [[ -f ${HOME}/.bashmatic/init.sh ]] && source  ${HOME}/.bashmatic/init.sh
+    ${BashMatic__Expr}
     h2 'Testing target: simple-script'
-    set -e
-    cd examples/simple_script
     bazel ${BAZEL_OPTS} info
     bazel ${BAZEL_OPTS} build ${BAZEL_BUILD_OPTS} -- //...
     bazel ${BAZEL_OPTS} test ${BAZEL_BUILD_OPTS} ${BAZEL_TEST_OPTS} -- //...
@@ -58,10 +57,8 @@ test::simple-script() {
 
 test::workspace() {
   /usr/bin/env bash -c "
-    source ${SHELL_INIT}
-    [[ -f ${HOME}/.bashmatic/init.sh ]] && source  ${HOME}/.bashmatic/init.sh
+    ${BashMatic__Expr}
     h2 'Testing target: workspace'
-    set -e
     bazel ${BAZEL_OPTS} build ${BAZEL_BUILD_OPTS} -- //...
     bazel ${BAZEL_OPTS} test ${BAZEL_BUILD_OPTS} ${BAZEL_TEST_OPTS} -- //...
   "
@@ -70,10 +67,8 @@ test::workspace() {
 # Private 
 test::bazel-info() {
   /usr/bin/env bash -c "
-    source ${SHELL_INIT}
-    [[ -f ${HOME}/.bashmatic/init.sh ]] && source  ${HOME}/.bashmatic/init.sh
+    ${BashMatic__Expr}
     h2 'Testing target: bazel-info'
-    set -e
     bazel ${BAZEL_OPTS} version
     bazel ${BAZEL_OPTS} info
     bazel ${BAZEL_OPTS} fetch --curses=no -- '//ruby/...'

bin/pre-commit

@@ -2,7 +2,8 @@
 #
 # Main dependency shell script that installs BashMatic Library in ~/.bashmatic folder.
 
-export COLUMNS=90
+export COLUMNS=75
+
 set -e
 # shellcheck disable=SC1091
 source "bin/deps"
@@ -13,10 +14,68 @@ source "bin/deps"
   setup::main gems
 }
 
-info "Running Rubocop, please wait..."
-run "rubocop -F -D || rubocop -a -D -P || true"
-run::set-next show-output-on
-run "rubocop -D -P"
+run::set-all abort-on-error
+
+fix::rubocop() {
+  h2::green "Running Rubocop, please wait..."
+  run       "rubocop || rubocop -a"
+  inf       "Rubocop completed."
+  ok:
+}
+
+# runs buildifer from the PATH if it exists, otherwise bazel target
+# we want pre-commit to be as fast as possible.
+fix::buildifier() {
+  h2::green "Buildifier"
+  if [[ -n $(command -v buildifier) ]]; then
+    info "Running $(command -v buildifier).."
+    run "find . -name 'BUILD*' -o -name 'WORKSPACE' -o -name '*.bzl'  | grep -v '.git' | xargs buildifier -v"
+  else
+    info "Running ${bldylw}bazel run :buildifier..."
+    run "bazel run :buildifier"
+  fi
+}
+
+__fix::cleanup() {
+  h2::green "Cleaning up..."
+  for f in "${__git_pre}" "${__git_post}"; do
+    [[ -f ${f} ]] && run "rm -f ${f}"
+  done
+}
+
+export __git_pre="/tmp/rules-ruby-git-status-pre-commit.$$"
+export __git_post="/tmp/rules-ruby-git-status-post-commit.$$"
+
+trap __fix::cleanup EXIT
+
+fix() {
+  # number of modified files
+  local changes_before=$(git status -s | md5)
+
+  git status -s >"${__git_pre}"
+
+  fix::rubocop
+  fix::buildifier
+
+  git status -s >"${__git_post}"
+
+  local changes_after=$(git status -s | md5)
+  if [[ ${changes_before} != "${changes_after}" ]]; then
+    hl::subtle "Git status -s output changed after pre-commit hook."
+    info "Changes before pre-commit hook:"
+    hr
+    diff "${pre}" "${post}"
+    hr
+    echo
+    info "ACTION: ${bldylw}Please add any respective files to the commit and retry."
+    exit 1
+  else
+    hr
+    info "No changes detected, commit will proceed."
+    exit 0
+  fi
+}
 
-success "RuboCop ran successfully."
+fix "$@"
 
+exit 0

bin/setup

@@ -11,7 +11,11 @@ setup::gems() {
 }
 
 setup::git() {
-  [[ -L .git/hooks/pre-commit ]] || run "ln -nfs bin/pre-commit .git/hooks/pre-commit"
+  inf 'Installing git pre-commit hook'
+  set -e
+  [[ -L .git/hooks/pre-commit ]] || run "cd .git/hooks && ln -nfs ../../bin/pre-commit pre-commit"
+  set +e
+  ok:
 }
 
 setup::install() {

bin/setup-darwin

@@ -25,11 +25,7 @@ __setup::brew-install-jdk() {
 }
 
 __setup::brew-deps() {
-  if ! brew list | grep -q xz; then
-    "brew install xz"
-  else
-    info: "Found brew package xz."
-  fi
+  lib::brew::install::packages 'xz' 'ydiff'
 }
 
 __setup::is-bazelisk-installed() {

ruby/private/toolchains/BUILD.runtime.tpl

@@ -9,8 +9,8 @@ package(default_visibility = ["//visibility:public"])
 ruby_toolchain(
     name = "toolchain",
     interpreter = "//:ruby_bin",
-    runtime = "//:runtime",
     rules_ruby_workspace = "{rules_ruby_workspace}",
+    runtime = "//:runtime",
     # TODO(yugui) Extract platform info from RbConfig
     # exec_compatible_with = [],
     # target_compatible_with = [],
@@ -25,14 +25,14 @@ sh_binary(
 cc_import(
     name = "libruby",
     hdrs = glob({hdrs}),
-    static_library = {static_library},
     shared_library = {shared_library},
+    static_library = {static_library},
 )
 
 cc_library(
     name = "headers",
-    includes = {includes},
     hdrs = glob({hdrs}),
+    includes = {includes},
 )
 
 filegroup(