@principles/pd-cli 1.92.0 → 1.94.0

package/tests/commands/cli-command-tree.test.ts

@@ -5,10 +5,11 @@
  */
 import { describe, it, expect } from 'vitest';
 import { execFileSync } from 'node:child_process';
+import { getBuiltPdCliPath } from '../helpers/pd-cli-path.js';
 
 function runPdHelp(args: string[]): string {
   try {
-    return execFileSync('node', ['packages/pd-cli/dist/index.js', ...args], {
+    return execFileSync('node', [getBuiltPdCliPath(), ...args], {
       encoding: 'utf8',
       cwd: process.cwd(),
     });

package/tests/commands/console-open.test.ts

@@ -21,6 +21,7 @@ import * as path from 'node:path';
 import { execFileSync } from 'node:child_process';
 import * as childProcessModule from 'node:child_process';
 import { EventEmitter } from 'node:events';
+import { getBuiltPdCliPath } from '../helpers/pd-cli-path.js';
 import {
   isLoopbackHost,
   normalizeLoopbackHost,
@@ -757,9 +758,18 @@ describe('CLI command wiring (pd console open)', () => {
 
 function runPd(args: string[], cwd: string): string {
   try {
-    return execFileSync('node', ['packages/pd-cli/dist/index.js', ...args], {
+    const env: Record<string, string> = { ...process.env };
+    if (!args.includes('--workspace') && !args.includes('--help') && !args.includes('-h')) {
+      env.USERPROFILE = '/nonexistent';
+      env.HOME = '/nonexistent';
+      env.HOMEPATH = '/nonexistent';
+      env.HOMEDRIVE = '/nonexistent';
+      delete env.PD_WORKSPACE_DIR;
+    }
+    return execFileSync('node', [getBuiltPdCliPath(), ...args], {
       encoding: 'utf8',
       cwd,
+      env,
     });
   } catch (err: unknown) {
     if (err && typeof err === 'object' && Object.hasOwn(err, 'stdout')) {

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,7 +84,12 @@ vi.mock('@principles/core/runtime-v2', () => {
       agentId: 'main',
     }),
     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: vi.fn().mockResolvedValue({
       status: 'succeeded',
       taskId: 'test-task-1',
@@ -106,6 +117,11 @@ vi.mock('../../src/config-reader.js', () => ({
   readOutputLanguageFromWorkspace: vi.fn().mockReturnValue({ outputLanguage: 'zh-CN' }),
 }));
 
+vi.mock('../../src/services/pd-config-loader.js', () => ({
+  loadPdConfig: vi.fn().mockReturnValue({ ok: true, effective: { config: {}, source: 'defaults', warnings: [] } }),
+  computeFlagsFromLoadResult: vi.fn().mockReturnValue({}),
+}));
+
 import { handleDiagnoseRun, type DiagnoseRunOptions } from '../../src/commands/diagnose.js';
 
 const SUCCEEDED_RESULT = {

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/commands/runtime-internalization.test.ts

@@ -6,12 +6,13 @@
  */
 import { describe, it, expect } from 'vitest';
 import { execFileSync } from 'node:child_process';
+import { getBuiltPdCliPath } from '../helpers/pd-cli-path.js';
 
 function runPdHelp(args: string[]): string {
   try {
-    return execFileSync('node', ['packages/pd-cli/dist/index.js', ...args], {
+    return execFileSync('node', [getBuiltPdCliPath(), ...args], {
       encoding: 'utf8',
-      cwd: 'D:/Code/principles',
+      cwd: process.cwd(),
     });
   } catch (err: unknown) {
     if (err && typeof err === 'object' && 'stdout' in err) {

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

@@ -4,11 +4,12 @@ import { mkdtempSync, rmSync, existsSync, readFileSync, mkdirSync } from 'fs';
 import { join } from 'path';
 import { tmpdir } from 'os';
 import Database from 'better-sqlite3';
+import { getBuiltPdCliPath } from '../helpers/pd-cli-path.js';
 
 describe('E2E: pd candidate intake flow', () => {
   let tempWorkspace: string;
   let db: Database.Database;
-  const pdCliPath = join(process.cwd(), 'dist/index.js');
+  const pdCliPath = getBuiltPdCliPath();
 
   beforeEach(() => {
     tempWorkspace = mkdtempSync(join(tmpdir(), 'pd-e2e-'));
@@ -148,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', () => {

package/tests/helpers/pd-cli-path.ts

@@ -0,0 +1,5 @@
+import { join } from 'node:path';
+
+export function getBuiltPdCliPath(): string {
+  return join(process.cwd(), 'dist', 'index.js');
+}

package/tests/pd-cli-smoke.test.ts

@@ -1,9 +1,10 @@
 import { describe, it, expect } from 'vitest';
 import { execFileSync } from 'node:child_process';
+import { getBuiltPdCliPath } from './helpers/pd-cli-path.js';
 
 describe('PD CLI Smoke Test', () => {
   it('should run successfully and load all dependencies including better-sqlite3 without errors', () => {
-    const output = execFileSync('node', ['packages/pd-cli/dist/index.js', '--help'], {
+    const output = execFileSync('node', [getBuiltPdCliPath(), '--help'], {
       encoding: 'utf8',
       cwd: process.cwd(),
     });

package/vitest.config.ts

@@ -0,0 +1,9 @@
+import { defineConfig } from 'vitest/config';
+
+export default defineConfig({
+  test: {
+    environment: 'node',
+    include: ['tests/**/*.test.ts', 'src/**/*.test.ts'],
+    exclude: ['**/node_modules/**', '**/dist/**'],
+  },
+});