principles-disciple 1.184.0 → 1.185.0

package/dist/service/evolution-dedup.d.ts

@@ -4,7 +4,7 @@
  * Dedup logic for preventing duplicate pain tasks and redundant reflections.
  * Extracted from evolution-worker.ts.
  */
-import type { EvolutionQueueItem } from './evolution-queue-migration.js';
+import type { EvolutionQueueItem } from '../core/evolution-types.js';
 /**
  * Dedup window for pain queue tasks (30 minutes).
  */

package/dist/service/evolution-worker.d.ts

@@ -1,6 +1,5 @@
 import type { OpenClawPluginServiceContext, OpenClawPluginApi, PluginLogger } from '../openclaw-sdk.js';
 import { WorkspaceContext } from '../core/workspace-context.js';
-import type { TaskKind, TaskPriority } from '../core/trajectory-types.js';
 export type { TaskKind, TaskPriority } from '../core/trajectory-types.js';
 export { loadEvolutionQueue, saveEvolutionQueue, withQueueLock, acquireQueueLock, requireQueueLock } from './queue-io.js';
 export { EVOLUTION_QUEUE_LOCK_SUFFIX, LOCK_MAX_RETRIES, LOCK_RETRY_DELAY_MS, LOCK_STALE_MS } from './queue-io.js';
@@ -24,36 +23,13 @@ export type { WatchdogResult };
  * Pain diagnosis is Runtime v2 only: after_tool_call / pd pain record ->
  * PainSignalBridge -> SplitDiagnosticianRunner. EvolutionWorker does not read
  * .pain_flag or process pain_diagnosis queue items.
+ *
+ * Types (QueueStatus / TaskResolution / EvolutionQueueItem) are re-exported
+ * from queue-migration.ts, which sources them from evolution-types.ts
+ * (canonical single source of truth).
  */
-/** @deprecated Use PDTaskStatus from '@principles/core/runtime-v2'. M2 migration will replace this. */
-export type QueueStatus = 'pending' | 'in_progress' | 'completed' | 'failed' | 'canceled';
-export type TaskResolution = 'marker_detected' | 'auto_completed_timeout' | 'failed_max_retries' | 'runtime_unavailable' | 'canceled' | 'late_marker_principle_created' | 'late_marker_no_principle' | 'stub_fallback' | 'skipped_thin_violation' | 'noise_classified' | 'retired';
-export interface EvolutionQueueItem {
-    id: string;
-    taskKind: TaskKind;
-    priority: TaskPriority;
-    source: string;
-    traceId?: string;
-    task?: string;
-    score: number;
-    reason: string;
-    timestamp: string;
-    enqueued_at?: string;
-    started_at?: string;
-    completed_at?: string;
-    assigned_session_key?: string;
-    trigger_text_preview?: string;
-    status: QueueStatus;
-    resolution?: TaskResolution;
-    session_id?: string;
-    agent_id?: string;
-    retryCount: number;
-    maxRetries: number;
-    lastError?: string;
-    resultRef?: string;
-    painEventId?: number;
-}
-import { migrateToV2, isLegacyQueueItem, migrateQueueToV2, LegacyEvolutionQueueItem, DEFAULT_TASK_KIND, DEFAULT_PRIORITY, DEFAULT_MAX_RETRIES, validateQueueItem, VALID_TASK_KINDS, isValidQueueItem, type RawQueueItem } from './queue-migration.js';
+export type { QueueStatus, TaskResolution, EvolutionQueueItem } from './queue-migration.js';
+import { migrateToV2, isLegacyQueueItem, migrateQueueToV2, LegacyEvolutionQueueItem, DEFAULT_TASK_KIND, DEFAULT_PRIORITY, DEFAULT_MAX_RETRIES, validateQueueItem, VALID_TASK_KINDS, isValidQueueItem, type RawQueueItem, type EvolutionQueueItem } from './queue-migration.js';
 export { migrateToV2, isLegacyQueueItem, migrateQueueToV2, LegacyEvolutionQueueItem, DEFAULT_TASK_KIND, DEFAULT_PRIORITY, DEFAULT_MAX_RETRIES, validateQueueItem, VALID_TASK_KINDS, isValidQueueItem };
 export type { RawQueueItem };
 export declare function extractEvolutionTaskId(task: string): string | null;

package/dist/service/evolution-worker.js

@@ -347,11 +347,6 @@ export async function registerEvolutionTaskSession(workspaceResolve, taskId, ses
         // Previously this inlined JSON.parse + migrate cast without validating,
         // so malformed queue items could reach the find() below. Reusing the
         // shared loader keeps a single chokepoint for queue validation.
-        // Note: loadEvolutionQueue returns core/evolution-types.EvolutionQueueItem,
-        // which saveEvolutionQueue also accepts. We deliberately avoid annotating
-        // with the local EvolutionQueueItem interface (line 87) to prevent the
-        // known type-drift between the two definitions (see follow-up: unify the
-        // three EvolutionQueueItem copies).
         const queue = loadEvolutionQueue(queuePath);
         const task = queue.find((item) => item.id === taskId && item.status === 'in_progress');
         if (!task) {
@@ -388,7 +383,11 @@ async function processEvolutionQueueWithResult(wctx, logger, eventLog, api) {
         if (!fs.existsSync(queuePath)) {
             return { queue: queueResult, errors };
         }
-        const queue = JSON.parse(fs.readFileSync(queuePath, 'utf8'));
+        // rc-1/rc-2/rc-4 (ERR-001/ERR-005/ERR-007): use the canonical loader
+        // instead of raw JSON.parse + cast. loadEvolutionQueue handles parse,
+        // migrate, and element-wise validation; malformed items are dropped
+        // with a console.warn (rc-9: no silent fallback).
+        const queue = loadEvolutionQueue(queuePath);
         // Purge stale failed tasks before processing (keeps queue lean)
         const purgeResult = purgeStaleFailedTasks(queue, logger);
         if (purgeResult.purged > 0) {

package/dist/service/queue-migration.d.ts

@@ -5,32 +5,8 @@
  * to the V2 schema. Zero I/O, zero imports from evolution-worker.ts.
  */
 import type { TaskKind, TaskPriority } from '../core/trajectory-types.js';
-export type QueueStatus = 'pending' | 'in_progress' | 'completed' | 'failed';
-export type TaskResolution = 'success' | 'failure' | 'skipped';
-export interface EvolutionQueueItem {
-    id: string;
-    taskKind: TaskKind;
-    priority: TaskPriority;
-    source: string;
-    traceId?: string;
-    task?: string;
-    score: number;
-    reason: string;
-    timestamp: string;
-    enqueued_at?: string;
-    started_at?: string;
-    completed_at?: string;
-    assigned_session_key?: string;
-    trigger_text_preview?: string;
-    status: QueueStatus;
-    resolution?: TaskResolution;
-    session_id?: string;
-    agent_id?: string;
-    retryCount: number;
-    maxRetries: number;
-    lastError?: string;
-    resultRef?: string;
-}
+import type { EvolutionQueueItem } from '../core/evolution-types.js';
+export type { QueueStatus, TaskResolution, EvolutionQueueItem } from '../core/evolution-types.js';
 /**
  * Legacy queue item shape (pre-V2) for migration compatibility.
  * These items lack taskKind, priority, retryCount, maxRetries, lastError fields.

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.184.0",
+  "version": "1.185.0",
   "activation": {
     "onCapabilities": [
       "hook"

package/package.json

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

package/dist/service/evolution-queue-migration.d.ts

@@ -1,128 +0,0 @@
-/**
- * Evolution Queue Migration — V1 to V2 Schema
- *
- * Pure transformation functions and shared queue types.
- */
-import type { TaskKind, TaskPriority } from '../core/trajectory-types.js';
-export type { TaskKind, TaskPriority } from '../core/trajectory-types.js';
-/**
- * Queue item status values.
- */
-export type QueueStatus = 'pending' | 'in_progress' | 'completed' | 'failed' | 'canceled';
-/**
- * Task resolution strings.
- */
-export type TaskResolution = 'marker_detected' | 'auto_completed_timeout' | 'failed_max_retries' | 'runtime_unavailable' | 'canceled' | 'late_marker_principle_created' | 'late_marker_no_principle' | 'stub_fallback' | 'skipped_thin_violation' | 'noise_classified' | 'retired';
-/**
- * Recent pain context for sleep_reflection tasks.
- * Attached to queue items to provide pain signal context.
- */
-export interface RecentPainContext {
-    mostRecent: {
-        score: number;
-        source: string;
-        reason: string;
-        timestamp: string;
-        sessionId: string;
-    } | null;
-    recentPainCount: number;
-    recentMaxPainScore: number;
-}
-/**
- * Default values for new V2 fields when migrating legacy items.
- */
-export declare const DEFAULT_TASK_KIND: TaskKind;
-export declare const DEFAULT_PRIORITY: TaskPriority;
-export declare const DEFAULT_MAX_RETRIES = 3;
-/**
- * Legacy (pre-V2) queue item schema.
- */
-export interface LegacyEvolutionQueueItem {
-    id: string;
-    source: string;
-    traceId?: string;
-    task?: string;
-    score: number;
-    reason: string;
-    timestamp: string;
-    enqueued_at?: string;
-    started_at?: string;
-    completed_at?: string;
-    assigned_session_key?: string;
-    trigger_text_preview?: string;
-    status?: string;
-    resolution?: string;
-    session_id?: string;
-    agent_id?: string;
-    taskKind?: string;
-    priority?: string;
-    retryCount?: number;
-    maxRetries?: number;
-    lastError?: string;
-    resultRef?: string;
-}
-/**
- * V2 queue item schema.
- */
-export interface EvolutionQueueItem {
-    id: string;
-    taskKind: TaskKind;
-    priority: TaskPriority;
-    source: string;
-    traceId?: string;
-    task?: string;
-    score: number;
-    reason: string;
-    timestamp: string;
-    enqueued_at?: string;
-    started_at?: string;
-    completed_at?: string;
-    assigned_session_key?: string;
-    trigger_text_preview?: string;
-    status: QueueStatus;
-    resolution?: TaskResolution;
-    session_id?: string;
-    agent_id?: string;
-    retryCount: number;
-    maxRetries: number;
-    lastError?: string;
-    resultRef?: string;
-    recentPainContext?: RecentPainContext;
-}
-export type RawQueueItem = Record<string, unknown>;
-/**
- * Migrate a legacy queue item to V2 schema.
- * Old items without taskKind are assumed to be pain_diagnosis for backward compatibility.
- */
-export declare function migrateToV2(item: LegacyEvolutionQueueItem): {
-    id: string;
-    taskKind: TaskKind;
-    priority: TaskPriority;
-    source: string;
-    traceId?: string;
-    task?: string;
-    score: number;
-    reason: string;
-    timestamp: string;
-    enqueued_at?: string;
-    started_at?: string;
-    completed_at?: string;
-    assigned_session_key?: string;
-    trigger_text_preview?: string;
-    status: QueueStatus;
-    resolution?: TaskResolution;
-    session_id?: string;
-    agent_id?: string;
-    retryCount: number;
-    maxRetries: number;
-    lastError?: string;
-    resultRef?: string;
-};
-/**
- * Check if an item is a legacy (pre-V2) queue item.
- */
-export declare function isLegacyQueueItem(item: RawQueueItem): boolean;
-/**
- * Migrate entire queue to V2 schema if needed.
- */
-export declare function migrateQueueToV2(queue: RawQueueItem[]): ReturnType<typeof migrateToV2>[];

package/dist/service/evolution-queue-migration.js

@@ -1,53 +0,0 @@
-/**
- * Evolution Queue Migration — V1 to V2 Schema
- *
- * Pure transformation functions and shared queue types.
- */
-/**
- * Default values for new V2 fields when migrating legacy items.
- */
-export const DEFAULT_TASK_KIND = 'pain_diagnosis';
-export const DEFAULT_PRIORITY = 'medium';
-export const DEFAULT_MAX_RETRIES = 3;
-/**
- * Migrate a legacy queue item to V2 schema.
- * Old items without taskKind are assumed to be pain_diagnosis for backward compatibility.
- */
-export function migrateToV2(item) {
-    return {
-        id: item.id,
-        taskKind: item.taskKind || DEFAULT_TASK_KIND,
-        priority: item.priority || DEFAULT_PRIORITY,
-        source: item.source,
-        traceId: item.traceId,
-        task: item.task,
-        score: item.score,
-        reason: item.reason,
-        timestamp: item.timestamp,
-        enqueued_at: item.enqueued_at,
-        started_at: item.started_at,
-        completed_at: item.completed_at,
-        assigned_session_key: item.assigned_session_key,
-        trigger_text_preview: item.trigger_text_preview,
-        status: item.status || 'pending',
-        resolution: item.resolution,
-        session_id: item.session_id,
-        agent_id: item.agent_id,
-        retryCount: item.retryCount || 0,
-        maxRetries: item.maxRetries || DEFAULT_MAX_RETRIES,
-        lastError: item.lastError,
-        resultRef: item.resultRef,
-    };
-}
-/**
- * Check if an item is a legacy (pre-V2) queue item.
- */
-export function isLegacyQueueItem(item) {
-    return item && typeof item === 'object' && !('taskKind' in item);
-}
-/**
- * Migrate entire queue to V2 schema if needed.
- */
-export function migrateQueueToV2(queue) {
-    return queue.map(item => isLegacyQueueItem(item) ? migrateToV2(item) : item);
-}