@happycastle/oh-my-openclaw 0.9.0 → 0.10.0

package/dist/hooks/persona-injector.d.ts

@@ -1,2 +1,6 @@
 import { OmocPluginApi } from '../types.js';
+export declare function resetPersonaInjectorState(): void;
+export declare function getPersonaInjectorState(): {
+    lastInjectedPersonaId: string | null;
+};
 export declare function registerPersonaInjector(api: OmocPluginApi): void;

package/dist/hooks/persona-injector.js

@@ -1,5 +1,12 @@
 import { getActivePersona } from '../utils/persona-state.js';
 import { readPersonaPromptSync } from '../agents/persona-prompts.js';
+let lastInjectedPersonaId = null;
+export function resetPersonaInjectorState() {
+    lastInjectedPersonaId = null;
+}
+export function getPersonaInjectorState() {
+    return { lastInjectedPersonaId };
+}
 export function registerPersonaInjector(api) {
     api.registerHook('agent:bootstrap', (event) => {
         const personaId = getActivePersona();
@@ -9,12 +16,20 @@ export function registerPersonaInjector(api) {
         if (!event.context.bootstrapFiles) {
             event.context.bootstrapFiles = [];
         }
+        const alreadyInFiles = event.context.bootstrapFiles.some((f) => f.path.startsWith('omoc://persona/'));
+        if (alreadyInFiles) {
+            return;
+        }
+        if (lastInjectedPersonaId === personaId) {
+            return;
+        }
         try {
             const content = readPersonaPromptSync(personaId);
             event.context.bootstrapFiles.push({
                 path: `omoc://persona/${personaId}`,
                 content,
             });
+            lastInjectedPersonaId = personaId;
             api.logger.info(`[omoc] Persona injected: ${personaId}`);
         }
         catch (err) {
@@ -22,6 +37,6 @@ export function registerPersonaInjector(api) {
         }
     }, {
         name: 'oh-my-openclaw.persona-injector',
-        description: 'Injects active persona prompt into agent bootstrap',
+        description: 'Injects active persona prompt once per persona change',
     });
 }

package/dist/hooks/todo-enforcer.d.ts

@@ -1,8 +1,6 @@
 import { OmocPluginApi } from '../types.js';
+export type AgentRole = 'orchestrator' | 'worker' | 'lightweight';
+export declare function classifyAgentRole(agentId?: string): AgentRole;
 export declare function resetEnforcerState(): void;
-export declare function getEnforcerState(): {
-    lastInjectionTime: number;
-    consecutiveFailures: number;
-    disabledByFailures: boolean;
-};
+export declare function getEnforcerState(): {};
 export declare function registerTodoEnforcer(api: OmocPluginApi): void;

package/dist/hooks/todo-enforcer.js

@@ -1,28 +1,46 @@
 import { getConfig } from '../utils/config.js';
-const DIRECTIVE_TEXT = `[SYSTEM DIRECTIVE: OH-MY-OPENCLAW - TODO CONTINUATION]
-You MUST continue working on incomplete todos.
-- Do NOT stop until all tasks are marked complete
-- Do NOT ask for permission to continue
+const ORCHESTRATOR_IDS = new Set([
+    'omoc_prometheus',
+    'omoc_atlas',
+]);
+const WORKER_IDS = new Set([
+    'omoc_sisyphus',
+    'omoc_hephaestus',
+    'omoc_frontend',
+]);
+export function classifyAgentRole(agentId) {
+    if (!agentId)
+        return 'orchestrator';
+    if (ORCHESTRATOR_IDS.has(agentId))
+        return 'orchestrator';
+    if (WORKER_IDS.has(agentId))
+        return 'worker';
+    return 'lightweight';
+}
+const ORCHESTRATOR_DIRECTIVE = `[SYSTEM REMINDER - TODO CONTINUATION]
+If you have incomplete todos, continue working on them.
 - Mark each task complete immediately when finished
 - If blocked, document the blocker and move to next task
+- Do NOT restate prior messages — output only deltas and next concrete action
+- If no actionable next step remains, declare tasks complete and stop
 
-[SUBAGENT COMPLETION RULE]
-When you receive a subagent completion notification ("✅ Subagent finished"):
-- This is an ACTION TRIGGER, not an FYI
-- IMMEDIATELY check the subagent's result
-- Verify against success criteria
-- Then proceed to the next task/phase
-- NEVER stop after receiving a completion notification`;
-let lastInjectionTime = 0;
-let consecutiveFailures = 0;
-let disabledByFailures = false;
+When you receive a subagent completion notification:
+- Check the subagent's result against success criteria
+- Then proceed to the next task/phase`;
+const WORKER_DIRECTIVE = `[SYSTEM REMINDER - TASK COMPLETION]
+Complete your assigned task, return the result, then stop.
+- Do NOT restate prior messages — output only new findings or changes
+- If blocked, report the blocker and stop`;
+const DIRECTIVES = {
+    orchestrator: ORCHESTRATOR_DIRECTIVE,
+    worker: WORKER_DIRECTIVE,
+    lightweight: null,
+};
 export function resetEnforcerState() {
-    lastInjectionTime = 0;
-    consecutiveFailures = 0;
-    disabledByFailures = false;
+    // no-op — kept for API compatibility (no global state to reset)
 }
 export function getEnforcerState() {
-    return { lastInjectionTime, consecutiveFailures, disabledByFailures };
+    return {};
 }
 export function registerTodoEnforcer(api) {
     api.registerHook('agent:bootstrap', (event) => {
@@ -30,36 +48,30 @@ export function registerTodoEnforcer(api) {
         if (!config.todo_enforcer_enabled) {
             return;
         }
-        if (disabledByFailures) {
-            api.logger.warn('[omoc] Todo enforcer disabled due to consecutive failures');
-            return;
-        }
-        const now = Date.now();
-        if (config.todo_enforcer_cooldown_ms > 0 && (now - lastInjectionTime) < config.todo_enforcer_cooldown_ms) {
-            api.logger.info('[omoc] Todo enforcer skipped (cooldown)');
+        const role = classifyAgentRole(event.context.agentId);
+        const directive = DIRECTIVES[role];
+        if (!directive) {
             return;
         }
         if (!event.context.bootstrapFiles) {
             event.context.bootstrapFiles = [];
         }
+        const alreadyInjected = event.context.bootstrapFiles.some((f) => f.path === 'omoc://todo-enforcer');
+        if (alreadyInjected) {
+            return;
+        }
         try {
             event.context.bootstrapFiles.push({
                 path: 'omoc://todo-enforcer',
-                content: DIRECTIVE_TEXT,
+                content: directive,
             });
-            lastInjectionTime = now;
-            consecutiveFailures = 0;
-            api.logger.info('[omoc] Todo enforcer directive injected');
+            api.logger.info(`[omoc] Todo enforcer injected (role: ${role})`);
         }
-        catch {
-            consecutiveFailures++;
-            if (config.todo_enforcer_max_failures > 0 && consecutiveFailures >= config.todo_enforcer_max_failures) {
-                disabledByFailures = true;
-                api.logger.error(`[omoc] Todo enforcer disabled after ${consecutiveFailures} consecutive failures`);
-            }
+        catch (err) {
+            api.logger.error('[omoc] Todo enforcer injection failed:', err);
         }
     }, {
         name: 'oh-my-openclaw.todo-enforcer',
-        description: 'Injects TODO continuation directive into agent bootstrap',
+        description: 'Injects role-aware TODO directive into agent bootstrap',
     });
 }

package/package.json

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