@scenarist/core 0.3.0 → 0.3.1

package/dist/domain/log-events.d.ts

@@ -52,6 +52,10 @@ export declare const LogEvents: {
     readonly STATE_CAPTURED: "state_captured";
     /** State injected into response */
     readonly STATE_INJECTED: "state_injected";
+    /** State set via afterResponse.setState */
+    readonly STATE_SET: "state_set";
+    /** State response resolved (condition matched or default) */
+    readonly STATE_RESPONSE_RESOLVED: "state_response_resolved";
     /** Sequence advanced to next response */
     readonly SEQUENCE_ADVANCED: "sequence_advanced";
     /** Sequence exhausted all responses */

package/dist/domain/log-events.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"log-events.d.ts","sourceRoot":"","sources":["../../src/domain/log-events.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAEH;;;;;;;;GAQG;AACH,eAAO,MAAM,aAAa;IACxB,oEAAoE;;IAEpE,yCAAyC;;IAEzC,yCAAyC;;IAEzC,kCAAkC;;IAElC,8BAA8B;;CAEtB,CAAC;AAEX;;GAEG;AACH,eAAO,MAAM,SAAS;IAEpB,uCAAuC;;IAEvC,mDAAmD;;IAEnD,mCAAmC;;IAEnC,kCAAkC;;IAIlC,2CAA2C;;IAE3C,4CAA4C;;IAE5C,sCAAsC;;IAEtC,kCAAkC;;IAIlC,mCAAmC;;IAEnC,mCAAmC;;IAInC,yCAAyC;;IAEzC,uCAAuC;;IAIvC,yCAAyC;;IAEzC,8CAA8C;;IAE9C,qCAAqC;;CAE7B,CAAC;AAEX,MAAM,MAAM,QAAQ,GAAG,CAAC,OAAO,SAAS,CAAC,CAAC,MAAM,OAAO,SAAS,CAAC,CAAC"}
+{"version":3,"file":"log-events.d.ts","sourceRoot":"","sources":["../../src/domain/log-events.ts"],"names":[],"mappings":"AAAA;;;;;;;;GAQG;AAEH;;;;;;;;GAQG;AACH,eAAO,MAAM,aAAa;IACxB,oEAAoE;;IAEpE,yCAAyC;;IAEzC,yCAAyC;;IAEzC,kCAAkC;;IAElC,8BAA8B;;CAEtB,CAAC;AAEX;;GAEG;AACH,eAAO,MAAM,SAAS;IAEpB,uCAAuC;;IAEvC,mDAAmD;;IAEnD,mCAAmC;;IAEnC,kCAAkC;;IAIlC,2CAA2C;;IAE3C,4CAA4C;;IAE5C,sCAAsC;;IAEtC,kCAAkC;;IAIlC,mCAAmC;;IAEnC,mCAAmC;;IAEnC,2CAA2C;;IAE3C,6DAA6D;;IAI7D,yCAAyC;;IAEzC,uCAAuC;;IAIvC,yCAAyC;;IAEzC,8CAA8C;;IAE9C,qCAAqC;;CAE7B,CAAC;AAEX,MAAM,MAAM,QAAQ,GAAG,CAAC,OAAO,SAAS,CAAC,CAAC,MAAM,OAAO,SAAS,CAAC,CAAC"}

package/dist/domain/log-events.js

@@ -55,6 +55,10 @@ export const LogEvents = {
     STATE_CAPTURED: "state_captured",
     /** State injected into response */
     STATE_INJECTED: "state_injected",
+    /** State set via afterResponse.setState */
+    STATE_SET: "state_set",
+    /** State response resolved (condition matched or default) */
+    STATE_RESPONSE_RESOLVED: "state_response_resolved",
     // Sequence progression
     /** Sequence advanced to next response */
     SEQUENCE_ADVANCED: "sequence_advanced",

package/dist/domain/response-selector.d.ts.map

@@ -1 +1 @@
-{"version":3,"file":"response-selector.d.ts","sourceRoot":"","sources":["../../src/domain/response-selector.ts"],"names":[],"mappings":"AAOA,OAAO,KAAK,EACV,gBAAgB,EAChB,eAAe,EACf,YAAY,EACZ,MAAM,EACP,MAAM,mBAAmB,CAAC;AAiB3B;;GAEG;AACH,KAAK,6BAA6B,GAAG;IACnC,eAAe,CAAC,EAAE,eAAe,CAAC;IAClC,YAAY,CAAC,EAAE,YAAY,CAAC;IAC5B,MAAM,CAAC,EAAE,MAAM,CAAC;CACjB,CAAC;AAEF;;;;;;;;;;GAUG;AACH,eAAO,MAAM,sBAAsB,GACjC,UAAS,6BAAkC,KAC1C,gBAgQF,CAAC"}
+{"version":3,"file":"response-selector.d.ts","sourceRoot":"","sources":["../../src/domain/response-selector.ts"],"names":[],"mappings":"AAOA,OAAO,KAAK,EACV,gBAAgB,EAChB,eAAe,EACf,YAAY,EACZ,MAAM,EACP,MAAM,mBAAmB,CAAC;AAiB3B;;GAEG;AACH,KAAK,6BAA6B,GAAG;IACnC,eAAe,CAAC,EAAE,eAAe,CAAC;IAClC,YAAY,CAAC,EAAE,YAAY,CAAC;IAC5B,MAAM,CAAC,EAAE,MAAM,CAAC;CACjB,CAAC;AAEF;;;;;;;;;;GAUG;AACH,eAAO,MAAM,sBAAsB,GACjC,UAAS,6BAAkC,KAC1C,gBAuQF,CAAC"}

package/dist/domain/response-selector.js

@@ -72,11 +72,14 @@ export const createResponseSelector = (options = {}) => {
                     continue;
                 }
                 // No match criteria = fallback mock (always matches)
-                // Log fallback evaluation
+                // Log fallback evaluation with response type info for debugging Issue #328
                 logger.debug(LogCategories.MATCHING, LogEvents.MOCK_MATCH_EVALUATED, logContext, {
                     mockIndex,
                     matched: true,
                     hasCriteria: false,
+                    hasSequence: !!mock.sequence,
+                    hasStateResponse: !!mock.stateResponse,
+                    hasResponse: !!mock.response,
                 });
                 // Dynamic response types (sequence, stateResponse) get higher priority than simple responses
                 // This ensures they are selected over simple fallback responses
@@ -105,7 +108,7 @@ export const createResponseSelector = (options = {}) => {
                     specificity,
                 });
                 // Select response (single, sequence, or stateResponse)
-                const response = selectResponseFromMock(testId, scenarioId, mockIndex, mock, sequenceTracker, stateManager);
+                const response = selectResponseFromMock(testId, scenarioId, mockIndex, mock, sequenceTracker, stateManager, logger);
                 if (!response) {
                     return {
                         success: false,
@@ -141,6 +144,9 @@ export const createResponseSelector = (options = {}) => {
                 // Apply afterResponse.setState to mutate state for subsequent requests
                 if (mock.afterResponse?.setState && stateManager) {
                     stateManager.merge(testId, mock.afterResponse.setState);
+                    logger.debug(LogCategories.STATE, LogEvents.STATE_SET, logContext, {
+                        setState: mock.afterResponse.setState,
+                    });
                 }
                 return { success: true, data: finalResponse };
             }
@@ -199,9 +205,11 @@ export const createResponseSelector = (options = {}) => {
  * @param mock - The mock definition
  * @param sequenceTracker - Optional sequence tracker for Phase 2
  * @param stateManager - Optional state manager for stateResponse
+ * @param logger - Logger for debugging
  * @returns ScenaristResponse or null if mock has no response type
  */
-const selectResponseFromMock = (testId, scenarioId, mockIndex, mock, sequenceTracker, stateManager) => {
+const selectResponseFromMock = (testId, scenarioId, mockIndex, mock, sequenceTracker, stateManager, logger) => {
+    const logContext = { testId, scenarioId };
     // Phase 2: If mock has a sequence, use sequence tracker
     if (mock.sequence) {
         if (!sequenceTracker) {
@@ -222,7 +230,7 @@ const selectResponseFromMock = (testId, scenarioId, mockIndex, mock, sequenceTra
     }
     // State-aware response: evaluate conditions against current state
     if (mock.stateResponse) {
-        return resolveStateResponse(testId, mock.stateResponse, stateManager);
+        return resolveStateResponse(testId, mock.stateResponse, stateManager, logger, logContext);
     }
     // Phase 1: Single response
     if (mock.response) {
@@ -240,18 +248,36 @@ const selectResponseFromMock = (testId, scenarioId, mockIndex, mock, sequenceTra
  * @param testId - Test ID for state isolation
  * @param stateResponse - The stateResponse configuration
  * @param stateManager - Optional state manager for state lookup
+ * @param logger - Logger for debugging
+ * @param logContext - Context for log messages
  * @returns The resolved response (matching condition or default)
  */
-const resolveStateResponse = (testId, stateResponse, stateManager) => {
+const resolveStateResponse = (testId, stateResponse, stateManager, logger, logContext) => {
     // Without stateManager, always return default
     if (!stateManager) {
+        logger.debug(LogCategories.STATE, LogEvents.STATE_RESPONSE_RESOLVED, logContext, {
+            result: "default",
+            reason: "no_state_manager",
+        });
         return stateResponse.default;
     }
     // Get current state for this test
     const currentState = stateManager.getAll(testId);
     // Create resolver and evaluate conditions
     const resolver = createStateResponseResolver();
-    return resolver.resolveResponse(stateResponse, currentState);
+    const response = resolver.resolveResponse(stateResponse, currentState);
+    // Log which response was selected and why
+    const isDefault = response === stateResponse.default;
+    const matchedCondition = isDefault
+        ? null
+        : stateResponse.conditions.find((c) => resolver.resolveResponse({ default: stateResponse.default, conditions: [c] }, currentState) !== stateResponse.default);
+    logger.debug(LogCategories.STATE, LogEvents.STATE_RESPONSE_RESOLVED, logContext, {
+        result: isDefault ? "default" : "condition",
+        currentState,
+        conditionsCount: stateResponse.conditions.length,
+        matchedWhen: matchedCondition?.when ?? null,
+    });
+    return response;
 };
 /**
  * Calculate specificity score for match criteria.

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@scenarist/core",
-  "version": "0.3.0",
+  "version": "0.3.1",
   "description": "Internal: Hexagonal architecture core for scenario-based testing with MSW",
   "author": "Paul Hammond (citypaul) <paul@packsoftware.co.uk>",
   "license": "MIT",