@principles/pd-cli 1.148.0 → 1.149.1

package/tests/commands/runtime-init-empty-workspace.test.ts

@@ -73,10 +73,6 @@ const EXPECTED_TRAJECTORY_TABLES = [
   'exports_audit', 'evolution_tasks', 'evolution_events',
 ];
 
-const EXPECTED_WORKFLOW_TABLES = [
-  'schema_version', 'subagent_workflows', 'subagent_workflow_events',
-];
-
 const EXPECTED_TRAJECTORY_INDEXES = [
   'idx_assistant_turns_session_id',
   'idx_assistant_turns_created_at',
@@ -110,11 +106,11 @@ describe('pd runtime init — empty workspace integration', () => {
   // ── EMPTY-01: initialize empty workspace ───────────────────────────────────
 
   describe('EMPTY-01: initialize empty workspace', () => {
-    it('returns ok=true with 3 initialized databases', () => {
+    it('returns ok=true with 2 initialized databases', () => {
       const output = buildRuntimeInitOutput(tmpDir, true);
       expect(output.ok).toBe(true);
       expect(output.mode).toBe('confirm');
-      expect(output.databases).toHaveLength(3);
+      expect(output.databases).toHaveLength(2);
       for (const db of output.databases) {
         expect(db.status).toBe('initialized');
       }
@@ -150,14 +146,10 @@ describe('pd runtime init — empty workspace integration', () => {
       }
     });
 
-    it('creates subagent_workflows.db with all expected tables', () => {
+    it('no longer creates subagent_workflows.db (legacy workflow store retired in PRI-737)', () => {
       buildRuntimeInitOutput(tmpDir, true);
       const wfDbPath = path.join(tmpDir, '.state', 'subagent_workflows.db');
-      expect(fs.existsSync(wfDbPath)).toBe(true);
-      const tables = getTableNames(wfDbPath);
-      for (const expected of EXPECTED_WORKFLOW_TABLES) {
-        expect(tables).toContain(expected);
-      }
+      expect(fs.existsSync(wfDbPath)).toBe(false);
     });
 
     it('pain_events table has canonical_pain_id and runtime_task_id columns', () => {
@@ -229,7 +221,7 @@ describe('pd runtime init — empty workspace integration', () => {
       // Second initialization (should be idempotent)
       const output2 = buildRuntimeInitOutput(tmpDir, true);
       expect(output2.ok).toBe(true);
-      expect(output2.databases).toHaveLength(3);
+      expect(output2.databases).toHaveLength(2);
       for (const db of output2.databases) {
         expect(db.status).toBe('initialized');
       }

package/tests/commands/runtime-init.test.ts

@@ -2,8 +2,8 @@
  * runtime-init tests — pd runtime init command (unit tests with mocked DB layer).
  *
  * Covers:
- *   - INIT-01: --dry-run (default) reports 3 DBs as 'skipped', no real writes
- *   - INIT-02: --confirm calls all 3 init functions, output shows 'initialized'
+ *   - INIT-01: --dry-run (default) reports 2 DBs as 'skipped', no real writes
+ *   - INIT-02: --confirm calls all init functions, output shows 'initialized'
  *   - INIT-03: --json outputs single parseable JSON object (cli-1)
  *   - INIT-04: --dry-run + --confirm mutually exclusive (cli-4)
  *   - INIT-05: --confirm failure sets exitCode=1 with reason + nextAction (cli-6)
@@ -24,7 +24,6 @@ import * as fs from 'node:fs';
 const mockState = vi.hoisted(() => {
   return {
     initTrajectorySchema: vi.fn(),
-    initWorkflowSchema: vi.fn(),
     sqliteConnectionCtor: vi.fn(),
     sqliteConnectionGetDb: vi.fn(),
     sqliteConnectionGetWarnings: vi.fn(),
@@ -37,7 +36,6 @@ const mockState = vi.hoisted(() => {
 
 vi.mock('principles-disciple', () => ({
   initTrajectorySchema: mockState.initTrajectorySchema,
-  initWorkflowSchema: mockState.initWorkflowSchema,
 }));
 
 vi.mock('@principles/core/runtime-v2', async () => {
@@ -87,12 +85,6 @@ function setupDefaultConfirmMocks(): void {
     warnings: [],
   });
 
-  // initWorkflowSchema mock
-  mockState.initWorkflowSchema.mockReturnValue({
-    tables: ['schema_version', 'subagent_workflows', 'subagent_workflow_events'],
-    warnings: [],
-  });
-
   // SchemaConformanceReadModel mock
   mockState.schemaConformanceCtor.mockImplementation(function () {
     return {
@@ -113,19 +105,18 @@ describe('pd runtime init', () => {
   // ── INIT-01: dry-run (default) ─────────────────────────────────────────────
 
   describe('INIT-01: dry-run (default)', () => {
-    it('reports 3 DBs as skipped without calling init functions', () => {
+    it('reports 2 DBs as skipped without calling init functions', () => {
       const tmp = mkTmpDir();
       try {
         const output = buildRuntimeInitOutput(tmp, false);
         expect(output.ok).toBe(true);
         expect(output.mode).toBe('dry-run');
-        expect(output.databases).toHaveLength(3);
+        expect(output.databases).toHaveLength(2);
         for (const db of output.databases) {
           expect(db.status).toBe('skipped');
         }
         // No init functions should be called in dry-run mode
         expect(mockState.initTrajectorySchema).not.toHaveBeenCalled();
-        expect(mockState.initWorkflowSchema).not.toHaveBeenCalled();
         expect(mockState.sqliteConnectionCtor).not.toHaveBeenCalled();
       } finally { rmTmpDir(tmp); }
     });
@@ -136,12 +127,10 @@ describe('pd runtime init', () => {
         const output = buildRuntimeInitOutput(tmp, false);
         const stateDb = output.databases.find(d => d.name === 'state.db');
         const trajDb = output.databases.find(d => d.name === 'trajectory.db');
-        const wfDb = output.databases.find(d => d.name === 'subagent_workflows.db');
         expect(stateDb?.tables).toContain('tasks');
         expect(stateDb?.tables).toContain('runs');
         expect(trajDb?.tables).toContain('pain_events');
         expect(trajDb?.tables).toContain('sessions');
-        expect(wfDb?.tables).toContain('subagent_workflows');
       } finally { rmTmpDir(tmp); }
     });
   });
@@ -149,20 +138,19 @@ describe('pd runtime init', () => {
   // ── INIT-02: --confirm ─────────────────────────────────────────────────────
 
   describe('INIT-02: --confirm', () => {
-    it('calls all 3 init functions and reports initialized status', () => {
+    it('calls all init functions and reports initialized status', () => {
       const tmp = mkTmpDir();
       try {
         const output = buildRuntimeInitOutput(tmp, true);
         expect(output.ok).toBe(true);
         expect(output.mode).toBe('confirm');
-        expect(output.databases).toHaveLength(3);
+        expect(output.databases).toHaveLength(2);
         for (const db of output.databases) {
           expect(db.status).toBe('initialized');
         }
         expect(mockState.sqliteConnectionCtor).toHaveBeenCalledTimes(1);
         expect(mockState.sqliteConnectionGetDb).toHaveBeenCalledTimes(1);
         expect(mockState.initTrajectorySchema).toHaveBeenCalledWith(tmp);
-        expect(mockState.initWorkflowSchema).toHaveBeenCalledWith(tmp);
       } finally { rmTmpDir(tmp); }
     });
 
@@ -256,7 +244,6 @@ describe('pd runtime init', () => {
     it('does not call any init functions on flag conflict', async () => {
       await handleRuntimeInit({ dryRun: true, confirm: true, json: true });
       expect(mockState.initTrajectorySchema).not.toHaveBeenCalled();
-      expect(mockState.initWorkflowSchema).not.toHaveBeenCalled();
       expect(mockState.sqliteConnectionCtor).not.toHaveBeenCalled();
     });
   });
@@ -305,19 +292,6 @@ describe('pd runtime init', () => {
       } finally { rmTmpDir(tmp); }
     });
 
-    it('sets exitCode=1 and includes reason when subagent_workflows.db fails', () => {
-      const tmp = mkTmpDir();
-      try {
-        mockState.initWorkflowSchema.mockImplementation(() => {
-          throw new Error('locked');
-        });
-        const output = buildRuntimeInitOutput(tmp, true);
-        expect(output.ok).toBe(false);
-        expect(output.reason).toContain('subagent_workflows.db');
-        expect(output.reason).toContain('locked');
-      } finally { rmTmpDir(tmp); }
-    });
-
     it('handler sets process.exitCode=1 on failure with --json', async () => {
       const tmp = mkTmpDir();
       try {

package/tests/commands/runtime-internalization-run-once.test.ts

@@ -86,6 +86,7 @@ vi.mock('@principles/core/runtime-v2', () => ({
   ArtificerRunner: vi.fn().mockImplementation(function () {
     return { run: mockRun };
   }),
+  buildArtificerHostSemanticContext: (registry) => ({ hostKinds: [], tools: registry.hostMappings() }),
   EvaluatorRunner: vi.fn().mockImplementation(function () {
     return { run: mockRun };
   }),
@@ -197,6 +198,10 @@ const { mockCreateEvaluatorRuntimeContext, mockCreateRolloutGovernanceDeps, mock
   const mockCreateEvaluatorRuntimeContext = vi.fn().mockReturnValue({
     ok: true,
     gateDeps: { evaluateInSandbox: vi.fn() },
+    // PRI-741: the context resolution also carries the single registry
+    // snapshot the parity projection derives from.
+    registry: { resolve: vi.fn(), hostMappings: vi.fn().mockReturnValue([{ rawToolName: 'write', canonicalKind: 'write' }]) },
+    hostKinds: ['openclaw'],
   });
   const mockCreateRolloutGovernanceDeps = vi.fn().mockReturnValue({
     dispatchActivation: vi.fn(),
@@ -204,7 +209,9 @@ const { mockCreateEvaluatorRuntimeContext, mockCreateRolloutGovernanceDeps, mock
   });
   const mockResolveWorkspaceHostToolSemantics = vi.fn().mockReturnValue({
     ok: true,
-    registry: { resolve: vi.fn() },
+    // PRI-741: the projection reads the host layer via hostMappings() — the
+    // mock registry carries it like a real buildToolSemanticRegistry product.
+    registry: { resolve: vi.fn(), hostMappings: vi.fn().mockReturnValue([{ rawToolName: 'write', canonicalKind: 'write' }]) },
     hostKinds: ['openclaw'],
   });
   return { mockCreateEvaluatorRuntimeContext, mockCreateRolloutGovernanceDeps, mockResolveWorkspaceHostToolSemantics };