scripts: harden slidev docker helpers

Populate titles in the generated template, allow selecting host ports,
and expose NODE_OPTIONS to bump the Playwright container heap. Also
ignore crash dumps and temporary vite config files created while
rendering.

Signed-off-by: Piotr Król <[email protected]>

Author: pietrushnic

.gitignore

@@ -11,3 +11,5 @@ venv
 output
 tmp.*
 slides
+core
+vite.config.ts

scripts/ci/gen_slides.sh

@@ -6,10 +6,15 @@ set -e
 gen_slides() {
     local day
     local escaped_file
+    local escaped_title
     local filename
     local input_file
+    local input_file_abs
+    local input_file_rel
+    local node_max_old_space
     local range
     local output_file
+    local title
 
     # Extract slide information from YAML using yq with pipe as a separator
     IFS=$'\n' # Set Internal Field Separator to newline for reading line by line
@@ -18,25 +23,46 @@ gen_slides() {
         IFS='|' read -r input_file range output_file <<< "$entry"
 
         # Derive other necessary variables from the input_file
-        filename=$(basename "$input_file")
+        if ! input_file_abs=$(realpath "$input_file"); then
+          error_exit "Could not resolve path for $input_file"
+        fi
+
+        if [ ! -f "$input_file_abs" ]; then
+          error_exit "$input_file doesn't exist"
+        fi
+
+        input_file_rel=$(realpath --relative-to "$PWD/slidev-template" "$input_file_abs")
+        if [ -z "$input_file_rel" ]; then
+          error_exit "Couldn't compute relative path for $input_file_abs"
+        fi
+
+        filename=$(basename "$input_file_abs")
         filename=${filename%.*}
         day=${filename:0:1}
+        title=$(awk 'BEGIN{in_frontmatter=0} \
+          /^---[\r\n]*$/ { if (in_frontmatter) exit; in_frontmatter=1; next } \
+          in_frontmatter && /^title:[[:space:]]*/ { sub(/^title:[[:space:]]*/,"",$0); print; exit }' "$input_file_abs")
+        if [ -z "$title" ]; then
+          title="3mdeb Presentation"
+        fi
         copyright=${COPYRIGHT:-3mdeb Sp. z o.o. Licensed under the CC BY-SA 4.0}
         copyright=${copyright//\"/}
 
         # Escape strings for sed
-        escaped_file=$(printf 'slides/%s\n' "$input_file" | sed -e 's/[\/&$]/\\&/g')
+        escaped_file=$(printf '%s\n' "$input_file_rel" | sed -e 's/[\/&$]/\\&/g')
+        escaped_title=$(printf '%s\n' "$title" | sed -e 's/[\/&$]/\\&/g')
 
         # Create temporary markdown file using the slide template
-        sed -e "s/<SRC>/$escaped_file/g" -e "s/<DAY>/$day/g" -e "s/<COPYRIGHT>/$copyright/g" \
+        sed -e "s/<SRC>/$escaped_file/g" -e "s/<DAY>/$day/g" -e "s/<COPYRIGHT>/$copyright/g" -e "s/<TITLE>/$escaped_title/g" \
           slides-template.md >slidev-template/slides.md
 
         cat slidev-template/slides.md
 
+        node_max_old_space=${SLIDEV_NODE_MAX_OLD_SPACE:-4096}
         if [ -n "$USE_DOCKER" ]; then
           docker run -it --rm --user $(id -u):$(id -g) \
             -v "$PWD:/repo" \
-            -p 8000:8000 \
+            -e NODE_OPTIONS=--max-old-space-size="$node_max_old_space" \
             mcr.microsoft.com/playwright:v1.53.2-noble \
             bash -c "
               cd /repo/slidev-template && npm run export slides.md -- \
@@ -57,7 +83,6 @@ check_dependencies() {
   if [ -n "$USE_DOCKER" ]; then
     docker run -it --rm --user $(id -u):$(id -g) \
       -v "$PWD:/repo" \
-      -p 8000:8000 \
       mcr.microsoft.com/playwright:v1.53.2-noble \
       bash -c "cd /repo/slidev-template && npm install"
   else
@@ -135,11 +160,5 @@ if ! check_dependencies; then
   error_exit "Missing dependencies"
 fi
 
-if [ -L "slidev-template/slides" ]; then
-  unlink slidev-template/slides
-fi
-
-ln -sr . slidev-template/slides
-
 # Call the function with the provided YAML file
 gen_slides "$1"

scripts/render-slides.sh

@@ -4,30 +4,82 @@
 render_slides() {
     local day
     local escaped_file
+    local escaped_title
     local filename
-    local input_file
+    local input_arg
+    local input_file_abs
+    local input_file_rel
+    local node_max_old_space
+    local slidev_port
+    local title
+
+    # Remove any surrounding quotes from the argument to avoid breaking realpath
+    input_arg=${1//\"/}
+
+    if ! input_file_abs=$(realpath "$input_arg"); then
+      error_exit "Could not resolve path for $input_arg"
+    fi
+
+    if [ ! -f "$input_file_abs" ]; then
+      error_exit "$input_arg doesn't exist"
+    fi
+
+    title=$(awk 'BEGIN{in_frontmatter=0} \
+      /^---[\r\n]*$/ { if (in_frontmatter) exit; in_frontmatter=1; next } \
+      in_frontmatter && /^title:[[:space:]]*/ { sub(/^title:[[:space:]]*/,"",$0); print; exit }' "$input_file_abs")
+    if [ -z "$title" ]; then
+      title="3mdeb Presentation"
+    fi
+
+    input_file_rel=$(realpath --relative-to "$PWD/slidev-template" "$input_file_abs")
+    if [ -z "$input_file_rel" ]; then
+      error_exit "Couldn't compute relative path for $input_file_abs"
+    fi
 
-    # Remove any surrounding quotes from the variables
-    input_file=${1//\"/}
-    input_file="slides/$(realpath --relative-to slidev-template/slides/ "$input_file")"
     # Derive other necessary variables from the input_file
-    filename=$(basename "$input_file")
+    filename=$(basename "$input_file_abs")
     filename=${filename%.*}
     day=${filename:0:1}
     copyright=${COPYRIGHT:-3mdeb Sp. z o.o. Licensed under the CC BY-SA 4.0}
     copyright=${copyright//\"/}
 
     # Escape strings for sed
-    escaped_file=$(printf '%s\n' "$input_file" | sed -e 's/[\/&$]/\\&/g')
+    escaped_file=$(printf '%s\n' "$input_file_rel" | sed -e 's/[\/&$]/\\&/g')
+    escaped_title=$(printf '%s\n' "$title" | sed -e 's/[\/&$]/\\&/g')
 
     # Create temporary markdown file using the slide template
-    sed -e "s/<SRC>/$escaped_file/g" -e "s/<DAY>/$day/g" -e "s/<COPYRIGHT>/$copyright/g" \
+    sed -e "s/<SRC>/$escaped_file/g" -e "s/<DAY>/$day/g" -e "s/<COPYRIGHT>/$copyright/g" -e "s/<TITLE>/$escaped_title/g" \
       slides-template.md >slidev-template/slides.md
     cp slidev-template/vite.config.ts .
 
+    slidev_port=${SLIDEV_PORT:-8000}
+    node_max_old_space=${SLIDEV_NODE_MAX_OLD_SPACE:-4096}
+
+    if command -v python3 >/dev/null 2>&1; then
+      if ! python3 - "$slidev_port" <<'PY'
+import socket
+import sys
+
+port = int(sys.argv[1])
+s = socket.socket(socket.AF_INET, socket.SOCK_STREAM)
+s.setsockopt(socket.SOL_SOCKET, socket.SO_REUSEADDR, 1)
+try:
+    s.bind(("0.0.0.0", port))
+except OSError:
+    sys.exit(1)
+finally:
+    s.close()
+PY
+      then
+        print_error "Port $slidev_port is already in use. Set SLIDEV_PORT to a free port."
+        exit 1
+      fi
+    fi
+
     docker run -it --rm --user $(id -u):$(id -g) \
       -v "$PWD:/repo" \
-      -p 8000:8000 \
+      -p "$slidev_port":8000 \
+      -e NODE_OPTIONS=--max-old-space-size="$node_max_old_space" \
           mcr.microsoft.com/playwright:v1.53.2-noble \
       bash -c "cd /repo/slidev-template && npm run dev slides.md -- -o false -p 8000 --remote --force"
 
@@ -39,7 +91,6 @@ render_slides() {
 check_dependencies() {
   docker run -it --rm --user $(id -u):$(id -g) \
     -v "$PWD:/repo" \
-    -p 8000:8000 \
     mcr.microsoft.com/playwright:v1.53.2-noble \
     bash -c "cd /repo/slidev-template && npm install"
   return $?
@@ -101,21 +152,9 @@ if [ $# -ne 1 ]; then
   print_usage_error "Script accepts 1 positional arguments, got $#"
 fi
 
-if [ ! -f "$1" ]; then
-  error_exit "$1 doesn't exist"
-fi
-
 if ! check_dependencies; then
   error_exit "Missing dependencies"
 fi
 
-if [ -L "slidev-template/slides" ]; then
-  unlink slidev-template/slides
-fi
-
-if ! ln -sr . slidev-template/slides; then
-  error_exit "Couldn't create slidev-template/slides symlink"
-fi
-
 # Call the function with the provided YAML file
 render_slides "$1"