@happycastle/oh-my-openclaw 0.12.2 → 0.12.4

package/dist/commands/persona-commands.js

@@ -1,29 +1,39 @@
 import { getActivePersona, setActivePersona, resetPersonaState } from '../utils/persona-state.js';
 import { resolvePersonaId, listPersonas, DEFAULT_PERSONA_ID } from '../agents/persona-prompts.js';
+import { resetPersonaInjectorState } from '../hooks/persona-injector.js';
+function getDisplayName(personaId) {
+    const persona = listPersonas().find((p) => p.id === personaId);
+    return persona ? `${persona.emoji} ${persona.displayName}` : personaId;
+}
 export function registerPersonaCommands(api) {
     api.registerCommand({
         name: 'omoc',
         description: 'OmOC mode — activate, switch, or list personas',
         acceptsArgs: true,
         handler: async (ctx) => {
+            api.logger.info(`[omoc] /omoc command received — raw ctx.args: ${JSON.stringify(ctx.args)}, ctx keys: ${JSON.stringify(Object.keys(ctx))}`);
             const args = (ctx.args ?? '').trim().toLowerCase();
+            api.logger.info(`[omoc] Parsed args: "${args}" (length: ${args.length})`);
             if (!args) {
+                const previousId = getActivePersona();
+                resetPersonaInjectorState();
                 setActivePersona(DEFAULT_PERSONA_ID);
-                const personas = listPersonas();
-                const defaultPersona = personas.find((p) => p.id === DEFAULT_PERSONA_ID);
-                const name = defaultPersona
-                    ? `${defaultPersona.emoji} ${defaultPersona.displayName}`
-                    : DEFAULT_PERSONA_ID;
+                const name = getDisplayName(DEFAULT_PERSONA_ID);
+                const switchNote = previousId && previousId !== DEFAULT_PERSONA_ID
+                    ? `\n\nSwitched from **${getDisplayName(previousId)}**.`
+                    : '';
                 return {
-                    text: `# OmOC Mode: ON\n\nActive persona: **${name}**\n\nThe persona prompt will be injected into all new agent sessions.\n\nUse \`/omoc list\` to see available personas, or \`/omoc <name>\` to switch.`,
+                    text: `# OmOC Mode: ON\n\nActive persona: **${name}**${switchNote}\n\nApplied immediately — your next message will use this persona.\n\nUse \`/omoc list\` to see available personas, or \`/omoc <name>\` to switch.`,
                 };
             }
             if (args === 'off') {
                 const wasActive = getActivePersona();
+                const wasName = wasActive ? getDisplayName(wasActive) : null;
                 resetPersonaState();
+                resetPersonaInjectorState();
                 return {
-                    text: wasActive
-                        ? `# OmOC Mode: OFF\n\nPersona **${wasActive}** deactivated. Agent sessions will use default behavior.`
+                    text: wasName
+                        ? `# OmOC Mode: OFF\n\nPersona **${wasName}** deactivated. Applied immediately — your next message will use default behavior.`
                         : '# OmOC Mode: OFF\n\nNo persona was active.',
                 };
             }
@@ -38,7 +48,7 @@ export function registerPersonaCommands(api) {
                     text: [
                         '# OmOC Personas',
                         '',
-                        `Active: ${activeId ? `**${activeId}**` : '_none_'}`,
+                        `Active: ${activeId ? `**${getDisplayName(activeId)}**` : '_none_'}`,
                         '',
                         '| | Command | Name | Role |',
                         '|---|---------|------|------|',
@@ -56,14 +66,16 @@ export function registerPersonaCommands(api) {
                     text: `# Unknown Persona: "${args}"\n\nAvailable personas: ${available}\n\nUse \`/omoc list\` for details.`,
                 };
             }
+            const previousId = getActivePersona();
+            resetPersonaInjectorState();
             setActivePersona(resolvedId);
-            const personas = listPersonas();
-            const switched = personas.find((p) => p.id === resolvedId);
-            const displayName = switched
-                ? `${switched.emoji} ${switched.displayName}`
-                : resolvedId;
+            const displayName = getDisplayName(resolvedId);
+            const switched = listPersonas().find((p) => p.id === resolvedId);
+            const switchNote = previousId && previousId !== resolvedId
+                ? `\n\nSwitched from **${getDisplayName(previousId)}**.`
+                : '';
             return {
-                text: `# Persona Switched\n\nActive persona: **${displayName}**\n\nThe ${switched?.theme ?? 'persona'} prompt will be injected into all new agent sessions.`,
+                text: `# Persona Switched\n\nActive persona: **${displayName}**${switchNote}\n\nApplied immediately — your next message will use the ${switched?.theme ?? 'persona'} prompt.`,
             };
         },
     });

package/dist/hooks/persona-injector.js

@@ -26,6 +26,11 @@ export function registerPersonaInjector(api) {
         const personaId = getActivePersona();
         const sessionKey = event.context.agentId || 'default';
         if (!personaId) {
+            if (lastInjectedPersonaId) {
+                contextCollector.unregister(sessionKey, `persona/${lastInjectedPersonaId}`);
+                lastInjectedPersonaId = null;
+                api.logger.info(`[omoc] Persona context cleared for ${sessionKey}`);
+            }
             return;
         }
         if (lastInjectedPersonaId === personaId) {

package/dist/index.js

@@ -16,7 +16,17 @@ import { registerPersonaCommands } from './commands/persona-commands.js';
 import { registerPersonaInjector } from './hooks/persona-injector.js';
 import { registerContextInjector } from './hooks/context-injector.js';
 import { registerSetupCli } from './cli/setup.js';
-let initialized = false;
+/**
+ * Generation counter for multi-registration handling.
+ *
+ * OpenClaw may call register() multiple times with different api instances.
+ * Only the LAST call's api connects to the live hook dispatcher.
+ * We track a generation number so that hooks from stale registrations
+ * become no-ops, solving both:
+ *   - hooks not firing (stale api from early call)
+ *   - triple firing (all 3 registrations' hooks executing)
+ */
+let generation = 0;
 const registry = {
     hooks: [],
     services: [],
@@ -24,16 +34,30 @@ const registry = {
     commands: [],
     cli: [],
 };
+function guardedApi(api, gen) {
+    return {
+        ...api,
+        registerHook: (event, handler, meta) => {
+            api.registerHook(event, (evt) => {
+                if (gen !== generation)
+                    return evt;
+                return handler(evt);
+            }, meta);
+        },
+    };
+}
 export default function register(api) {
-    if (initialized) {
-        api.logger.warn(`[${PLUGIN_ID}] Plugin already initialized — skipping duplicate register() call`);
-        return;
-    }
-    initialized = true;
+    const gen = ++generation;
+    registry.hooks = [];
+    registry.services = [];
+    registry.tools = [];
+    registry.commands = [];
+    registry.cli = [];
     const config = getConfig(api);
-    api.logger.info(`[${PLUGIN_ID}] Initializing plugin v${VERSION}`);
+    api.logger.info(`[${PLUGIN_ID}] Initializing plugin v${VERSION} (register call #${gen})`);
+    const guarded = guardedApi(api, gen);
     try {
-        registerTodoEnforcer(api);
+        registerTodoEnforcer(guarded);
         registry.hooks.push('todo-enforcer');
         api.logger.info(`[${PLUGIN_ID}] Todo Enforcer hook registered (enabled: ${config.todo_enforcer_enabled})`);
     }
@@ -41,7 +65,7 @@ export default function register(api) {
         api.logger.error(`[${PLUGIN_ID}] Failed to register Todo Enforcer:`, err);
     }
     try {
-        registerCommentChecker(api);
+        registerCommentChecker(guarded);
         registry.hooks.push('comment-checker');
         api.logger.info(`[${PLUGIN_ID}] Comment Checker hook registered (enabled: ${config.comment_checker_enabled})`);
     }
@@ -49,7 +73,7 @@ export default function register(api) {
         api.logger.error(`[${PLUGIN_ID}] Failed to register Comment Checker:`, err);
     }
     try {
-        registerMessageMonitor(api);
+        registerMessageMonitor(guarded);
         registry.hooks.push('message-monitor', 'message-received-monitor');
         api.logger.info(`[${PLUGIN_ID}] Message Monitor hook registered`);
     }
@@ -57,7 +81,7 @@ export default function register(api) {
         api.logger.error(`[${PLUGIN_ID}] Failed to register Message Monitor:`, err);
     }
     try {
-        registerStartupHook(api);
+        registerStartupHook(guarded);
         registry.hooks.push('gateway-startup');
         api.logger.info(`[${PLUGIN_ID}] Gateway startup hook registered`);
     }
@@ -65,7 +89,7 @@ export default function register(api) {
         api.logger.error(`[${PLUGIN_ID}] Failed to register startup hook:`, err);
     }
     try {
-        registerPersonaInjector(api);
+        registerPersonaInjector(guarded);
         registry.hooks.push('persona-injector');
         api.logger.info(`[${PLUGIN_ID}] Persona injector hook registered`);
     }
@@ -73,7 +97,7 @@ export default function register(api) {
         api.logger.error(`[${PLUGIN_ID}] Failed to register Persona Injector:`, err);
     }
     try {
-        registerContextInjector(api);
+        registerContextInjector(guarded);
         registry.hooks.push('context-injector');
         api.logger.info(`[${PLUGIN_ID}] Context injector hook registered (before_prompt_build)`);
     }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@happycastle/oh-my-openclaw",
-  "version": "0.12.2",
+  "version": "0.12.4",
   "description": "Oh-My-OpenClaw plugin — multi-agent orchestration, todo enforcer, ralph loop, and custom tools for OpenClaw",
   "type": "module",
   "main": "dist/index.js",