nyrn
/
openclaw
mirror of https://github.com/openclaw/openclaw.git
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

fix: address Codex review — shallow-copy meta before namespacing, keep hardcoded fallback after pattern strip

This commit is contained in:
scotthuang 2026-03-15 18:39:57 +08:00
parent b113519c6f
commit 66b35e00b6
2 changed files with 7 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/plugins/hooks.ts
View File

@ -186,7 +186,9 @@ function namespaceMessageMeta(
if (!result?.messageMeta || Object.keys(result.messageMeta).length === 0) {
return;
}
result.messageMeta = { [pluginId]: result.messageMeta };
// Shallow-copy the original meta before wrapping so we never mutate the
// plugin's returned object (which may be a reused module-level constant).
result.messageMeta = { [pluginId]: { ...result.messageMeta } };
}
/**

5
ui/src/ui/chat/message-extract.ts
View File

@ -36,7 +36,10 @@ function stripDisplayPatterns(text: string, message: unknown): string {
// skip invalid regex
}
}
return result.trimStart();
// Still run hardcoded fallback after pattern-driven strip: if the
// plugin-provided regex was malformed or missed a variant format,
// the safety net catches it.
return stripRelevantMemoriesTags(result.trimStart());
}
// Hardcoded fallback: strip <relevant-memories> tags even when messageMeta