fix(openapi-resource-mocks): catch mode no longer disrupts existing mock value on enable

Previously, enabling catch mode immediately called setLoading() on the current ref,
blowing away any resolved value before a new request had even fired. Now setCatchMode
only registers the key — the existing onRequest listener already handles interception
when the next actual request fires. Also removed the stale re-intercept logic in the
reset control handler, and unified applyBehavior into a single onRequest listener so
reload correctly re-applies initialBehavior when catch mode is off.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>

Author: constantant

tools/openapi-resource-mocks/src/lib/mock-resource-bus.ts

@@ -60,13 +60,7 @@ export class MockResourceBus {
 
   setCatchMode(key: string, enabled: boolean): void {
     if (enabled) {
-      if (this.catchModeKeys.has(key)) return; // idempotent — avoids duplicate caught events
-      this.catchModeKeys.add(key);
-      const ref = this.refs.get(key);
-      if (ref) {
-        ref.setLoading();
-        this.dispatchDomEvent(key, { type: 'caught', args: [], requestId: this.nextReqId(), ts: Date.now() });
-      }
+      this.catchModeKeys.add(key); // idempotent — next request will be caught, current value untouched
     } else {
       this.catchModeKeys.delete(key);
     }
@@ -196,17 +190,7 @@ export class MockResourceBus {
         case 'setLoading':       ref.setLoading();                                       break;
         case 'fail':             ref.fail(detail.value);                                 break;
         case 'reload':           window.__openApiMocks__?.[detail.key]?.reload();         break;
-        case 'reset': {
-          ref.reset();
-          // If catch mode is still on, immediately re-intercept the resource.
-          if (this.catchModeKeys.has(detail.key)) {
-            ref.setLoading();
-            this.dispatchDomEvent(detail.key, {
-              type: 'caught', args: [], requestId: this.nextReqId(), ts: Date.now(),
-            });
-          }
-          break;
-        }
+        case 'reset':          ref.reset();                                                break;
         case 'setProgress':      ref.setProgress(detail.progressType!, detail.loaded!, detail.total); break;
         case 'simulateProgress': window.__openApiMocks__?.[detail.key]?.simulateProgress(
                                    detail.progressType!,

tools/openapi-resource-mocks/src/lib/provide-mock-resource.ts

@@ -57,14 +57,18 @@ export function provideMockResource<T>(
         bus.register(effectiveKey, ref, meta);
         const internal = ref as MockResourceRefInternal<T>;
 
+        if (initialBehavior) {
+          // Re-apply on every request (initial, param change, reload) unless catch mode
+          // is intercepting. The bus's onRequest listener (registered in bus.register)
+          // runs first, so isCatchMode reflects whether this request was just caught.
+          ref.onRequest(() => {
+            if (!bus.isCatchMode(effectiveKey)) applyBehavior(ref, initialBehavior);
+          });
+        }
+
         // untracked() prevents thunk signal-reads from leaking into the outer reactive
         // context — component field initializers run during Angular change detection.
-        untracked(() => {
-          internal._notifyRequest(args);
-          // Skip initialBehavior when the bus caught the request: catch mode takes
-          // precedence and the response must come from the DevTools panel.
-          if (initialBehavior && !bus.isCatchMode(effectiveKey)) applyBehavior(ref, initialBehavior);
-        });
+        untracked(() => { internal._notifyRequest(args); });
 
         // When any arg is a reactive thunk, track signal changes so each new set of
         // params fires a new request event (mirroring how httpResource re-fires on
@@ -76,9 +80,8 @@ export function provideMockResource<T>(
             // Resolve thunks inside the effect body so their signals are tracked.
             const resolved = args.map((a) => (typeof a === 'function' ? (a as () => unknown)() : a));
             untracked(() => {
-              if (first) { first = false; return; } // first run already handled above
+              if (first) { first = false; return; } // first run already handled by _notifyRequest above
               internal._notifyRequest(resolved);
-              if (initialBehavior && !bus.isCatchMode(effectiveKey)) applyBehavior(ref, initialBehavior);
             });
           });
         }