@yeaft/webchat-agent 1.0.51 → 1.0.53

package/connection/upgrade.js

@@ -32,7 +32,7 @@ const isWin = platform() === 'win32';
 // macOS/Linux: use absolute path — launchd/systemd have minimal PATH.
 const npmPath = isWin ? 'npm' : join(nodeBinDir, 'npm');
 const pm2Path = isWin ? 'pm2' : join(nodeBinDir, 'pm2');
-const shellOpt = isWin ? { shell: true } : {};
+const shellOpt = isWin ? { shell: true, windowsHide: true } : {};
 
 // Ensure PATH includes nodeBinDir so that `#!/usr/bin/env node` shebangs
 // can locate node in launchd/systemd environments with minimal PATH.

package/crew/worktree.js

@@ -27,7 +27,7 @@ export async function initWorktrees(projectDir, roles) {
   // 获取 git 已知的 worktree 列表
   let knownWorktrees = new Set();
   try {
-    const { stdout } = await execFile('git', ['worktree', 'list', '--porcelain'], { cwd: projectDir });
+    const { stdout } = await execFile('git', ['worktree', 'list', '--porcelain'], { cwd: projectDir, windowsHide: true });
     for (const line of stdout.split('\n')) {
       if (line.startsWith('worktree ')) {
         knownWorktrees.add(line.slice('worktree '.length).trim());
@@ -66,13 +66,13 @@ export async function initWorktrees(projectDir, roles) {
     try {
       // 创建分支（如果不存在）
       try {
-        await execFile('git', ['branch', branch], { cwd: projectDir });
+        await execFile('git', ['branch', branch], { cwd: projectDir, windowsHide: true });
       } catch {
         // 分支已存在，忽略
       }
 
       // 创建 worktree
-      await execFile('git', ['worktree', 'add', wtDir, branch], { cwd: projectDir });
+      await execFile('git', ['worktree', 'add', wtDir, branch], { cwd: projectDir, windowsHide: true });
       console.log(`[Crew] Created worktree: ${wtDir} on branch ${branch}`);
       worktreeMap.set(idx, wtDir);
     } catch (e) {
@@ -104,14 +104,14 @@ export async function cleanupWorktrees(projectDir) {
       const branch = `crew/${entry}`;
 
       try {
-        await execFile('git', ['worktree', 'remove', wtDir, '--force'], { cwd: projectDir });
+        await execFile('git', ['worktree', 'remove', wtDir, '--force'], { cwd: projectDir, windowsHide: true });
         console.log(`[Crew] Removed worktree: ${wtDir}`);
       } catch (e) {
         console.warn(`[Crew] Failed to remove worktree ${wtDir}:`, e.message);
       }
 
       try {
-        await execFile('git', ['branch', '-D', branch], { cwd: projectDir });
+        await execFile('git', ['branch', '-D', branch], { cwd: projectDir, windowsHide: true });
         console.log(`[Crew] Deleted branch: ${branch}`);
       } catch (e) {
         console.warn(`[Crew] Failed to delete branch ${branch}:`, e.message);

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@yeaft/webchat-agent",
-  "version": "1.0.51",
+  "version": "1.0.53",
   "description": "Remote worker agent for Yeaft Web Code Agent — connects the native Yeaft engine, CLI providers, and workbench tools",
   "main": "index.js",
   "type": "module",

package/providers/copilot-models.js

@@ -99,7 +99,7 @@ function _probeAcpModels() {
       if (err) reject(err); else resolve(models || []);
     };
     try {
-      child = spawn('copilot', ['--acp'], { stdio: ['pipe', 'pipe', 'pipe'] });
+      child = spawn('copilot', ['--acp'], { stdio: ['pipe', 'pipe', 'pipe'], windowsHide: true });
     } catch (e) { return reject(e); }
     const timer = setTimeout(() => finish(new Error('ACP probe timeout')), PROBE_TIMEOUT_MS);
     child.on('error', (e) => finish(e));

package/yeaft/mcp.js

@@ -81,6 +81,7 @@ class MCPServerConnection extends EventEmitter {
         this.#process = spawn(this.config.command, this.config.args || [], {
           stdio: ['pipe', 'pipe', 'pipe'],
           env: { ...process.env, ...this.config.env },
+          windowsHide: true,
         });
 
         this.#process.stdout.on('data', (data) => {

package/yeaft/prompts.js

@@ -26,7 +26,6 @@ import { readFileSync, existsSync } from 'fs';
 import { join, dirname } from 'path';
 import { fileURLToPath } from 'url';
 import { getRuntimePlatformInfo, renderRuntimePlatformPrompt } from './runtime-platform.js';
-import { DEFAULT_VPS } from './vp/seed-defaults.js';
 
 // ─── Template Loading (one-time at startup) ──────────────────────
 
@@ -463,50 +462,17 @@ function selectVpPersonaName(vpPersona, effectiveLang) {
   return typeof vpPersona.displayName === 'string' ? vpPersona.displayName.trim() : '';
 }
 
-function normalizePersonaBodyForMatch(value) {
-  return typeof value === 'string' ? value.trim() : '';
-}
-
-function selectMigratedStockPersonaBody(vpPersona, body, effectiveLang) {
-  const vpId = typeof vpPersona?.vpId === 'string' ? vpPersona.vpId.trim() : '';
-  const persistedBody = normalizePersonaBodyForMatch(body);
-  if (!vpId || !persistedBody) return null;
-
-  const stock = DEFAULT_VPS.find(vp => vp.vpId === vpId);
-  if (!stock) return null;
-
-  const acceptedBodies = [
-    stock.legacyPersonaEn,
-    stock.legacyPersona,
-    stock.personaEn,
-    ...(Array.isArray(stock.legacyPersonas) ? stock.legacyPersonas : []),
-  ]
-    .map(normalizePersonaBodyForMatch)
-    .filter(Boolean);
-
-  if (!acceptedBodies.includes(persistedBody)) return null;
-  const selected = effectiveLang === 'zh' ? stock.personaZh : stock.personaEn;
-  return normalizePersonaBodyForMatch(selected) || null;
-}
-
 function selectVpPersonaBody(vpPersona, effectiveLang) {
   const body = typeof vpPersona.persona === 'string' ? vpPersona.persona.trim() : '';
 
-  // role.md persona is the canonical soul. Exact stock legacy bodies are mapped
-  // to the current authored source so old seeded role.md files do not leak
-  // bilingual/English souls into localized system prompts before top-up runs.
-  if (body) {
-    const migratedStockBody = selectMigratedStockPersonaBody(vpPersona, body, effectiveLang);
-    if (migratedStockBody) return migratedStockBody;
-
-    if (body.includes('<!-- lang:')) {
-      const selected = extractExactLangSection(body, effectiveLang);
-      return selected !== null ? selected : body;
-    }
-    return body;
+  // role.md is the canonical soul source. Stock legacy migration belongs in
+  // seed-topup, not in prompt rendering; prompt rendering should only select
+  // authored language sections when they are present.
+  if (body && body.includes('<!-- lang:')) {
+    const selected = extractExactLangSection(body, effectiveLang);
+    return selected !== null ? selected : body;
   }
-
-  return '';
+  return body;
 }
 
 

package/yeaft/tools/bash.js

@@ -45,6 +45,7 @@ function runCommand(command, { cwd, timeout, signal, runtimePlatform }) {
       env,
       stdio: ['ignore', 'pipe', 'pipe'],
       detached: !platform.isWindows,
+      windowsHide: true,
     });
 
     let stdout = '';

package/yeaft/tools/enter-worktree.js

@@ -14,6 +14,12 @@ import { existsSync, mkdirSync } from 'fs';
 import { join, resolve } from 'path';
 import { randomUUID } from 'crypto';
 
+const HIDDEN_PROCESS_OPTIONS = { windowsHide: true };
+
+function gitExecFileSync(args, options = {}) {
+  return execFileSync('git', args, { ...options, ...HIDDEN_PROCESS_OPTIONS });
+}
+
 export default defineTool({
   name: 'EnterWorktree',
   description: {
@@ -61,7 +67,7 @@ Worktree 创建在 .yeaft/worktrees/ 中，基于 HEAD 创建新分支。返回
 
     // Verify we're in a git repo
     try {
-      execFileSync('git', ['rev-parse', '--git-dir'], { cwd, stdio: 'pipe' });
+      gitExecFileSync(['rev-parse', '--git-dir'], { cwd, stdio: 'pipe' });
     } catch {
       return JSON.stringify({ error: 'Not in a git repository' });
     }
@@ -93,7 +99,7 @@ Worktree 创建在 .yeaft/worktrees/ 中，基于 HEAD 创建新分支。返回
     try {
       // Create worktree with new branch. Use execFileSync args instead of
       // shell quoting so Windows drive letters and spaces in paths survive.
-      execFileSync('git', ['worktree', 'add', '-b', branchName, worktreeDir, baseRef], { cwd, stdio: 'pipe' });
+      gitExecFileSync(['worktree', 'add', '-b', branchName, worktreeDir, baseRef], { cwd, stdio: 'pipe' });
 
       return JSON.stringify({
         success: true,

package/yeaft/tools/exit-worktree.js

@@ -12,6 +12,12 @@ import { execFileSync } from 'child_process';
 import { existsSync } from 'fs';
 import { resolve } from 'path';
 
+const HIDDEN_PROCESS_OPTIONS = { windowsHide: true };
+
+function gitExecFileSync(args, options = {}) {
+  return execFileSync('git', args, { ...options, ...HIDDEN_PROCESS_OPTIONS });
+}
+
 export default defineTool({
   name: 'ExitWorktree',
   description: {
@@ -84,7 +90,7 @@ unless discard_changes is set to true.`,
       // Check for uncommitted changes
       if (!input.discard_changes) {
         try {
-          const status = execFileSync('git', ['status', '--porcelain'], {
+          const status = gitExecFileSync(['status', '--porcelain'], {
             cwd: worktreePath,
             encoding: 'utf8',
             stdio: ['pipe', 'pipe', 'pipe'],
@@ -104,7 +110,7 @@ unless discard_changes is set to true.`,
       // Get branch name before removal
       let branchName = null;
       try {
-        branchName = execFileSync('git', ['rev-parse', '--abbrev-ref', 'HEAD'], {
+        branchName = gitExecFileSync(['rev-parse', '--abbrev-ref', 'HEAD'], {
           cwd: worktreePath,
           encoding: 'utf8',
           stdio: ['pipe', 'pipe', 'pipe'],
@@ -115,7 +121,7 @@ unless discard_changes is set to true.`,
 
       // Remove worktree. Use argv form so Windows paths with spaces or drive
       // letters are passed to git unchanged.
-      execFileSync('git', ['worktree', 'remove', ...(input.discard_changes ? ['--force'] : []), worktreePath], {
+      gitExecFileSync(['worktree', 'remove', ...(input.discard_changes ? ['--force'] : []), worktreePath], {
         cwd: mainCwd,
         stdio: 'pipe',
       });
@@ -123,7 +129,7 @@ unless discard_changes is set to true.`,
       // Remove the branch if it was a yeaft worktree branch
       if (branchName && branchName.startsWith('yeaft-wt/')) {
         try {
-          execFileSync('git', ['branch', '-D', branchName], {
+          gitExecFileSync(['branch', '-D', branchName], {
             cwd: mainCwd,
             stdio: 'pipe',
           });

package/yeaft/tools/grep.js

@@ -30,7 +30,7 @@ const BINARY_EXTS = new Set([
  */
 function hasRipgrep() {
   return new Promise((resolve) => {
-    const proc = spawn('rg', ['--version'], { stdio: 'pipe' });
+    const proc = spawn('rg', ['--version'], { stdio: 'pipe', windowsHide: true });
     proc.on('close', (code) => resolve(code === 0));
     proc.on('error', () => resolve(false));
   });
@@ -60,7 +60,7 @@ function runRipgrep(pattern, searchPath, options) {
     if (options.multiline) args.push('-U', '--multiline-dotall');
     args.push('--max-count', String(options.maxResults || 500));
 
-    const proc = spawn('rg', args, { stdio: ['ignore', 'pipe', 'pipe'] });
+    const proc = spawn('rg', args, { stdio: ['ignore', 'pipe', 'pipe'], windowsHide: true });
     let stdout = '';
     let stderr = '';
 

package/yeaft/vp/seed-topup.js

@@ -9,24 +9,30 @@
  * (b) a forced overwrite that would clobber their hand edits.
  *
  * This module runs on every agent start alongside `seedDefaultVps` and does
- * two minimal, additive things:
+ * three safe stock-maintenance things:
  *
  *   1. **Top-up missing stock VPs**. If a vpId from `DEFAULT_VPS` is not
  *      on disk, `createVp()` it. This keeps product-owned defaults such as
  *      Omni and the expanded role roster visible in session/member pickers.
  *
- *   2. **Backfill the `area` frontmatter line** on existing seeded VPs whose
- *      role.md predates the area field. The body is left BYTE-IDENTICAL —
- *      we splice a single `area: <bucket>` line into the YAML frontmatter
- *      and write nothing else. If the user has authored their own `area`,
- *      we keep theirs.
+ *   2. **Backfill missing stock frontmatter** (`area`, `nameZh`, `roleZh`) on
+ *      existing seeded VPs whose role.md predates those fields. The persona body
+ *      is left BYTE-IDENTICAL for these metadata backfills. If the user has
+ *      authored their own value, we keep theirs.
+ *
+ *   3. **Migrate exact historical stock bodies to canonical localized bodies**.
+ *      This is intentionally a write-time migration, not a prompt-rendering
+ *      fallback: once top-up sees an old shipped Omni/Linus/etc. body, role.md is
+ *      rewritten to the current `<!-- lang:en -->` / `<!-- lang:zh -->` stock
+ *      persona. User-edited bodies are not touched.
  *
  * Hard rules:
- *   - **Never** overwrite a VP that is on disk. The user might have edited
- *     persona/role/traits; that is their truth, not ours.
- *   - **Keep stock defaults available.** If a shipped stock VP is missing,
- *     recreate it, but never overwrite an on-disk VP. The session/member picker
- *     depends on these product-owned defaults being present.
+ *   - **Never** overwrite a user-edited VP. A persona body is only rewritten when
+ *     it exactly matches a historical body shipped by us.
+ *   - **Keep stock defaults canonical and available.** If a shipped stock VP is
+ *     missing, recreate it. If it exists with an old exact stock body, rewrite
+ *     that body to the current canonical localized body. The session/member
+ *     picker depends on these product-owned defaults being present and current.
  *   - Best-effort: any failure is logged, never thrown.
  *
  * Pre-ledger deletion caveat: on the very first top-up against an existing
@@ -47,8 +53,9 @@
  *     }
  *   }
  *
- * The hash is reserved for future "the default persona changed; offer the
- * user a migration" semantics. We do not auto-upgrade today.
+ * The hash tracks the canonical stock persona currently written by top-up. Exact
+ * historical stock bodies are auto-migrated to this canonical localized body;
+ * user-edited bodies are deliberately left alone.
  */
 
 import { existsSync, mkdirSync, readFileSync, readdirSync, renameSync, statSync, writeFileSync } from 'fs';
@@ -231,7 +238,7 @@ function replaceRoleBody(source, body) {
   return `${match[1]}${String(body || '').trim()}\n`;
 }
 
-function backfillLocalizedPersonaBody(source, vp) {
+export function backfillLocalizedPersonaBody(source, vp) {
   const body = roleBodyOf(source).trim();
   const nextBody = typeof vp.persona === 'string' ? vp.persona.trim() : '';
   if (!body || !nextBody || body === nextBody) return null;
@@ -301,9 +308,9 @@ export function topUpDefaultVps(libDir = DEFAULT_VP_LIB_DIR) {
     const inLedger = Object.prototype.hasOwnProperty.call(versions.seeded, vpId);
 
     if (onDisk) {
-      // Already there — never overwrite. Possibly backfill `area` /
-      // `nameZh`. Each backfill is independent: a role.md with `area`
-      // already set but no `nameZh` should still get `nameZh`.
+      // Already there — preserve user-authored content, but keep shipped stock
+      // defaults canonical. Metadata backfills only add missing frontmatter;
+      // persona migration only rewrites exact historical stock bodies.
       skippedExisting.push(vpId);
       let currentSrc = null;
       const rolePath = join(libDir, vpId, 'role.md');
@@ -388,6 +395,7 @@ export function topUpDefaultVps(libDir = DEFAULT_VP_LIB_DIR) {
             if (patched != null && patched !== src) {
               writeFileSync(rolePath, patched, 'utf-8');
               currentSrc = patched;
+              versions.seeded[vpId] = personaHash(vp.persona);
               personaBackfilled.push(vpId);
             }
           }