fix: overhaul CI + improve package definitions

Sunrisepeak · Sunrisepeak · commit fc91defd33bd · 2026-05-13T15:26:45.000+08:00
CI:
- setup-toolchain action: use quick_install.sh, persist PATH via
  GITHUB_PATH, add Windows xlings install step
- ci.yml: dynamic change detection (replaces dorny/paths-filter),
  remove all manual PATH exports, trigger on .github/** and .xlings.json

Package definitions:
- llmapi: add 0.2.5/0.2.6, move tinyhttps dep to on_load with
  version guard (only &gt;=0.2.0), remove global add_deps
- tinyhttps: add 0.2.2/0.2.3, move mbedtls dep to on_load,
  add target="tinyhttps" to install
diff --git a/.github/actions/setup-toolchain/action.yml b/.github/actions/setup-toolchain/action.yml
@@ -1,61 +1,33 @@
 name: Setup C++23 Toolchain
-description: Install xlings and C++23 toolchain for the current platform
+description: Install xlings and project dependencies declared in .xlings.json
 
 runs:
   using: composite
   steps:
-    - name: Setup (linux)
-      if: runner.os == 'Linux'
-      shell: bash
-      run: |
-        LATEST_VERSION=$(gh release view --repo d2learn/xlings --json tagName -q '.tagName')
-        VERSION_NUM=${LATEST_VERSION#v}
-        TARBALL="xlings-${VERSION_NUM}-linux-x86_64.tar.gz"
-        gh release download "$LATEST_VERSION" --repo d2learn/xlings --pattern "$TARBALL" --dir /tmp
-        mkdir -p "$HOME/.xlings"
-        tar -xzf "/tmp/$TARBALL" -C "$HOME/.xlings" --strip-components=1
-        "$HOME/.xlings/bin/xlings" self install
-      env:
-        GH_TOKEN: ${{ github.token }}
-
-    - name: Setup (macos)
-      if: runner.os == 'macOS'
+    - name: Install xlings (unix)
+      if: runner.os != 'Windows'
       shell: bash
       run: |
-        LATEST_VERSION=$(gh release view --repo d2learn/xlings --json tagName -q '.tagName')
-        VERSION_NUM=${LATEST_VERSION#v}
-        ARCH=$(uname -m)
-        TARBALL="xlings-${VERSION_NUM}-macosx-${ARCH}.tar.gz"
-        gh release download "$LATEST_VERSION" --repo d2learn/xlings --pattern "$TARBALL" --dir /tmp
-        mkdir -p "$HOME/.xlings"
-        tar -xzf "/tmp/$TARBALL" -C "$HOME/.xlings" --strip-components=1
-        xattr -dr com.apple.quarantine "$HOME/.xlings" 2>/dev/null || true
-        "$HOME/.xlings/bin/xlings" self install
-      env:
-        GH_TOKEN: ${{ github.token }}
+        curl -fsSL https://raw.githubusercontent.com/openxlings/xlings/main/tools/other/quick_install.sh | bash
+        echo "XLINGS_HOME=$HOME/.xlings" >> "$GITHUB_ENV"
+        echo "$HOME/.xlings/subos/current/bin" >> "$GITHUB_PATH"
+        echo "$HOME/.xlings/bin" >> "$GITHUB_PATH"
 
-    - name: Setup (windows)
+    - name: Install xlings (windows)
       if: runner.os == 'Windows'
       shell: pwsh
-      run: irm https://raw.githubusercontent.com/d2learn/xlings/refs/heads/main/tools/other/quick_install.ps1 | iex
+      run: |
+        irm https://raw.githubusercontent.com/openxlings/xlings/main/tools/other/quick_install.ps1 | iex
+        "XLINGS_HOME=$env:USERPROFILE\.xlings" >> $env:GITHUB_ENV
+        "$env:USERPROFILE\.xlings\subos\current\bin" >> $env:GITHUB_PATH
+        "$env:USERPROFILE\.xlings\bin" >> $env:GITHUB_PATH
 
-    - name: Refresh package index (unix)
+    - name: Install dependencies
       if: runner.os != 'Windows'
       shell: bash
-      run: |
-        export PATH="$HOME/.xlings/subos/current/bin:$PATH"
-        xlings update
+      run: xlings install -y
 
-    - name: Refresh package index (windows)
+    - name: Install dependencies (windows)
       if: runner.os == 'Windows'
       shell: pwsh
-      run: |
-        $env:PATH = "$env:USERPROFILE\.xlings\subos\current\bin;$env:PATH"
-        xlings update
-
-    - name: Install toolchain (unix)
-      if: runner.os != 'Windows'
-      shell: bash
-      run: |
-        export PATH="$HOME/.xlings/subos/current/bin:$PATH"
-        xlings install -y
+      run: xlings install -y
diff --git a/.github/workflows/ci.yml b/.github/workflows/ci.yml
@@ -6,13 +6,15 @@ on:
     paths:
       - 'packages/**'
       - 'tests/**'
-      - '.github/workflows/ci.yml'
+      - '.github/**'
+      - '.xlings.json'
   pull_request:
     branches: [main]
     paths:
       - 'packages/**'
       - 'tests/**'
-      - '.github/workflows/ci.yml'
+      - '.github/**'
+      - '.xlings.json'
 
 env:
   XLINGS_NON_INTERACTIVE: 1
@@ -21,46 +23,48 @@ jobs:
   detect-changes:
     runs-on: ubuntu-24.04
     outputs:
-      templates: ${{ steps.filter.outputs.templates }}
-      cmdline: ${{ steps.filter.outputs.cmdline }}
-      llmapi: ${{ steps.filter.outputs.llmapi }}
-      lua: ${{ steps.filter.outputs.lua }}
-      xpkg: ${{ steps.filter.outputs.xpkg }}
+      packages: ${{ steps.detect.outputs.packages }}
+      build_all: ${{ steps.detect.outputs.build_all }}
     steps:
       - uses: actions/checkout@v4
-      - uses: dorny/paths-filter@v3
-        id: filter
         with:
-          filters: |
-            templates:
-              - 'packages/t/templates/**'
-              - 'tests/t/templates/**'
-              - '.github/workflows/ci.yml'
-            cmdline:
-              - 'packages/c/cmdline/**'
-              - 'tests/c/cmdline/**'
-              - '.github/workflows/ci.yml'
-            llmapi:
-              - 'packages/l/llmapi/**'
-              - 'tests/l/llmapi/**'
-              - '.github/workflows/ci.yml'
-            lua:
-              - 'packages/m/mcpplibs-capi-lua/**'
-              - 'tests/l/lua/**'
-              - '.github/workflows/ci.yml'
-            xpkg:
-              - 'packages/m/mcpplibs-xpkg/**'
-              - 'tests/m/mcpplibs-xpkg/**'
-              - '.github/workflows/ci.yml'
+          fetch-depth: 0
+
+      - name: Detect changed packages
+        id: detect
+        run: |
+          if [ "${{ github.event_name }}" = "pull_request" ]; then
+            BASE="${{ github.event.pull_request.base.sha }}"
+          else
+            BASE="${{ github.event.before }}"
+          fi
+
+          CI_CHANGED=$(git diff --name-only "$BASE" HEAD -- .github/ .xlings.json || true)
+          if [ -n "$CI_CHANGED" ]; then
+            echo "build_all=true" >> "$GITHUB_OUTPUT"
+            echo "CI config changed, will build all packages"
+          else
+            echo "build_all=false" >> "$GITHUB_OUTPUT"
+          fi
+
+          CHANGED_DIRS=$(git diff --name-only "$BASE" HEAD -- packages/ tests/ | \
+            sed -n 's|^packages/./\([^/]*\)/.*|\1|p; s|^tests/./\([^/]*\)/.*|\1|p' | \
+            sort -u)
+
+          JSON="["
+          FIRST=true
+          for pkg in $CHANGED_DIRS; do
+            if [ "$FIRST" = true ]; then FIRST=false; else JSON="$JSON,"; fi
+            JSON="$JSON\"$pkg\""
+          done
+          JSON="$JSON]"
+
+          echo "packages=$JSON" >> "$GITHUB_OUTPUT"
+          echo "Changed packages: $JSON"
 
   build:
     needs: detect-changes
-    if: >-
-      needs.detect-changes.outputs.templates == 'true' ||
-      needs.detect-changes.outputs.cmdline == 'true' ||
-      needs.detect-changes.outputs.llmapi == 'true' ||
-      needs.detect-changes.outputs.lua == 'true' ||
-      needs.detect-changes.outputs.xpkg == 'true'
+    if: needs.detect-changes.outputs.build_all == 'true' || needs.detect-changes.outputs.packages != '[]'
     strategy:
       fail-fast: false
       matrix:
@@ -77,98 +81,121 @@ jobs:
       - name: Configure (linux)
         if: runner.os == 'Linux'
         working-directory: tests
-        run: |
-          export PATH="$HOME/.xlings/subos/current/bin:$PATH"
-          xmake f -P . -y
+        run: xmake f -P . -y -vv
 
       - name: Configure (macos)
         if: runner.os == 'macOS'
         working-directory: tests
-        run: |
-          export PATH="$HOME/.xlings/subos/current/bin:$PATH"
-          xmake f -P . -y --toolchain=llvm
+        run: xmake f -P . -y -vv --toolchain=llvm
 
       - name: Configure (windows)
         if: runner.os == 'Windows'
         working-directory: tests
-        run: |
-          $env:PATH = "$env:USERPROFILE\.xlings\subos\current\bin;$env:PATH"
-          xmake f -P . -y
+        run: xmake f -P . -y -vv
 
-      # templates
-      - name: templates (unix)
-        if: runner.os != 'Windows' && needs.detect-changes.outputs.templates == 'true'
-        working-directory: tests
-        run: |
-          export PATH="$HOME/.xlings/subos/current/bin:$PATH"
-          xmake build -P . -y templates_test
-          xmake run -P . templates_test
-      - name: templates (windows)
-        if: runner.os == 'Windows' && needs.detect-changes.outputs.templates == 'true'
+      - name: Build and test (unix)
+        if: runner.os != 'Windows'
         working-directory: tests
+        env:
+          BUILD_ALL: ${{ needs.detect-changes.outputs.build_all }}
+          CHANGED_PACKAGES: ${{ needs.detect-changes.outputs.packages }}
         run: |
-          $env:PATH = "$env:USERPROFILE\.xlings\subos\current\bin;$env:PATH"
-          xmake build -P . -y templates_test
-          xmake run -P . templates_test
+          TARGETS=""
+          if [ "$BUILD_ALL" = "true" ]; then
+            TARGETS=$(grep -r 'target("' */*/xmake.lua 2>/dev/null | sed 's/.*target("\([^"]*\)").*/\1/' | sort -u)
+            echo "Building ALL targets: $TARGETS"
+          else
+            for pkg in $(echo "$CHANGED_PACKAGES" | tr -d '[]"' | tr ',' ' '); do
+              TARGET="${pkg}_test"
+              if grep -rq "target(\"$TARGET\")" */*/xmake.lua 2>/dev/null; then
+                TARGETS="$TARGETS $TARGET"
+              else
+                echo "Warning: no test target '$TARGET' found for package '$pkg', skipping"
+              fi
+            done
+            echo "Building changed targets: $TARGETS"
+          fi
 
-      # cmdline
-      - name: cmdline (unix)
-        if: runner.os != 'Windows' && needs.detect-changes.outputs.cmdline == 'true'
-        working-directory: tests
-        run: |
-          export PATH="$HOME/.xlings/subos/current/bin:$PATH"
-          xmake build -P . -y cmdline_test
-          xmake run -P . cmdline_test test_input
-      - name: cmdline (windows)
-        if: runner.os == 'Windows' && needs.detect-changes.outputs.cmdline == 'true'
-        working-directory: tests
-        run: |
-          $env:PATH = "$env:USERPROFILE\.xlings\subos\current\bin;$env:PATH"
-          xmake build -P . -y cmdline_test
-          xmake run -P . cmdline_test test_input
+          FAILED=""
+          for target in $TARGETS; do
+            echo "=== Building $target ==="
+            if ! xmake build -P . -y "$target"; then
+              echo "FAILED to build $target"
+              FAILED="$FAILED $target"
+              continue
+            fi
 
-      # llmapi (build only, needs API key to run)
-      - name: llmapi (unix)
-        if: runner.os != 'Windows' && needs.detect-changes.outputs.llmapi == 'true'
-        working-directory: tests
-        run: |
-          export PATH="$HOME/.xlings/subos/current/bin:$PATH"
-          xmake build -P . -y llmapi_test
-      - name: llmapi (windows)
-        if: runner.os == 'Windows' && needs.detect-changes.outputs.llmapi == 'true'
-        working-directory: tests
-        run: |
-          $env:PATH = "$env:USERPROFILE\.xlings\subos\current\bin;$env:PATH"
-          xmake build -P . -y llmapi_test
+            if [ "$target" = "llmapi_test" ]; then
+              echo "Skipping run for $target (requires API key)"
+              continue
+            fi
 
-      # lua
-      - name: lua (unix)
-        if: runner.os != 'Windows' && needs.detect-changes.outputs.lua == 'true'
-        working-directory: tests
-        run: |
-          export PATH="$HOME/.xlings/subos/current/bin:$PATH"
-          xmake build -P . -y lua_test
-          xmake run -P . lua_test
-      - name: lua (windows)
-        if: runner.os == 'Windows' && needs.detect-changes.outputs.lua == 'true'
-        working-directory: tests
-        run: |
-          $env:PATH = "$env:USERPROFILE\.xlings\subos\current\bin;$env:PATH"
-          xmake build -P . -y lua_test
-          xmake run -P . lua_test
+            echo "=== Running $target ==="
+            if [ "$target" = "cmdline_test" ]; then
+              xmake run -P . "$target" test_input
+            else
+              xmake run -P . "$target"
+            fi
+          done
 
-      # mcpplibs-xpkg
-      - name: mcpplibs-xpkg (unix)
-        if: runner.os != 'Windows' && needs.detect-changes.outputs.xpkg == 'true'
-        working-directory: tests
-        run: |
-          export PATH="$HOME/.xlings/subos/current/bin:$PATH"
-          xmake build -P . -y mcpplibs-xpkg_test
-          xmake run -P . mcpplibs-xpkg_test
-      - name: mcpplibs-xpkg (windows)
-        if: runner.os == 'Windows' && needs.detect-changes.outputs.xpkg == 'true'
+          if [ -n "$FAILED" ]; then
+            echo "::error::Failed targets:$FAILED"
+            exit 1
+          fi
+
+      - name: Build and test (windows)
+        if: runner.os == 'Windows'
         working-directory: tests
+        env:
+          BUILD_ALL: ${{ needs.detect-changes.outputs.build_all }}
+          CHANGED_PACKAGES: ${{ needs.detect-changes.outputs.packages }}
         run: |
-          $env:PATH = "$env:USERPROFILE\.xlings\subos\current\bin;$env:PATH"
-          xmake build -P . -y mcpplibs-xpkg_test
-          xmake run -P . mcpplibs-xpkg_test
+          $targets = @()
+          if ($env:BUILD_ALL -eq "true") {
+            $targets = Get-ChildItem -Recurse -Filter "xmake.lua" -Path "*/*" |
+              Select-String 'target\("([^"]+)"\)' |
+              ForEach-Object { $_.Matches[0].Groups[1].Value } |
+              Sort-Object -Unique
+            Write-Host "Building ALL targets: $($targets -join ', ')"
+          } else {
+            $packages = $env:CHANGED_PACKAGES | ConvertFrom-Json
+            foreach ($pkg in $packages) {
+              $target = "${pkg}_test"
+              $found = Get-ChildItem -Recurse -Filter "xmake.lua" -Path "*/*" |
+                Select-String "target\(`"$target`"\)" -Quiet
+              if ($found) {
+                $targets += $target
+              } else {
+                Write-Host "Warning: no test target '$target' found for package '$pkg', skipping"
+              }
+            }
+            Write-Host "Building changed targets: $($targets -join ', ')"
+          }
+
+          $failed = @()
+          foreach ($target in $targets) {
+            Write-Host "=== Building $target ==="
+            xmake build -P . -y $target
+            if ($LASTEXITCODE -ne 0) {
+              Write-Host "FAILED to build $target"
+              $failed += $target
+              continue
+            }
+
+            if ($target -eq "llmapi_test") {
+              Write-Host "Skipping run for $target (requires API key)"
+              continue
+            }
+
+            Write-Host "=== Running $target ==="
+            if ($target -eq "cmdline_test") {
+              xmake run -P . $target test_input
+            } else {
+              xmake run -P . $target
+            }
+          }
+
+          if ($failed.Count -gt 0) {
+            Write-Host "::error::Failed targets: $($failed -join ', ')"
+            exit 1
+          }
diff --git a/packages/l/llmapi/xmake.lua b/packages/l/llmapi/xmake.lua
@@ -19,10 +19,12 @@ package("llmapi")
     add_versions("0.0.1", "174f86d3afdf48a57ad1cc9688718d1f1100a78a7e56686c823c573c3ccf99f4")
 
     add_includedirs("include")
-    add_deps("mcpplibs-tinyhttps 0.2.2")
 
     on_load(function (package)
         package:add("links", "llmapi")
+        if package:version():ge("0.2.0") then
+            package:add("deps", "mcpplibs-tinyhttps >=0.2.0")
+        end
     end)
 
     on_install(function (package)
diff --git a/packages/m/mcpplibs-tinyhttps/xmake.lua b/packages/m/mcpplibs-tinyhttps/xmake.lua
