principles-disciple 1.72.0 → 1.74.0

package/src/tools/write-pain-flag.ts

@@ -1,215 +0,0 @@
-import type { OpenClawPluginApi } from '../openclaw-sdk.js';
-import { Type } from '@sinclair/typebox';
-import { buildPainFlag } from '../core/pain.js';
-import { resolveWorkspaceDirFromApi } from '../core/path-resolver.js';
-import { TrajectoryRegistry } from '../core/trajectory.js';
-import * as fs from 'fs';
-import * as path from 'path';
-import { atomicWriteFileSync } from '../utils/io.js';
-
-// Pain flag contract required fields
-const PAIN_FLAG_REQUIRED_FIELDS = ['source', 'score', 'time', 'reason'] as const;
-
-/**
- * Atomic file write: write to temp file then rename.
- * Prevents corruption if process crashes mid-write.
- */
-function writePainFlagAtomic(filePath: string, content: string): void {
-    const dir = path.dirname(filePath);
-    if (!fs.existsSync(dir)) {
-        fs.mkdirSync(dir, { recursive: true });
-    }
-    atomicWriteFileSync(filePath, content);
-}
-
-/**
- * Creates the `write_pain_flag` tool.
- *
- * This tool allows the agent to record a pain signal when it recognizes
- * that it made a mistake, violated a principle, or needs to flag an issue
- * for later reflection.
- *
- * The tool wraps `buildPainFlag` + atomic `writePainFlag` to ensure:
- * - Correct KV format serialization (never [object Object] corruption)
- * - Atomic writes (temp file + rename, crash-safe)
- * - Full contract compliance (source, score, time, reason)
- *
- * The agent should NEVER write to .pain_flag directly.
- */
-export function createWritePainFlagTool(api: OpenClawPluginApi) {
-    return {
-        name: 'write_pain_flag',
-        description:
-            'Record a pain signal to flag mistakes, principle violations, or issues for later reflection. ' +
-            'Use this tool INSTEAD of writing .pain_flag directly. ' +
-            'Pain signals are processed by the evolution system on the next heartbeat cycle.',
-        parameters: Type.Object({
-            reason: Type.String({
-                description:
-                    'Describe specifically what went wrong. ' +
-                    'Include the error, the violated principle, or the issue. ' +
-                    'Be concrete: "I edited config.ts without reading it first, breaking the export" ' +
-                    'is better than "I made a mistake".',
-            }),
-            score: Type.Optional(Type.Number({
-                description:
-                    'Pain severity score (0-100). Default: 80. ' +
-                    'Guidelines: 30-50 (minor issue), 50-70 (moderate error), ' +
-                    '70-100 (severe principle violation or data loss risk).',
-                minimum: 0,
-                maximum: 100,
-            })),
-            source: Type.Optional(Type.String({
-                description:
-                    'Source of the pain signal. ' +
-                    'Values: manual (user flagged), tool_failure (tool error), ' +
-                    'user_empathy (user frustration), principle_violation (principle broken), ' +
-                    'human_intervention (user manually intervened). ' +
-                    'Default: manual.',
-            })),
-            session_id: Type.Optional(Type.String({
-                description:
-                    'Session ID where the pain occurred. ' +
-                    'If not provided, the system will use the current session.',
-            })),
-            is_risky: Type.Optional(Type.Boolean({
-                description:
-                    'Whether this involves a high-risk operation (e.g., writing to sensitive files). ' +
-                    'Default: false.',
-            })),
-        }),
-
-        async execute(
-            _toolCallId: string,
-            rawParams: Record<string, unknown>
-        ): Promise<{ content: { type: string; text: string }[] }> {
-            const reason = typeof rawParams.reason === 'string' ? rawParams.reason.trim() : '';
-            const score = typeof rawParams.score === 'number' ? Math.max(0, Math.min(100, Math.round(rawParams.score))) : 80;
-            const source = typeof rawParams.source === 'string' && rawParams.source.trim() ? rawParams.source.trim() : 'manual';
-            const sessionId = typeof rawParams.session_id === 'string' ? rawParams.session_id.trim() : '';
-            const isRisky = rawParams.is_risky === true;
-
-            // ── Validate required fields ──
-            if (!reason) {
-                api.logger?.warn?.('[PD:write_pain_flag] Missing required field: reason');
-                return {
-                    content: [{
-                        type: 'text',
-                        text: '❌ Error: The `reason` parameter is required.\n' +
-                            'Describe specifically what went wrong. Example:\n' +
-                            '"I edited config.ts without reading it first, breaking the export"',
-                    }],
-                };
-            }
-
-            // ── Resolve workspace ──
-            const workspaceDir = resolveWorkspaceDirFromApi(api);
-            if (!workspaceDir) {
-                api.logger?.error?.('[PD:write_pain_flag] Cannot resolve workspace directory');
-                return {
-                    content: [{
-                        type: 'text',
-                        text: '❌ Error: Cannot determine the workspace directory. ' +
-                            'Please ensure you are in an active workspace.',
-                    }],
-                };
-            }
-
-            try {
-                // ── Record pain event to trajectory DB first (before flag file) ──
-                // This ensures we have a real AUTOINCREMENT ID that flows through
-                // the entire pain→principle→compile chain (painEventId propagation).
-                let painEventId: number | undefined;
-                try {
-                    const trajectory = TrajectoryRegistry.get(workspaceDir);
-                    painEventId = trajectory.recordPainEvent({
-                        sessionId: sessionId || 'unknown',
-                        source,
-                        score,
-                        reason,
-                        severity: null,
-                        origin: 'manual',
-                        confidence: null,
-                        text: undefined,
-                    });
-                } catch (trajErr) {
-                    // If trajectory write fails, log but continue — the pain flag
-                    // itself is still valid and should be processed. The pain event
-                    // ID simply won't be available for the compiler's exact matching.
-                    api.logger?.warn?.(`[PD:write_pain_flag] Failed to record pain event to trajectory: ${String(trajErr)}`);
-                }
-
-                // ── Build pain flag data (KV format) ──
-                const painData = buildPainFlag({
-                    source,
-                    score: String(score),
-                    reason,
-                    session_id: sessionId,
-                    is_risky: isRisky,
-                    pain_event_id: painEventId !== undefined ? String(painEventId) : undefined,
-                });
-
-                // ── Validate contract compliance ──
-                const missingFields: string[] = [];
-                for (const field of PAIN_FLAG_REQUIRED_FIELDS) {
-                    if (!painData[field] || painData[field].trim() === '') {
-                        missingFields.push(field);
-                    }
-                }
-                if (missingFields.length > 0) {
-                    api.logger?.error?.(`[PD:write_pain_flag] Pain flag missing required fields: ${missingFields.join(', ')}`);
-                    return {
-                        content: [{
-                            type: 'text',
-                            text: `❌ Error: Pain flag is missing required fields: ${missingFields.join(', ')}. ` +
-                                'This is an internal error — please report it.',
-                        }],
-                    };
-                }
-
-                // ── Atomic write (temp file + rename) ──
-                const painFlagPath = path.join(workspaceDir, '.state', '.pain_flag');
-                const { serializeKvLines } = await import('../utils/io.js');
-                const content = serializeKvLines(painData);
-                writePainFlagAtomic(painFlagPath, content);
-
-                // ── Log success ──
-                api.logger?.info?.(
-                    `[PD:write_pain_flag] Pain signal recorded: source=${source}, score=${score}, ` +
-                    `reason="${reason.slice(0, 80)}"${reason.length > 80 ? '...' : ''}"`
-                );
-
-                // ── Agent feedback ──
-                return {
-                    content: [{
-                        type: 'text',
-                        text: `✅ Pain signal recorded successfully.\n\n` +
-                            `- **Reason**: ${reason}\n` +
-                            `- **Score**: ${score}/100\n` +
-                            `- **Source**: ${source}\n` +
-                            `- **Risk**: ${isRisky ? 'Yes' : 'No'}\n` +
-                            `- **Session**: ${sessionId || '(current)'}\n\n` +
-                            `The evolution system will process this signal on the next heartbeat cycle ` +
-                            `(typically within 60 seconds).`,
-                    }],
-                };
-            } catch (err) {
-                // ── Log failure with stack trace ──
-                const errorMsg = err instanceof Error ? err.message : String(err);
-                const stack = err instanceof Error ? err.stack?.split('\n').slice(0, 3).join(' → ') : '';
-                api.logger?.error?.(
-                    `[PD:write_pain_flag] Failed to write pain flag: ${errorMsg}` +
-                    (stack ? `\n  Stack: ${stack}` : '')
-                );
-
-                return {
-                    content: [{
-                        type: 'text',
-                        text: `❌ Failed to record pain signal: ${errorMsg}\n\n` +
-                            'The error has been logged. Please try again or report this issue.',
-                    }],
-                };
-            }
-        },
-    };
-}

package/templates/langs/en/skills/plan-script/SKILL.md

@@ -1,32 +0,0 @@
----
-name: plan-script
-description: Create a step-by-step movie-script style execution plan. Includes target files, verification metrics, and rollback strategy.
-disable-model-invocation: true
----
-
-# Plan Script
-
-**Goal**: Produce a "foolproof" executable plan to ensure controlled execution.
-
-Please generate plan in the following structure:
-
-## 1. Target Files (Authorization List)
-- List file paths **uniquely authorized** for modification in this plan.
-- Format: `- path/to/file`
-
-## 2. Steps (Execution Steps)
-1. Operations specific to filenames and tool calls.
-2. Each step includes expected intermediate state.
-
-## 3. Metrics (Verification Metrics)
-- How to quantitatively prove this plan succeeded? (e.g., tests pass, command returns 0, specific string appears in logs).
-
-## 4. Active Mental Models
-- Select exactly **2** meta-cognitive models from `.principles/THINKING_OS.md` that are most relevant to the current task.
-- Format: `- [T-0X] Model Name: Why is it needed for this specific task?`
-
-## 5. Rollback (Rollback Strategy)
-- If step 2 fails, how to one-click restore to safe state?
-
----
-**Action**: Update above content to `PLAN.md` and set `STATUS: READY`.

package/templates/langs/zh/skills/plan-script/SKILL.md

@@ -1,32 +0,0 @@
----
-name: plan-script
-description: Create a step-by-step movie-script style execution plan. Includes target files, verification metrics, and rollback strategy.
-disable-model-invocation: true
----
-
-# Plan Script (计划编排)
-
-**目标**: 产生一份“傻瓜式”可执行计划，确保执行过程受控。
-
-请按以下结构生成计划：
-
-## 1. Target Files (授权清单)
-- 列出本次计划**唯一授权**修改的文件路径。
-- 格式：`- path/to/file`
-
-## 2. Steps (执行步骤)
-1. 具体到文件名和工具调用的操作。
-2. 每个步骤包含预期的中间状态。
-
-## 3. Metrics (验证指标)
-- 如何量化证明本计划成功了？(如：测试通过、命令返回 0、日志出现特定字符串)。
-
-## 4. Active Mental Models (激活的思维模型)
-- 从 `.principles/THINKING_OS.md` 中挑选 **2 个** 最适合当前任务的元认知模型。
-- 格式：`- [T-0X] 模型名称：为什么在这个任务中需要它？`
-
-## 5. Rollback (回滚方案)
-- 如果步骤 2 失败，如何一键恢复到安全状态？
-
----
-**动作**: 请将以上内容更新至 `PLAN.md`，并设置 `STATUS: READY`。