create-openclaw-bot 5.1.0 → 5.1.2

package/tests/smoke-cli-logic.mjs

@@ -38,6 +38,30 @@ checks.push(() => expectMatch(
   'Docker branch must still auto-install Docker when missing'
 ));
 
+checks.push(() => expectMatch(
+  cli,
+  /function shouldReuseInstalledGlobals\(\) \{[\s\S]*OPENCLAW_SETUP_REUSE_GLOBALS[\s\S]*trim\(\)\.toLowerCase\(\)/,
+  'CLI must expose an env-flag helper for fast test runs that reuse installed global packages'
+));
+
+checks.push(() => expectMatch(
+  cli,
+  /function installLatestOpenClaw\(\{ isVi, osChoice \}\) \{[\s\S]*installGlobalPackage\('openclaw@latest', \{ isVi, osChoice, displayName: 'openclaw' \}\)[\s\S]*process\.exit\(1\)/,
+  'CLI must provide a shared helper that always installs or upgrades openclaw@latest'
+));
+
+checks.push(() => expectMatch(
+  cli,
+  /function installLatestOpenClaw\(\{ isVi, osChoice \}\) \{[\s\S]*shouldReuseInstalledGlobals\(\) && isOpenClawInstalled\(\)[\s\S]*Reusing the installed openclaw/s,
+  'CLI fast-test mode must be able to reuse an existing openclaw install'
+));
+
+checks.push(() => expectMatch(
+  cli,
+  /installLatestOpenClaw\(\{ isVi, osChoice \}\);\s*if \(deployMode === 'docker'\) \{/,
+  'CLI must install or upgrade openclaw before entering the Docker/native branches'
+));
+
 checks.push(() => expectMatch(
   cli,
   /if \(!isOpenClawInstalled\(\)\) \{[\s\S]*installGlobalPackage\('openclaw@latest', \{ isVi, osChoice, displayName: 'openclaw' \}\)/,
@@ -52,19 +76,32 @@ checks.push(() => expectMatch(
 
 checks.push(() => expectMatch(
   cli,
-  /if \(providerKey === '9router' && !is9RouterInstalled\(\)\) \{[\s\S]*installGlobalPackage\('9router@latest', \{ isVi, osChoice, displayName: '9Router' \}\)/,
+  /if \(providerKey === '9router'\) \{[\s\S]*else if \(!is9RouterInstalled\(\)\) \{[\s\S]*installGlobalPackage\('9router@latest', \{ isVi, osChoice, displayName: '9Router' \}\)/,
   'Native 9Router flow must auto-install 9Router'
 ));
 
 checks.push(() => expectMatch(
   cli,
-  /async function writeNative9RouterSyncScript\(projectDir\) \{[\s\S]*9router-smart-route-sync\.js[\s\S]*providerConnections[\s\S]*smart-route/s,
-  'Native 9Router flow must write a smart-route sync script based on ~/.9router/db.json'
+  /if \(providerKey === '9router'\) \{[\s\S]*shouldReuseInstalledGlobals\(\) && is9RouterInstalled\(\)[\s\S]*Reusing the installed 9Router[\s\S]*else if \(!is9RouterInstalled\(\)\)/s,
+  'CLI fast-test mode must be able to reuse an existing 9Router install'
+));
+
+checks.push(() => expectMatch(
+  cli,
+  /function build9RouterSmartRouteSyncScript\(dbPath\) \{[\s\S]*safeDbPath[\s\S]*providerConnections[\s\S]*smart-route[\s\S]*async function writeNative9RouterSyncScript\(projectDir\) \{[\s\S]*getNative9RouterDataDir\(\), 'db\.json'/s,
+  'Native 9Router flow must write a smart-route sync script based on the platform-specific 9Router data directory'
 ));
 
 checks.push(() => expectMatch(
   cli,
-  /Removed smart-route \(no active providers\)[\s\S]*if \(!a\.length\) \{[\s\S]*removeSmartRoute\(\)[\s\S]*if \(!m\.length\) \{[\s\S]*removeSmartRoute\(\)/s,
+  /function getNative9RouterDataDir\(\) \{[\s\S]*process\.platform === 'win32'[\s\S]*AppData[\s\S]*Roaming[\s\S]*9router[\s\S]*os\.homedir\(\)[\s\S]*\.9router/s,
+  'CLI must resolve the correct native 9Router data directory on both Windows and Unix'
+));
+
+checks.push(() => expect(
+  cli.includes("Removed smart-route (no active providers)")
+    && cli.includes("if(!a.length){removeSmartRoute();return;}")
+    && cli.includes("if(!m.length){removeSmartRoute();return;}"),
   '9Router sync logic in CLI must remove stale smart-route combos when providers are disabled'
 ));
 
@@ -94,10 +131,46 @@ checks.push(() => expectMatch(
 
 checks.push(() => expectMatch(
   cli,
-  /function printZaloPersonalLoginInfo\(\{ isVi, deployMode, projectDir \}\) \{[\s\S]*docker compose exec -it ai-bot openclaw channels login --channel zalouser --verbose[\s\S]*const copyCmd = deployMode === 'native'[\s\S]*docker compose cp ai-bot:\$\{qrPath\} \.\/zalo-login-qr\.png/s,
+  /function printZaloPersonalLoginInfo\(\{ isVi, deployMode, projectDir \}\) \{[\s\S]*docker compose exec -it ai-bot openclaw channels login --channel zalouser --verbose[\s\S]*openclaw-zalouser-qr-default\.png[\s\S]*Copy-Item[\s\S]*docker compose cp ai-bot:\$\{qrPath\} \.\/zalo-login-qr\.png/s,
   'CLI must print the dedicated Docker/native Zalo Personal login commands and QR copy path instead of onboarding'
 ));
 
+checks.push(() => expectMatch(
+  cli,
+  /function extractZaloPairingCode\(text\) \{[\s\S]*openclaw pairing approve zalouser[\s\S]*Pairing code:/s,
+  'CLI must be able to extract a Zalo pairing code from the login output'
+));
+
+checks.push(() => expectMatch(
+  cli,
+  /function approveZaloPairingCode\(\{ pairingCode, projectDir, isVi \}\) \{[\s\S]*openclaw pairing approve zalouser \$\{pairingCode\}/s,
+  'CLI must be able to auto-approve a detected Zalo pairing code'
+));
+
+checks.push(() => expectMatch(
+  cli,
+  /async function runNativeZaloPersonalLoginFlow\(\{ isVi, projectDir \}\) \{[\s\S]*spawn\('openclaw', \['channels', 'login', '--channel', 'zalouser', '--verbose'\]/s,
+  'Native Zalo flow must run the zalouser login command'
+));
+
+checks.push(() => expectMatch(
+  cli,
+  /async function runNativeZaloPersonalLoginFlow\(\{ isVi, projectDir \}\) \{[\s\S]*fs\.remove\(qrSourcePath\)[\s\S]*fs\.remove\(qrProjectPath\)[\s\S]*fs\.copy\(qrSourcePath, qrProjectPath, \{ overwrite: true \}\)/s,
+  'Native Zalo flow must clear stale QR files and copy the fresh QR into the project folder'
+));
+
+checks.push(() => expectMatch(
+  cli,
+  /async function runNativeZaloPersonalLoginFlow\(\{ isVi, projectDir \}\) \{[\s\S]*outputBuffer[\s\S]*extractZaloPairingCode\(outputBuffer\)[\s\S]*approveZaloPairingCode\(\{ pairingCode, projectDir, isVi \}\)/s,
+  'Native Zalo flow must auto-approve pairing when the login command emits a pairing code'
+));
+
+checks.push(() => expectMatch(
+  cli,
+  /async function runNativeZaloPersonalLoginFlow\(\{ isVi, projectDir \}\) \{[\s\S]*let loginSucceeded = false[\s\S]*successPattern[\s\S]*if \(exitCode !== 0 && !loginSucceeded\)[\s\S]*else if \(loginSucceeded && exitCode !== 0\)/s,
+  'Native Zalo flow must tolerate non-standard exit codes when the login output already reports success'
+));
+
 checks.push(() => expectMatch(
   cli,
   /baseUrl: deployMode === 'native' \? 'http:\/\/localhost:20128\/v1' : 'http:\/\/9router:20128\/v1'/,
@@ -116,6 +189,36 @@ checks.push(() => expectMatch(
   'VPS native 9Router flow must start a standalone 9Router dashboard on port 20128 via PM2'
 ));
 
+checks.push(() => expectMatch(
+  cli,
+  /function spawnBackgroundProcess\(command, args, options = \{\}\) \{[\s\S]*if \(process\.platform === 'win32'\)[\s\S]*resolveWindowsCommand[\s\S]*Start-Process -WindowStyle Hidden[\s\S]*powershell\.exe/s,
+  'Native desktop background helpers must use hidden Start-Process launches on Windows'
+));
+
+checks.push(() => expectMatch(
+  cli,
+  /function resolveNative9RouterDesktopLaunch\(\) \{[\s\S]*process\.platform === 'win32'[\s\S]*npm root -g[\s\S]*9router', 'app', 'server\.js'[\s\S]*PORT: '20128'[\s\S]*HOSTNAME: '0\.0\.0\.0'[\s\S]*command: '9router'[\s\S]*\['-n', '-t', '-l', '-H', '0\.0\.0\.0', '-p', '20128', '--skip-update'\]/s,
+  'Native desktop 9Router launch must bypass the interactive CLI menu on Windows while preserving the standard CLI launch elsewhere'
+));
+
+checks.push(() => expectMatch(
+  cli,
+  /const native9RouterLaunch = resolveNative9RouterDesktopLaunch\(\);[\s\S]*spawnBackgroundProcess\(native9RouterLaunch\.command, native9RouterLaunch\.args, \{[\s\S]*env: native9RouterLaunch\.env/s,
+  'Native desktop 9Router flow must launch through the background helper with the resolved launch spec'
+));
+
+checks.push(() => expectMatch(
+  cli,
+  /const routerHealth = await waitFor9RouterApiReady\(\);[\s\S]*admin API chua san sang[\s\S]*admin API is not ready yet/s,
+  'Native desktop 9Router flow must warn when the dashboard port opens but the admin API never becomes ready'
+));
+
+checks.push(() => expectMatch(
+  cli,
+  /spawnBackgroundProcess\(process\.execPath, \[native9RouterSyncScriptPath\]/,
+  'Native desktop 9Router sync loop must launch through the background helper'
+));
+
 checks.push(() => expectMatch(
   cli,
   /function runPm2Save\(\{ projectDir, isVi \}\) \{[\s\S]*execSync\('pm2 save'[\s\S]*PM2 save did not complete/s,
@@ -124,8 +227,8 @@ checks.push(() => expectMatch(
 
 checks.push(() => expectMatch(
   cli,
-  /const child = spawn\('openclaw', \['gateway', 'run'\], \{/,
-  'Native desktop flows must start openclaw in foreground'
+  /if \(channelKey === 'zalo-personal'\) \{\s*await runNativeZaloPersonalLoginFlow\(\{ isVi, projectDir \}\);\s*\}[\s\S]*const child = spawn\('openclaw', \['gateway', 'run'\], \{/s,
+  'Native desktop flows must finish the Zalo login flow before starting openclaw in foreground'
 ));
 
 checks.push(() => expectOrder(
@@ -179,19 +282,20 @@ checks.push(() => expectMatch(
 
 checks.push(() => expectMatch(
   setup,
-  /function providerLines\(arr, shell\) \{[\s\S]*npm install -g 9router[\s\S]*9router -n -t -l -H 0\.0\.0\.0 -p 20128 --skip-update/s,
+  /function providerLines\(arr, shell\) \{[\s\S]*npm install -g 9router[\s\S]*npm root -g[\s\S]*Join-Path \$npmRoot[\s\S]*Start-Process -WindowStyle Hidden -FilePath ''node\.exe''[\s\S]*9router-smart-route-sync\.js/s,
   'Native script generation must install and start a standalone 9Router dashboard on port 20128'
 ));
 
 checks.push(() => expectMatch(
   setup,
-  /function native9RouterSyncScriptContent\(\) \{[\s\S]*providerConnections[\s\S]*smart-route/s,
-  'Native script generation must embed a 9Router smart-route sync script'
+  /function native9RouterSyncScriptContent\(\) \{[\s\S]*process\.platform==='win32'[\s\S]*AppData[\s\S]*Roaming[\s\S]*providerConnections[\s\S]*smart-route/s,
+  'Native script generation must embed a 9Router smart-route sync script with the correct Windows data directory'
 ));
 
-checks.push(() => expectMatch(
-  setup,
-  /Removed smart-route \(no active providers\)[\s\S]*if \(!a\.length\) \{[\s\S]*removeSmartRoute\(\)[\s\S]*if \(!m\.length\) \{[\s\S]*removeSmartRoute\(\)/s,
+checks.push(() => expect(
+  setup.includes("Removed smart-route (no active providers)")
+    && setup.includes("if(!a.length){removeSmartRoute();return;}")
+    && setup.includes("if(!m.length){removeSmartRoute();return;}"),
   '9Router sync logic in setup.js must remove stale smart-route combos when providers are disabled'
 ));
 
@@ -245,8 +349,8 @@ checks.push(() => expectMatch(
 
 checks.push(() => expectMatch(
   setup,
-  /docker compose exec -it ai-bot openclaw channels login --channel zalouser --verbose[\s\S]*docker compose cp ai-bot:\/tmp\/openclaw\/openclaw-zalouser-qr-default\.png \.\/zalo-login-qr\.png/s,
-  'Wizard copy must use the dedicated zalouser login command and Docker QR copy path'
+  /Native setup now auto-runs the login flow and copies the QR into the project folder[\s\S]*docker compose exec -it ai-bot openclaw channels login --channel zalouser --verbose[\s\S]*docker compose cp ai-bot:\/tmp\/openclaw\/openclaw-zalouser-qr-default\.png \.\/zalo-login-qr\.png/s,
+  'Wizard copy must mention native auto-login and still show the dedicated Docker QR login command'
 ));
 
 for (const check of checks) {

package/tmp/live-enable-zalouser.cjs

@@ -1,20 +0,0 @@
-const fs = require('fs');
-
-const configPath = process.argv[2];
-
-if (!configPath) {
-  console.error('Usage: node live-enable-zalouser.cjs <configPath>');
-  process.exit(1);
-}
-
-const config = JSON.parse(fs.readFileSync(configPath, 'utf8'));
-config.channels = config.channels || {};
-config.channels.zalouser = {
-  ...(config.channels.zalouser || {}),
-  enabled: true,
-  dmPolicy: 'pairing',
-  autoReply: true
-};
-
-fs.writeFileSync(configPath, JSON.stringify(config, null, 2));
-console.log(`Enabled channels.zalouser in ${configPath}`);

package/tmp/live-enable-zalouser.js

@@ -1,20 +0,0 @@
-const fs = require('fs');
-
-const configPath = process.argv[2];
-
-if (!configPath) {
-  console.error('Usage: node live-enable-zalouser.js <configPath>');
-  process.exit(1);
-}
-
-const config = JSON.parse(fs.readFileSync(configPath, 'utf8'));
-config.channels = config.channels || {};
-config.channels.zalouser = {
-  ...(config.channels.zalouser || {}),
-  enabled: true,
-  dmPolicy: 'pairing',
-  autoReply: true
-};
-
-fs.writeFileSync(configPath, JSON.stringify(config, null, 2));
-console.log(`Enabled channels.zalouser in ${configPath}`);

package/tmp/live-patch-9router.cjs

@@ -1,40 +0,0 @@
-const fs = require('fs');
-
-const dbPath = process.argv[2];
-
-if (!dbPath) {
-  console.error('Usage: node live-patch-9router.cjs <dbPath>');
-  process.exit(1);
-}
-
-const models = [
-  'cx/gpt-5.4',
-  'cx/gpt-5.3-codex',
-  'cx/gpt-5.3-codex-high',
-  'cx/gpt-5.2-codex',
-  'cx/gpt-5.2',
-  'cx/gpt-5.1-codex-max',
-  'cx/gpt-5.1-codex',
-  'cx/gpt-5.1',
-  'cx/gpt-5-codex'
-];
-
-const db = JSON.parse(fs.readFileSync(dbPath, 'utf8'));
-db.combos = db.combos || [];
-
-const combo = {
-  id: 'smart-route',
-  name: 'smart-route',
-  alias: 'smart-route',
-  models
-};
-
-const index = db.combos.findIndex((entry) => entry && entry.id === 'smart-route');
-if (index >= 0) {
-  db.combos[index] = combo;
-} else {
-  db.combos.push(combo);
-}
-
-fs.writeFileSync(dbPath, JSON.stringify(db, null, 2));
-console.log(`Patched smart-route in ${dbPath}`);

package/tmp/live-patch-9router.js

@@ -1,40 +0,0 @@
-const fs = require('fs');
-
-const dbPath = process.argv[2];
-
-if (!dbPath) {
-  console.error('Usage: node live-patch-9router.js <dbPath>');
-  process.exit(1);
-}
-
-const models = [
-  'cx/gpt-5.4',
-  'cx/gpt-5.3-codex',
-  'cx/gpt-5.3-codex-high',
-  'cx/gpt-5.2-codex',
-  'cx/gpt-5.2',
-  'cx/gpt-5.1-codex-max',
-  'cx/gpt-5.1-codex',
-  'cx/gpt-5.1',
-  'cx/gpt-5-codex'
-];
-
-const db = JSON.parse(fs.readFileSync(dbPath, 'utf8'));
-db.combos = db.combos || [];
-
-const combo = {
-  id: 'smart-route',
-  name: 'smart-route',
-  alias: 'smart-route',
-  models
-};
-
-const index = db.combos.findIndex((entry) => entry && entry.id === 'smart-route');
-if (index >= 0) {
-  db.combos[index] = combo;
-} else {
-  db.combos.push(combo);
-}
-
-fs.writeFileSync(dbPath, JSON.stringify(db, null, 2));
-console.log(`Patched smart-route in ${dbPath}`);