@cratis/chronicle 6.34.0 → 6.35.0

package/reducers/Reducers.ts

@@ -105,6 +105,7 @@ export class Reducers implements IReducers {
 
     private readonly _logger: ReturnType<typeof createLogger>;
     private readonly _lifecycle: ConnectionLifecycle;
+    private readonly _unsubscribeDisconnected: () => void;
     private readonly _reducers = new Map<string, Constructor>();
     private readonly _queues = new Map<string, AsyncQueue<ReducerMessage>>();
     private readonly _observations = new Map<string, AbortController>();
@@ -136,7 +137,7 @@ export class Reducers implements IReducers {
     ) {
         this._logger = createLogger('@cratis/chronicle/reducers', _connection.logger);
         this._lifecycle = lifecycle;
-        lifecycle.onDisconnected(async () => {
+        this._unsubscribeDisconnected = lifecycle.onDisconnected(async () => {
             this._logger.info('Disconnected — stopping all reducer observations');
             this._registered = false;
             this.disconnectAll();
@@ -145,6 +146,7 @@ export class Reducers implements IReducers {
 
     /** Stops observations permanently when the owning client is disposed. */
     dispose(): void {
+        this._unsubscribeDisconnected();
         this._disposed = true;
         this._registered = false;
         this.disconnectAll();
@@ -172,13 +174,16 @@ export class Reducers implements IReducers {
             await this.discover();
         }
 
+        const registrations = [...this._reducers].map(([id, type]) => ({
+            id, type, dispatcher: new ReducerEventDispatcher(type, this._clientArtifacts.eventTypes)
+        }));
         assertUniqueReadModelIds(this._clientArtifacts.readModels);
         await this.registerReadModels();
 
         if (this._disposed) return;
         this._logger.info('Registering reducers', { count: this._reducers.size });
-        for (const [id, reducerType] of this._reducers) {
-            this.startObservation(id, reducerType);
+        for (const { id, type, dispatcher } of registrations) {
+            this.startObservation(id, type, dispatcher);
         }
 
         this._registered = true;
@@ -236,12 +241,12 @@ export class Reducers implements IReducers {
         return JSON.stringify(minimalSchema);
     }
 
-    private startObservation(id: string, reducerType: Constructor): void {
+    private startObservation(id: string, reducerType: Constructor,
+        dispatcher: ReducerEventDispatcher): void {
         runInBackgroundContext(() => {
             if (this._disposed) return;
             const metadata = getReducerMetadata(reducerType)!;
             const eventSequenceId = metadata.eventSequenceId ?? EventSequenceId.eventLog.value;
-            const dispatcher = new ReducerEventDispatcher(reducerType, this._clientArtifacts.eventTypes);
             const readModelName = this.getReducerReadModelIdentifier(reducerType);
 
             this._logger.info('Starting reducer observation', {
@@ -269,7 +274,7 @@ export class Reducers implements IReducers {
             this._logger.error('Reducer observation loop exited with error', { reducerId: id, error });
         }
 
-        this.scheduleReobserve(id, reducerType);
+        this.scheduleReobserve(id, reducerType, dispatcher);
     }
 
     /**
@@ -282,7 +287,7 @@ export class Reducers implements IReducers {
      * retried, and the delay keeps a stream that keeps ending from becoming a hot
      * loop.
      */
-    private scheduleReobserve(id: string, reducerType: Constructor): void {
+    private scheduleReobserve(id: string, reducerType: Constructor, dispatcher: ReducerEventDispatcher): void {
         // A disconnect clears the registration; the reconnect re-registers every
         // reducer from scratch, so retrying here as well would double up.
         if (!this._registered || this._disposed) {
@@ -295,7 +300,7 @@ export class Reducers implements IReducers {
             }
 
             this._logger.info('Re-establishing reducer observation', { reducerId: id });
-            this.startObservation(id, reducerType);
+            this.startObservation(id, reducerType, dispatcher);
         }, Reducers._reobserveDelayMs));
 
         handle.unref?.();
@@ -353,6 +358,7 @@ export class Reducers implements IReducers {
 
         try {
             const reducerInstance = this._artifactActivator ? undefined : new (reducerType as new () => Record<string, Function>)();
+            if (reducerInstance) dispatcher.validateInstance(reducerInstance);
 
             for await (const operation of this._connection.reducers.observe(queue, { signal: controller.signal })) {
                 let lastSuccessfullyObservedEvent = SEQUENCE_NUMBER_UNAVAILABLE;
@@ -376,9 +382,12 @@ export class Reducers implements IReducers {
                             readModels: this._eventStore.readModels, eventSequenceId, partition: operation.Partition,
                             signal: controller.signal, delivery: ArtifactDelivery.ReplayNotification,
                             replayState: operation.ReplayState
-                        }, this._artifactActivator, artifact => runActivated(artifact, () =>
-                            notifyReplayLifecycle(artifact.instance, operation.ReplayState, operation.Partition),
-                            { delivery: ArtifactDelivery.ReplayNotification, replayState: operation.ReplayState }), this._connection.logger);
+                        }, this._artifactActivator, artifact => {
+                            dispatcher.validateInstance(artifact.instance);
+                            return runActivated(artifact, () =>
+                                notifyReplayLifecycle(artifact.instance, operation.ReplayState, operation.Partition),
+                                { delivery: ArtifactDelivery.ReplayNotification, replayState: operation.ReplayState });
+                        }, this._connection.logger);
                     } else if (reducerInstance) {
                         await notifyReplayLifecycle(reducerInstance, operation.ReplayState, operation.Partition);
                     }
@@ -448,7 +457,10 @@ export class Reducers implements IReducers {
                             readModels: this._eventStore.readModels, eventSequenceId, partition: operation.Partition,
                             signal: controller.signal, delivery: ArtifactDelivery.Events,
                             eventContext: toClientEventContext(firstInvocableEvent.Context!)
-                        }, this._artifactActivator, processEvents, this._connection.logger);
+                        }, this._artifactActivator, artifact => {
+                            dispatcher.validateInstance(artifact.instance);
+                            return processEvents(artifact);
+                        }, this._connection.logger);
                     } catch (err) {
                         this._logger.error('Error activating reducer', { reducerId: id, error: err });
                         exceptionMessages.push(String(err));

package/reducers/for_ReducerEventDispatcher/when_invoking_an_explicit_handler.spec.ts

@@ -0,0 +1,51 @@
+// Copyright (c) Cratis. All rights reserved.
+// Licensed under the MIT license. See LICENSE file in the project root for full license information.
+
+import { beforeEach, chai, describe, it } from 'vitest';
+import { eventType } from '../../events/eventTypeDecorator.js';
+import { EventType } from '../../events/EventType.js';
+import type { EventContext } from '../../events/EventContext.js';
+import { handles } from '../index.js';
+import { ReducerEventDispatcher } from '../ReducerEventDispatcher.js';
+
+chai.should();
+
+@eventType('explicit-reducer-delivery')
+class AuthorRegistered { constructor(readonly name: string) {} }
+
+class AuthorReducer {
+    @handles(AuthorRegistered)
+    update(event: AuthorRegistered, state: { names: string[] } | undefined, context: EventContext) {
+        return { names: [...(state?.names ?? []), event.name], id: context.eventSourceId };
+    }
+}
+
+class FieldReducer extends AuthorReducer {
+    override update = () => ({ names: [], id: 'hidden' });
+}
+
+describe('when an instance field hides an inherited reducer handler', () => {
+    it('should reject the field against the preflight method', () => {
+        const dispatcher = new ReducerEventDispatcher(FieldReducer, [AuthorRegistered]);
+        (() => dispatcher.validateInstance(new FieldReducer())).should.throw(
+            "Override 'update' on 'FieldReducer' hides @handles(AuthorRegistered) declared on 'AuthorReducer'; use a method with @handles instead of an instance field.");
+    });
+});
+
+describe('when invoking an explicit reducer handler', () => {
+    let result: unknown;
+    beforeEach(async () => {
+        const dispatcher = new ReducerEventDispatcher(AuthorReducer, [AuthorRegistered]);
+        const instance = new AuthorReducer() as unknown as Record<string, Function>;
+        const context: EventContext = {
+            sequenceNumber: 1n, eventSourceId: 'author', eventType: EventType.parse('explicit-reducer-delivery'),
+            occurred: new Date(), correlationId: '', causation: [], tags: []
+        };
+        const handler = dispatcher.handlerFor('explicit-reducer-delivery')!;
+        const initial = await dispatcher.invoke(instance, handler, new AuthorRegistered('Ada'), undefined, context);
+        result = await dispatcher.invoke(instance, handler, new AuthorRegistered('Grace'), initial, context);
+    });
+    it('should pass the event state and context to the named method', () => {
+        (result as object).should.deep.equal({ names: ['Ada', 'Grace'], id: 'author' });
+    });
+});

package/reducers/for_ReducerEventDispatcher/when_invoking_instance_handlers.spec.ts

@@ -0,0 +1,69 @@
+// Copyright (c) Cratis. All rights reserved.
+// Licensed under the MIT license. See LICENSE file in the project root for full license information.
+
+import { beforeEach, chai, describe, it } from 'vitest';
+import { eventType } from '../../events/eventTypeDecorator.js';
+import { EventType } from '../../events/EventType.js';
+import type { EventContext } from '../../events/EventContext.js';
+import { handles } from '../index.js';
+import { ReducerEventDispatcher } from '../ReducerEventDispatcher.js';
+
+chai.should();
+
+@eventType('instance-reducer-event')
+class AuthorRegistered { constructor(readonly name: string) {} }
+
+class BoundConventional {
+    readonly prefix = 'conventional';
+    constructor() { this.authorRegistered = this.authorRegistered.bind(this); }
+    authorRegistered(event: AuthorRegistered, state: string[] | undefined, context: EventContext) {
+        return [...(state ?? []), `${this.prefix}:${event.name}:${context.eventSourceId}`];
+    }
+}
+
+class BoundExplicit {
+    readonly prefix = 'explicit';
+    constructor() { this.update = this.update.bind(this); }
+    @handles(AuthorRegistered)
+    update(event: AuthorRegistered, state: string[] | undefined, context: EventContext) {
+        return [...(state ?? []), `${this.prefix}:${event.name}:${context.eventSourceId}`];
+    }
+}
+
+class Conventional {
+    authorRegistered(event: AuthorRegistered, state: string[] | undefined, context: EventContext) {
+        return [...(state ?? []), `base:${event.name}:${context.eventSourceId}`];
+    }
+}
+
+class FieldConventional extends Conventional {
+    override authorRegistered = (event: AuthorRegistered, state: string[] | undefined, context: EventContext) =>
+        [...(state ?? []), `field:${event.name}:${context.eventSourceId}`];
+}
+
+for (const { type, prefix } of [
+    { type: BoundConventional, prefix: 'conventional' },
+    { type: BoundExplicit, prefix: 'explicit' },
+    { type: FieldConventional, prefix: 'field' }
+]) {
+    describe(`when invoking the ${prefix} reducer instance handler`, () => {
+        let result: unknown;
+        beforeEach(async () => {
+            const dispatcher = new ReducerEventDispatcher(type, [AuthorRegistered]);
+            const instance = new type();
+            dispatcher.validateInstance(instance);
+            const context: EventContext = {
+                sequenceNumber: 1n, eventSourceId: 'author', eventType: EventType.parse('instance-reducer-event'),
+                occurred: new Date(), correlationId: '', causation: [], tags: []
+            };
+            const handler = dispatcher.handlerFor('instance-reducer-event')!;
+            const initial = await dispatcher.invoke(instance as unknown as Record<string, Function>, handler,
+                new AuthorRegistered('Ada'), undefined, context);
+            result = await dispatcher.invoke(instance as unknown as Record<string, Function>, handler,
+                new AuthorRegistered('Grace'), initial, context);
+        });
+        it('should invoke the instance handler with the event state and context', () => {
+            (result as string[]).should.deep.equal([`${prefix}:Ada:author`, `${prefix}:Grace:author`]);
+        });
+    });
+}

package/reducers/for_Reducers/when_activating_a_field_override.spec.ts

@@ -0,0 +1,63 @@
+// Copyright (c) Cratis. All rights reserved.
+// Licensed under the MIT license. See LICENSE file in the project root for full license information.
+
+import { EventContext, EventType, ObservationState, ReplayState, type ReducerMessage } from '@cratis/chronicle.contracts';
+import { beforeEach, chai, describe, it } from 'vitest';
+import type { IClientArtifactsProvider } from '../../artifacts/index.js';
+import type { ChronicleConnection } from '../../connection/index.js';
+import { ConnectionLifecycle } from '../../connection/ConnectionLifecycle.js';
+import type { IEventStore } from '../../IEventStore.js';
+import { eventType } from '../../events/eventTypeDecorator.js';
+import { handles, reducer, Reducers } from '../index.js';
+
+chai.should();
+
+@eventType('field-reducer-event')
+class AuthorRegistered {}
+const calls: string[] = [];
+class Base {
+    @handles(AuthorRegistered) update() { calls.push('base'); return {}; }
+    beginReplay() { calls.push('replay'); }
+    endReplay() {}
+}
+@reducer('field-reducer')
+class Derived extends Base {
+    override update = () => { calls.push('field'); return {}; };
+}
+
+for (const replayState of [ReplayState.REPLAY_STATE_None, ReplayState.BeginReplay]) {
+    describe(`when activating a reducer with a field override for replay state ${replayState}`, () => {
+        let result: ReducerMessage;
+        let disposed: boolean;
+        beforeEach(async () => {
+            calls.length = 0;
+            disposed = false;
+            const lifecycle = new ConnectionLifecycle();
+            let finish!: () => void;
+            const finished = new Promise<void>(resolve => { finish = resolve; });
+            async function* stream(queue: AsyncIterable<ReducerMessage>) {
+                const messages = queue[Symbol.asyncIterator]();
+                await messages.next();
+                yield { Events: [{ Content: '{}', Context: EventContext.fromPartial({
+                    EventType: EventType.fromPartial({ Id: 'field-reducer-event', Generation: 1 }), SequenceNumber: 1n
+                }) }], Partition: 'author', ReplayState: replayState, InitialState: '' };
+                result = (await messages.next()).value!;
+                await lifecycle.disconnected(error => { throw error; });
+                finish();
+            }
+            const reducers = new Reducers({ reducers: [Derived], eventTypes: [AuthorRegistered], readModels: [] } as unknown as IClientArtifactsProvider,
+                { reducers: { observe: stream }, readModels: { registerMany: async () => ({}) } } as unknown as ChronicleConnection,
+                'store', 'tenant', lifecycle, 'sink', { readModels: {} } as IEventStore,
+                type => ({ instance: new type(), dispose: () => { disposed = true; } }));
+            try { await reducers.register(); await finished; } finally { reducers.dispose(); }
+        });
+        it('should fail delivery without invoking the field or replay notification', () => {
+            calls.should.have.lengthOf(0);
+            result.Content!.Value1!.State.should.equal(ObservationState.Failed);
+            result.Content!.Value1!.ReadModelState.should.equal('');
+            result.Content!.Value1!.ExceptionMessages.join(' ').should.contain(
+                "Override 'update' on 'Derived' hides @handles(AuthorRegistered) declared on 'Base'; use a method with @handles instead of an instance field.");
+        });
+        it('should release the rejected activation lease', () => { disposed.should.be.true; });
+    });
+}

package/reducers/for_Reducers/when_observing_explicit_handlers.legacy.spec.ts

@@ -0,0 +1,73 @@
+// Copyright (c) Cratis. All rights reserved.
+// Licensed under the MIT license. See LICENSE file in the project root for full license information.
+
+import { EventContext, EventType, ObservationState, type ReducerMessage } from '@cratis/chronicle.contracts';
+import { beforeEach, chai, describe, it } from 'vitest';
+import type { IClientArtifactsProvider } from '../../artifacts/index.js';
+import type { ChronicleConnection } from '../../connection/index.js';
+import { ConnectionLifecycle } from '../../connection/ConnectionLifecycle.js';
+import { eventType } from '../../events/eventTypeDecorator.js';
+import type { EventContext as ClientEventContext } from '../../events/EventContext.js';
+import { handles, reducer, Reducers } from '../index.js';
+
+chai.should();
+
+@eventType('legacy-reduced-handler')
+class AuthorRegistered { constructor(readonly name = '') {} }
+
+@reducer('legacy-reduction')
+class AuthorReducer {
+    @handles(AuthorRegistered)
+    update(event: AuthorRegistered, state: { names: string[] } | undefined, context: ClientEventContext) {
+        return { names: [...(state?.names ?? []), event.name], id: context.eventSourceId };
+    }
+}
+
+describe('when observing explicit legacy reducer handlers', () => {
+    let registration: ReducerMessage;
+    let result: ReducerMessage;
+    beforeEach(async () => {
+        const lifecycle = new ConnectionLifecycle();
+        let finish!: () => void;
+        const finished = new Promise<void>(resolve => { finish = resolve; });
+        async function* stream(queue: AsyncIterable<ReducerMessage>) {
+            const messages = queue[Symbol.asyncIterator]();
+            registration = (await messages.next()).value!;
+            yield {
+                Events: ['Ada', 'Grace'].map((name, index) => ({
+                    Context: EventContext.fromPartial({
+                        EventType: EventType.fromPartial({ Id: 'legacy-reduced-handler', Generation: 1 }),
+                        EventSourceId: 'author', SequenceNumber: BigInt(index + 1)
+                    }),
+                    Content: JSON.stringify({ name })
+                })),
+                Partition: 'author', ReplayState: 0, InitialState: ''
+            };
+            result = (await messages.next()).value!;
+            await lifecycle.disconnected(error => { throw error; });
+            finish();
+        }
+        const connection = {
+            reducers: { observe: stream }, readModels: { registerMany: async () => ({}) }
+        } as unknown as ChronicleConnection;
+        const artifacts = {
+            reducers: [AuthorReducer], eventTypes: [AuthorRegistered], readModels: []
+        } as unknown as IClientArtifactsProvider;
+        const reducers = new Reducers(artifacts, connection, 'store', 'tenant', lifecycle, 'sink');
+        try {
+            await reducers.register();
+            await finished;
+        } finally { reducers.dispose(); }
+    });
+    it('should subscribe to the explicitly named event type', () => {
+        registration.Content!.Value0!.Reducer!.EventTypes.map(entry => entry.EventType!.Id)
+            .should.deep.equal(['legacy-reduced-handler']);
+    });
+    it('should pass the event state and context through the named method', () => {
+        JSON.parse(result.Content!.Value1!.ReadModelState).should.deep.equal({ names: ['Ada', 'Grace'], id: 'author' });
+    });
+    it('should acknowledge both deliveries successfully', () => {
+        result.Content!.Value1!.State.should.equal(ObservationState.Success);
+        result.Content!.Value1!.LastSuccessfulObservation.should.equal(2n);
+    });
+});

package/reducers/for_Reducers/when_registering_explicit_handlers.spec.ts

@@ -0,0 +1,57 @@
+// Copyright (c) Cratis. All rights reserved.
+// Licensed under the MIT license. See LICENSE file in the project root for full license information.
+
+import { beforeEach, chai, describe, it, vi } from 'vitest';
+import type { IClientArtifactsProvider } from '../../artifacts/index.js';
+import type { ChronicleConnection } from '../../connection/index.js';
+import { ConnectionLifecycle } from '../../connection/ConnectionLifecycle.js';
+import { eventType } from '../../events/eventTypeDecorator.js';
+import { handles, reducer, Reducers } from '../index.js';
+
+chai.should();
+
+@eventType('reducer-handled-event')
+class AuthorRegistered {}
+@eventType('unselected-reducer-event')
+class Unregistered {}
+
+@reducer('valid-handlers')
+class Valid { @handles(AuthorRegistered) update() {} }
+@reducer('missing-handler-event')
+class Missing { @handles(Unregistered) update() {} }
+@reducer('duplicate-handlers')
+class Duplicate {
+    @handles(AuthorRegistered) update() {}
+    authorRegistered() {}
+}
+
+for (const [type, message] of [
+    [Missing, /update.*Missing.*Unregistered.*no registered event type/],
+    [Duplicate, /multiple handlers.*reducer-handled-event.*update.*authorRegistered/]
+] as const) {
+    describe(`when registering a reducer with invalid explicit handlers on ${type.name}`, () => {
+        const observe = vi.fn();
+        const registerMany = vi.fn();
+        let error: unknown;
+        beforeEach(async () => {
+            observe.mockReset();
+            registerMany.mockReset();
+            error = undefined;
+            const artifacts = { reducers: [Valid, type], eventTypes: [AuthorRegistered], readModels: [] } as unknown as IClientArtifactsProvider;
+            const connection = { reducers: { observe }, readModels: { registerMany } } as unknown as ChronicleConnection;
+            const reducers = new Reducers(artifacts, connection, 'store', 'tenant', new ConnectionLifecycle(), 'sink');
+            try { await reducers.register(); } catch (caught) { error = caught; }
+            finally { reducers.dispose(); }
+        });
+        it('should reject registration with an actionable error', () => {
+            (error instanceof Error).should.be.true;
+            (error as Error).message.should.match(message);
+        });
+        it('should not register read models before validating all handlers', () => {
+            registerMany.mock.calls.should.have.lengthOf(0);
+        });
+        it('should not start an observation before validating all reducers', () => {
+            observe.mock.calls.should.have.lengthOf(0);
+        });
+    });
+}

package/reducers/index.ts

@@ -1,6 +1,7 @@
 // Copyright (c) Cratis. All rights reserved.
 // Licensed under the MIT license. See LICENSE file in the project root for full license information.
 
+export { handles } from '../events/handles.js';
 export { ReducerId } from './ReducerId.js';
 export { reducer, getReducerMetadata, isReducer } from './reducer.js';
 export type { ReducerMetadata } from './reducer.js';

package/testing/ReactorScenario.ts

@@ -21,6 +21,7 @@ import { EventStoreNamespaceName } from '../EventStoreNamespaceName.js';
 import type { IEventStore } from '../IEventStore.js';
 import type { IReadModels } from '../readModels/IReadModels.js';
 import { getReactorEventTypes, invokeReactorHandler, selectReactorHandler } from '../reactors/ReactorDispatcher.js';
+import { createHandlerInstanceValidator } from '../observation/createHandlerInstanceValidator.js';
 import { getReactorMetadata } from '../reactors/reactor.js';
 import { normalizeReactorSideEffects } from '../reactors/ReactorSideEffects.js';
 import type { ReactorServices } from '../reactors/ReactorServices.js';
@@ -40,6 +41,7 @@ export class ReactorScenario {
     private readonly _entries: ReturnType<typeof getReactorEventTypes>;
     private readonly _eventTypes: readonly Constructor[];
     private readonly _instance?: Record<string, Function>;
+    private readonly _validateInstance: (instance: object) => void;
     private readonly _store: IEventStore;
     private readonly _controller = new AbortController();
     private readonly _effects: RecordedReactorSideEffect[] = [];
@@ -76,9 +78,12 @@ export class ReactorScenario {
             eventTypeMigrations: eventTypeMigrations === undefined ? undefined : [...eventTypeMigrations]
         } });
         this._entries = getReactorEventTypes(_reactor, this._eventTypes);
+        this._validateInstance = createHandlerInstanceValidator(_reactor,
+            this._entries.flatMap(entry => [entry.methodName, entry.replayMethodName].filter((name): name is string => !!name)));
         // Every unsupported event-sequence operation is latched at its source, whichever result or stream carries it.
         observeUnsupportedOperations(this._events.eventSequence, error => { this.latch(error); });
         this._instance = _options.artifactActivator ? undefined : new (_reactor as new () => Record<string, Function>)();
+        if (this._instance) this._validateInstance(this._instance);
         this._store = _options.servicesEventStore ?? this.scenarioStore();
         this.given = { forEventSource: id => ({ events: (...events) => this.deliver(id, events, true) }) };
         this.when = { forEventSource: id => ({ events: (...events) => this.deliver(id, events, false) }) };
@@ -246,7 +251,10 @@ export class ReactorScenario {
                         artifactId: getReactorMetadata(this._reactor)!.id.value, eventStore: this._store,
                         readModels: services.readModels, eventSequenceId: 'event-log', partition: sourceId,
                         signal: services.signal, delivery: ArtifactDelivery.Events, eventContext: first.context },
-                    this._options.artifactActivator, process);
+                    this._options.artifactActivator, artifact => {
+                        this._validateInstance(artifact.instance);
+                        return process(artifact);
+                    });
                 } else {
                     await process({ instance: this._instance ?? {} });
                 }

package/testing/ReducerReadModelProcessor.ts

@@ -18,6 +18,7 @@ export class ReducerReadModelProcessor<TReadModel extends object> implements IRe
     async process(events: readonly ScenarioEvent[]): Promise<Map<string, ReadModelState<TReadModel>>> {
         const results = new Map<string, ReadModelState<TReadModel>>();
         const reducer = new (this._reducerType as new () => Record<string, Function>)();
+        this._dispatcher.validateInstance(reducer);
         for (const event of events) {
             const handler = this._dispatcher.handlerFor(event.context.eventType.id.value);
             if (!handler) continue;

package/testing/for_ReactorScenario/when_delivering_to_an_explicit_handler.spec.ts

@@ -0,0 +1,100 @@
+// Copyright (c) Cratis. All rights reserved.
+// Licensed under the MIT license. See LICENSE file in the project root for full license information.
+
+import { field } from '@cratis/fundamentals';
+import { beforeEach, chai, describe, it } from 'vitest';
+import { eventType, handles, onceOnly, reactor, type EventContext, type ReactorServices } from '../../index.js';
+import { ReactorScenario } from '../index.js';
+
+chai.should();
+
+@eventType('scenario-explicit-author')
+class AuthorRegistered {
+    @field(String) name: string;
+    constructor(name: string) { this.name = name; }
+}
+@eventType('scenario-explicit-notification')
+class AuthorNotified {
+    @field(String) name: string;
+    constructor(name: string) { this.name = name; }
+}
+
+const calls: { name: string; source: string; method: string }[] = [];
+@reactor('explicit-scenario-handler')
+class AuthorReactor {
+    @onceOnly()
+    @handles(AuthorRegistered)
+    notify(event: AuthorRegistered, context: EventContext, services: ReactorServices) {
+        (services.readModels === services.eventStore.readModels).should.be.true;
+        calls.push({ name: event.name, source: context.eventSourceId, method: 'notify' });
+        return new AuthorNotified(event.name);
+    }
+}
+
+const options = { artifacts: { eventTypes: [AuthorRegistered, AuthorNotified] }, constraints: 'disabled' as const };
+
+describe('when delivering a live event to an explicitly named onceOnly handler in ReactorScenario', () => {
+    let scenario: ReactorScenario;
+    beforeEach(async () => {
+        calls.length = 0;
+        scenario = new ReactorScenario(AuthorReactor, options);
+        await scenario.when.forEventSource('author-1').events(new AuthorRegistered('Ada'));
+    });
+    it('should invoke the method with the event and context', () => {
+        calls.should.deep.equal([{ name: 'Ada', source: 'author-1', method: 'notify' }]);
+    });
+    it('should process returned side effects', () => {
+        scenario.shouldHaveProduced(AuthorNotified, event => event.name === 'Ada');
+    });
+    it('should complete delivery', () => {
+        scenario.results[0].completed.should.be.true;
+    });
+});
+
+describe('when ReactorScenario has an unregistered explicit handler', () => {
+    it('should reject it before constructing the reactor', () => {
+        let constructed = false;
+        @reactor('invalid-explicit-scenario')
+        class Invalid {
+            constructor() { constructed = true; }
+            @handles(AuthorRegistered) notify() {}
+        }
+        (() => new ReactorScenario(Invalid, { ...options, artifacts: { eventTypes: [AuthorNotified] } }))
+            .should.throw(/notify.*AuthorRegistered.*no registered event type/);
+        constructed.should.be.false;
+    });
+});
+
+@reactor('field-shadowing-scenario')
+class FieldReactor extends AuthorReactor {
+    override notify = () => new AuthorNotified('hidden');
+}
+
+describe('when an instance field hides an inherited ReactorScenario handler', () => {
+    it('should reject the field after constructing the reactor', () => {
+        (() => new ReactorScenario(FieldReactor, options)).should.throw(
+            "Override 'notify' on 'FieldReactor' hides @handles(AuthorRegistered) declared on 'AuthorReactor'; use a method with @handles instead of an instance field.");
+    });
+    it('should reject an activator-created field before delivery and dispose its lease', async () => {
+        let disposed = false;
+        const scenario = new ReactorScenario(FieldReactor, { ...options, artifactActivator: type => ({
+            instance: new type(), dispose: () => { disposed = true; }
+        }) });
+        const [result] = await Promise.allSettled([scenario.when.forEventSource('author').events(new AuthorRegistered('Ada'))]);
+        result.status.should.equal('rejected');
+        if (result.status === 'rejected') (result.reason as Error).message.should.match(/Override 'notify'.*use a method/);
+        scenario.produced.should.have.lengthOf(0);
+        disposed.should.be.true;
+    });
+});
+
+describe('when ReactorScenario has duplicate explicit and conventional handlers', () => {
+    @reactor('duplicate-explicit-scenario')
+    class Invalid {
+        @handles(AuthorRegistered) notify() {}
+        authorRegistered() {}
+    }
+    it('should reject the ambiguous event delivery', () => {
+        (() => new ReactorScenario(Invalid, options)).should.throw(/multiple handlers.*scenario-explicit-author/);
+    });
+});

package/testing/for_ReducerReadModelProcessor/when_an_instance_field_hides_a_handler.spec.ts

@@ -0,0 +1,29 @@
+// Copyright (c) Cratis. All rights reserved.
+// Licensed under the MIT license. See LICENSE file in the project root for full license information.
+
+import { chai, describe, it } from 'vitest';
+import { eventType } from '../../events/eventTypeDecorator.js';
+import { handles } from '../../events/handles.js';
+import { ReducerReadModelProcessor } from '../ReducerReadModelProcessor.js';
+
+chai.should();
+
+@eventType('scenario-field-reducer')
+class AuthorRegistered {}
+class Model {}
+class Base {
+    @handles(AuthorRegistered) update() { return new Model(); }
+}
+class Derived extends Base {
+    override update = () => new Model();
+}
+
+describe('when a reducer scenario instance field hides an inherited handler', () => {
+    it('should reject the instance before processing events', async () => {
+        const processor = new ReducerReadModelProcessor(Model, Derived, [AuthorRegistered]);
+        const [result] = await Promise.allSettled([processor.process([])]);
+        result.status.should.equal('rejected');
+        if (result.status === 'rejected') (result.reason as Error).message.should.equal(
+            "Override 'update' on 'Derived' hides @handles(AuthorRegistered) declared on 'Base'; use a method with @handles instead of an instance field.");
+    });
+});