refactor(openclaw): drop unused retain prefix config

Author: Aldoustheorchestrator

hindsight-integrations/openclaw/README.md

@@ -51,8 +51,6 @@ Optional settings in `~/.openclaw/openclaw.json` under `plugins.entries.hindsigh
 | `bankIdPrefix` | — | Prefix for bank IDs (e.g. `"prod"`) |
 | `retainTags` | `[]` | Tags applied to every retained document, useful for cross-agent/source labeling (e.g. `source_system:openclaw`, `agent:agentname`) |
 | `retainSource` | `"openclaw"` | `source` value written into retained document metadata |
-| `retainUserPrefix` | `"User"` | Reserved for future transcript-label parity work. Current retained transcripts still use structured role markers. |
-| `retainAssistantPrefix` | `"Assistant"` | Reserved for future transcript-label parity work. Current retained transcripts still use structured role markers. |
 | `dynamicBankGranularity` | `["agent", "channel", "user"]` | Fields used to derive bank ID. Options: `agent`, `channel`, `user`, `provider` |
 | `excludeProviders` | `["heartbeat"]` | Message providers to skip for recall/retain (e.g. `heartbeat`, `slack`, `telegram`, `discord`) |
 | `autoRecall` | `true` | Auto-inject memories before each turn. Set to `false` when the agent has its own recall tool. |

hindsight-integrations/openclaw/openclaw.plugin.json

@@ -88,16 +88,6 @@
         "description": "Source value written into retained document metadata. Defaults to 'openclaw'.",
         "default": "openclaw"
       },
-      "retainUserPrefix": {
-        "type": "string",
-        "description": "Reserved for future transcript-label parity work. Currently retained transcripts continue using structured role markers.",
-        "default": "User"
-      },
-      "retainAssistantPrefix": {
-        "type": "string",
-        "description": "Reserved for future transcript-label parity work. Currently retained transcripts continue using structured role markers.",
-        "default": "Assistant"
-      },
       "autoRecall": {
         "type": "boolean",
         "description": "Automatically recall memories on every prompt and inject them as context. Set to false when agent has its own recall tool.",

hindsight-integrations/openclaw/src/index.test.ts

@@ -317,8 +317,6 @@ describe('prepareRetentionTranscript', () => {
   const baseConfig: PluginConfig = {
     dynamicBankId: true,
     retainRoles: ['user', 'assistant'],
-    retainUserPrefix: 'User',
-    retainAssistantPrefix: 'Assistant',
   };
 
   it('returns null if no user message found (turn boundary)', () => {
@@ -344,17 +342,6 @@ describe('prepareRetentionTranscript', () => {
     expect(result?.transcript).not.toContain('Old user');
   });
 
-  it('retains structured role markers even when prefixes are configured', () => {
-    const config: PluginConfig = { ...baseConfig, retainUserPrefix: 'Operator', retainAssistantPrefix: 'Aldous' };
-    const messages = [
-      { role: 'user', content: 'Hello there' },
-      { role: 'assistant', content: 'General Kenobi' }
-    ];
-    const result = prepareRetentionTranscript(messages, config);
-    expect(result?.transcript).toContain('[role: user]\nHello there\n[user:end]');
-    expect(result?.transcript).toContain('[role: assistant]\nGeneral Kenobi\n[assistant:end]');
-  });
-
   it('filters out excluded roles', () => {
     const config: PluginConfig = { ...baseConfig, retainRoles: ['user'] };
     const messages = [

hindsight-integrations/openclaw/src/index.ts

@@ -840,8 +840,6 @@ function getPluginConfig(api: MoltbotPluginAPI): PluginConfig {
     bankIdPrefix: config.bankIdPrefix,
     retainTags: normalizeRetainTags(config.retainTags),
     retainSource: typeof config.retainSource === 'string' && config.retainSource.trim().length > 0 ? config.retainSource.trim() : undefined,
-    retainUserPrefix: typeof config.retainUserPrefix === 'string' && config.retainUserPrefix.trim().length > 0 ? config.retainUserPrefix.trim() : 'User',
-    retainAssistantPrefix: typeof config.retainAssistantPrefix === 'string' && config.retainAssistantPrefix.trim().length > 0 ? config.retainAssistantPrefix.trim() : 'Assistant',
     excludeProviders: Array.isArray(config.excludeProviders)
       ? Array.from(new Set(['heartbeat', ...config.excludeProviders.filter((provider): provider is string => typeof provider === 'string')]))
       : ['heartbeat'],

hindsight-integrations/openclaw/src/types.ts

@@ -66,8 +66,6 @@ export interface PluginConfig {
   bankIdPrefix?: string; // Prefix for bank IDs (e.g. 'prod' -> 'prod-slack-C123')
   retainTags?: string[]; // Default tags applied to retained documents; merged with per-call tags when provided
   retainSource?: string; // Source written into retained document metadata (default: 'openclaw')
-  retainUserPrefix?: string; // Label used before user turns in retained transcripts (default: 'User')
-  retainAssistantPrefix?: string; // Label used before assistant turns in retained transcripts (default: 'Assistant')
   excludeProviders?: string[]; // Message providers to exclude from recall/retain (e.g. ['telegram', 'discord'])
   autoRecall?: boolean; // Auto-recall memories on every prompt (default: true). Set to false when agent has its own recall tool.
   dynamicBankGranularity?: Array<'agent' | 'provider' | 'channel' | 'user'>; // Fields for bank ID derivation. Default: ['agent', 'channel', 'user']