@pennyfarthing/cyclist 11.0.0 → 11.1.1

package/dist/notification-sound.js

@@ -1,219 +0,0 @@
-/**
- * Notification Sound Service (Story MSSCI-14191)
- *
- * Provides audio notifications for workflow events using Web Audio API.
- * Different tones for different event types:
- * - HANDOFF: Low tone (agent transition)
- * - QUESTION: Higher tone (user input needed)
- * - COMPLETION: Chord (workflow complete)
- *
- * State is managed in-memory but can sync with settings via loadNotificationSoundSetting().
- */
-// =============================================================================
-// Types
-// =============================================================================
-/**
- * Notification event types that trigger sounds
- */
-export const NotificationEvent = {
-    HANDOFF: 'handoff', // Agent handoff / phase transition
-    QUESTION: 'question', // User input required
-    COMPLETION: 'completion', // Workflow completion
-};
-/**
- * Frequency configurations for each event type (in Hz)
- */
-const EVENT_FREQUENCIES = {
-    [NotificationEvent.HANDOFF]: [330], // E4 - single low tone
-    [NotificationEvent.QUESTION]: [523], // C5 - higher, attention-grabbing
-    [NotificationEvent.COMPLETION]: [523, 659, 784], // C5-E5-G5 major chord
-};
-// =============================================================================
-// State
-// =============================================================================
-let soundEnabled = false;
-let audioContext = null;
-// For testing: injectable AudioContext constructor
-// eslint-disable-next-line @typescript-eslint/no-explicit-any
-let _audioContextClass = null;
-// =============================================================================
-// Public API
-// =============================================================================
-/**
- * Service class for notification sounds (exported for type checking)
- */
-export class NotificationSound {
-    static isEnabled() {
-        return isNotificationSoundEnabled();
-    }
-    static setEnabled(enabled) {
-        setNotificationSoundEnabled(enabled);
-    }
-    static async play(event) {
-        return playNotificationSound(event);
-    }
-}
-/**
- * Check if notification sounds are enabled
- */
-export function isNotificationSoundEnabled() {
-    return soundEnabled;
-}
-/**
- * Enable or disable notification sounds
- */
-export function setNotificationSoundEnabled(enabled) {
-    soundEnabled = enabled;
-}
-/**
- * Load notification sound setting from REST API
- */
-export async function loadNotificationSoundSetting() {
-    try {
-        const response = await fetch('/api/settings');
-        if (response.ok) {
-            const settings = await response.json();
-            soundEnabled = settings?.notifications?.sound ?? false;
-        }
-    }
-    catch {
-        // Silently fail - keep current state
-    }
-}
-/**
- * Check if audio playback is permitted (handles autoplay restrictions)
- * Returns true if audio can play, false if user gesture is required
- */
-export async function checkAudioPermission() {
-    try {
-        const ctx = getOrCreateAudioContext();
-        if (!ctx)
-            return false;
-        if (ctx.state === 'suspended') {
-            await ctx.resume();
-        }
-        return ctx.state === 'running';
-    }
-    catch {
-        return false;
-    }
-}
-/**
- * Play a notification sound for the given event type
- */
-export async function playNotificationSound(event) {
-    // Exit early if disabled
-    if (!soundEnabled) {
-        return;
-    }
-    try {
-        const ctx = getOrCreateAudioContext();
-        if (!ctx) {
-            return;
-        }
-        // Handle suspended state (autoplay restrictions)
-        if (ctx.state === 'suspended') {
-            await ctx.resume();
-        }
-        const frequencies = EVENT_FREQUENCIES[event] || EVENT_FREQUENCIES[NotificationEvent.HANDOFF];
-        // Play each frequency in the pattern
-        for (let i = 0; i < frequencies.length; i++) {
-            const freq = frequencies[i];
-            const startTime = ctx.currentTime + i * 0.1; // Slight offset for chord/arpeggio effect
-            playTone(ctx, freq, startTime, 0.2);
-        }
-    }
-    catch {
-        // Silently fail - audio is optional
-    }
-}
-// =============================================================================
-// Internal Helpers
-// =============================================================================
-/**
- * Get the AudioContext constructor (allows for mocking in tests)
- * Checks multiple locations to support different environments and test mocking
- */
-function getAudioContextClass() {
-    // Check if a test-injected class is set
-    if (_audioContextClass !== null) {
-        return _audioContextClass;
-    }
-    // Check various locations where AudioContext might be defined
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    const g = globalThis;
-    if (g.AudioContext) {
-        return g.AudioContext;
-    }
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    if (typeof window !== 'undefined' && window.AudioContext) {
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any
-        return window.AudioContext;
-    }
-    // webkitAudioContext for Safari
-    // eslint-disable-next-line @typescript-eslint/no-explicit-any
-    if (typeof window !== 'undefined' && window.webkitAudioContext) {
-        // eslint-disable-next-line @typescript-eslint/no-explicit-any
-        return window.webkitAudioContext;
-    }
-    return null;
-}
-/**
- * Get or create the shared AudioContext
- */
-function getOrCreateAudioContext() {
-    const AudioContextClass = getAudioContextClass();
-    if (!AudioContextClass) {
-        return null;
-    }
-    if (!audioContext) {
-        try {
-            audioContext = new AudioContextClass();
-        }
-        catch {
-            return null;
-        }
-    }
-    return audioContext;
-}
-/**
- * Play a single tone using Web Audio API
- */
-function playTone(ctx, frequency, startTime, duration) {
-    const oscillator = ctx.createOscillator();
-    const gainNode = ctx.createGain();
-    oscillator.connect(gainNode);
-    gainNode.connect(ctx.destination);
-    oscillator.type = 'sine';
-    oscillator.frequency.setValueAtTime(frequency, startTime);
-    // Envelope: quick attack, gradual decay
-    gainNode.gain.setValueAtTime(0.3, startTime);
-    gainNode.gain.exponentialRampToValueAtTime(0.01, startTime + duration);
-    oscillator.start(startTime);
-    oscillator.stop(startTime + duration);
-}
-/**
- * Reset state (for testing)
- */
-export function resetNotificationSoundState() {
-    soundEnabled = false;
-    if (audioContext) {
-        try {
-            audioContext.close().catch(() => { });
-        }
-        catch {
-            // Ignore errors closing mock audio context
-        }
-        audioContext = null;
-    }
-}
-/**
- * Inject a custom AudioContext class (for testing)
- * Pass null to use the default detection
- */
-export function setAudioContextClass(cls) {
-    _audioContextClass = cls;
-    // Reset the existing context so the new class is used
-    audioContext = null;
-}
-//# sourceMappingURL=notification-sound.js.map

package/dist/notification-sound.js.map

@@ -1 +0,0 @@
-{"version":3,"file":"notification-sound.js","sourceRoot":"","sources":["../src/notification-sound.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;GAUG;AAEH,gFAAgF;AAChF,QAAQ;AACR,gFAAgF;AAEhF;;GAEG;AACH,MAAM,CAAC,MAAM,iBAAiB,GAAG;IAC/B,OAAO,EAAE,SAAS,EAAE,mCAAmC;IACvD,QAAQ,EAAE,UAAU,EAAE,sBAAsB;IAC5C,UAAU,EAAE,YAAY,EAAE,sBAAsB;CACxC,CAAC;AAIX;;GAEG;AACH,MAAM,iBAAiB,GAA4C;IACjE,CAAC,iBAAiB,CAAC,OAAO,CAAC,EAAE,CAAC,GAAG,CAAC,EAAE,uBAAuB;IAC3D,CAAC,iBAAiB,CAAC,QAAQ,CAAC,EAAE,CAAC,GAAG,CAAC,EAAE,kCAAkC;IACvE,CAAC,iBAAiB,CAAC,UAAU,CAAC,EAAE,CAAC,GAAG,EAAE,GAAG,EAAE,GAAG,CAAC,EAAE,uBAAuB;CACzE,CAAC;AAEF,gFAAgF;AAChF,QAAQ;AACR,gFAAgF;AAEhF,IAAI,YAAY,GAAG,KAAK,CAAC;AACzB,IAAI,YAAY,GAAwB,IAAI,CAAC;AAE7C,mDAAmD;AACnD,8DAA8D;AAC9D,IAAI,kBAAkB,GAAoC,IAAI,CAAC;AAE/D,gFAAgF;AAChF,aAAa;AACb,gFAAgF;AAEhF;;GAEG;AACH,MAAM,OAAO,iBAAiB;IAC5B,MAAM,CAAC,SAAS;QACd,OAAO,0BAA0B,EAAE,CAAC;IACtC,CAAC;IAED,MAAM,CAAC,UAAU,CAAC,OAAgB;QAChC,2BAA2B,CAAC,OAAO,CAAC,CAAC;IACvC,CAAC;IAED,MAAM,CAAC,KAAK,CAAC,IAAI,CAAC,KAA4B;QAC5C,OAAO,qBAAqB,CAAC,KAAK,CAAC,CAAC;IACtC,CAAC;CACF;AAED;;GAEG;AACH,MAAM,UAAU,0BAA0B;IACxC,OAAO,YAAY,CAAC;AACtB,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,2BAA2B,CAAC,OAAgB;IAC1D,YAAY,GAAG,OAAO,CAAC;AACzB,CAAC;AAED;;GAEG;AACH,MAAM,CAAC,KAAK,UAAU,4BAA4B;IAChD,IAAI,CAAC;QACH,MAAM,QAAQ,GAAG,MAAM,KAAK,CAAC,eAAe,CAAC,CAAC;QAC9C,IAAI,QAAQ,CAAC,EAAE,EAAE,CAAC;YAChB,MAAM,QAAQ,GAAG,MAAM,QAAQ,CAAC,IAAI,EAAE,CAAC;YACvC,YAAY,GAAG,QAAQ,EAAE,aAAa,EAAE,KAAK,IAAI,KAAK,CAAC;QACzD,CAAC;IACH,CAAC;IAAC,MAAM,CAAC;QACP,qCAAqC;IACvC,CAAC;AACH,CAAC;AAED;;;GAGG;AACH,MAAM,CAAC,KAAK,UAAU,oBAAoB;IACxC,IAAI,CAAC;QACH,MAAM,GAAG,GAAG,uBAAuB,EAAE,CAAC;QACtC,IAAI,CAAC,GAAG;YAAE,OAAO,KAAK,CAAC;QAEvB,IAAI,GAAG,CAAC,KAAK,KAAK,WAAW,EAAE,CAAC;YAC9B,MAAM,GAAG,CAAC,MAAM,EAAE,CAAC;QACrB,CAAC;QAED,OAAO,GAAG,CAAC,KAAK,KAAK,SAAS,CAAC;IACjC,CAAC;IAAC,MAAM,CAAC;QACP,OAAO,KAAK,CAAC;IACf,CAAC;AACH,CAAC;AAED;;GAEG;AACH,MAAM,CAAC,KAAK,UAAU,qBAAqB,CAAC,KAA4B;IACtE,yBAAyB;IACzB,IAAI,CAAC,YAAY,EAAE,CAAC;QAClB,OAAO;IACT,CAAC;IAED,IAAI,CAAC;QACH,MAAM,GAAG,GAAG,uBAAuB,EAAE,CAAC;QACtC,IAAI,CAAC,GAAG,EAAE,CAAC;YACT,OAAO;QACT,CAAC;QAED,iDAAiD;QACjD,IAAI,GAAG,CAAC,KAAK,KAAK,WAAW,EAAE,CAAC;YAC9B,MAAM,GAAG,CAAC,MAAM,EAAE,CAAC;QACrB,CAAC;QAED,MAAM,WAAW,GAAG,iBAAiB,CAAC,KAAK,CAAC,IAAI,iBAAiB,CAAC,iBAAiB,CAAC,OAAO,CAAC,CAAC;QAE7F,qCAAqC;QACrC,KAAK,IAAI,CAAC,GAAG,CAAC,EAAE,CAAC,GAAG,WAAW,CAAC,MAAM,EAAE,CAAC,EAAE,EAAE,CAAC;YAC5C,MAAM,IAAI,GAAG,WAAW,CAAC,CAAC,CAAC,CAAC;YAC5B,MAAM,SAAS,GAAG,GAAG,CAAC,WAAW,GAAG,CAAC,GAAG,GAAG,CAAC,CAAC,0CAA0C;YACvF,QAAQ,CAAC,GAAG,EAAE,IAAI,EAAE,SAAS,EAAE,GAAG,CAAC,CAAC;QACtC,CAAC;IACH,CAAC;IAAC,MAAM,CAAC;QACP,oCAAoC;IACtC,CAAC;AACH,CAAC;AAED,gFAAgF;AAChF,mBAAmB;AACnB,gFAAgF;AAEhF;;;GAGG;AACH,SAAS,oBAAoB;IAC3B,wCAAwC;IACxC,IAAI,kBAAkB,KAAK,IAAI,EAAE,CAAC;QAChC,OAAO,kBAAkB,CAAC;IAC5B,CAAC;IAED,8DAA8D;IAC9D,8DAA8D;IAC9D,MAAM,CAAC,GAAG,UAAiB,CAAC;IAC5B,IAAI,CAAC,CAAC,YAAY,EAAE,CAAC;QACnB,OAAO,CAAC,CAAC,YAAY,CAAC;IACxB,CAAC;IACD,8DAA8D;IAC9D,IAAI,OAAO,MAAM,KAAK,WAAW,IAAK,MAAc,CAAC,YAAY,EAAE,CAAC;QAClE,8DAA8D;QAC9D,OAAQ,MAAc,CAAC,YAAY,CAAC;IACtC,CAAC;IACD,gCAAgC;IAChC,8DAA8D;IAC9D,IAAI,OAAO,MAAM,KAAK,WAAW,IAAK,MAAc,CAAC,kBAAkB,EAAE,CAAC;QACxE,8DAA8D;QAC9D,OAAQ,MAAc,CAAC,kBAAkB,CAAC;IAC5C,CAAC;IACD,OAAO,IAAI,CAAC;AACd,CAAC;AAED;;GAEG;AACH,SAAS,uBAAuB;IAC9B,MAAM,iBAAiB,GAAG,oBAAoB,EAAE,CAAC;IACjD,IAAI,CAAC,iBAAiB,EAAE,CAAC;QACvB,OAAO,IAAI,CAAC;IACd,CAAC;IAED,IAAI,CAAC,YAAY,EAAE,CAAC;QAClB,IAAI,CAAC;YACH,YAAY,GAAG,IAAI,iBAAiB,EAAE,CAAC;QACzC,CAAC;QAAC,MAAM,CAAC;YACP,OAAO,IAAI,CAAC;QACd,CAAC;IACH,CAAC;IAED,OAAO,YAAY,CAAC;AACtB,CAAC;AAED;;GAEG;AACH,SAAS,QAAQ,CAAC,GAAiB,EAAE,SAAiB,EAAE,SAAiB,EAAE,QAAgB;IACzF,MAAM,UAAU,GAAG,GAAG,CAAC,gBAAgB,EAAE,CAAC;IAC1C,MAAM,QAAQ,GAAG,GAAG,CAAC,UAAU,EAAE,CAAC;IAElC,UAAU,CAAC,OAAO,CAAC,QAAQ,CAAC,CAAC;IAC7B,QAAQ,CAAC,OAAO,CAAC,GAAG,CAAC,WAAW,CAAC,CAAC;IAElC,UAAU,CAAC,IAAI,GAAG,MAAM,CAAC;IACzB,UAAU,CAAC,SAAS,CAAC,cAAc,CAAC,SAAS,EAAE,SAAS,CAAC,CAAC;IAE1D,wCAAwC;IACxC,QAAQ,CAAC,IAAI,CAAC,cAAc,CAAC,GAAG,EAAE,SAAS,CAAC,CAAC;IAC7C,QAAQ,CAAC,IAAI,CAAC,4BAA4B,CAAC,IAAI,EAAE,SAAS,GAAG,QAAQ,CAAC,CAAC;IAEvE,UAAU,CAAC,KAAK,CAAC,SAAS,CAAC,CAAC;IAC5B,UAAU,CAAC,IAAI,CAAC,SAAS,GAAG,QAAQ,CAAC,CAAC;AACxC,CAAC;AAED;;GAEG;AACH,MAAM,UAAU,2BAA2B;IACzC,YAAY,GAAG,KAAK,CAAC;IACrB,IAAI,YAAY,EAAE,CAAC;QACjB,IAAI,CAAC;YACH,YAAY,CAAC,KAAK,EAAE,CAAC,KAAK,CAAC,GAAG,EAAE,GAAE,CAAC,CAAC,CAAC;QACvC,CAAC;QAAC,MAAM,CAAC;YACP,2CAA2C;QAC7C,CAAC;QACD,YAAY,GAAG,IAAI,CAAC;IACtB,CAAC;AACH,CAAC;AAED;;;GAGG;AACH,MAAM,UAAU,oBAAoB,CAAC,GAAoC;IACvE,kBAAkB,GAAG,GAAG,CAAC;IACzB,sDAAsD;IACtD,YAAY,GAAG,IAAI,CAAC;AACtB,CAAC"}

package/dist/plugin-loader.test.d.ts

@@ -1,17 +0,0 @@
-/**
- * Tests for Story 93-6: Plugin Router Loader for Cyclist
- *
- * Tests the dynamic discovery and mounting of API routers from installed
- * @pennyfarthing/* plugin packages. The loader uses the plugin discovery
- * system (93-3) to find plugins and mount their Express routers.
- *
- * Test categories:
- * 1. initPluginRouters() - Core loading behavior
- * 2. Plugin with API router - Benchmark plugin integration
- * 3. Graceful degradation - Missing packages, failed imports
- * 4. Cyclist starts without plugins - Empty state
- *
- * Run with: cd packages/cyclist && pnpm test
- */
-export {};
-//# sourceMappingURL=plugin-loader.test.d.ts.map

package/dist/plugin-loader.test.d.ts.map

@@ -1 +0,0 @@
-{"version":3,"file":"plugin-loader.test.d.ts","sourceRoot":"","sources":["../src/plugin-loader.test.ts"],"names":[],"mappings":"AAAA;;;;;;;;;;;;;;GAcG"}

package/dist/plugin-loader.test.js

@@ -1,407 +0,0 @@
-/**
- * Tests for Story 93-6: Plugin Router Loader for Cyclist
- *
- * Tests the dynamic discovery and mounting of API routers from installed
- * @pennyfarthing/* plugin packages. The loader uses the plugin discovery
- * system (93-3) to find plugins and mount their Express routers.
- *
- * Test categories:
- * 1. initPluginRouters() - Core loading behavior
- * 2. Plugin with API router - Benchmark plugin integration
- * 3. Graceful degradation - Missing packages, failed imports
- * 4. Cyclist starts without plugins - Empty state
- *
- * Run with: cd packages/cyclist && pnpm test
- */
-import { describe, it, expect, vi, beforeEach } from 'vitest';
-import express from 'express';
-import { initPluginRouters } from './plugin-loader.js';
-// Mock @pennyfarthing/core plugin discovery
-vi.mock('@pennyfarthing/core', async () => {
-    const actual = await vi.importActual('@pennyfarthing/core');
-    return {
-        ...actual,
-        discoverPlugins: vi.fn(),
-        getPluginRouters: vi.fn(),
-    };
-});
-// Get mocked functions for test control
-import { discoverPlugins, getPluginRouters } from '@pennyfarthing/core';
-const mockDiscoverPlugins = vi.mocked(discoverPlugins);
-const mockGetPluginRouters = vi.mocked(getPluginRouters);
-// Helper: create a fake Express Router
-function createFakeRouter() {
-    const router = express.Router();
-    router.get('/test', (_req, res) => res.json({ ok: true }));
-    return router;
-}
-describe('initPluginRouters()', () => {
-    let app;
-    beforeEach(() => {
-        app = express();
-        vi.clearAllMocks();
-    });
-    it('should return PluginLoadResult with correct shape', async () => {
-        mockDiscoverPlugins.mockReturnValue([]);
-        mockGetPluginRouters.mockReturnValue([]);
-        const result = await initPluginRouters(app, '/fake/project');
-        expect(result).toHaveProperty('discovered');
-        expect(result).toHaveProperty('loaded');
-        expect(result).toHaveProperty('failed');
-        expect(result).toHaveProperty('routers');
-        expect(typeof result.discovered).toBe('number');
-        expect(typeof result.loaded).toBe('number');
-        expect(typeof result.failed).toBe('number');
-        expect(Array.isArray(result.routers)).toBe(true);
-    });
-    it('should report zero when no plugins are discovered', async () => {
-        mockDiscoverPlugins.mockReturnValue([]);
-        mockGetPluginRouters.mockReturnValue([]);
-        const result = await initPluginRouters(app, '/fake/project');
-        expect(result.discovered).toBe(0);
-        expect(result.loaded).toBe(0);
-        expect(result.failed).toBe(0);
-        expect(result.routers).toHaveLength(0);
-    });
-    it('should call discoverPlugins with the project root', async () => {
-        mockDiscoverPlugins.mockReturnValue([]);
-        mockGetPluginRouters.mockReturnValue([]);
-        await initPluginRouters(app, '/my/project');
-        expect(mockDiscoverPlugins).toHaveBeenCalledWith('/my/project');
-    });
-    it('should pass discovered plugins to getPluginRouters', async () => {
-        const fakePlugins = [
-            {
-                name: '@pennyfarthing/benchmark',
-                path: '/node_modules/@pennyfarthing/benchmark',
-                manifest: { commands: 'commands/', skills: 'skills/' },
-            },
-        ];
-        mockDiscoverPlugins.mockReturnValue(fakePlugins);
-        mockGetPluginRouters.mockReturnValue([]);
-        await initPluginRouters(app, '/fake/project');
-        expect(mockGetPluginRouters).toHaveBeenCalledWith(fakePlugins);
-    });
-    it('should report discovered count matching number of plugins', async () => {
-        const fakePlugins = [
-            {
-                name: '@pennyfarthing/benchmark',
-                path: '/node_modules/@pennyfarthing/benchmark',
-                manifest: { commands: 'commands/' },
-            },
-            {
-                name: '@pennyfarthing/analytics',
-                path: '/node_modules/@pennyfarthing/analytics',
-                manifest: { commands: 'commands/' },
-            },
-        ];
-        mockDiscoverPlugins.mockReturnValue(fakePlugins);
-        mockGetPluginRouters.mockReturnValue([]);
-        const result = await initPluginRouters(app, '/fake/project');
-        expect(result.discovered).toBe(2);
-    });
-});
-describe('Plugin with API router', () => {
-    let app;
-    beforeEach(() => {
-        app = express();
-        vi.clearAllMocks();
-    });
-    it('should dynamically import and mount a plugin router', async () => {
-        const fakePlugins = [
-            {
-                name: '@pennyfarthing/benchmark',
-                path: '/node_modules/@pennyfarthing/benchmark',
-                manifest: {
-                    commands: 'commands/',
-                    api: {
-                        path: '/api/benchmark',
-                        module: './dist/api/benchmark.js',
-                        export: 'createBenchmarkRouter',
-                    },
-                },
-            },
-        ];
-        const fakeRouters = [
-            {
-                mountPath: '/api/benchmark',
-                modulePath: '/node_modules/@pennyfarthing/benchmark/dist/api/benchmark.js',
-                exportName: 'createBenchmarkRouter',
-                plugin: '@pennyfarthing/benchmark',
-            },
-        ];
-        mockDiscoverPlugins.mockReturnValue(fakePlugins);
-        mockGetPluginRouters.mockReturnValue(fakeRouters);
-        // Mock the dynamic import to return a module with the router factory
-        const fakeRouter = createFakeRouter();
-        vi.stubGlobal('import', undefined); // Clear any previous stubs
-        // We need to mock the dynamic import that initPluginRouters will use
-        // The implementation should use: const mod = await import(modulePath)
-        // We mock it via vi.mock for the specific module path
-        const originalImport = globalThis.import;
-        // @ts-expect-error - mocking dynamic import
-        globalThis.__pluginImportMock = vi.fn().mockResolvedValue({
-            createBenchmarkRouter: () => fakeRouter,
-        });
-        const result = await initPluginRouters(app, '/fake/project');
-        expect(result.loaded).toBe(1);
-        expect(result.failed).toBe(0);
-        expect(result.routers).toHaveLength(1);
-        expect(result.routers[0]).toEqual(expect.objectContaining({
-            mountPath: '/api/benchmark',
-            plugin: '@pennyfarthing/benchmark',
-            success: true,
-        }));
-    });
-    it('should mount router at the declared mount path', async () => {
-        const fakePlugins = [
-            {
-                name: '@pennyfarthing/benchmark',
-                path: '/node_modules/@pennyfarthing/benchmark',
-                manifest: {
-                    api: {
-                        path: '/api/benchmark',
-                        module: './dist/api/benchmark.js',
-                        export: 'createBenchmarkRouter',
-                    },
-                },
-            },
-        ];
-        const fakeRouters = [
-            {
-                mountPath: '/api/benchmark',
-                modulePath: '/node_modules/@pennyfarthing/benchmark/dist/api/benchmark.js',
-                exportName: 'createBenchmarkRouter',
-                plugin: '@pennyfarthing/benchmark',
-            },
-        ];
-        mockDiscoverPlugins.mockReturnValue(fakePlugins);
-        mockGetPluginRouters.mockReturnValue(fakeRouters);
-        // Spy on app.use to verify the mount path
-        const useSpy = vi.spyOn(app, 'use');
-        const result = await initPluginRouters(app, '/fake/project');
-        // Verify app.use was called with the correct mount path
-        if (result.loaded > 0) {
-            const mountCalls = useSpy.mock.calls.filter((call) => call[0] === '/api/benchmark');
-            expect(mountCalls.length).toBeGreaterThan(0);
-        }
-    });
-    it('should pass getProjectDir to router factory if it accepts arguments', async () => {
-        // Some routers take a getProjectDir function as argument
-        // The plugin loader should detect this and pass it
-        const fakePlugins = [
-            {
-                name: '@pennyfarthing/benchmark',
-                path: '/node_modules/@pennyfarthing/benchmark',
-                manifest: {
-                    api: {
-                        path: '/api/benchmark',
-                        module: './dist/api/benchmark.js',
-                        export: 'createBenchmarkRouter',
-                    },
-                },
-            },
-        ];
-        const fakeRouters = [
-            {
-                mountPath: '/api/benchmark',
-                modulePath: '/node_modules/@pennyfarthing/benchmark/dist/api/benchmark.js',
-                exportName: 'createBenchmarkRouter',
-                plugin: '@pennyfarthing/benchmark',
-            },
-        ];
-        mockDiscoverPlugins.mockReturnValue(fakePlugins);
-        mockGetPluginRouters.mockReturnValue(fakeRouters);
-        // The result should include the mounted router
-        const result = await initPluginRouters(app, '/fake/project');
-        // If loaded successfully, the factory was called
-        expect(result).toBeDefined();
-    });
-});
-describe('Graceful degradation', () => {
-    let app;
-    beforeEach(() => {
-        app = express();
-        vi.clearAllMocks();
-    });
-    it('should handle failed dynamic import gracefully', async () => {
-        const fakePlugins = [
-            {
-                name: '@pennyfarthing/broken',
-                path: '/node_modules/@pennyfarthing/broken',
-                manifest: {
-                    api: {
-                        path: '/api/broken',
-                        module: './dist/api/broken.js',
-                        export: 'createBrokenRouter',
-                    },
-                },
-            },
-        ];
-        const fakeRouters = [
-            {
-                mountPath: '/api/broken',
-                modulePath: '/node_modules/@pennyfarthing/broken/dist/api/broken.js',
-                exportName: 'createBrokenRouter',
-                plugin: '@pennyfarthing/broken',
-            },
-        ];
-        mockDiscoverPlugins.mockReturnValue(fakePlugins);
-        mockGetPluginRouters.mockReturnValue(fakeRouters);
-        // The module path doesn't exist, so dynamic import will fail
-        const result = await initPluginRouters(app, '/fake/project');
-        // Should NOT throw — graceful degradation
-        expect(result.loaded).toBe(0);
-        expect(result.failed).toBe(1);
-        expect(result.routers[0]).toEqual(expect.objectContaining({
-            mountPath: '/api/broken',
-            plugin: '@pennyfarthing/broken',
-            success: false,
-        }));
-        expect(result.routers[0].error).toBeDefined();
-    });
-    it('should handle missing export name in module gracefully', async () => {
-        const fakePlugins = [
-            {
-                name: '@pennyfarthing/noexport',
-                path: '/node_modules/@pennyfarthing/noexport',
-                manifest: {
-                    api: {
-                        path: '/api/noexport',
-                        module: './dist/api/noexport.js',
-                        export: 'createNonexistentRouter',
-                    },
-                },
-            },
-        ];
-        const fakeRouters = [
-            {
-                mountPath: '/api/noexport',
-                modulePath: '/node_modules/@pennyfarthing/noexport/dist/api/noexport.js',
-                exportName: 'createNonexistentRouter',
-                plugin: '@pennyfarthing/noexport',
-            },
-        ];
-        mockDiscoverPlugins.mockReturnValue(fakePlugins);
-        mockGetPluginRouters.mockReturnValue(fakeRouters);
-        const result = await initPluginRouters(app, '/fake/project');
-        // Should fail gracefully when the export doesn't exist
-        expect(result.failed).toBeGreaterThanOrEqual(1);
-        const routerResult = result.routers.find((r) => r.plugin === '@pennyfarthing/noexport');
-        expect(routerResult?.success).toBe(false);
-    });
-    it('should continue loading other plugins when one fails', async () => {
-        const fakePlugins = [
-            {
-                name: '@pennyfarthing/broken',
-                path: '/node_modules/@pennyfarthing/broken',
-                manifest: {
-                    api: {
-                        path: '/api/broken',
-                        module: './dist/api/broken.js',
-                        export: 'createBrokenRouter',
-                    },
-                },
-            },
-            {
-                name: '@pennyfarthing/good',
-                path: '/node_modules/@pennyfarthing/good',
-                manifest: {
-                    api: {
-                        path: '/api/good',
-                        module: './dist/api/good.js',
-                        export: 'createGoodRouter',
-                    },
-                },
-            },
-        ];
-        const fakeRouters = [
-            {
-                mountPath: '/api/broken',
-                modulePath: '/node_modules/@pennyfarthing/broken/dist/api/broken.js',
-                exportName: 'createBrokenRouter',
-                plugin: '@pennyfarthing/broken',
-            },
-            {
-                mountPath: '/api/good',
-                modulePath: '/node_modules/@pennyfarthing/good/dist/api/good.js',
-                exportName: 'createGoodRouter',
-                plugin: '@pennyfarthing/good',
-            },
-        ];
-        mockDiscoverPlugins.mockReturnValue(fakePlugins);
-        mockGetPluginRouters.mockReturnValue(fakeRouters);
-        const result = await initPluginRouters(app, '/fake/project');
-        // Both routers should be attempted
-        expect(result.routers).toHaveLength(2);
-        // At least the broken one should fail
-        const brokenResult = result.routers.find((r) => r.plugin === '@pennyfarthing/broken');
-        expect(brokenResult?.success).toBe(false);
-    });
-    it('should never throw from initPluginRouters', async () => {
-        // Even if discoverPlugins itself throws, we should catch it
-        mockDiscoverPlugins.mockImplementation(() => {
-            throw new Error('Catastrophic failure');
-        });
-        const result = await initPluginRouters(app, '/fake/project');
-        expect(result.discovered).toBe(0);
-        expect(result.loaded).toBe(0);
-        expect(result.failed).toBe(0);
-    });
-});
-describe('Cyclist starts without plugins', () => {
-    let app;
-    beforeEach(() => {
-        app = express();
-        vi.clearAllMocks();
-    });
-    it('should return clean result when no @pennyfarthing packages exist', async () => {
-        mockDiscoverPlugins.mockReturnValue([]);
-        mockGetPluginRouters.mockReturnValue([]);
-        const result = await initPluginRouters(app, '/fake/project');
-        expect(result.discovered).toBe(0);
-        expect(result.loaded).toBe(0);
-        expect(result.failed).toBe(0);
-        expect(result.routers).toEqual([]);
-    });
-    it('should not modify the Express app when no plugins have routers', async () => {
-        const fakePlugins = [
-            {
-                name: '@pennyfarthing/benchmark',
-                path: '/node_modules/@pennyfarthing/benchmark',
-                manifest: { commands: 'commands/', skills: 'skills/' },
-                // Note: no api field — plugin has commands/skills but no router
-            },
-        ];
-        mockDiscoverPlugins.mockReturnValue(fakePlugins);
-        mockGetPluginRouters.mockReturnValue([]); // No routers declared
-        const useSpy = vi.spyOn(app, 'use');
-        const routesBefore = app._router?.stack?.length ?? 0;
-        const result = await initPluginRouters(app, '/fake/project');
-        // Plugin was discovered but has no router
-        expect(result.discovered).toBe(1);
-        expect(result.loaded).toBe(0);
-        // app.use should not have been called for plugin routers
-        const pluginMountCalls = useSpy.mock.calls.filter((call) => typeof call[0] === 'string' && call[0].startsWith('/api/'));
-        expect(pluginMountCalls).toHaveLength(0);
-    });
-    it('should log discovered plugins for observability', async () => {
-        const consoleSpy = vi.spyOn(console, 'log').mockImplementation(() => { });
-        mockDiscoverPlugins.mockReturnValue([
-            {
-                name: '@pennyfarthing/benchmark',
-                path: '/node_modules/@pennyfarthing/benchmark',
-                manifest: { commands: 'commands/' },
-            },
-        ]);
-        mockGetPluginRouters.mockReturnValue([]);
-        await initPluginRouters(app, '/fake/project');
-        // Should log something about plugin discovery
-        expect(consoleSpy).toHaveBeenCalled();
-        const logMessages = consoleSpy.mock.calls.map((call) => call.join(' '));
-        const hasPluginLog = logMessages.some((msg) => msg.includes('plugin') || msg.includes('Plugin'));
-        expect(hasPluginLog).toBe(true);
-        consoleSpy.mockRestore();
-    });
-});
-//# sourceMappingURL=plugin-loader.test.js.map