@principles/pd-cli 1.93.0 → 1.94.0

package/tests/commands/diagnose.test.ts

@@ -57,11 +57,17 @@ vi.mock('@principles/core/runtime-v2', () => {
     SqliteSourceTraceLocator: vi.fn().mockImplementation(function () { return {}; }),
     StoreEventEmitter: vi.fn().mockImplementation(function () { return {}; }),
     storeEmitter: { emitTelemetry: vi.fn() },
-    DiagnosticianRunner: vi.fn().mockImplementation(function () { return {}; }),
-    PassThroughValidator: vi.fn().mockImplementation(function () { return {}; }),
-    DefaultDiagnosticianValidator: vi.fn().mockImplementation(function () { return {}; }),
+    SplitDiagnosticianRunner: vi.fn().mockImplementation(function () { return {}; }),
+    DiagRootCauseRunner: vi.fn().mockImplementation(function () { return {}; }),
+    DiagDistillerRunner: vi.fn().mockImplementation(function () { return {}; }),
+    DiagRouterRunner: vi.fn().mockImplementation(function () { return {}; }),
+    DefaultDiagRootCauseValidator: vi.fn().mockImplementation(function () { return {}; }),
+    DefaultDiagDistillerValidator: vi.fn().mockImplementation(function () { return {}; }),
+    DisabledDiagnosticianRunner: vi.fn().mockImplementation(function () { return {}; }),
     TestDoubleRuntimeAdapter: vi.fn().mockImplementation(function () { return {}; }),
     OpenClawCliRuntimeAdapter: vi.fn().mockImplementation(function () { return {}; }),
+    PiAiRuntimeAdapter: vi.fn().mockImplementation(function () { return {}; }),
+    SPLIT_PIPELINE_TOTAL_TIMEOUT_MS: 300000,
     PDRuntimeError: class PDRuntimeError extends Error {
       constructor(public category: string, message: string) {
         super(message);
@@ -78,8 +84,12 @@ vi.mock('@principles/core/runtime-v2', () => {
       agentId: 'main',
     }),
     isRuntimeConfigError: vi.fn().mockReturnValue(false),
-    isFeatureEnabled: vi.fn().mockReturnValue(false),
+    isFeatureEnabled: vi.fn().mockReturnValue(true),
     resolveOutputLanguage: vi.fn().mockReturnValue({ outputLanguage: 'zh-CN' }),
+    validatePdConfig: vi.fn().mockReturnValue({ valid: true, errors: [] }),
+    computeEffectivePdConfig: vi.fn().mockReturnValue({ config: {}, source: 'defaults', warnings: [] }),
+    computeFeatureFlagsFromConfig: vi.fn().mockReturnValue({}),
+    redactPdConfig: vi.fn().mockImplementation((c) => c),
     run: vi.fn().mockResolvedValue({
       status: 'succeeded',
       taskId: 'test-task-1',

package/tests/commands/pain-retry.test.ts

@@ -89,11 +89,17 @@ vi.mock('@principles/core/runtime-v2', () => {
     SqliteSourceTraceLocator: vi.fn().mockImplementation(function () { return {}; }),
     StoreEventEmitter: vi.fn().mockImplementation(function () { return {}; }),
     storeEmitter: { emitTelemetry: vi.fn() },
-    DiagnosticianRunner: vi.fn().mockImplementation(function () { return {}; }),
-    DefaultDiagnosticianValidator: vi.fn().mockImplementation(function () { return {}; }),
+    SplitDiagnosticianRunner: vi.fn().mockImplementation(function () { return {}; }),
+    DiagRootCauseRunner: vi.fn().mockImplementation(function () { return {}; }),
+    DiagDistillerRunner: vi.fn().mockImplementation(function () { return {}; }),
+    DiagRouterRunner: vi.fn().mockImplementation(function () { return {}; }),
+    DefaultDiagRootCauseValidator: vi.fn().mockImplementation(function () { return {}; }),
+    DefaultDiagDistillerValidator: vi.fn().mockImplementation(function () { return {}; }),
+    DisabledDiagnosticianRunner: vi.fn().mockImplementation(function () { return {}; }),
     TestDoubleRuntimeAdapter: vi.fn().mockImplementation(function () { return {}; }),
     OpenClawCliRuntimeAdapter: vi.fn().mockImplementation(function () { return {}; }),
     PiAiRuntimeAdapter: vi.fn().mockImplementation(function () { return {}; }),
+    SPLIT_PIPELINE_TOTAL_TIMEOUT_MS: 300000,
     PDRuntimeError: class PDRuntimeError extends Error {
       constructor(public category: string, message: string) {
         super(message);
@@ -103,7 +109,12 @@ vi.mock('@principles/core/runtime-v2', () => {
     CandidateIntakeService: MockCandidateIntakeService,
     resolveRuntimeConfig: mockResolveRuntimeConfig,
     isRuntimeConfigError: vi.fn().mockReturnValue(false),
+    isFeatureEnabled: vi.fn().mockReturnValue(true),
     resolveOutputLanguage: vi.fn().mockReturnValue({ outputLanguage: 'zh-CN' }),
+    validatePdConfig: vi.fn().mockReturnValue({ valid: true, errors: [] }),
+    computeEffectivePdConfig: vi.fn().mockReturnValue({ config: {}, source: 'defaults', warnings: [] }),
+    computeFeatureFlagsFromConfig: vi.fn().mockReturnValue({}),
+    redactPdConfig: vi.fn().mockImplementation((c) => c),
     run: mockRun,
     status: vi.fn(),
   };

package/tests/e2e/candidate-intake-e2e.test.ts

@@ -149,12 +149,33 @@ describe('E2E: pd candidate intake flow', () => {
     }
   }
 
-  function readLedgerFile(workspace: string): any[] {
-    const ledgerPath = join(workspace, '.pd', 'principle-tree-ledger.json');
+  function readLedgerFile(workspace: string): Array<{ id: string; sourceRef: string; status: string; evaluability: string }> {
+    const ledgerPath = join(workspace, '.state', 'principle_training_state.json');
     if (!existsSync(ledgerPath)) return [];
     const content = readFileSync(ledgerPath, 'utf-8');
-    const data = JSON.parse(content);
-    return data.principles || [];
+    const data: unknown = JSON.parse(content);
+    if (typeof data !== 'object' || data === null) return [];
+    const tree = (data as Record<string, unknown>)._tree ?? (data as Record<string, unknown>).tree ?? {};
+    if (typeof tree !== 'object' || tree === null) return [];
+    const principles = (tree as Record<string, unknown>).principles ?? {};
+    if (typeof principles !== 'object' || principles === null) return [];
+    return Object.values(principles).map((p: unknown) => {
+      if (typeof p !== 'object' || p === null) {
+        return { id: '', sourceRef: 'candidate://', status: '', evaluability: '' };
+      }
+      const pObj = p as Record<string, unknown>;
+      const painIds = pObj.derivedFromPainIds;
+      const candidateId = Array.isArray(painIds) && typeof painIds[0] === 'string' ? painIds[0] : '';
+      const id = typeof pObj.id === 'string' ? pObj.id : '';
+      const rawStatus = typeof pObj.status === 'string' ? pObj.status : '';
+      const evaluability = typeof pObj.evaluability === 'string' ? pObj.evaluability : '';
+      return {
+        id,
+        sourceRef: `candidate://${candidateId}`,
+        status: rawStatus === 'candidate' ? 'probation' : rawStatus,
+        evaluability,
+      };
+    });
   }
 
   it('Test 1 (Happy path E2E): pending candidate → intake → consumed → ledgerEntryId in output', () => {