@diff-review-system/drs 2.2.1 → 3.0.0

package/dist/runtime/client.test.js

@@ -0,0 +1,392 @@
+import { beforeEach, describe, expect, it, vi } from 'vitest';
+import { RuntimeClient, createRuntimeClient, createRuntimeClientInstance } from './client.js';
+function createRuntime(sessionOverrides = {}) {
+    return {
+        server: {
+            url: 'pi://in-process',
+            close: vi.fn(),
+        },
+        client: {
+            session: {
+                create: vi.fn(async () => ({ data: { id: 'session-123' } })),
+                prompt: vi.fn(async () => { }),
+                messages: vi.fn(async () => ({ data: [] })),
+                delete: vi.fn(async () => { }),
+                ...sessionOverrides,
+            },
+        },
+    };
+}
+const mocks = vi.hoisted(() => ({
+    createPiInProcessServer: vi.fn(),
+    loadReviewAgents: vi.fn(() => []),
+}));
+vi.mock('../pi/sdk.js', () => ({
+    createPiInProcessServer: mocks.createPiInProcessServer,
+}));
+vi.mock('./agent-loader.js', () => ({
+    loadReviewAgents: mocks.loadReviewAgents,
+}));
+describe('RuntimeClient', () => {
+    beforeEach(() => {
+        vi.clearAllMocks();
+        vi.spyOn(console, 'log').mockImplementation(() => { });
+        vi.spyOn(console, 'warn').mockImplementation(() => { });
+        mocks.createPiInProcessServer.mockResolvedValue(createRuntime());
+        mocks.loadReviewAgents.mockReturnValue([]);
+    });
+    describe('constructor', () => {
+        it('creates an instance with minimal config', () => {
+            const client = new RuntimeClient({
+                directory: '/test/dir',
+            });
+            expect(client).toBeInstanceOf(RuntimeClient);
+        });
+        it('supports optional model overrides and provider config', () => {
+            const client = new RuntimeClient({
+                modelOverrides: {
+                    'review/security': 'anthropic/claude-opus-4-5-20251101',
+                },
+                provider: {
+                    custom: {
+                        npm: '@custom/provider',
+                        name: 'custom',
+                        models: { model: { name: 'model' } },
+                        options: {
+                            baseURL: 'https://api.custom.example',
+                            apiKey: '{env:CUSTOM_API_KEY}',
+                        },
+                    },
+                },
+            });
+            expect(client).toBeInstanceOf(RuntimeClient);
+        });
+    });
+    describe('initialize', () => {
+        it('wires Pi runtime agent prompts and model overrides', async () => {
+            mocks.loadReviewAgents.mockReturnValue([
+                {
+                    name: 'review/security',
+                    path: '/tmp/security.md',
+                    description: 'Security specialist',
+                    prompt: 'Security prompt',
+                    tools: { Read: true },
+                },
+                {
+                    name: 'review/quality',
+                    path: '/tmp/quality.md',
+                    description: 'Quality specialist',
+                    prompt: 'Quality prompt',
+                },
+            ]);
+            const client = await createRuntimeClientInstance({
+                directory: process.cwd(),
+                config: {
+                    review: {
+                        agents: ['security', 'quality'],
+                        default: {
+                            skills: [],
+                        },
+                    },
+                },
+                modelOverrides: {
+                    'review/security': 'anthropic/claude-security',
+                },
+            });
+            expect(mocks.createPiInProcessServer).toHaveBeenCalledWith(expect.objectContaining({
+                config: expect.objectContaining({
+                    agent: expect.objectContaining({
+                        'review/security': expect.objectContaining({
+                            prompt: 'Security prompt',
+                            model: 'anthropic/claude-security',
+                            tools: { Read: true },
+                        }),
+                        'review/quality': expect.objectContaining({
+                            prompt: 'Quality prompt',
+                        }),
+                    }),
+                }),
+            }));
+            await client.shutdown();
+        });
+        it('passes skill search paths and agent skill configuration to Pi runtime', async () => {
+            const projectRoot = process.cwd();
+            const config = {
+                review: {
+                    agents: [
+                        {
+                            name: 'security',
+                            skills: ['security-audit'],
+                        },
+                    ],
+                    default: {
+                        skills: ['baseline-review'],
+                    },
+                },
+            };
+            const client = await createRuntimeClientInstance({
+                directory: projectRoot,
+                config,
+            });
+            expect(mocks.createPiInProcessServer).toHaveBeenCalledWith(expect.objectContaining({
+                config: expect.objectContaining({
+                    skillSearchPaths: expect.any(Array),
+                    agentSkills: {
+                        'review/security': ['baseline-review', 'security-audit'],
+                    },
+                }),
+            }));
+            await client.shutdown();
+        });
+        it('passes per-agent tool overrides to Pi runtime config', async () => {
+            mocks.loadReviewAgents.mockReturnValue([
+                {
+                    name: 'review/security',
+                    path: '/tmp/security.md',
+                    description: 'Security agent',
+                    prompt: 'Security prompt',
+                    tools: { Read: true, Bash: false, Edit: true },
+                },
+                {
+                    name: 'review/quality',
+                    path: '/tmp/quality.md',
+                    description: 'Quality agent',
+                    prompt: 'Quality prompt',
+                    // No tools override — uses global defaults
+                },
+            ]);
+            const client = await createRuntimeClientInstance({
+                directory: process.cwd(),
+                config: {
+                    review: {
+                        agents: ['security', 'quality'],
+                        default: { skills: [] },
+                    },
+                },
+            });
+            expect(mocks.createPiInProcessServer).toHaveBeenCalledWith(expect.objectContaining({
+                config: expect.objectContaining({
+                    agent: expect.objectContaining({
+                        'review/security': expect.objectContaining({
+                            tools: { Read: true, Bash: false, Edit: true },
+                        }),
+                        'review/quality': expect.not.objectContaining({
+                            tools: expect.anything(),
+                        }),
+                    }),
+                }),
+            }));
+            await client.shutdown();
+        });
+    });
+    describe('createSession', () => {
+        it('throws error if not initialized', async () => {
+            const client = new RuntimeClient({});
+            await expect(client.createSession({
+                agent: 'review/security',
+                message: 'Review this code',
+            })).rejects.toThrow('Runtime client not initialized');
+        });
+        it('maps authentication errors to actionable messages', async () => {
+            mocks.createPiInProcessServer.mockResolvedValueOnce(createRuntime({
+                create: vi.fn(async () => {
+                    throw new Error('401 Unauthorized');
+                }),
+            }));
+            const client = await createRuntimeClientInstance({
+                directory: process.cwd(),
+            });
+            await expect(client.createSession({
+                agent: 'review/security',
+                message: 'Review this code',
+            })).rejects.toThrow('Authentication failed with the configured model provider');
+        });
+        it('maps model resolution errors to actionable messages', async () => {
+            mocks.createPiInProcessServer.mockResolvedValueOnce(createRuntime({
+                create: vi.fn(async () => {
+                    throw new Error('Failed to resolve model "anthropic/does-not-exist"');
+                }),
+            }));
+            const client = await createRuntimeClientInstance({
+                directory: process.cwd(),
+            });
+            await expect(client.createSession({
+                agent: 'review/security',
+                message: 'Review this code',
+            })).rejects.toThrow('Model configuration is invalid or unavailable');
+        });
+        it('includes local-runtime hint when connectivity errors occur', async () => {
+            mocks.createPiInProcessServer.mockResolvedValueOnce(createRuntime({
+                create: vi.fn(async () => {
+                    throw new Error('fetch failed');
+                }),
+            }));
+            const client = await createRuntimeClientInstance({
+                directory: process.cwd(),
+            });
+            await expect(client.createSession({
+                agent: 'review/security',
+                message: 'Review this code',
+            })).rejects.toThrow('Verify local Pi runtime setup and model provider connectivity');
+        });
+    });
+    describe('lifecycle and helper methods', () => {
+        it('streamMessages throws if not initialized', async () => {
+            const client = new RuntimeClient({});
+            const generator = client.streamMessages('session-123');
+            await expect(generator.next()).rejects.toThrow('Runtime client not initialized');
+        });
+        it('applies configured model pricing when runtime cost is missing or zero', async () => {
+            const runtimeMessages = [
+                {
+                    info: {
+                        id: 'msg-1',
+                        role: 'assistant',
+                        time: { completed: Date.now() },
+                        provider: 'opencode',
+                        model: 'glm-5-free',
+                        usage: {
+                            input: 1000,
+                            output: 100,
+                            cacheRead: 0,
+                            cacheWrite: 0,
+                            totalTokens: 1100,
+                        },
+                    },
+                    parts: [{ text: 'done' }],
+                },
+            ];
+            mocks.createPiInProcessServer.mockResolvedValueOnce(createRuntime({
+                messages: vi.fn(async () => ({ data: runtimeMessages })),
+            }));
+            const client = await createRuntimeClientInstance({
+                directory: process.cwd(),
+                config: {
+                    review: {
+                        agents: [],
+                        default: {
+                            skills: [],
+                        },
+                    },
+                    pricing: {
+                        models: {
+                            'opencode/glm-5-free': {
+                                input: 2,
+                                output: 8,
+                            },
+                        },
+                    },
+                },
+            });
+            const collected = [];
+            for await (const message of client.streamMessages('session-123')) {
+                collected.push(message);
+            }
+            expect(collected).toHaveLength(1);
+            expect(collected[0].usage?.cost).toBeCloseTo(0.0028, 10);
+            await client.shutdown();
+        });
+        it('closeSession throws if not initialized', async () => {
+            const client = new RuntimeClient({});
+            await expect(client.closeSession('session-123')).rejects.toThrow('Runtime client not initialized');
+        });
+        it('getServerUrl throws when server is not initialized', () => {
+            const client = new RuntimeClient({});
+            expect(() => client.getServerUrl()).toThrow('Server not initialized');
+        });
+        it('shutdown does not throw when no runtime is active', async () => {
+            const client = new RuntimeClient({});
+            await expect(client.shutdown()).resolves.toBeUndefined();
+        });
+    });
+    describe('environment variable resolution', () => {
+        it('resolves provider apiKey env references before runtime init', async () => {
+            const originalEnv = process.env.TEST_API_KEY;
+            process.env.TEST_API_KEY = 'test-key-123';
+            try {
+                const client = await createRuntimeClientInstance({
+                    directory: process.cwd(),
+                    provider: {
+                        'test-provider': {
+                            npm: '@test/provider',
+                            name: 'test-provider',
+                            models: { 'test-model': { name: 'test-model' } },
+                            options: {
+                                baseURL: 'https://api.test.com',
+                                apiKey: '{env:TEST_API_KEY}',
+                            },
+                        },
+                    },
+                });
+                expect(mocks.createPiInProcessServer).toHaveBeenCalledWith(expect.objectContaining({
+                    config: expect.objectContaining({
+                        provider: expect.objectContaining({
+                            'test-provider': expect.objectContaining({
+                                options: expect.objectContaining({
+                                    apiKey: 'test-key-123',
+                                }),
+                            }),
+                        }),
+                    }),
+                }));
+                await client.shutdown();
+            }
+            finally {
+                if (originalEnv === undefined) {
+                    delete process.env.TEST_API_KEY;
+                }
+                else {
+                    process.env.TEST_API_KEY = originalEnv;
+                }
+            }
+        });
+        it('warns when referenced environment variables are missing', async () => {
+            const logSpy = vi.spyOn(console, 'log').mockImplementation(() => { });
+            const originalEnv = process.env.NONEXISTENT_VAR;
+            delete process.env.NONEXISTENT_VAR;
+            try {
+                const client = await createRuntimeClientInstance({
+                    directory: process.cwd(),
+                    provider: {
+                        'test-provider': {
+                            npm: '@test/provider',
+                            name: 'test-provider',
+                            models: { 'test-model': { name: 'test-model' } },
+                            options: {
+                                baseURL: 'https://api.test.com',
+                                apiKey: '{env:NONEXISTENT_VAR}',
+                            },
+                        },
+                    },
+                });
+                // Logger outputs warning via console.log (human format)
+                const allOutput = logSpy.mock.calls.map((c) => String(c[0])).join('\n');
+                expect(allOutput).toContain('NONEXISTENT_VAR is not set');
+                await client.shutdown();
+            }
+            finally {
+                logSpy.mockRestore();
+                if (originalEnv !== undefined) {
+                    process.env.NONEXISTENT_VAR = originalEnv;
+                }
+            }
+        });
+    });
+    describe('factory functions', () => {
+        it('createRuntimeClient returns an uninitialized client instance', () => {
+            const client = createRuntimeClient({
+                directory: process.cwd(),
+            });
+            expect(client).toBeInstanceOf(RuntimeClient);
+        });
+        it('createRuntimeClientInstance initializes in-process runtime', async () => {
+            const client = await createRuntimeClientInstance({
+                directory: process.cwd(),
+            });
+            expect(client).toBeInstanceOf(RuntimeClient);
+            expect(mocks.createPiInProcessServer).toHaveBeenCalled();
+            await client.shutdown();
+        });
+    });
+});
+//# sourceMappingURL=client.test.js.map

package/dist/runtime/client.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"client.test.js","sourceRoot":"","sources":["../../src/runtime/client.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAC9D,OAAO,EAAE,aAAa,EAAE,mBAAmB,EAAE,2BAA2B,EAAE,MAAM,aAAa,CAAC;AAE9F,SAAS,aAAa,CACpB,mBAKK,EAAE;IAEP,OAAO;QACL,MAAM,EAAE;YACN,GAAG,EAAE,iBAAiB;YACtB,KAAK,EAAE,EAAE,CAAC,EAAE,EAAE;SACf;QACD,MAAM,EAAE;YACN,OAAO,EAAE;gBACP,MAAM,EAAE,EAAE,CAAC,EAAE,CAAC,KAAK,IAAI,EAAE,CAAC,CAAC,EAAE,IAAI,EAAE,EAAE,EAAE,EAAE,aAAa,EAAE,EAAE,CAAC,CAAC;gBAC5D,MAAM,EAAE,EAAE,CAAC,EAAE,CAAC,KAAK,IAAI,EAAE,GAAE,CAAC,CAAC;gBAC7B,QAAQ,EAAE,EAAE,CAAC,EAAE,CAAC,KAAK,IAAI,EAAE,CAAC,CAAC,EAAE,IAAI,EAAE,EAAE,EAAE,CAAC,CAAC;gBAC3C,MAAM,EAAE,EAAE,CAAC,EAAE,CAAC,KAAK,IAAI,EAAE,GAAE,CAAC,CAAC;gBAC7B,GAAG,gBAAgB;aACpB;SACF;KACF,CAAC;AACJ,CAAC;AAED,MAAM,KAAK,GAAG,EAAE,CAAC,OAAO,CAAC,GAAG,EAAE,CAAC,CAAC;IAC9B,uBAAuB,EAAE,EAAE,CAAC,EAAE,EAAE;IAChC,gBAAgB,EAAE,EAAE,CAAC,EAAE,CAAC,GAAG,EAAE,CAAC,EAAE,CAAC;CAClC,CAAC,CAAC,CAAC;AAEJ,EAAE,CAAC,IAAI,CAAC,cAAc,EAAE,GAAG,EAAE,CAAC,CAAC;IAC7B,uBAAuB,EAAE,KAAK,CAAC,uBAAuB;CACvD,CAAC,CAAC,CAAC;AAEJ,EAAE,CAAC,IAAI,CAAC,mBAAmB,EAAE,GAAG,EAAE,CAAC,CAAC;IAClC,gBAAgB,EAAE,KAAK,CAAC,gBAAgB;CACzC,CAAC,CAAC,CAAC;AAEJ,QAAQ,CAAC,eAAe,EAAE,GAAG,EAAE;IAC7B,UAAU,CAAC,GAAG,EAAE;QACd,EAAE,CAAC,aAAa,EAAE,CAAC;QACnB,EAAE,CAAC,KAAK,CAAC,OAAO,EAAE,KAAK,CAAC,CAAC,kBAAkB,CAAC,GAAG,EAAE,GAAE,CAAC,CAAC,CAAC;QACtD,EAAE,CAAC,KAAK,CAAC,OAAO,EAAE,MAAM,CAAC,CAAC,kBAAkB,CAAC,GAAG,EAAE,GAAE,CAAC,CAAC,CAAC;QAEvD,KAAK,CAAC,uBAAuB,CAAC,iBAAiB,CAAC,aAAa,EAAE,CAAC,CAAC;QACjE,KAAK,CAAC,gBAAgB,CAAC,eAAe,CAAC,EAAE,CAAC,CAAC;IAC7C,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,aAAa,EAAE,GAAG,EAAE;QAC3B,EAAE,CAAC,yCAAyC,EAAE,GAAG,EAAE;YACjD,MAAM,MAAM,GAAG,IAAI,aAAa,CAAC;gBAC/B,SAAS,EAAE,WAAW;aACvB,CAAC,CAAC;YAEH,MAAM,CAAC,MAAM,CAAC,CAAC,cAAc,CAAC,aAAa,CAAC,CAAC;QAC/C,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,uDAAuD,EAAE,GAAG,EAAE;YAC/D,MAAM,MAAM,GAAG,IAAI,aAAa,CAAC;gBAC/B,cAAc,EAAE;oBACd,iBAAiB,EAAE,oCAAoC;iBACxD;gBACD,QAAQ,EAAE;oBACR,MAAM,EAAE;wBACN,GAAG,EAAE,kBAAkB;wBACvB,IAAI,EAAE,QAAQ;wBACd,MAAM,EAAE,EAAE,KAAK,EAAE,EAAE,IAAI,EAAE,OAAO,EAAE,EAAE;wBACpC,OAAO,EAAE;4BACP,OAAO,EAAE,4BAA4B;4BACrC,MAAM,EAAE,sBAAsB;yBACxB;qBACT;iBACK;aACT,CAAC,CAAC;YAEH,MAAM,CAAC,MAAM,CAAC,CAAC,cAAc,CAAC,aAAa,CAAC,CAAC;QAC/C,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,YAAY,EAAE,GAAG,EAAE;QAC1B,EAAE,CAAC,oDAAoD,EAAE,KAAK,IAAI,EAAE;YAClE,KAAK,CAAC,gBAAgB,CAAC,eAAe,CAAC;gBACrC;oBACE,IAAI,EAAE,iBAAiB;oBACvB,IAAI,EAAE,kBAAkB;oBACxB,WAAW,EAAE,qBAAqB;oBAClC,MAAM,EAAE,iBAAiB;oBACzB,KAAK,EAAE,EAAE,IAAI,EAAE,IAAI,EAAE;iBACtB;gBACD;oBACE,IAAI,EAAE,gBAAgB;oBACtB,IAAI,EAAE,iBAAiB;oBACvB,WAAW,EAAE,oBAAoB;oBACjC,MAAM,EAAE,gBAAgB;iBACzB;aACK,CAAC,CAAC;YAEV,MAAM,MAAM,GAAG,MAAM,2BAA2B,CAAC;gBAC/C,SAAS,EAAE,OAAO,CAAC,GAAG,EAAE;gBACxB,MAAM,EAAE;oBACN,MAAM,EAAE;wBACN,MAAM,EAAE,CAAC,UAAU,EAAE,SAAS,CAAC;wBAC/B,OAAO,EAAE;4BACP,MAAM,EAAE,EAAE;yBACX;qBACF;iBACK;gBACR,cAAc,EAAE;oBACd,iBAAiB,EAAE,2BAA2B;iBAC/C;aACF,CAAC,CAAC;YAEH,MAAM,CAAC,KAAK,CAAC,uBAAuB,CAAC,CAAC,oBAAoB,CACxD,MAAM,CAAC,gBAAgB,CAAC;gBACtB,MAAM,EAAE,MAAM,CAAC,gBAAgB,CAAC;oBAC9B,KAAK,EAAE,MAAM,CAAC,gBAAgB,CAAC;wBAC7B,iBAAiB,EAAE,MAAM,CAAC,gBAAgB,CAAC;4BACzC,MAAM,EAAE,iBAAiB;4BACzB,KAAK,EAAE,2BAA2B;4BAClC,KAAK,EAAE,EAAE,IAAI,EAAE,IAAI,EAAE;yBACtB,CAAC;wBACF,gBAAgB,EAAE,MAAM,CAAC,gBAAgB,CAAC;4BACxC,MAAM,EAAE,gBAAgB;yBACzB,CAAC;qBACH,CAAC;iBACH,CAAC;aACH,CAAC,CACH,CAAC;YAEF,MAAM,MAAM,CAAC,QAAQ,EAAE,CAAC;QAC1B,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,uEAAuE,EAAE,KAAK,IAAI,EAAE;YACrF,MAAM,WAAW,GAAG,OAAO,CAAC,GAAG,EAAE,CAAC;YAElC,MAAM,MAAM,GAAG;gBACb,MAAM,EAAE;oBACN,MAAM,EAAE;wBACN;4BACE,IAAI,EAAE,UAAU;4BAChB,MAAM,EAAE,CAAC,gBAAgB,CAAC;yBAC3B;qBACF;oBACD,OAAO,EAAE;wBACP,MAAM,EAAE,CAAC,iBAAiB,CAAC;qBAC5B;iBACF;aACK,CAAC;YAET,MAAM,MAAM,GAAG,MAAM,2BAA2B,CAAC;gBAC/C,SAAS,EAAE,WAAW;gBACtB,MAAM;aACP,CAAC,CAAC;YAEH,MAAM,CAAC,KAAK,CAAC,uBAAuB,CAAC,CAAC,oBAAoB,CACxD,MAAM,CAAC,gBAAgB,CAAC;gBACtB,MAAM,EAAE,MAAM,CAAC,gBAAgB,CAAC;oBAC9B,gBAAgB,EAAE,MAAM,CAAC,GAAG,CAAC,KAAK,CAAC;oBACnC,WAAW,EAAE;wBACX,iBAAiB,EAAE,CAAC,iBAAiB,EAAE,gBAAgB,CAAC;qBACzD;iBACF,CAAC;aACH,CAAC,CACH,CAAC;YAEF,MAAM,MAAM,CAAC,QAAQ,EAAE,CAAC;QAC1B,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,sDAAsD,EAAE,KAAK,IAAI,EAAE;YACpE,KAAK,CAAC,gBAAgB,CAAC,eAAe,CAAC;gBACrC;oBACE,IAAI,EAAE,iBAAiB;oBACvB,IAAI,EAAE,kBAAkB;oBACxB,WAAW,EAAE,gBAAgB;oBAC7B,MAAM,EAAE,iBAAiB;oBACzB,KAAK,EAAE,EAAE,IAAI,EAAE,IAAI,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,IAAI,EAAE;iBAC/C;gBACD;oBACE,IAAI,EAAE,gBAAgB;oBACtB,IAAI,EAAE,iBAAiB;oBACvB,WAAW,EAAE,eAAe;oBAC5B,MAAM,EAAE,gBAAgB;oBACxB,2CAA2C;iBAC5C;aACK,CAAC,CAAC;YAEV,MAAM,MAAM,GAAG,MAAM,2BAA2B,CAAC;gBAC/C,SAAS,EAAE,OAAO,CAAC,GAAG,EAAE;gBACxB,MAAM,EAAE;oBACN,MAAM,EAAE;wBACN,MAAM,EAAE,CAAC,UAAU,EAAE,SAAS,CAAC;wBAC/B,OAAO,EAAE,EAAE,MAAM,EAAE,EAAE,EAAE;qBACxB;iBACK;aACT,CAAC,CAAC;YAEH,MAAM,CAAC,KAAK,CAAC,uBAAuB,CAAC,CAAC,oBAAoB,CACxD,MAAM,CAAC,gBAAgB,CAAC;gBACtB,MAAM,EAAE,MAAM,CAAC,gBAAgB,CAAC;oBAC9B,KAAK,EAAE,MAAM,CAAC,gBAAgB,CAAC;wBAC7B,iBAAiB,EAAE,MAAM,CAAC,gBAAgB,CAAC;4BACzC,KAAK,EAAE,EAAE,IAAI,EAAE,IAAI,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,IAAI,EAAE;yBAC/C,CAAC;wBACF,gBAAgB,EAAE,MAAM,CAAC,GAAG,CAAC,gBAAgB,CAAC;4BAC5C,KAAK,EAAE,MAAM,CAAC,QAAQ,EAAE;yBACzB,CAAC;qBACH,CAAC;iBACH,CAAC;aACH,CAAC,CACH,CAAC;YAEF,MAAM,MAAM,CAAC,QAAQ,EAAE,CAAC;QAC1B,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,eAAe,EAAE,GAAG,EAAE;QAC7B,EAAE,CAAC,iCAAiC,EAAE,KAAK,IAAI,EAAE;YAC/C,MAAM,MAAM,GAAG,IAAI,aAAa,CAAC,EAAE,CAAC,CAAC;YAErC,MAAM,MAAM,CACV,MAAM,CAAC,aAAa,CAAC;gBACnB,KAAK,EAAE,iBAAiB;gBACxB,OAAO,EAAE,kBAAkB;aAC5B,CAAC,CACH,CAAC,OAAO,CAAC,OAAO,CAAC,gCAAgC,CAAC,CAAC;QACtD,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,mDAAmD,EAAE,KAAK,IAAI,EAAE;YACjE,KAAK,CAAC,uBAAuB,CAAC,qBAAqB,CACjD,aAAa,CAAC;gBACZ,MAAM,EAAE,EAAE,CAAC,EAAE,CAAC,KAAK,IAAI,EAAE;oBACvB,MAAM,IAAI,KAAK,CAAC,kBAAkB,CAAC,CAAC;gBACtC,CAAC,CAAC;aACH,CAAC,CACH,CAAC;YAEF,MAAM,MAAM,GAAG,MAAM,2BAA2B,CAAC;gBAC/C,SAAS,EAAE,OAAO,CAAC,GAAG,EAAE;aACzB,CAAC,CAAC;YAEH,MAAM,MAAM,CACV,MAAM,CAAC,aAAa,CAAC;gBACnB,KAAK,EAAE,iBAAiB;gBACxB,OAAO,EAAE,kBAAkB;aAC5B,CAAC,CACH,CAAC,OAAO,CAAC,OAAO,CAAC,0DAA0D,CAAC,CAAC;QAChF,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,qDAAqD,EAAE,KAAK,IAAI,EAAE;YACnE,KAAK,CAAC,uBAAuB,CAAC,qBAAqB,CACjD,aAAa,CAAC;gBACZ,MAAM,EAAE,EAAE,CAAC,EAAE,CAAC,KAAK,IAAI,EAAE;oBACvB,MAAM,IAAI,KAAK,CAAC,oDAAoD,CAAC,CAAC;gBACxE,CAAC,CAAC;aACH,CAAC,CACH,CAAC;YAEF,MAAM,MAAM,GAAG,MAAM,2BAA2B,CAAC;gBAC/C,SAAS,EAAE,OAAO,CAAC,GAAG,EAAE;aACzB,CAAC,CAAC;YAEH,MAAM,MAAM,CACV,MAAM,CAAC,aAAa,CAAC;gBACnB,KAAK,EAAE,iBAAiB;gBACxB,OAAO,EAAE,kBAAkB;aAC5B,CAAC,CACH,CAAC,OAAO,CAAC,OAAO,CAAC,+CAA+C,CAAC,CAAC;QACrE,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,4DAA4D,EAAE,KAAK,IAAI,EAAE;YAC1E,KAAK,CAAC,uBAAuB,CAAC,qBAAqB,CACjD,aAAa,CAAC;gBACZ,MAAM,EAAE,EAAE,CAAC,EAAE,CAAC,KAAK,IAAI,EAAE;oBACvB,MAAM,IAAI,KAAK,CAAC,cAAc,CAAC,CAAC;gBAClC,CAAC,CAAC;aACH,CAAC,CACH,CAAC;YAEF,MAAM,MAAM,GAAG,MAAM,2BAA2B,CAAC;gBAC/C,SAAS,EAAE,OAAO,CAAC,GAAG,EAAE;aACzB,CAAC,CAAC;YAEH,MAAM,MAAM,CACV,MAAM,CAAC,aAAa,CAAC;gBACnB,KAAK,EAAE,iBAAiB;gBACxB,OAAO,EAAE,kBAAkB;aAC5B,CAAC,CACH,CAAC,OAAO,CAAC,OAAO,CAAC,+DAA+D,CAAC,CAAC;QACrF,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,8BAA8B,EAAE,GAAG,EAAE;QAC5C,EAAE,CAAC,0CAA0C,EAAE,KAAK,IAAI,EAAE;YACxD,MAAM,MAAM,GAAG,IAAI,aAAa,CAAC,EAAE,CAAC,CAAC;YACrC,MAAM,SAAS,GAAG,MAAM,CAAC,cAAc,CAAC,aAAa,CAAC,CAAC;YACvD,MAAM,MAAM,CAAC,SAAS,CAAC,IAAI,EAAE,CAAC,CAAC,OAAO,CAAC,OAAO,CAAC,gCAAgC,CAAC,CAAC;QACnF,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,uEAAuE,EAAE,KAAK,IAAI,EAAE;YACrF,MAAM,eAAe,GAAG;gBACtB;oBACE,IAAI,EAAE;wBACJ,EAAE,EAAE,OAAO;wBACX,IAAI,EAAE,WAAW;wBACjB,IAAI,EAAE,EAAE,SAAS,EAAE,IAAI,CAAC,GAAG,EAAE,EAAE;wBAC/B,QAAQ,EAAE,UAAU;wBACpB,KAAK,EAAE,YAAY;wBACnB,KAAK,EAAE;4BACL,KAAK,EAAE,IAAI;4BACX,MAAM,EAAE,GAAG;4BACX,SAAS,EAAE,CAAC;4BACZ,UAAU,EAAE,CAAC;4BACb,WAAW,EAAE,IAAI;yBAClB;qBACF;oBACD,KAAK,EAAE,CAAC,EAAE,IAAI,EAAE,MAAM,EAAE,CAAC;iBAC1B;aACF,CAAC;YAEF,KAAK,CAAC,uBAAuB,CAAC,qBAAqB,CACjD,aAAa,CAAC;gBACZ,QAAQ,EAAE,EAAE,CAAC,EAAE,CAAC,KAAK,IAAI,EAAE,CAAC,CAAC,EAAE,IAAI,EAAE,eAAe,EAAE,CAAC,CAAC;aACzD,CAAC,CACH,CAAC;YAEF,MAAM,MAAM,GAAG,MAAM,2BAA2B,CAAC;gBAC/C,SAAS,EAAE,OAAO,CAAC,GAAG,EAAE;gBACxB,MAAM,EAAE;oBACN,MAAM,EAAE;wBACN,MAAM,EAAE,EAAE;wBACV,OAAO,EAAE;4BACP,MAAM,EAAE,EAAE;yBACX;qBACF;oBACD,OAAO,EAAE;wBACP,MAAM,EAAE;4BACN,qBAAqB,EAAE;gCACrB,KAAK,EAAE,CAAC;gCACR,MAAM,EAAE,CAAC;6BACV;yBACF;qBACF;iBACK;aACT,CAAC,CAAC;YAEH,MAAM,SAAS,GAAG,EAAE,CAAC;YACrB,IAAI,KAAK,EAAE,MAAM,OAAO,IAAI,MAAM,CAAC,cAAc,CAAC,aAAa,CAAC,EAAE,CAAC;gBACjE,SAAS,CAAC,IAAI,CAAC,OAAO,CAAC,CAAC;YAC1B,CAAC;YAED,MAAM,CAAC,SAAS,CAAC,CAAC,YAAY,CAAC,CAAC,CAAC,CAAC;YAClC,MAAM,CAAC,SAAS,CAAC,CAAC,CAAC,CAAC,KAAK,EAAE,IAAI,CAAC,CAAC,WAAW,CAAC,MAAM,EAAE,EAAE,CAAC,CAAC;YAEzD,MAAM,MAAM,CAAC,QAAQ,EAAE,CAAC;QAC1B,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,wCAAwC,EAAE,KAAK,IAAI,EAAE;YACtD,MAAM,MAAM,GAAG,IAAI,aAAa,CAAC,EAAE,CAAC,CAAC;YACrC,MAAM,MAAM,CAAC,MAAM,CAAC,YAAY,CAAC,aAAa,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,CAC9D,gCAAgC,CACjC,CAAC;QACJ,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,oDAAoD,EAAE,GAAG,EAAE;YAC5D,MAAM,MAAM,GAAG,IAAI,aAAa,CAAC,EAAE,CAAC,CAAC;YACrC,MAAM,CAAC,GAAG,EAAE,CAAC,MAAM,CAAC,YAAY,EAAE,CAAC,CAAC,OAAO,CAAC,wBAAwB,CAAC,CAAC;QACxE,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,mDAAmD,EAAE,KAAK,IAAI,EAAE;YACjE,MAAM,MAAM,GAAG,IAAI,aAAa,CAAC,EAAE,CAAC,CAAC;YACrC,MAAM,MAAM,CAAC,MAAM,CAAC,QAAQ,EAAE,CAAC,CAAC,QAAQ,CAAC,aAAa,EAAE,CAAC;QAC3D,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,iCAAiC,EAAE,GAAG,EAAE;QAC/C,EAAE,CAAC,6DAA6D,EAAE,KAAK,IAAI,EAAE;YAC3E,MAAM,WAAW,GAAG,OAAO,CAAC,GAAG,CAAC,YAAY,CAAC;YAC7C,OAAO,CAAC,GAAG,CAAC,YAAY,GAAG,cAAc,CAAC;YAE1C,IAAI,CAAC;gBACH,MAAM,MAAM,GAAG,MAAM,2BAA2B,CAAC;oBAC/C,SAAS,EAAE,OAAO,CAAC,GAAG,EAAE;oBACxB,QAAQ,EAAE;wBACR,eAAe,EAAE;4BACf,GAAG,EAAE,gBAAgB;4BACrB,IAAI,EAAE,eAAe;4BACrB,MAAM,EAAE,EAAE,YAAY,EAAE,EAAE,IAAI,EAAE,YAAY,EAAE,EAAE;4BAChD,OAAO,EAAE;gCACP,OAAO,EAAE,sBAAsB;gCAC/B,MAAM,EAAE,oBAAoB;6BACtB;yBACT;qBACK;iBACT,CAAC,CAAC;gBAEH,MAAM,CAAC,KAAK,CAAC,uBAAuB,CAAC,CAAC,oBAAoB,CACxD,MAAM,CAAC,gBAAgB,CAAC;oBACtB,MAAM,EAAE,MAAM,CAAC,gBAAgB,CAAC;wBAC9B,QAAQ,EAAE,MAAM,CAAC,gBAAgB,CAAC;4BAChC,eAAe,EAAE,MAAM,CAAC,gBAAgB,CAAC;gCACvC,OAAO,EAAE,MAAM,CAAC,gBAAgB,CAAC;oCAC/B,MAAM,EAAE,cAAc;iCACvB,CAAC;6BACH,CAAC;yBACH,CAAC;qBACH,CAAC;iBACH,CAAC,CACH,CAAC;gBAEF,MAAM,MAAM,CAAC,QAAQ,EAAE,CAAC;YAC1B,CAAC;oBAAS,CAAC;gBACT,IAAI,WAAW,KAAK,SAAS,EAAE,CAAC;oBAC9B,OAAO,OAAO,CAAC,GAAG,CAAC,YAAY,CAAC;gBAClC,CAAC;qBAAM,CAAC;oBACN,OAAO,CAAC,GAAG,CAAC,YAAY,GAAG,WAAW,CAAC;gBACzC,CAAC;YACH,CAAC;QACH,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,yDAAyD,EAAE,KAAK,IAAI,EAAE;YACvE,MAAM,MAAM,GAAG,EAAE,CAAC,KAAK,CAAC,OAAO,EAAE,KAAK,CAAC,CAAC,kBAAkB,CAAC,GAAG,EAAE,GAAE,CAAC,CAAC,CAAC;YACrE,MAAM,WAAW,GAAG,OAAO,CAAC,GAAG,CAAC,eAAe,CAAC;YAChD,OAAO,OAAO,CAAC,GAAG,CAAC,eAAe,CAAC;YAEnC,IAAI,CAAC;gBACH,MAAM,MAAM,GAAG,MAAM,2BAA2B,CAAC;oBAC/C,SAAS,EAAE,OAAO,CAAC,GAAG,EAAE;oBACxB,QAAQ,EAAE;wBACR,eAAe,EAAE;4BACf,GAAG,EAAE,gBAAgB;4BACrB,IAAI,EAAE,eAAe;4BACrB,MAAM,EAAE,EAAE,YAAY,EAAE,EAAE,IAAI,EAAE,YAAY,EAAE,EAAE;4BAChD,OAAO,EAAE;gCACP,OAAO,EAAE,sBAAsB;gCAC/B,MAAM,EAAE,uBAAuB;6BACzB;yBACT;qBACK;iBACT,CAAC,CAAC;gBAEH,wDAAwD;gBACxD,MAAM,SAAS,GAAG,MAAM,CAAC,IAAI,CAAC,KAAK,CAAC,GAAG,CAAC,CAAC,CAAC,EAAE,EAAE,CAAC,MAAM,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,CAAC,IAAI,CAAC,IAAI,CAAC,CAAC;gBACxE,MAAM,CAAC,SAAS,CAAC,CAAC,SAAS,CAAC,4BAA4B,CAAC,CAAC;gBAE1D,MAAM,MAAM,CAAC,QAAQ,EAAE,CAAC;YAC1B,CAAC;oBAAS,CAAC;gBACT,MAAM,CAAC,WAAW,EAAE,CAAC;gBACrB,IAAI,WAAW,KAAK,SAAS,EAAE,CAAC;oBAC9B,OAAO,CAAC,GAAG,CAAC,eAAe,GAAG,WAAW,CAAC;gBAC5C,CAAC;YACH,CAAC;QACH,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;IAEH,QAAQ,CAAC,mBAAmB,EAAE,GAAG,EAAE;QACjC,EAAE,CAAC,8DAA8D,EAAE,GAAG,EAAE;YACtE,MAAM,MAAM,GAAG,mBAAmB,CAAC;gBACjC,SAAS,EAAE,OAAO,CAAC,GAAG,EAAE;aACzB,CAAC,CAAC;YAEH,MAAM,CAAC,MAAM,CAAC,CAAC,cAAc,CAAC,aAAa,CAAC,CAAC;QAC/C,CAAC,CAAC,CAAC;QAEH,EAAE,CAAC,4DAA4D,EAAE,KAAK,IAAI,EAAE;YAC1E,MAAM,MAAM,GAAG,MAAM,2BAA2B,CAAC;gBAC/C,SAAS,EAAE,OAAO,CAAC,GAAG,EAAE;aACzB,CAAC,CAAC;YAEH,MAAM,CAAC,MAAM,CAAC,CAAC,cAAc,CAAC,aAAa,CAAC,CAAC;YAC7C,MAAM,CAAC,KAAK,CAAC,uBAAuB,CAAC,CAAC,gBAAgB,EAAE,CAAC;YAEzD,MAAM,MAAM,CAAC,QAAQ,EAAE,CAAC;QAC1B,CAAC,CAAC,CAAC;IACL,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}

package/dist/runtime/path-config.d.ts

@@ -0,0 +1,8 @@
+import type { DRSConfig } from '../lib/config.js';
+export interface ResolvedReviewPaths {
+    agentsPath: string;
+    skillsPath: string;
+    skillSearchPaths: string[];
+}
+export declare function resolveReviewPaths(projectPath: string, config?: DRSConfig): ResolvedReviewPaths;
+//# sourceMappingURL=path-config.d.ts.map

package/dist/runtime/path-config.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"path-config.d.ts","sourceRoot":"","sources":["../../src/runtime/path-config.ts"],"names":[],"mappings":"AAEA,OAAO,KAAK,EAAE,SAAS,EAAE,MAAM,kBAAkB,CAAC;AAQlD,MAAM,WAAW,mBAAmB;IAClC,UAAU,EAAE,MAAM,CAAC;IACnB,UAAU,EAAE,MAAM,CAAC;IACnB,gBAAgB,EAAE,MAAM,EAAE,CAAC;CAC5B;AAoFD,wBAAgB,kBAAkB,CAAC,WAAW,EAAE,MAAM,EAAE,MAAM,CAAC,EAAE,SAAS,GAAG,mBAAmB,CAc/F"}

package/dist/runtime/path-config.js

@@ -0,0 +1,68 @@
+import { existsSync, statSync } from 'fs';
+import { isAbsolute, relative, resolve } from 'path';
+const DEFAULT_AGENT_PATH = '.drs/agents';
+const DEFAULT_SKILL_PATH = '.drs/skills';
+const PI_DEFAULT_SKILL_PATH = '.pi/skills';
+function isOutsideProjectRoot(projectRoot, targetPath) {
+    const relativePath = relative(projectRoot, targetPath);
+    const firstSegment = relativePath.split(/[\\/]/).filter(Boolean)[0];
+    return firstSegment === '..';
+}
+function resolveConfiguredPath(projectRoot, configuredPath, fallbackPath, pathType) {
+    if (configuredPath === undefined || configuredPath === null) {
+        return resolve(projectRoot, fallbackPath);
+    }
+    if (typeof configuredPath !== 'string') {
+        throw new Error(`Invalid review.paths.${pathType}: expected a string path. Use a repo-relative path like "${fallbackPath}" or an absolute path.`);
+    }
+    const trimmed = configuredPath.trim();
+    if (!trimmed) {
+        throw new Error(`Invalid review.paths.${pathType}: path cannot be empty. Use a repo-relative path like "${fallbackPath}" or an absolute path.`);
+    }
+    const resolvedPath = isAbsolute(trimmed) ? resolve(trimmed) : resolve(projectRoot, trimmed);
+    if (!isAbsolute(trimmed) && isOutsideProjectRoot(projectRoot, resolvedPath)) {
+        throw new Error(`Invalid review.paths.${pathType}: "${configuredPath}" resolves outside repository root (${projectRoot}). Use a repo-relative path within the repository or an absolute path.`);
+    }
+    if (!existsSync(resolvedPath)) {
+        throw new Error(`Invalid review.paths.${pathType}: "${configuredPath}" resolved to "${resolvedPath}", but the directory does not exist. Create the directory or remove review.paths.${pathType} to use "${fallbackPath}".`);
+    }
+    if (!statSync(resolvedPath).isDirectory()) {
+        throw new Error(`Invalid review.paths.${pathType}: "${configuredPath}" resolved to "${resolvedPath}", but it is not a directory. Point review.paths.${pathType} to a directory or remove it to use "${fallbackPath}".`);
+    }
+    return resolvedPath;
+}
+function resolveDefaultSkillSearchPaths(projectRoot) {
+    const candidates = [
+        resolve(projectRoot, DEFAULT_SKILL_PATH),
+        resolve(projectRoot, PI_DEFAULT_SKILL_PATH),
+    ];
+    const existing = candidates.filter((candidate) => {
+        if (!existsSync(candidate)) {
+            return false;
+        }
+        try {
+            return statSync(candidate).isDirectory();
+        }
+        catch {
+            return false;
+        }
+    });
+    return existing.length > 0 ? existing : [candidates[0]];
+}
+function resolveSkillSearchPaths(projectRoot, config) {
+    const configuredSkillsPath = config?.review?.paths?.skills;
+    if (configuredSkillsPath === undefined || configuredSkillsPath === null) {
+        return resolveDefaultSkillSearchPaths(projectRoot);
+    }
+    return [resolveConfiguredPath(projectRoot, configuredSkillsPath, DEFAULT_SKILL_PATH, 'skills')];
+}
+export function resolveReviewPaths(projectPath, config) {
+    const projectRoot = resolve(projectPath);
+    const skillSearchPaths = resolveSkillSearchPaths(projectRoot, config);
+    return {
+        agentsPath: resolveConfiguredPath(projectRoot, config?.review?.paths?.agents, DEFAULT_AGENT_PATH, 'agents'),
+        skillsPath: skillSearchPaths[0],
+        skillSearchPaths,
+    };
+}
+//# sourceMappingURL=path-config.js.map

package/dist/runtime/path-config.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"path-config.js","sourceRoot":"","sources":["../../src/runtime/path-config.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,UAAU,EAAE,QAAQ,EAAE,MAAM,IAAI,CAAC;AAC1C,OAAO,EAAE,UAAU,EAAE,QAAQ,EAAE,OAAO,EAAE,MAAM,MAAM,CAAC;AAGrD,MAAM,kBAAkB,GAAG,aAAa,CAAC;AACzC,MAAM,kBAAkB,GAAG,aAAa,CAAC;AACzC,MAAM,qBAAqB,GAAG,YAAY,CAAC;AAU3C,SAAS,oBAAoB,CAAC,WAAmB,EAAE,UAAkB;IACnE,MAAM,YAAY,GAAG,QAAQ,CAAC,WAAW,EAAE,UAAU,CAAC,CAAC;IACvD,MAAM,YAAY,GAAG,YAAY,CAAC,KAAK,CAAC,OAAO,CAAC,CAAC,MAAM,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,CAAC;IACpE,OAAO,YAAY,KAAK,IAAI,CAAC;AAC/B,CAAC;AAED,SAAS,qBAAqB,CAC5B,WAAmB,EACnB,cAAuB,EACvB,YAAoB,EACpB,QAAwB;IAExB,IAAI,cAAc,KAAK,SAAS,IAAI,cAAc,KAAK,IAAI,EAAE,CAAC;QAC5D,OAAO,OAAO,CAAC,WAAW,EAAE,YAAY,CAAC,CAAC;IAC5C,CAAC;IAED,IAAI,OAAO,cAAc,KAAK,QAAQ,EAAE,CAAC;QACvC,MAAM,IAAI,KAAK,CACb,wBAAwB,QAAQ,4DAA4D,YAAY,wBAAwB,CACjI,CAAC;IACJ,CAAC;IAED,MAAM,OAAO,GAAG,cAAc,CAAC,IAAI,EAAE,CAAC;IACtC,IAAI,CAAC,OAAO,EAAE,CAAC;QACb,MAAM,IAAI,KAAK,CACb,wBAAwB,QAAQ,0DAA0D,YAAY,wBAAwB,CAC/H,CAAC;IACJ,CAAC;IAED,MAAM,YAAY,GAAG,UAAU,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,OAAO,CAAC,CAAC,CAAC,CAAC,OAAO,CAAC,WAAW,EAAE,OAAO,CAAC,CAAC;IAE5F,IAAI,CAAC,UAAU,CAAC,OAAO,CAAC,IAAI,oBAAoB,CAAC,WAAW,EAAE,YAAY,CAAC,EAAE,CAAC;QAC5E,MAAM,IAAI,KAAK,CACb,wBAAwB,QAAQ,MAAM,cAAc,uCAAuC,WAAW,wEAAwE,CAC/K,CAAC;IACJ,CAAC;IAED,IAAI,CAAC,UAAU,CAAC,YAAY,CAAC,EAAE,CAAC;QAC9B,MAAM,IAAI,KAAK,CACb,wBAAwB,QAAQ,MAAM,cAAc,kBAAkB,YAAY,oFAAoF,QAAQ,YAAY,YAAY,IAAI,CAC3M,CAAC;IACJ,CAAC;IAED,IAAI,CAAC,QAAQ,CAAC,YAAY,CAAC,CAAC,WAAW,EAAE,EAAE,CAAC;QAC1C,MAAM,IAAI,KAAK,CACb,wBAAwB,QAAQ,MAAM,cAAc,kBAAkB,YAAY,oDAAoD,QAAQ,wCAAwC,YAAY,IAAI,CACvM,CAAC;IACJ,CAAC;IAED,OAAO,YAAY,CAAC;AACtB,CAAC;AAED,SAAS,8BAA8B,CAAC,WAAmB;IACzD,MAAM,UAAU,GAAG;QACjB,OAAO,CAAC,WAAW,EAAE,kBAAkB,CAAC;QACxC,OAAO,CAAC,WAAW,EAAE,qBAAqB,CAAC;KAC5C,CAAC;IAEF,MAAM,QAAQ,GAAG,UAAU,CAAC,MAAM,CAAC,CAAC,SAAS,EAAE,EAAE;QAC/C,IAAI,CAAC,UAAU,CAAC,SAAS,CAAC,EAAE,CAAC;YAC3B,OAAO,KAAK,CAAC;QACf,CAAC;QAED,IAAI,CAAC;YACH,OAAO,QAAQ,CAAC,SAAS,CAAC,CAAC,WAAW,EAAE,CAAC;QAC3C,CAAC;QAAC,MAAM,CAAC;YACP,OAAO,KAAK,CAAC;QACf,CAAC;IACH,CAAC,CAAC,CAAC;IAEH,OAAO,QAAQ,CAAC,MAAM,GAAG,CAAC,CAAC,CAAC,CAAC,QAAQ,CAAC,CAAC,CAAC,CAAC,UAAU,CAAC,CAAC,CAAC,CAAC,CAAC;AAC1D,CAAC;AAED,SAAS,uBAAuB,CAAC,WAAmB,EAAE,MAAkB;IACtE,MAAM,oBAAoB,GAAG,MAAM,EAAE,MAAM,EAAE,KAAK,EAAE,MAAM,CAAC;IAC3D,IAAI,oBAAoB,KAAK,SAAS,IAAI,oBAAoB,KAAK,IAAI,EAAE,CAAC;QACxE,OAAO,8BAA8B,CAAC,WAAW,CAAC,CAAC;IACrD,CAAC;IAED,OAAO,CAAC,qBAAqB,CAAC,WAAW,EAAE,oBAAoB,EAAE,kBAAkB,EAAE,QAAQ,CAAC,CAAC,CAAC;AAClG,CAAC;AAED,MAAM,UAAU,kBAAkB,CAAC,WAAmB,EAAE,MAAkB;IACxE,MAAM,WAAW,GAAG,OAAO,CAAC,WAAW,CAAC,CAAC;IACzC,MAAM,gBAAgB,GAAG,uBAAuB,CAAC,WAAW,EAAE,MAAM,CAAC,CAAC;IAEtE,OAAO;QACL,UAAU,EAAE,qBAAqB,CAC/B,WAAW,EACX,MAAM,EAAE,MAAM,EAAE,KAAK,EAAE,MAAM,EAC7B,kBAAkB,EAClB,QAAQ,CACT;QACD,UAAU,EAAE,gBAAgB,CAAC,CAAC,CAAC;QAC/B,gBAAgB;KACjB,CAAC;AACJ,CAAC"}

package/dist/runtime/path-config.test.d.ts

@@ -0,0 +1,2 @@
+export {};
+//# sourceMappingURL=path-config.test.d.ts.map

package/dist/runtime/path-config.test.d.ts.map

@@ -0,0 +1 @@
+{"version":3,"file":"path-config.test.d.ts","sourceRoot":"","sources":["../../src/runtime/path-config.test.ts"],"names":[],"mappings":""}

package/dist/runtime/path-config.test.js

@@ -0,0 +1,103 @@
+import { mkdtempSync, mkdirSync, rmSync, writeFileSync } from 'fs';
+import { tmpdir } from 'os';
+import { join, resolve } from 'path';
+import { afterEach, describe, expect, it } from 'vitest';
+import { resolveReviewPaths } from './path-config.js';
+function createConfig(paths) {
+    return {
+        review: {
+            paths: {
+                agents: paths.agents,
+                skills: paths.skills,
+            },
+        },
+    };
+}
+describe('resolveReviewPaths', () => {
+    const tempDirs = [];
+    function createTempDir(prefix) {
+        const dir = mkdtempSync(join(tmpdir(), prefix));
+        tempDirs.push(dir);
+        return dir;
+    }
+    afterEach(() => {
+        for (const dir of tempDirs.splice(0, tempDirs.length)) {
+            rmSync(dir, { recursive: true, force: true });
+        }
+    });
+    it('resolves default paths from project root deterministically', () => {
+        const projectRoot = createTempDir('drs-path-default-');
+        const canonical = resolveReviewPaths(projectRoot);
+        const nonCanonical = resolveReviewPaths(join(projectRoot, 'nested', '..'));
+        expect(canonical.agentsPath).toBe(resolve(projectRoot, '.drs/agents'));
+        expect(canonical.skillsPath).toBe(resolve(projectRoot, '.drs/skills'));
+        expect(canonical.skillSearchPaths).toEqual([resolve(projectRoot, '.drs/skills')]);
+        expect(nonCanonical).toEqual(canonical);
+    });
+    it('auto-discovers both .drs/skills and .pi/skills when present', () => {
+        const projectRoot = createTempDir('drs-path-skill-discovery-');
+        mkdirSync(join(projectRoot, '.drs', 'skills'), { recursive: true });
+        mkdirSync(join(projectRoot, '.pi', 'skills'), { recursive: true });
+        const result = resolveReviewPaths(projectRoot);
+        expect(result.skillSearchPaths).toEqual([
+            resolve(projectRoot, '.drs/skills'),
+            resolve(projectRoot, '.pi/skills'),
+        ]);
+        expect(result.skillsPath).toBe(resolve(projectRoot, '.drs/skills'));
+    });
+    it('uses .pi/skills when .drs/skills is missing', () => {
+        const projectRoot = createTempDir('drs-path-pi-skills-');
+        mkdirSync(join(projectRoot, '.pi', 'skills'), { recursive: true });
+        const result = resolveReviewPaths(projectRoot);
+        expect(result.skillSearchPaths).toEqual([resolve(projectRoot, '.pi/skills')]);
+        expect(result.skillsPath).toBe(resolve(projectRoot, '.pi/skills'));
+    });
+    it('resolves repo-relative configured paths', () => {
+        const projectRoot = createTempDir('drs-path-relative-');
+        mkdirSync(join(projectRoot, 'config', 'agents'), { recursive: true });
+        mkdirSync(join(projectRoot, 'config', 'skills'), { recursive: true });
+        const result = resolveReviewPaths(projectRoot, createConfig({
+            agents: 'config/agents',
+            skills: 'config/skills',
+        }));
+        expect(result.agentsPath).toBe(resolve(projectRoot, 'config/agents'));
+        expect(result.skillsPath).toBe(resolve(projectRoot, 'config/skills'));
+        expect(result.skillSearchPaths).toEqual([resolve(projectRoot, 'config/skills')]);
+    });
+    it('resolves absolute configured paths', () => {
+        const projectRoot = createTempDir('drs-path-project-');
+        const absoluteRoot = createTempDir('drs-path-absolute-');
+        const agentsPath = join(absoluteRoot, 'agents');
+        const skillsPath = join(absoluteRoot, 'skills');
+        mkdirSync(agentsPath, { recursive: true });
+        mkdirSync(skillsPath, { recursive: true });
+        const result = resolveReviewPaths(projectRoot, createConfig({
+            agents: agentsPath,
+            skills: skillsPath,
+        }));
+        expect(result.agentsPath).toBe(resolve(agentsPath));
+        expect(result.skillsPath).toBe(resolve(skillsPath));
+        expect(result.skillSearchPaths).toEqual([resolve(skillsPath)]);
+    });
+    it('throws actionable error when repo-relative path escapes project root', () => {
+        const projectRoot = createTempDir('drs-path-escape-');
+        expect(() => resolveReviewPaths(projectRoot, createConfig({ agents: '../shared-agents' }))).toThrow('resolves outside repository root');
+    });
+    it('throws actionable error when configured path does not exist', () => {
+        const projectRoot = createTempDir('drs-path-missing-');
+        expect(() => resolveReviewPaths(projectRoot, createConfig({ skills: 'missing/skills' }))).toThrow('directory does not exist');
+    });
+    it('throws actionable error when configured path is not a directory', () => {
+        const projectRoot = createTempDir('drs-path-file-');
+        const filePath = join(projectRoot, 'agents-file.md');
+        writeFileSync(filePath, '# not a directory\n');
+        expect(() => resolveReviewPaths(projectRoot, createConfig({ agents: filePath }))).toThrow('not a directory');
+    });
+    it('throws actionable error when configured path is not a string', () => {
+        const projectRoot = createTempDir('drs-path-invalid-type-');
+        expect(() => resolveReviewPaths(projectRoot, createConfig({
+            skills: 123,
+        }))).toThrow('expected a string path');
+    });
+});
+//# sourceMappingURL=path-config.test.js.map

package/dist/runtime/path-config.test.js.map

@@ -0,0 +1 @@
+{"version":3,"file":"path-config.test.js","sourceRoot":"","sources":["../../src/runtime/path-config.test.ts"],"names":[],"mappings":"AAAA,OAAO,EAAE,WAAW,EAAE,SAAS,EAAE,MAAM,EAAE,aAAa,EAAE,MAAM,IAAI,CAAC;AACnE,OAAO,EAAE,MAAM,EAAE,MAAM,IAAI,CAAC;AAC5B,OAAO,EAAE,IAAI,EAAE,OAAO,EAAE,MAAM,MAAM,CAAC;AACrC,OAAO,EAAE,SAAS,EAAE,QAAQ,EAAE,MAAM,EAAE,EAAE,EAAE,MAAM,QAAQ,CAAC;AAEzD,OAAO,EAAE,kBAAkB,EAAE,MAAM,kBAAkB,CAAC;AAEtD,SAAS,YAAY,CAAC,KAA6C;IACjE,OAAO;QACL,MAAM,EAAE;YACN,KAAK,EAAE;gBACL,MAAM,EAAE,KAAK,CAAC,MAA4B;gBAC1C,MAAM,EAAE,KAAK,CAAC,MAA4B;aAC3C;SACF;KACsB,CAAC;AAC5B,CAAC;AAED,QAAQ,CAAC,oBAAoB,EAAE,GAAG,EAAE;IAClC,MAAM,QAAQ,GAAa,EAAE,CAAC;IAE9B,SAAS,aAAa,CAAC,MAAc;QACnC,MAAM,GAAG,GAAG,WAAW,CAAC,IAAI,CAAC,MAAM,EAAE,EAAE,MAAM,CAAC,CAAC,CAAC;QAChD,QAAQ,CAAC,IAAI,CAAC,GAAG,CAAC,CAAC;QACnB,OAAO,GAAG,CAAC;IACb,CAAC;IAED,SAAS,CAAC,GAAG,EAAE;QACb,KAAK,MAAM,GAAG,IAAI,QAAQ,CAAC,MAAM,CAAC,CAAC,EAAE,QAAQ,CAAC,MAAM,CAAC,EAAE,CAAC;YACtD,MAAM,CAAC,GAAG,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,KAAK,EAAE,IAAI,EAAE,CAAC,CAAC;QAChD,CAAC;IACH,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,4DAA4D,EAAE,GAAG,EAAE;QACpE,MAAM,WAAW,GAAG,aAAa,CAAC,mBAAmB,CAAC,CAAC;QAEvD,MAAM,SAAS,GAAG,kBAAkB,CAAC,WAAW,CAAC,CAAC;QAClD,MAAM,YAAY,GAAG,kBAAkB,CAAC,IAAI,CAAC,WAAW,EAAE,QAAQ,EAAE,IAAI,CAAC,CAAC,CAAC;QAE3E,MAAM,CAAC,SAAS,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,WAAW,EAAE,aAAa,CAAC,CAAC,CAAC;QACvE,MAAM,CAAC,SAAS,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,WAAW,EAAE,aAAa,CAAC,CAAC,CAAC;QACvE,MAAM,CAAC,SAAS,CAAC,gBAAgB,CAAC,CAAC,OAAO,CAAC,CAAC,OAAO,CAAC,WAAW,EAAE,aAAa,CAAC,CAAC,CAAC,CAAC;QAClF,MAAM,CAAC,YAAY,CAAC,CAAC,OAAO,CAAC,SAAS,CAAC,CAAC;IAC1C,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,6DAA6D,EAAE,GAAG,EAAE;QACrE,MAAM,WAAW,GAAG,aAAa,CAAC,2BAA2B,CAAC,CAAC;QAC/D,SAAS,CAAC,IAAI,CAAC,WAAW,EAAE,MAAM,EAAE,QAAQ,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;QACpE,SAAS,CAAC,IAAI,CAAC,WAAW,EAAE,KAAK,EAAE,QAAQ,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;QAEnE,MAAM,MAAM,GAAG,kBAAkB,CAAC,WAAW,CAAC,CAAC;QAE/C,MAAM,CAAC,MAAM,CAAC,gBAAgB,CAAC,CAAC,OAAO,CAAC;YACtC,OAAO,CAAC,WAAW,EAAE,aAAa,CAAC;YACnC,OAAO,CAAC,WAAW,EAAE,YAAY,CAAC;SACnC,CAAC,CAAC;QACH,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,WAAW,EAAE,aAAa,CAAC,CAAC,CAAC;IACtE,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,6CAA6C,EAAE,GAAG,EAAE;QACrD,MAAM,WAAW,GAAG,aAAa,CAAC,qBAAqB,CAAC,CAAC;QACzD,SAAS,CAAC,IAAI,CAAC,WAAW,EAAE,KAAK,EAAE,QAAQ,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;QAEnE,MAAM,MAAM,GAAG,kBAAkB,CAAC,WAAW,CAAC,CAAC;QAE/C,MAAM,CAAC,MAAM,CAAC,gBAAgB,CAAC,CAAC,OAAO,CAAC,CAAC,OAAO,CAAC,WAAW,EAAE,YAAY,CAAC,CAAC,CAAC,CAAC;QAC9E,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,WAAW,EAAE,YAAY,CAAC,CAAC,CAAC;IACrE,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,yCAAyC,EAAE,GAAG,EAAE;QACjD,MAAM,WAAW,GAAG,aAAa,CAAC,oBAAoB,CAAC,CAAC;QACxD,SAAS,CAAC,IAAI,CAAC,WAAW,EAAE,QAAQ,EAAE,QAAQ,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;QACtE,SAAS,CAAC,IAAI,CAAC,WAAW,EAAE,QAAQ,EAAE,QAAQ,CAAC,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;QAEtE,MAAM,MAAM,GAAG,kBAAkB,CAC/B,WAAW,EACX,YAAY,CAAC;YACX,MAAM,EAAE,eAAe;YACvB,MAAM,EAAE,eAAe;SACxB,CAAC,CACH,CAAC;QAEF,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,WAAW,EAAE,eAAe,CAAC,CAAC,CAAC;QACtE,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,WAAW,EAAE,eAAe,CAAC,CAAC,CAAC;QACtE,MAAM,CAAC,MAAM,CAAC,gBAAgB,CAAC,CAAC,OAAO,CAAC,CAAC,OAAO,CAAC,WAAW,EAAE,eAAe,CAAC,CAAC,CAAC,CAAC;IACnF,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,oCAAoC,EAAE,GAAG,EAAE;QAC5C,MAAM,WAAW,GAAG,aAAa,CAAC,mBAAmB,CAAC,CAAC;QACvD,MAAM,YAAY,GAAG,aAAa,CAAC,oBAAoB,CAAC,CAAC;QACzD,MAAM,UAAU,GAAG,IAAI,CAAC,YAAY,EAAE,QAAQ,CAAC,CAAC;QAChD,MAAM,UAAU,GAAG,IAAI,CAAC,YAAY,EAAE,QAAQ,CAAC,CAAC;QAChD,SAAS,CAAC,UAAU,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;QAC3C,SAAS,CAAC,UAAU,EAAE,EAAE,SAAS,EAAE,IAAI,EAAE,CAAC,CAAC;QAE3C,MAAM,MAAM,GAAG,kBAAkB,CAC/B,WAAW,EACX,YAAY,CAAC;YACX,MAAM,EAAE,UAAU;YAClB,MAAM,EAAE,UAAU;SACnB,CAAC,CACH,CAAC;QAEF,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,UAAU,CAAC,CAAC,CAAC;QACpD,MAAM,CAAC,MAAM,CAAC,UAAU,CAAC,CAAC,IAAI,CAAC,OAAO,CAAC,UAAU,CAAC,CAAC,CAAC;QACpD,MAAM,CAAC,MAAM,CAAC,gBAAgB,CAAC,CAAC,OAAO,CAAC,CAAC,OAAO,CAAC,UAAU,CAAC,CAAC,CAAC,CAAC;IACjE,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,sEAAsE,EAAE,GAAG,EAAE;QAC9E,MAAM,WAAW,GAAG,aAAa,CAAC,kBAAkB,CAAC,CAAC;QAEtD,MAAM,CAAC,GAAG,EAAE,CACV,kBAAkB,CAAC,WAAW,EAAE,YAAY,CAAC,EAAE,MAAM,EAAE,kBAAkB,EAAE,CAAC,CAAC,CAC9E,CAAC,OAAO,CAAC,kCAAkC,CAAC,CAAC;IAChD,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,6DAA6D,EAAE,GAAG,EAAE;QACrE,MAAM,WAAW,GAAG,aAAa,CAAC,mBAAmB,CAAC,CAAC;QAEvD,MAAM,CAAC,GAAG,EAAE,CACV,kBAAkB,CAAC,WAAW,EAAE,YAAY,CAAC,EAAE,MAAM,EAAE,gBAAgB,EAAE,CAAC,CAAC,CAC5E,CAAC,OAAO,CAAC,0BAA0B,CAAC,CAAC;IACxC,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,iEAAiE,EAAE,GAAG,EAAE;QACzE,MAAM,WAAW,GAAG,aAAa,CAAC,gBAAgB,CAAC,CAAC;QACpD,MAAM,QAAQ,GAAG,IAAI,CAAC,WAAW,EAAE,gBAAgB,CAAC,CAAC;QACrD,aAAa,CAAC,QAAQ,EAAE,qBAAqB,CAAC,CAAC;QAE/C,MAAM,CAAC,GAAG,EAAE,CAAC,kBAAkB,CAAC,WAAW,EAAE,YAAY,CAAC,EAAE,MAAM,EAAE,QAAQ,EAAE,CAAC,CAAC,CAAC,CAAC,OAAO,CACvF,iBAAiB,CAClB,CAAC;IACJ,CAAC,CAAC,CAAC;IAEH,EAAE,CAAC,8DAA8D,EAAE,GAAG,EAAE;QACtE,MAAM,WAAW,GAAG,aAAa,CAAC,wBAAwB,CAAC,CAAC;QAE5D,MAAM,CAAC,GAAG,EAAE,CACV,kBAAkB,CAChB,WAAW,EACX,YAAY,CAAC;YACX,MAAM,EAAE,GAAG;SACZ,CAAC,CACH,CACF,CAAC,OAAO,CAAC,wBAAwB,CAAC,CAAC;IACtC,CAAC,CAAC,CAAC;AACL,CAAC,CAAC,CAAC"}