@objectstack/runtime 4.0.3 → 4.0.5

package/src/http-dispatcher.test.ts

@@ -1,1312 +0,0 @@
-
-import { describe, it, expect, vi, beforeEach } from 'vitest';
-import { HttpDispatcher } from './http-dispatcher.js';
-import { ObjectKernel } from '@objectstack/core';
-
-describe('HttpDispatcher', () => {
-    let kernel: ObjectKernel;
-    let dispatcher: HttpDispatcher;
-    let mockProtocol: any;
-    let mockBroker: any;
-
-    beforeEach(() => {
-        // Mock Kernel
-        mockProtocol = {
-            saveMetaItem: vi.fn().mockResolvedValue({ success: true, message: 'Saved' }),
-            getMetaItem: vi.fn().mockResolvedValue({ success: true, item: { foo: 'bar' } })
-        };
-        
-        mockBroker = {
-            call: vi.fn(),
-        };
-
-        kernel = {
-            broker: mockBroker,
-            context: {
-                getService: (name: string) => {
-                    if (name === 'protocol') return mockProtocol;
-                    return null;
-                }
-            }
-        } as any;
-
-        dispatcher = new HttpDispatcher(kernel);
-    });
-
-    describe('handleMetadata', () => {
-        it('should handle PUT /metadata/:type/:name by calling protocol.saveMetaItem', async () => {
-            const context = { request: {} };
-            const body = { label: 'New Label' };
-            const path = '/objects/my_obj';
-
-            const result = await dispatcher.handleMetadata(path, context, 'PUT', body);
-
-            expect(result.handled).toBe(true);
-            expect(result.response?.status).toBe(200);
-            expect(mockProtocol.saveMetaItem).toHaveBeenCalledWith({
-                type: 'objects',
-                name: 'my_obj',
-                item: body
-            });
-            expect(result.response?.body).toEqual({
-                success: true,
-                data: { success: true, message: 'Saved' },
-                meta: undefined
-            });
-        });
-
-        it('should fallback to broker call if protocol is missing saveMetaItem', async () => {
-             // Mock protocol without saveMetaItem
-            (kernel as any).context.getService = () => ({}); 
-            // Mock broker success
-            mockBroker.call.mockResolvedValue({ success: true, fromBroker: true });
-
-            const context = { request: {} };
-            const body = { label: 'Fallback' };
-            const path = '/objects/my_obj';
-
-            const result = await dispatcher.handleMetadata(path, context, 'PUT', body);
-
-            expect(result.handled).toBe(true);
-            expect(mockBroker.call).toHaveBeenCalledWith(
-                'metadata.saveItem',
-                { type: 'objects', name: 'my_obj', item: body },
-                { request: context.request }
-            );
-            expect(result.response?.body?.data).toEqual({ success: true, fromBroker: true });
-        });
-
-        it('should return error if save fails', async () => {
-            mockProtocol.saveMetaItem.mockRejectedValue(new Error('Save failed'));
-
-            const context = { request: {} };
-            const body = {};
-            const path = '/objects/bad_obj';
-
-            const result = await dispatcher.handleMetadata(path, context, 'PUT', body);
-
-            expect(result.handled).toBe(true);
-            expect(result.response?.status).toBe(400);
-            expect(result.response?.body?.error?.message).toBe('Save failed');
-        });
-        
-        it('should handle READ operations as before', async () => {
-             mockBroker.call.mockResolvedValue({ name: 'my_obj' });
-             
-             const context = { request: {} };
-             const result = await dispatcher.handleMetadata('/objects/my_obj', context, 'GET');
-             
-             expect(result.handled).toBe(true);
-             expect(mockBroker.call).toHaveBeenCalledWith(
-                 'metadata.getObject', 
-                 { objectName: 'my_obj' }, 
-                 { request: context.request }
-             );
-        });
-    });
-
-    describe('handleAutomation', () => {
-        let mockAutomationService: any;
-
-        beforeEach(() => {
-            mockAutomationService = {
-                listFlows: vi.fn().mockResolvedValue(['flow_a', 'flow_b']),
-                getFlow: vi.fn().mockResolvedValue({ name: 'flow_a', label: 'Flow A' }),
-                registerFlow: vi.fn(),
-                unregisterFlow: vi.fn(),
-                execute: vi.fn().mockResolvedValue({ success: true, output: {} }),
-                toggleFlow: vi.fn().mockResolvedValue(undefined),
-                listRuns: vi.fn().mockResolvedValue([{ id: 'run_1', status: 'completed' }]),
-                getRun: vi.fn().mockResolvedValue({ id: 'run_1', status: 'completed' }),
-                trigger: vi.fn().mockResolvedValue({ success: true }),
-            };
-
-            // Set up kernel services to include automation
-            (kernel as any).services = new Map([
-                ['automation', mockAutomationService],
-            ]);
-        });
-
-        it('should list flows via GET /', async () => {
-            const result = await dispatcher.handleAutomation('', 'GET', {}, { request: {} });
-            expect(result.handled).toBe(true);
-            expect(result.response?.body?.data?.flows).toEqual(['flow_a', 'flow_b']);
-        });
-
-        it('should get a flow via GET /:name', async () => {
-            const result = await dispatcher.handleAutomation('flow_a', 'GET', {}, { request: {} });
-            expect(result.handled).toBe(true);
-            expect(result.response?.body?.data?.name).toBe('flow_a');
-        });
-
-        it('should return 404 for non-existent flow via GET /:name', async () => {
-            mockAutomationService.getFlow.mockResolvedValue(null);
-            const result = await dispatcher.handleAutomation('missing', 'GET', {}, { request: {} });
-            expect(result.handled).toBe(true);
-            expect(result.response?.status).toBe(404);
-        });
-
-        it('should create a flow via POST /', async () => {
-            const body = { name: 'new_flow', label: 'New Flow' };
-            const result = await dispatcher.handleAutomation('', 'POST', body, { request: {} });
-            expect(result.handled).toBe(true);
-            expect(mockAutomationService.registerFlow).toHaveBeenCalledWith('new_flow', body);
-        });
-
-        it('should update a flow via PUT /:name', async () => {
-            const body = { definition: { label: 'Updated' } };
-            const result = await dispatcher.handleAutomation('flow_a', 'PUT', body, { request: {} });
-            expect(result.handled).toBe(true);
-            expect(mockAutomationService.registerFlow).toHaveBeenCalledWith('flow_a', { label: 'Updated' });
-        });
-
-        it('should delete a flow via DELETE /:name', async () => {
-            const result = await dispatcher.handleAutomation('flow_a', 'DELETE', {}, { request: {} });
-            expect(result.handled).toBe(true);
-            expect(mockAutomationService.unregisterFlow).toHaveBeenCalledWith('flow_a');
-            expect(result.response?.body?.data?.deleted).toBe(true);
-        });
-
-        it('should trigger a flow via POST /:name/trigger', async () => {
-            const result = await dispatcher.handleAutomation('flow_a/trigger', 'POST', { key: 'val' }, { request: {} });
-            expect(result.handled).toBe(true);
-            expect(mockAutomationService.execute).toHaveBeenCalledWith('flow_a', { key: 'val' });
-        });
-
-        it('should toggle a flow via POST /:name/toggle', async () => {
-            const result = await dispatcher.handleAutomation('flow_a/toggle', 'POST', { enabled: false }, { request: {} });
-            expect(result.handled).toBe(true);
-            expect(mockAutomationService.toggleFlow).toHaveBeenCalledWith('flow_a', false);
-        });
-
-        it('should list runs via GET /:name/runs', async () => {
-            const result = await dispatcher.handleAutomation('flow_a/runs', 'GET', {}, { request: {} });
-            expect(result.handled).toBe(true);
-            expect(result.response?.body?.data?.runs).toHaveLength(1);
-        });
-
-        it('should get a run via GET /:name/runs/:runId', async () => {
-            const result = await dispatcher.handleAutomation('flow_a/runs/run_1', 'GET', {}, { request: {} });
-            expect(result.handled).toBe(true);
-            expect(result.response?.body?.data?.id).toBe('run_1');
-        });
-
-        it('should return 404 for non-existent run', async () => {
-            mockAutomationService.getRun.mockResolvedValue(null);
-            const result = await dispatcher.handleAutomation('flow_a/runs/missing', 'GET', {}, { request: {} });
-            expect(result.handled).toBe(true);
-            expect(result.response?.status).toBe(404);
-        });
-
-        it('should handle legacy trigger path POST /trigger/:name', async () => {
-            const result = await dispatcher.handleAutomation('trigger/flow_a', 'POST', { data: 1 }, { request: {} });
-            expect(result.handled).toBe(true);
-            expect(mockAutomationService.trigger).toHaveBeenCalledWith('flow_a', { data: 1 }, { request: {} });
-        });
-    });
-
-    // ═══════════════════════════════════════════════════════════════
-    // Async Service Resolution Tests
-    // Covers: getService awaits Promise-based (async factory) services
-    // ═══════════════════════════════════════════════════════════════
-
-    describe('Async service resolution (Promise-based injection)', () => {
-
-        describe('handleAnalytics with async service', () => {
-            it('should resolve analytics service from Promise (async factory)', async () => {
-                const mockAnalytics = {
-                    query: vi.fn().mockResolvedValue({ rows: [{ id: 1 }], total: 1 }),
-                    getMeta: vi.fn().mockResolvedValue({ tables: ['t1'] }),
-                    generateSql: vi.fn().mockResolvedValue({ sql: 'SELECT 1' }),
-                };
-                // Inject as Promise (simulates async factory registration)
-                (kernel as any).getService = vi.fn().mockImplementation((name: string) => {
-                    if (name === 'analytics') return Promise.resolve(mockAnalytics);
-                    return null;
-                });
-
-                const result = await dispatcher.handleAnalytics('query', 'POST', { sql: 'SELECT 1' }, { request: {} });
-                expect(result.handled).toBe(true);
-                expect(result.response?.status).toBe(200);
-                expect(mockAnalytics.query).toHaveBeenCalled();
-            });
-
-            it('should handle POST /analytics/sql with async service', async () => {
-                const mockAnalytics = {
-                    generateSql: vi.fn().mockResolvedValue({ sql: 'SELECT * FROM t' }),
-                };
-                (kernel as any).getService = vi.fn().mockResolvedValue(mockAnalytics);
-
-                const result = await dispatcher.handleAnalytics('sql', 'POST', { object: 'test' }, { request: {} });
-                expect(result.handled).toBe(true);
-                expect(result.response?.status).toBe(200);
-                expect(mockAnalytics.generateSql).toHaveBeenCalled();
-            });
-
-            it('should handle GET /analytics/meta with async service', async () => {
-                const mockAnalytics = {
-                    getMeta: vi.fn().mockResolvedValue({ tables: ['users', 'orders'] }),
-                };
-                (kernel as any).getService = vi.fn().mockResolvedValue(mockAnalytics);
-
-                const result = await dispatcher.handleAnalytics('meta', 'GET', {}, { request: {} });
-                expect(result.handled).toBe(true);
-                expect(result.response?.status).toBe(200);
-                expect(result.response?.body?.data?.tables).toEqual(['users', 'orders']);
-            });
-
-            it('should return unhandled when analytics service is not registered', async () => {
-                (kernel as any).getService = vi.fn().mockResolvedValue(null);
-                (kernel as any).services = new Map();
-
-                const result = await dispatcher.handleAnalytics('query', 'POST', {}, { request: {} });
-                expect(result.handled).toBe(false);
-            });
-
-            it('should return unhandled for unknown analytics sub-path', async () => {
-                const mockAnalytics = { query: vi.fn() };
-                (kernel as any).getService = vi.fn().mockResolvedValue(mockAnalytics);
-
-                const result = await dispatcher.handleAnalytics('unknown', 'POST', {}, { request: {} });
-                expect(result.handled).toBe(false);
-            });
-        });
-
-        describe('handleAuth with async service', () => {
-            it('should resolve auth service from Promise', async () => {
-                const mockAuth = {
-                    handler: vi.fn().mockResolvedValue({ user: { id: '1' } }),
-                };
-                (kernel as any).getService = vi.fn().mockImplementation((name: string) => {
-                    if (name === 'auth') return Promise.resolve(mockAuth);
-                    return null;
-                });
-
-                const result = await dispatcher.handleAuth('', 'POST', {}, { request: {}, response: {} });
-                expect(result.handled).toBe(true);
-                expect(mockAuth.handler).toHaveBeenCalled();
-            });
-
-            it('should fallback to legacy login when async auth service has no handler', async () => {
-                (kernel as any).getService = vi.fn().mockResolvedValue({});
-                mockBroker.call.mockResolvedValue({ token: 'abc' });
-
-                const result = await dispatcher.handleAuth('/login', 'POST', { user: 'a' }, { request: {} });
-                expect(result.handled).toBe(true);
-                expect(mockBroker.call).toHaveBeenCalledWith('auth.login', { user: 'a' }, { request: {} });
-            });
-
-            it('should return unhandled when auth service not registered and no legacy match', async () => {
-                (kernel as any).getService = vi.fn().mockResolvedValue(null);
-                (kernel as any).services = new Map();
-
-                const result = await dispatcher.handleAuth('/profile', 'GET', {}, { request: {} });
-                expect(result.handled).toBe(false);
-            });
-        });
-
-        describe('handleAuth mock fallback (MSW/test mode)', () => {
-            beforeEach(() => {
-                // No auth service, no broker — simulates MSW/mock mode
-                (kernel as any).getService = vi.fn().mockResolvedValue(null);
-                (kernel as any).services = new Map();
-                (kernel as any).broker = null;
-            });
-
-            it('should mock sign-up/email endpoint', async () => {
-                const result = await dispatcher.handleAuth('/sign-up/email', 'POST', { email: 'test@example.com', name: 'Test' }, { request: {} });
-                expect(result.handled).toBe(true);
-                expect(result.response?.status).toBe(200);
-                expect(result.response?.body.user).toBeDefined();
-                expect(result.response?.body.user.email).toBe('test@example.com');
-                expect(result.response?.body.session).toBeDefined();
-            });
-
-            it('should mock sign-in/email endpoint', async () => {
-                const result = await dispatcher.handleAuth('/sign-in/email', 'POST', { email: 'test@example.com' }, { request: {} });
-                expect(result.handled).toBe(true);
-                expect(result.response?.status).toBe(200);
-                expect(result.response?.body.user).toBeDefined();
-                expect(result.response?.body.session).toBeDefined();
-            });
-
-            it('should mock get-session endpoint', async () => {
-                const result = await dispatcher.handleAuth('/get-session', 'GET', {}, { request: {} });
-                expect(result.handled).toBe(true);
-                expect(result.response?.status).toBe(200);
-                expect(result.response?.body).toEqual({ session: null, user: null });
-            });
-
-            it('should mock sign-out endpoint', async () => {
-                const result = await dispatcher.handleAuth('/sign-out', 'POST', {}, { request: {} });
-                expect(result.handled).toBe(true);
-                expect(result.response?.status).toBe(200);
-                expect(result.response?.body).toEqual({ success: true });
-            });
-
-            it('should mock login fallback when broker unavailable', async () => {
-                const result = await dispatcher.handleAuth('/login', 'POST', { email: 'test@example.com' }, { request: {} });
-                expect(result.handled).toBe(true);
-                expect(result.response?.status).toBe(200);
-                expect(result.response?.body.user).toBeDefined();
-                expect(result.response?.body.session).toBeDefined();
-            });
-
-            it('should return unhandled for unknown auth path in mock mode', async () => {
-                const result = await dispatcher.handleAuth('/unknown', 'GET', {}, { request: {} });
-                expect(result.handled).toBe(false);
-            });
-        });
-
-        describe('handleStorage with async service', () => {
-            it('should resolve storage service from Promise', async () => {
-                const mockStorage = {
-                    upload: vi.fn().mockResolvedValue({ id: 'file_1', url: '/files/1' }),
-                };
-                (kernel as any).getService = vi.fn().mockImplementation((name: string) => {
-                    if (name === 'file-storage') return Promise.resolve(mockStorage);
-                    return null;
-                });
-
-                const result = await dispatcher.handleStorage('/upload', 'POST', { name: 'test.txt' }, { request: {} });
-                expect(result.handled).toBe(true);
-                expect(result.response?.status).toBe(200);
-                expect(mockStorage.upload).toHaveBeenCalled();
-            });
-
-            it('should return 501 when storage service is not registered (async null)', async () => {
-                (kernel as any).getService = vi.fn().mockResolvedValue(null);
-                (kernel as any).services = new Map();
-
-                const result = await dispatcher.handleStorage('/upload', 'POST', {}, { request: {} });
-                expect(result.handled).toBe(true);
-                expect(result.response?.status).toBe(501);
-                expect(result.response?.body?.error?.message).toBe('File storage not configured');
-            });
-
-            it('should handle GET /storage/file/:id with async service', async () => {
-                const mockStorage = {
-                    download: vi.fn().mockResolvedValue({ data: 'content', mimeType: 'text/plain' }),
-                };
-                (kernel as any).getService = vi.fn().mockImplementation((name: string) => {
-                    if (name === 'file-storage') return Promise.resolve(mockStorage);
-                    return null;
-                });
-
-                const result = await dispatcher.handleStorage('/file/abc123', 'GET', null, { request: {} });
-                expect(result.handled).toBe(true);
-                expect(mockStorage.download).toHaveBeenCalledWith('abc123', { request: {} });
-            });
-
-            it('should return 400 when upload has no file', async () => {
-                const mockStorage = { upload: vi.fn() };
-                (kernel as any).getService = vi.fn().mockResolvedValue(mockStorage);
-
-                const result = await dispatcher.handleStorage('/upload', 'POST', null, { request: {} });
-                expect(result.handled).toBe(true);
-                expect(result.response?.status).toBe(400);
-                expect(result.response?.body?.error?.message).toBe('No file provided');
-            });
-        });
-
-        describe('handleAutomation with async service', () => {
-            it('should resolve automation service from Promise (async factory)', async () => {
-                const mockAuto = {
-                    listFlows: vi.fn().mockResolvedValue(['f1']),
-                };
-                (kernel as any).getService = vi.fn().mockImplementation((name: string) => {
-                    if (name === 'automation') return Promise.resolve(mockAuto);
-                    return null;
-                });
-
-                const result = await dispatcher.handleAutomation('', 'GET', {}, { request: {} });
-                expect(result.handled).toBe(true);
-                expect(result.response?.body?.data?.flows).toEqual(['f1']);
-            });
-
-            it('should return unhandled when automation service not registered', async () => {
-                (kernel as any).getService = vi.fn().mockResolvedValue(null);
-                (kernel as any).services = new Map();
-
-                const result = await dispatcher.handleAutomation('', 'GET', {}, { request: {} });
-                expect(result.handled).toBe(false);
-            });
-        });
-
-        describe('handleMetadata with async protocol service', () => {
-            it('should resolve protocol service from async getService', async () => {
-                const asyncProtocol = {
-                    saveMetaItem: vi.fn().mockResolvedValue({ success: true }),
-                };
-                (kernel as any).context.getService = vi.fn().mockImplementation((name: string) => {
-                    if (name === 'protocol') return Promise.resolve(asyncProtocol);
-                    return null;
-                });
-
-                const result = await dispatcher.handleMetadata('/objects/my_obj', { request: {} }, 'PUT', { label: 'Test' });
-                expect(result.handled).toBe(true);
-                expect(result.response?.status).toBe(200);
-                expect(asyncProtocol.saveMetaItem).toHaveBeenCalled();
-            });
-
-            it('should fallback to broker when async protocol returns null', async () => {
-                (kernel as any).context.getService = vi.fn().mockResolvedValue(null);
-                mockBroker.call.mockResolvedValue({ name: 'my_obj' });
-
-                const result = await dispatcher.handleMetadata('/objects/my_obj', { request: {} }, 'GET');
-                expect(result.handled).toBe(true);
-                expect(mockBroker.call).toHaveBeenCalledWith(
-                    'metadata.getObject',
-                    { objectName: 'my_obj' },
-                    { request: {} }
-                );
-            });
-        });
-    });
-
-    // ═══════════════════════════════════════════════════════════════
-    // Synchronous service resolution (backward compatibility)
-    // ═══════════════════════════════════════════════════════════════
-
-    describe('Synchronous service resolution (backward compat)', () => {
-        it('should work with synchronous service from services Map', async () => {
-            const syncAnalytics = {
-                query: vi.fn().mockResolvedValue({ rows: [], total: 0 }),
-            };
-            (kernel as any).services = new Map([['analytics', syncAnalytics]]);
-
-            const result = await dispatcher.handleAnalytics('query', 'POST', {}, { request: {} });
-            expect(result.handled).toBe(true);
-            expect(syncAnalytics.query).toHaveBeenCalled();
-        });
-
-        it('should work with synchronous getService returning service directly', async () => {
-            const syncAuto = {
-                listFlows: vi.fn().mockResolvedValue(['flow_x']),
-            };
-            (kernel as any).getService = vi.fn().mockReturnValue(syncAuto);
-
-            const result = await dispatcher.handleAutomation('', 'GET', {}, { request: {} });
-            expect(result.handled).toBe(true);
-            expect(result.response?.body?.data?.flows).toEqual(['flow_x']);
-        });
-    });
-
-    // ═══════════════════════════════════════════════════════════════
-    // getServiceAsync preferred path
-    // ═══════════════════════════════════════════════════════════════
-
-    describe('getServiceAsync preferred path', () => {
-        it('should prefer getServiceAsync over getService for analytics', async () => {
-            const asyncAnalytics = {
-                query: vi.fn().mockResolvedValue({ rows: [1], total: 1 }),
-            };
-            (kernel as any).getServiceAsync = vi.fn().mockResolvedValue(asyncAnalytics);
-            (kernel as any).getService = vi.fn().mockImplementation(() => {
-                throw new Error("Service 'analytics' is async - use await");
-            });
-
-            const result = await dispatcher.handleAnalytics('query', 'POST', {}, { request: {} });
-            expect(result.handled).toBe(true);
-            expect(asyncAnalytics.query).toHaveBeenCalled();
-            expect((kernel as any).getServiceAsync).toHaveBeenCalledWith('analytics');
-        });
-
-        it('should prefer getServiceAsync over getService for auth', async () => {
-            const asyncAuth = {
-                handler: vi.fn().mockResolvedValue({ user: { id: '1' } }),
-            };
-            (kernel as any).getServiceAsync = vi.fn().mockResolvedValue(asyncAuth);
-            (kernel as any).getService = vi.fn().mockImplementation(() => {
-                throw new Error("Service 'auth' is async - use await");
-            });
-
-            const result = await dispatcher.handleAuth('', 'POST', {}, { request: {}, response: {} });
-            expect(result.handled).toBe(true);
-            expect(asyncAuth.handler).toHaveBeenCalled();
-            expect((kernel as any).getServiceAsync).toHaveBeenCalledWith('auth');
-        });
-
-        it('should prefer getServiceAsync over getService for automation', async () => {
-            const asyncAuto = {
-                listFlows: vi.fn().mockResolvedValue(['flow_async']),
-            };
-            (kernel as any).getServiceAsync = vi.fn().mockResolvedValue(asyncAuto);
-
-            const result = await dispatcher.handleAutomation('', 'GET', {}, { request: {} });
-            expect(result.handled).toBe(true);
-            expect(result.response?.body?.data?.flows).toEqual(['flow_async']);
-            expect((kernel as any).getServiceAsync).toHaveBeenCalledWith('automation');
-        });
-
-        it('should prefer getServiceAsync over getService for file-storage', async () => {
-            const asyncStorage = {
-                upload: vi.fn().mockResolvedValue({ id: 'file_1', url: '/files/1' }),
-            };
-            (kernel as any).getServiceAsync = vi.fn().mockResolvedValue(asyncStorage);
-
-            const result = await dispatcher.handleStorage('/upload', 'POST', { name: 'test.txt' }, { request: {} });
-            expect(result.handled).toBe(true);
-            expect(result.response?.status).toBe(200);
-            expect((kernel as any).getServiceAsync).toHaveBeenCalledWith('file-storage');
-        });
-
-        it('should resolve protocol service via getServiceAsync for handleMetadata', async () => {
-            const asyncProtocol = {
-                saveMetaItem: vi.fn().mockResolvedValue({ success: true }),
-            };
-            (kernel as any).getServiceAsync = vi.fn().mockImplementation((name: string) => {
-                if (name === 'protocol') return Promise.resolve(asyncProtocol);
-                return Promise.resolve(null);
-            });
-            // Remove context.getService to ensure getServiceAsync is used
-            (kernel as any).context = {};
-
-            const result = await dispatcher.handleMetadata('/objects/my_obj', { request: {} }, 'PUT', { label: 'Test' });
-            expect(result.handled).toBe(true);
-            expect(result.response?.status).toBe(200);
-            expect(asyncProtocol.saveMetaItem).toHaveBeenCalled();
-            expect((kernel as any).getServiceAsync).toHaveBeenCalledWith('protocol');
-        });
-
-        it('should fall through when getServiceAsync returns null', async () => {
-            (kernel as any).getServiceAsync = vi.fn().mockResolvedValue(null);
-            const syncAnalytics = {
-                query: vi.fn().mockResolvedValue({ rows: [], total: 0 }),
-            };
-            (kernel as any).services = new Map([['analytics', syncAnalytics]]);
-
-            const result = await dispatcher.handleAnalytics('query', 'POST', {}, { request: {} });
-            expect(result.handled).toBe(true);
-            expect(syncAnalytics.query).toHaveBeenCalled();
-        });
-
-        it('should fall through when getServiceAsync throws', async () => {
-            (kernel as any).getServiceAsync = vi.fn().mockRejectedValue(new Error('not found'));
-            const syncAnalytics = {
-                query: vi.fn().mockResolvedValue({ rows: [], total: 0 }),
-            };
-            (kernel as any).services = new Map([['analytics', syncAnalytics]]);
-
-            const result = await dispatcher.handleAnalytics('query', 'POST', {}, { request: {} });
-            expect(result.handled).toBe(true);
-            expect(syncAnalytics.query).toHaveBeenCalled();
-        });
-    });
-
-    // ═══════════════════════════════════════════════════════════════
-    // handleData — expand/populate parameter flow
-    // ═══════════════════════════════════════════════════════════════
-
-    describe('handleData', () => {
-        it('should pass expand and select to broker for GET /data/:object/:id', async () => {
-            mockBroker.call.mockResolvedValue({ object: 'order_item', id: 'oi_1', record: { id: 'oi_1' } });
-
-            const result = await dispatcher.handleData(
-                '/order_item/oi_1', 'GET', {},
-                { expand: 'order,product', select: 'name,total' },
-                { request: {} }
-            );
-
-            expect(result.handled).toBe(true);
-            expect(result.response?.status).toBe(200);
-            expect(mockBroker.call).toHaveBeenCalledWith(
-                'data.get',
-                { object: 'order_item', id: 'oi_1', expand: 'order,product', select: 'name,total' },
-                { request: {} }
-            );
-        });
-
-        it('should NOT pass non-allowlisted params for GET /data/:object/:id', async () => {
-            mockBroker.call.mockResolvedValue({ object: 'task', id: 't1', record: {} });
-
-            await dispatcher.handleData(
-                '/task/t1', 'GET', {},
-                { expand: 'assignee', malicious: 'drop_table', filter: 'hack' },
-                { request: {} }
-            );
-
-            // Only expand is passed; malicious and filter are dropped
-            expect(mockBroker.call).toHaveBeenCalledWith(
-                'data.get',
-                { object: 'task', id: 't1', expand: 'assignee' },
-                { request: {} }
-            );
-        });
-
-        it('should pass full query (with expand/populate) for GET /data/:object list', async () => {
-            mockBroker.call.mockResolvedValue({ object: 'task', records: [], total: 0 });
-
-            const query = { populate: 'assignee,project', top: '10', skip: '0' };
-            const result = await dispatcher.handleData(
-                '/task', 'GET', {},
-                query,
-                { request: {} }
-            );
-
-            expect(result.handled).toBe(true);
-            // top → limit and skip → offset are normalized by the dispatcher
-            expect(mockBroker.call).toHaveBeenCalledWith(
-                'data.query',
-                { object: 'task', query: { populate: 'assignee,project', limit: '10', offset: '0' } },
-                { request: {} }
-            );
-        });
-
-        it('should pass expand in query for GET /data/:object list', async () => {
-            mockBroker.call.mockResolvedValue({ object: 'order', records: [], total: 0 });
-
-            const query = { expand: 'customer,products' };
-            await dispatcher.handleData('/order', 'GET', {}, query, { request: {} });
-
-            expect(mockBroker.call).toHaveBeenCalledWith(
-                'data.query',
-                { object: 'order', query: { expand: 'customer,products' } },
-                { request: {} }
-            );
-        });
-
-        it('should return error if object name is missing', async () => {
-            const result = await dispatcher.handleData('/', 'GET', {}, {}, { request: {} });
-            expect(result.handled).toBe(true);
-            expect(result.response?.status).toBe(400);
-        });
-
-        it('should handle POST /data/:object/query with body containing expand', async () => {
-            mockBroker.call.mockResolvedValue({ object: 'task', records: [] });
-
-            await dispatcher.handleData(
-                '/task/query', 'POST',
-                { filter: { status: 'active' }, populate: ['assignee'] },
-                {},
-                { request: {} }
-            );
-
-            expect(mockBroker.call).toHaveBeenCalledWith(
-                'data.query',
-                { object: 'task', filter: { status: 'active' }, populate: ['assignee'] },
-                { request: {} }
-            );
-        });
-    });
-
-    // ═══════════════════════════════════════════════════════════════
-    // Error handling for service method failures
-    // ═══════════════════════════════════════════════════════════════
-
-    describe('Service method error handling', () => {
-        it('should propagate analytics query error', async () => {
-            const badAnalytics = {
-                query: vi.fn().mockRejectedValue(new Error('Query timeout')),
-            };
-            (kernel as any).getService = vi.fn().mockResolvedValue(badAnalytics);
-
-            await expect(
-                dispatcher.handleAnalytics('query', 'POST', {}, { request: {} })
-            ).rejects.toThrow('Query timeout');
-        });
-
-        it('should propagate storage upload error', async () => {
-            const badStorage = {
-                upload: vi.fn().mockRejectedValue(new Error('Disk full')),
-            };
-            (kernel as any).getService = vi.fn().mockImplementation((name: string) => {
-                if (name === 'file-storage') return Promise.resolve(badStorage);
-                return null;
-            });
-
-            await expect(
-                dispatcher.handleStorage('/upload', 'POST', { data: 'file' }, { request: {} })
-            ).rejects.toThrow('Disk full');
-        });
-    });
-
-    // ═══════════════════════════════════════════════════════════════
-    // Package Publish / Revert Endpoints
-    // ═══════════════════════════════════════════════════════════════
-
-    describe('Package publish/revert endpoints', () => {
-        it('should handle POST /packages/:id/publish via metadata service', async () => {
-            const mockMetadata = {
-                publishPackage: vi.fn().mockResolvedValue({
-                    success: true,
-                    packageId: 'com.acme.crm',
-                    version: 2,
-                    publishedAt: '2025-06-01T00:00:00Z',
-                    itemsPublished: 3,
-                }),
-            };
-            const mockRegistry = {
-                getAllPackages: vi.fn().mockReturnValue([]),
-                enablePackage: vi.fn(),
-                disablePackage: vi.fn(),
-            };
-            (kernel as any).getService = vi.fn().mockImplementation((name: string) => {
-                if (name === 'metadata') return Promise.resolve(mockMetadata);
-                if (name === 'objectql') return Promise.resolve({ registry: mockRegistry });
-                return null;
-            });
-
-            const result = await dispatcher.handlePackages('/com.acme.crm/publish', 'POST', { publishedBy: 'admin' }, {}, { request: {} });
-            expect(result.handled).toBe(true);
-            expect(result.response?.status).toBe(200);
-            expect(mockMetadata.publishPackage).toHaveBeenCalledWith('com.acme.crm', { publishedBy: 'admin' });
-        });
-
-        it('should handle POST /packages/:id/revert via metadata service', async () => {
-            const mockMetadata = {
-                revertPackage: vi.fn().mockResolvedValue(undefined),
-            };
-            const mockRegistry = {
-                getAllPackages: vi.fn().mockReturnValue([]),
-                enablePackage: vi.fn(),
-                disablePackage: vi.fn(),
-            };
-            (kernel as any).getService = vi.fn().mockImplementation((name: string) => {
-                if (name === 'metadata') return Promise.resolve(mockMetadata);
-                if (name === 'objectql') return Promise.resolve({ registry: mockRegistry });
-                return null;
-            });
-
-            const result = await dispatcher.handlePackages('/com.acme.crm/revert', 'POST', {}, {}, { request: {} });
-            expect(result.handled).toBe(true);
-            expect(result.response?.status).toBe(200);
-            expect(mockMetadata.revertPackage).toHaveBeenCalledWith('com.acme.crm');
-        });
-
-        it('should fallback to broker for publish when metadata service unavailable', async () => {
-            const mockRegistry = {
-                getAllPackages: vi.fn().mockReturnValue([]),
-            };
-            (kernel as any).getService = vi.fn().mockImplementation((name: string) => {
-                if (name === 'metadata') return Promise.resolve(null);
-                if (name === 'objectql') return Promise.resolve({ registry: mockRegistry });
-                return null;
-            });
-            mockBroker.call.mockResolvedValue({ success: true, packageId: 'crm', version: 1, publishedAt: '2025-01-01T00:00:00Z', itemsPublished: 2 });
-
-            const result = await dispatcher.handlePackages('/crm/publish', 'POST', {}, {}, { request: {} });
-            expect(result.handled).toBe(true);
-            expect(mockBroker.call).toHaveBeenCalledWith('metadata.publishPackage', { packageId: 'crm' }, { request: {} });
-        });
-    });
-
-    // ═══════════════════════════════════════════════════════════════
-    // Metadata getPublished Endpoint
-    // ═══════════════════════════════════════════════════════════════
-
-    describe('Metadata getPublished endpoint', () => {
-        it('should handle GET /metadata/:type/:name/published via metadata service', async () => {
-            const mockMetadata = {
-                getPublished: vi.fn().mockResolvedValue({ name: 'account', label: 'Account' }),
-            };
-            (kernel as any).getService = vi.fn().mockImplementation((name: string) => {
-                if (name === 'metadata') return Promise.resolve(mockMetadata);
-                return null;
-            });
-
-            const result = await dispatcher.handleMetadata('/object/account/published', { request: {} }, 'GET');
-            expect(result.handled).toBe(true);
-            expect(result.response?.status).toBe(200);
-            expect(result.response?.body?.data).toEqual({ name: 'account', label: 'Account' });
-            expect(mockMetadata.getPublished).toHaveBeenCalledWith('object', 'account');
-        });
-
-        it('should return 404 when published item not found', async () => {
-            const mockMetadata = {
-                getPublished: vi.fn().mockResolvedValue(undefined),
-            };
-            (kernel as any).getService = vi.fn().mockImplementation((name: string) => {
-                if (name === 'metadata') return Promise.resolve(mockMetadata);
-                return null;
-            });
-
-            const result = await dispatcher.handleMetadata('/object/nonexistent/published', { request: {} }, 'GET');
-            expect(result.handled).toBe(true);
-            expect(result.response?.status).toBe(404);
-        });
-
-        it('should fallback to broker for getPublished when metadata service unavailable', async () => {
-            (kernel as any).getService = vi.fn().mockResolvedValue(null);
-            mockBroker.call.mockResolvedValue({ name: 'account', fields: ['name'] });
-
-            const result = await dispatcher.handleMetadata('/object/account/published', { request: {} }, 'GET');
-            expect(result.handled).toBe(true);
-            expect(result.response?.status).toBe(200);
-            expect(mockBroker.call).toHaveBeenCalledWith(
-                'metadata.getPublished',
-                { type: 'object', name: 'account' },
-                { request: {} }
-            );
-        });
-    });
-
-    // ═══════════════════════════════════════════════════════════════
-    // handleI18n — i18n route dispatching
-    // ═══════════════════════════════════════════════════════════════
-
-    describe('handleI18n', () => {
-        let mockI18nService: any;
-
-        beforeEach(() => {
-            mockI18nService = {
-                getLocales: vi.fn().mockReturnValue(['en', 'zh-CN', 'ja']),
-                getTranslations: vi.fn().mockReturnValue({ 'o.account.label': '客户', 'o.account.fields.name': '名称' }),
-                getFieldLabels: vi.fn().mockReturnValue({ name: '名称', industry: '行业' }),
-            };
-
-            (kernel as any).getService = vi.fn().mockImplementation((name: string) => {
-                if (name === 'i18n') return mockI18nService;
-                return null;
-            });
-        });
-
-        it('should list locales via GET /locales', async () => {
-            const result = await dispatcher.handleI18n('/locales', 'GET', {}, { request: {} });
-            expect(result.handled).toBe(true);
-            expect(result.response?.status).toBe(200);
-            expect(result.response?.body?.data?.locales).toEqual(['en', 'zh-CN', 'ja']);
-            expect(mockI18nService.getLocales).toHaveBeenCalled();
-        });
-
-        it('should get translations via GET /translations/:locale', async () => {
-            const result = await dispatcher.handleI18n('/translations/zh-CN', 'GET', {}, { request: {} });
-            expect(result.handled).toBe(true);
-            expect(result.response?.status).toBe(200);
-            expect(result.response?.body?.data?.locale).toBe('zh-CN');
-            expect(result.response?.body?.data?.translations).toEqual({ 'o.account.label': '客户', 'o.account.fields.name': '名称' });
-            expect(mockI18nService.getTranslations).toHaveBeenCalledWith('zh-CN');
-        });
-
-        it('should get translations via GET /translations?locale=zh-CN (query param)', async () => {
-            const result = await dispatcher.handleI18n('/translations', 'GET', { locale: 'zh-CN' }, { request: {} });
-            expect(result.handled).toBe(true);
-            expect(result.response?.status).toBe(200);
-            expect(result.response?.body?.data?.locale).toBe('zh-CN');
-            expect(mockI18nService.getTranslations).toHaveBeenCalledWith('zh-CN');
-        });
-
-        it('should return 400 when translations requested without locale', async () => {
-            const result = await dispatcher.handleI18n('/translations', 'GET', {}, { request: {} });
-            expect(result.handled).toBe(true);
-            expect(result.response?.status).toBe(400);
-            expect(result.response?.body?.error?.message).toBe('Missing locale parameter');
-        });
-
-        it('should get field labels via GET /labels/:object/:locale', async () => {
-            const result = await dispatcher.handleI18n('/labels/account/zh-CN', 'GET', {}, { request: {} });
-            expect(result.handled).toBe(true);
-            expect(result.response?.status).toBe(200);
-            expect(result.response?.body?.data?.object).toBe('account');
-            expect(result.response?.body?.data?.locale).toBe('zh-CN');
-            expect(result.response?.body?.data?.labels).toEqual({ name: '名称', industry: '行业' });
-            expect(mockI18nService.getFieldLabels).toHaveBeenCalledWith('account', 'zh-CN');
-        });
-
-        it('should get field labels via GET /labels/:object?locale=zh-CN (query param)', async () => {
-            const result = await dispatcher.handleI18n('/labels/account', 'GET', { locale: 'zh-CN' }, { request: {} });
-            expect(result.handled).toBe(true);
-            expect(result.response?.status).toBe(200);
-            expect(result.response?.body?.data?.object).toBe('account');
-            expect(mockI18nService.getFieldLabels).toHaveBeenCalledWith('account', 'zh-CN');
-        });
-
-        it('should return 400 when labels requested without locale', async () => {
-            const result = await dispatcher.handleI18n('/labels/account', 'GET', {}, { request: {} });
-            expect(result.handled).toBe(true);
-            expect(result.response?.status).toBe(400);
-            expect(result.response?.body?.error?.message).toBe('Missing locale parameter');
-        });
-
-        it('should fallback to deriving labels from translations when getFieldLabels is missing', async () => {
-            delete mockI18nService.getFieldLabels;
-            mockI18nService.getTranslations.mockReturnValue({
-                'o.contact.fields.first_name': 'First Name',
-                'o.contact.fields.email': 'Email',
-                'o.contact.label': 'Contact',
-            });
-
-            const result = await dispatcher.handleI18n('/labels/contact/en', 'GET', {}, { request: {} });
-            expect(result.handled).toBe(true);
-            expect(result.response?.status).toBe(200);
-            expect(result.response?.body?.data?.labels).toEqual({
-                first_name: 'First Name',
-                email: 'Email',
-            });
-        });
-
-        it('should return 501 when i18n service is not available', async () => {
-            (kernel as any).getService = vi.fn().mockResolvedValue(null);
-            (kernel as any).services = new Map();
-
-            const result = await dispatcher.handleI18n('/locales', 'GET', {}, { request: {} });
-            expect(result.handled).toBe(true);
-            expect(result.response?.status).toBe(501);
-        });
-
-        it('should return unhandled for non-GET methods', async () => {
-            const result = await dispatcher.handleI18n('/locales', 'POST', {}, { request: {} });
-            expect(result.handled).toBe(false);
-        });
-
-        it('should dispatch /i18n routes via dispatch()', async () => {
-            const result = await dispatcher.dispatch('GET', '/i18n/locales', undefined, {}, { request: {} });
-            expect(result.handled).toBe(true);
-            expect(result.response?.body?.data?.locales).toEqual(['en', 'zh-CN', 'ja']);
-        });
-
-        it('should resolve locale via fallback (zh → zh-CN) for translations', async () => {
-            // Override mock to be locale-aware: only 'zh-CN' has data, 'zh' returns empty
-            mockI18nService.getTranslations = vi.fn().mockImplementation((locale: string) => {
-                if (locale === 'zh-CN') return { 'o.task.label': '任务' };
-                return {};
-            });
-
-            const result = await dispatcher.handleI18n('/translations/zh', 'GET', {}, { request: {} });
-            expect(result.handled).toBe(true);
-            expect(result.response?.status).toBe(200);
-            const data = result.response?.body?.data;
-            expect(data.locale).toBe('zh-CN');
-            expect(data.requestedLocale).toBe('zh');
-            expect(data.translations).toEqual({ 'o.task.label': '任务' });
-        });
-
-        it('should resolve locale via case-insensitive fallback (ZH-CN → zh-CN) for translations', async () => {
-            // Override mock to be locale-aware: 'ZH-CN' returns empty, 'zh-CN' has data
-            mockI18nService.getTranslations = vi.fn().mockImplementation((locale: string) => {
-                if (locale === 'zh-CN') return { 'o.task.label': '任务' };
-                return {};
-            });
-
-            const result = await dispatcher.handleI18n('/translations/ZH-CN', 'GET', {}, { request: {} });
-            expect(result.handled).toBe(true);
-            expect(result.response?.status).toBe(200);
-            const data = result.response?.body?.data;
-            expect(data.locale).toBe('zh-CN');
-            expect(data.translations).toEqual({ 'o.task.label': '任务' });
-        });
-    });
-
-    // ═══════════════════════════════════════════════════════════════
-    // Discovery ↔ Handler i18n consistency
-    // ═══════════════════════════════════════════════════════════════
-
-    describe('discovery-handler i18n consistency', () => {
-        it('should report i18n as available in discovery when service is registered', async () => {
-            const mockI18nService = {
-                getLocales: vi.fn().mockReturnValue(['en', 'zh-CN', 'ja']),
-                getTranslations: vi.fn().mockReturnValue({}),
-                getDefaultLocale: vi.fn().mockReturnValue('en'),
-            };
-
-            (kernel as any).getService = vi.fn().mockImplementation((name: string) => {
-                if (name === 'i18n') return mockI18nService;
-                return null;
-            });
-
-            const info = await dispatcher.getDiscoveryInfo('/api/v1');
-            expect(info.services.i18n.enabled).toBe(true);
-            expect(info.services.i18n.status).toBe('available');
-            expect(info.routes.i18n).toBe('/api/v1/i18n');
-            expect(info.features.i18n).toBe(true);
-        });
-
-        it('should report i18n as unavailable in discovery when service is not registered', async () => {
-            (kernel as any).getService = vi.fn().mockResolvedValue(null);
-            (kernel as any).services = new Map();
-
-            const info = await dispatcher.getDiscoveryInfo('/api/v1');
-            expect(info.services.i18n.enabled).toBe(false);
-            expect(info.services.i18n.status).toBe('unavailable');
-            expect(info.routes.i18n).toBeUndefined();
-            expect(info.features.i18n).toBe(false);
-        });
-
-        it('should detect i18n via getServiceAsync (async factory) in discovery', async () => {
-            const mockI18nService = {
-                getLocales: vi.fn().mockReturnValue(['en', 'fr']),
-                getTranslations: vi.fn().mockReturnValue({}),
-                getDefaultLocale: vi.fn().mockReturnValue('fr'),
-            };
-
-            // Service NOT in sync map, only accessible via async factory
-            (kernel as any).services = new Map();
-            (kernel as any).getServiceAsync = vi.fn().mockImplementation(async (name: string) => {
-                if (name === 'i18n') return mockI18nService;
-                return null;
-            });
-
-            const info = await dispatcher.getDiscoveryInfo('/api/v1');
-            expect(info.services.i18n.enabled).toBe(true);
-            expect(info.services.i18n.status).toBe('available');
-
-            // Handler should also find it
-            const result = await dispatcher.handleI18n('/locales', 'GET', {}, { request: {} });
-            expect(result.handled).toBe(true);
-            expect(result.response?.status).toBe(200);
-            expect(result.response?.body?.data?.locales).toEqual(['en', 'fr']);
-        });
-
-        it('should populate locale from actual i18n service', async () => {
-            const mockI18nService = {
-                getLocales: vi.fn().mockReturnValue(['en', 'zh-CN', 'ja']),
-                getTranslations: vi.fn().mockReturnValue({}),
-                getDefaultLocale: vi.fn().mockReturnValue('zh-CN'),
-            };
-
-            (kernel as any).getService = vi.fn().mockImplementation((name: string) => {
-                if (name === 'i18n') return mockI18nService;
-                return null;
-            });
-
-            const info = await dispatcher.getDiscoveryInfo('/api/v1');
-            expect(info.locale.default).toBe('zh-CN');
-            expect(info.locale.supported).toEqual(['en', 'zh-CN', 'ja']);
-        });
-
-        it('should use default locale when i18n service is not available', async () => {
-            (kernel as any).getService = vi.fn().mockResolvedValue(null);
-            (kernel as any).services = new Map();
-
-            const info = await dispatcher.getDiscoveryInfo('/api/v1');
-            expect(info.locale.default).toBe('en');
-            expect(info.locale.supported).toEqual(['en']);
-            expect(info.locale.timezone).toBe('UTC');
-        });
-
-        it('should ensure discovery and dispatch are consistent for root path', async () => {
-            const mockI18nService = {
-                getLocales: vi.fn().mockReturnValue(['en']),
-                getTranslations: vi.fn().mockReturnValue({}),
-                getDefaultLocale: vi.fn().mockReturnValue('en'),
-            };
-
-            (kernel as any).getService = vi.fn().mockImplementation((name: string) => {
-                if (name === 'i18n') return mockI18nService;
-                return null;
-            });
-
-            // Dispatch to root should return the same discovery data
-            const result = await dispatcher.dispatch('GET', '', undefined, {}, { request: {} });
-            expect(result.handled).toBe(true);
-            const data = result.response?.body?.data;
-            expect(data.services.i18n.enabled).toBe(true);
-            expect(data.locale.default).toBe('en');
-        });
-    });
-
-    // ═══════════════════════════════════════════════════════════════
-    // i18n across server/dev/mock environments
-    // ═══════════════════════════════════════════════════════════════
-
-    describe('i18n environment consistency', () => {
-        it('should work with dev stub i18n service (in-memory translations)', async () => {
-            // Simulate dev plugin i18n stub — Map-backed, all sync
-            const translations = new Map<string, Record<string, unknown>>();
-            let defaultLocale = 'en';
-            const devI18nStub = {
-                t: (key: string, locale: string) => {
-                    const t = translations.get(locale);
-                    return (t?.[key] as string) ?? key;
-                },
-                getTranslations: (locale: string) => translations.get(locale) ?? {},
-                loadTranslations: (locale: string, data: Record<string, unknown>) => {
-                    translations.set(locale, { ...translations.get(locale), ...data });
-                },
-                getLocales: () => [...translations.keys()],
-                getDefaultLocale: () => defaultLocale,
-                setDefaultLocale: (locale: string) => { defaultLocale = locale; },
-            };
-
-            // Load data like AppPlugin would
-            devI18nStub.loadTranslations('en', { 'o.task.label': 'Task' });
-            devI18nStub.loadTranslations('zh-CN', { 'o.task.label': '任务' });
-
-            (kernel as any).getService = vi.fn().mockImplementation((name: string) => {
-                if (name === 'i18n') return devI18nStub;
-                return null;
-            });
-
-            // Discovery should reflect loaded locales
-            const info = await dispatcher.getDiscoveryInfo('/api/v1');
-            expect(info.services.i18n.enabled).toBe(true);
-            expect(info.locale.supported).toEqual(['en', 'zh-CN']);
-
-            // Handler should serve translations
-            const result = await dispatcher.handleI18n('/translations/zh-CN', 'GET', {}, { request: {} });
-            expect(result.response?.status).toBe(200);
-            expect(result.response?.body?.data?.translations['o.task.label']).toBe('任务');
-        });
-
-        it('should handle MSW catch-all dispatch pattern for i18n', async () => {
-            // MSW routes all requests through dispatcher.dispatch()
-            const mockI18nService = {
-                getLocales: vi.fn().mockReturnValue(['en', 'de']),
-                getTranslations: vi.fn().mockReturnValue({ 'o.account.label': 'Konto' }),
-                getDefaultLocale: vi.fn().mockReturnValue('de'),
-            };
-
-            (kernel as any).getService = vi.fn().mockImplementation((name: string) => {
-                if (name === 'i18n') return mockI18nService;
-                return null;
-            });
-
-            // MSW-style dispatch: full path stripped to relative
-            const localesResult = await dispatcher.dispatch('GET', '/i18n/locales', undefined, {}, { request: {} });
-            expect(localesResult.handled).toBe(true);
-            expect(localesResult.response?.body?.data?.locales).toEqual(['en', 'de']);
-
-            const translationsResult = await dispatcher.dispatch('GET', '/i18n/translations/de', undefined, {}, { request: {} });
-            expect(translationsResult.handled).toBe(true);
-            expect(translationsResult.response?.body?.data?.translations['o.account.label']).toBe('Konto');
-
-            // Discovery and handler agree
-            const discovery = await dispatcher.getDiscoveryInfo('/api/v1');
-            expect(discovery.services.i18n.enabled).toBe(true);
-            expect(discovery.locale.default).toBe('de');
-        });
-
-        it('should return 501 consistently when i18n is unavailable in both discovery and handler', async () => {
-            (kernel as any).getService = vi.fn().mockResolvedValue(null);
-            (kernel as any).services = new Map();
-
-            // Discovery: unavailable
-            const info = await dispatcher.getDiscoveryInfo('/api/v1');
-            expect(info.services.i18n.enabled).toBe(false);
-            expect(info.services.i18n.status).toBe('unavailable');
-
-            // Handler: 501
-            const result = await dispatcher.handleI18n('/locales', 'GET', {}, { request: {} });
-            expect(result.response?.status).toBe(501);
-
-            // Dispatch: also 501
-            const dispatchResult = await dispatcher.dispatch('GET', '/i18n/locales', undefined, {}, { request: {} });
-            expect(dispatchResult.response?.status).toBe(501);
-        });
-
-        it('should handle context-based service resolution (mock kernel)', async () => {
-            // Simulate a kernel that only provides i18n through context.getService
-            const mockI18n = {
-                getLocales: vi.fn().mockReturnValue(['en']),
-                getTranslations: vi.fn().mockReturnValue({}),
-                getDefaultLocale: vi.fn().mockReturnValue('en'),
-            };
-
-            (kernel as any).services = new Map();
-            (kernel as any).getService = undefined;
-            (kernel as any).getServiceAsync = undefined;
-            (kernel as any).context = {
-                getService: vi.fn().mockImplementation((name: string) => {
-                    if (name === 'i18n') return mockI18n;
-                    return null;
-                }),
-            };
-
-            const info = await dispatcher.getDiscoveryInfo('/api/v1');
-            expect(info.services.i18n.enabled).toBe(true);
-
-            const result = await dispatcher.handleI18n('/locales', 'GET', {}, { request: {} });
-            expect(result.response?.status).toBe(200);
-        });
-    });
-
-    describe('handleMetadata without broker (serverless degradation)', () => {
-        let brokerlessKernel: any;
-        let brokerlessDispatcher: HttpDispatcher;
-
-        beforeEach(() => {
-            // Kernel with NO broker — simulates a lightweight/serverless setup
-            // where only the protocol service and/or ObjectQL registry are available.
-            brokerlessKernel = {
-                broker: null,
-                context: {
-                    getService: vi.fn().mockReturnValue(null),
-                },
-            };
-            brokerlessDispatcher = new HttpDispatcher(brokerlessKernel);
-        });
-
-        it('GET /meta should return default types when broker is missing', async () => {
-            const context = { request: {} };
-            const result = await brokerlessDispatcher.handleMetadata('', context, 'GET');
-            expect(result.handled).toBe(true);
-            expect(result.response?.status).toBe(200);
-            expect(result.response?.body?.data?.types).toContain('object');
-        });
-
-        it('GET /meta/types should return default types when broker is missing', async () => {
-            const context = { request: {} };
-            const result = await brokerlessDispatcher.handleMetadata('/types', context, 'GET');
-            expect(result.handled).toBe(true);
-            expect(result.response?.status).toBe(200);
-            expect(result.response?.body?.data?.types).toContain('object');
-        });
-
-        it('GET /meta/objects should use ObjectQL registry when broker is missing', async () => {
-            const mockRegistry = {
-                getAllObjects: vi.fn().mockReturnValue([{ name: 'account' }]),
-                getObject: vi.fn(),
-            };
-            brokerlessKernel.context.getService = vi.fn().mockImplementation((name: string) => {
-                if (name === 'objectql') return { registry: mockRegistry };
-                return null;
-            });
-
-            const context = { request: {} };
-            const result = await brokerlessDispatcher.handleMetadata('/objects', context, 'GET');
-            expect(result.handled).toBe(true);
-            expect(result.response?.status).toBe(200);
-            expect(mockRegistry.getAllObjects).toHaveBeenCalled();
-        });
-
-        it('GET /meta/objects/:name should use ObjectQL registry when broker is missing', async () => {
-            const mockRegistry = {
-                registry: {
-                    getObject: vi.fn().mockReturnValue({ name: 'account', fields: {} }),
-                },
-            };
-            brokerlessKernel.context.getService = vi.fn().mockImplementation((name: string) => {
-                if (name === 'objectql') return mockRegistry;
-                return null;
-            });
-
-            const context = { request: {} };
-            const result = await brokerlessDispatcher.handleMetadata('/objects/account', context, 'GET');
-            expect(result.handled).toBe(true);
-            expect(result.response?.status).toBe(200);
-            expect(mockRegistry.registry.getObject).toHaveBeenCalledWith('account');
-        });
-
-        it('GET /meta/:type/:name/published should return 404 when broker is missing and metadata service is unavailable', async () => {
-            const context = { request: {} };
-            const result = await brokerlessDispatcher.handleMetadata('/object/my_obj/published', context, 'GET');
-            expect(result.handled).toBe(true);
-            expect(result.response?.status).toBe(404);
-        });
-
-        it('PUT /meta/:type/:name should return 501 when broker is missing and protocol is unavailable', async () => {
-            const context = { request: {} };
-            const body = { label: 'Test' };
-            const result = await brokerlessDispatcher.handleMetadata('/objects/my_obj', context, 'PUT', body);
-            expect(result.handled).toBe(true);
-            expect(result.response?.status).toBe(501);
-        });
-
-        it('should use protocol service even when broker is missing', async () => {
-            const mockProtocolLocal = {
-                getMetaTypes: vi.fn().mockResolvedValue({ types: ['custom_type'] }),
-            };
-            brokerlessKernel.context.getService = vi.fn().mockImplementation((name: string) => {
-                if (name === 'protocol') return mockProtocolLocal;
-                return null;
-            });
-
-            const context = { request: {} };
-            const result = await brokerlessDispatcher.handleMetadata('/types', context, 'GET');
-            expect(result.handled).toBe(true);
-            expect(result.response?.status).toBe(200);
-            expect(mockProtocolLocal.getMetaTypes).toHaveBeenCalled();
-            expect(result.response?.body?.data?.types).toContain('custom_type');
-        });
-    });
-});