principles-disciple 1.65.0 → 1.67.0

package/ADVANCED_CONFIG_ZH.md

@@ -44,10 +44,6 @@
     *   `success_base` (默认: 1): 成功干完一个小任务，加 1 分。
     *   `streak_bonus` (默认: 5): 连续 5 次成功（连杀奖励），额外加 5 分。
 
-### 5. 元认知深度反思 (`deep_reflection`)
-*   `enabled`: 是否允许 AI 停下来深思熟虑。
-*   `auto_trigger_conditions.error_rate_threshold` (默认: 0.3): 当近期操作的错误率超过 30% 时，强制让 AI 停下手头的活，调用 `deep_reflect` 工具分析自己是不是大方向搞错了。
-
 ---
 
 ### ⚠️ 高风险操作警告！

package/README.md

@@ -44,15 +44,6 @@ All commands support **short aliases** for easier input:
 | `/nocturnal-rollout` | Nocturnal rollout and promotion |
 | `/pd-workflow-debug` | Debug workflow state |
 
-### Tools
-
-**`deep_reflect`** - Executes deep meta-cognitive reflection to analyze potential risks, logical gaps, or architectural improvements in the current task.
-
-Parameters:
-- `context` (required): Task context, code snippet, or current difficulty
-- `depth` (optional): Reflection depth 1-3 (default: 2)
-- `model_id` (optional): Force specific thinking model
-
 ### Configuration
 
 The plugin accepts the following configuration options:
@@ -62,8 +53,6 @@ The plugin accepts the following configuration options:
 | `language` | `zh` | Interaction language (`en` or `zh`) |
 | `auditLevel` | `medium` | Security guardrail level (`low`, `medium`, `high`) |
 | `riskPaths` | `[]` | High-risk directories requiring explicit authorization |
-| `deep_reflection.enabled` | `true` | Enable AI deep reflection |
-| `deep_reflection.mode` | `auto` | Reflection trigger mode (`auto` or `forced`) |
 
 ## Part of the principles monorepo
 

package/openclaw.plugin.json

@@ -2,7 +2,7 @@
   "id": "principles-disciple",
   "name": "Principles Disciple",
   "description": "Evolutionary programming agent framework with strategic guardrails and reflection loops.",
-  "version": "1.65.0",
+  "version": "1.67.0",
   "skills": [
     "./skills"
   ],
@@ -37,27 +37,6 @@
         "default": [],
         "description": "自定义高危目录（例如 .git/, prod_db/）。AI 试图修改这些目录前，将被强制拦截并要求出具安全计划。"
       },
-      "deep_reflection": {
-        "type": "object",
-        "additionalProperties": false,
-        "description": "当 AI 遇到复杂问题或连续报错时，是否允许它停下来进行深度自我反思？",
-        "properties": {
-          "enabled": {
-            "type": "boolean",
-            "default": true,
-            "description": "开启 AI 深度反思功能"
-          },
-          "mode": {
-            "type": "string",
-            "enum": [
-              "auto",
-              "forced"
-            ],
-            "default": "auto",
-            "description": "auto: 遇到困难自动触发; forced: 每次回答前都强制反思 (极耗时间，不推荐)"
-          }
-        }
-      }
     }
   },
   "uiHints": {
@@ -70,9 +49,6 @@
     },
     "riskPaths": {
       "label": "⚠️ 绝对高危目录 (空表示不设限)"
-    },
-    "deep_reflection": {
-      "label": "💡 AI 深度反思功能"
     }
   },
   "buildFingerprint": {

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "principles-disciple",
-  "version": "1.65.0",
+  "version": "1.67.0",
   "description": "Native OpenClaw plugin for Principles Disciple",
   "type": "module",
   "main": "./dist/bundle.js",

package/src/commands/context.ts

@@ -74,7 +74,6 @@ function showStatus(workspaceDir: string, isZh: boolean): string {
 |------|------|------|
 | 核心原则 | ✅ 始终开启 | 不可关闭 |
 | 思维模型 | ${config.thinkingOs ? '✅ 开启' : '❌ 关闭'} | /pd-context thinking on/off |
-| 反思日志 | ${config.reflectionLog ? '✅ 开启' : '❌ 关闭'} | /pd-context reflection on/off |
 | 项目上下文 | ${formatProjectFocus(config.projectFocus, isZh)} | /pd-context focus full/summary/off |
 
 💡 输入 \`/pd-context help\` 查看更多选项
@@ -87,7 +86,6 @@ function showStatus(workspaceDir: string, isZh: boolean): string {
 |---------|--------|---------|
 | Core Principles | ✅ Always ON | Not configurable |
 | Thinking OS | ${config.thinkingOs ? '✅ ON' : '❌ OFF'} | /pd-context thinking on/off |
-| Reflection Log | ${config.reflectionLog ? '✅ ON' : '❌ OFF'} | /pd-context reflection on/off |
 | Project Context | ${formatProjectFocus(config.projectFocus, isZh)} | /pd-context focus full/summary/off |
 
 💡 Type \`/pd-context help\` for more options
@@ -102,26 +100,26 @@ function showStatus(workspaceDir: string, isZh: boolean): string {
  
 function toggleSetting(
     workspaceDir: string,
-    key: 'thinkingOs' | 'reflectionLog',
+    key: 'thinkingOs',
     value: string,
     isZh: boolean
 ): string {
     const config = loadContextInjectionConfig(workspaceDir);
     const oldValue = config[key];
-    
+
     if (value === 'on') {
         config[key] = true;
     } else if (value === 'off') {
         config[key] = false;
     } else {
-        return isZh 
+        return isZh
             ? `❌ 无效值: "${value}"。使用 "on" 或 "off"。`
             : `❌ Invalid value: "${value}". Use "on" or "off".`;
     }
-    
+
     const newValue = config[key];
-    const keyName = isZh 
-        ? { thinkingOs: '思维模型', reflectionLog: '反思日志' }[key]
+    const keyName = isZh
+        ? { thinkingOs: '思维模型' }[key]
         : key;
     
     // No change needed
@@ -222,7 +220,6 @@ function applyPreset(
         case 'minimal':
             config = {
                 thinkingOs: false,
-                reflectionLog: false,
                 projectFocus: 'off',
                 evolutionContext: { ...defaultContextConfig.evolutionContext }
             };
@@ -230,7 +227,6 @@ function applyPreset(
         case 'standard':
             config = {
                 thinkingOs: true,
-                reflectionLog: false,
                 projectFocus: 'off',
                 evolutionContext: { ...defaultContextConfig.evolutionContext }
             };
@@ -238,7 +234,6 @@ function applyPreset(
         case 'full':
             config = {
                 thinkingOs: true,
-                reflectionLog: true,
                 projectFocus: 'summary',
                 evolutionContext: { ...defaultContextConfig.evolutionContext }
             };
@@ -328,9 +323,6 @@ export function handleContextCommand(ctx: PluginCommandContext): PluginCommandRe
         case 'thinking':
             result = toggleSetting(workspaceDir, 'thinkingOs', value, isZh);
             break;
-        case 'reflection':
-            result = toggleSetting(workspaceDir, 'reflectionLog', value, isZh);
-            break;
         case 'focus':
             result = setProjectFocus(workspaceDir, value, isZh);
             break;

package/src/commands/evolution-status.ts

@@ -100,6 +100,17 @@ function buildEnglishOutput(
     `- generatedAt: ${summary.metadata.generatedAt}`,
   ];
 
+  // E: YAML-driven Workflow Funnels (Phase 6)
+  if (summary.workflowFunnels && summary.workflowFunnels.length > 0) {
+    lines.push('');
+    for (const funnel of summary.workflowFunnels) {
+      lines.push(`Workflow Funnel: ${funnel.funnelLabel}`);
+      for (const stage of funnel.stages) {
+        lines.push(`  - ${stage.label}: ${stage.count}`);
+      }
+    }
+  }
+
   if (warnings.length > 0) {
     lines.push('', 'Warnings');
     for (const warning of warnings) {
@@ -110,8 +121,8 @@ function buildEnglishOutput(
   return lines.join('\n');
 }
 
- 
-     
+
+
 function buildChineseOutput(
   workspaceDir: string,
   sessionId: string | null,
@@ -161,6 +172,17 @@ function buildChineseOutput(
     `- 生成时间: ${summary.metadata.generatedAt}`,
   ];
 
+  // E: YAML驱动的Workflow Funnels (Phase 6)
+  if (summary.workflowFunnels && summary.workflowFunnels.length > 0) {
+    lines.push('');
+    for (const funnel of summary.workflowFunnels) {
+      lines.push(`Workflow 漏斗: ${funnel.funnelLabel}`);
+      for (const stage of funnel.stages) {
+        lines.push(`  - ${stage.label}: ${stage.count}`);
+      }
+    }
+  }
+
   if (warnings.length > 0) {
     lines.push('', '警告');
     for (const warning of warnings) {
@@ -183,7 +205,11 @@ export function handleEvolutionStatusCommand(ctx: PluginCommandContext): { text:
   const loader = new WorkflowFunnelLoader(stateDir);
   loader.watch();
   try {
-    const summary = RuntimeSummaryService.getSummary(workspaceDir, { sessionId, loaderWarnings: loader.getWarnings() });
+    const summary = RuntimeSummaryService.getSummary(workspaceDir, {
+      sessionId,
+      loaderWarnings: loader.getWarnings(),
+      funnels: loader.getAllFunnels(),
+    });
     const recommendations = WorkspaceContext.fromHookContext({ workspaceDir })
       .principleLifecycle
       .recomputeAll()

package/src/commands/pain.ts

@@ -156,8 +156,7 @@ export function handlePainCommand(ctx: PluginCommandContext): PluginCommandResul
             suggestionText = `
 💡 **建议 (系统检测到您当前遇到较大阻力)**:
    1. 执行 \`/pd-status reset\` 清零疲劳值。
-   2. 让 AI 调用 \`deep_reflect\` 工具进行深度反思。
-   3. 如果当前上下文太乱，考虑使用 \`/clear\` 开启新会话。`;
+   2. 如果当前上下文太乱，考虑使用 \`/clear\` 开启新会话。`;
         }
         else if (gfi > 50) healthLabel = '遇到阻力 🟡';
         else if (gfi > 20) healthLabel = '轻微受挫 🟢';
@@ -168,8 +167,8 @@ export function handlePainCommand(ctx: PluginCommandContext): PluginCommandResul
             suggestionText = `
 💡 **Suggestion (High friction detected)**:
    1. Run \`/pd-status reset\` to clear friction.
-   2. Ask the AI to use the \`deep_reflect\` tool.
-   4. Consider starting a new session with \`/clear\`.`;
+   2. Ask the AI to reflect deeply before continuing.
+   3. Consider starting a new session with \`/clear\`.`;
         }
         else if (gfi > 50) healthLabel = 'High Friction 🟡';
         else if (gfi > 20) healthLabel = 'Minor Issues 🟢';

package/src/config/defaults/runtime.ts

@@ -115,8 +115,6 @@ export const MAX_FILE_SIZE_BYTES = 10 * 1024 * 1024;
 
 // ── Workflow TTL Settings ───────────────────────────────────────────────────────
 
-/** Deep-reflect workflow TTL (10 minutes) */
-export const DEEP_REFLECT_TTL_MS = 10 * ONE_MINUTE_MS;
 
 // ── Time Window Constants ───────────────────────────────────────────────────────
 

package/src/constants/tools.ts

@@ -6,7 +6,6 @@ export const READ_ONLY_TOOL_NAMES = [
   'resolve-library-id', 'get-library-docs',
   'memory_recall', 'save_memory', 'todo_read', 'todo_write',
   'ask_user', 'ask_user_question',
-  'deep_reflect',
   'pd-status', 'report',
 ] as const;
 

package/src/core/config.ts

@@ -3,21 +3,6 @@ import * as fs from 'fs';
 import * as path from 'path';
 import { atomicWriteFileSync } from '../utils/io.js';
 
-export interface DeepReflectionSettings {
-    enabled: boolean;
-    mode: 'auto' | 'forced' | 'disabled';
-    force_checkpoint?: boolean;
-    checkpoint_message?: string;
-    auto_trigger_conditions?: {
-        min_tool_calls?: number;
-        error_rate_threshold?: number;
-        complexity_keywords?: string[];
-    };
-    default_model?: string;
-    default_depth?: number;
-    timeout_ms?: number;
-    modelsDir?: string;
-}
 
 export interface GfiGateSettings {
     enabled: boolean;
@@ -83,7 +68,6 @@ export interface PainSettings {
         initial_delay_ms: number;
         task_timeout_ms: number;
     };
-    deep_reflection?: DeepReflectionSettings;
     empathy_engine?: {
         enabled?: boolean;
         dedupe_window_ms?: number;
@@ -159,20 +143,6 @@ export const DEFAULT_SETTINGS: PainSettings = {
         initial_delay_ms: 5000,
         task_timeout_ms: 60 * 60 * 1000  // 1 hour, matching evolution-worker.ts default
     },
-    deep_reflection: {
-        enabled: true,
-        mode: 'auto',
-        force_checkpoint: true,
-        checkpoint_message: 'Before responding, quick self-check: 1. Task complexity (simple/medium/complex) 2. Information sufficiency (sufficient/need more) 3. If complex or insufficient info, call deep_reflect tool',
-        auto_trigger_conditions: {
-            min_tool_calls: 5,
-            error_rate_threshold: 0.3,
-            complexity_keywords: ['refactor', 'architecture', 'design', 'optimize', 'security', 'critical']
-        },
-        default_model: 'T-01',
-        default_depth: 2,
-        timeout_ms: 60000
-    },
     empathy_engine: {
         enabled: true,
         dedupe_window_ms: 60000,

package/src/core/event-log.ts

@@ -16,7 +16,6 @@ import type {
   GateBypassEventData,
   PlanApprovalEventData,
   EvolutionTaskEventData,
-  DeepReflectionEventData,
   EmpathyRollbackEventData,
   // C: New event data types
   DiagnosisTaskEventData,
@@ -178,11 +177,6 @@ export class EventLog {
     this.record('evolution_task', 'completed', undefined, data);
   }
   
-  recordDeepReflection(sessionId: string | undefined, data: DeepReflectionEventData): void {
-    const category = data.passed ? 'passed' : data.timeout ? 'failure' : 'completed';
-    this.record('deep_reflection', category, sessionId, data);
-  }
-
   recordEmpathyRollback(sessionId: string | undefined, data: EmpathyRollbackEventData): void {
     this.record('empathy_rollback', 'rolled_back', sessionId, data);
   }
@@ -416,12 +410,10 @@ export class EventLog {
         }
       } else if (Object.prototype.hasOwnProperty.call(raw, 'success')) {
         // Legacy format: { success: boolean }
-        // Apply agreed default semantics: treat as 'success' if true, 'missing_json' if false
-        if (raw['success']) {
-          stats.evolution.diagnosticianReportsWritten++;
-        }
-        // Note: legacy 'false' entries are not counted in any sub-counter since
-        // the old system had no such breakdown; they are invisible in sub-stats.
+        // Agreed fix: count ALL legacy events in diagnosticianReportsWritten (+1 for both true and false).
+        // Sub-counters (reportsMissingJson/reportsIncompleteFields) stay untouched because
+        // legacy events lack enough information to distinguish sub-category — preserve total, don't fake breakdown.
+        stats.evolution.diagnosticianReportsWritten++;
       }
     } else if (entry.type === 'principle_candidate') {
       stats.evolution.principleCandidatesCreated++;

package/src/hooks/prompt.ts

@@ -789,7 +789,6 @@ ${taskBlocks}${processingNote}
 - **STOP** aggressive file modifications.
 - **START** every response with a sincere, non-defensive apology.
 - **ACTION**: Explain why you failed, and propose a highly cautious recovery plan.
-- Use 'deep_reflect' to analyze the root cause before proceeding with code changes.
 `;
   } else if (currentGfi >= 40) {
     attitudeDirective = `
@@ -839,18 +838,6 @@ ${taskBlocks}${processingNote}
     }
   }
 
-  // Reflection Log (configurable) - moved to appendSystemContext for WebUI UX
-  let reflectionLogContent = '';
-  if (contextConfig.reflectionLog) {
-    const reflectionLogPath = wctx.resolve('REFLECTION_LOG');
-    try {
-      const cached = cachedReadFile(reflectionLogPath);
-      if (cached) reflectionLogContent = cached.trim();
-    } catch (e) {
-      logger?.error(`[PD:Prompt] Failed to read REFLECTION_LOG: ${String(e)}`);
-    }
-  }
-
   // Project Context (configurable: full/summary/off) - moved to appendSystemContext for WebUI UX
   let projectContextContent = '';
   let workingMemoryContent = '';
@@ -988,17 +975,12 @@ ${empathySilenceConstraint}
     appendParts.push(workingMemoryContent);
   }
 
-  // 2. Reflection Log
-  if (reflectionLogContent) {
-    appendParts.push(`<reflection_log>\n${reflectionLogContent}\n</reflection_log>`);
-  }
-
-  // 3. Thinking OS (configurable)
+  // 2. Thinking OS (configurable)
   if (thinkingOsContent) {
     appendParts.push(`<thinking_os>\n${thinkingOsContent}\n</thinking_os>`);
   }
 
-  // 4. Evolution Loop principles (active/probation)
+  // 3. Evolution Loop principles (active/probation)
   if (evolutionPrinciplesContent) {
     appendParts.push(`<evolution_principles>\n${evolutionPrinciplesContent}\n</evolution_principles>`);
   }
@@ -1163,22 +1145,8 @@ ${attitudeDirective}
       }
     }
 
-    // 2. Truncate reflection_log if still over limit
+    // 2. Final check
     let newSize = prependSystemContext.length + prependContext.length + appendSystemContext.length;
-    if (newSize > MAX_SIZE && reflectionLogContent && appendSystemContext.includes('<reflection_log>')) {
-      const lines = reflectionLogContent.split('\n');
-      if (lines.length > 30) {
-        const truncated = lines.slice(0, 30).join('\n') + '\n...[truncated]';
-        appendSystemContext = appendSystemContext.replace(
-          `<reflection_log>\n${reflectionLogContent}\n</reflection_log>`,
-          `<reflection_log>\n${truncated}\n</reflection_log>`
-        );
-        truncationLog.push('reflection_log');
-      }
-    }
-
-    // 3. Final check
-    newSize = prependSystemContext.length + prependContext.length + appendSystemContext.length;
     if (newSize > MAX_SIZE) {
       // NOTE: We still return the content even if over limit, as truncating more
       // could lose critical context like principles or evolution directives.

package/src/hooks/subagent.ts

@@ -6,7 +6,6 @@ import { extractAgentIdFromSessionKey } from '../utils/session-key.js';
 import { recordEvolutionSuccess } from '../core/evolution-engine.js';
 import { WorkflowStore } from '../service/subagent-workflow/workflow-store.js';
 import { EmpathyObserverWorkflowManager } from '../service/subagent-workflow/empathy-observer-workflow-manager.js';
-import { DeepReflectWorkflowManager } from '../service/subagent-workflow/deep-reflect-workflow-manager.js';
 import type { WorkflowManager } from '../service/subagent-workflow/types.js';
 
 /**
@@ -35,12 +34,6 @@ function createWorkflowManagerForType(
                 logger: loggerAdapter,
                 subagent,
             });
-        case 'deep-reflect':
-            return new DeepReflectWorkflowManager({
-                workspaceDir,
-                logger: loggerAdapter,
-                subagent,
-            });
         default:
             return null;
     }

package/src/index.ts

@@ -55,7 +55,6 @@ import { CentralSyncService } from './service/central-sync-service.js';
 import { ensureWorkspaceTemplates } from './core/init.js';
 import { migrateDirectoryStructure } from './core/migration.js';
 import { SystemLogger } from './core/system-logger.js';
-import { createDeepReflectTool } from './tools/deep-reflect.js';
 import { createWritePainFlagTool } from './tools/write-pain-flag.js';
 import { PathResolver } from './core/path-resolver.js';
 import { createPrinciplesConsoleRoute } from './http/principles-console-route.js';
@@ -750,7 +749,6 @@ const plugin = {
       }
     });
 
-    api.registerTool(createDeepReflectTool(api));
     api.registerTool(createWritePainFlagTool(api));
   }
 };

package/src/service/evolution-worker.ts

@@ -31,7 +31,6 @@ import { checkWorkspaceIdle, checkCooldown, recordCooldown } from './nocturnal-r
 import { loadCooldownEscalationConfig, loadNocturnalConfigMerged } from './nocturnal-config.js';
 import { WorkflowStore } from './subagent-workflow/workflow-store.js';
 import { EmpathyObserverWorkflowManager } from './subagent-workflow/empathy-observer-workflow-manager.js';
-import { DeepReflectWorkflowManager } from './subagent-workflow/deep-reflect-workflow-manager.js';
 import { NocturnalWorkflowManager, nocturnalWorkflowSpec } from './subagent-workflow/nocturnal-workflow-manager.js';
 import {
     createNocturnalTrajectoryExtractor,
@@ -2443,18 +2442,6 @@ export const EvolutionWorkerService: ExtendedEvolutionWorkerService = {
                             empathyMgr.dispose();
                         }
 
-                        const deepReflectMgr = new DeepReflectWorkflowManager({
-                            workspaceDir: wctx.workspaceDir,
-                            logger: api.logger,
-                            subagent: subagentRuntime,
-                            agentSession,
-                        });
-                        try {
-                            swept += await deepReflectMgr.sweepExpiredWorkflows(WORKFLOW_TTL_MS);
-                        } finally {
-                            deepReflectMgr.dispose();
-                        }
-
                         // #183 + #188: Sweep Nocturnal workflows too (with gateway-safe fallback)
                         try {
                             const nocturnalMgr = new NocturnalWorkflowManager({