@inquiryon/openclaw-amp-governance 1.0.3 → 1.0.5

package/index.js

@@ -1,4 +1,5 @@
 import * as fs from 'fs';
+import { exec } from 'child_process';
 
 console.log('[AMP Governance] Plugin module loaded — Phase 4.');
 
@@ -23,6 +24,9 @@ try {
 // Module-level instance cache so we only init once per process lifetime
 let _instanceId = null;
 
+// Last known sender — populated by message_received, used for HITL notifications
+let _lastSender = null;  // { from: string, channelId: string }
+
 function readSession() {
   try {
     return JSON.parse(fs.readFileSync(SESSION_FILE, 'utf-8'));
@@ -111,6 +115,23 @@ async function ampLog(instanceId, message, level = 'INFO') {
   }
 }
 
+// ── PROACTIVE NOTIFICATIONS ──────────────────────────────────────────────────
+
+/**
+ * Send a proactive WhatsApp message to the user without waiting for the agent
+ * to finish. Uses the openclaw CLI so we don't need to reverse-engineer the
+ * gateway REST API.  Fire-and-forget — errors are logged but never thrown.
+ */
+function notifyUser(sender, message) {
+  if (!sender?.from || !sender?.channelId) return;
+  const safeMsg = message.replace(/"/g, '\\"');
+  const cmd = `openclaw message send --channel ${sender.channelId} --target "${sender.from}" --message "${safeMsg}"`;
+  console.log(`[AMP Governance] Sending notification to ${sender.from}: ${message}`);
+  exec(cmd, (err) => {
+    if (err) console.warn('[AMP Governance] notifyUser failed:', err.message);
+  });
+}
+
 // ── HITL POLICY ENGINE ───────────────────────────────────────────────────────
 
 /**
@@ -120,7 +141,7 @@ async function ampLog(instanceId, message, level = 'INFO') {
 async function requestHitlEval(instanceId, tool, params) {
   const res = await fetch(`${config.AMP_BACKEND_URL}/api/hitl/request`, {
     method: 'POST',
-    headers: { 'Content-Type': 'application/json' },
+    headers: { 'X-API-Key': config.AMP_API_KEY, 'Content-Type': 'application/json' },
     body: JSON.stringify({
       caller_id:  instanceId,
       instance_id: instanceId,
@@ -209,12 +230,16 @@ async function checkToolPolicy(instanceId, tool, params) {
     console.log(`[AMP Governance] HITL required for "${tool}" — waiting for human decision...`);
     await ampLog(instanceId, `HITL requested for tool: ${tool} — awaiting human approval in AMP`, 'WARN');
 
+    // Notify the user immediately so they aren't staring at silence
+    notifyUser(_lastSender, `I need a reviewer to approve "${tool}" before I can proceed. I'll follow up once the decision is made — this may take a few minutes.`);
+
     let decision;
     try {
       decision = await pollHitlDecision(instanceId);
     } catch (err) {
       const msg = `Tool call "${tool}" timed out waiting for human approval — blocked.`;
       await ampLog(instanceId, msg, 'ERROR');
+      notifyUser(_lastSender, `No response from the reviewer — "${tool}" was blocked due to timeout.`);
       return { block: true, blockReason: msg };
     }
 
@@ -223,11 +248,13 @@ async function checkToolPolicy(instanceId, tool, params) {
 
     if (resolution === 'approve' || resolution === 'approved') {
       await ampLog(instanceId, `HITL approved: ${tool} | workitem: ${decision.workitem_id}`);
+      notifyUser(_lastSender, `The reviewer approved "${tool}" — continuing now.`);
       return {};
     }
 
     if (resolution === 'modify' || resolution === 'modified') {
       await ampLog(instanceId, `HITL approved with modification: ${tool} | workitem: ${decision.workitem_id}`);
+      notifyUser(_lastSender, `The reviewer approved "${tool}" (with modifications) — continuing now.`);
       return {};
     }
 
@@ -235,6 +262,7 @@ async function checkToolPolicy(instanceId, tool, params) {
     const info = decision.information ? ` Reviewer note: ${decision.information}` : '';
     const msg = `Tool call "${tool}" was rejected by a human reviewer.${info}`;
     await ampLog(instanceId, `HITL rejected: ${tool} | workitem: ${decision.workitem_id}`, 'WARN');
+    notifyUser(_lastSender, `The reviewer rejected "${tool}".${info}`);
     return { block: true, blockReason: msg };
   }
 
@@ -360,6 +388,14 @@ export default {
   register(api) {
     api.logger.info('AMP Governance registered. Phase 4 - eval policy enforcement active.');
 
+    // ── INBOUND MESSAGE: cache sender for HITL notifications ─────────────────
+    api.on('message_received', (event, ctx) => {
+      if (event.from && ctx.channelId) {
+        _lastSender = { from: event.from, channelId: ctx.channelId };
+        console.log(`[AMP Governance] Sender cached: ${event.from} on ${ctx.channelId}`);
+      }
+    });
+
     // ── BEFORE TOOL CALL: governance check + logging ─────────────────────────
     api.on('before_tool_call', async (event) => {
       const tool = event.toolName || event.name || 'unknown-tool';
@@ -398,7 +434,7 @@ export default {
       await ampLog(instanceId, message, event.success === false ? 'ERROR' : 'INFO');
     });
 
-    // ── OUTBOUND MESSAGE LOGGING ──────────────────────────────────────────────
+    // ── OUTBOUND MESSAGE LOGGING + PREFIX ────────────────────────────────────
     api.on('message_sending', async (event) => {
       const instanceId = _instanceId || readSession()?.instanceId;
       const msgText    = event.content || event.text || event.message || JSON.stringify(event);
@@ -407,6 +443,8 @@ export default {
       if (instanceId) {
         await ampLog(instanceId, `Agent reply: ${preview}`);
       }
+      // Prefix every outbound message so the user can distinguish agent replies
+      return { content: `[OpenClaw]\n${msgText}` };
     });
   },
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@inquiryon/openclaw-amp-governance",
-  "version": "1.0.3",
+  "version": "1.0.5",
   "description": "AMP governance plugin for OpenClaw",
   "type": "module",
   "files": [