@principles/core 1.177.4 → 1.177.6

package/dist/runtime-v2/__tests__/architecture-regression.test.js

@@ -1804,6 +1804,19 @@ describe('PRI-111 ArtificerRunner boundary', () => {
         expect(src).toContain('ArtificerRuleOutputSchema');
     });
 });
+// ── PRI-458: PhilosopherRunner barrel boundary (MVP-Quiet) ─────────────────────
+describe('PRI-458 PhilosopherRunner barrel boundary', () => {
+    it('BARREL_EXPORTS: internalization/index.ts does NOT export PhilosopherRunner (PRI-458 MVP-Quiet)', async () => {
+        // PRI-458: PhilosopherRunner is de-surfaced from the internal barrel (MVP-Quiet).
+        // It remains in runtime-v2/index.ts (public barrel) because rulehost-pipeline-runner.ts
+        // imports it from there (EP-02 exception).
+        const { readFileSync } = await import('node:fs');
+        const { resolve } = await import('node:path');
+        const src = readFileSync(resolve(__dirname, '..', 'internalization', 'index.ts'), 'utf-8');
+        expect(src).not.toContain("from './philosopher-runner.js'");
+        expect(src).not.toContain("from './philosopher-output.js'");
+    });
+});
 // ── PRI-EVAL: EvaluatorRunner boundary guards ──────────────────────────────────
 describe('PRI-EVAL EvaluatorRunner boundary', () => {
     it('evaluator source files exist in internalization directory', async () => {
@@ -1844,13 +1857,17 @@ describe('PRI-EVAL EvaluatorRunner boundary', () => {
         expect(src).not.toContain('node:fs');
         expect(src).not.toContain('node:cron');
     });
-    it('BARREL_EXPORTS: internalization/index.ts exports EvaluatorRunner and EvaluatorOutput', async () => {
+    it('BARREL_EXPORTS: internalization/index.ts does NOT export EvaluatorRunner (PRI-458 MVP-Quiet)', async () => {
+        // PRI-458: EvaluatorRunner is de-surfaced from the internal barrel (MVP-Quiet).
+        // It remains in runtime-v2/index.ts (public barrel) because rulehost-pipeline-runner.ts
+        // imports it from there (EP-02 exception). These negative assertions lock the boundary:
+        // if someone re-aggregates the evaluator exports into this barrel, the test will fail.
         const { readFileSync } = await import('node:fs');
         const { resolve } = await import('node:path');
         const src = readFileSync(resolve(__dirname, '..', 'internalization', 'index.ts'), 'utf-8');
-        expect(src).toContain('EvaluatorRunner');
-        expect(src).toContain('EvaluatorOutput');
-        expect(src).toContain('DefaultEvaluatorValidator');
+        expect(src).not.toContain("from './evaluator-runner.js'");
+        expect(src).not.toContain("from './evaluator-output.js'");
+        expect(src).not.toContain("from './evaluator-prompt-builder.js'");
     });
     it('SCHEMA_REGISTRY: pi-ai-runtime-adapter.ts registers evaluator-output-v1 schema', async () => {
         const { readFileSync } = await import('node:fs');
@@ -1900,13 +1917,16 @@ describe('PRI-RR RolloutReviewerRunner boundary', () => {
         expect(src).not.toContain('node:fs');
         expect(src).not.toContain('node:cron');
     });
-    it('BARREL_EXPORTS: internalization/index.ts exports RolloutReviewerRunner and RolloutReviewerOutput', async () => {
+    it('BARREL_EXPORTS: internalization/index.ts does NOT export RolloutReviewerRunner (PRI-458 MVP-Quiet)', async () => {
+        // PRI-458: RolloutReviewerRunner is de-surfaced from the internal barrel (MVP-Quiet).
+        // It remains in runtime-v2/index.ts (public barrel) because runtime-internalization-run-once.ts
+        // imports it from there and package.json has no deep import path available (EP-02 exception).
         const { readFileSync } = await import('node:fs');
         const { resolve } = await import('node:path');
         const src = readFileSync(resolve(__dirname, '..', 'internalization', 'index.ts'), 'utf-8');
-        expect(src).toContain('RolloutReviewerRunner');
-        expect(src).toContain('RolloutReviewerOutput');
-        expect(src).toContain('DefaultRolloutReviewerValidator');
+        expect(src).not.toContain("from './rollout-reviewer-runner.js'");
+        expect(src).not.toContain("from './rollout-reviewer-output.js'");
+        expect(src).not.toContain("from './rollout-reviewer-prompt-builder.js'");
     });
     it('SCHEMA_REGISTRY: pi-ai-runtime-adapter.ts registers rollout-reviewer-output-v1 schema', async () => {
         const { readFileSync } = await import('node:fs');
@@ -2176,12 +2196,14 @@ describe('PRI-117 Nocturnal god-class freeze', () => {
             expect(src).not.toContain('invokeStubScribe');
         }
     });
-    it('RUNTIME_V2_USES_PEER_RUNNERS: DreamerRunner/PhilosopherRunner/ScribeRunner are the canonical entry points', async () => {
+    it('RUNTIME_V2_USES_PEER_RUNNERS: DreamerRunner/ScribeRunner/ArtificerRunner are the canonical MVP-Core entry points', async () => {
+        // PRI-458: narrowed to MVP-Core runners only. PhilosopherRunner (MVP-Quiet)
+        // remains in runtime-v2/index.ts for rulehost-pipeline-runner.ts compatibility
+        // but is no longer asserted as a canonical entry point.
         const { readFileSync } = await import('node:fs');
         const { resolve } = await import('node:path');
         const src = readFileSync(resolve(__dirname, '..', 'index.ts'), 'utf-8');
         expect(src).toContain('DreamerRunner');
-        expect(src).toContain('PhilosopherRunner');
         expect(src).toContain('ScribeRunner');
         expect(src).toContain('ArtificerRunner');
     });