@principles/pd-cli 1.149.0 → 1.149.2

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 };

package/tests/commands/samples-list.test.ts

@@ -0,0 +1,43 @@
+/**
+ * pd samples list command tests — PRI-753 (Trajectory Reader Reality Fix).
+ *
+ * A missing workspace trajectory database is an operator-visible failure with
+ * a next action (cli-6), never a silent "no samples" answer (rc-9).
+ */
+import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
+
+vi.mock('../../src/resolve-workspace.js', () => ({
+  resolveWorkspaceDir: vi.fn().mockReturnValue('/tmp/pri753-missing-workspace'),
+}));
+
+import { handleSamplesList } from '../../src/commands/samples-list.js';
+
+describe('handleSamplesList — unavailable database (PRI-753)', () => {
+  let errorSpy: ReturnType<typeof vi.spyOn>;
+  let exitSpy: ReturnType<typeof vi.spyOn>;
+
+  beforeEach(() => {
+    errorSpy = vi.spyOn(console, 'error').mockImplementation(() => {});
+    exitSpy = vi.spyOn(process, 'exit').mockImplementation(((code: number) => {
+      throw new Error(`process.exit:${String(code)}`);
+    }) as never);
+  });
+
+  afterEach(() => {
+    vi.restoreAllMocks();
+  });
+
+  it('fails loud with reason and next action when trajectory.db is missing', async () => {
+    await expect(handleSamplesList({})).rejects.toThrow('process.exit:1');
+
+    const stderrText = errorSpy.mock.calls.map((call) => String(call[0])).join('');
+    expect(stderrText).toContain('Trajectory database unavailable');
+    expect(stderrText).toContain(joinPath('.state', 'trajectory.db'));
+    expect(stderrText).toContain('pd runtime init --confirm');
+    expect(exitSpy).toHaveBeenCalledWith(1);
+  });
+});
+
+function joinPath(a: string, b: string): string {
+  return `${a}${process.platform === 'win32' ? '\\' : '/'}${b}`;
+}

package/tests/commands/samples-review.test.ts

@@ -0,0 +1,41 @@
+/**
+ * pd samples review command tests — PRI-753 (Trajectory Reader Reality Fix).
+ *
+ * A missing workspace trajectory database is an operator-visible failure with
+ * a next action (cli-6); failure exits before any mutation (cli-2/cli-5).
+ */
+import { describe, it, expect, vi, beforeEach, afterEach } from 'vitest';
+
+vi.mock('../../src/resolve-workspace.js', () => ({
+  resolveWorkspaceDir: vi.fn().mockReturnValue('/tmp/pri753-missing-workspace'),
+}));
+
+import { handleSamplesReview } from '../../src/commands/samples-review.js';
+
+describe('handleSamplesReview — unavailable database (PRI-753)', () => {
+  let errorSpy: ReturnType<typeof vi.spyOn>;
+  let exitSpy: ReturnType<typeof vi.spyOn>;
+
+  beforeEach(() => {
+    errorSpy = vi.spyOn(console, 'error').mockImplementation(() => {});
+    exitSpy = vi.spyOn(process, 'exit').mockImplementation(((code: number) => {
+      throw new Error(`process.exit:${String(code)}`);
+    }) as never);
+  });
+
+  afterEach(() => {
+    vi.restoreAllMocks();
+  });
+
+  it('fails loud with reason and next action when trajectory.db is missing', async () => {
+    await expect(handleSamplesReview({ sampleId: 'sample-001', decision: 'approve' })).rejects.toThrow(
+      'process.exit:1'
+    );
+
+    const stderrText = errorSpy.mock.calls.map((call) => String(call[0])).join('');
+    expect(stderrText).toContain('Trajectory database unavailable');
+    expect(stderrText).toContain('trajectory.db');
+    expect(stderrText).toContain('pd runtime init --confirm');
+    expect(exitSpy).toHaveBeenCalledWith(1);
+  });
+});