@goodandready/dsh-goal 0.2.20 → 0.2.22

package/lib/addon-store.js

@@ -22,6 +22,15 @@ export class AddonStore {
   getGoalData(goalId) {
     if (!goalId) return null;
     if (!this.goals.has(goalId)) {
+      if (this.goals.size > 100) {
+        for (const [k, v] of this.goals.entries()) {
+          const isInactive = (!v.milestones || v.milestones.length === 0 || v.milestones.every(m => m.completed));
+          if (isInactive) {
+            this.goals.delete(k);
+            if (this.goals.size <= 80) break;
+          }
+        }
+      }
       this.goals.set(goalId, {
         milestones: [],
         snapshots: [],
@@ -32,6 +41,13 @@ export class AddonStore {
     return this.goals.get(goalId);
   }
 
+  removeGoalData(goalId) {
+    if (!goalId) return false;
+    const deleted = this.goals.delete(goalId);
+    if (deleted) this.scheduleSave();
+    return deleted;
+  }
+
   getMilestones(goalId) {
     const data = this.getGoalData(goalId);
     return data ? data.milestones : [];
@@ -102,13 +118,14 @@ export class AddonStore {
       return;
     }
     if (!this.saveTimer) {
-      this.saveTimer = setTimeout(() => {
+      const timer = setTimeout(() => {
         this.saveTimer = null;
         this.writeToDiskSync();
       }, 200);
-      if (typeof this.saveTimer.unref === 'function') {
-        this.saveTimer.unref();
+      if (typeof timer?.unref === 'function') {
+        timer.unref();
       }
+      this.saveTimer = timer;
     }
   }
 
@@ -192,10 +209,12 @@ export class AddonStore {
         migratedAny = true;
       }
 
-      data._migratedToAddon = true;
-      data._migratedAt = Date.now();
-      fs.writeFileSync(legacyPath, JSON.stringify(data, null, 2), 'utf8');
-      this.scheduleSave(true);
+      if (migratedAny) {
+        data._migratedToAddon = true;
+        data._migratedAt = Date.now();
+        fs.writeFileSync(legacyPath, JSON.stringify(data, null, 2), 'utf8');
+        this.scheduleSave(true);
+      }
       return migratedAny;
     } catch (err) {
       this.logger.warn?.('[AddonStore] Legacy migration failed:', err.message);

package/lib/engine-milestones.js

@@ -116,7 +116,16 @@ export function applyMilestoneUpdate(target, status, notes = '', checklist = nul
     }
   }
 
-  if (status && (validStatuses.length === 0 || validStatuses.includes(status))) {
+  if (status) {
+    if (validStatuses.length > 0 && !validStatuses.includes(status)) {
+      return {
+        prevStatus,
+        updatedStatus: prevStatus,
+        blocked: false,
+        invalidStatus: true,
+        error: `Invalid milestone status: ${status}`,
+      };
+    }
     target.status = status;
   }
   if (notes !== undefined && notes !== null && notes !== '') {

package/lib/engine-prompt.js

@@ -1,3 +1,4 @@
+import { detectLanguage } from "./goal-engine.js";
 import { isMilestoneBlocked } from './engine-milestones.js';
 
 /**
@@ -100,3 +101,29 @@ export function buildStatePromptInjection(snapshot, pendingNudge) {
     `3. When all milestones are completed, call tool goal_finish with a detailed summary of achieved results.`
   );
 }
+
+
+/**
+ * Format goal start prompt with mandatory milestone contract
+ * @param {string} goalText
+ * @param {string} [explicitLang]
+ * @returns {string}
+ */
+export function formatGoalStartPrompt(goalText, explicitLang) {
+  const lang = explicitLang || detectLanguage(goalText);
+  if (lang === "zh") {
+    return `🎯 目标模式已激活 (Goal Mode): "${goalText}"\n\n` +
+      `自主执行契约：\n` +
+      `1. 第一步必选动作：在开展任何工作或回复之前，你的第一个动作必须是调用 goal_set_milestones 工具，提交包含 3-7 个具体连续步骤的工作计划。用户将在界面中实时查看此计划。\n` +
+      `2. 第二步：计划确立后开始执行任务。在开始每个步骤前，通过 goal_update_progress(milestone_id, "in_progress") 标记其为进行中。\n` +
+      `3. 第三步：完成某一步骤后，通过 goal_update_progress(milestone_id, "completed", "阶段简要成果") 标记为已完成。\n` +
+      `4. 第四步：当所有里程碑全部完成后，调用 goal_finish(summary) 工具并附带详尽的最终成果总结。`;
+  }
+
+  return `🎯 Goal Mode activated: "${goalText}"\n\n` +
+    `STRICT AUTONOMOUS CONTRACT:\n` +
+    `1. MANDATORY STEP 1: Your FIRST action BEFORE doing any other work or reply MUST be calling tool goal_set_milestones with an array of 3-7 concrete, sequential milestones. The user sees this plan in real time.\n` +
+    `2. STEP 2: After establishing the plan, proceed to execute milestones. Before starting each milestone, mark it as in_progress via goal_update_progress(milestone_id, "in_progress").\n` +
+    `3. STEP 3: Upon completing a milestone, mark it as completed via goal_update_progress(milestone_id, "completed", "brief milestone outcome").\n` +
+    `4. STEP 4: When all milestones are completed, call tool goal_finish(summary) with a comprehensive summary of achieved results.`;
+}

package/lib/engine-reports.js

@@ -184,7 +184,7 @@ export function exportReportGitHubPR(state) {
     md += `\n</details>\n\n`;
   }
 
-  md += `*Automated by [@goodandready/dsh-goal](https://github.com/GooDAnDReaDY/dsh-goal)*\n`;
+  md += `*Automated by [@goodandready/dsh-goal](https://github.com/goodandready/dsh-goal)*\n`;
   return md;
 }
 
@@ -261,7 +261,7 @@ export function rollbackToCheckpoint(commitHash, cwd) {
  * @param {string} [cwd]
  * @returns {{ relativePath: string, fullPath: string, content: string }|null}
  */
-export function saveGoalArtifact(state, cwd) {
+export function saveGoalArtifact(state, cwd, addonData = {}) {
   if (!state || (!state.title && !state.objective)) return null;
   try {
     const root = cwd || process.cwd();
@@ -281,7 +281,7 @@ export function saveGoalArtifact(state, cwd) {
     const fullPath = path.join(targetDir, filename);
     const relativePath = path.join('.dsh', 'goals', filename).replace(/\\/g, '/');
 
-    const content = exportReportMarkdown(state);
+    const content = exportReportMarkdown(state, addonData);
     fs.writeFileSync(fullPath, content, 'utf8');
 
     return { relativePath, fullPath, content };
@@ -295,5 +295,5 @@ export function saveGoalArtifact(state, cwd) {
  */
 export function saveGoalReportFile(goal, addonData = {}, baseDir = null) {
   const root = baseDir || process.cwd();
-  return saveGoalArtifact(goal, root)?.fullPath || null;
+  return saveGoalArtifact(goal, root, addonData)?.fullPath || null;
 }

package/lib/engine-retrospective.js

@@ -1,5 +1,6 @@
 import { execFileSync } from 'node:child_process';
 import { MilestoneStatus, calculateElapsedSeconds } from './goal-engine-constants.js';
+import { isValidGitRef } from './engine-git-branch.js';
 
 /**
  * Generate post-goal retrospective analytics card
@@ -33,8 +34,9 @@ export function generateRetrospectiveData(goal, cwd = process.cwd()) {
   let gitDiffStat = '';
   let filesChangedCount = 0;
   try {
-    const startRef = goal.startCommit || 'HEAD~1';
-    const rawStat = execFileSync('git', ['diff', '--stat', `${startRef}..HEAD`], {
+    const rawStartRef = goal.gitStartCommit || goal.startCommit || 'HEAD~1';
+    const startRef = (rawStartRef && isValidGitRef(rawStartRef)) ? rawStartRef : 'HEAD~1';
+    const rawStat = execFileSync('git', ['diff', '--stat', `${startRef}..HEAD`, '--'], {
       cwd,
       encoding: 'utf8',
       stdio: ['ignore', 'pipe', 'ignore'],

package/lib/engine-sessions.js

@@ -32,6 +32,7 @@ export function addNudgeToGoal(goal, text) {
   if (!Array.isArray(goal.nudges)) goal.nudges = [];
   const now = Date.now();
   goal.nudges.push({ text: clean, timestamp: now });
+  if (goal.nudges.length > 50) goal.nudges = goal.nudges.slice(-50);
   goal.pendingNudge = clean;
   goal.logs.push({
     timestamp: now,

package/lib/engine-store.js

@@ -93,13 +93,14 @@ export class EngineStore {
       return;
     }
     if (!this.saveTimer) {
-      this.saveTimer = setTimeout(() => {
+      const timer = setTimeout(() => {
         this.saveTimer = null;
         this.writeStateToDiskSync();
       }, 250);
-      if (typeof this.saveTimer.unref === 'function') {
-        this.saveTimer.unref();
+      if (typeof timer?.unref === 'function') {
+        timer.unref();
       }
+      this.saveTimer = timer;
     }
   }
 

package/lib/goal-engine.js

@@ -18,7 +18,9 @@ export { getGitCurrentCommit, exportReportMarkdown, exportReportGitHubPR, create
  */
 export class GoalEngine {
   constructor(options = {}) {
-    this.defaultMaxIterations = options.defaultMaxIterations ?? 25;
+    this.logger = options.logger || console;
+    const rawDefIters = options.defaultMaxIterations ?? 25;
+    this.defaultMaxIterations = typeof rawDefIters === "number" ? rawDefIters : (typeof rawDefIters?.value === "number" ? rawDefIters.value : 25);
     this.autoDrive = options.autoDrive ?? true;
     this.enableSound = options.enableSound ?? true;
     this.showQuickLaunchButton = options.showQuickLaunchButton ?? true;
@@ -95,11 +97,15 @@ export class GoalEngine {
   }
 
   updateConfig(config = {}) {
-    if (typeof config.defaultMaxIterations === 'number' && config.defaultMaxIterations >= 1) {
+    const rawMax = config.defaultMaxIterations ?? config.maxIterations;
+    const maxVal = typeof rawMax === 'number' ? rawMax : (typeof rawMax?.value === 'number' ? rawMax.value : undefined);
+    if (typeof maxVal === 'number' && maxVal >= 1) {
       const prev = this.defaultMaxIterations;
-      this.defaultMaxIterations = config.defaultMaxIterations;
+      this.defaultMaxIterations = maxVal;
       for (const [_, goal] of this.goals) {
-        if (goal && goal.maxIterations === prev) goal.maxIterations = config.defaultMaxIterations;
+        if (goal && (goal.maxIterations === prev || typeof goal.maxIterations !== 'number' || typeof goal.maxIterations === 'object')) {
+          goal.maxIterations = maxVal;
+        }
       }
     }
     if (typeof config.autoDrive === 'boolean') this.autoDrive = config.autoDrive;
@@ -373,13 +379,18 @@ export class GoalEngine {
       enforceDependencies: this.enableMilestoneDependencies,
       allMilestones: goal.milestones,
     });
-    if (res.blocked) {
-      recordActivity(goal, {
-        type: 'milestone',
-        tool: 'updateMilestone',
-        status: 'error',
-        details: `Milestone ${id} blocked by ${res.missingDependencies?.join(', ')}`,
-      });
+    if (res.invalidStatus || res.blocked) {
+      if (res.invalidStatus) {
+        this.logger?.warn?.(`[dsh-goal] Invalid milestone status "${status}" for milestone ${id}`);
+      }
+      if (res.blocked) {
+        recordActivity(goal, {
+          type: 'milestone',
+          tool: 'updateMilestone',
+          status: 'error',
+          details: `Milestone ${id} blocked by ${res.missingDependencies?.join(', ')}`,
+        });
+      }
       return false;
     }
 
@@ -401,7 +412,7 @@ export class GoalEngine {
             recordActivity(goal, { type: 'issue', tool: 'syncIssueChecklist', status: 'success', details: `Issue #${syncRes.issueNumber} updated` });
             this.emit(sid);
           }
-        }).catch(() => {});
+        }).catch((err) => { this.logger?.warn?.("[dsh-goal] Failed to sync milestone with issue:", err?.message || err); });
       }
     }
 

package/lib/index.js

@@ -1,3 +1,4 @@
+import fs from 'node:fs';
 import path from 'node:path';
 import os from 'node:os';
 import z from '@deepseek-ai/schemastery';
@@ -130,9 +131,12 @@ export function apply(ctx, config = {}) {
 
   // Attempt migration of legacy state if present
   try {
-    const agentsSvc = safeService(ctx, "agents");
-    const firstAgent = agentsSvc?.get?.(agentsSvc?.keys?.()?.next()?.value);
-    addonStore.migrateLegacyState(storagePath, safeService(ctx, "goals"), firstAgent);
+    const legacyStatePath = path.resolve(defaultStorageDir, 'dsh-goal-state.json');
+    if (fs.existsSync(legacyStatePath)) {
+      const agentsSvc = safeService(ctx, "agents");
+      const firstAgent = agentsSvc?.get?.(agentsSvc?.keys?.()?.next()?.value);
+      addonStore.migrateLegacyState(legacyStatePath, safeService(ctx, "goals"), firstAgent);
+    }
   } catch (migErr) {
     logger.warn?.('[dsh-goal] Legacy migration check failed:', migErr.message);
   }
@@ -162,6 +166,7 @@ export function apply(ctx, config = {}) {
       autoScaleBudgetNearCompletion: currentSettings.autoScaleBudgetNearCompletion,
       enableTemplatesDrawer: currentSettings.enableTemplatesDrawer,
       enableMilestoneDependencies: currentSettings.enableMilestoneDependencies,
+      logger,
     });
   }
 
@@ -192,14 +197,55 @@ export function apply(ctx, config = {}) {
     }
   }
 
+  const sessionAgents = new Map();
+
+  const stopRunningAgents = (sessionId = 'default') => {
+    try {
+      const agentsSvc = safeService(ctx, "agents");
+      const agent = sessionAgents.get(sessionId) || agentsSvc?.get?.(sessionId) || agentsSvc?.get?.(agentsSvc?.keys?.()?.next()?.value);
+      if (agent && typeof agent.abort === 'function') {
+        agent.abort();
+      } else if (agent && typeof agent.cancel === 'function') {
+        agent.cancel();
+      } else if (agent && typeof agent.stop === 'function') {
+        agent.stop();
+      }
+    } catch (err) {
+      logger.warn?.('[dsh-goal] Failed to stop agent:', err?.message || err);
+    }
+  };
+
+  const resumeActiveAgent = (prompt, sessionId = 'default') => {
+    try {
+      const agentsSvc = safeService(ctx, "agents");
+      const agent = sessionAgents.get(sessionId) || agentsSvc?.get?.(sessionId) || agentsSvc?.get?.(agentsSvc?.keys?.()?.next()?.value);
+      if (agent) {
+        if (typeof agent.resume === 'function') {
+          agent.resume(prompt);
+        } else if (typeof agent.step === 'function') {
+          agent.step(prompt);
+        } else if (typeof agent.send === 'function' && prompt) {
+          agent.send(prompt);
+        }
+      }
+    } catch (err) {
+      logger.warn?.('[dsh-goal] Failed to resume agent:', err?.message || err);
+    }
+  };
+
   // 2. WebServer HTTP & SSE routes (registered after updater so it remains primary)
   const { broadcastEvent } = registerRoutes(ctx, {
     engine: fallbackEngine,
     milestoneManager,
+    snapshotManager: gitSnapshotManager,
     gitSnapshotManager,
+    budgetGuard: tokenBudgetGuard,
     tokenBudgetGuard,
     addonStore,
     goalsService: safeService(ctx, "goals"),
+    stopRunningAgents,
+    resumeActiveAgent,
+    sessionAgents,
     getConfig: () => currentSettings,
     sseClients,
     logger,
@@ -243,6 +289,7 @@ export function apply(ctx, config = {}) {
 
     // Inject milestone prompts and check budget during agent turns
     ctx.on('agent/pre-step', async ({ agent, messages, step, signal }, next) => {
+      if (agent) { sessionAgents.set(sessionIdOf(agent, "default"), agent); }
       const goalsSvc = safeService(ctx, "goals");
       if (goalsSvc && typeof goalsSvc.get === "function" && agent) {
         try {
@@ -284,7 +331,7 @@ export function apply(ctx, config = {}) {
 
     // Compatibility events for turn/end and approval/asked
     ctx.on('turn/end', (event) => {
-      // noop or session isolation
+      // session isolation turn tracking
     });
 
     ctx.on('approval/asked', (event) => {

package/lib/milestone-manager.js

@@ -95,11 +95,15 @@ export class MilestoneManager {
 
     // Best-effort Gitea issue checklist sync if issue data exists
     const goalData = this.store.getGoalData(goalId);
-    if (goalData?.issue?.number) {
+    if (goalData?.issueRef?.issueNumber || goalData?.issue?.number) {
       try {
-        syncMilestoneWithIssue(goalData.issue.number, m.title, true);
-      } catch (e) {
-        // sync error non-fatal
+        const issueRef = goalData.issueRef || { issueNumber: goalData.issue.number };
+        const fakeGoal = { issueRef, cwd: workspaceRoot };
+        syncMilestoneWithIssue(fakeGoal, m, { cwd: workspaceRoot }).catch((err) => {
+          this.logger?.warn?.('[MilestoneManager] Issue sync failed:', err?.message || err);
+        });
+      } catch (err) {
+        this.logger?.warn?.('[MilestoneManager] Issue sync error:', err?.message || err);
       }
     }
 
@@ -132,8 +136,8 @@ export class MilestoneManager {
 
     const uncompletedChecklist = (active.checklist || []).filter(c => !c.done);
     let clText = '';
-    if (active.checklist?.length > 0) {
-      clText = ` | Next items: ${active.checklist.map(c => `[${c.done ? 'x' : ' '}] ${c.text}`).join(', ')}`;
+    if (uncompletedChecklist.length > 0) {
+      clText = ` | Next items: ${uncompletedChecklist.map(c => `[ ] ${c.text}`).join(', ')}`;
     }
 
     return `[Goal Milestones]: Current step: "${active.title}"${clText}`;

package/lib/routes.js

@@ -79,10 +79,14 @@ export function registerRoutes(ctx, {
   sseClients = new Map(),
   addonStore,
   budgetGuard,
+  tokenBudgetGuard,
   snapshotManager,
+  gitSnapshotManager,
   milestoneManager,
   logger = console,
 } = {}) {
+  const activeBudgetGuard = budgetGuard || tokenBudgetGuard;
+  const activeSnapshotManager = snapshotManager || gitSnapshotManager;
   // Subscribe to engine if provided (backward compatibility)
   if (engine && typeof engine.subscribe === 'function') {
     engine.subscribe((snapshot, sid) => {
@@ -396,6 +400,9 @@ export function registerRoutes(ctx, {
                   if (engine) {
                     result = engine.clear(sid);
                   }
+                  if (addonStore && typeof addonStore.removeGoalData === 'function') {
+                    addonStore.removeGoalData(sid);
+                  }
                   if (typeof stopRunningAgents === 'function') {
                     stopRunningAgents(sid);
                   }

package/lib/tools.js

@@ -141,7 +141,6 @@ export function registerTools(ctx, { milestoneManager, goalsService, logger = co
       },
       output: JSON_OUTPUT,
       execute: handleGoalMilestones,
-      handler: handleGoalMilestones,
     });
 
     // Tool 2: goal_checklist
@@ -181,7 +180,99 @@ export function registerTools(ctx, { milestoneManager, goalsService, logger = co
       },
       output: JSON_OUTPUT,
       execute: handleGoalChecklist,
-      handler: handleGoalChecklist,
+    });
+    // Tool 3: goal_set_milestones (Issue #127 alias matching autonomous-contract prompt)
+    const handleGoalSetMilestones = async (args = {}, toolCtx = {}) => {
+      const goalId = args.goalId || args.goal_id || resolveGoalId(toolCtx);
+      let list = args.milestones;
+      if (Array.isArray(list)) {
+        list = list.map((item, idx) => {
+          if (typeof item === 'string') {
+            return { id: `m-${idx + 1}`, title: item };
+          }
+          return item;
+        });
+      } else {
+        list = [];
+      }
+      const res = mm.setMilestones(goalId, list);
+      return { ok: true, goalId, count: res.length, milestones: res };
+    };
+
+    safeRegister({
+      name: 'goal_set_milestones',
+      description: 'Set initial work plan milestones for the active goal.',
+      parameters: {
+        type: 'object',
+        properties: {
+          milestones: {
+            type: 'array',
+            items: { type: ['string', 'object'] },
+            description: 'Array of milestone titles or objects',
+          },
+        },
+        required: ['milestones'],
+      },
+      output: JSON_OUTPUT,
+      execute: handleGoalSetMilestones,
+    });
+
+    // Tool 4: goal_update_progress (Issue #127 alias matching autonomous-contract prompt)
+    const handleGoalUpdateProgress = async (args = {}, toolCtx = {}) => {
+      const goalId = args.goalId || args.goal_id || resolveGoalId(toolCtx);
+      const milestoneId = args.milestone_id || args.milestoneId;
+      const status = args.status || 'in_progress';
+      const notes = typeof args.notes === 'string' ? args.notes : '';
+      if (!milestoneId) throw new Error('milestone_id is required');
+      if (status === 'completed') {
+        const rawCwd = toolCtx?.session?.cwd || toolCtx?.cwd || process.cwd();
+        const safeCwd = getSafeCwd(rawCwd);
+        const completed = await mm.completeMilestone(goalId, milestoneId, notes, safeCwd);
+        return { ok: true, goalId, milestoneId, status: 'completed', milestone: completed };
+      }
+      const updated = mm.updateMilestone(goalId, milestoneId, { status, notes, checklist: args.checklist });
+      return { ok: true, goalId, milestoneId, status, milestone: updated };
+    };
+
+    safeRegister({
+      name: 'goal_update_progress',
+      description: 'Update progress and status of a milestone.',
+      parameters: {
+        type: 'object',
+        properties: {
+          milestone_id: { type: 'string', description: 'Target milestone ID' },
+          status: { type: 'string', enum: ['pending', 'in_progress', 'completed', 'blocked'], description: 'Milestone status' },
+          notes: { type: 'string', description: 'Brief outcome or status notes' },
+          checklist: { type: 'array', items: { type: 'string' }, description: 'Subtask items' },
+        },
+        required: ['milestone_id', 'status'],
+      },
+      output: JSON_OUTPUT,
+      execute: handleGoalUpdateProgress,
+    });
+
+    // Tool 5: goal_finish (Issue #127 alias matching autonomous-contract prompt)
+    const handleGoalFinish = async (args = {}, toolCtx = {}) => {
+      const goalId = args.goalId || args.goal_id || resolveGoalId(toolCtx);
+      const summary = typeof args.summary === 'string' ? args.summary : '';
+      if (goalsService && typeof goalsService.complete === 'function') {
+        try { goalsService.complete(toolCtx?.agent, summary); } catch (e) { /* ignore */ }
+      }
+      return { ok: true, goalId, completed: true, summary };
+    };
+
+    safeRegister({
+      name: 'goal_finish',
+      description: 'Finish active goal with final achieved outcome summary.',
+      parameters: {
+        type: 'object',
+        properties: {
+          summary: { type: 'string', description: 'Comprehensive summary of results' },
+        },
+        required: ['summary'],
+      },
+      output: JSON_OUTPUT,
+      execute: handleGoalFinish,
     });
   });
 }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@goodandready/dsh-goal",
-  "version": "0.2.20",
+  "version": "0.2.22",
   "description": "Core Goal Addon (Token Budget, Milestones, Git Snapshots & Retrospective Reports) for DeepSeek Harness",
   "type": "module",
   "main": "./lib/index.js",
@@ -55,18 +55,9 @@
   "peerDependencies": {
     "@deepseek-ai/cordis": "^4.0.1",
     "@deepseek-ai/dsh-host-webserver": "^0.1.7-rc.2 || ^0.2.0-rc.1",
-    "@deepseek-ai/schemastery": "^3.18.4",
-    "@deepseek-ai/dsh-goal": "^0.1.7-rc.2 || ^0.2.0-rc.1"
-  },
-  "devDependencies": {
     "@deepseek-ai/schemastery": "^3.18.4"
   },
-  "dependencies": {
+  "devDependencies": {
     "@deepseek-ai/schemastery": "^3.18.4"
-  },
-  "peerDependenciesMeta": {
-    "@deepseek-ai/dsh-goal": {
-      "optional": true
-    }
   }
 }

package/lib/card-form-state.js

@@ -1,147 +0,0 @@
-/**
- * Pure state and validation logic for dsh-goal settings form.
- * Independent from DOM/React, 100% testable via node --test.
- */
-
-export const DEFAULT_SETTINGS = {
-  maxIterations: 25,
-  autoDrive: true,
-  enableSound: true,
-  showQuickLaunchButton: true,
-  consecutiveToolFailureLimit: 3,
-  enableBrowserNotifications: true,
-  maxTokenBudget: 0,
-  budgetWarningThreshold: 80,
-  autoCheckpointOnMilestone: false,
-  storagePath: '',
-};
-
-export const NUMERIC_FIELDS = {
-  maxIterations: 1,
-  consecutiveToolFailureLimit: 0,
-  maxTokenBudget: 0,
-  budgetWarningThreshold: 0,
-};
-
-/**
- * Parse numeric settings field:
- * - empty string / null / undefined -> undefined (reset override)
- * - valid integer >= min -> number
- * - otherwise -> NaN (invalid)
- * @param {any} raw
- * @param {number} [min=1]
- * @returns {number|undefined}
- */
-export function parseNumberField(raw, min = 1) {
-  if (raw === '' || raw === null || raw === undefined) {
-    return undefined;
-  }
-  const n = Number(raw);
-  if (!Number.isFinite(n) || n < min || !Number.isInteger(n)) {
-    return NaN;
-  }
-  return n;
-}
-
-/**
- * Compute status for a specific setting field
- * @param {string} field
- * @param {any} draftValue
- * @param {Object} snap - { value, base, user }
- */
-export function getFieldStatus(field, draftValue, snap) {
-  const baseVal = snap?.base?.[field] ?? DEFAULT_SETTINGS[field];
-  const userVal = snap?.user?.[field];
-  const effectiveVal = snap?.value?.[field] ?? baseVal;
-
-  const isOverridden = userVal !== undefined && userVal !== null;
-  const currentVal = draftValue !== undefined ? draftValue : effectiveVal;
-
-  let invalid = false;
-  let isDirty = false;
-
-  if (field in NUMERIC_FIELDS) {
-    if (draftValue !== undefined) {
-      if (draftValue === '') {
-        invalid = false;
-        isDirty = userVal !== undefined;
-      } else {
-        const min = NUMERIC_FIELDS[field];
-        const parsed = parseNumberField(draftValue, min);
-        if (Number.isNaN(parsed)) {
-          invalid = true;
-          isDirty = true;
-        } else {
-          isDirty = parsed !== (userVal !== undefined ? userVal : baseVal);
-        }
-      }
-    }
-  } else if (field === 'storagePath') {
-    if (draftValue !== undefined) {
-      const cleanVal = typeof draftValue === 'string' ? draftValue.trim() : '';
-      isDirty = cleanVal !== (userVal !== undefined ? userVal : (baseVal || ''));
-    }
-  } else {
-    // boolean fields (autoDrive, enableSound, showQuickLaunchButton, enableBrowserNotifications, autoCheckpointOnMilestone)
-    if (draftValue !== undefined) {
-      isDirty = Boolean(draftValue) !== (userVal !== undefined ? Boolean(userVal) : Boolean(baseVal));
-    }
-  }
-
-  return {
-    value: currentVal,
-    baseValue: baseVal,
-    userValue: userVal,
-    isOverridden,
-    isDirty,
-    invalid,
-  };
-}
-
-/**
- * Compute list of key-value writes for scope.set
- * @param {Object|null} draft
- * @param {Object} snap - { value, base, user }
- * @returns {Array<[string, any]>}
- */
-export function computeSavePlan(draft, snap) {
-  if (!draft) return [];
-
-  const writes = [];
-  const fields = [
-    'maxIterations',
-    'autoDrive',
-    'enableSound',
-    'showQuickLaunchButton',
-    'consecutiveToolFailureLimit',
-    'enableBrowserNotifications',
-    'maxTokenBudget',
-    'budgetWarningThreshold',
-    'autoCheckpointOnMilestone',
-    'storagePath',
-  ];
-
-  for (const f of fields) {
-    if (draft[f] === undefined) continue;
-
-    const status = getFieldStatus(f, draft[f], snap);
-    if (status.invalid) continue;
-
-    if (status.isDirty) {
-      if (f in NUMERIC_FIELDS) {
-        if (draft[f] === '') {
-          writes.push([f, undefined]);
-        } else {
-          writes.push([f, parseNumberField(draft[f], NUMERIC_FIELDS[f])]);
-        }
-      } else if (f === 'storagePath') {
-        const clean = typeof draft[f] === 'string' ? draft[f].trim() : '';
-        writes.push([f, clean ? clean : undefined]);
-      } else {
-        writes.push([f, Boolean(draft[f])]);
-      }
-    }
-  }
-
-  return writes;
-}

package/lib/command-handler.js

@@ -1,242 +0,0 @@
-import { randomUUID } from 'node:crypto';
-import { GoalState, detectLanguage } from './goal-engine.js';
-
-export const USAGE_EN = 'Usage: /goal [<goal>|clear|pause|resume]';
-export const USAGE_ZH = '用法: /goal [<目标描述>|clear|pause|resume]';
-
-/**
- * Parse input string for /goal command
- * @param {string} rawInput
- * @returns {{ action: string, text?: string }}
- */
-export function parseGoalInput(rawInput = '') {
-  const input = String(rawInput || '').trim();
-  if (!input) {
-    return { action: 'show' };
-  }
-
-  const lower = input.toLowerCase();
-  if (lower === 'clear') return { action: 'clear' };
-  if (lower === 'pause') return { action: 'pause' };
-  if (lower === 'resume') return { action: 'resume' };
-
-  return { action: 'start', text: input };
-}
-
-/**
- * Create a valid identified user message for agent followup
- * @param {string} text
- * @returns {Object}
- */
-export function createGoalUserMessage(text) {
-  return {
-    id: randomUUID(),
-    role: 'user',
-    content: [{ type: 'text', text }],
-    source: { kind: 'user' },
-  };
-}
-
-/**
- * Format goal start prompt with mandatory milestone contract
- * @param {string} goalText
- * @param {string} [explicitLang]
- * @returns {string}
- */
-export function formatGoalStartPrompt(goalText, explicitLang) {
-  const lang = explicitLang || detectLanguage(goalText);
-  if (lang === 'zh') {
-    return `🎯 目标模式已激活 (Goal Mode): "${goalText}"\n\n` +
-      `自主执行契约：\n` +
-      `1. 第一步必选动作：在开展任何工作或回复之前，你的第一个动作必须是调用 goal_set_milestones 工具，提交包含 3-7 个具体连续步骤的工作计划。用户将在界面中实时查看此计划。\n` +
-      `2. 第二步：计划确立后开始执行任务。在开始每个步骤前，通过 goal_update_progress(milestone_id, "in_progress") 标记其为进行中。\n` +
-      `3. 第三步：完成某一步骤后，通过 goal_update_progress(milestone_id, "completed", "阶段简要成果") 标记为已完成。\n` +
-      `4. 第四步：当所有里程碑全部完成后，调用 goal_finish(summary) 工具并附带详尽的最终成果总结。`;
-  }
-
-  return `🎯 Goal Mode activated: "${goalText}"\n\n` +
-    `STRICT AUTONOMOUS CONTRACT:\n` +
-    `1. MANDATORY STEP 1: Your FIRST action BEFORE doing any other work or reply MUST be calling tool goal_set_milestones with an array of 3-7 concrete, sequential milestones. The user sees this plan in real time.\n` +
-    `2. STEP 2: After establishing the plan, proceed to execute milestones. Before starting each milestone, mark it as in_progress via goal_update_progress(milestone_id, "in_progress").\n` +
-    `3. STEP 3: Upon completing a milestone, mark it as completed via goal_update_progress(milestone_id, "completed", "brief milestone outcome").\n` +
-    `4. STEP 4: When all milestones are completed, call tool goal_finish(summary) with a comprehensive summary of achieved results.`;
-}
-
-/**
- * Format response for UI and manage agent task execution
- * @param {import('./goal-engine.js').GoalEngine} engine
- * @param {{ action: string, text?: string }} parsed
- * @param {Object} [config]
- * @param {Object} [agent]
- * @param {string} [sessionId='default']
- * @returns {{ kind: string, text: string }}
- */
-export function executeGoalSlashCommand(engine, parsed, config = {}, agent = null, sessionId = 'default') {
-  const sid = sessionId || 'default';
-  const snap = engine.getGoalSnapshot(sid);
-  const lang = snap.lang || (parsed.text ? detectLanguage(parsed.text) : 'en');
-  const isZh = lang === 'zh';
-  const usage = isZh ? USAGE_ZH : USAGE_EN;
-
-  switch (parsed.action) {
-    case 'show': {
-      if (!snap.hasActiveGoal) {
-        return {
-          kind: 'success',
-          text: isZh ? `目标模式: 当前未设置活动目标。\n${usage}` : `Goal Mode: no active goal set.\n${usage}`,
-        };
-      }
-
-      const milestonesInfo = snap.milestones && snap.milestones.length > 0
-        ? `\n\n${isZh ? '工作计划' : 'Work Plan'}:\n` +
-          snap.milestones
-            .map((m, i) => `  ${i + 1}. [${m.status.toUpperCase()}] ${m.title}${m.notes ? ` (${m.notes})` : ''}`)
-            .join('\n')
-        : '';
-
-      const etaText = snap.formattedETA ? ` (ETA: ${snap.formattedETA})` : '';
-      const tokensInfo = (snap.tokensUsage && snap.tokensUsage.totalTokens > 0)
-        ? (isZh
-            ? `\nToken: ${snap.tokensUsage.totalTokens.toLocaleString()} (输入: ${snap.tokensUsage.promptTokens.toLocaleString()}, 输出: ${snap.tokensUsage.completionTokens.toLocaleString()})`
-            : `\nTokens: ${snap.tokensUsage.totalTokens.toLocaleString()} (in: ${snap.tokensUsage.promptTokens.toLocaleString()}, out: ${snap.tokensUsage.completionTokens.toLocaleString()})`)
-        : '';
-
-      return {
-        kind: 'success',
-        text: isZh
-          ? `🎯 目标: "${snap.title}"\n` +
-            `状态: ${snap.state}\n` +
-            `耗时: ${snap.formattedElapsed}${etaText}\n` +
-            `迭代轮次: ${snap.iterationsCount}/${snap.maxIterations}` +
-            tokensInfo +
-            milestonesInfo +
-            `\n\n快捷指令: /goal pause, /goal resume, /goal clear, /goal <新目标>`
-          : `🎯 Goal: "${snap.title}"\n` +
-            `Status: ${snap.state}\n` +
-            `Duration: ${snap.formattedElapsed}${etaText}\n` +
-            `Iterations: ${snap.iterationsCount}/${snap.maxIterations}` +
-            tokensInfo +
-            milestonesInfo +
-            `\n\nCommands: /goal pause, /goal resume, /goal clear, /goal <new goal>`,
-      };
-    }
-
-    case 'clear': {
-      if (!snap.hasActiveGoal) {
-        return {
-          kind: 'success',
-          text: isZh ? '当前未设置目标。' : 'No active goal set.',
-        };
-      }
-      engine.clear(sid);
-      if (agent && typeof agent.cancel === 'function') {
-        try {
-          agent.cancel({ kind: 'user', reason: 'Goal cleared by user' });
-        } catch (err) {
-          // agent cancellation error handled defensively
-        }
-      }
-      return {
-        kind: 'success',
-        text: isZh ? '🎯 目标已清除。' : '🎯 Goal cleared.',
-      };
-    }
-
-    case 'pause': {
-      if (!snap.hasActiveGoal) {
-        return {
-          kind: 'error',
-          text: isZh
-            ? `无法暂停：未设置活动目标。\n${usage}`
-            : `Cannot pause: no active goal.\n${usage}`,
-        };
-      }
-      if (snap.state === GoalState.PAUSED) {
-        return {
-          kind: 'success',
-          text: isZh ? '⏸ 目标当前已处于暂停状态。' : '⏸ Goal is already paused.',
-        };
-      }
-      engine.pause(isZh ? '用户命令暂停 (/goal pause)' : 'Paused by user command (/goal pause)', sid);
-      if (agent && typeof agent.cancel === 'function') {
-        try {
-          agent.cancel({ kind: 'user', reason: 'Goal paused by user' });
-        } catch (err) {
-          // agent cancellation error handled defensively
-        }
-      }
-      return {
-        kind: 'success',
-        text: isZh
-          ? `⏸ 目标已暂停: "${snap.title}"。智能体已停止，使用 /goal resume 或 ▶️ 按钮继续。`
-          : `⏸ Goal paused: "${snap.title}". Agent stopped. Use /goal resume or ▶️ button to continue.`,
-      };
-    }
-
-    case 'resume': {
-      if (!snap.hasActiveGoal) {
-        return {
-          kind: 'error',
-          text: isZh
-            ? `无法恢复：未设置活动目标。\n${usage}`
-            : `Cannot resume: no active goal.\n${usage}`,
-        };
-      }
-      if (snap.state === GoalState.RUNNING) {
-        return {
-          kind: 'success',
-          text: isZh ? '▶️ 目标已在运行中。' : '▶️ Goal is already running.',
-        };
-      }
-      engine.resume(sid);
-      if (agent && typeof agent.followup === 'function') {
-        try {
-          const resumePrompt = isZh
-            ? `▶️ 用户已恢复目标运行。请继续推进工作计划中尚未完成的步骤。通过 goal_update_progress 实时更新进度。`
-            : `▶️ Goal resumed by user. Continue executing the work plan from where you stopped. Update steps via goal_update_progress.`;
-          agent.followup(createGoalUserMessage(resumePrompt));
-        } catch (err) {
-          // agent followup error handled defensively
-        }
-      }
-      return {
-        kind: 'success',
-        text: isZh
-          ? `▶️ 目标已恢复: "${snap.title}"。智能体继续工作中。`
-          : `▶️ Goal resumed: "${snap.title}". Agent continues working.`,
-      };
-    }
-
-    case 'start': {
-      const goalLang = detectLanguage(parsed.text);
-      const isZhStart = goalLang === 'zh';
-      const maxIterations = config.maxIterations ?? snap.maxIterations ?? 25;
-      const cwd = config.cwd;
-      const newSnap = engine.startGoal(parsed.text, { maxIterations, lang: goalLang, cwd }, sid);
-
-      if (agent && typeof agent.followup === 'function') {
-        try {
-          const userMsg = createGoalUserMessage(formatGoalStartPrompt(parsed.text, goalLang));
-          agent.followup(userMsg);
-        } catch (err) {
-          // agent dispatch error handled defensively
-        }
-      }
-
-      return {
-        kind: 'success',
-        text: isZhStart
-          ? `🎯 目标已激活: "${newSnap.title}" (上限 ${newSnap.maxIterations} 轮迭代)。\n` +
-            `暂停: /goal pause | 清除: /goal clear`
-          : `🎯 Goal activated: "${newSnap.title}" (max ${newSnap.maxIterations} iterations).\n` +
-            `To pause: /goal pause | To clear: /goal clear`,
-      };
-    }
-
-    default:
-      return {
-        kind: 'error',
-        text: isZh ? `未知指令。${usage}` : `Unknown command. ${usage}`,
-      };
-  }
-}