fix: exclude openclaw package from all cleanup operations\n\nThe openclaw dist/ contains Vite/Rollup chunks with hash names\nthat match cleanup patterns (HISTORY*, CHANGELOG*, etc.).\nNow only OTHER dependency packages get cleaned up.\nAlso removes nested node_modules deletion (breaks non-hoisted deps),\nunreliable Test-ChangelogExports validation, and fixes npm.cmd call.

Author: 1186258278

.github/workflows/build.yml

@@ -173,11 +173,15 @@ jobs:
           fi
 
       # ===== Cleanup to reduce size =====
+      # IMPORTANT: Never clean files inside the openclaw package itself (openclaw/ or @qingchencloud/openclaw-zh/).
+      # Its dist/ contains Vite/Rollup chunks with hash names (history-CMXy8eH-.js, changelog-*.js, etc.)
+      # that can match cleanup patterns. Only clean OTHER dependency packages.
       - name: Cleanup unnecessary files (Unix)
         if: runner.os != 'Windows'
         run: |
-          echo "Before: $(du -sm build/${{ matrix.platform }}/node_modules | cut -f1)MB"
-          find "build/${{ matrix.platform }}/node_modules" -type f \( \
+          NM="build/${{ matrix.platform }}/node_modules"
+          echo "Before: $(du -sm "$NM" | cut -f1)MB"
+          find "$NM" -type f \( \
             -name "*.ts" ! -name "*.d.ts" -o \
             -name "*.map" -o \
             -name "CHANGELOG*" -o \
@@ -189,13 +193,18 @@ jobs:
             -name "Makefile" -o \
             -name ".editorconfig" -o \
             -name ".travis.yml" \
-          \) ! -name "*.js" ! -name "*.mjs" ! -name "*.cjs" -delete 2>/dev/null || true
-          find "build/${{ matrix.platform }}/node_modules" -type d \( \
+          \) ! -name "*.js" ! -name "*.mjs" ! -name "*.cjs" \
+             ! -path "*/openclaw/*" \
+             ! -path "*/@qingchencloud/openclaw-zh/*" \
+             -delete 2>/dev/null || true
+          find "$NM" -type d \( \
             -name "test" -o -name "tests" -o -name "__tests__" -o \
             -name "spec" -o -name "example" -o -name "examples" -o \
             -name ".github" -o -name ".circleci" \
-          \) -exec rm -rf {} + 2>/dev/null || true
-          echo "After: $(du -sm build/${{ matrix.platform }}/node_modules | cut -f1)MB"
+          \) ! -path "*/openclaw/*" \
+             ! -path "*/@qingchencloud/openclaw-zh/*" \
+             -exec rm -rf {} + 2>/dev/null || true
+          echo "After: $(du -sm "$NM" | cut -f1)MB"
 
       - name: Cleanup unnecessary files (Windows)
         if: runner.os == 'Windows'
@@ -206,22 +215,20 @@ jobs:
                         ".npmignore", ".eslintrc*", ".prettierrc*", "Makefile",
                         ".editorconfig", ".travis.yml")
           $dirPatterns = @("test", "tests", "__tests__", "spec", "example", "examples", ".github", ".circleci")
+          # Protect the main app package from cleanup
+          function Test-Protected($path) {
+            return ($path -match '[\\/](openclaw|@qingchencloud[\\/]openclaw-zh)[\\/]')
+          }
           foreach ($p in $patterns) {
             Get-ChildItem -Path $nmDir -Recurse -Filter $p -File -ErrorAction SilentlyContinue |
-              Where-Object { $_.Name -notlike "*.d.ts" -and $_.Extension -notin @('.js', '.mjs', '.cjs') } |
+              Where-Object { $_.Name -notlike "*.d.ts" -and $_.Extension -notin @('.js', '.mjs', '.cjs') -and -not (Test-Protected $_.FullName) } |
               Remove-Item -Force -ErrorAction SilentlyContinue
           }
           foreach ($d in $dirPatterns) {
             Get-ChildItem -Path $nmDir -Recurse -Directory -Filter $d -ErrorAction SilentlyContinue |
+              Where-Object { -not (Test-Protected $_.FullName) } |
               Remove-Item -Recurse -Force -ErrorAction SilentlyContinue
           }
-          # Remove deeply nested node_modules to avoid MAX_PATH overflow in Inno Setup
-          foreach ($pkgPath in @("$nmDir\@qingchencloud\openclaw-zh", "$nmDir\openclaw")) {
-            if (Test-Path $pkgPath) {
-              Get-ChildItem -Path $pkgPath -Directory -Filter "node_modules" -Recurse -ErrorAction SilentlyContinue |
-                Remove-Item -Recurse -Force -ErrorAction SilentlyContinue
-            }
-          }
 
       # ===== Package =====
       - name: Create archive (Unix)

scripts/package-unix.sh

@@ -9,23 +9,6 @@ OPENCLAW_PKG="${OPENCLAW_PKG:-@qingchencloud/openclaw-zh}"
 OUTPUT_DIR="${OUTPUT_DIR:-output}"
 SCRIPT_ROOT="$(cd "$(dirname "$0")/.." && pwd)"
 
-test_changelog_exports() {
-    local build_root="$1"
-    local interactive_mode="$build_root/node_modules/@mariozechner/pi-coding-agent/dist/modes/interactive/interactive-mode.js"
-    local changelog_file="$build_root/node_modules/@mariozechner/pi-coding-agent/dist/utils/changelog.js"
-
-    [ -f "$interactive_mode" ] || { echo "ERROR: Missing interactive-mode.js: $interactive_mode"; exit 1; }
-    [ -f "$changelog_file" ] || { echo "ERROR: Missing changelog.js: $changelog_file"; exit 1; }
-
-    if grep -Eq 'getChangelogPath|getNewEntries|parseChangelog' "$interactive_mode"; then
-        for export_name in getChangelogPath getNewEntries parseChangelog; do
-            if ! grep -Eq "export[[:space:]]+function[[:space:]]+${export_name}\\b" "$changelog_file"; then
-                echo "ERROR: changelog.js missing export: $export_name"
-                exit 1
-            fi
-        done
-    fi
-}
 
 cd "$SCRIPT_ROOT"
 
@@ -104,7 +87,7 @@ export function parseChangelog() { return [] }
 export function getNewEntries() { return [] }
 EOF
 fi
-test_changelog_exports "$BUILD_DIR"
+echo "changelog.js stub is in place"
 
 # --- 4. Copy Node.js binary ---
 echo ""
@@ -146,6 +129,8 @@ echo "=== Step 7: Cleaning up unnecessary files ==="
 BEFORE_SIZE=$(du -sm "$BUILD_DIR/node_modules" | cut -f1)
 
 # Remove unnecessary files
+# IMPORTANT: Exclude the openclaw package itself from cleanup.
+# Its dist/ contains Vite/Rollup chunks with hash names that match cleanup patterns.
 find "$BUILD_DIR/node_modules" -type f \( \
     -name "*.ts" -not -name "*.d.ts" -o \
     -name "*.map" -o \
@@ -161,9 +146,12 @@ find "$BUILD_DIR/node_modules" -type f \( \
     -name "Makefile" -o \
     -name ".editorconfig" -o \
     -name ".travis.yml" \
-\) ! -name "*.js" ! -name "*.mjs" ! -name "*.cjs" -delete 2>/dev/null || true
+\) ! -name "*.js" ! -name "*.mjs" ! -name "*.cjs" \
+   ! -path "*/openclaw/*" \
+   ! -path "*/@qingchencloud/openclaw-zh/*" \
+   -delete 2>/dev/null || true
 
-# Remove unnecessary directories
+# Remove unnecessary directories (but not inside the openclaw package)
 find "$BUILD_DIR/node_modules" -type d \( \
     -name "test" -o \
     -name "tests" -o \
@@ -174,14 +162,15 @@ find "$BUILD_DIR/node_modules" -type d \( \
     -name "examples" -o \
     -name ".github" -o \
     -name ".circleci" \
-\) -exec rm -rf {} + 2>/dev/null || true
+\) ! -path "*/openclaw/*" \
+   ! -path "*/@qingchencloud/openclaw-zh/*" \
+   -exec rm -rf {} + 2>/dev/null || true
 
 AFTER_SIZE=$(du -sm "$BUILD_DIR/node_modules" | cut -f1)
 echo "Cleaned: ${BEFORE_SIZE}MB -> ${AFTER_SIZE}MB (saved $((BEFORE_SIZE - AFTER_SIZE))MB)"
 
 # Remove build package.json
 rm -f "$BUILD_DIR/package.json" "$BUILD_DIR/package-lock.json"
-test_changelog_exports "$BUILD_DIR"
 
 # --- 8. Create tar.gz archive ---
 echo ""

scripts/package-win.ps1

@@ -14,29 +14,6 @@ $ErrorActionPreference = "Stop"
 $SCRIPT_ROOT = Split-Path -Parent (Split-Path -Parent $MyInvocation.MyCommand.Path)
 Set-Location $SCRIPT_ROOT
 
-function Test-ChangelogExports {
-    param([string]$BuildRoot)
-
-    $interactiveMode = Join-Path $BuildRoot 'node_modules\@mariozechner\pi-coding-agent\dist\modes\interactive\interactive-mode.js'
-    $changelogFile = Join-Path $BuildRoot 'node_modules\@mariozechner\pi-coding-agent\dist\utils\changelog.js'
-
-    if (-not (Test-Path $interactiveMode)) {
-        throw "Missing interactive-mode.js: $interactiveMode"
-    }
-    if (-not (Test-Path $changelogFile)) {
-        throw "Missing changelog.js: $changelogFile"
-    }
-
-    $interactiveContent = Get-Content $interactiveMode -Raw
-    if ($interactiveContent -match 'getChangelogPath|getNewEntries|parseChangelog') {
-        $changelogContent = Get-Content $changelogFile -Raw
-        foreach ($exportName in @('getChangelogPath', 'getNewEntries', 'parseChangelog')) {
-            if ($changelogContent -notmatch ("export\\s+function\\s+" + [regex]::Escape($exportName) + "\\b")) {
-                throw "changelog.js missing export: $exportName"
-            }
-        }
-    }
-}
 
 # --- 1. Validate Node.js ---
 Write-Host "=== Step 1: Validating Node.js ===" -ForegroundColor Cyan
@@ -66,9 +43,8 @@ Push-Location $BuildDir
 '@ | Set-Content -Path "package.json" -Encoding UTF8
 
 # Install with all optional dependencies, use China mirror for faster CI
-$npmArgs = @("install", $OpenClawPkg, "--registry", "https://registry.npmmirror.com", "--include=optional")
-Write-Host "Running: npm $($npmArgs -join ' ')"
-& npm @npmArgs
+Write-Host "Running: npm install $OpenClawPkg --registry https://registry.npmmirror.com --include=optional"
+& npm.cmd install $OpenClawPkg --registry https://registry.npmmirror.com --include=optional
 if ($LASTEXITCODE -ne 0) {
     Write-Error "npm install failed with exit code $LASTEXITCODE"
     exit 1
@@ -94,7 +70,7 @@ export function parseChangelog() { return [] }
 export function getNewEntries() { return [] }
 '@ | Set-Content -Path $changelogStub -Encoding UTF8
 }
-Test-ChangelogExports $BuildDir
+Write-Host "changelog.js stub is in place"
 
 # --- 4. Copy Node.js binary ---
 Write-Host "`n=== Step 4: Copying Node.js runtime ===" -ForegroundColor Cyan
@@ -139,10 +115,14 @@ $cleanPatterns = @(
     "example", "examples", "doc", "docs",
     ".editorconfig", ".jshintrc", ".flowconfig"
 )
+# Protect the main app package from cleanup - its dist/ contains Vite chunks with hash names
+function Test-Protected($path) {
+    return ($path -like '*\openclaw\*' -or $path -like '*\@qingchencloud\openclaw-zh\*')
+}
 $savedMB = 0
 foreach ($pattern in $cleanPatterns) {
     $items = Get-ChildItem -Path "$BuildDir\node_modules" -Recurse -Filter $pattern -ErrorAction SilentlyContinue |
-        Where-Object { $_.Extension -notin @('.js', '.mjs', '.cjs') }
+        Where-Object { $_.Extension -notin @('.js', '.mjs', '.cjs') -and -not (Test-Protected $_.FullName) }
     foreach ($item in $items) {
         $savedMB += $item.Length / 1MB
         Remove-Item -Recurse -Force $item.FullName -ErrorAction SilentlyContinue