codemini-cli 0.5.8 → 0.5.10

package/package.json

@@ -1,67 +1,67 @@
-{
-  "name": "codemini-cli",
-  "version": "0.5.8",
-  "description": "Coding CLI optimized for small-model workflows and Windows PowerShell",
-  "keywords": [
-    "cli",
-    "ai",
-    "coding-assistant",
-    "developer-tools",
-    "terminal",
-    "windows",
-    "powershell"
-  ],
-  "type": "module",
-  "homepage": "https://github.com/havingautism/Codemini-CLI#readme",
-  "bugs": {
-    "url": "https://github.com/havingautism/Codemini-CLI/issues"
-  },
-  "repository": {
-    "type": "git",
-    "url": "git+https://github.com/havingautism/Codemini-CLI.git"
-  },
-  "bin": {
-    "codemini": "bin/coder.js",
-    "coder": "bin/coder.js"
-  },
-  "scripts": {
-    "start": "node bin/coder.js",
-    "test": "node --test tests/*.test.js",
-    "build:web": "npm install --prefix codemini-web && npm run build --prefix codemini-web",
-    "prepack": "npm run build:web",
-    "pack:offline": "npm pack",
-    "bump:patch": "npm version patch --no-git-tag-version",
-    "bump:minor": "npm version minor --no-git-tag-version",
-    "bump:major": "npm version major --no-git-tag-version"
-  },
-  "files": [
-    "bin",
-    "src",
-    "codemini-web/server.js",
-    "codemini-web/lib",
-    "codemini-web/dist",
-    "codemini-web/codemini_logo.png",
-    "souls",
-    "templates",
-    "skills",
-    "README.md",
-    "OPERATIONS.md",
-    "deployment.md"
-  ],
-  "engines": {
-    "node": ">=22"
-  },
-  "publishConfig": {
-    "access": "public"
-  },
-  "dependencies": {
-    "@cursorless/tree-sitter-wasms": "^0.8.1",
-    "cheerio": "^1.1.2",
-    "cli-truncate": "^6.0.0",
-    "ink": "^7.0.0",
-    "react": "^19.2.5",
-    "strip-ansi": "^7.2.0",
-    "web-tree-sitter": "^0.26.8"
-  },
-  "license": "MIT"
-}
+{
+  "name": "codemini-cli",
+  "version": "0.5.10",
+  "description": "Coding CLI optimized for small-model workflows and Windows PowerShell",
+  "keywords": [
+    "cli",
+    "ai",
+    "coding-assistant",
+    "developer-tools",
+    "terminal",
+    "windows",
+    "powershell"
+  ],
+  "type": "module",
+  "homepage": "https://github.com/havingautism/Codemini-CLI#readme",
+  "bugs": {
+    "url": "https://github.com/havingautism/Codemini-CLI/issues"
+  },
+  "repository": {
+    "type": "git",
+    "url": "git+https://github.com/havingautism/Codemini-CLI.git"
+  },
+  "bin": {
+    "codemini": "bin/coder.js",
+    "coder": "bin/coder.js"
+  },
+  "scripts": {
+    "start": "node bin/coder.js",
+    "test": "node --test tests/*.test.js",
+    "build:web": "npm install --prefix codemini-web && npm run build --prefix codemini-web",
+    "prepack": "npm run build:web",
+    "pack:offline": "npm pack",
+    "bump:patch": "npm version patch --no-git-tag-version",
+    "bump:minor": "npm version minor --no-git-tag-version",
+    "bump:major": "npm version major --no-git-tag-version"
+  },
+  "files": [
+    "bin",
+    "src",
+    "codemini-web/server.js",
+    "codemini-web/lib",
+    "codemini-web/dist",
+    "codemini-web/codemini_logo.png",
+    "souls",
+    "templates",
+    "skills",
+    "README.md",
+    "OPERATIONS.md",
+    "deployment.md"
+  ],
+  "engines": {
+    "node": ">=22"
+  },
+  "publishConfig": {
+    "access": "public"
+  },
+  "dependencies": {
+    "@cursorless/tree-sitter-wasms": "^0.8.1",
+    "cheerio": "^1.1.2",
+    "cli-truncate": "^6.0.0",
+    "ink": "^7.0.0",
+    "react": "^19.2.5",
+    "strip-ansi": "^7.2.0",
+    "web-tree-sitter": "^0.26.8"
+  },
+  "license": "MIT"
+}

package/skills/brainstorm/SKILL.md

@@ -1,7 +1,7 @@
 ---
 name: brainstorm
 description: Lightweight brainstorming skill. Use when a feature or behavior request has multiple reasonable approaches and the missing piece is user preference, tradeoff choice, or key constraint.
-version: 0.2.0
+version: 0.2.1
 ---
 
 Use this skill only when the task needs clarification or option comparison before coding.
@@ -18,6 +18,7 @@ Do NOT skip this skill for tasks that appear straightforward but lack clear requ
 2. **Give 2-3 short options** only when the blocking constraint is already clear. Keep options concrete and focused on the main tradeoff.
 3. **Present conclusions as suggested decisions**, not final choices.
 4. **Do NOT write code, pseudo-code, file edits, or broad repo exploration** while direction is still being chosen.
+5. **Stop after your brainstorm response.** Do not say "I will start", "starting now", "I'll edit", or otherwise transition into implementation in the same turn.
 
 ## Output Formats
 
@@ -54,6 +55,8 @@ Suggested decision:
 - reason: <why>
 ```
 
+After Mode B, STOP. Wait for the user to approve, reject, or revise the suggested decision.
+
 ## Self-Review
 
 Before presenting options or a suggested decision, quickly check:
@@ -64,9 +67,11 @@ Before presenting options or a suggested decision, quickly check:
 
 ## Exit
 
-After the user approves a direction:
+Brainstorm ends only when the user sends a later message that clearly approves a direction, for example "use option 2", "按这个做", "确认，开始实现", or "直接写代码".
+
+After that later user approval:
 
 - If the task is small and clear enough to implement directly → proceed to code.
 - If the task is non-trivial or touches multiple areas → YOU MUST invoke `writing-plans` to create an implementation plan before coding.
 
-Do NOT stop at the brainstorm conclusion when the natural next step is planning.
+Do NOT treat your own suggested decision as approval. Do NOT continue from the brainstorm conclusion into planning or implementation until the user has explicitly approved a direction in a separate message.

package/skills/codemini.skills.json

@@ -0,0 +1,40 @@
+{
+  "version": 1,
+  "skills": {
+    "superpowers-lite": {
+      "description": "Default concise workflow for coding tasks; keep context tight, choose the right route, and verify before claiming success.",
+      "mode": "always",
+      "triggers": [],
+      "enabled": true,
+      "priority": 100
+    },
+    "brainstorm": {
+      "description": "Use when a feature or behavior request has multiple reasonable approaches and the missing piece is user preference, tradeoff choice, or key constraint.",
+      "mode": "agent_requested",
+      "triggers": [],
+      "enabled": true,
+      "priority": 70
+    },
+    "writing-plans": {
+      "description": "Use when you have a clear goal or approved design for a non-trivial task, before touching code.",
+      "mode": "agent_requested",
+      "triggers": [],
+      "enabled": true,
+      "priority": 75
+    },
+    "grill-me": {
+      "description": "Use when the user explicitly asks to pressure-test a plan, architecture choice, PR, launch, or product idea.",
+      "mode": "agent_requested",
+      "triggers": [],
+      "enabled": true,
+      "priority": 65
+    },
+    "project-requirements": {
+      "description": "Create a project requirements report with repository inspection and structured output artifacts.",
+      "mode": "manual",
+      "triggers": [],
+      "enabled": true,
+      "priority": 50
+    }
+  }
+}

package/src/commands/skill.js

@@ -77,9 +77,15 @@ export async function listSkillEntries({ scope = 'all', cwd = process.cwd() } =
       name: command.name,
       version: command.metadata?.version || '0.0.0',
       description: command.metadata?.description || '',
+      mode: command.metadata?.mode || '',
+      triggers: Array.isArray(command.metadata?.triggers) ? command.metadata.triggers : [],
       scope: itemScope,
       path: command.path,
-      enabled: itemScope === 'builtin' ? true : config.skills?.enabled?.[command.name] !== false
+      enabled: command.metadata?.enabled === false
+        ? false
+        : itemScope === 'builtin'
+          ? true
+          : config.skills?.enabled?.[command.name] !== false
     });
   }
   return entries.sort((a, b) => `${a.scope}:${a.name}`.localeCompare(`${b.scope}:${b.name}`));
@@ -93,14 +99,19 @@ async function readSkillMeta(name, { scope = 'all', cwd = process.cwd() } = {})
   }
   const dir = path.dirname(found.path);
   const manifestPath = path.join(dir, 'manifest.json');
+  const catalogPath = path.join(path.dirname(dir), 'codemini.skills.json');
   let manifest = null;
   try {
-    manifest = JSON.parse(await fs.readFile(manifestPath, 'utf8'));
+    const catalog = JSON.parse(await fs.readFile(catalogPath, 'utf8'));
+    manifest = catalog?.skills?.[found.name] || null;
   } catch {
-    manifest = null;
+    try {
+      manifest = JSON.parse(await fs.readFile(manifestPath, 'utf8'));
+    } catch {
+      manifest = null;
+    }
   }
-  const entryFile = manifest?.entry || 'SKILL.md';
-  const skillPath = found.path || path.join(dir, entryFile);
+  const skillPath = found.path || path.join(dir, 'SKILL.md');
   try {
     const content = await fs.readFile(skillPath, 'utf8');
     const firstLines = content.split('\n').slice(0, 20).join('\n');

package/src/core/ast.js

@@ -4,6 +4,7 @@ import { createRequire } from 'node:module';
 import { Parser, Language, Query } from 'web-tree-sitter';
 import { LANGUAGE_ALIASES, EXTENSION_LANGUAGE_MAP } from './constants.js';
 import { sha256Prefixed as sha256 } from './crypto-utils.js';
+import { BoundedCache } from './bounded-cache.js';
 
 const require = createRequire(import.meta.url);
 
@@ -43,7 +44,7 @@ const parserInitPromise = Parser.init({
     return scriptName === 'web-tree-sitter.wasm' ? TREE_SITTER_WASM_PATH : scriptName;
   }
 });
-const languageCache = new Map();
+const languageCache = new BoundedCache({ maxSize: 16, ttlMs: 60 * 60 * 1000 });
 
 function clipText(text, maxLen = 220) {
   const normalized = String(text || '').replace(/\s+/g, ' ').trim();
@@ -113,15 +114,34 @@ async function loadLanguage(language) {
   if (languageCache.has(language)) return languageCache.get(language);
   const wasmPath = LANGUAGE_WASM_PATHS[language];
   if (!wasmPath) throw new Error(`Unsupported Tree-sitter language: ${language}`);
-  const loaded = await Language.load(wasmPath);
-  languageCache.set(language, loaded);
-  return loaded;
+  const loadPromise = Language.load(wasmPath);
+  languageCache.set(language, loadPromise);
+  try {
+    return await loadPromise;
+  } catch (error) {
+    languageCache.delete(language);
+    throw error;
+  }
 }
 
-async function parseContent(content, language) {
+async function getParser(language) {
   const loadedLanguage = await loadLanguage(language);
   const parser = new Parser();
   parser.setLanguage(loadedLanguage);
+  return { parser, loadedLanguage };
+}
+
+function deleteParsed(parsed) {
+  try {
+    parsed?.tree?.delete?.();
+  } catch {}
+  try {
+    parsed?.parser?.delete?.();
+  } catch {}
+}
+
+async function parseContent(content, language) {
+  const { parser, loadedLanguage } = await getParser(language);
   const tree = parser.parse(content);
   return { parser, tree, loadedLanguage };
 }
@@ -188,8 +208,7 @@ export async function findEnclosingSymbol(content, filePath, line) {
   } catch {
     return null;
   } finally {
-    if (tree) tree.delete();
-    if (parser) parser.delete();
+    deleteParsed({ tree, parser });
   }
 }
 
@@ -223,8 +242,7 @@ export async function queryAst(root, args) {
   }
 
   query.delete();
-  parsed.tree.delete();
-  parsed.parser.delete();
+  deleteParsed(parsed);
 
   return {
     path: relativePath,
@@ -261,8 +279,7 @@ export async function readAstNode(root, args) {
     child_summaries: node.namedChildren.slice(0, 8).map((child) => summarizeNode(child))
   };
 
-  parsed.tree.delete();
-  parsed.parser.delete();
+  deleteParsed(parsed);
   return result;
 }
 
@@ -277,15 +294,13 @@ export async function resolveAstTarget(root, relativePath, astTarget) {
   const parsed = await parseFile(root, relativePath, astTarget.language);
   const node = exactNodeForTarget(parsed.tree.rootNode, astTarget);
   if (!node) {
-    parsed.tree.delete();
-    parsed.parser.delete();
+    deleteParsed(parsed);
     throw new Error('AST target no longer matches the current file');
   }
 
   const currentHash = sha256(node.text);
   if (String(astTarget.range_hash || '') !== currentHash) {
-    parsed.tree.delete();
-    parsed.parser.delete();
+    deleteParsed(parsed);
     throw new Error('ast_target range_hash mismatch; the selected node changed and is now stale');
   }
 

package/src/core/chat-runtime.js

@@ -128,6 +128,8 @@ function getCompletionCopy(language = 'zh') {
         'context.prompt_budget_audit': 'Prompt 预算审计开关',
         'context.microcompact_enabled': '微压缩(micro-compact)开关',
         'context.microcompact_keep_recent': '微压缩保留最近工具结果数',
+        'context.project_instructions_enabled': '项目 AGENTS.md 注入开关',
+        'context.project_instructions_max_chars': '项目 AGENTS.md 字符上限',
         'sessions.max_sessions': '会话保留上限',
         'sessions.retention_days': '会话保留天数',
         'shell.default': '默认 shell',
@@ -136,6 +138,7 @@ function getCompletionCopy(language = 'zh') {
         'soul.preset': 'soul 预设',
         'soul.custom_path': '自定义 soul 路径',
         'policy.safe_mode': '安全模式开关',
+        'policy.allowed_paths': '安全模式目录白名单',
         'policy.allow_dangerous_commands': '危险命令开关'
       },
       optionHints: {
@@ -145,8 +148,11 @@ function getCompletionCopy(language = 'zh') {
         'execution.mode': '可选：auto | normal | plan',
         'shell.default': '常用：bash | powershell',
         'policy.safe_mode': '可选：true | false',
+        'policy.allowed_paths': 'JSON 数组，例如 ["D:\\\\shared"]',
         'policy.allow_dangerous_commands': '可选：true | false',
-        'context.prompt_budget_audit': '可选：true | false'
+        'context.prompt_budget_audit': '可选：true | false',
+        'context.project_instructions_enabled': '可选：true | false',
+        'context.project_instructions_max_chars': '建议：8000-12000'
       },
       describeSet: (label, hint) => `设置${label}${hint ? `（${hint}）` : ''}`,
       describeGet: (label, hint) => `查看${label}${hint ? `（${hint}）` : ''}`,
@@ -235,6 +241,8 @@ function getCompletionCopy(language = 'zh') {
         'context.prompt_budget_audit': 'prompt budget audit switch',
         'context.microcompact_enabled': 'micro-compact enabled',
         'context.microcompact_keep_recent': 'micro-compact keep recent tool results',
+        'context.project_instructions_enabled': 'project AGENTS.md injection switch',
+        'context.project_instructions_max_chars': 'project AGENTS.md character limit',
         'sessions.max_sessions': 'stored session limit',
         'sessions.retention_days': 'session retention days',
         'shell.default': 'default shell',
@@ -243,6 +251,7 @@ function getCompletionCopy(language = 'zh') {
         'soul.preset': 'soul preset',
         'soul.custom_path': 'custom soul prompt path',
         'policy.safe_mode': 'safe mode switch',
+        'policy.allowed_paths': 'safe-mode allowed path roots',
         'policy.allow_dangerous_commands': 'dangerous command allowance'
       },
       optionHints: {
@@ -252,8 +261,11 @@ function getCompletionCopy(language = 'zh') {
         'execution.mode': 'options: auto | normal | plan',
         'shell.default': 'common: bash | powershell',
         'policy.safe_mode': 'options: true | false',
+        'policy.allowed_paths': 'JSON array, for example ["D:\\\\shared"]',
         'policy.allow_dangerous_commands': 'options: true | false',
-        'context.prompt_budget_audit': 'options: true | false'
+        'context.prompt_budget_audit': 'options: true | false',
+        'context.project_instructions_enabled': 'options: true | false',
+        'context.project_instructions_max_chars': 'recommended: 8000-12000'
       },
       describeSet: (label, hint) => `set the ${label}${hint ? ` (${hint})` : ''}`,
       describeGet: (label, hint) => `show the ${label}${hint ? ` (${hint})` : ''}`,
@@ -1265,6 +1277,7 @@ function isBundledSkillCommand(command) {
 }
 
 function isSkillEnabled(config, name, command = null) {
+  if (command?.metadata?.enabled === false) return false;
   if (isBundledSkillCommand(command)) return true;
   return config.skills?.enabled?.[name] !== false;
 }
@@ -2282,7 +2295,10 @@ function summarizePromptBudgetAudit(audit) {
 }
 
 function buildRuntimeStateSnapshot({ currentSession, config, model, executionMode, extraSession }) {
-  const parentTokens = estimateMessagesTokens(currentSession?.messages || []);
+  const activeParentMessages = Array.isArray(currentSession?.compact?.view) && currentSession.compact.view.length > 0
+    ? currentSession.compact.view
+    : currentSession?.messages || [];
+  const parentTokens = estimateMessagesTokens(activeParentMessages);
   const subTokens = extraSession ? estimateMessagesTokens(extraSession.messages || []) : 0;
   const currentContextTokens = parentTokens + subTokens;
   const maxContextTokens = effectiveMaxContextTokens(config);
@@ -2301,6 +2317,9 @@ function buildRuntimeStateSnapshot({ currentSession, config, model, executionMod
     maxContextTokens,
     pendingPlanApproval: planState?.status === 'pending_approval'
       ? { goal: planState.goal, summary: planState.finalSummary || planState.summary, filePath: planState.filePath, steps: planState.steps || [] }
+      : null,
+    pendingReflectSkill: planState?.status === 'pending_reflect_skill'
+      ? buildPendingReflectSkillSnapshot(planState)
       : null
   };
   Object.defineProperties(snapshot, {
@@ -2314,11 +2333,6 @@ function buildRuntimeStateSnapshot({ currentSession, config, model, executionMod
       enumerable: false,
       writable: false
     },
-    pendingReflectSkill: {
-      value: currentSession?.planState?.status === 'pending_reflect_skill',
-      enumerable: false,
-      writable: false
-    },
     replyLanguage: {
       value: getReplyLanguage(config),
       enumerable: false,
@@ -2329,7 +2343,6 @@ function buildRuntimeStateSnapshot({ currentSession, config, model, executionMod
         ...snapshot,
         currentContextTokens,
         contextUsagePct,
-        pendingReflectSkill: currentSession?.planState?.status === 'pending_reflect_skill',
         replyLanguage: getReplyLanguage(config)
       }),
       enumerable: false,
@@ -2513,6 +2526,21 @@ function buildPendingReflectSkillMessage(reflectState) {
   return lines.join('\n');
 }
 
+function buildPendingReflectSkillSnapshot(reflectState) {
+  const candidates = Array.isArray(reflectState?.candidates) ? reflectState.candidates : [];
+  const candidate = candidates[0] || null;
+  if (!candidate) return null;
+  return {
+    scope: reflectState?.targetScope || 'project',
+    request: reflectState?.request || '',
+    name: candidate.name || '',
+    description: candidate.description || '',
+    confidence: Number(candidate.confidence ?? 0.75),
+    targetPath: candidate.targetPath || '',
+    content: candidate.content || ''
+  };
+}
+
 function buildApprovedPlanExecutionPrompt(planState, approvalText = '') {
   const requirementPacket = buildGoalRequirementPacket(planState?.goal || '', 'coder');
   const lines = [
@@ -2740,8 +2768,10 @@ async function askModel({
     }
   }
 
+  const shouldGenerateTitle = text
+    ? !session.messages.some((msg) => msg?.role === 'user')
+    : false;
   if (text) {
-    const shouldGenerateTitle = !session.messages.some((msg) => msg?.role === 'user');
     const modelExtra =
       typeof modelText === 'string' && modelText && modelText !== text ? { model_content: modelText } : {};
     const userMessage = stampedMessage('user', text, modelExtra);
@@ -2773,7 +2803,16 @@ async function askModel({
 
   const { definitions, handlers, formatters, deferredDefinitions, dispose: disposeTools } = getBuiltinTools({
     workspaceRoot: process.cwd(),
-    config,
+    config: {
+      ...config,
+      policy: {
+        ...(config.policy || {}),
+        allowed_paths: [
+          ...(Array.isArray(config.policy?.allowed_paths) ? config.policy.allowed_paths : []),
+          path.join(getSessionsDir(), String(session.id))
+        ]
+      }
+    },
     sessionId: session.id,
     onSystemEvent: onAgentEvent,
     getTodos: () => normalizeTodos(session.todos),
@@ -3004,7 +3043,7 @@ async function askModel({
     await flushScheduledSave();
     await saveSession(session);
     // Generate a better title asynchronously after saving
-    if (shouldReplaceSessionTitle(session.title)) {
+    if (shouldGenerateTitle) {
       const titleSessionId = session.id;
       generateSessionTitle({
         userText: text,
@@ -4242,11 +4281,22 @@ export async function createChatRuntime({
   if (hasPendingPlanApproval(currentSession)) {
     executionMode = 'plan';
   }
+  let compactState = null;
+  const normalizeCompactThreshold = (value, fallback = 60) => {
+    const num = Number(value);
+    if (!Number.isFinite(num)) return fallback;
+    return Math.min(95, Math.max(50, num));
+  };
+  const syncCompactStateFromConfig = () => {
+    if (!compactState) return;
+    compactState.threshold = normalizeCompactThreshold(config.context?.preflight_trigger_pct, 60);
+  };
   const syncRuntimeFromConfig = async ({ model: nextModel } = {}) => {
     const configuredMode = String(config.execution?.mode || 'normal');
     executionMode = hasPendingPlanApproval(currentSession)
       ? 'plan'
       : (['normal', 'auto', 'plan'].includes(configuredMode) ? configuredMode : 'normal');
+    syncCompactStateFromConfig();
 
     const resolvedModel = String(nextModel || '').trim();
     if (resolvedModel) {
@@ -4269,10 +4319,10 @@ export async function createChatRuntime({
   // Set up tool result store under session directory
   const sessionResultsDir = path.join(getSessionsDir(), String(currentSession.id));
   setResultDir(sessionResultsDir);
-  const compactState = {
+  compactState = {
     backupMessages: null,
     autoEnabled: true,
-    threshold: 60,
+    threshold: normalizeCompactThreshold(config.context?.preflight_trigger_pct, 60),
     mode: 'conservative'
   };
   let compactedForModel = currentSession.compact?.view || null;
@@ -4349,6 +4399,8 @@ export async function createChatRuntime({
     'context.read_file_max_chars',
     'context.microcompact_enabled',
     'context.microcompact_keep_recent',
+    'context.project_instructions_enabled',
+    'context.project_instructions_max_chars',
     'sessions.max_sessions',
     'sessions.retention_days',
     'shell.timeout_ms',
@@ -4356,6 +4408,7 @@ export async function createChatRuntime({
     'soul.preset',
     'soul.custom_path',
     'policy.safe_mode',
+    'policy.allowed_paths',
     'policy.allow_dangerous_commands'
   ];
 
@@ -4777,6 +4830,11 @@ export async function createChatRuntime({
   };
 
   const persistAssistantExchange = async (userText, assistantText, { includeUser = true, extra = {} } = {}) => {
+    const priorUserCount = currentSession.messages.filter((msg) => msg?.role === 'user').length;
+    const priorAssistantCount = currentSession.messages.filter((msg) => msg?.role === 'assistant').length;
+    const shouldGenerateTitle =
+      (includeUser && userText && priorUserCount === 0) ||
+      (!includeUser && userText && priorUserCount === 1 && priorAssistantCount === 0);
     if (includeUser && userText) {
       appendSessionMessage(stampedMessage('user', userText));
     }
@@ -4787,7 +4845,7 @@ export async function createChatRuntime({
     currentSession.mode = executionMode || config.execution?.mode || 'normal';
     await saveSession(currentSession);
     // Generate a better title asynchronously after saving
-    if (shouldReplaceSessionTitle(currentSession.title)) {
+    if (shouldGenerateTitle || shouldReplaceSessionTitle(currentSession.title)) {
       const titleSessionId = currentSession.id;
       generateSessionTitle({
         userText,
@@ -5115,6 +5173,7 @@ export async function createChatRuntime({
           });
           currentSession.planState = null;
           executionMode = 'auto';
+          if (onAgentEvent) onAgentEvent({ type: 'reflect:approval_cleared' });
           await reloadCommandsAndSkills();
           const text = `Reflect skill written and loaded: /${written.draft.name}\nPath: ${written.filePath}`;
           await persistLocalExchange(line, text, { includeUser: false });
@@ -5172,6 +5231,12 @@ export async function createChatRuntime({
               workspaceRoot: process.cwd()
             })
           };
+          if (onAgentEvent) {
+            onAgentEvent({
+              type: 'reflect:pending_approval',
+              draft: buildPendingReflectSkillSnapshot(currentSession.planState)
+            });
+          }
           const text = `Reflect skill draft revised.\n${buildPendingReflectSkillMessage(currentSession.planState)}`;
           await persistLocalExchange(line, text);
           return { type: 'system', text };
@@ -5209,6 +5274,7 @@ export async function createChatRuntime({
         if (hasPendingReflectSkill(currentSession)) {
           currentSession.planState = null;
           executionMode = 'auto';
+          if (onAgentEvent) onAgentEvent({ type: 'reflect:approval_cleared' });
           const text = 'Reflect skill draft discarded.';
           await persistLocalExchange(line, text, { includeUser: false });
           return { type: 'system', text };
@@ -5668,6 +5734,12 @@ export async function createChatRuntime({
           request: parsedReflect.request,
           candidates
         };
+        if (onAgentEvent) {
+          onAgentEvent({
+            type: 'reflect:pending_approval',
+            draft: buildPendingReflectSkillSnapshot(currentSession.planState)
+          });
+        }
         const text = buildPendingReflectSkillMessage(currentSession.planState);
         await persistLocalExchange(line, text);
         return { type: 'system', text };
@@ -5735,7 +5807,7 @@ export async function createChatRuntime({
           await resetConfig();
           config = await loadConfig();
           await syncRuntimeFromConfig({ model: resolveDefaultModel(config) });
-          compactState.threshold = 60;
+          syncCompactStateFromConfig();
           compactState.mode = 'conservative';
           compactState.autoEnabled = true;
           const text = 'Config reset complete';