Rename script files to match bb tasks names

While moving to bb tasks I found that my task names were better
than my script filenames. This change renames script filenames to
match task names - just to keep me sane.

Also now using tasks from scripts and GitHub Action workflows.

And updated change log.

Author: lread

.github/workflows/code-coverage.yml

@@ -43,7 +43,7 @@ jobs:
       run: clojure -Spath
 
     - name: Run Tests with Code Coverage
-      run: bb ./script/coverage.clj
+      run: bb test-coverage
 
     - name: Upload Code Coverage Results
       run: |

.github/workflows/libs-test.yml

@@ -18,7 +18,7 @@ jobs:
           ~/.m2/repository
           ~/.gitlibs
         # base cache on libs_test.clj script for now... lib deps are embedded in it
-        key: $${ runner.os }}-clj-libs-deps-${{ hashFiles('script/libs_tests.clj') }}
+        key: $${ runner.os }}-clj-libs-deps-${{ hashFiles('script/test_libs.clj') }}
         restore-keys: $${ runner.os }}-clj-libs-deps-
 
     - name: Setup Java
@@ -53,4 +53,4 @@ jobs:
         planck --version
 
     - name: Run Libs Tests
-      run: bb ./script/libs_tests.clj run
+      run: bb test-libs run

.github/workflows/native-image-test.yml

@@ -9,7 +9,7 @@ jobs:
       matrix:
         os: [ windows-latest, ubuntu-latest, macos-latest ]
         graal-java: [ java11 ]
-        test: [ pure, sci ]
+        test: [ test-native, test-native-sci ]
 
     name: ${{ matrix.os }},${{ matrix.graal-java }},${{ matrix.test }}
 
@@ -109,9 +109,9 @@ jobs:
       run: |
         # it is overly awkward to get vcvars64.bat loaded into powershell, so we use cmd shell instead
         call "C:\Program Files (x86)\Microsoft Visual Studio\2019\Enterprise\VC\Auxiliary\Build\vcvars64.bat"
-        bb script\${{ matrix.test }}_native_test.clj
+        bb ${{ matrix.test }}
       shell: cmd
       if: matrix.os == 'windows-latest'
     - name: Execute ${{ matrix.test }} native test (macos, linux)
-      run: bb script/${{ matrix.test }}_native_test.clj
+      run: bb ${{ matrix.test }}
       if: matrix.os != 'windows-latest'

.github/workflows/release.yml

@@ -56,16 +56,16 @@ jobs:
       run: clojure -Spath
 
     - name: Release Prep (step 1 of 4)
-      run: bb ./script/release.clj prep
+      run: bb ci-release prep
 
     - name: Release Deploy (step 2 of 4)
       env:
         CLOJARS_USERNAME: ${{ secrets.CLOJARS_USERNAME }}
         CLOJARS_PASSWORD: ${{ secrets.CLOJARS_PASSWORD }}
-      run: bb ./script/release.clj deploy-remote
+      run: bb ci-release deploy-remote
 
     - name: Release Commit (step 3 of 4)
-      run: bb ./script/release.clj commit
+      run: bb ci-release commit
 
     - name: Make GitHub Actions aware of target version tag
       run: echo "::set-output name=tag::v$(cat ./target/target-version.txt)"

.github/workflows/unit-test.yml

@@ -146,4 +146,4 @@ jobs:
     # Run tests
     #
     - name: Run CI tests
-      run: bb ./script/ci_tests.clj
+      run: bb ci-unit-tests

CHANGELOG.adoc

@@ -17,6 +17,8 @@ For a list of breaking changes see link:#v1-breaking[breaking changes].
 === Unreleased
 
 * rewrite-clj now exports clj-kondo config for its public API https://github.com/clj-commons/rewrite-clj/issues/146[#146]
+* Internal rewrite-clj developer facing:
+  * Switched from babashka scripts to babashka tasks, developer guide updated accordingly
 
 === v1.0.605-alpha
 
@@ -362,7 +364,7 @@ If you were using this internal namespace you can opt to switch to, the also int
 ** `prepend-space` in favour of `insert-space-left`
 ** `append-newline` in favour of `insert-newline-right`
 ** `prepend-newline` in favour of `insert-newline-left`
-* fix insertion of nodes in the presense of existing whitespace. (see #33, #34 -
+* fix insertion of nodes in the presence of existing whitespace. (see #33, #34 -
   thanks @eraserhd!)
 * `edn` and `edn*` now take a `:track-position?` option that activates a custom
   zipper implementation allowing `position` to be called on. (see #41, #45 -

bb.edn

@@ -9,26 +9,27 @@
         lread/status-line {:git/url "https://github.com/lread/status-line.git"
                            :sha "35ed39645038e81b42cb15ed6753b8462e60a06d"}}
  :tasks {;; setup
-         :requires ([lread.status-line :as status])
-         :enter (let [{:keys [name]} (current-task)] (status/line :head "TASK %s" name))
+         :requires ([clojure.string :as string]
+                    [lread.status-line :as status])
+         :enter (let [{:keys [name]} (current-task)] (status/line :head "TASK %s %s" name (string/join " " *command-line-args*)))
          :leave (let [{:keys [name]} (current-task)] (status/line :detail "\nTASK %s done." name))
          ;; commands
          apply-import-vars {:task apply-import-vars/-main :doc "(check|gen-code) - export APIs statically from templates"}
          lint              {:task lint/-main              :doc "[--rebuild-cache]"}
-         test-clj          {:task clj-tests/-main         :doc "[--clojure-version (1.9|1.10)]"}
-         test-cljs         {:task cljs-tests/-main        :doc "use --help for args"}
-         test-shadow-cljs  {:task shadow-cljs-test/-main}
-         test-native       {:task pure-native-test/-main  :doc "run rewrite-clj and tests after both compiled with GraalVM native-image"}
-         test-native-sci   {:task sci-native-test/-main   :doc "interpret rewrite-clj tests via sci from rewrite-clj native image"}
-         test-jvm-sci      {:task sci-jvm-test/-main      :doc "JVM only sanity test for test-native-sci"}
-         test-clj-watch    {:task clj-watch/-main         :doc "[kaocha args]"}
-         test-cljs-watch   {:task cljs-watch/-main        :doc "watch cljs test with fighweel main"}
-         test-coverage     {:task coverage/-main          :doc "generate code coverage reports for Clojure tests"}
-         test-doc          {:task doc-tests/-main         :doc "test doc code blocks"}
-         test-libs         {:task libs-tests/-main        :doc "(list|run|outdated) - verify that libs using rewrite-clj* work with current rewrite-clj"}
+         test-clj          {:task test-clj/-main          :doc "[--clojure-version (1.9|1.10)]"}
+         test-cljs         {:task test-cljs/-main         :doc "use --help for args"}
+         test-shadow-cljs  {:task test-shadow-cljs/-main}
+         test-native       {:task test-native/-main       :doc "run rewrite-clj and tests after both compiled with GraalVM native-image"}
+         test-native-sci   {:task test-native-sci/-main   :doc "interpret rewrite-clj tests via sci from rewrite-clj native image"}
+         test-jvm-sci      {:task test-jvm-sci/-main      :doc "JVM only sanity test for test-native-sci"}
+         test-clj-watch    {:task test-clj-watch/-main    :doc "[kaocha args]"}
+         test-cljs-watch   {:task test-cljs-watch/-main   :doc "watch cljs test with fighweel main"}
+         test-coverage     {:task test-coverage/-main     :doc "generate code coverage reports for Clojure tests"}
+         test-doc          {:task test-doc/-main          :doc "test doc code blocks"}
+         test-libs         {:task test-libs/-main         :doc "(list|run|outdated) - verify that libs using rewrite-clj* work with current rewrite-clj"}
          outdated          {:task outdated/-main          :doc "report on outdated Clojure and npm dependencies"}
-         doc-api-diffs     {:task gen-api-diffs/-main     :doc "generate diff docs for rewrite-clj* APIs"}
-         doc-update-readme {:task update-readme/-main     :doc "honour our contributors in README"}
+         doc-api-diffs     {:task doc-api-diffs/-main     :doc "generate diff docs for rewrite-clj* APIs"}
+         doc-update-readme {:task doc-update-readme/-main :doc "honour our contributors in README"}
          cljdoc-preview    {:task cljdoc-preview/-main    :doc "preview what docs will look like on cljdoc, use --help for args"}
-         ci-unit-tests     {:task ci-tests/-main}
-         ci-release        {:task release/-main           :doc "release tasks, use --help for args"}}}
+         ci-unit-tests     {:task ci-unit-tests/-main}
+         ci-release        {:task ci-release/-main        :doc "release tasks, use --help for args"}}}

script/release.clj renamed to script/ci_release.clj

@@ -4,7 +4,7 @@
 ;; This script is ultimately run from GitHub Actions
 ;;
 
-(ns release
+(ns ci-release
   (:require [clojure.java.io :as io]
             [clojure.string :as string]
             [helper.env :as env]

script/ci_tests.clj renamed to script/ci_unit_tests.clj

@@ -1,6 +1,6 @@
 #!/usr/bin/env bb
 
-(ns ci-tests
+(ns ci-unit-tests
   (:require [helper.env :as env]
             [helper.fs :as fs]
             [helper.main :as main]
@@ -12,29 +12,29 @@
     (fs/delete-file-recursively dir true)))
 
 (defn lint []
-  (shell/command ["bb" "./script/lint.clj"]))
+  (shell/command ["bb" "lint"]))
 
 (defn check-import-vars []
-  (shell/command ["bb" "./script/apply_import_vars.clj" "check"]))
+  (shell/command ["bb" "apply-import-vars" "check"]))
 
 (defn doc-tests[]
-  (shell/command ["bb" "./script/doc_tests.clj"]))
+  (shell/command ["bb" "test-doc"]))
 
 (defn clojure-tests []
   (doseq [version ["1.9" "1.10"]]
-    (shell/command ["bb" "./script/clj_tests.clj" "--clojure-version" version])) )
+    (shell/command ["bb" "test-clj" "--clojure-version" version])) )
 
 (defn cljs-tests []
   (doseq [env ["node" "chrome-headless"]
           opt ["none" "advanced"]]
-    (shell/command ["bb" "./script/cljs_tests.clj" "--env" env "--optimizations" opt])))
+    (shell/command ["bb" "test-cljs" "--env" env "--optimizations" opt])))
 
 (defn shadow-cljs-tests []
-  (shell/command ["bb" "./script/shadow_cljs_test.clj"]))
+  (shell/command ["bb" "test-shadow-cljs"]))
 
 (defn cljs-bootstrap-tests []
   (if (some #{(env/get-os)} '(:mac :unix))
-    (shell/command ["bb" "./script/cljs_tests.clj" "--env" "planck" "--optimizations" "none"])
+    (shell/command ["bb" "test-cljs" "--env" "planck" "--optimizations" "none"])
     (status/line :warn "skipping planck tests, they can only be run on linux and macOS")) )
 
 (defn -main [& args]

script/gen_api_diffs.clj renamed to script/doc_api_diffs.clj

@@ -1,6 +1,6 @@
 #!/usr/bin/env bb
 
-(ns gen-api-diffs
+(ns doc-api-diffs
   (:require [clojure.java.io :as io]
             [clojure.string :as string]
             [helper.env :as env]