principles-disciple 1.101.0 → 1.102.1

package/openclaw.plugin.json

@@ -2,7 +2,7 @@
   "id": "principles-disciple",
   "name": "Principles Disciple",
   "description": "Evolutionary programming agent framework with strategic guardrails and reflection loops.",
-  "version": "1.101.0",
+  "version": "1.102.1",
   "activation": {
     "onCapabilities": [
       "hook"

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "principles-disciple",
-  "version": "1.101.0",
+  "version": "1.102.1",
   "description": "Native OpenClaw plugin for Principles Disciple",
   "type": "module",
   "main": "./dist/bundle.js",

package/src/core/config-health.ts

@@ -10,6 +10,20 @@
  * needs to check conversation access state (PRI-346).
  */
 
+/**
+ * PRI-348: Extract the full plugin entry (including hooks) from the global OpenClaw config.
+ * Unlike api.pluginConfig (which is only the entry.config sub-object), this returns
+ * the entire entry including hooks, config, enabled, etc.
+ */
+export function getPluginEntry(config: unknown, pluginId: string): unknown {
+  if (!config || typeof config !== 'object' || Array.isArray(config)) return undefined;
+  const plugins = (config as Record<string, unknown>).plugins;
+  if (!plugins || typeof plugins !== 'object' || Array.isArray(plugins)) return undefined;
+  const entries = (plugins as Record<string, unknown>).entries;
+  if (!entries || typeof entries !== 'object' || Array.isArray(entries)) return undefined;
+  return (entries as Record<string, unknown>)[pluginId];
+}
+
 /** Keep in sync with @principles/core CONVERSATION_ACCESS_CONFIG_KEY */
 const CONVERSATION_ACCESS_CONFIG_KEY = 'allowConversationAccess' as const;
 

package/src/core/runtime-v2-prompt-activation-reader.ts

@@ -50,9 +50,9 @@ export class PromptActivationReader {
         }
 
         if (artifactRow === null) {
-          const warning = `artifact_not_found: artifactId=${activation.artifactId}; nextAction=check_pi_artifacts_table`;
+          const warning = `artifact_not_found: artifactId=${activation.artifactId}; nextAction=check_pi_artifacts_table_or_remove_stale_activation`;
           warnings.push(warning);
-          this.deps.logger?.warn?.(`[PD:RuntimeV2] ${warning}`);
+          this.deps.logger?.info?.(`[PD:RuntimeV2] ${warning}`);
           continue;
         }
 

package/src/index.ts

@@ -20,8 +20,8 @@ import type {
 } from './openclaw-sdk.js';
 import * as path from 'path';
 import { loadFeatureFlagFromConfig } from './core/pd-config-loader.js';
-import { checkConversationAccessConfig } from './core/config-health.js';
-export { checkConversationAccessConfig } from './core/config-health.js';
+import { checkConversationAccessConfig, getPluginEntry } from './core/config-health.js';
+export { checkConversationAccessConfig, getPluginEntry } from './core/config-health.js';
 export type { ConversationAccessCheckResult } from './core/config-health.js';
 import { classifyTask } from './core/local-worker-routing.js';
 import { completeShadowObservation, recordShadowRouting } from './core/shadow-observation-registry.js';
@@ -171,7 +171,7 @@ const plugin = {
       }
 
       // PRI-343: Check allowConversationAccess — warn if llm_output/trajectory hooks blocked
-      const accessCheck = checkConversationAccessConfig(api.pluginConfig);
+      const accessCheck = checkConversationAccessConfig(getPluginEntry(api.config, api.id));
       if (!accessCheck.authorized) {
         api.logger.error(
           `[PD:health] conversation hooks (llm_output / trajectory) will be BLOCKED by OpenClaw.\n` +
@@ -539,7 +539,7 @@ const plugin = {
           TrajectoryCollector.handleBeforeMessageWrite(event, {
             ...ctx,
             workspaceDir: wsResult.workspaceDir,
-            pluginConfig: api.pluginConfig,
+            pluginConfig: getPluginEntry(api.config, api.id),
           });
         } catch (err) {
           // Non-critical: don't surface to user

package/src/openclaw-sdk.ts

@@ -80,6 +80,7 @@ export interface OpenClawPluginService {
 }
 
 export interface OpenClawPluginApi {
+  id: string;
   rootDir?: string;
   pluginConfig?: Record<string, unknown>;
   logger: PluginLogger;

package/tests/hooks/runtime-v2-prompt-activation.test.ts

@@ -387,13 +387,13 @@ describe('Runtime V2 prompt activation injection', () => {
 
     await insertPromptActivation({ artifactId, principleId });
 
-    const warnSpy = vi.fn();
+    const infoSpy = vi.fn();
     const ctx = {
       workspaceDir: tempWorkspaceDir,
       trigger: 'user',
       sessionId: 'test-session-v2',
       api: {
-        logger: { info: vi.fn(), warn: warnSpy, error: vi.fn() },
+        logger: { info: infoSpy, warn: vi.fn(), error: vi.fn() },
         runtime: {},
         config: {},
       },
@@ -404,8 +404,8 @@ describe('Runtime V2 prompt activation injection', () => {
 
     expect(result).toBeDefined();
     expect(result?.appendSystemContext).not.toContain(TEST_PRINCIPLE_TEXT);
-    const warnCalls = warnSpy.mock.calls.map((c: unknown[]) => String(c[0]));
-    const hasActivationWarning = warnCalls.some((c: string) => c.includes('artifact_not_found') || c.includes('artifact_query_unexpected') || c.includes('activation'));
+    const infoCalls = infoSpy.mock.calls.map((c: unknown[]) => String(c[0]));
+    const hasActivationWarning = infoCalls.some((c: string) => c.includes('artifact_not_found') || c.includes('artifact_query_unexpected') || c.includes('activation'));
     expect(hasActivationWarning).toBe(true);
   });