Write machine specs to benchmark logs, default to not write to log

Author: grega

README.md

@@ -130,8 +130,8 @@ This outputs `demo.gif` from the tape file.
 Static benchmark READMEs in `bench/` (small, medium, large, stress) exercise every parsing path at different scales. Run benchmarks with:
 
 ```bash
-./bench/run              # run and append to bench/results.md
-./bench/run --no-log     # print results only, don't log
+./bench/run              # run benchmarks, print results
+./bench/run --log        # also save to bench/results.csv (should only be used by release script / run when creating a new release)
 ```
 
 Benchmarks run automatically during `./release` and are recorded in `bench/results.csv`.

bench/results.csv

@@ -1,13 +1,13 @@
-timestamp,version,fixture,lines,median_ms
-2026-03-17T22:32:43Z,0.7.0,small,21,98.8
-2026-03-17T22:32:43Z,0.7.0,medium,164,466.8
-2026-03-17T22:32:43Z,0.7.0,large,439,915.3
-2026-03-17T22:32:43Z,0.7.0,stress,561,1084.2
-2026-03-17T22:37:47Z,0.8.0,small,21,100.3
-2026-03-17T22:37:47Z,0.8.0,medium,164,474.8
-2026-03-17T22:37:47Z,0.8.0,large,439,989.7
-2026-03-17T22:37:47Z,0.8.0,stress,561,1064.1
-2026-03-17T22:39:13Z,0.9.0,small,21,99.8
-2026-03-17T22:39:13Z,0.9.0,medium,164,472.3
-2026-03-17T22:39:13Z,0.9.0,large,439,925.7
-2026-03-17T22:39:13Z,0.9.0,stress,561,1058.8
+timestamp,version,machine,fixture,lines,median_ms
+2026-03-17T22:32:43Z,0.7.0,Apple_M1_Pro_8c,small,21,98.8
+2026-03-17T22:32:43Z,0.7.0,Apple_M1_Pro_8c,medium,164,466.8
+2026-03-17T22:32:43Z,0.7.0,Apple_M1_Pro_8c,large,439,915.3
+2026-03-17T22:32:43Z,0.7.0,Apple_M1_Pro_8c,stress,561,1084.2
+2026-03-17T22:37:47Z,0.8.0,Apple_M1_Pro_8c,small,21,100.3
+2026-03-17T22:37:47Z,0.8.0,Apple_M1_Pro_8c,medium,164,474.8
+2026-03-17T22:37:47Z,0.8.0,Apple_M1_Pro_8c,large,439,989.7
+2026-03-17T22:37:47Z,0.8.0,Apple_M1_Pro_8c,stress,561,1064.1
+2026-03-17T22:53:19Z,0.9.0,Apple_M1_Pro_8c,small,21,102.8
+2026-03-17T22:53:19Z,0.9.0,Apple_M1_Pro_8c,medium,164,502.4
+2026-03-17T22:53:19Z,0.9.0,Apple_M1_Pro_8c,large,439,973.9
+2026-03-17T22:53:19Z,0.9.0,Apple_M1_Pro_8c,stress,561,1100.5

bench/run

@@ -2,8 +2,8 @@
 # bench/run — benchmark hdi against static READMEs and report timings.
 #
 # Usage:
-#   ./bench/run              # run benchmarks, print results, append to log
-#   ./bench/run --no-log     # run benchmarks, print results only
+#   ./bench/run              # run benchmarks, print results only
+#   ./bench/run --log        # also save to bench/results.csv (used by release)
 
 set -euo pipefail
 
@@ -19,19 +19,39 @@ median() {
   sort -n | awk '{a[NR]=$1} END {print a[int((NR+1)/2)]}'
 }
 
-time_ms() {
-  local start end
-  start=$(/usr/bin/perl -MTime::HiRes=time -e 'printf "%.6f\n", time')
-  "$@" >/dev/null 2>&1
-  end=$(/usr/bin/perl -MTime::HiRes=time -e 'printf "%.6f\n", time')
-  /usr/bin/perl -e "printf \"%.1f\n\", ($end - $start) * 1000"
-}
+if [[ "$(uname)" == "Darwin" ]]; then
+  time_ms() {
+    local start end
+    start=$(/usr/bin/perl -MTime::HiRes=time -e 'printf "%.6f\n", time')
+    "$@" >/dev/null 2>&1
+    end=$(/usr/bin/perl -MTime::HiRes=time -e 'printf "%.6f\n", time')
+    /usr/bin/perl -e "printf \"%.1f\n\", ($end - $start) * 1000"
+  }
+else
+  time_ms() {
+    local start end
+    start=$(date +%s%N)
+    "$@" >/dev/null 2>&1
+    end=$(date +%s%N)
+    echo "$(( (end - start) / 1000000 )).$(( (end - start) / 100000 % 10 ))"
+  }
+fi
 
-# ── Version info ─────────────────────────────────────────────────────────────
+# ── Version and machine info ──────────────────────────────────────────────────
 
 version=$(grep -m1 '^VERSION=' "$HDI" | sed 's/VERSION="//;s/"//')
 timestamp=$(date -u +"%Y-%m-%dT%H:%M:%SZ")
 
+# Build a short machine identifier: Brand_Model_Cores
+if [[ "$(uname)" == "Darwin" ]]; then
+  chip=$(sysctl -n machdep.cpu.brand_string 2>/dev/null | sed 's/ /_/g')
+  cores=$(sysctl -n hw.ncpu 2>/dev/null)
+else
+  chip=$(grep -m1 'model name' /proc/cpuinfo 2>/dev/null | sed 's/.*: //;s/ /_/g')
+  cores=$(nproc 2>/dev/null)
+fi
+machine="${chip:-unknown}_${cores:-?}c"
+
 # ── Benchmark ────────────────────────────────────────────────────────────────
 
 declare -a names=()
@@ -62,7 +82,7 @@ done
 # ── Print results ────────────────────────────────────────────────────────────
 
 printf "\n"
-printf "  hdi %s\n" "$version"
+printf "  hdi %s on %s\n" "$version" "$machine"
 printf "  %s — %d runs per fixture\n\n" "$timestamp" "$RUNS"
 printf "  %-12s %6s %8s\n" "Fixture" "Lines" "Median"
 printf "  %-12s %6s %8s\n" "───────" "─────" "──────"
@@ -73,12 +93,12 @@ printf "\n"
 
 # ── Log to CSV ───────────────────────────────────────────────────────────────
 
-if [[ "${1:-}" == "--no-log" ]]; then
+if [[ "${1:-}" != "--log" ]]; then
   exit 0
 fi
 
 if [[ ! -f "$LOG" ]]; then
-  echo "timestamp,version,fixture,lines,median_ms" > "$LOG"
+  echo "timestamp,version,machine,fixture,lines,median_ms" > "$LOG"
 fi
 
 # Remove any existing rows for this version (keep header + other versions)
@@ -88,7 +108,7 @@ mv "$tmpfile" "$LOG"
 
 # Append new rows
 for i in "${!names[@]}"; do
-  echo "$timestamp,$version,${names[$i]},${lines[$i]},${medians[$i]}" >> "$LOG"
+  echo "$timestamp,$version,$machine,${names[$i]},${lines[$i]},${medians[$i]}" >> "$LOG"
 done
 
 printf "  Logged to bench/results.csv\n\n"

release

@@ -66,7 +66,7 @@ sed -i.bak "s/^VERSION=\"$current\"/VERSION=\"$new\"/" "$HDI" && rm -f "$HDI.bak
 # Run benchmarks against the new version
 if [[ -x "$SCRIPT_DIR/bench/run" ]]; then
   echo ""
-  "$SCRIPT_DIR/bench/run"
+  "$SCRIPT_DIR/bench/run" --log
 fi
 
 git add hdi bench/results.csv