Release v0.0.130

.cargo/config.toml

@@ -3,6 +3,11 @@
 # Moved from src-tauri/.cargo/config.toml to the workspace root so that all
 # workspace members share the same build configuration.
 #
+# RLX (optional): Cargo resolves `rlx` via ../rlx/rlx (workspace.dependencies).
+#   CI — .github/actions/checkout-rlx clones https://github.com/MIT-RLX/rlx.git
+#   Local — `npm run setup:rlx` or direnv (.envrc) symlinks ../rlx -> RLX_ROOT
+#           (default /Users/Shared/rlx; override in gitignored rlx.path).
+#
 # `relative = true` paths are resolved relative to this file's parent
 # directory (the project root).
 
@@ -79,6 +84,17 @@ target-dir = "src-tauri/target"
 # On non-Windows platforms cmake ignores this variable.
 CMAKE_MSVC_RUNTIME_LIBRARY = "MultiThreadedDLL"
 
+# ── macOS: cmake / sccache paths ──────────────────────────────────────────────
+# Homebrew cmake/sccache are not on the default PATH when cargo invokes build
+# scripts. The cmake-0.1.x crate respects the CMAKE env var as the binary path.
+#
+# These are NOT set here because cargo's [env] table is unconditional: a value
+# like "/opt/homebrew/bin/cmake" leaks to Windows / Linux runners where the
+# path doesn't exist, causing cmake-rs to panic with "is `cmake` not installed?"
+# (os error 3 / ENOENT). Instead they are exported only on macOS by:
+#   - .envrc (for `cargo build` via direnv)
+#   - scripts/tauri-build.js (for `npm run tauri:build`)
+
 # ── macOS deployment target ───────────────────────────────────────────────────
 #
 # 14.0 satisfies both:

.dockerignore

@@ -0,0 +1,36 @@
+# Heavy artifacts that should never be sent to the Docker build context.
+# Without this, `docker build` for Dockerfile.upgrade-test transfers tens of
+# GB of local target/ output and chokes on I/O.
+
+# Rust build artifacts
+target/
+src-tauri/target/
+crates/*/target/
+**/*.rlib
+
+# Node / frontend
+node_modules/
+.svelte-kit/
+build/
+.vite/
+
+# IDE / OS
+.git/
+.idea/
+.vscode/
+.DS_Store
+
+# Logs / test results
+test-results/
+playwright-report/
+logs/
+*.log
+*.log.zst
+
+# Local data
+.skill/
+data/
+fixtures/
+
+# Cargo registry should come from the volume mount, not the context
+.cargo/

.envrc

@@ -18,10 +18,28 @@ if [ "$(uname)" = "Darwin" ]; then
   if [ -x "$GAR" ]; then
     export AR="$GAR"
   fi
+
+  # Homebrew cmake/sccache are not on the default PATH when cargo invokes
+  # build scripts. The cmake-0.1.x crate respects CMAKE as the binary path.
+  # Kept out of .cargo/config.toml because [env] is unconditional and would
+  # leak these macOS paths to Windows / Linux runners (cmake-rs panics with
+  # "is `cmake` not installed?" / os error 3).
+  if [ -z "${CMAKE:-}" ] && [ -x "/opt/homebrew/bin/cmake" ]; then
+    export CMAKE="/opt/homebrew/bin/cmake"
+  fi
+  if [ -z "${SCCACHE_PATH:-}" ] && [ -x "/opt/homebrew/bin/sccache" ]; then
+    export SCCACHE_PATH="/opt/homebrew/bin/sccache"
+  fi
 fi
 
 # Use prebuilt llama.cpp if available (skip cmake rebuild).
 # Run `npm run setup:llama-prebuilt` to download.
 if [ -d ".llama-prebuilt/lib" ]; then
   export LLAMA_PREBUILT_DIR="$(pwd)/.llama-prebuilt"
 fi
+
+# Sibling RLX checkout for optional llm-rlx / text-embeddings-rlx features.
+# Default: symlink ../rlx -> /Users/Shared/rlx (override via rlx.path or RLX_ROOT).
+if [ -f "scripts/ensure-rlx.sh" ]; then
+  bash scripts/ensure-rlx.sh
+fi

.githooks/pre-commit

@@ -30,8 +30,22 @@ fi
 # Run cargo fmt on staged Rust files
 if echo "$STAGED_FILES" | grep -q '\.rs$'; then
   echo "🦀 Running cargo fmt…"
-  cargo fmt
-  git add $( echo "$STAGED_FILES" | grep '\.rs$' )
+  # GUI git clients (editors, Tower, etc.) launch hooks in a non-interactive
+  # shell that doesn't source ~/.zshrc, so rustup's ~/.cargo/bin is missing
+  # from PATH. Source the env file rustup ships, or fall back to the default
+  # install path.
+  if [ -f "$HOME/.cargo/env" ]; then
+    . "$HOME/.cargo/env"
+  elif [ -d "$HOME/.cargo/bin" ]; then
+    export PATH="$HOME/.cargo/bin:$PATH"
+  fi
+  if ! command -v cargo >/dev/null 2>&1; then
+    echo "⚠️  cargo not found on PATH; skipping cargo fmt." >&2
+    echo "   Install rustup (https://rustup.rs) or add ~/.cargo/bin to your shell's PATH." >&2
+  else
+    cargo fmt
+    git add $( echo "$STAGED_FILES" | grep '\.rs$' )
+  fi
 fi
 
 echo "✅ Pre-commit checks passed (basic validation only)."

.github/actions/checkout-rlx/action.yml

@@ -0,0 +1,21 @@
+name: Checkout RLX
+description: >-
+  Clone https://github.com/MIT-RLX/rlx next to the skill repo so path
+  dependencies (../../../rlx/rlx) resolve in CI.
+
+inputs:
+  ref:
+    description: Branch, tag, or SHA to checkout
+    required: false
+    default: main
+
+runs:
+  using: composite
+  steps:
+    - name: Clone MIT-RLX/rlx
+      shell: bash
+      env:
+        RLX_REF: ${{ inputs.ref }}
+        RLX_URL: https://github.com/MIT-RLX/rlx.git
+        GITHUB_ACTIONS: "true"
+      run: bash scripts/ensure-rlx.sh

.github/workflows/ci.yml

@@ -100,6 +100,9 @@ jobs:
       - name: Fetch skills submodule
         run: git submodule update --init --depth=1 skills
 
+      - name: Checkout RLX
+        uses: ./.github/actions/checkout-rlx
+
       - name: Setup Rust bootstrap (Linux)
         uses: ./.github/actions/setup-rust-bootstrap-linux
 
@@ -470,6 +473,9 @@ jobs:
         with:
           fetch-depth: 1
 
+      - name: Checkout RLX
+        uses: ./.github/actions/checkout-rlx
+
       - name: Validate Windows app manifest
         shell: bash
         run: node scripts/check-windows-manifest.mjs src-tauri/manifest.xml
@@ -610,6 +616,10 @@ jobs:
       # only happens once per change to the install script.  On cache hit the
       # install script detects the SDK via filesystem and skips the download.
       - name: Cache Vulkan SDK
+        # Don't fail the job on transient cache-service errors (actions/cache@v5
+        # occasionally returns "failure" instead of "miss"). The install script
+        # below handles a missing SDK by downloading it on the spot.
+        continue-on-error: true
         uses: actions/cache@v5
         with:
           path: C:\VulkanSDK
@@ -742,6 +752,9 @@ jobs:
         with:
           fetch-depth: 1
 
+      - name: Checkout RLX
+        uses: ./.github/actions/checkout-rlx
+
       - name: Setup Rust bootstrap (Linux)
         uses: ./.github/actions/setup-rust-bootstrap-linux
 
@@ -851,6 +864,9 @@ jobs:
         with:
           fetch-depth: 1
 
+      - name: Checkout RLX
+        uses: ./.github/actions/checkout-rlx
+
       - name: Setup Rust bootstrap (Linux)
         uses: ./.github/actions/setup-rust-bootstrap-linux
 
@@ -936,6 +952,9 @@ jobs:
       - name: Fetch skills submodule
         run: git submodule update --init --depth=1 skills
 
+      - name: Checkout RLX
+        uses: ./.github/actions/checkout-rlx
+
       - name: Free disk space
         run: node scripts/ci.mjs free-disk-space
 
@@ -1022,6 +1041,9 @@ jobs:
         with:
           fetch-depth: 1
 
+      - name: Checkout RLX
+        uses: ./.github/actions/checkout-rlx
+
       - name: Setup Node.js
         uses: actions/setup-node@v6
         with:

.github/workflows/pr-build.yml

@@ -164,6 +164,9 @@ jobs:
           ref:         ${{ steps.meta.outputs.ref }}
           fetch-depth: 0
 
+      - name: Checkout RLX
+        uses: ./.github/actions/checkout-rlx
+
       # ── Build label (version string used in artifact names + PR comment) ────
       - name: Compute build label
         id: label

.github/workflows/release-linux.yml

@@ -38,6 +38,9 @@ jobs:
         with:
           fetch-depth: 0   # full history for git tag --format and changelog generation
 
+      - name: Checkout RLX
+        uses: ./.github/actions/checkout-rlx
+
       # ── Verify tag matches tauri.conf.json version ────────────────────────────
       - name: Resolve version
         id: version_meta
@@ -164,12 +167,15 @@ jobs:
         shell: bash
         run: |
           set -euo pipefail
-          # Build both in one invocation for consistent feature unification
+          # Build both in one invocation for consistent feature unification.
+          # NOTE: `|| return $?` is required — `set -e` is inhibited inside a
+          # function called via `if ! cmd`, so a failing cargo build would
+          # silently fall through and run_cmd would still return 0.
           run_cmd() {
             # Build daemon first — its default features compile llama-cpp-sys-4
             # with mtmd+vulkan+q1 via skill-llm's target-specific deps.
-            cargo build -p skill-daemon --release --locked --target x86_64-unknown-linux-gnu --timings
-            cargo build -p skill --release --locked --target x86_64-unknown-linux-gnu --features custom-protocol --timings
+            cargo build -p skill-daemon --release --locked --target x86_64-unknown-linux-gnu --timings || return $?
+            cargo build -p skill --release --locked --target x86_64-unknown-linux-gnu --features custom-protocol --timings || return $?
           }
           if ! run_cmd; then
             if [[ -n "${LLAMA_PREBUILT_DIR:-}" ]]; then

.github/workflows/release-mac.yml

@@ -26,6 +26,9 @@ jobs:
         with:
           fetch-depth: 0   # full history so git tag --format can read the annotation
 
+      - name: Checkout RLX
+        uses: ./.github/actions/checkout-rlx
+
       # ── Verify tag matches tauri.conf.json version ────────────────────────────
       # Catches "forgot to bump the version" before wasting 20 min on a build.
       - name: Resolve version
@@ -174,32 +177,39 @@ jobs:
           # feature unification produces one consistent llama-cpp-sys-4 build
           # (with mtmd + metal + q1).  Two separate invocations can cause cargo
           # to reuse a cached llama-cpp-sys-4 static lib that lacks mtmd symbols.
+          # NOTE: `|| return $?` after each cargo invocation is required.
+          # `set -e` is inhibited inside a function called via `if ! cmd` /
+          # captured exit code, so without explicit `|| return` a failing
+          # cargo build would silently continue and run_cmd would still return 0.
           run_cmd() {
             # Build daemon first — its default features (llm + embed-exg) compile
             # llama-cpp-sys-4 with mtmd+metal+q1 via skill-llm's target-specific deps.
             # Building skill second reuses the cached native libs.
-            cargo build -p skill-daemon --release --locked --target aarch64-apple-darwin --timings
-            cargo build -p skill --release --locked --target aarch64-apple-darwin --features custom-protocol --timings
-
+            cargo build -p skill-daemon --release --locked --target aarch64-apple-darwin --timings || return $?
+            cargo build -p skill-tty --release --locked --target aarch64-apple-darwin --timings || return $?
+            cargo build -p skill --release --locked --target aarch64-apple-darwin --features custom-protocol --timings || return $?
+
             # Fix @rpath for skill-daemon immediately after build
             # llama-cpp-4 builds .dylib files by default; we need to embed them
             # in the .app bundle and update the binary's search paths
             DAEMON_BIN="src-tauri/target/aarch64-apple-darwin/release/skill-daemon"
-            if [ -f "$DAEMON_BIN" ]; then
-              echo "Fixing @rpath for skill-daemon…"
-
-              # Create Frameworks directory if it doesn't exist
-              mkdir -p "src-tauri/target/aarch64-apple-darwin/release/Frameworks"
-
-              # Copy any .dylib files that the daemon depends on
-              for LIB in libggml-base.0.dylib libllama.1.dylib; do
-                if [ -f "src-tauri/target/aarch64-apple-darwin/release/$LIB" ]; then
-                  cp "src-tauri/target/aarch64-apple-darwin/release/$LIB" \
-                     "src-tauri/target/aarch64-apple-darwin/release/Frameworks/"
-                  install_name_tool -change @rpath/$LIB @executable_path/../Frameworks/$LIB "$DAEMON_BIN"
-                fi
-              done
+            if [ ! -f "$DAEMON_BIN" ]; then
+              echo "::error::Daemon binary missing after successful cargo build: $DAEMON_BIN"
+              return 1
             fi
+            echo "Fixing @rpath for skill-daemon…"
+
+            # Create Frameworks directory if it doesn't exist
+            mkdir -p "src-tauri/target/aarch64-apple-darwin/release/Frameworks"
+
+            # Copy any .dylib files that the daemon depends on
+            for LIB in libggml-base.0.dylib libllama.1.dylib; do
+              if [ -f "src-tauri/target/aarch64-apple-darwin/release/$LIB" ]; then
+                cp "src-tauri/target/aarch64-apple-darwin/release/$LIB" \
+                   "src-tauri/target/aarch64-apple-darwin/release/Frameworks/" || return $?
+                install_name_tool -change @rpath/$LIB @executable_path/../Frameworks/$LIB "$DAEMON_BIN" || return $?
+              fi
+            done
           }
           if ! run_cmd; then
             if [[ -n "${LLAMA_PREBUILT_DIR:-}" ]]; then

.github/workflows/release-windows.yml

@@ -35,6 +35,9 @@ jobs:
         with:
           fetch-depth: 0
 
+      - name: Checkout RLX
+        uses: ./.github/actions/checkout-rlx
+
       - name: Validate Windows app manifest
         shell: bash
         run: node scripts/check-windows-manifest.mjs src-tauri/manifest.xml
@@ -225,6 +228,10 @@ jobs:
       # toolchain.  Installing them in a single step via background jobs cuts
       # ~1-2 min of sequential wait time on cache-miss runs.
       - name: Cache Vulkan SDK
+        # Don't fail the job on transient cache-service errors (actions/cache@v5
+        # occasionally returns "failure" instead of "miss"). The install step
+        # below handles a missing SDK by downloading it on the spot.
+        continue-on-error: true
         uses: actions/cache@v5
         with:
           path: C:\VulkanSDK
@@ -546,8 +553,11 @@ jobs:
 
           # Build both packages in a single cargo invocation so feature
           # unification happens once and shared dependencies compile only once.
+          # NOTE: `|| return $?` is required — `set -e` is inhibited inside a
+          # function called via `if ! cmd`, so a failing cargo build would
+          # silently fall through and run_cmd would still return 0.
           run_cmd() {
-            cargo build -p skill-daemon -p skill --release --locked --target x86_64-pc-windows-msvc --features custom-protocol --timings
+            cargo build -p skill-daemon -p skill --release --locked --target x86_64-pc-windows-msvc --features custom-protocol --timings || return $?
           }
 
           if ! run_cmd; then
@@ -906,7 +916,7 @@ jobs:
 
       # ── Discord notification ──────────────────────────────────────────────────
       - name: Notify Discord of release
-        if: always() && steps.version_meta.outputs.dry_run != 'true'
+        if: always() && steps.version_meta.outputs.is_release == 'true'
         shell: bash
         env:
           DISCORD_WEBHOOK_URL: ${{ secrets.DISCORD_WEBHOOK_URL }}

.gitignore

@@ -7,6 +7,7 @@ node_modules
 .env.*
 !.env.example
 !.envrc
+rlx.path
 vite.config.js.timestamp-*
 vite.config.ts.timestamp-*