dominds 1.2.5 → 1.2.7

package/dist/tools/fs.js

@@ -17,6 +17,60 @@ const readline_1 = require("readline");
 const access_control_1 = require("../access-control");
 const log_1 = require("../log");
 const runtime_language_1 = require("../shared/runtime-language");
+async function statWithSymlinkInfo(absPath) {
+    const lstat = await promises_1.default.lstat(absPath);
+    if (!lstat.isSymbolicLink()) {
+        return {
+            lstat,
+            followStat: lstat,
+            isSymlink: false,
+        };
+    }
+    let symlinkTarget;
+    try {
+        symlinkTarget = await promises_1.default.readlink(absPath);
+    }
+    catch (err) {
+        log_1.log.warn(`Failed to read symlink ${absPath}:`, err);
+    }
+    const followStat = await promises_1.default.stat(absPath);
+    return {
+        lstat,
+        followStat,
+        isSymlink: true,
+        symlinkTarget,
+    };
+}
+function symlinkTargetText(target) {
+    return target ?? '<unreadable>';
+}
+function symlinkFollowNotice(workLanguage, relPath, target, asKind) {
+    const kindLabel = asKind === 'dir'
+        ? workLanguage === 'zh'
+            ? '目录'
+            : 'directory'
+        : workLanguage === 'zh'
+            ? '文件'
+            : 'file';
+    const targetText = symlinkTargetText(target);
+    if (workLanguage === 'zh') {
+        return `🔗 说明：\`${relPath}\` 是符号链接（→ \`${targetText}\`），已按${kindLabel}跟随处理。`;
+    }
+    return `🔗 Note: \`${relPath}\` is a symlink (→ \`${targetText}\`) and was followed as a ${kindLabel}.`;
+}
+function symlinkRemovalNotice(workLanguage, relPath, target) {
+    const targetText = symlinkTargetText(target);
+    if (workLanguage === 'zh') {
+        return `🔗 说明：删除的是符号链接路径 \`${relPath}\` 本身（→ \`${targetText}\`），不会直接删除其目标。`;
+    }
+    return `🔗 Note: removed symlink path \`${relPath}\` itself (→ \`${targetText}\`), not the target directly.`;
+}
+function appendSymlinkYamlFields(lines, keyPrefix, info) {
+    if (!info || !info.isSymlink)
+        return;
+    lines.push(`${keyPrefix}_kind: symlink`);
+    lines.push(`${keyPrefix}_symlink_target: ${yamlQuote(symlinkTargetText(info.symlinkTarget))}`);
+}
 function formatSize(bytes) {
     if (bytes === 0)
         return '0 B';
@@ -99,6 +153,9 @@ exports.listDirTool = {
                 notDir: (p) => `❌ **错误**\n\n路径 \`${p}\` 不是目录。`,
                 readDirFailed: (msg) => `❌ **错误**\n\n读取目录失败：${msg}`,
                 dirHeader: '📁 **目录：**',
+                symlinkPathNotice: (p, target) => target
+                    ? `🔗 **说明：** \`${p}\` 是符号链接（→ \`${target}\`），已按目录跟随读取。`
+                    : `🔗 **说明：** \`${p}\` 是符号链接，已按目录跟随读取。`,
                 emptyDir: '_此目录为空。_',
                 table: {
                     name: '名称',
@@ -114,6 +171,9 @@ exports.listDirTool = {
                 notDir: (p) => `❌ **Error**\n\nPath \`${p}\` is not a directory.`,
                 readDirFailed: (msg) => `❌ **Error**\n\nFailed to read directory: ${msg}`,
                 dirHeader: '📁 **Directory:**',
+                symlinkPathNotice: (p, target) => target
+                    ? `🔗 **Note:** \`${p}\` is a symlink (→ \`${target}\`), and was followed as a directory.`
+                    : `🔗 **Note:** \`${p}\` is a symlink and was followed as a directory.`,
                 emptyDir: '_This directory is empty._',
                 table: {
                     name: 'Name',
@@ -141,9 +201,13 @@ exports.listDirTool = {
             return content;
         }
         try {
+            let inputPathIsSymlink = false;
+            let inputPathSymlinkTarget;
             try {
-                const stats = await promises_1.default.lstat(dir);
-                if (!stats.isDirectory()) {
+                const statsInfo = await statWithSymlinkInfo(dir);
+                inputPathIsSymlink = statsInfo.isSymlink;
+                inputPathSymlinkTarget = statsInfo.symlinkTarget;
+                if (!statsInfo.followStat.isDirectory()) {
                     const content = labels.notDir(rel);
                     return content;
                 }
@@ -188,21 +252,31 @@ exports.listDirTool = {
                         try {
                             const target = await promises_1.default.readlink(entryPath);
                             dirEntry.target = target;
-                            // If symlink points to a text file, count lines from the target
                             try {
                                 const targetStats = await promises_1.default.stat(entryPath); // Follow the symlink
-                                if (targetStats.isFile() && isTextFile(entry.name)) {
+                                if (targetStats.isDirectory()) {
+                                    dirEntry.symlinkResolvedType = 'dir';
+                                }
+                                else if (targetStats.isFile()) {
+                                    dirEntry.symlinkResolvedType = 'file';
+                                }
+                                else {
+                                    dirEntry.symlinkResolvedType = 'other';
+                                }
+                                // If symlink points to a text file, count lines from the target
+                                if (targetStats.isFile() && (isTextFile(entry.name) || isTextFile(target))) {
                                     dirEntry.lines = await countLines(entryPath);
                                 }
                             }
                             catch (err) {
                                 log_1.log.warn(`Failed to stat symlink target ${entryPath}:`, err);
-                                // Target doesn't exist or can't be accessed
+                                dirEntry.symlinkResolvedType = 'broken';
                             }
                         }
                         catch (err) {
                             log_1.log.warn(`Failed to read symlink ${entryPath}:`, err);
                             dirEntry.target = '<unreadable>';
+                            dirEntry.symlinkResolvedType = 'other';
                         }
                     }
                     else {
@@ -228,6 +302,9 @@ exports.listDirTool = {
             const relativeDir = path_1.default.relative(cwd, dir) || '.';
             // Create markdown table for directory entries
             let markdown = `${labels.dirHeader} \`${relativeDir}\`\n\n`;
+            if (inputPathIsSymlink) {
+                markdown += `${labels.symlinkPathNotice(rel, inputPathSymlinkTarget)}\n\n`;
+            }
             if (data.length === 0) {
                 markdown += labels.emptyDir;
             }
@@ -244,7 +321,10 @@ exports.listDirTool = {
                                 : '❓';
                     const sizeStr = entry.size ? formatSize(entry.size) : '-';
                     const linesStr = entry.lines ? entry.lines.toString() : '-';
-                    const targetStr = entry.target ? `→ ${entry.target}` : '-';
+                    const targetTypeStr = entry.type === 'symlink' && entry.symlinkResolvedType
+                        ? ` (${entry.symlinkResolvedType})`
+                        : '';
+                    const targetStr = entry.target ? `→ ${entry.target}${targetTypeStr}` : '-';
                     markdown += `| ${typeIcon} \`${entry.name}\` | ${entry.type} | ${sizeStr} | ${linesStr} | ${targetStr} |\n`;
                 }
             }
@@ -335,20 +415,27 @@ exports.rmDirTool = {
             return (0, access_control_1.getAccessDeniedMessage)('write', rel, workLanguage);
         }
         try {
+            let pathInfo;
             // Check if path exists and is a directory
-            const stats = await promises_1.default.lstat(targetPath);
-            if (!stats.isDirectory()) {
-                return labels.notDir(rel);
+            pathInfo = await statWithSymlinkInfo(targetPath);
+            const followNotice = pathInfo.isSymlink
+                ? `\n\n${symlinkFollowNotice(workLanguage, rel, pathInfo.symlinkTarget, 'dir')}`
+                : '';
+            if (!pathInfo.followStat.isDirectory()) {
+                return `${labels.notDir(rel)}${followNotice}`;
             }
             // Check if directory is empty when not using recursive
             if (!recursive) {
                 const entries = await promises_1.default.readdir(targetPath);
                 if (entries.length > 0) {
-                    return labels.notEmpty(rel);
+                    return `${labels.notEmpty(rel)}${followNotice}`;
                 }
             }
             // Node.js >=22+ types deprecate recursive rmdir in favor of rm.
             await promises_1.default.rm(targetPath, { recursive, force: false });
+            if (pathInfo.isSymlink) {
+                return `${labels.removed(rel)}\n\n${symlinkFollowNotice(workLanguage, rel, pathInfo.symlinkTarget, 'dir')}\n\n${symlinkRemovalNotice(workLanguage, rel, pathInfo.symlinkTarget)}`;
+            }
             return labels.removed(rel);
         }
         catch (error) {
@@ -416,13 +503,20 @@ exports.rmFileTool = {
             return (0, access_control_1.getAccessDeniedMessage)('write', rel, workLanguage);
         }
         try {
+            let pathInfo;
             // Check if path exists and is a file
-            const stats = await promises_1.default.lstat(targetPath);
-            if (!stats.isFile()) {
-                return labels.notFile(rel);
+            pathInfo = await statWithSymlinkInfo(targetPath);
+            const followNotice = pathInfo.isSymlink
+                ? `\n\n${symlinkFollowNotice(workLanguage, rel, pathInfo.symlinkTarget, 'file')}`
+                : '';
+            if (!pathInfo.followStat.isFile()) {
+                return `${labels.notFile(rel)}${followNotice}`;
             }
             // Remove the file
             await promises_1.default.unlink(targetPath);
+            if (pathInfo.isSymlink) {
+                return `${labels.removed(rel)}\n\n${symlinkFollowNotice(workLanguage, rel, pathInfo.symlinkTarget, 'file')}\n\n${symlinkRemovalNotice(workLanguage, rel, pathInfo.symlinkTarget)}`;
+            }
             return labels.removed(rel);
         }
         catch (error) {
@@ -518,7 +612,7 @@ exports.mkDirTool = {
             return (0, access_control_1.getAccessDeniedMessage)('write', rel, workLanguage);
         }
         try {
-            const st = await promises_1.default.lstat(targetPath).catch((err) => {
+            const pathInfo = await statWithSymlinkInfo(targetPath).catch((err) => {
                 if (typeof err === 'object' &&
                     err !== null &&
                     'code' in err &&
@@ -527,25 +621,30 @@ exports.mkDirTool = {
                 }
                 throw err;
             });
-            if (st) {
-                if (!st.isDirectory()) {
-                    const yaml = [
+            if (pathInfo) {
+                const symlinkSummary = pathInfo.isSymlink
+                    ? ` ${symlinkFollowNotice(workLanguage, rel, pathInfo.symlinkTarget, 'dir')}`
+                    : '';
+                if (!pathInfo.followStat.isDirectory()) {
+                    const yamlLines = [
                         `status: error`,
                         `path: ${yamlQuote(rel)}`,
                         `error: PATH_EXISTS_NOT_DIR`,
                         `summary: ${yamlQuote(workLanguage === 'zh'
-                            ? 'Mk-dir failed: path exists and is not a directory.'
-                            : 'Mk-dir failed: path exists and is not a directory.')}`,
-                    ].join('\n');
-                    return formatYamlCodeBlock(yaml);
+                            ? `Mk-dir failed: path exists and is not a directory.${symlinkSummary}`
+                            : `Mk-dir failed: path exists and is not a directory.${symlinkSummary}`)}`,
+                    ];
+                    appendSymlinkYamlFields(yamlLines, 'path', pathInfo);
+                    return formatYamlCodeBlock(yamlLines.join('\n'));
                 }
-                const yaml = [
+                const yamlLines = [
                     `status: ok`,
                     `path: ${yamlQuote(rel)}`,
                     `created: false`,
-                    `summary: ${yamlQuote(`Mk-dir: ${rel} (parents=${parents}).`)}`,
-                ].join('\n');
-                return formatYamlCodeBlock(yaml);
+                    `summary: ${yamlQuote(`Mk-dir: ${rel} (parents=${parents}).${symlinkSummary}`)}`,
+                ];
+                appendSymlinkYamlFields(yamlLines, 'path', pathInfo);
+                return formatYamlCodeBlock(yamlLines.join('\n'));
             }
             await promises_1.default.mkdir(targetPath, { recursive: parents });
             const yaml = [
@@ -620,32 +719,40 @@ exports.moveFileTool = {
             return (0, access_control_1.getAccessDeniedMessage)('write', from, workLanguage);
         }
         try {
-            const st = await promises_1.default.lstat(absFrom);
-            if (!st.isFile()) {
-                const yaml = [
+            const fromInfo = await statWithSymlinkInfo(absFrom);
+            const fromSymlinkSummary = fromInfo.isSymlink
+                ? ` ${symlinkFollowNotice(workLanguage, from, fromInfo.symlinkTarget, 'file')}`
+                : '';
+            if (!fromInfo.followStat.isFile()) {
+                const yamlLines = [
                     `status: error`,
                     `from: ${yamlQuote(from)}`,
                     `to: ${yamlQuote(to)}`,
                     `error: FROM_NOT_FILE`,
                     `summary: ${yamlQuote(workLanguage === 'zh'
-                        ? 'Move-file failed: from is not a file.'
-                        : 'Move-file failed: from is not a file.')}`,
-                ].join('\n');
-                return formatYamlCodeBlock(yaml);
+                        ? `Move-file failed: from is not a file.${fromSymlinkSummary}`
+                        : `Move-file failed: from is not a file.${fromSymlinkSummary}`)}`,
+                ];
+                appendSymlinkYamlFields(yamlLines, 'from_path', fromInfo);
+                return formatYamlCodeBlock(yamlLines.join('\n'));
             }
             const toParent = path_1.default.dirname(absTo);
-            const toParentSt = await promises_1.default.lstat(toParent).catch(() => undefined);
-            if (!toParentSt || !toParentSt.isDirectory()) {
-                const yaml = [
+            const toParentInfo = await statWithSymlinkInfo(toParent).catch(() => undefined);
+            const toParentSymlinkSummary = toParentInfo && toParentInfo.isSymlink
+                ? ` ${symlinkFollowNotice(workLanguage, path_1.default.relative(cwd, toParent) || '.', toParentInfo.symlinkTarget, 'dir')}`
+                : '';
+            if (!toParentInfo || !toParentInfo.followStat.isDirectory()) {
+                const yamlLines = [
                     `status: error`,
                     `from: ${yamlQuote(from)}`,
                     `to: ${yamlQuote(to)}`,
                     `error: TO_PARENT_NOT_DIR`,
                     `summary: ${yamlQuote(workLanguage === 'zh'
-                        ? 'Move-file failed: destination parent directory does not exist. Use mk_dir first.'
-                        : 'Move-file failed: destination parent directory does not exist. Use mk_dir first.')}`,
-                ].join('\n');
-                return formatYamlCodeBlock(yaml);
+                        ? `Move-file failed: destination parent directory does not exist. Use mk_dir first.${toParentSymlinkSummary}`
+                        : `Move-file failed: destination parent directory does not exist. Use mk_dir first.${toParentSymlinkSummary}`)}`,
+                ];
+                appendSymlinkYamlFields(yamlLines, 'to_parent_path', toParentInfo);
+                return formatYamlCodeBlock(yamlLines.join('\n'));
             }
             const toExists = await promises_1.default
                 .lstat(absTo)
@@ -660,7 +767,7 @@ exports.moveFileTool = {
                 throw err;
             });
             if (toExists) {
-                const yaml = [
+                const yamlLines = [
                     `status: error`,
                     `from: ${yamlQuote(from)}`,
                     `to: ${yamlQuote(to)}`,
@@ -668,17 +775,21 @@ exports.moveFileTool = {
                     `summary: ${yamlQuote(workLanguage === 'zh'
                         ? 'Move-file failed: destination already exists.'
                         : 'Move-file failed: destination already exists.')}`,
-                ].join('\n');
-                return formatYamlCodeBlock(yaml);
+                ];
+                appendSymlinkYamlFields(yamlLines, 'from_path', fromInfo);
+                appendSymlinkYamlFields(yamlLines, 'to_parent_path', toParentInfo);
+                return formatYamlCodeBlock(yamlLines.join('\n'));
             }
             await promises_1.default.rename(absFrom, absTo);
-            const yaml = [
+            const yamlLines = [
                 `status: ok`,
                 `from: ${yamlQuote(from)}`,
                 `to: ${yamlQuote(to)}`,
-                `summary: ${yamlQuote(`Move-file: ${from} \u2192 ${to}.`)}`,
-            ].join('\n');
-            return formatYamlCodeBlock(yaml);
+                `summary: ${yamlQuote(`Move-file: ${from} \u2192 ${to}.${fromSymlinkSummary}${toParentSymlinkSummary}`)}`,
+            ];
+            appendSymlinkYamlFields(yamlLines, 'from_path', fromInfo);
+            appendSymlinkYamlFields(yamlLines, 'to_parent_path', toParentInfo);
+            return formatYamlCodeBlock(yamlLines.join('\n'));
         }
         catch (error) {
             const yaml = [
@@ -745,32 +856,40 @@ exports.moveDirTool = {
             return (0, access_control_1.getAccessDeniedMessage)('write', from, workLanguage);
         }
         try {
-            const st = await promises_1.default.lstat(absFrom);
-            if (!st.isDirectory()) {
-                const yaml = [
+            const fromInfo = await statWithSymlinkInfo(absFrom);
+            const fromSymlinkSummary = fromInfo.isSymlink
+                ? ` ${symlinkFollowNotice(workLanguage, from, fromInfo.symlinkTarget, 'dir')}`
+                : '';
+            if (!fromInfo.followStat.isDirectory()) {
+                const yamlLines = [
                     `status: error`,
                     `from: ${yamlQuote(from)}`,
                     `to: ${yamlQuote(to)}`,
                     `error: FROM_NOT_DIR`,
                     `summary: ${yamlQuote(workLanguage === 'zh'
-                        ? 'Move-dir failed: from is not a directory.'
-                        : 'Move-dir failed: from is not a directory.')}`,
-                ].join('\n');
-                return formatYamlCodeBlock(yaml);
+                        ? `Move-dir failed: from is not a directory.${fromSymlinkSummary}`
+                        : `Move-dir failed: from is not a directory.${fromSymlinkSummary}`)}`,
+                ];
+                appendSymlinkYamlFields(yamlLines, 'from_path', fromInfo);
+                return formatYamlCodeBlock(yamlLines.join('\n'));
             }
             const toParent = path_1.default.dirname(absTo);
-            const toParentSt = await promises_1.default.lstat(toParent).catch(() => undefined);
-            if (!toParentSt || !toParentSt.isDirectory()) {
-                const yaml = [
+            const toParentInfo = await statWithSymlinkInfo(toParent).catch(() => undefined);
+            const toParentSymlinkSummary = toParentInfo && toParentInfo.isSymlink
+                ? ` ${symlinkFollowNotice(workLanguage, path_1.default.relative(cwd, toParent) || '.', toParentInfo.symlinkTarget, 'dir')}`
+                : '';
+            if (!toParentInfo || !toParentInfo.followStat.isDirectory()) {
+                const yamlLines = [
                     `status: error`,
                     `from: ${yamlQuote(from)}`,
                     `to: ${yamlQuote(to)}`,
                     `error: TO_PARENT_NOT_DIR`,
                     `summary: ${yamlQuote(workLanguage === 'zh'
-                        ? 'Move-dir failed: destination parent directory does not exist. Use mk_dir first.'
-                        : 'Move-dir failed: destination parent directory does not exist. Use mk_dir first.')}`,
-                ].join('\n');
-                return formatYamlCodeBlock(yaml);
+                        ? `Move-dir failed: destination parent directory does not exist. Use mk_dir first.${toParentSymlinkSummary}`
+                        : `Move-dir failed: destination parent directory does not exist. Use mk_dir first.${toParentSymlinkSummary}`)}`,
+                ];
+                appendSymlinkYamlFields(yamlLines, 'to_parent_path', toParentInfo);
+                return formatYamlCodeBlock(yamlLines.join('\n'));
             }
             const toExists = await promises_1.default
                 .lstat(absTo)
@@ -785,7 +904,7 @@ exports.moveDirTool = {
                 throw err;
             });
             if (toExists) {
-                const yaml = [
+                const yamlLines = [
                     `status: error`,
                     `from: ${yamlQuote(from)}`,
                     `to: ${yamlQuote(to)}`,
@@ -793,19 +912,26 @@ exports.moveDirTool = {
                     `summary: ${yamlQuote(workLanguage === 'zh'
                         ? 'Move-dir failed: destination already exists.'
                         : 'Move-dir failed: destination already exists.')}`,
-                ].join('\n');
-                return formatYamlCodeBlock(yaml);
+                ];
+                appendSymlinkYamlFields(yamlLines, 'from_path', fromInfo);
+                appendSymlinkYamlFields(yamlLines, 'to_parent_path', toParentInfo);
+                return formatYamlCodeBlock(yamlLines.join('\n'));
             }
-            const movedEntryCount = await countDirEntries(absFrom);
+            const movedEntryCount = fromInfo.isSymlink ? 1 : await countDirEntries(absFrom);
             await promises_1.default.rename(absFrom, absTo);
-            const yaml = [
+            const symlinkMoveNotice = fromInfo.isSymlink
+                ? ` ${symlinkRemovalNotice(workLanguage, from, fromInfo.symlinkTarget)}`
+                : '';
+            const yamlLines = [
                 `status: ok`,
                 `from: ${yamlQuote(from)}`,
                 `to: ${yamlQuote(to)}`,
                 `moved_entry_count: ${movedEntryCount}`,
-                `summary: ${yamlQuote(`Move-dir: ${from} \u2192 ${to} (${movedEntryCount} entries).`)}`,
-            ].join('\n');
-            return formatYamlCodeBlock(yaml);
+                `summary: ${yamlQuote(`Move-dir: ${from} \u2192 ${to} (${movedEntryCount} entries).${fromSymlinkSummary}${toParentSymlinkSummary}${symlinkMoveNotice}`)}`,
+            ];
+            appendSymlinkYamlFields(yamlLines, 'from_path', fromInfo);
+            appendSymlinkYamlFields(yamlLines, 'to_parent_path', toParentInfo);
+            return formatYamlCodeBlock(yamlLines.join('\n'));
         }
         catch (error) {
             const yaml = [

package/dist/tools/prompts/memory/en/principles.md

@@ -51,14 +51,20 @@ The memory toolset uses a **path key-value storage** model:
 ### 1. Path Naming Conventions
 
 - Use descriptive paths: `project/architecture`, `user/preferences/language`
-- Avoid special characters: Do not include `/`, `\`, `*`, etc. in paths
-- Use hierarchical structure: Organize memory by topic, e.g., `project/todo`, `project/done`
+- Use `/` to organize hierarchy (this is the recommended usage), with guardrails:
+  - Absolute paths are forbidden (must not start with `/`)
+  - Path traversal is forbidden (`..` is not allowed)
+  - Avoid `\\` (cross-platform readability; prefer `/`)
+- Keep paths _flat_: prefer a small number of topic files rather than a deep directory tree.
 
 ### 2. Content Format
 
-- Keep content concise: Each memory should cover only one topic
-- Use structured format: Can use Markdown format to organize content
-- Regular cleanup: Periodically check and delete outdated memories
+- Treat personal memory as a **carry-along stable-facts memo**: enable **0 ripgrep** startup within your scope.
+- Store stable facts only: **anchor points (file/symbol) + 1-line meaning + key contracts/priorities (≤3)**.
+- Fewer memory files is better: group facts that will be updated together into one file; avoid adding extra “directory-of-directory” layers.
+- Do not store task progress or daily state here:
+  - Team-visible progress belongs in Taskdoc `progress`
+  - Short-term working set belongs in reminders
 
 ### 3. Usage Scenarios
 
@@ -66,6 +72,8 @@ The memory toolset uses a **path key-value storage** model:
 - **Context memory**: Save important information from conversation context
 - **Preference settings**: Save user preferences and configuration information
 
+> Note: If you find yourself using personal memory to store “current progress of this run”, it likely belongs in Taskdoc `progress` or reminders instead.
+
 ## Relationship with Other Tools
 
 - **team_memory**: Team shared memory, visible to all members

package/dist/tools/prompts/memory/en/tools.md

@@ -30,12 +30,8 @@ Create new memory (when path does not exist).
 
 **Returns:**
 
-```yaml
-status: ok|error
-path: <memory path>
-content_size: <content size in bytes>
-created_at: <creation timestamp>
-```
+- Success: a short plain-text message in the work language (e.g. `Added`).
+- Failure: a plain-text error message (often starts with `Error:`) with an actionable next step (e.g. use `replace_memory`).
 
 **Errors:**
 
@@ -52,12 +48,8 @@ Update existing memory (when path exists).
 
 **Returns:**
 
-```yaml
-status: ok|error
-path: <memory path>
-content_size: <content size in bytes>
-updated_at: <update timestamp>
-```
+- Success: a short plain-text message in the work language (e.g. `Updated`).
+- Failure: a plain-text error message (often starts with `Error:`) with an actionable next step (e.g. use `add_memory`).
 
 **Errors:**
 
@@ -73,11 +65,8 @@ Delete specified memory.
 
 **Returns:**
 
-```yaml
-status: ok|error
-path: <memory path>
-deleted_at: <deletion timestamp>
-```
+- Success: a short plain-text message in the work language (e.g. `Deleted`).
+- Failure: a plain-text error message (often starts with `Error:`).
 
 **Errors:**
 
@@ -93,11 +82,8 @@ Clear all personal memory.
 
 **Returns:**
 
-```yaml
-status: ok|error
-cleared_count: <number of memories deleted>
-cleared_at: <deletion timestamp>
-```
+- Success: a short plain-text message in the work language (e.g. `Cleared`). If there is nothing to clear, returns a message like `No personal memory to clear.`
+- Failure: a plain-text error message.
 
 **Errors:**
 
@@ -137,18 +123,7 @@ drop_memory({
 clear_memory({});
 ```
 
-## YAML Output Contract
+## Output and Language
 
-All tool outputs use YAML format for programmatic processing:
-
-- `status`: Operation status, `ok` for success, `error` for failure
-- `path`: Memory path
-- Other fields: Additional information for specific operations
-
-On error, returns:
-
-```yaml
-status: error
-error_code: <error code>
-message: <error message>
-```
+- Output is **plain text**, not structured JSON/YAML.
+- The message language follows the current **work language**.

package/dist/tools/prompts/memory/zh/principles.md

@@ -51,14 +51,20 @@ memory 工具集采用**路径键值存储**模型：
 ### 1. 路径命名规范
 
 - 使用描述性路径：`project/architecture`、`user/preferences/language`
-- 避免使用特殊字符：路径中不要包含 `/`、`\`、`*` 等
-- 使用层级结构：按主题组织记忆，例如 `project/todo`、`project/done`
+- 允许用 `/` 组织层级（这就是推荐用法），但必须遵守护栏：
+  - 不允许绝对路径（不能以 `/` 开头）
+  - 不允许路径穿越（`..` 禁止）
+  - 避免使用 `\\`（跨平台可读性差，统一用 `/`）
+- 保持路径扁平：优先少量主题文件，而不是深层目录树（个人记忆的目标是“快速索引”，不是建文件系统）。
 
 ### 2. 内容格式
 
-- 保持内容简洁：每条记忆只包含一个主题
-- 使用结构化格式：可以使用 Markdown 格式组织内容
-- 定期清理：定期检查并删除过时的记忆
+- 把个人记忆当作“长期随身携带的稳定事实备忘”：让你在职责范围内尽量做到 **0 次 ripgrep 就能开工**。
+- 内容只写稳定事实：**落点（文件/符号）+ 一句话语义 + 关键契约/优先级（≤3 条）**。
+- 记忆文件更少更好：按“未来会一起更新的内容”合并在同一文件里；不要为了“看起来整齐”引入额外的“目录套目录”。
+- 不要把任务进度/当天状态写进个人记忆：
+  - 任务协作公告板用差遣牒（Taskdoc 的 `progress`）
+  - 短期工作集用提醒项（reminders）
 
 ### 3. 使用场景
 
@@ -66,6 +72,8 @@ memory 工具集采用**路径键值存储**模型：
 - **上下文记忆**：保存对话上下文中的重要信息
 - **偏好设置**：保存用户偏好和配置信息
 
+> 注：如果你发现自己在用个人记忆保存“这次对话的当前进展”，大概率应该写到 Taskdoc `progress` 或 reminders。
+
 ## 与其他工具的关系
 
 - **team_memory**：团队共享记忆，所有成员可见
@@ -74,6 +82,8 @@ memory 工具集采用**路径键值存储**模型：
 
 ## 限制与注意事项
 
-1. 记忆内容有大小限制（单条记忆最大 1MB）
-2. 记忆路径不能超过 255 个字符
-3. `clear_memory` 会删除所有记忆，**不可恢复**
+1. 路径护栏：不允许绝对路径；不允许包含 `..`。
+2. 内容护栏：`content` 不能为空字符串。
+3. `clear_memory` 会删除当前智能体的全部个人记忆，**不可恢复**。
+
+> 建议：即使当前实现没有硬性 size 限制，也应保持记忆内容精炼，否则会显著增加上下文成本与过期风险。