Refactor tools/

Signed-off-by: Andrew Stein <steinlink@gmail.com>

Author: texodus

rust/perspective-viewer/src/css/column-selector.css

@@ -334,17 +334,22 @@
         padding: var(--column_type--padding, 0px 0px 0px 0px);
         position: absolute;
         height: 28px;
-        top: 0;
+        top: -28px;
         display: inline-flex;
         align-items: flex-end;
         content: var(--psp-label--all-columns--content, "All Columns");
     }
 
-    #sub-columns .scroll-panel-container:first-child {
-        padding-top: 28px;
+    #sub-columns .scroll-panel-container {
+        margin-top: 28px;
         margin-bottom: 4px;
     }
 
+    /* #sub-acolumns .scroll-panel-container #add-expression {
+        padding-top: 28px;
+        margin-top: 0;
+    }   */
+
     #sub-columns .column-selector-column.column-selector-column-hidden {
         overflow: hidden;
         height: 0;

rust/perspective-viewer/src/css/config-selector.css

@@ -310,7 +310,7 @@
             padding-top: 15px;
             font-size: var(--label--font-size, 0.75em);
             select {
-                font-size: var(--label--font-size, 0.75em);
+                font-size: var(--label--font-size);
             }
         }
 

tools/scripts/bench.mjs

@@ -11,13 +11,14 @@
 // ┗━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┛
 
 import * as dotenv from "dotenv";
-import sh from "./sh.mjs";
 import { get_scope } from "./sh_perspective.mjs";
 
+import "zx/globals";
+
 dotenv.config({ path: "./.perspectiverc", quiet: true });
 const scope = get_scope();
 if (scope.includes("client")) {
-    sh`pnpm run --recursive --filter bench bench_js`.runSync();
+    $.sync`pnpm run --recursive --filter bench bench_js`;
 } else if (scope.includes("python")) {
-    sh`pnpm run --recursive --filter bench bench_python`.runSync();
+    $.sync`pnpm run --recursive --filter bench bench_python`;
 }

tools/scripts/fix_cpp.mjs

@@ -14,7 +14,6 @@ import path from "path";
 import os from "os";
 import fs from "fs";
 import * as dotenv from "dotenv";
-import sh from "./sh.mjs";
 import { getWorkspaceRoot, getWorkspacePackageJson } from "./workspace.mjs";
 
 import "zx/globals";
@@ -26,27 +25,23 @@ const emscripten = pkg.emscripten;
 dotenv.config({ path: "./.perspectiverc", quiet: true });
 
 function base() {
-    return sh.path`${getWorkspaceRoot()}/.emsdk`;
+    return path.resolve(getWorkspaceRoot(), ".emsdk");
 }
 
 function emsdk_checkout() {
-    function git(args) {
-        sh`git ${args}`.runSync();
-    }
-
-    git(["clone", "https://github.com/emscripten-core/emsdk.git", base()]);
+    $.sync`git clone https://github.com/emscripten-core/emsdk.git ${base()}`;
 }
 
 function emsdk(...args) {
     const basedir = base();
     const suffix = os.type() == "Windows_NT" ? ".bat" : "";
-    const emsdk = path.join(basedir, "emsdk" + suffix);
-    sh`${emsdk} ${args}`.runSync();
+    const emsdkBin = path.join(basedir, "emsdk" + suffix);
+    $.sync`${emsdkBin} ${args}`;
 }
 
 function toolchain_install() {
     console.log(`-- Installing Emscripten ${emscripten}`);
-    sh`git pull`.cwd(".emsdk").runSync();
+    $.sync({ cwd: ".emsdk" })`git pull`;
     emsdk("install", emscripten);
     emsdk("activate", emscripten);
     console.log(`-- Emscripten ${emscripten} installed`);

tools/scripts/install_emsdk.mjs

@@ -21,7 +21,8 @@ export async function lint_js(is_fix = false) {
     await $`prettier ${prettier_flags} "examples/**/*.js" "examples/**/*.tsx" "tools/scripts/*.mjs" "rust/**/*.ts" "rust/**/*.js" "packages/**/*.js" "packages/**/*.ts"`.verbose();
     await $`prettier --prose-wrap=always ${prettier_flags} "rust/*/docs/**/*.md"`.verbose();
     // cmd.sh`prettier ${prettier_flags} "**/*.yaml"`;
-    await $`prettier ${prettier_flags} "**/less/*.less"`.verbose();
+    await $`prettier ${prettier_flags} "**/css/**/*.css"`.verbose();
+    await $`prettier ${prettier_flags} "**/themes/**/*.css"`.verbose();
     await $`prettier ${prettier_flags} "**/html/*.html"`.verbose();
     await $`prettier ${prettier_flags} "packages/**/package.json" "rust/**/package.json" "examples/**/package.json" "docs/package.json"`.verbose();
 

tools/scripts/lint.mjs

@@ -28,7 +28,6 @@ const FIX_PATHS = [
     ["**/*.mjs", ["//", "/*", " *", " */"]],
     // ["**/*.css", ["//", "/*", " *", " */"]],
     ["**/*.scss", ["//", "/*", " *", " */"]],
-    ["**/*.less", ["//", "/*", " *", " */"]],
     ["**/*.toml", ["# ", "#"]],
     ["**/*.yaml", ["# ", "#"]],
     ["**/*.py", ["# ", "#"]],

tools/scripts/lint_headers.mjs

@@ -11,7 +11,6 @@
 // ┗━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┛
 
 import { Octokit } from "octokit";
-import sh from "./sh.mjs";
 import fs from "node:fs/promises";
 
 import "zx/globals";
@@ -21,11 +20,11 @@ const OCTOKIT = new Octokit({
     auth: process.env.GITHUB_TOKEN,
 });
 
-const CURRENT_TAG = sh`git describe --exact-match --tags`.execSync();
+const CURRENT_TAG = $.sync`git describe --exact-match --tags`.toString().trim();
 
-const IS_DIRTY = await sh`git status --untracked-files=no --porcelain`
-    .exec()
-    .then((x) => x.length > 0);
+const IS_DIRTY =
+    (await $`git status --untracked-files=no --porcelain`).stdout.trim()
+        .length > 0;
 
 async function get_release_assets() {
     const resp = await OCTOKIT.request("GET /repos/{owner}/{repo}/releases", {
@@ -71,9 +70,9 @@ async function publish_release_assets(releases) {
                     release.name.endsWith("tar.gz")) &&
                 release.name.indexOf("wasm") === -1
             ) {
-                sh`twine upload ${release.name}`.runSync();
+                $.sync`twine upload ${release.name}`;
             } else if (release.name.endsWith(".tgz")) {
-                sh`npm publish ${release.name}`.runSync();
+                $.sync`npm publish ${release.name}`;
             } else {
                 console.log(`Skipping  "${release.name}"`);
             }

tools/scripts/publish.mjs

@@ -10,10 +10,10 @@
 // ┃ of the [Apache License 2.0](https://www.apache.org/licenses/LICENSE-2.0). ┃
 // ┗━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━━┛
 
-import sh from "./sh.mjs";
 import inquirer from "inquirer";
 import fs from "fs";
 import * as dotenv from "dotenv";
+import { execSync } from "child_process";
 
 const original = dotenv.config({
     path: "./.perspectiverc",
@@ -50,7 +50,7 @@ const CONFIG = new Proxy(
         write() {
             fs.writeFileSync("./.perspectiverc", this.config.join("\n"));
             if (process.env.PSP_BUILD_IMMEDIATELY || process.env.PSP_ONCE) {
-                sh`node tools/scripts/build.mjs`.runSync();
+                execSync("node tools/scripts/build.mjs", { stdio: "inherit" });
                 if (process.env.PSP_ONCE) {
                     while (this.config.length > 0) {
                         this.config.pop();